Repository navigation
#196 — repin views-appwrite to main's tip; the first pin was an unmerged branch - #198
Merged
Merged
Conversation
…as an unmerged branch S3 (#184/#195) pinned the Appwrite Seam Contract at a commit resolved with `git -C ../views-appwrite rev-parse HEAD`. That checkout was sitting on the unmerged `feat/s1-single-writer-rule` branch, so the pin captured a commit that never reached `main` and has now been withdrawn (views-appwrite #30, #27). It declared registry v1.4.0; main carries the ratified v1.3.0. Verified before repinning, not taken on the issue's word: the withdrawn commit is reachable only from that feature branch, 47172af is origin/main's tip, its registry declares version = "1.3.0" (RATIFIED — þing-02), and both cited files exist at it. Four references repointed — appwrite_env.py's docstring and ADR-013 §7d's two links — plus the version labels, the §7d correction paragraph, and Erratum E2, all of which asserted a 1.3.0 -> 1.4.0 move that never landed. The withdrawn sha is deliberately not repeated anywhere, including in the prose narrating its withdrawal, so that grepping for it finds nothing. #196's acceptance criteria are met literally. The lesson is narrow and now recorded in both places a future contributor will look: resolve a cross-repo pin from the tip of the other repo's main, never from whatever its working copy has checked out. A pin is a claim about what the contract says; HEAD answers a different question — what someone was working on. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This was referenced Aug 2, 2026
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.
Closes #196. Corrects S3 (#184/#195), merged three hours earlier.
What went wrong, and it was mine
S3 pinned the Appwrite Seam Contract with:
That checkout was sitting on the unmerged
feat/s1-single-writer-rulebranch. So the pin captured a commit that never reachedmainand has since been withdrawn unmerged (views-appwrite #30, #27). It declared registry v1.4.0;maincarries the ratified v1.3.0.HEADof a sibling working copy is not the tip of itsmain. It is whatever someone happened to have checked out — which, in a repo being actively worked on, is more likely than not to be a feature branch.Verified, not taken on trust
The issue arrived labelled URGENT and asked for a specific substitution. Every claim was checked against the views-appwrite repo before acting:
maingit branch -a --contains→ onlyfeat/s1-single-writer-rule;merge-base --is-ancestor origin/main→ false47172afismain's tipgit rev-parse --short origin/main→47172afversion = "1.3.0" # RATIFIED — þing-02version = "1.4.0" # AMENDED, NOT re-ratifiedappwrite_seam_contract.mdandcoordinate_registry.tomlboth presentWhat changed
Four pin references —
appwrite_env.py's docstring and ADR-013 §7d's two links — plus three prose blocks that asserted a 1.3.0 → 1.4.0 move that never landed: the §7d version label, the §7d correction paragraph, and Erratum E2.v1.4.0mentions elsewhere in the repo are views-datafactory's version, unrelated, and untouched.The withdrawn sha appears nowhere
Including in the prose describing its withdrawal. #196's acceptance criterion is
grep -rn b54928f .returning nothing, and a sha left in narration would still show up there and read as live. The story survives without it:The lesson, recorded in both places a contributor will look
Resolve a cross-repo pin from the tip of the other repo's
main, never from its working copy'sHEAD. A pin is a claim about what the contract says;HEADanswers a different question.Worth adding to S6 (#187)'s scope: its drift test should assert the pinned commit is reachable from the sibling's
main, not merely that it exists. That is the check that would have caught this in the same hour rather than requiring another seat to notice.Noted from #196, not acted on here
The withdrawn clause defined its objection window as ending "on the first consumer pinning v1.4.0" — so this repo pinning promptly and correctly ended a review period before any seat could use it. That is a defect in the withdrawn clause and belongs upstream; recording it here only so the causal chain is legible.
Verification
🤖 Generated with Claude Code