Skip to content

sync: back-sync the language bindings and their shared contract#158

Merged
suchintan merged 1 commit into
mainfrom
suchintan/backsync-bindings
Jul 24, 2026
Merged

sync: back-sync the language bindings and their shared contract#158
suchintan merged 1 commit into
mainfrom
suchintan/backsync-bindings

Conversation

@suchintan

Copy link
Copy Markdown
Member

Summary

Completes the back-sync map: bindings/, go/, java/, csharp/, ruby/, and php/ now sync from this repo to the development repo, so external contributions to the language bindings flow back like the rest of the shared tree (the same gap #154 closed for cli/ and mcp-rs/). All six directories exist on main — the precondition for deleteOrphaned mappings.

Sync-map edit applied with explicit maintainer approval.

Smoke test

PyYAML parse: 36 entries, all six new sources present exactly once, no duplicates across the map.

🤖 Generated with Claude Code

The back-sync map covered the core tree but not bindings/, go/, java/,
csharp/, ruby/, or php/, so an external contribution to any language
binding silently never reached the development repo. All six directories
exist on this repository's main, which the deleteOrphaned mappings
require before they may be added.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@claude

claude Bot commented Jul 24, 2026

Copy link
Copy Markdown

Summary

Straightforward, low-risk CI/infra change — .github/sync.yml only, no engine/binding code touched. Verified the stated precondition and found no issues.

🔴 Critical Issues (0)

None found.

🟡 Suggestions (0)

None.

📝 Minor / Style (2)
  • Verified locally: bindings/, go/, java/, csharp/, ruby/, php/ all exist on main and are non-empty, so the deleteOrphaned precondition called out in the new comment actually holds. Also confirmed bindings/ (contract docs + JSON schemas under bindings/cases/) doesn't path-overlap with the per-language dirs, so there's no double-mapping/conflict in the sync action.
  • mcp/ (the Python MCP server) is still unmapped while its Rust sibling mcp-rs/ was added in mcp: back-sync the native server and CLI; run mcp-rs tests in CI; deprecate the Python server #154 — not a blocker here since it's a different kind of package, not a language binding, but worth a follow-up sync entry if external contributions to mcp/ should also flow to rustwright-cloud.

@suchintan
suchintan merged commit 506506b into main Jul 24, 2026
13 checks passed
@suchintan
suchintan deleted the suchintan/backsync-bindings branch July 24, 2026 12:55
@github-actions

Copy link
Copy Markdown

Synced to rustwright-cloud: https://github.com/Skyvern-AI/rustwright-cloud/pull/137

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant