Skip to content

Dev#2

Merged
Wembie merged 2 commits into
mainfrom
dev
May 9, 2026
Merged

Dev#2
Wembie merged 2 commits into
mainfrom
dev

Conversation

@Wembie
Copy link
Copy Markdown
Owner

@Wembie Wembie commented May 9, 2026

No description provided.

@Wembie Wembie requested a review from Copilot May 9, 2026 01:33
@Wembie Wembie self-assigned this May 9, 2026
@Wembie Wembie merged commit 7af77bf into main May 9, 2026
2 of 3 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a small release-tooling suite to keep VERSION and multiple static manifests in sync, and enforces that synchronization in CI/CD. This also bumps the project/manifest versions to 1.0.0.

Changes:

  • Introduce scripts/release/sync_version.py with a new --check mode and a callable sync_static_manifests() API.
  • Add scripts/release/bump_version.py to bump VERSION and sync manifests (optionally committing the changes).
  • Add CI/CD gates for manifest/version sync and add tests covering the bump/sync workflow; update static manifest versions to 1.0.0.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/test_version_scripts.py Adds tests for the new bump/sync scripts using a temp repo root.
scripts/release/sync_version.py Implements manifest synchronization, semver validation, and --check mode for CI/CD enforcement.
scripts/release/bump_version.py Adds a CLI to bump VERSION, sync manifests, and optionally commit the resulting changes.
scripts/release/README.md Documents the release script commands and suggested release flow.
.github/workflows/ci.yml Adds a CI step to verify manifests are synced to VERSION via --check.
.github/workflows/cd.yml Replaces ad-hoc sync validation with sync_version.py --check.
sentinel.skill Updates manifest version to 1.0.0.
gemini-extension.json Updates manifest version to 1.0.0.
.claude-plugin/plugin.json Updates manifest version to 1.0.0.
.agents/plugins/marketplace.json Updates manifest version to 1.0.0.
Comments suppressed due to low confidence (2)

scripts/release/sync_version.py:46

  • read_version() calls sys.exit(...) on missing/invalid VERSION. Since this function is now part of the module’s public API (no leading underscore) and is used by sync_static_manifests(), it will terminate the interpreter if someone imports and calls it programmatically. Prefer raising an exception (e.g., ValueError / FileNotFoundError) and letting main() convert that into an exit code/message.
    scripts/release/sync_version.py:122
  • The new --check/check_only behavior is now used in CI/CD, but there are no tests exercising the check path (e.g., out-of-sync manifests should cause a non-zero exit, and in-sync should exit cleanly without writing). Adding a test around sync_static_manifests(check_only=True) and/or main() would help prevent release/CI regressions.

Comment on lines +67 to +71
if not sync_version.is_valid_semver(target):
raise ValueError(
f"Invalid version {target!r}. Use semver like 1.0.1 or a bump kind."
)
return target
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.

2 participants