Skip to content

docs(architecture): add testability rules and restructure skill#1460

Merged
piorpua merged 1 commit intomainfrom
docs/architecture-testability
Mar 19, 2026
Merged

docs(architecture): add testability rules and restructure skill#1460
piorpua merged 1 commit intomainfrom
docs/architecture-testability

Conversation

@kaizhou-lab
Copy link
Copy Markdown
Collaborator

Summary

  • Add testability rules (pure logic vs IO separation, dependency injection) and test file mapping conventions to architecture docs
  • Sync conventions to docs/conventions/file-structure.md
  • Restructure architecture skill using progressive disclosure: split 572-line SKILL.md into 140-line core + 3 reference files (renderer, process, project-layout)

Test plan

  • Verify architecture skill triggers correctly on file placement questions
  • Verify reference files are loaded when detailed context is needed

…closure

Split the 572-line SKILL.md into a concise 140-line core file with
three reference files for renderer, process, and project layout details.
Follows skill-creator best practices: keep SKILL.md under 500 lines,
load detailed references only when needed.
@piorpua piorpua merged commit 4a4b3db into main Mar 19, 2026
4 checks passed
@piorpua piorpua deleted the docs/architecture-testability branch March 19, 2026 09:54
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