Implementation Plan: Remove AGENTS.md Per-File Catalogs and Retire Their Checker - #4436
Merged
Trecek merged 7 commits intoJul 31, 2026
Merged
Conversation
Trecek
commented
Jul 31, 2026
Trecek
left a comment
Collaborator
Author
There was a problem hiding this comment.
AutoSkillit PR Review — Verdict: approved_with_comments
Summary: 13 inline findings (4 warning, 9 info). No critical/blocking issues. See inline comments for detail. Warnings route to resolve-review.
⚠️ Outside Diff Range
These findings target lines not in the diff and could not be posted as inline comments:
src/autoskillit/cli/session/AGENTS.md
- L385 [warning/cohesion]: Asymmetric prose preservation across CLI siblings — cli/session/AGENTS.md and cli/ui/AGENTS.md preserve critical behavio
src/autoskillit/hooks/_capture/AGENTS.md
- L870 [info/cohesion]: hooks/_capture/AGENTS.md is a pure deletion (-21 lines, no prose preservation) while its sibling hooks/AGENTS.md and hoo
src/autoskillit/hooks/guards/AGENTS.md
- L970 [warning/cohesion]: hooks/guards/AGENTS.md is the only file in the PR to receive a major restructure — a brand-new 33-line 'Guard Capabiliti
tests/docs/test_sub_claude_md_completeness.py
- L2591 [warning/cohesion]: Hard-coded family and total counts in test_tracked_guidance_families_and_counts — `assert tuple(len(family) for family i
Trecek
enabled auto-merge
July 31, 2026 02:34
Trecek
deleted the
impl-issue-4430-remove-agents-catalogs-20260730-161350
branch
July 31, 2026 02:48
8 tasks
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.
Summary
Remove exhaustive per-file inventories from the repository guidance hierarchy while retaining all 63 tracked
AGENTS.mdguides, all 62 existingCLAUDE.mdadapters, and every non-catalog instruction. Directory/package indexes and behavioral contract tables remain.Retire
scripts/check_sub_agents_md.py, its pre-commit hook, direct tests, catalog-positive assertions, module-name coverage assertion, and test-filter route. Replace those positive filename-completeness checks with tracked-file parity, catalog-absence detection, and explicit preservation canaries for Channel B,_captureisolation, guard invariants, and catalog-carried behavior.Validation
pre-commit run --all-filesAUTOSKILLIT_TEST_BASE_REF=develop task test-filtered— 32,518 passed, 559 skipped, 27 xfailedtask test-all— 34,081 passed, 630 skipped, 27 xfailedCloses #4430
Implementation Plan
Plan file:
/home/talon/projects/generic_automation_mcp/.autoskillit/temp/make-plan/issue_4430_remove_agents_catalogs_plan_2026-07-30_153128.md🤖 Generated with Codex via AutoSkillit