chore: cleanup project structure and sync rules #417
Merged
Conversation
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
- Remove test/debug files from production (malformed-agent.md, test-agent.md) - Remove duplicate e2e-test-engineer agent (frameworks version is canonical) - Remove historical documentation (PHASE1-*, PHASE2-*, PHASE3_*, TASK-*) - Remove archived rules (rules-archive directory) - Remove old vitepress config.js (config.ts is active) - Remove disabled tests and migration artifacts - Create missing includes/task-creation-excellence.md (fixes 19 broken @include) - Sync rules from autopm/ to root .claude/rules/: - agent-mandatory-optimized.md - command-pipelines.md - context-optimization.md - context7-enforcement-optimized.md - github-operations.md Total reduction: ~150KB of redundant files
Contributor
There was a problem hiding this comment.
Pull request overview
This PR performs a comprehensive cleanup of the project structure, removing historical documentation, test artifacts, and deprecated files while establishing a cleaner organization.
Key changes:
- Removed 20+ obsolete files including disabled tests, historical phase documentation, and archived rules
- Created missing
task-creation-excellence.mdinclude file referenced by 7 agent definitions - Synced 5 rule files from autopm/ to root .claude/rules/ for consistency
- Removed duplicate e2e-test-engineer agent and test/debug agent files
Reviewed changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| test/unit/self-maintenance.test.js.disabled | Removed disabled unit tests for self-maintenance (544 lines) |
| test/test-report.json | Removed old test report artifact from September 2025 |
| test/jest-tests/migrate-commands.js | Removed migration helper script (115 lines) |
| test/jest-tests/jest-migration-helper.js | Removed Jest migration utilities (195 lines) |
| test/e2e/pm-commands.test.js.disabled | Removed disabled E2E tests (277 lines) |
| test/RUN-SIMULATION.md | Removed Polish-language simulation guide (124 lines) |
| docs/TASK-*.md | Removed 2 historical task completion documents (~640 lines total) |
| docs/PHASE*.md | Removed 5 phase planning/status documents (~2750 lines total) |
| docs-site/docs/.vitepress/config.js | Removed old VitePress config (config.ts is active) |
| .claude/rules/*.md | Added 5 synced rule files from autopm/ (github-operations, context7-enforcement-optimized, context-optimization, command-pipelines, agent-mandatory-optimized) |
| .claude/rules-archive/*.md | Removed 3 archived rule documents (performance-guidelines, golden-rules, definition-of-done) |
| .claude/agents/testing/e2e-test-engineer.md | Removed duplicate agent (frameworks version is canonical) |
| .claude/agents/core/test-agent.md | Removed test/debug agent file (60 lines) |
| .claude/agents/core/malformed-agent.md | Removed malformed test file |
| .claude/agents/includes/task-creation-excellence.md | Created missing include file referenced by agents (80 lines) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,80 @@ | |||
| ## Task Creation Excellence | |||
There was a problem hiding this comment.
The PR description claims this file "fixes 19 broken @include" references, but a search across the .claude/agents directory only found 7 files that reference task-creation-excellence.md. This discrepancy should be clarified - either the count is incorrect or there were additional references that have already been removed in this PR.
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
Changes
Net reduction: ~5000 lines of redundant code/documentation
Test plan