Skip to content

Reorganize diagram scripts into scripts/diagrams subdirectory#2433

Merged
SatoryKono merged 8 commits intomainfrom
claude/list-recent-branches-ynTMs
Mar 1, 2026
Merged

Reorganize diagram scripts into scripts/diagrams subdirectory#2433
SatoryKono merged 8 commits intomainfrom
claude/list-recent-branches-ynTMs

Conversation

@SatoryKono
Copy link
Owner

Summary

Consolidates diagram-related scripts into a dedicated scripts/diagrams/ subdirectory to improve code organization and reduce clutter in the root scripts/ directory. Creates deprecation wrappers in the original locations to maintain backward compatibility during transition.

Changes

  • Script reorganization: Moved 15+ diagram validation and processing scripts to scripts/diagrams/:

    • check_diagram_quality_gates.py
    • check_svg_text_visibility.py
    • check_diagram_artifacts.py
    • check_diagram_visual_smoke.py
    • add_svg_text_fallback.py
    • inject_svg_styles.py
    • lint_diagrams.py
    • normalize_node_sizes.py
    • uniform_diagram_sizes.py
    • And others
  • Backward compatibility: Created thin deprecation wrappers at original script locations that delegate to the new subdirectory versions

  • Documentation: Added scripts/README.md explaining the new layout and consolidation rationale

  • CI/workflow updates: Updated GitHub Actions workflows (.github/workflows/docs.yml, diagram-nightly.yml) and Makefile to reference scripts in new location

  • Generated artifacts: Added diagram index files (PNG/SVG) and catalog HTMLs for architecture, foundation, class-diagrams, and views diagram sets

Checklist

  • Scripts maintain identical functionality (moved, not modified)
  • Deprecation wrappers preserve backward compatibility
  • CI workflows updated to use new script paths
  • No hardcoded secrets or credentials
  • Documentation added for new structure

https://claude.ai/code/session_016A29BqCCCnqbBCTyGKKZzS

SatoryKono and others added 8 commits March 1, 2026 16:34
Merges diagram rendering, scripts, views, and architecture diagrams
from TMP01-01. Conflicts resolved with TMP01-01 priority.

https://claude.ai/code/session_016A29BqCCCnqbBCTyGKKZzS
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@SatoryKono SatoryKono merged commit 2ccdaf5 into main Mar 1, 2026
7 of 32 checks passed
@SatoryKono SatoryKono deleted the claude/list-recent-branches-ynTMs branch March 1, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants