Skip to content

shard(tick): 2146Z — post-merge cleanup of 2139Z broken relative-link bug#3271

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

shard(tick): 2146Z — post-merge cleanup of 2139Z broken relative-link bug#3271
AceHack merged 1 commit into
mainfrom
shard/tick-2146Z-broken-relative-links-fix-otto-cli-2026-05-14

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 14, 2026

Summary

Tick 2026-05-14T21:46Z shard. Substantive work in #3269 — post-merge cleanup of 3 broken relative markdown links in 2139Z.md that Copilot caught after #3264 merged.

What landed

Real substantive catch

Markdown links like [text](.claude/rules/foo.md) from a tick shard at docs/hygiene-history/ticks/2026/05/14/ resolve relative to that directory — they'd render as docs/hygiene-history/ticks/2026/05/14/.claude/rules/foo.md (404).

Convention codified

Use plain backticks for path references in tick shards. Two acceptable patterns:

  1. `.claude/rules/foo.md` (preferred — repo convention)
  2. [link](/path/to/foo.md) (root-relative if linking is needed)

Scan result

grep -nE "\[\?[^]]`?](.claude/|[[^]]](docs/hygiene"` across all today's shards: 0 other instances. Bug isolated to 2139Z.

Filed for future tick

audit-shard-broken-links.ts audit-discipline candidate (cousin to audit-md032-plus-linestart.ts).

Prior-tick PRs merged this batch

  • #3263 (shard 2135Z) → 29c195e
  • #3264 (shard 2139Z) → 5a88735
  • #3265 (Lior PR-archive batch) → 1eafd60

Session running tally: 27 merged + 2 wait-ci (#3267 + #3269) + this shard's PR

Test plan

  • markdownlint-cli2 + audit-md032-plus-linestart clean
  • No real broken links in this shard
  • Composite branch-guard + gh pr create --head used
  • CI clears
  • Auto-merge

🤖 Generated with Claude Code

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

… bug

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>
Copilot AI review requested due to automatic review settings May 14, 2026 21:50
@AceHack AceHack enabled auto-merge (squash) May 14, 2026 21:50
@AceHack AceHack merged commit 9691495 into main May 14, 2026
24 checks passed
@AceHack AceHack deleted the shard/tick-2146Z-broken-relative-links-fix-otto-cli-2026-05-14 branch May 14, 2026 21:52
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 adds a tick shard documenting post-merge cleanup for broken relative markdown links in the 2139Z shard and records the follow-up PR/thread resolution context.

Changes:

  • Adds the 2146Z hygiene-history tick shard.
  • Documents the broken relative-link class and preferred shard path-reference convention.
  • Records verification and current PR visibility state.

Comment thread docs/hygiene-history/ticks/2026/05/14/2146Z.md
Comment thread docs/hygiene-history/ticks/2026/05/14/2146Z.md
AceHack added a commit that referenced this pull request May 14, 2026
… operands (#3274)

Two substantive Copilot catches post-merge on PR #3271:

1. P1 rendering: single-backtick inline code span contained literal
   backticks inside the regex (`?` and `\`?`). Markdown terminates the
   span at the first inner backtick, rendering the rest as broken
   markup. Fix: convert to fenced ```bash code block.

2. P2 reproducibility: the command shown was missing file/path
   operands. As written, `grep -nE "..."` reads stdin instead of
   scanning the shard files. Fix: added
   `docs/hygiene-history/ticks/2026/05/14/2*.md` operand.

Also wrapped the regex in single-quotes (instead of double-quotes) so
the shell doesn't interpret the special characters.

Verified the new command runs correctly + produces the claimed
"zero other instances" result (grep exit code 1 = no matches).

Post-merge cleanup PR; #3271 already merged the shard with the bug.

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

Tick output:
- PR #3274 (fix(shard-2146Z) grep rendering + operands): converted
  inline grep command to fenced bash code block + added file operand.
  Two Copilot catches addressed (P1 rendering bug + P2 reproducibility).
  Verified corrected command exits 1 (no matches) matching claim.
  Auto-merge armed.
- Both threads on #3271 acknowledged with reply pointing to #3274;
  resolved.
- This shard.

Pre-push lint also caught an MD038 issue in the shard table (literal
triple-backticks inside single-backtick code span); fixed before push.
Pre-push lint continues to pay off.

Discipline codified:
- Inline-code-span: choose single/double/fenced based on content
- Reproducibility: every quoted command should be runnable as-is

Audit-discipline candidates filing list:
- audit-shard-broken-links.ts (filed 2146Z)
- temporal-provenance audit (filed 2151Z)
- audit-shard-runnable-commands.ts (this tick — new)

Prior-tick PRs merged this batch:
- #3271 (shard 2146Z) MERGED as 9691495
- #3273 (shard 2151Z) MERGED as 1862096

Session tally: 31 merged + 1 wait-ci (#3274) + this shard's PR.

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