refactor: reorganize automation directory structure#3164
Merged
MarkusNeusinger merged 2 commits intomainfrom Jan 3, 2026
Merged
refactor: reorganize automation directory structure#3164MarkusNeusinger merged 2 commits intomainfrom
MarkusNeusinger merged 2 commits intomainfrom
Conversation
Reorganize directory structure to better separate concerns: - automation/ → Only workflow-specific scripts used by GitHub Actions - scripts/ → One-time migrations and manual utilities - core/generators/ → Reusable business logic components Changes: - Move one-time migration scripts to scripts/: - backfill_review_metadata.py - fix_library_versions.py - migrate_metadata_format.py - migrate_to_new_structure.py - upgrade_specs.py - upgrade_specs_ai.py - Move reusable generators to core/generators/: - plot_generator.py (from automation/generators/) - Update imports in tests and documentation - Update repository.md with new directory structure - Add sections explaining automation/ vs scripts/ principles All tests pass successfully.
Update internal documentation in migration scripts to reflect new paths in scripts/ directory instead of automation/scripts/.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR reorganizes the repository's directory structure to better separate concerns between workflow automation, one-time migration scripts, and reusable business logic. The primary goal is to clarify the purpose of different script types and improve maintainability.
Key Changes:
- Move reusable plot generation utilities from
automation/generators/tocore/generators/ - Move one-time migration scripts to a new
scripts/directory - Update documentation to explain the principles behind
automation/vsscripts/separation
Reviewed changes
Copilot reviewed 6 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/unit/core/generators/test_plot_generator.py | Updated import path to reflect plot_generator move to core.generators |
| scripts/upgrade_specs_ai.py | New file: AI-powered spec upgrader using Claude (moved from automation) |
| scripts/upgrade_specs.py | New file: Structural spec version upgrader (moved from automation) |
| scripts/migrate_to_new_structure.py | Updated usage documentation path from automation/scripts to scripts/ |
| scripts/migrate_metadata_format.py | Updated usage documentation path from automation/scripts to scripts/ |
| scripts/fix_library_versions.py | New file: Library version synchronization script (moved from automation) |
| scripts/backfill_review_metadata.py | Updated usage documentation paths from automation/scripts to scripts/ |
| docs/concepts/claude-skill-plot-generation.md | Updated example import path from automation.generators to core.generators |
| docs/architecture/repository.md | Added detailed documentation of new directory structure and separation principles |
| core/generators/plot_generator.py | New file: Reusable plot code generator moved from automation (now in core) |
| core/generators/init.py | New file: Package initialization for core.generators module |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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.
Reorganize directory structure to better separate concerns:
Changes:
Move one-time migration scripts to scripts/:
Move reusable generators to core/generators/:
Update imports in tests and documentation
Update repository.md with new directory structure
Add sections explaining automation/ vs scripts/ principles
All tests pass successfully.