Adopt coherence v0.4.1; drop the docs-portability workaround - #8
Merged
Conversation
coherence v0.4.1 fixes the docs-freshness gate upstream: `docs --check` now normalizes absRoot (the absolute checkout path) as well as the timestamp, so the gate compares only the derived graph, not where it was generated. That makes the local normalize shim redundant. - re-pin coherence-harness → github:daniloc/coherence#v0.4.1 - coherence:docs / coherence:docs:check → back to the plain tool commands - delete scripts/coherence-normalize.mjs (the stop-gap) - regenerate the committed docs with the fixed tool (natural output) Verified cross-env: with the fixed tool wired, tampering graph.json/_graph.html's absRoot to a foreign /home/runner/... path still reports "docs current". Full set green locally: conventions / injection-lint / atlas / coherence:docs / coherence: verify / typecheck / 725 tests. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
mnemion | 2432f93 | Jun 22 2026, 06:09 PM |
daniloc
added a commit
that referenced
this pull request
Jun 23, 2026
) Two-layer prevention for the class where a source edit drifts the committed coherence graph and CI fails late on coherence:docs:check: 1. Tool fix — bump coherence v0.5.0 → v0.5.1, whose freshness gate now normalizes out line numbers. A comment/line-shift edit no longer drifts graph.json/_graph.html, so the common false positive is gone and committed graph diffs show only real structural change. (This is the deeper fix for the cross-env gate finickiness #7/#8 worked around.) 2. Shift-left — a pre-push hook (scripts/git-hooks/pre-push, auto-installed via the `prepare` script setting core.hooksPath) runs coherence:docs:check before a push. With v0.5.1 it fires ONLY on genuine structural drift committed without a regenerate — blocking it on your machine instead of in CI, with the exact fix command. Bypass with `git push --no-verify`. Co-authored-by: Claude Opus 4.8 (1M context) <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
1 participant
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.
Follow-up to the CI-green work (#6, #7): the docs-freshness failure was a coherence-tool bug —
docs --checknormalized thegeneratedAttimestamp but notabsRoot(the absolute checkout path baked intograph.json/_graph.html), so the committed artifacts only matched the machine that generated them.Fixed upstream in coherence v0.4.1 (
docs --checknow normalizesabsRoottoo), so the mnemion-side shim is no longer needed:coherence-harness→github:daniloc/coherence#v0.4.1coherence:docs/coherence:docs:check→ back to the plain tool commandsscripts/coherence-normalize.mjs(the stop-gap from CI: make coherence docs portable (green the freshness gate) #7)absRoot)Verified cross-env: with the fixed tool wired, tampering the committed
absRootto a foreign/home/runner/…path still reportsdocs current. Full set green locally: conventions / injection-lint / atlas / coherence:docs / coherence:verify / typecheck / 725 tests.🤖 Generated with Claude Code