chore: organize project root directory structure#24
Merged
Conversation
Moved analysis documents, tests, and scripts to appropriate locations. Removed development artifacts. **Changes:** - Moved IMPLEMENTATION_SUMMARY.md → docs/architecture/hooks-system-implementation.md - Moved SHOWCASE_ANALYSIS_SUMMARY.md → docs/research/showcase-repository-analysis-2025-11-02.md - Moved ST-001-AUTO-ACTIVATION-ANALYSIS.md → docs/design-decisions/auto-activation-mechanism-analysis.md - Moved test_semantic_integration.py → tests/integration/test_semantic_integration.py - Moved run_semantic_tests.sh → scripts/run_semantic_tests.sh (updated test path) - Removed mcp_config.json (development artifact, actual config lives in .claude/) - Created ROOT_DIRECTORY_AUDIT.json for cleanup planning **Rationale:** - Analysis docs belong in docs/architecture/, docs/research/, docs/design-decisions/ - Tests belong in tests/integration/ - Scripts belong in scripts/ - mcp_config.json was a leftover development file (mapify creates .claude/mcp_config.json) - RELEASING.md and requirements files stay in root (standard locations) **Verification:** - All git moves preserve history - Generated files (coverage.json, verification JSON) already gitignored - Test runner script updated to reference new test location Completed as part of MAP efficient workflow cleanup_root_1762179205.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR reorganizes the repository structure by moving misplaced files to their appropriate locations and removing an obsolete configuration file. The changes improve project organization and maintainability.
Key changes:
- Moves integration test file and test runner script from root to proper directories
- Relocates analysis/research documentation to appropriate docs subdirectories
- Deletes obsolete MCP configuration file (duplicates source code template)
Reviewed Changes
Copilot reviewed 3 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/integration/test_semantic_integration.py | Integration test file moved from root to tests/integration/ directory |
| scripts/run_semantic_tests.sh | Test runner script path updated to reflect new test location |
| mcp_config.json | Obsolete MCP config file removed (duplicates template in source) |
| docs/research/showcase-repository-analysis-2025-11-02.md | Research analysis document moved from root to docs/research/ |
| docs/design-decisions/auto-activation-mechanism-analysis.md | Design analysis document moved from root to docs/design-decisions/ |
| docs/architecture/hooks-system-implementation.md | Implementation summary moved from root to docs/architecture/ |
| ROOT_DIRECTORY_AUDIT.json | New audit file documenting root directory cleanup plan |
Comments suppressed due to low confidence (1)
scripts/run_semantic_tests.sh:8
- The test script uses plain
pythoninstead ofpython3, which could cause issues on systems wherepythonpoints to Python 2.x. The shebang intest_semantic_integration.pyline 1 usespython3, creating an inconsistency. Consider usingpython3here for consistency and compatibility.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Create conftest.py with manager fixture - Fix sys.path.insert to use correct project root - Convert test_initialization to use fixture - Tests now work from tests/integration/ subdirectory
azalio
added a commit
that referenced
this pull request
Feb 13, 2026
…-35) MEDIUM fixes: - #8: Remove dead RETRY_LOOP phase from orchestrator STEP_PHASES - #10: Fix plan path to branch-scoped .map/<branch>/task_plan_<branch>.md - #11: Fix findings path to branch-scoped .map/<branch>/findings_<branch>.md - #12: Remove references to non-existent ralph-loop-config.json - #13/#14: Rewrite map-resume to use step_state.json instead of progress.md - #15: Fix INIT_PLAN heading format (### ST-XXX with - **Status:** prefix) - #16: Fix regex in step_runner to match plan format (### heading, - **Status:**) - #17: Fix map-learn contradiction about automatic learning LOW fixes: - #9/#31: Document dual state file system (step_state.json vs workflow_state.json) - #19: Document intentional Evaluator/Reflector/Curator omission in map-efficient - #20: Fix line count reference (~150 → ~540 lines) - #21: Standardize all AskUserQuestion to Python function call syntax - #22: Rename Steps 2.5/2.6 to 2a/2b to avoid phase number collision - #23/#24: Fix map-debate comparison table (map-efficient uses single Actor) - #25: Replace cat commands with Read tool comments in map-check - #28/#29: Replace undefined thrashing_detected()/max_redecompositions - #30: Add - **Status:** pending field to map-plan template - #32: Note map-fast max 3 vs map-efficient max 5 intentional difference - #33: Remove Evaluator from map-fast skipped agents list - #34: Move AskUserQuestion to "Built-in Tools" section in map-release - #35: Replace parallel bash & processes with sequential && in map-release Template sync: All .claude/ changes mirrored to src/mapify_cli/templates/
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
Comprehensive root directory cleanup to improve project organization and maintainability.
Changes
Files Moved
IMPLEMENTATION_SUMMARY.md→docs/architecture/hooks-system-implementation.mdSHOWCASE_ANALYSIS_SUMMARY.md→docs/research/showcase-repository-analysis-2025-11-02.mdST-001-AUTO-ACTIVATION-ANALYSIS.md→docs/design-decisions/auto-activation-mechanism-analysis.mdtest_semantic_integration.py→tests/integration/test_semantic_integration.pyrun_semantic_tests.sh→scripts/run_semantic_tests.sh(updated test path)Files Removed
mcp_config.json- Development artifact (actual config lives in.claude/mcp_config.json)coverage.json,mapify_cli_verification_*.json- Generated files (already gitignored)Files Created
ROOT_DIRECTORY_AUDIT.json- Comprehensive audit for future referenceFiles Kept in Root (with documented rationale)
RELEASING.md- Standard location for release documentationrequirements-*.txt- Python ecosystem conventionRationale
Before: 20 files in root (many misplaced)
After: 13 files in root (all standard project files)
Analysis documents now organized in:
docs/architecture/- Implementation summariesdocs/research/- External research analysisdocs/design-decisions/- Technical decision documentationTests and scripts moved to conventional locations.
Testing
git mvto preserve historyChecklist