Repository navigation
fix(build): remove dead sync-extension-compiler script and stale log - #189
Merged
Merged
Conversation
`scripts/sync-extension-compiler.mjs` was a hand-copy script that synced compiled `dist/*.js` files and schemas into `extension/compiler/` and `extension/schemas/`. It has been dead code since `build-compiler-bundle.mjs` was introduced: that script bundles the same sources via esbuild, syncs the same schemas, installs runtime deps, and has been the active implementation ever since. No external reference to sync-extension-compiler remained in package.json, CI workflows, or docs. Also removes the stale `backends → extension/backends/` from the build-compiler-bundle.mjs success log. The Python/svgbob backend was removed in a prior pass; the script already cleans that directory up rather than populating it, so the log entry was misleading. No behaviour change. 839 tests pass. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Valerii Korobeinikov <vkgeorgia@icloud.com>
📊 Metrics Regression Report✅ All metrics within tolerance Summary
Detailsai-expense-approval.bpmn.transitrix.yaml
feature-release.bpmn.transitrix.yaml
large-cyclic-workflow.bpmn.transitrix.yaml
order-fulfillment.bpmn.transitrix.yaml
simple-approval.bpmn.transitrix.yaml
simple-linear.bpmn.transitrix.yaml
small-dense-approval.bpmn.transitrix.yaml
xlarge-stress-test.bpmn.transitrix.yaml
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
scripts/sync-extension-compiler.mjs— a hand-copy script that synced compileddist/*.jsand schemas toextension/compiler/andextension/schemas/. It has been dead code sincebuild-compiler-bundle.mjswas introduced: that script bundles the same entries via esbuild, syncs the same schemas, and installs runtime deps. Zero external references remained inpackage.json, CI, or docs.backends → extension/backends/fragment from thebuild-compiler-bundle.mjssuccess log. The Python/svgbob backend was removed in a prior pass; the script now cleans that directory rather than populating it, so the log entry was misleading.No behaviour change. 839 tests pass.
Test plan
npm test— 839 passed (56 test files)npm run compile— type-check cleansync-extension-compilerinpackage.json,.github/,docs/,scripts/🤖 Generated with Claude Code