No tag fetching — CI checkouts didn't fetch tags, causing duplicate release commits
Weak idempotency — release lock checked tag existence but not tag-commit alignment
No pre-flight check — script didn't verify existing tags were consistent before releasing
No verification tool — no way to audit tag/version/changelog alignment
Closes DEVX-10
## Summary
- Add `fetch_tags()` to sync remote tags before release (prevents duplicate release commits in CI)
- Add `verify_tag_consistency()` pre-flight check that aborts if tags are misaligned
- Add `--verify` flag to check tag/version/changelog/release alignment without making changes
- Strengthen `create_and_push_tag()` to verify tag points to HEAD (raises on misalignment)
- Fix `has_unreleased_changes()` to exclude release commits (prevents duplicate commits)
- Fix CHANGELOG.md: remove duplicate `[0.1.0]` section, add missing blank lines
## Root causes fixed
1. **No tag fetching** — CI checkouts didn't fetch tags, causing duplicate release commits
2. **Weak idempotency** — release lock checked tag existence but not tag-commit alignment
3. **No pre-flight check** — script didn't verify existing tags were consistent before releasing
4. **No verification tool** — no way to audit tag/version/changelog alignment
Closes DEVX-10
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: OK
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: OK
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
## Automated PR Review
- Architecture compliance: OK
- Best practices: OK
- Security: OK
- i18n: OK
- Resource management: OK
- Documentation: OK
- Tests: OK
- Commit conventions: OK
No issues found by automated checks.
---
**Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emil
merged commit 465f80fbbb into master2026-06-23 01:28:28 +00:00
emil
deleted branch devx-10-fix-release-script2026-06-23 01:28:28 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
emil
Summary
fetch_tags()to sync remote tags before release (prevents duplicate release commits in CI)verify_tag_consistency()pre-flight check that aborts if tags are misaligned--verifyflag to check tag/version/changelog/release alignment without making changescreate_and_push_tag()to verify tag points to HEAD (raises on misalignment)has_unreleased_changes()to exclude release commits (prevents duplicate commits)[0.1.0]section, add missing blank linesRoot causes fixed
Closes DEVX-10
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.
Automated PR Review
No issues found by automated checks.
Auto-merge: If all CI checks pass, this PR will be merged automatically.