Consolidate rules into CLAUDE.md and fix broken agent paths#85
Merged
Conversation
Migrate from .agents/ to .claude/ as the single skills/rules home: - Fold the four .claude/rules/*.md files into CLAUDE.md as compact sections (Writing Style, PyPI Packaging, Worktree Map) and delete the rules/ folder to simplify the project. - Fix every stale .agents/... path in CLAUDE.md to .claude/... - Remove references to non-existent AGENTS.md symlink and MEMORY.md. - Add missing modules (gui.py, gui/) to the worktree map. - Fix the html-report-builder template path to its real location under .claude/skills/html-report-builder/assets/. - Trim and tighten all skill SKILL.md files: drop duplicated checklists, inline HTML templates, and redundant guidance. - Net reduction of ~234 lines while keeping all substance.
Resolve modify/delete conflict on AGENTS.md by keeping the deletion (CLAUDE.md is the canonical home). Fold main's new Token efficiency section into CLAUDE.md. Update worktree map to reflect the gui.py -> gui/ package split from PR #84.
Code Coverage OverviewLanguages: Python Python / code-coverage/pytestThe overall coverage remains at 74%, unchanged from the branch. Code Coverage is in Public Preview. Learn more and provide us with your feedback. |
Contributor
There was a problem hiding this comment.
Pull request overview
Consolidates agent-facing repo guidance by removing legacy .agents/ / AGENTS.md content, standardizing on .claude/ as canonical home for skills, and expanding/refreshing the Related Projects reference corpus used for PKG/PFS/exFAT research.
Changes:
- Remove legacy
AGENTS.md+ old.agents/rules/*and.agents/skills/*trees. - Add/refresh
.claude/skills/*/SKILL.mdskills (tests, PR writing, report builder, related-project add/index). - Add multiple
.claude/skills/related-projects-index/references/*.mdreference notes and update the index formatting.
Reviewed changes
Copilot reviewed 17 out of 33 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| AGENTS.md | Removes legacy duplicated development-guideline content. |
| .claude/skills/running-unit-tests/SKILL.md | Adds canonical test/format/lint workflow guidance under .claude/. |
| .claude/skills/related-projects-index/SKILL.md | Cleans up index phrasing/format and points to references/. |
| .claude/skills/related-projects-index/references/wololo-fpkg-flatz.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/tlou-psarc-tool.md | Adds upstream reference note (contains local-path mention to fix). |
| .claude/skills/related-projects-index/references/shadpkg-howworks.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/shadowmountplus.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/psdevwiki-pkg-files.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/psdevwiki-pfs.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/ps5-game-compressor.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/pkgtool.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/pkg-passcode-finder.md | Adds upstream reference note (contains local-path mention to fix). |
| .claude/skills/related-projects-index/references/oodle-kraken-tools.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/liborbispkg.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/liborbispkg-wiki.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/kstuff-lite.md | Adds upstream reference note. |
| .claude/skills/related-projects-index/references/goodle.md | Adds upstream reference note (contains local-path mention to fix). |
| .claude/skills/related-project-add/SKILL.md | Adds canonical workflow for adding related-project entries. |
| .claude/skills/pr-and-commit-writing/SKILL.md | Adds canonical public-text guidance for commits/PRs/comments. |
| .claude/skills/html-report-builder/SKILL.md | Adds canonical HTML-report workflow guidance. |
| .claude/skills/html-report-builder/assets/report-template.html | Adds report template at new canonical path. |
| .agents/skills/running-unit-tests/SKILL.md | Removes legacy skill copy under .agents/. |
| .agents/skills/related-project-add/SKILL.md | Removes legacy skill copy under .agents/. |
| .agents/skills/pr-and-commit-writing/SKILL.md | Removes legacy skill copy under .agents/. |
| .agents/skills/html-report-builder/SKILL.md | Removes legacy skill copy under .agents/. |
| .agents/rules/writing-style.md | Removes legacy rule file under .agents/. |
| .agents/rules/worktree.md | Removes legacy rule file under .agents/. |
| .agents/rules/tmp-usage.md | Removes legacy rule file under .agents/. |
| .agents/rules/pypi-project-guidelines.md | Removes legacy rule file under .agents/. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Consolidate rules into CLAUDE.md and fix broken agent paths
🤔 Why?
.claude/rules/folder with four small markdown files plus stale.agents/...path references in CLAUDE.md that pointed to folders/symlinks that no longer exist (.agents/,AGENTS.md,MEMORY.md).🔧 What changed
.agents/→.claude/as the single home for skills and rules; removed the old.agents/tree,AGENTS.md, and.pisymlink..claude/rules/folder — simpler project, smaller context footprint..agents/...pointers now read.claude/...; removed references to non-existentAGENTS.mdandMEMORY.md.gui.py→gui/package split (from Split the monolith gui.py into a focused package #84) and added the Token efficiency section that previously lived only inAGENTS.md..claude/skills/html-report-builder/assets/.🧪 How to test
cat .claude/skills/html-report-builder/assets/report-template.htmland grep CLAUDE.md for.agents(should be empty).💬 Notes for non-technical readers