Defame1297/holocron
0
0
Fork 0
Code Issues 14 Pull Requests Actions Packages Projects Releases 3 Wiki Activity

refactor(bin): move non-spec root files into references/ across four skills #133

Merged
Defame1297 merged 2 commits from refactor/122-move-skill-root-files into main 2026-09-10 19:36:16 +00:00
Conversation 1 Commits 2 Files Changed 40 +66 -66
Claude commented 2026-09-09 20:12:58 +00:00
Collaborator
Copy Link

Summary

Fixes #122. grill-with-docs, improve-codebase-architecture, tdd, and triage kept non-spec markdown files at their skill root, violating skill-audit's file-structure.md rule (only SKILL.md/README.md belong at the root; everything else lives in scripts/, references/, assets/, or tests/). A root-level file is invisible to the ADR-0020 dangling-reference gate, which only resolves unqualified references/... pointers.

Moved and renamed to lowercase-kebab-case under references/ (specced in a grill-with-docs design session before implementation):

  • grill-with-docs: ADR-FORMAT.md, CONTEXT-FORMAT.md
  • improve-codebase-architecture: DEEPENING.md, INTERFACE-DESIGN.md, LANGUAGE.md
  • tdd: 5 files (casing was already fine, just moved)
  • triage: AGENT-BRIEF.md, OUT-OF-SCOPE.md

Also:

  • Updated every in-skill link to the new references/ paths, including link text that still showed the old uppercase filenames.
  • Fixed improve-codebase-architecture/SKILL.md's cross-skill citation of grill-with-docs's two files to the sanctioned possessive form (per file-structure.md) with the references/ segment included.
  • Updated all four skills' README.md file tables.
  • Regenerated the flat content mirror via scripts/sync-plugin-content.sh --all.

Test plan

  • bash tests/run-tests.sh --strict — 26/26 suites passed, 0 skipped, 0 failed (includes ADR-0020 dangling-reference and frontmatter checks)
  • pre-push gate (pre-commit run --hook-stage pre-push --all-files) — passes

🤖 Generated with Claude Code

https://claude.ai/code/session_01PDj6F7SPXzh3FtPN78dZ88

## Summary Fixes #122. `grill-with-docs`, `improve-codebase-architecture`, `tdd`, and `triage` kept non-spec markdown files at their skill root, violating `skill-audit`'s `file-structure.md` rule (only `SKILL.md`/`README.md` belong at the root; everything else lives in `scripts/`, `references/`, `assets/`, or `tests/`). A root-level file is invisible to the ADR-0020 dangling-reference gate, which only resolves unqualified `references/...` pointers. Moved and renamed to lowercase-kebab-case under `references/` (specced in a grill-with-docs design session before implementation): - `grill-with-docs`: `ADR-FORMAT.md`, `CONTEXT-FORMAT.md` - `improve-codebase-architecture`: `DEEPENING.md`, `INTERFACE-DESIGN.md`, `LANGUAGE.md` - `tdd`: 5 files (casing was already fine, just moved) - `triage`: `AGENT-BRIEF.md`, `OUT-OF-SCOPE.md` Also: - Updated every in-skill link to the new `references/` paths, including link text that still showed the old uppercase filenames. - Fixed `improve-codebase-architecture/SKILL.md`'s cross-skill citation of `grill-with-docs`'s two files to the sanctioned possessive form (per `file-structure.md`) with the `references/` segment included. - Updated all four skills' `README.md` file tables. - Regenerated the flat content mirror via `scripts/sync-plugin-content.sh --all`. ## Test plan - [x] `bash tests/run-tests.sh --strict` — 26/26 suites passed, 0 skipped, 0 failed (includes ADR-0020 dangling-reference and frontmatter checks) - [x] pre-push gate (`pre-commit run --hook-stage pre-push --all-files`) — passes 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01PDj6F7SPXzh3FtPN78dZ88
Claude added 1 commit 2026-09-09 20:12:58 +00:00
refactor(bin): move non-spec root files into references/ across four skills 4f49b2a249 
grill-with-docs, improve-codebase-architecture, tdd, and triage kept
non-spec markdown files at their skill root, in violation of
skill-audit's file-structure.md rule (only SKILL.md/README.md belong
at the root; everything else lives in scripts/, references/, assets/
or tests/). A root-level file is invisible to the ADR-0020
dangling-reference gate, which only resolves unqualified
`references/...` pointers.

- Moved and renamed to lowercase-kebab-case under references/:
  grill-with-docs (ADR-FORMAT.md, CONTEXT-FORMAT.md),
  improve-codebase-architecture (DEEPENING.md, INTERFACE-DESIGN.md,
  LANGUAGE.md), tdd (five files, casing was already fine), triage
  (AGENT-BRIEF.md, OUT-OF-SCOPE.md).
- Updated every in-skill link to the new references/ paths, including
  link text that still showed the old uppercase filenames.
- Fixed improve-codebase-architecture/SKILL.md's cross-skill citation
  of grill-with-docs's two files to the sanctioned possessive form
  with the references/ segment included.
- Updated all four skills' README.md file tables to match.
- Regenerated the flat content mirror via
  scripts/sync-plugin-content.sh --all.

Fixes #122.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PDj6F7SPXzh3FtPN78dZ88
Defame1297 added 1 commit 2026-09-10 19:17:23 +00:00
fix(bin): repoint improve-codebase-architecture README's LANGUAGE.md mention c75e4ef4f3 
PR #133 renamed the skill's root-level LANGUAGE.md to references/language.md
but missed a prose mention (not a markdown link) in the overview paragraph.
Fix both the .apm/ source and its generated flat mirror.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PDj6F7SPXzh3FtPN78dZ88
Defame1297 approved these changes 2026-09-10 19:36:12 +00:00
Defame1297 merged commit 5d7c76d797 into main 2026-09-10 19:36:16 +00:00
Defame1297 deleted branch refactor/122-move-skill-root-files 2026-09-10 19:36:16 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Defame1297
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Claude renovate-bot Defame1297
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Defame1297/holocron#133
Delete Branch "refactor/122-move-skill-root-files"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?