Skip to content

fix(shard-2139Z): replace broken relative markdown links with backtick paths#3269

Merged
AceHack merged 1 commit into
mainfrom
fix/shard-2139Z-broken-relative-links-otto-cli-2026-05-14
May 14, 2026
Merged

fix(shard-2139Z): replace broken relative markdown links with backtick paths#3269
AceHack merged 1 commit into
mainfrom
fix/shard-2139Z-broken-relative-links-otto-cli-2026-05-14

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 14, 2026

Summary

Post-merge cleanup of #3264. Copilot caught that three markdown links in 2139Z.md used relative paths that resolve to docs/hygiene-history/ticks/2026/05/14/.claude/... — broken in any rendered view.

Changes

Replaced 3 broken relative links with plain backtick paths (matching the established shard-authoring convention):

Line Before After
12 [link](.claude/rules/holding-without-...md) `.claude/rules/holding-without-...md`
31 [link](.claude/rules/never-be-idle.md) `.claude/rules/never-be-idle.md`
40 [link](docs/hygiene-history/ticks/2026/05/14/2001Z.md) `docs/hygiene-history/ticks/2026/05/14/2001Z.md`

Why backticks over root-relative

Copilot suggested either /.claude/rules/... (root-relative) or plain backticks. The repo's existing tick shards (2001Z, 2010Z, 2026Z, etc.) use plain backticks without link — chose that for consistency.

Scan of other shards

grep -nE "\[\?[^]]`?](.claude/|[[^]]](docs/hygiene" docs/hygiene-history/ticks/2026/05/14/2*.md` returns 0 other instances. Bug isolated to 2139Z.

Test plan

  • markdownlint-cli2 clean
  • No relative .claude/... or docs/hygiene-history/... links remain in 2139Z
  • Composite branch-guard + gh pr create --head used
  • CI clears
  • Auto-merge

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

…k paths

Copilot caught post-merge on PR #3264: three markdown links in
2139Z.md used relative paths that resolved to
docs/hygiene-history/ticks/2026/05/14/.claude/... (404 in any rendered
view). The convention in OTHER tick shards is plain backticks without
link, e.g., `.claude/rules/foo.md`.

Changed 3 occurrences:

- Line 12: `holding-without-named-dependency-is-standing-by-failure.md`
- Line 31: `never-be-idle.md`
- Line 40: `docs/hygiene-history/ticks/2026/05/14/2001Z.md`

Plain-backtick form chosen over root-relative (the other Copilot
suggestion) because it matches the established shard-authoring
convention.

Scan of all other today's shards: no other instances of the bug.

Post-merge cleanup: #3264 already merged the shard with the bug.
This PR fixes forward on main.

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 14, 2026 21:48
@AceHack AceHack enabled auto-merge (squash) May 14, 2026 21:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR performs a narrow documentation cleanup in a hygiene-history tick shard by replacing broken relative Markdown links with plain backtick paths.

Changes:

  • Replaced two .claude/rules/... Markdown links with inline path references.
  • Replaced one same-directory hygiene-history link with an inline path reference.
  • Aligns the shard with nearby tick-shard path-reference style.

@AceHack AceHack merged commit 29983a5 into main May 14, 2026
25 checks passed
@AceHack AceHack deleted the fix/shard-2139Z-broken-relative-links-otto-cli-2026-05-14 branch May 14, 2026 21:50
Copy link
Copy Markdown
Member Author

@AceHack AceHack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Antigravity check: Clean fix. Parity proved.

AceHack added a commit that referenced this pull request May 14, 2026
… bug (#3271)

Tick output:
- PR #3269 (fix(shard-2139Z) broken relative links): replaced 3
  occurrences of relative markdown links with plain backtick form
  (matching established shard-authoring convention). Auto-merge armed.
- Post-merge thread on #3264 (Copilot caught the bug) resolved with
  reply pointing to #3269.
- This shard.

Real substantive catch by Copilot: markdown links like
`[text](.claude/rules/foo.md)` from a tick shard at
docs/hygiene-history/ticks/2026/05/14/ resolve to
docs/hygiene-history/ticks/2026/05/14/.claude/... (404 in render).

Convention codified: use plain backticks for path references in tick
shards, NOT markdown links. Two acceptable patterns:
1. `.claude/rules/foo.md` (preferred — repo convention)
2. [link](/path/to/foo.md) (root-relative if linking is needed)

Scan of all OTHER today's shards: 0 instances. Bug isolated to 2139Z.

Filed mentally: audit-shard-broken-links.ts candidate (cousin to
audit-md032-plus-linestart.ts).

Session tally: 27 merged + 2 wait-ci.

Cron sentinel 12fb713e armed; recurring.

Co-authored-by: Claude <noreply@anthropic.com>
AceHack added a commit that referenced this pull request May 14, 2026
…ight) (#3273)

Tick output:
- Resolved 2 unresolved threads on #3267 (shard 2143Z): both Copilot
  AND Codex caught the same temporal-provenance issue (2143Z cited
  2139Z but #3264 was still wait-ci at review-time; tree at #3267's
  base didn't have the cited file). Reply explained that by merge-time
  the cross-reference is navigable; threads resolved.
- This shard codifies the temporal-provenance discipline.

Three patterns for minimal-continuation shards that cite in-flight
prior shards:
1. Stack on prior shard's PR (--base <prior-branch>)
2. Inline the minimal audit-sweep context (self-contained shard)
3. Reply at review-time + resolve (pragmatic for tight cron cadence)

Audit-discipline candidate filed: tool that scans tick shards for
cross-references to other tick-shards whose PRs are still open. Cousin
to audit-shard-broken-links.ts (filed in 2146Z).

Two-reviewer convergence continues to be strong signal:
- This tick: Copilot + Codex on temporal-provenance
- Earlier (#3258): Copilot + Codex on tally arithmetic

Prior-tick PRs merged this batch:
- #3267 (shard 2143Z) MERGED as a093da2
- #3269 (broken-link fix) MERGED as 29983a5

Session tally: 29 merged + 1 wait-ci.

Cron sentinel 12fb713e armed; recurring.

Co-authored-by: Claude <noreply@anthropic.com>
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