-
Notifications
You must be signed in to change notification settings - Fork 1
shard(tick): 2146Z — post-merge cleanup of 2139Z broken relative-link bug #3271
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
AceHack
merged 1 commit into
main
from
shard/tick-2146Z-broken-relative-links-fix-otto-cli-2026-05-14
May 14, 2026
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,91 @@ | ||
| # Tick 2026-05-14T21:46Z — post-merge cleanup: 2139Z broken relative-link fix | ||
|
|
||
| ## Refresh (step 1) | ||
|
|
||
| - Cron sentinel `12fb713e` live. | ||
| - Prior-tick PRs polled: | ||
| - **#3263** (shard 2135Z) merged earlier as `29c195e` | ||
| - **#3264** (shard 2139Z) — merged as `5a88735`, post-merge thread (Copilot) | ||
| - **#3267** (shard 2143Z minimal-continuation) — wait-ci, autoMerge armed | ||
| - **#3265** (Lior PR-archive batch from another Otto) — merged as `1eafd60` | ||
|
|
||
| ## Holding discipline (step 2) | ||
|
|
||
| Post-merge Copilot thread on #3264. Per | ||
| `.claude/rules/blocked-green-ci-investigate-threads.md`: investigate. | ||
|
|
||
| ## Work (step 3) — post-merge cleanup of broken relative links | ||
|
|
||
| Copilot catch on `docs/hygiene-history/ticks/2026/05/14/2139Z.md`: | ||
|
|
||
| > P1 (xref): These markdown links to `.claude/rules/...` are relative | ||
| > to the tick file's directory, so they resolve to | ||
| > `docs/hygiene-history/ticks/2026/05/14/.claude/...` and will 404. | ||
| > This issue also appears in the following locations of the same file: | ||
| > lines 30, 39. | ||
|
|
||
| Real substantive catch — broken rendered-view links. Three occurrences: | ||
|
|
||
| - Line 12: `.claude/rules/holding-without-named-dependency-is-standing-by-failure.md` | ||
| - Line 31: `.claude/rules/never-be-idle.md` | ||
| - Line 40: `docs/hygiene-history/ticks/2026/05/14/2001Z.md` | ||
|
|
||
| The session's other tick shards (2010Z, 2026Z, 2030Z, 2034Z, 2046Z, etc.) | ||
| use plain backticks WITHOUT markdown links — matching the established | ||
| convention. 2139Z accidentally introduced the markdown-link pattern. | ||
|
|
||
| Substantive landing: | ||
|
|
||
| - PR **#3269** (fix(shard-2139Z) broken relative links): replaced 3 | ||
| occurrences with plain backtick form. Auto-merge armed. | ||
| - Thread on #3264 acknowledged with reply pointing to #3269; resolved. | ||
|
|
||
| Scan of OTHER today's shards for the same bug: | ||
| `grep -nE "\[\`?[^]]*\`?\]\(\.claude/|\[[^]]*\]\(docs/hygiene"` — | ||
|
AceHack marked this conversation as resolved.
|
||
| **zero other instances**. Bug isolated to 2139Z. | ||
|
|
||
| ## Verify (step 4) | ||
|
|
||
| - `markdownlint-cli2` clean on the fixed shard | ||
| - No relative `.claude/...` or `docs/hygiene-history/...` markdown links | ||
| remain in 2139Z | ||
| - Thread resolved via GraphQL (`isResolved=true`) | ||
| - Composite branch-guard + `gh pr create --head` used | ||
|
|
||
| ## Shard (step 5) | ||
|
|
||
| This file. | ||
|
|
||
| ## CronList (step 6) | ||
|
|
||
| Sentinel `12fb713e` armed; one entry, recurring. | ||
|
|
||
| ## Visibility (step 7) | ||
|
|
||
| - **PR #3269** (broken-link fix) — wait-ci, autoMerge armed | ||
| - **PR #3267** (shard 2143Z) — still wait-ci, autoMerge armed | ||
| - **#3263 / #3264 / #3265** merged this batch | ||
| - Session running tally: **27 merged + 2 wait-ci** (then plus this shard's PR) | ||
|
|
||
| ## Notes for future-Otto | ||
|
|
||
| **Shard-authoring convention codified**: use plain backticks for path | ||
| references, NOT markdown links. Markdown links from | ||
| `docs/hygiene-history/ticks/YYYY/MM/DD/HHMMZ.md` resolve relative to | ||
| that directory, so any path starting with `.claude/` or `docs/` would | ||
| 404 in rendered views. | ||
|
|
||
| Two acceptable patterns: | ||
|
|
||
| 1. **Plain backticks (preferred)**: `` `.claude/rules/foo.md` `` | ||
| 2. **Root-relative link**: `[link](/path/to/foo.md)` | ||
|
|
||
| The repo convention is (1). Future-Otto authoring tick shards should | ||
| use plain backticks unless the linked content is genuinely actionable | ||
| (e.g., a backlog row a reviewer needs to navigate). Even then, prefer | ||
| root-relative over directory-relative. | ||
|
|
||
| Pre-push lint catch candidate: `audit-shard-broken-links.ts` could scan | ||
| for `](\.claude/` or `](docs/` patterns inside tick-shard files. Filed | ||
| mentally as a future audit-discipline candidate (cousin to | ||
| `audit-md032-plus-linestart.ts`). | ||
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.