docs(now): README matches the gates that read docs/now entries - #5962
Merged
Merged
Conversation
Name the shape gate (check-now-freshness.yml job `check`, tools/check_now_entry_shape.py) and point at its docstring; note its stricter slug pattern; say the local pre-commit readers only check directory freshness; record the Archive-Repair rule for docs/NOW.md; say `tri now add` needs the tri binary and name `tri now check` and `tri hooks pre-push`. Docs only; no gate changes. Closes #5961 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Contributor
|
📓 NotebookLM Notebook linked to this PR
This notebook contains session context, decisions, and artifacts for this work. |
gHashTag
added a commit
that referenced
this pull request
Oct 4, 2026
…on, drop their 'required' wording (Refs #5935) Brings remove-now-gate (0d5424b) up to date with master (4499e27). Three files conflicted. - CONTRIBUTING.md: master's #5952 section is kept for what an entry is -- one new docs/now/<YYYY-MM-DD>-<slug>.md, the shape pointed at the docstring of tools/check_now_entry_shape.py, `tri now add` needing `cargo build --release -p tri`, hand-written entries, the two local commands. Its "every PR adds one entry", the two CI workflow bullets (now-sync-gate.yml / check-now-freshness.yml) and the pre-commit freshness/exit-2 paragraph are replaced with this branch's truth: the NOW gate was removed by owner decision 2026-10-04 (#5935), an entry is optional, and tri now check, tri hooks pre-push, tri hooks now-gate and t27c check-now remain as tools anyone may run. Heading is now "NOW entries (optional, one file each)" and item 4's anchor follows it. - docs/now/README.md: #5962's `cargo build --release -p tri` note and the shape-checker paragraph (docstring pointer, lowercase-slug caveat) are kept; "a second required gate", "scripts/verify.sh previews both questions" (that preview was removed in this branch) and the .githooks/pre-commit / scripts/pre-commit freshness claims are dropped. The History paragraph no longer implies CI refuses a docs/NOW.md edit: now-sync-gate-diff.sh still does when someone runs it, and no CI job has run it since #5935. Also corrected the branch's own line that said `tri hooks now-gate` runs the script (it reads the directory; only `tri hooks pre-push` runs the script), and "It cannot satisfy the gate" under "Files that are not entries". - tools/census/*.txt: not hand-merged; re-blessed with `tri census pin --bless`, then `tri census pin --gate` printed PASS. Census, merged tree against each parent: - shell: workflow files read 63 (master 64, branch 63); jobs 85 (master 86, branch 84); run: steps 278 (master 285, branch 276); runner-named 257 (master 264, branch 255). Master added one job (loop-tools-gate.yml `harness`, one run: step) and one run: step in cli-tri.yml; this branch removed now-sync-gate.yml (one job, four run: steps) and three run: steps of the old `check` job. 86 - 1 = 85, 285 - 7 = 278, 264 - 7 = 257. - quiet: equal to the branch (workflow files read 63, "named a path but not quiet" 159); master's 64 / 162 move by the same deleted file. - fetches: equal to master's ledger (files read 46, lines naming either spelling 74, fetch sites 33, --paginate 11, asks whether the page filled 11); the merge moves nothing there, the branch's older 69 / 28 / 7 / 10 came from before master's cli/tri changes. Checked: scripts/ci/check_pr_branch_filters.py CLEAN (63 workflow files); scripts/ci/test_now_gate_writes_nothing.py ok. Nothing compiled locally. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
gHashTag
added a commit
that referenced
this pull request
Oct 4, 2026
…efs #5776) PR #5951 (Closes #5935) merged as 57c2cfa and PR #5958 (Closes #5957) as 6765d63, both on the owner's word. Master conflicts with #5952/#5962 were resolved before the merge; required checks and cli-tri green on the merged heads. The live hooks on this machine come from the main checkout's .githooks and stay old until that checkout takes master. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Summary
docs/now/README.mdis the single home for thedocs/now/entry layout (CONTRIBUTING.md points at it after #5952). Four statements in it no longer matched the code; this fixes them minimally and points attools/check_now_entry_shape.py's docstring for the entry shape instead of copying it.Evidence (origin/master 93868bf)
.github/workflows/check-now-freshness.yml:36(jobcheck) runstools/check_now_entry_shape.py(shape defined in its docstring, lines 1-45). Its filename regextools/check_now_entry_shape.py:56(lowercase[a-z0-9-]slug) is stricter thanscripts/ci/now-sync-gate-diff.sh:56([A-Za-z0-9._-]+). README now names the gate and the difference..githooks/pre-commit,scripts/pre-commit,t27c check-now,tri hooks now-gateenforce "the same three conditions". They read the wholedocs/now/directory (bootstrap/src/suite.rs:3158-3175,cli/tri/src/hooks.rs:298,scripts/pre-commit:14-50);cli/tri/src/nownote.rs:265-273,294-298says so. The commit hook on this very branch passed on an unrelated 2026-10-03 entry before this branch's entry was staged. README now calls them weaker and namesscripts/verify.sh,tri now check,tri hooks pre-pushas the range/shape previews.docs/NOW.md"is no longer read by any gate";scripts/ci/now-sync-gate-diff.sh:126-179refuses an edit without anArchive-Repair:trailer. README now says so.tri now addneedscargo build --release -p triand that a hand-written entry works.Docs only; no gate changes.
Closes #5961
🤖 Generated with Claude Code