docs(scripts): the version-gate roster names the two repositories that carry it - #1075
Merged
Conversation
…t carry it The header claimed these files are kept byte-identical in three repositories. Two carry them. agent-eval has never had check-version-bump.mjs, check-api-surface.mjs, lib/api-surface.mjs or api-surface.json on main, and no CI step that runs them. The only copy sits on `feat/export-surface-gate`, written 2026-08-18, unmerged, and 59 commits behind main on 2026-09-01. That branch's own copy of check-version-bump.mjs had already drifted from this one before the published-baseline fix existed, so the three-name roster was never true. The correction names exactly the two repositories that carry the file and says why the third is absent, so nobody restores it on the strength of the sentence alone. The stale count in check-api-surface.mjs goes with it: "three copies of one rule end up with three different bugs" was tied to the same roster, and now reads count-free. Adopting the gate in agent-eval is a decision about that repository's release discipline, not an edit this file is owed. It has an open release train, and a new gate switched on mid-flight would red it. Tracked as tangle-network/agent-eval#729. Documentation only. No behavior changes.
tangletools
approved these changes
Sep 1, 2026
tangletools
left a comment
Contributor
There was a problem hiding this comment.
✅ Auto-approved drewstone PR — a9b952f0
This PR was opened by the trusted drewstone account.
The full PR reviewer audit still runs separately and will publish findings if it detects issues.
This approval is provisional. It rests on the audit running. If the audit cannot run — for example the CLI bridge rejects it — this approval is dismissed rather than left standing, so an unrun check never reads as a passing one.
tangletools · auto-approval · reason: drewstone_author · 2026-09-01T23:57:44Z
drewstone
added a commit
to tangle-network/agent-knowledge
that referenced
this pull request
Sep 2, 2026
…t carry it (#187) The header claimed these files are kept byte-identical in three repositories. Two carry them. agent-eval has never had check-version-bump.mjs, check-api-surface.mjs, lib/api-surface.mjs or api-surface.json on main, and no CI step that runs them. The only copy sits on `feat/export-surface-gate`, written 2026-08-18, unmerged, and 59 commits behind main on 2026-09-01. That branch's own copy of check-version-bump.mjs had already drifted from this one before the published-baseline fix existed, so the three-name roster was never true. The correction names exactly the two repositories that carry the file and says why the third is absent, so nobody restores it on the strength of the sentence alone. The stale count in check-api-surface.mjs goes with it: "three copies of one rule end up with three different bugs" was tied to the same roster, and now reads count-free. Adopting the gate in agent-eval is a decision about that repository's release discipline, not an edit this file is owed. It has an open release train, and a new gate switched on mid-flight would red it. Tracked as tangle-network/agent-eval#729. Byte-identity with agent-runtime is preserved: the same patch lands in tangle-network/agent-runtime#1075. Documentation only. No behavior changes.
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.
The header claimed these files are kept byte-identical in three repositories.
Two carry them.
agent-eval has never had
check-version-bump.mjs,check-api-surface.mjs,lib/api-surface.mjsorapi-surface.jsonon main, and no CI step that runsthem. The only copy sits on
feat/export-surface-gate, written 2026-08-18,unmerged, and 59 commits behind main as of 2026-09-01. That branch's own copy of
check-version-bump.mjshad already drifted from this one before thepublished-baseline fix existed, so the three-name roster was never true — not
since the day it was written.
The correction names exactly the two repositories that carry the file, and says
plainly why the third is absent, so nobody restores it on the strength of the
sentence alone.
The stale count in
check-api-surface.mjsgoes with it: "three copies of onerule end up with three different bugs" was tied to the same roster. It reads
count-free now, which keeps the argument and drops the number.
Adopting the gate in agent-eval is a decision about that repository's release
discipline, not an edit this file is owed. It has an open release train
(0.173.0), and switching a new gate on mid-flight would red it. Tracked as its
own issue: tangle-network/agent-eval#729
Documentation only. No behavior changes.
Byte-identity
Verified by checksum after the edit — the same patch is applied to both
repositories, so
check-version-bump.mjs,check-api-surface.mjs,lib/api-surface.mjsandtests/version-bump-check.test.tsremain identicalacross agent-runtime and agent-knowledge. The companion PR is tangle-network/agent-knowledge (opened next).
Gates
node --checkon both edited scripts — cleanpnpm run lint— cleannpx vitest run tests/version-bump-check.test.ts— 36 passedpnpm run check:version-bump— passes (no consumer surface change)