Fine-tune 0.3.1: rename schema key to sync_version, clean upgrade prompt - #2
Merged
Conversation
…ure hint to read CHANGELOG breaking notes Permanent frozen key, picked the clean final name now (touches engine/docs/examples/ci). intelligence-update skill: on any failure code, re-read CHANGELOG entries current<release<=engine (esp. ### Breaking) before retry/escalate. No new release/version added. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…o blockquote); removed duplicate prompt in 0.3.1 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…breaking notes, base plan on them, latest scripts, retry) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Renames the frozen schema-version contract key from intelligence_sync_version to sync_version across the codebase (engine, examples, CI, docs) and tidies the CHANGELOG upgrade instruction into a clean fenced copy-paste prompt (de-duplicating the prior inline 0.3.1 note). No behavioral changes; pure token rename plus documentation cleanup, all under the existing ## [0.3.1] entry.
Changes:
- Rename
intelligence_sync_version→sync_versionin engine (lib/migrations.shIS_VERSION_KEY,sync.sherror text), examples, CI assertions, INIT, and docs. - Replace blockquote-style upgrade instruction in
CHANGELOG.mdwith a single fenced code block and remove the duplicate hint inside the 0.3.1 entry. - Update SKILL.md to add an on-failure CHANGELOG re-read hint and propagate the rename in all references.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates schema key name in the update-flow description. |
| intelligence/sync/skills/intelligence-update/SKILL.md | Renames key references; adds on-failure CHANGELOG re-read hint. |
| intelligence/sync/scripts/sync.sh | Updates error message and comment to new key name. |
| intelligence/sync/scripts/lib/migrations.sh | Changes IS_VERSION_KEY value to sync_version. |
| intelligence/sync/INIT.md | Updates bootstrap key emission references. |
| intelligence/sync/docs/CONVENTIONS.md | Renames key throughout the schema-contract section. |
| docs/CONVENTIONS.md | Mirrors the vendored CONVENTIONS rename. |
| CLAUDE.md | Updates architecture overview key reference. |
| CHANGELOG.md | Cleans upgrade prompt into a fenced block; de-duplicates 0.3.1 hint; renames key. |
| examples/{go-api,dotnet-api-with-react-frontend,platform-with-submodules}/config.yaml | Updates example configs to the new key. |
| .github/workflows/ci.yml | Updates 9 CI assertions/configs to the new key. |
Verified zero stale intelligence_sync_version occurrences remain repo-wide.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
Fine-tunes the 0.3.1 modular/update work from #1 (no new release): renames the permanent
config.yamlschema-version contract keyintelligence_sync_version→sync_versioneverywhere (it is a frozen key — picked the clean final name once), and makes theCHANGELOG.mdupgrade instruction a clean copy-paste prompt (fenced block, no blockquote>artifacts; single source — removed the duplicate inside the 0.3.1 entry). Theintelligence-updateskill's on-failure "read the CHANGELOG breaking notes" hint is kept as-is.Type of change
Verification
bash intelligence/sync/scripts/sync.shruns cleanly against an example project (CIsmoke×6 +migrationchain; locally revalidated stamp/read with the newsync_versionkey)lint_frontmatterproduces no new warningsCHANGELOG.md,README.md,docs/CONVENTIONS.md,CLAUDE.md,intelligence/sync/INIT.md, vendored docs)CHANGELOG.mdupdated (under existing## [0.3.1]; no new version by request)Checks: 0 stale
intelligence_sync_versionoccurrences repo-wide;sync_versionconsistent across engine /ci.yml(×9) /examples/*/INIT;bash -nclean on all scripts;ci.ymlvalid YAML; working tree clean.Notes for reviewers
sync_versionis a permanent, format-stable, top-level scalar inconfig.yaml— no migration may ever rename/move it (the rename is done now, before it's relied on at scale). Engine reads/writes it viaread_engine_stamp/stamp_version; CImigrationjob andexamples/*carry it.865f18e(rename + kept on-failure skill hint),f79f9fd(clean copy-paste upgrade prompt, de-duplicated). Squash-merge intended.