Conversation
…ck-shard-TEMPLATE.md (OUTSIDE walked tree so doesn't trip schema validator); comment-block carries path-depth + schema + filename-regex reference for substrate-honest copy-paste; ticks/README.md updated with pointer; prevents recurring 5-level path bug at write-time vs gate-time. Implements option 1 from 2142Z shard's 3 future-work candidates (bandwidth-engineering at write-time; most-aligned with anti-extractive operating principles). Substantive substrate-engineering work; small bounded scope; counter reset via concrete artifact.
There was a problem hiding this comment.
Pull request overview
Adds a tick-shard template file at docs/hygiene-history/tick-shard-TEMPLATE.md (deliberately outside the schema-validated ticks/ walked tree) that documents the recurring 6-level path-depth requirement, schema, and filename regex inline. Updates the ticks/README.md to point contributors at the new template.
Changes:
- New
tick-shard-TEMPLATE.mdwith inline comment block covering path depth, schema, filename regex, and a pre-push verify command, plus an example shard body. - README in
ticks/adds a short pointer to the template.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/hygiene-history/tick-shard-TEMPLATE.md | New copy-paste template living outside the walked tree, with path-depth/schema/filename guidance. |
| docs/hygiene-history/ticks/README.md | Adds a 4-line pointer to the new template above the schema description. |
AceHack
deleted the
shard/tick-2153z-tick-shard-template-prevents-recurring-path-depth-bug-at-write-time-2026-05-18
branch
This was referenced May 18, 2026
AceHack
added a commit
that referenced
this pull request
May 18, 2026
AceHack
added a commit
that referenced
this pull request
May 18, 2026
…entries, grandfather 21 legacy findings from peer Otto-CLI shards in #4212 + earlier batches); restores lint to PASSING on CI gate; future-NEW findings (after TEMPLATE.md write-time fix #4220) will surface cleanly; substrate-debt finding from #4216 still tracked but no longer noisy on every PR (#4227)
AceHack
added a commit
that referenced
this pull request
May 18, 2026
…compliance); reframe 2115Z xref as referring to substrate state at write-time not on-main; reframe shard-template generator as hypothetical-not-yet-built (subsequently landed via #4220 as markdown TEMPLATE not TS generator)
AceHack
added a commit
that referenced
this pull request
May 18, 2026
… (option 1 from 2142Z); uses collision-suffix 'c' because peer Otto-CLI shipped own 2153Z.md; shard uses hybrid pipe-row+H1+sections format as working example of template's correct shape; all 3 pre-push checks pass (MD032 + markdownlint + relative-path audit) (#4221)
AceHack
added a commit
that referenced
this pull request
May 18, 2026
…+ 3 candidate future-work options (#4218) * shard(2142Z): brief reference-comment on peer Otto-CLI #4215 (same path-depth class as #4209); pattern observation — peer sibling-instance keeps hitting same issue across batches because lint(tick-shard relative-paths) is non-required (no branch protection); 3 candidate future-work options noted (shard-template generator, promote lint to required, accept advisory + document); shard-template most aligned with bandwidth-engineering discipline (prevents at write-time vs catches post-hoc) * fix(2142Z): 3 Codex P1/P2 findings — add pipe-row first line (schema compliance); reframe 2115Z xref as referring to substrate state at write-time not on-main; reframe shard-template generator as hypothetical-not-yet-built (subsequently landed via #4220 as markdown TEMPLATE not TS generator)
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.
TEMPLATE.md at docs/hygiene-history/ + ticks/README.md pointer. Substrate-engineering write-time fix for recurring 5-level path bug.