Skip to content

docs: add project skills for architecture, commit, PR, testing, and i18n#1420

Merged
piorpua merged 12 commits intomainfrom
docs/add-project-skills
Mar 19, 2026
Merged

docs: add project skills for architecture, commit, PR, testing, and i18n#1420
piorpua merged 12 commits intomainfrom
docs/add-project-skills

Conversation

@kaizhou-lab
Copy link
Copy Markdown
Collaborator

Summary

  • Add 5 skill files (.claude/skills/) codifying project conventions: architecture, commit, PR, testing, i18n
  • Add docs/conventions/file-structure.md for directory naming and structure rules
  • Streamline AGENTS.md to reference skills instead of duplicating rules

Changes

New skills:

  • architecture — file/directory structure, process boundaries, module conversion patterns
  • commit — structured commit workflow with branch guard (prevents direct commits to main)
  • pr — pull request workflow with automatic issue creation and association
  • testing — test quality standards, workflow, and checklist
  • i18n — refactored internationalization workflow

Updated files:

  • AGENTS.md — simplified, now references skill files; added Skills Index table
  • CLAUDE.md — added @AGENTS.md reference
  • docs/conventions/file-structure.md — new file structure conventions document

Related Issue

Closes #1419

Test Plan

  • All skills are markdown-only, no code changes
  • Verify skill auto-triggers match expected scenarios
  • Confirm AGENTS.md renders correctly and links resolve

zk added 12 commits March 18, 2026 21:49
- Add architecture skill covering all process types, directory naming, and process boundary rules
- Add commit skill with branch guard, quality gates, and mandatory rules
- Add testing skill with quality standards, edge case checklist, and coverage requirements
- Update i18n skill to dynamically reference i18n-config.json instead of hardcoded values
- Add docs/conventions/file-structure.md for human-readable structure reference
- Simplify AGENTS.md by replacing inline rules with skill references
- Add Skills Index table listing all 4 skills with triggers
- Clarify that skills apply to all agents and contributors
- Add architecture and i18n skill usage notes to CLAUDE.md
@piorpua piorpua merged commit 79e85c1 into main Mar 19, 2026
4 checks passed
@piorpua piorpua deleted the docs/add-project-skills branch March 19, 2026 03:59
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.

docs: add project skills for architecture, commit, PR, testing, and i18n

2 participants