Skip to content

Require SDK release tags to reference protected main commits - #53

Merged
amanda-vapi merged 1 commit into
mainfrom
codex/approved-main-releases
Nov 3, 2026
Merged

amanda-vapi merged 1 commit into
mainfrom
codex/approved-main-releases

Conversation

@amanda-vapi

@amanda-vapi amanda-vapi commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

A version-matching tag could previously publish SDK code that had never reached protected main. Before package installation or building, fetch main, resolve the event commit (including annotated tags), verify the checkout matches it, and require that commit to be an ancestor of main. Older approved commits remain eligible for backports.

Preserve the new guard through Fern regeneration and test it with actual Git histories covering approved ancestors, annotated tags, unmerged commits, incorrect checkouts, missing refs, and branch refs masquerading as tags. Existing tag/version validation and isolated npm publishing remain in place.

Validation: all 25 release-validation tests pass locally. GitHub compile/test checks passed on both push and pull-request runs. No release tag was created and no package was published.

The repository rule was separately updated to dismiss stale approvals after code changes; its one-approval requirement, required compile/test checks, and empty bypass list remain in place.

@amanda-vapi
amanda-vapi merged commit 8fe071c into main Nov 3, 2026
9 checks passed
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