@cregis-dev/cckit 0.3.0 → 0.4.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +15 -3
- package/package.json +1 -1
- package/registry.json +234 -200
- package/src/commands/init.js +19 -15
- package/src/commands/update.js +9 -4
- package/src/core/plugin-installer.js +59 -13
- package/src/utils/fs.js +6 -1
- package/templates/bmad/_config/bmad-help.csv +2 -1
- package/templates/bmad/_config/files-manifest.csv +437 -431
- package/templates/bmad/_config/ides/claude-code.yaml +5 -5
- package/templates/bmad/_config/ides/trae.yaml +5 -5
- package/templates/bmad/_config/task-manifest.csv +7 -6
- package/templates/bmad/_config/workflow-manifest.csv +34 -34
- package/templates/bmad/bmm/agents/qa.md +1 -1
- package/templates/bmad/bmm/config.yaml +1 -1
- package/templates/bmad/bmm/module-help.csv +1 -1
- package/templates/bmad/bmm/workflows/1-analysis/create-product-brief/steps/step-06-complete.md +1 -1
- package/templates/bmad/bmm/workflows/1-analysis/create-product-brief/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/1-analysis/research/workflow-domain-research.md +1 -1
- package/templates/bmad/bmm/workflows/1-analysis/research/workflow-market-research.md +1 -1
- package/templates/bmad/bmm/workflows/1-analysis/research/workflow-technical-research.md +1 -1
- package/templates/bmad/bmm/workflows/2-plan-workflows/create-prd/steps-c/step-12-complete.md +1 -1
- package/templates/bmad/bmm/workflows/2-plan-workflows/create-prd/steps-v/step-v-13-report-complete.md +1 -1
- package/templates/bmad/bmm/workflows/2-plan-workflows/create-prd/workflow-create-prd.md +1 -1
- package/templates/bmad/bmm/workflows/2-plan-workflows/create-prd/workflow-edit-prd.md +1 -1
- package/templates/bmad/bmm/workflows/2-plan-workflows/create-prd/workflow-validate-prd.md +1 -1
- package/templates/bmad/bmm/workflows/2-plan-workflows/create-ux-design/steps/step-14-complete.md +1 -1
- package/templates/bmad/bmm/workflows/2-plan-workflows/create-ux-design/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/3-solutioning/check-implementation-readiness/steps/step-06-final-assessment.md +1 -1
- package/templates/bmad/bmm/workflows/3-solutioning/check-implementation-readiness/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/3-solutioning/create-architecture/steps/step-08-complete.md +1 -1
- package/templates/bmad/bmm/workflows/3-solutioning/create-architecture/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/3-solutioning/create-epics-and-stories/steps/step-04-final-validation.md +1 -1
- package/templates/bmad/bmm/workflows/3-solutioning/create-epics-and-stories/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/4-implementation/code-review/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/4-implementation/correct-course/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/4-implementation/create-story/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/4-implementation/dev-story/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/4-implementation/retrospective/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/4-implementation/sprint-planning/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/4-implementation/sprint-status/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/bmad-quick-flow/quick-dev/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/bmad-quick-flow/quick-spec/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/document-project/workflow.yaml +1 -1
- package/templates/bmad/bmm/workflows/generate-project-context/workflow.md +1 -1
- package/templates/bmad/bmm/workflows/qa-generate-e2e-tests/workflow.yaml +2 -2
- package/templates/bmad/commands/bmad-bmm-check-implementation-readiness.md +2 -2
- package/templates/bmad/commands/bmad-bmm-code-review.md +4 -4
- package/templates/bmad/commands/bmad-bmm-correct-course.md +4 -4
- package/templates/bmad/commands/bmad-bmm-create-architecture.md +2 -2
- package/templates/bmad/commands/bmad-bmm-create-epics-and-stories.md +2 -2
- package/templates/bmad/commands/bmad-bmm-create-prd.md +2 -2
- package/templates/bmad/commands/bmad-bmm-create-product-brief.md +2 -2
- package/templates/bmad/commands/bmad-bmm-create-story.md +4 -4
- package/templates/bmad/commands/bmad-bmm-create-ux-design.md +2 -2
- package/templates/bmad/commands/bmad-bmm-dev-story.md +4 -4
- package/templates/bmad/commands/bmad-bmm-document-project.md +4 -4
- package/templates/bmad/commands/bmad-bmm-domain-research.md +2 -2
- package/templates/bmad/commands/bmad-bmm-edit-prd.md +2 -2
- package/templates/bmad/commands/bmad-bmm-generate-project-context.md +2 -2
- package/templates/bmad/commands/bmad-bmm-market-research.md +2 -2
- package/templates/bmad/commands/bmad-bmm-qa-generate-e2e-tests.md +4 -4
- package/templates/bmad/commands/bmad-bmm-quick-dev.md +2 -2
- package/templates/bmad/commands/bmad-bmm-quick-spec.md +2 -2
- package/templates/bmad/commands/bmad-bmm-retrospective.md +4 -4
- package/templates/bmad/commands/bmad-bmm-sprint-planning.md +4 -4
- package/templates/bmad/commands/bmad-bmm-sprint-status.md +4 -4
- package/templates/bmad/commands/bmad-bmm-technical-research.md +2 -2
- package/templates/bmad/commands/bmad-bmm-validate-prd.md +2 -2
- package/templates/bmad/commands/bmad-brainstorming.md +2 -2
- package/templates/bmad/commands/bmad-editorial-review-prose.md +1 -1
- package/templates/bmad/commands/bmad-editorial-review-structure.md +1 -1
- package/templates/bmad/commands/bmad-help.md +1 -1
- package/templates/bmad/commands/bmad-index-docs.md +1 -1
- package/templates/bmad/commands/bmad-party-mode.md +2 -2
- package/templates/bmad/commands/bmad-review-adversarial-general.md +1 -1
- package/templates/bmad/commands/bmad-review-edge-case-hunter.md +10 -0
- package/templates/bmad/commands/bmad-shard-doc.md +1 -1
- package/templates/bmad/commands/bmad-tea-teach-me-testing.md +2 -2
- package/templates/bmad/commands/bmad-tea-testarch-atdd.md +4 -4
- package/templates/bmad/commands/bmad-tea-testarch-automate.md +4 -4
- package/templates/bmad/commands/bmad-tea-testarch-ci.md +4 -4
- package/templates/bmad/commands/bmad-tea-testarch-framework.md +4 -4
- package/templates/bmad/commands/bmad-tea-testarch-nfr.md +4 -4
- package/templates/bmad/commands/bmad-tea-testarch-test-design.md +4 -4
- package/templates/bmad/commands/bmad-tea-testarch-test-review.md +4 -4
- package/templates/bmad/commands/bmad-tea-testarch-trace.md +4 -4
- package/templates/bmad/core/config.yaml +1 -1
- package/templates/bmad/core/module-help.csv +1 -0
- package/templates/bmad/core/tasks/editorial-review-prose.xml +1 -1
- package/templates/bmad/core/tasks/editorial-review-structure.xml +1 -1
- package/templates/bmad/core/tasks/help.md +1 -1
- package/templates/bmad/core/tasks/index-docs.xml +1 -1
- package/templates/bmad/core/tasks/review-adversarial-general.xml +1 -1
- package/templates/bmad/core/tasks/review-edge-case-hunter.xml +63 -0
- package/templates/bmad/core/tasks/shard-doc.xml +1 -1
- package/templates/bmad/core/workflows/advanced-elicitation/workflow.xml +1 -1
- package/templates/bmad/core/workflows/brainstorming/steps/step-01-session-setup.md +31 -18
- package/templates/bmad/core/workflows/brainstorming/steps/step-01b-continue.md +1 -1
- package/templates/bmad/core/workflows/brainstorming/steps/step-03-technique-execution.md +2 -2
- package/templates/bmad/core/workflows/brainstorming/steps/step-04-idea-organization.md +2 -2
- package/templates/bmad/core/workflows/brainstorming/workflow.md +4 -2
- package/templates/bmad/core/workflows/party-mode/workflow.md +1 -1
- package/templates/bmad/tea/config.yaml +5 -1
- package/templates/bmad/tea/testarch/knowledge/contract-testing.md +24 -2
- package/templates/bmad/tea/testarch/knowledge/pact-mcp.md +204 -0
- package/templates/bmad/tea/testarch/knowledge/pactjs-utils-consumer-helpers.md +211 -0
- package/templates/bmad/tea/testarch/knowledge/pactjs-utils-overview.md +210 -0
- package/templates/bmad/tea/testarch/knowledge/pactjs-utils-provider-verifier.md +315 -0
- package/templates/bmad/tea/testarch/knowledge/pactjs-utils-request-filter.md +224 -0
- package/templates/bmad/tea/testarch/tea-index.csv +5 -0
- package/templates/bmad/tea/workflows/testarch/README.md +1 -1
- package/templates/bmad/tea/workflows/testarch/atdd/steps-c/step-01-preflight-and-context.md +30 -0
- package/templates/bmad/tea/workflows/testarch/atdd/steps-c/step-04-generate-tests.md +159 -57
- package/templates/bmad/tea/workflows/testarch/atdd/steps-c/step-04a-subagent-api-failing.md +215 -0
- package/templates/bmad/tea/workflows/testarch/atdd/steps-c/step-04b-subagent-e2e-failing.md +244 -0
- package/templates/bmad/tea/workflows/testarch/atdd/steps-c/step-04c-aggregate.md +31 -15
- package/templates/bmad/tea/workflows/testarch/atdd/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/atdd/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/atdd/workflow.yaml +2 -2
- package/templates/bmad/tea/workflows/testarch/automate/steps-c/step-01-preflight-and-context.md +32 -0
- package/templates/bmad/tea/workflows/testarch/automate/steps-c/step-03-generate-tests.md +215 -101
- package/templates/bmad/tea/workflows/testarch/automate/steps-c/step-03a-subagent-api.md +193 -0
- package/templates/bmad/tea/workflows/testarch/automate/steps-c/step-03b-subagent-backend.md +246 -0
- package/templates/bmad/tea/workflows/testarch/automate/steps-c/step-03b-subagent-e2e.md +213 -0
- package/templates/bmad/tea/workflows/testarch/automate/steps-c/step-03c-aggregate.md +38 -22
- package/templates/bmad/tea/workflows/testarch/automate/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/automate/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/automate/workflow.yaml +2 -2
- package/templates/bmad/tea/workflows/testarch/ci/steps-c/step-02-generate-pipeline.md +124 -1
- package/templates/bmad/tea/workflows/testarch/ci/steps-c/step-03-configure-quality-gates.md +7 -0
- package/templates/bmad/tea/workflows/testarch/ci/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/ci/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/ci/workflow.yaml +2 -2
- package/templates/bmad/tea/workflows/testarch/framework/steps-c/step-03-scaffold-framework.md +126 -3
- package/templates/bmad/tea/workflows/testarch/framework/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/framework/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/framework/workflow.yaml +2 -2
- package/templates/bmad/tea/workflows/testarch/nfr-assess/steps-c/step-04-evaluate-and-score.md +150 -36
- package/templates/bmad/tea/workflows/testarch/nfr-assess/steps-c/step-04a-subagent-security.md +138 -0
- package/templates/bmad/tea/workflows/testarch/nfr-assess/steps-c/step-04b-subagent-performance.md +84 -0
- package/templates/bmad/tea/workflows/testarch/nfr-assess/steps-c/step-04c-subagent-reliability.md +85 -0
- package/templates/bmad/tea/workflows/testarch/nfr-assess/steps-c/step-04d-subagent-scalability.md +88 -0
- package/templates/bmad/tea/workflows/testarch/nfr-assess/steps-c/step-04e-aggregate-nfr.md +27 -10
- package/templates/bmad/tea/workflows/testarch/nfr-assess/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/nfr-assess/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/nfr-assess/workflow.yaml +2 -2
- package/templates/bmad/tea/workflows/testarch/teach-me-testing/data/tea-resources-index.yaml +3 -3
- package/templates/bmad/tea/workflows/testarch/teach-me-testing/workflow-plan-teach-me-testing.md +6 -6
- package/templates/bmad/tea/workflows/testarch/teach-me-testing/workflow.md +1 -1
- package/templates/bmad/tea/workflows/testarch/test-design/steps-c/step-02-load-context.md +30 -0
- package/templates/bmad/tea/workflows/testarch/test-design/steps-c/step-05-generate-output.md +72 -1
- package/templates/bmad/tea/workflows/testarch/test-design/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/test-design/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/test-design/workflow.yaml +2 -2
- package/templates/bmad/tea/workflows/testarch/test-review/steps-c/step-01-load-context.md +29 -1
- package/templates/bmad/tea/workflows/testarch/test-review/steps-c/step-03-quality-evaluation.md +147 -46
- package/templates/bmad/tea/workflows/testarch/test-review/steps-c/step-03a-subagent-determinism.md +214 -0
- package/templates/bmad/tea/workflows/testarch/test-review/steps-c/step-03b-subagent-isolation.md +125 -0
- package/templates/bmad/tea/workflows/testarch/test-review/steps-c/step-03c-subagent-maintainability.md +102 -0
- package/templates/bmad/tea/workflows/testarch/test-review/steps-c/step-03e-subagent-performance.md +117 -0
- package/templates/bmad/tea/workflows/testarch/test-review/steps-c/step-03f-aggregate-scores.md +10 -10
- package/templates/bmad/tea/workflows/testarch/test-review/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/test-review/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/test-review/workflow.yaml +2 -2
- package/templates/bmad/tea/workflows/testarch/trace/steps-c/step-04-analyze-gaps.md +92 -1
- package/templates/bmad/tea/workflows/testarch/trace/validation-report-20260127-095021.md +1 -1
- package/templates/bmad/tea/workflows/testarch/trace/validation-report-20260127-102401.md +3 -3
- package/templates/bmad/tea/workflows/testarch/trace/workflow.yaml +2 -2
- package/templates/ecc/agents/chief-of-staff.md +151 -0
- package/templates/ecc/commands/claw.md +79 -0
- package/templates/ecc/rules/README.md +23 -2
- package/templates/ecc/rules/common/development-workflow.md +37 -0
- package/templates/ecc/rules/common/git-workflow.md +2 -23
- package/templates/ecc/rules/swift/coding-style.md +47 -0
- package/templates/ecc/rules/swift/hooks.md +20 -0
- package/templates/ecc/rules/swift/patterns.md +66 -0
- package/templates/ecc/rules/swift/security.md +33 -0
- package/templates/ecc/rules/swift/testing.md +45 -0
- package/templates/ecc/skills/api-design/SKILL.md +1 -0
- package/templates/ecc/skills/article-writing/SKILL.md +85 -0
- package/templates/ecc/skills/backend-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/clickhouse-io/SKILL.md +1 -0
- package/templates/ecc/skills/coding-standards/SKILL.md +1 -0
- package/templates/ecc/skills/configure-ecc/SKILL.md +32 -4
- package/templates/ecc/skills/content-engine/SKILL.md +88 -0
- package/templates/ecc/skills/content-hash-cache-pattern/SKILL.md +1 -0
- package/templates/ecc/skills/continuous-learning/SKILL.md +2 -1
- package/templates/ecc/skills/continuous-learning-v2/SKILL.md +4 -1
- package/templates/ecc/skills/continuous-learning-v2/hooks/observe.sh +14 -7
- package/templates/ecc/skills/cost-aware-llm-pipeline/SKILL.md +1 -0
- package/templates/ecc/skills/cpp-coding-standards/SKILL.md +1 -0
- package/templates/ecc/skills/cpp-testing/SKILL.md +1 -0
- package/templates/ecc/skills/database-migrations/SKILL.md +1 -0
- package/templates/ecc/skills/deployment-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/django-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/django-security/SKILL.md +1 -0
- package/templates/ecc/skills/django-tdd/SKILL.md +1 -0
- package/templates/ecc/skills/django-verification/SKILL.md +1 -0
- package/templates/ecc/skills/docker-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/e2e-testing/SKILL.md +1 -0
- package/templates/ecc/skills/eval-harness/SKILL.md +1 -0
- package/templates/ecc/skills/foundation-models-on-device/SKILL.md +243 -0
- package/templates/ecc/skills/frontend-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/frontend-slides/SKILL.md +184 -0
- package/templates/ecc/skills/frontend-slides/STYLE_PRESETS.md +330 -0
- package/templates/ecc/skills/golang-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/golang-testing/SKILL.md +1 -0
- package/templates/ecc/skills/investor-materials/SKILL.md +96 -0
- package/templates/ecc/skills/investor-outreach/SKILL.md +76 -0
- package/templates/ecc/skills/iterative-retrieval/SKILL.md +1 -0
- package/templates/ecc/skills/java-coding-standards/SKILL.md +1 -0
- package/templates/ecc/skills/jpa-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/liquid-glass-design/SKILL.md +279 -0
- package/templates/ecc/skills/market-research/SKILL.md +75 -0
- package/templates/ecc/skills/nutrient-document-processing/SKILL.md +1 -1
- package/templates/ecc/skills/postgres-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/project-guidelines-example/SKILL.md +1 -0
- package/templates/ecc/skills/python-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/python-testing/SKILL.md +1 -0
- package/templates/ecc/skills/regex-vs-llm-structured-text/SKILL.md +1 -0
- package/templates/ecc/skills/search-first/SKILL.md +3 -1
- package/templates/ecc/skills/security-review/SKILL.md +1 -0
- package/templates/ecc/skills/security-scan/SKILL.md +1 -0
- package/templates/ecc/skills/skill-stocktake/SKILL.md +176 -0
- package/templates/ecc/skills/skill-stocktake/scripts/quick-diff.sh +87 -0
- package/templates/ecc/skills/skill-stocktake/scripts/save-results.sh +56 -0
- package/templates/ecc/skills/skill-stocktake/scripts/scan.sh +170 -0
- package/templates/ecc/skills/springboot-patterns/SKILL.md +1 -0
- package/templates/ecc/skills/springboot-security/SKILL.md +1 -0
- package/templates/ecc/skills/springboot-tdd/SKILL.md +1 -0
- package/templates/ecc/skills/springboot-verification/SKILL.md +1 -0
- package/templates/ecc/skills/strategic-compact/SKILL.md +1 -0
- package/templates/ecc/skills/swift-actor-persistence/SKILL.md +1 -0
- package/templates/ecc/skills/swift-concurrency-6-2/SKILL.md +216 -0
- package/templates/ecc/skills/swift-protocol-di-testing/SKILL.md +1 -0
- package/templates/ecc/skills/swiftui-patterns/SKILL.md +259 -0
- package/templates/ecc/skills/tdd-workflow/SKILL.md +1 -0
- package/templates/ecc/skills/verification-loop/SKILL.md +1 -0
- package/templates/ecc/skills/visa-doc-translate/README.md +86 -0
- package/templates/ecc/skills/visa-doc-translate/SKILL.md +117 -0
- package/templates/ext-skills/pinchtab/SKILL.md +89 -486
- package/templates/ext-skills/pinchtab/TRUST.md +69 -0
- package/templates/ext-skills/pinchtab/references/api.md +297 -0
- package/templates/ext-skills/pinchtab/references/env.md +45 -0
- package/templates/ext-skills/pinchtab/references/profiles.md +107 -0
- package/templates/plugins/claude-code-setup/.claude-plugin/plugin.json +9 -0
- package/templates/plugins/claude-code-setup/LICENSE +202 -0
- package/templates/plugins/claude-code-setup/README.md +29 -0
- package/templates/plugins/claude-code-setup/automation-recommender-example.png +0 -0
- package/templates/plugins/claude-code-setup/skills/claude-automation-recommender/SKILL.md +288 -0
- package/templates/plugins/claude-code-setup/skills/claude-automation-recommender/references/hooks-patterns.md +226 -0
- package/templates/plugins/claude-code-setup/skills/claude-automation-recommender/references/mcp-servers.md +263 -0
- package/templates/plugins/claude-code-setup/skills/claude-automation-recommender/references/plugins-reference.md +98 -0
- package/templates/plugins/claude-code-setup/skills/claude-automation-recommender/references/skills-reference.md +408 -0
- package/templates/plugins/claude-code-setup/skills/claude-automation-recommender/references/subagent-templates.md +181 -0
- package/templates/plugins/claude-md-management/.claude-plugin/plugin.json +9 -0
- package/templates/plugins/claude-md-management/LICENSE +202 -0
- package/templates/plugins/claude-md-management/README.md +40 -0
- package/templates/plugins/claude-md-management/claude-md-improver-example.png +0 -0
- package/templates/plugins/claude-md-management/commands/revise-claude-md.md +54 -0
- package/templates/plugins/claude-md-management/revise-claude-md-example.png +0 -0
- package/templates/plugins/claude-md-management/skills/claude-md-improver/SKILL.md +179 -0
- package/templates/plugins/claude-md-management/skills/claude-md-improver/references/quality-criteria.md +109 -0
- package/templates/plugins/claude-md-management/skills/claude-md-improver/references/templates.md +253 -0
- package/templates/plugins/claude-md-management/skills/claude-md-improver/references/update-guidelines.md +150 -0
- package/templates/plugins/code-simplifier/.claude-plugin/plugin.json +9 -0
- package/templates/plugins/code-simplifier/LICENSE +202 -0
- package/templates/plugins/code-simplifier/agents/code-simplifier.md +52 -0
- package/templates/plugins/commit-commands/.claude-plugin/plugin.json +9 -0
- package/templates/plugins/commit-commands/LICENSE +202 -0
- package/templates/plugins/commit-commands/README.md +225 -0
- package/templates/plugins/commit-commands/commands/clean_gone.md +53 -0
- package/templates/plugins/commit-commands/commands/commit-push-pr.md +20 -0
- package/templates/plugins/commit-commands/commands/commit.md +17 -0
- package/templates/trae-bmad/rules/bmad-bmm-check-implementation-readiness.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-code-review.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-correct-course.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-create-architecture.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-create-epics-and-stories.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-create-prd.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-create-product-brief.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-create-story.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-create-ux-design.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-dev-story.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-document-project.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-domain-research.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-edit-prd.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-generate-project-context.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-market-research.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-qa-generate-e2e-tests.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-quick-dev.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-quick-spec.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-retrospective.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-sprint-planning.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-sprint-status.md +4 -4
- package/templates/trae-bmad/rules/bmad-bmm-technical-research.md +2 -2
- package/templates/trae-bmad/rules/bmad-bmm-validate-prd.md +2 -2
- package/templates/trae-bmad/rules/bmad-brainstorming.md +2 -2
- package/templates/trae-bmad/rules/bmad-editorial-review-prose.md +1 -1
- package/templates/trae-bmad/rules/bmad-editorial-review-structure.md +1 -1
- package/templates/trae-bmad/rules/bmad-help.md +1 -1
- package/templates/trae-bmad/rules/bmad-index-docs.md +1 -1
- package/templates/trae-bmad/rules/bmad-party-mode.md +2 -2
- package/templates/trae-bmad/rules/bmad-review-adversarial-general.md +1 -1
- package/templates/trae-bmad/rules/bmad-review-edge-case-hunter.md +10 -0
- package/templates/trae-bmad/rules/bmad-shard-doc.md +1 -1
- package/templates/trae-bmad/rules/bmad-tea-teach-me-testing.md +2 -2
- package/templates/trae-bmad/rules/bmad-tea-testarch-atdd.md +4 -4
- package/templates/trae-bmad/rules/bmad-tea-testarch-automate.md +4 -4
- package/templates/trae-bmad/rules/bmad-tea-testarch-ci.md +4 -4
- package/templates/trae-bmad/rules/bmad-tea-testarch-framework.md +4 -4
- package/templates/trae-bmad/rules/bmad-tea-testarch-nfr.md +4 -4
- package/templates/trae-bmad/rules/bmad-tea-testarch-test-design.md +4 -4
- package/templates/trae-bmad/rules/bmad-tea-testarch-test-review.md +4 -4
- package/templates/trae-bmad/rules/bmad-tea-testarch-trace.md +4 -4
|
@@ -0,0 +1,87 @@
|
|
|
1
|
+
#!/usr/bin/env bash
|
|
2
|
+
# quick-diff.sh — compare skill file mtimes against results.json evaluated_at
|
|
3
|
+
# Usage: quick-diff.sh RESULTS_JSON [CWD_SKILLS_DIR]
|
|
4
|
+
# Output: JSON array of changed/new files to stdout (empty [] if no changes)
|
|
5
|
+
#
|
|
6
|
+
# When CWD_SKILLS_DIR is omitted, defaults to $PWD/.claude/skills so the
|
|
7
|
+
# script always picks up project-level skills without relying on the caller.
|
|
8
|
+
#
|
|
9
|
+
# Environment:
|
|
10
|
+
# SKILL_STOCKTAKE_GLOBAL_DIR Override ~/.claude/skills (for testing only;
|
|
11
|
+
# do not set in production — intended for bats tests)
|
|
12
|
+
# SKILL_STOCKTAKE_PROJECT_DIR Override project dir detection (for testing only)
|
|
13
|
+
|
|
14
|
+
set -euo pipefail
|
|
15
|
+
|
|
16
|
+
RESULTS_JSON="${1:-}"
|
|
17
|
+
CWD_SKILLS_DIR="${SKILL_STOCKTAKE_PROJECT_DIR:-${2:-$PWD/.claude/skills}}"
|
|
18
|
+
GLOBAL_DIR="${SKILL_STOCKTAKE_GLOBAL_DIR:-$HOME/.claude/skills}"
|
|
19
|
+
|
|
20
|
+
if [[ -z "$RESULTS_JSON" || ! -f "$RESULTS_JSON" ]]; then
|
|
21
|
+
echo "Error: RESULTS_JSON not found: ${RESULTS_JSON:-<empty>}" >&2
|
|
22
|
+
exit 1
|
|
23
|
+
fi
|
|
24
|
+
|
|
25
|
+
# Validate CWD_SKILLS_DIR looks like a .claude/skills path (defense-in-depth).
|
|
26
|
+
# Only warn when the path exists — a nonexistent path poses no traversal risk.
|
|
27
|
+
if [[ -n "$CWD_SKILLS_DIR" && -d "$CWD_SKILLS_DIR" && "$CWD_SKILLS_DIR" != */.claude/skills* ]]; then
|
|
28
|
+
echo "Warning: CWD_SKILLS_DIR does not look like a .claude/skills path: $CWD_SKILLS_DIR" >&2
|
|
29
|
+
fi
|
|
30
|
+
|
|
31
|
+
evaluated_at=$(jq -r '.evaluated_at' "$RESULTS_JSON")
|
|
32
|
+
|
|
33
|
+
# Fail fast on a missing or malformed evaluated_at rather than producing
|
|
34
|
+
# unpredictable results from ISO 8601 string comparison against "null".
|
|
35
|
+
if [[ ! "$evaluated_at" =~ ^[0-9]{4}-[0-9]{2}-[0-9]{2}T[0-9]{2}:[0-9]{2}:[0-9]{2}Z$ ]]; then
|
|
36
|
+
echo "Error: invalid or missing evaluated_at in $RESULTS_JSON: $evaluated_at" >&2
|
|
37
|
+
exit 1
|
|
38
|
+
fi
|
|
39
|
+
|
|
40
|
+
# Pre-extract known paths from results.json once (O(1) lookup per file instead of O(n*m))
|
|
41
|
+
known_paths=$(jq -r '.skills[].path' "$RESULTS_JSON" 2>/dev/null)
|
|
42
|
+
|
|
43
|
+
tmpdir=$(mktemp -d)
|
|
44
|
+
# Use a function to avoid embedding $tmpdir in a quoted string (prevents injection
|
|
45
|
+
# if TMPDIR were crafted to contain shell metacharacters).
|
|
46
|
+
_cleanup() { rm -rf "$tmpdir"; }
|
|
47
|
+
trap _cleanup EXIT
|
|
48
|
+
|
|
49
|
+
# Shared counter across process_dir calls — intentionally NOT local
|
|
50
|
+
i=0
|
|
51
|
+
|
|
52
|
+
process_dir() {
|
|
53
|
+
local dir="$1"
|
|
54
|
+
while IFS= read -r file; do
|
|
55
|
+
local mtime dp is_new
|
|
56
|
+
mtime=$(date -u -r "$file" +%Y-%m-%dT%H:%M:%SZ)
|
|
57
|
+
dp="${file/#$HOME/~}"
|
|
58
|
+
|
|
59
|
+
# Check if this file is known to results.json (exact whole-line match to
|
|
60
|
+
# avoid substring false-positives, e.g. "python-patterns" matching "python-patterns-v2").
|
|
61
|
+
if echo "$known_paths" | grep -qxF "$dp"; then
|
|
62
|
+
is_new="false"
|
|
63
|
+
# Known file: only emit if mtime changed (ISO 8601 string comparison is safe)
|
|
64
|
+
[[ "$mtime" > "$evaluated_at" ]] || continue
|
|
65
|
+
else
|
|
66
|
+
is_new="true"
|
|
67
|
+
# New file: always emit regardless of mtime
|
|
68
|
+
fi
|
|
69
|
+
|
|
70
|
+
jq -n \
|
|
71
|
+
--arg path "$dp" \
|
|
72
|
+
--arg mtime "$mtime" \
|
|
73
|
+
--argjson is_new "$is_new" \
|
|
74
|
+
'{path:$path,mtime:$mtime,is_new:$is_new}' \
|
|
75
|
+
> "$tmpdir/$i.json"
|
|
76
|
+
i=$((i+1))
|
|
77
|
+
done < <(find "$dir" -name "*.md" -type f 2>/dev/null | sort)
|
|
78
|
+
}
|
|
79
|
+
|
|
80
|
+
[[ -d "$GLOBAL_DIR" ]] && process_dir "$GLOBAL_DIR"
|
|
81
|
+
[[ -n "$CWD_SKILLS_DIR" && -d "$CWD_SKILLS_DIR" ]] && process_dir "$CWD_SKILLS_DIR"
|
|
82
|
+
|
|
83
|
+
if [[ $i -eq 0 ]]; then
|
|
84
|
+
echo "[]"
|
|
85
|
+
else
|
|
86
|
+
jq -s '.' "$tmpdir"/*.json
|
|
87
|
+
fi
|
|
@@ -0,0 +1,56 @@
|
|
|
1
|
+
#!/usr/bin/env bash
|
|
2
|
+
# save-results.sh — merge evaluated skills into results.json with correct UTC timestamp
|
|
3
|
+
# Usage: save-results.sh RESULTS_JSON <<< "$EVAL_JSON"
|
|
4
|
+
#
|
|
5
|
+
# stdin format:
|
|
6
|
+
# { "skills": {...}, "mode"?: "full"|"quick", "batch_progress"?: {...} }
|
|
7
|
+
#
|
|
8
|
+
# Always sets evaluated_at to current UTC time via `date -u`.
|
|
9
|
+
# Merges stdin .skills into existing results.json (new entries override old).
|
|
10
|
+
# Optionally updates .mode and .batch_progress if present in stdin.
|
|
11
|
+
|
|
12
|
+
set -euo pipefail
|
|
13
|
+
|
|
14
|
+
RESULTS_JSON="${1:-}"
|
|
15
|
+
|
|
16
|
+
if [[ -z "$RESULTS_JSON" ]]; then
|
|
17
|
+
echo "Error: RESULTS_JSON argument required" >&2
|
|
18
|
+
echo "Usage: save-results.sh RESULTS_JSON <<< \"\$EVAL_JSON\"" >&2
|
|
19
|
+
exit 1
|
|
20
|
+
fi
|
|
21
|
+
|
|
22
|
+
EVALUATED_AT=$(date -u +%Y-%m-%dT%H:%M:%SZ)
|
|
23
|
+
|
|
24
|
+
# Read eval results from stdin and validate JSON before touching the results file
|
|
25
|
+
input_json=$(cat)
|
|
26
|
+
if ! echo "$input_json" | jq empty 2>/dev/null; then
|
|
27
|
+
echo "Error: stdin is not valid JSON" >&2
|
|
28
|
+
exit 1
|
|
29
|
+
fi
|
|
30
|
+
|
|
31
|
+
if [[ ! -f "$RESULTS_JSON" ]]; then
|
|
32
|
+
# Bootstrap: create new results.json from stdin JSON + current UTC timestamp
|
|
33
|
+
echo "$input_json" | jq --arg ea "$EVALUATED_AT" \
|
|
34
|
+
'. + { evaluated_at: $ea }' > "$RESULTS_JSON"
|
|
35
|
+
exit 0
|
|
36
|
+
fi
|
|
37
|
+
|
|
38
|
+
# Merge: new .skills override existing ones; old skills not in input_json are kept.
|
|
39
|
+
# Optionally update .mode and .batch_progress if provided.
|
|
40
|
+
#
|
|
41
|
+
# Use mktemp for a collision-safe temp file (concurrent runs on the same RESULTS_JSON
|
|
42
|
+
# would race on a predictable ".tmp" suffix; random suffix prevents silent overwrites).
|
|
43
|
+
tmp=$(mktemp "${RESULTS_JSON}.XXXXXX")
|
|
44
|
+
trap 'rm -f "$tmp"' EXIT
|
|
45
|
+
|
|
46
|
+
jq -s \
|
|
47
|
+
--arg ea "$EVALUATED_AT" \
|
|
48
|
+
'.[0] as $existing | .[1] as $new |
|
|
49
|
+
$existing |
|
|
50
|
+
.evaluated_at = $ea |
|
|
51
|
+
.skills = ($existing.skills + ($new.skills // {})) |
|
|
52
|
+
if ($new | has("mode")) then .mode = $new.mode else . end |
|
|
53
|
+
if ($new | has("batch_progress")) then .batch_progress = $new.batch_progress else . end' \
|
|
54
|
+
"$RESULTS_JSON" <(echo "$input_json") > "$tmp"
|
|
55
|
+
|
|
56
|
+
mv "$tmp" "$RESULTS_JSON"
|
|
@@ -0,0 +1,170 @@
|
|
|
1
|
+
#!/usr/bin/env bash
|
|
2
|
+
# scan.sh — enumerate skill files, extract frontmatter and UTC mtime
|
|
3
|
+
# Usage: scan.sh [CWD_SKILLS_DIR]
|
|
4
|
+
# Output: JSON to stdout
|
|
5
|
+
#
|
|
6
|
+
# When CWD_SKILLS_DIR is omitted, defaults to $PWD/.claude/skills so the
|
|
7
|
+
# script always picks up project-level skills without relying on the caller.
|
|
8
|
+
#
|
|
9
|
+
# Environment:
|
|
10
|
+
# SKILL_STOCKTAKE_GLOBAL_DIR Override ~/.claude/skills (for testing only;
|
|
11
|
+
# do not set in production — intended for bats tests)
|
|
12
|
+
# SKILL_STOCKTAKE_PROJECT_DIR Override project dir detection (for testing only)
|
|
13
|
+
|
|
14
|
+
set -euo pipefail
|
|
15
|
+
|
|
16
|
+
GLOBAL_DIR="${SKILL_STOCKTAKE_GLOBAL_DIR:-$HOME/.claude/skills}"
|
|
17
|
+
CWD_SKILLS_DIR="${SKILL_STOCKTAKE_PROJECT_DIR:-${1:-$PWD/.claude/skills}}"
|
|
18
|
+
# Path to JSONL file containing tool-use observations (optional; used for usage frequency counts).
|
|
19
|
+
# Override via SKILL_STOCKTAKE_OBSERVATIONS env var if your setup uses a different path.
|
|
20
|
+
OBSERVATIONS="${SKILL_STOCKTAKE_OBSERVATIONS:-$HOME/.claude/observations.jsonl}"
|
|
21
|
+
|
|
22
|
+
# Validate CWD_SKILLS_DIR looks like a .claude/skills path (defense-in-depth).
|
|
23
|
+
# Only warn when the path exists — a nonexistent path poses no traversal risk.
|
|
24
|
+
if [[ -n "$CWD_SKILLS_DIR" && -d "$CWD_SKILLS_DIR" && "$CWD_SKILLS_DIR" != */.claude/skills* ]]; then
|
|
25
|
+
echo "Warning: CWD_SKILLS_DIR does not look like a .claude/skills path: $CWD_SKILLS_DIR" >&2
|
|
26
|
+
fi
|
|
27
|
+
|
|
28
|
+
# Extract a frontmatter field (handles both quoted and unquoted single-line values).
|
|
29
|
+
# Does NOT support multi-line YAML blocks (| or >) or nested YAML keys.
|
|
30
|
+
extract_field() {
|
|
31
|
+
local file="$1" field="$2"
|
|
32
|
+
awk -v f="$field" '
|
|
33
|
+
BEGIN { fm=0 }
|
|
34
|
+
/^---$/ { fm++; next }
|
|
35
|
+
fm==1 {
|
|
36
|
+
n = length(f) + 2
|
|
37
|
+
if (substr($0, 1, n) == f ": ") {
|
|
38
|
+
val = substr($0, n+1)
|
|
39
|
+
gsub(/^"/, "", val)
|
|
40
|
+
gsub(/"$/, "", val)
|
|
41
|
+
print val
|
|
42
|
+
exit
|
|
43
|
+
}
|
|
44
|
+
}
|
|
45
|
+
fm>=2 { exit }
|
|
46
|
+
' "$file"
|
|
47
|
+
}
|
|
48
|
+
|
|
49
|
+
# Get UTC timestamp N days ago (supports both macOS and GNU date)
|
|
50
|
+
date_ago() {
|
|
51
|
+
local n="$1"
|
|
52
|
+
date -u -v-"${n}d" +%Y-%m-%dT%H:%M:%SZ 2>/dev/null ||
|
|
53
|
+
date -u -d "${n} days ago" +%Y-%m-%dT%H:%M:%SZ
|
|
54
|
+
}
|
|
55
|
+
|
|
56
|
+
# Count observations matching a file path since a cutoff timestamp
|
|
57
|
+
count_obs() {
|
|
58
|
+
local file="$1" cutoff="$2"
|
|
59
|
+
if [[ ! -f "$OBSERVATIONS" ]]; then
|
|
60
|
+
echo 0
|
|
61
|
+
return
|
|
62
|
+
fi
|
|
63
|
+
jq -r --arg p "$file" --arg c "$cutoff" \
|
|
64
|
+
'select(.tool=="Read" and .path==$p and .timestamp>=$c) | 1' \
|
|
65
|
+
"$OBSERVATIONS" 2>/dev/null | wc -l | tr -d ' '
|
|
66
|
+
}
|
|
67
|
+
|
|
68
|
+
# Scan a directory and produce a JSON array of skill objects
|
|
69
|
+
scan_dir_to_json() {
|
|
70
|
+
local dir="$1"
|
|
71
|
+
local c7 c30
|
|
72
|
+
c7=$(date_ago 7)
|
|
73
|
+
c30=$(date_ago 30)
|
|
74
|
+
|
|
75
|
+
local tmpdir
|
|
76
|
+
tmpdir=$(mktemp -d)
|
|
77
|
+
# Use a function to avoid embedding $tmpdir in a quoted string (prevents injection
|
|
78
|
+
# if TMPDIR were crafted to contain shell metacharacters).
|
|
79
|
+
local _scan_tmpdir="$tmpdir"
|
|
80
|
+
_scan_cleanup() { rm -rf "$_scan_tmpdir"; }
|
|
81
|
+
trap _scan_cleanup RETURN
|
|
82
|
+
|
|
83
|
+
# Pre-aggregate observation counts in two passes (one per window) instead of
|
|
84
|
+
# calling jq per-file — reduces from O(n*m) to O(n+m) jq invocations.
|
|
85
|
+
local obs_7d_counts obs_30d_counts
|
|
86
|
+
obs_7d_counts=""
|
|
87
|
+
obs_30d_counts=""
|
|
88
|
+
if [[ -f "$OBSERVATIONS" ]]; then
|
|
89
|
+
obs_7d_counts=$(jq -r --arg c "$c7" \
|
|
90
|
+
'select(.tool=="Read" and .timestamp>=$c) | .path' \
|
|
91
|
+
"$OBSERVATIONS" 2>/dev/null | sort | uniq -c)
|
|
92
|
+
obs_30d_counts=$(jq -r --arg c "$c30" \
|
|
93
|
+
'select(.tool=="Read" and .timestamp>=$c) | .path' \
|
|
94
|
+
"$OBSERVATIONS" 2>/dev/null | sort | uniq -c)
|
|
95
|
+
fi
|
|
96
|
+
|
|
97
|
+
local i=0
|
|
98
|
+
while IFS= read -r file; do
|
|
99
|
+
local name desc mtime u7 u30 dp
|
|
100
|
+
name=$(extract_field "$file" "name")
|
|
101
|
+
desc=$(extract_field "$file" "description")
|
|
102
|
+
mtime=$(date -u -r "$file" +%Y-%m-%dT%H:%M:%SZ)
|
|
103
|
+
# Use awk exact field match to avoid substring false-positives from grep -F.
|
|
104
|
+
# uniq -c output format: " N /path/to/file" — path is always field 2.
|
|
105
|
+
u7=$(echo "$obs_7d_counts" | awk -v f="$file" '$2 == f {print $1}' | head -1)
|
|
106
|
+
u7="${u7:-0}"
|
|
107
|
+
u30=$(echo "$obs_30d_counts" | awk -v f="$file" '$2 == f {print $1}' | head -1)
|
|
108
|
+
u30="${u30:-0}"
|
|
109
|
+
dp="${file/#$HOME/~}"
|
|
110
|
+
|
|
111
|
+
jq -n \
|
|
112
|
+
--arg path "$dp" \
|
|
113
|
+
--arg name "$name" \
|
|
114
|
+
--arg description "$desc" \
|
|
115
|
+
--arg mtime "$mtime" \
|
|
116
|
+
--argjson use_7d "$u7" \
|
|
117
|
+
--argjson use_30d "$u30" \
|
|
118
|
+
'{path:$path,name:$name,description:$description,use_7d:$use_7d,use_30d:$use_30d,mtime:$mtime}' \
|
|
119
|
+
> "$tmpdir/$i.json"
|
|
120
|
+
i=$((i+1))
|
|
121
|
+
done < <(find "$dir" -name "*.md" -type f 2>/dev/null | sort)
|
|
122
|
+
|
|
123
|
+
if [[ $i -eq 0 ]]; then
|
|
124
|
+
echo "[]"
|
|
125
|
+
else
|
|
126
|
+
jq -s '.' "$tmpdir"/*.json
|
|
127
|
+
fi
|
|
128
|
+
}
|
|
129
|
+
|
|
130
|
+
# --- Main ---
|
|
131
|
+
|
|
132
|
+
global_found="false"
|
|
133
|
+
global_count=0
|
|
134
|
+
global_skills="[]"
|
|
135
|
+
|
|
136
|
+
if [[ -d "$GLOBAL_DIR" ]]; then
|
|
137
|
+
global_found="true"
|
|
138
|
+
global_skills=$(scan_dir_to_json "$GLOBAL_DIR")
|
|
139
|
+
global_count=$(echo "$global_skills" | jq 'length')
|
|
140
|
+
fi
|
|
141
|
+
|
|
142
|
+
project_found="false"
|
|
143
|
+
project_path=""
|
|
144
|
+
project_count=0
|
|
145
|
+
project_skills="[]"
|
|
146
|
+
|
|
147
|
+
if [[ -n "$CWD_SKILLS_DIR" && -d "$CWD_SKILLS_DIR" ]]; then
|
|
148
|
+
project_found="true"
|
|
149
|
+
project_path="$CWD_SKILLS_DIR"
|
|
150
|
+
project_skills=$(scan_dir_to_json "$CWD_SKILLS_DIR")
|
|
151
|
+
project_count=$(echo "$project_skills" | jq 'length')
|
|
152
|
+
fi
|
|
153
|
+
|
|
154
|
+
# Merge global + project skills into one array
|
|
155
|
+
all_skills=$(jq -s 'add' <(echo "$global_skills") <(echo "$project_skills"))
|
|
156
|
+
|
|
157
|
+
jq -n \
|
|
158
|
+
--arg global_found "$global_found" \
|
|
159
|
+
--argjson global_count "$global_count" \
|
|
160
|
+
--arg project_found "$project_found" \
|
|
161
|
+
--arg project_path "$project_path" \
|
|
162
|
+
--argjson project_count "$project_count" \
|
|
163
|
+
--argjson skills "$all_skills" \
|
|
164
|
+
'{
|
|
165
|
+
scan_summary: {
|
|
166
|
+
global: { found: ($global_found == "true"), count: $global_count },
|
|
167
|
+
project: { found: ($project_found == "true"), path: $project_path, count: $project_count }
|
|
168
|
+
},
|
|
169
|
+
skills: $skills
|
|
170
|
+
}'
|
|
@@ -0,0 +1,216 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: swift-concurrency-6-2
|
|
3
|
+
description: Swift 6.2 Approachable Concurrency — single-threaded by default, @concurrent for explicit background offloading, isolated conformances for main actor types.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Swift 6.2 Approachable Concurrency
|
|
7
|
+
|
|
8
|
+
Patterns for adopting Swift 6.2's concurrency model where code runs single-threaded by default and concurrency is introduced explicitly. Eliminates common data-race errors without sacrificing performance.
|
|
9
|
+
|
|
10
|
+
## When to Activate
|
|
11
|
+
|
|
12
|
+
- Migrating Swift 5.x or 6.0/6.1 projects to Swift 6.2
|
|
13
|
+
- Resolving data-race safety compiler errors
|
|
14
|
+
- Designing MainActor-based app architecture
|
|
15
|
+
- Offloading CPU-intensive work to background threads
|
|
16
|
+
- Implementing protocol conformances on MainActor-isolated types
|
|
17
|
+
- Enabling Approachable Concurrency build settings in Xcode 26
|
|
18
|
+
|
|
19
|
+
## Core Problem: Implicit Background Offloading
|
|
20
|
+
|
|
21
|
+
In Swift 6.1 and earlier, async functions could be implicitly offloaded to background threads, causing data-race errors even in seemingly safe code:
|
|
22
|
+
|
|
23
|
+
```swift
|
|
24
|
+
// Swift 6.1: ERROR
|
|
25
|
+
@MainActor
|
|
26
|
+
final class StickerModel {
|
|
27
|
+
let photoProcessor = PhotoProcessor()
|
|
28
|
+
|
|
29
|
+
func extractSticker(_ item: PhotosPickerItem) async throws -> Sticker? {
|
|
30
|
+
guard let data = try await item.loadTransferable(type: Data.self) else { return nil }
|
|
31
|
+
|
|
32
|
+
// Error: Sending 'self.photoProcessor' risks causing data races
|
|
33
|
+
return await photoProcessor.extractSticker(data: data, with: item.itemIdentifier)
|
|
34
|
+
}
|
|
35
|
+
}
|
|
36
|
+
```
|
|
37
|
+
|
|
38
|
+
Swift 6.2 fixes this: async functions stay on the calling actor by default.
|
|
39
|
+
|
|
40
|
+
```swift
|
|
41
|
+
// Swift 6.2: OK — async stays on MainActor, no data race
|
|
42
|
+
@MainActor
|
|
43
|
+
final class StickerModel {
|
|
44
|
+
let photoProcessor = PhotoProcessor()
|
|
45
|
+
|
|
46
|
+
func extractSticker(_ item: PhotosPickerItem) async throws -> Sticker? {
|
|
47
|
+
guard let data = try await item.loadTransferable(type: Data.self) else { return nil }
|
|
48
|
+
return await photoProcessor.extractSticker(data: data, with: item.itemIdentifier)
|
|
49
|
+
}
|
|
50
|
+
}
|
|
51
|
+
```
|
|
52
|
+
|
|
53
|
+
## Core Pattern — Isolated Conformances
|
|
54
|
+
|
|
55
|
+
MainActor types can now conform to non-isolated protocols safely:
|
|
56
|
+
|
|
57
|
+
```swift
|
|
58
|
+
protocol Exportable {
|
|
59
|
+
func export()
|
|
60
|
+
}
|
|
61
|
+
|
|
62
|
+
// Swift 6.1: ERROR — crosses into main actor-isolated code
|
|
63
|
+
// Swift 6.2: OK with isolated conformance
|
|
64
|
+
extension StickerModel: @MainActor Exportable {
|
|
65
|
+
func export() {
|
|
66
|
+
photoProcessor.exportAsPNG()
|
|
67
|
+
}
|
|
68
|
+
}
|
|
69
|
+
```
|
|
70
|
+
|
|
71
|
+
The compiler ensures the conformance is only used on the main actor:
|
|
72
|
+
|
|
73
|
+
```swift
|
|
74
|
+
// OK — ImageExporter is also @MainActor
|
|
75
|
+
@MainActor
|
|
76
|
+
struct ImageExporter {
|
|
77
|
+
var items: [any Exportable]
|
|
78
|
+
|
|
79
|
+
mutating func add(_ item: StickerModel) {
|
|
80
|
+
items.append(item) // Safe: same actor isolation
|
|
81
|
+
}
|
|
82
|
+
}
|
|
83
|
+
|
|
84
|
+
// ERROR — nonisolated context can't use MainActor conformance
|
|
85
|
+
nonisolated struct ImageExporter {
|
|
86
|
+
var items: [any Exportable]
|
|
87
|
+
|
|
88
|
+
mutating func add(_ item: StickerModel) {
|
|
89
|
+
items.append(item) // Error: Main actor-isolated conformance cannot be used here
|
|
90
|
+
}
|
|
91
|
+
}
|
|
92
|
+
```
|
|
93
|
+
|
|
94
|
+
## Core Pattern — Global and Static Variables
|
|
95
|
+
|
|
96
|
+
Protect global/static state with MainActor:
|
|
97
|
+
|
|
98
|
+
```swift
|
|
99
|
+
// Swift 6.1: ERROR — non-Sendable type may have shared mutable state
|
|
100
|
+
final class StickerLibrary {
|
|
101
|
+
static let shared: StickerLibrary = .init() // Error
|
|
102
|
+
}
|
|
103
|
+
|
|
104
|
+
// Fix: Annotate with @MainActor
|
|
105
|
+
@MainActor
|
|
106
|
+
final class StickerLibrary {
|
|
107
|
+
static let shared: StickerLibrary = .init() // OK
|
|
108
|
+
}
|
|
109
|
+
```
|
|
110
|
+
|
|
111
|
+
### MainActor Default Inference Mode
|
|
112
|
+
|
|
113
|
+
Swift 6.2 introduces a mode where MainActor is inferred by default — no manual annotations needed:
|
|
114
|
+
|
|
115
|
+
```swift
|
|
116
|
+
// With MainActor default inference enabled:
|
|
117
|
+
final class StickerLibrary {
|
|
118
|
+
static let shared: StickerLibrary = .init() // Implicitly @MainActor
|
|
119
|
+
}
|
|
120
|
+
|
|
121
|
+
final class StickerModel {
|
|
122
|
+
let photoProcessor: PhotoProcessor
|
|
123
|
+
var selection: [PhotosPickerItem] // Implicitly @MainActor
|
|
124
|
+
}
|
|
125
|
+
|
|
126
|
+
extension StickerModel: Exportable { // Implicitly @MainActor conformance
|
|
127
|
+
func export() {
|
|
128
|
+
photoProcessor.exportAsPNG()
|
|
129
|
+
}
|
|
130
|
+
}
|
|
131
|
+
```
|
|
132
|
+
|
|
133
|
+
This mode is opt-in and recommended for apps, scripts, and other executable targets.
|
|
134
|
+
|
|
135
|
+
## Core Pattern — @concurrent for Background Work
|
|
136
|
+
|
|
137
|
+
When you need actual parallelism, explicitly offload with `@concurrent`:
|
|
138
|
+
|
|
139
|
+
> **Important:** This example requires Approachable Concurrency build settings — SE-0466 (MainActor default isolation) and SE-0461 (NonisolatedNonsendingByDefault). With these enabled, `extractSticker` stays on the caller's actor, making mutable state access safe. **Without these settings, this code has a data race** — the compiler will flag it.
|
|
140
|
+
|
|
141
|
+
```swift
|
|
142
|
+
nonisolated final class PhotoProcessor {
|
|
143
|
+
private var cachedStickers: [String: Sticker] = [:]
|
|
144
|
+
|
|
145
|
+
func extractSticker(data: Data, with id: String) async -> Sticker {
|
|
146
|
+
if let sticker = cachedStickers[id] {
|
|
147
|
+
return sticker
|
|
148
|
+
}
|
|
149
|
+
|
|
150
|
+
let sticker = await Self.extractSubject(from: data)
|
|
151
|
+
cachedStickers[id] = sticker
|
|
152
|
+
return sticker
|
|
153
|
+
}
|
|
154
|
+
|
|
155
|
+
// Offload expensive work to concurrent thread pool
|
|
156
|
+
@concurrent
|
|
157
|
+
static func extractSubject(from data: Data) async -> Sticker { /* ... */ }
|
|
158
|
+
}
|
|
159
|
+
|
|
160
|
+
// Callers must await
|
|
161
|
+
let processor = PhotoProcessor()
|
|
162
|
+
processedPhotos[item.id] = await processor.extractSticker(data: data, with: item.id)
|
|
163
|
+
```
|
|
164
|
+
|
|
165
|
+
To use `@concurrent`:
|
|
166
|
+
1. Mark the containing type as `nonisolated`
|
|
167
|
+
2. Add `@concurrent` to the function
|
|
168
|
+
3. Add `async` if not already asynchronous
|
|
169
|
+
4. Add `await` at call sites
|
|
170
|
+
|
|
171
|
+
## Key Design Decisions
|
|
172
|
+
|
|
173
|
+
| Decision | Rationale |
|
|
174
|
+
|----------|-----------|
|
|
175
|
+
| Single-threaded by default | Most natural code is data-race free; concurrency is opt-in |
|
|
176
|
+
| Async stays on calling actor | Eliminates implicit offloading that caused data-race errors |
|
|
177
|
+
| Isolated conformances | MainActor types can conform to protocols without unsafe workarounds |
|
|
178
|
+
| `@concurrent` explicit opt-in | Background execution is a deliberate performance choice, not accidental |
|
|
179
|
+
| MainActor default inference | Reduces boilerplate `@MainActor` annotations for app targets |
|
|
180
|
+
| Opt-in adoption | Non-breaking migration path — enable features incrementally |
|
|
181
|
+
|
|
182
|
+
## Migration Steps
|
|
183
|
+
|
|
184
|
+
1. **Enable in Xcode**: Swift Compiler > Concurrency section in Build Settings
|
|
185
|
+
2. **Enable in SPM**: Use `SwiftSettings` API in package manifest
|
|
186
|
+
3. **Use migration tooling**: Automatic code changes via swift.org/migration
|
|
187
|
+
4. **Start with MainActor defaults**: Enable inference mode for app targets
|
|
188
|
+
5. **Add `@concurrent` where needed**: Profile first, then offload hot paths
|
|
189
|
+
6. **Test thoroughly**: Data-race issues become compile-time errors
|
|
190
|
+
|
|
191
|
+
## Best Practices
|
|
192
|
+
|
|
193
|
+
- **Start on MainActor** — write single-threaded code first, optimize later
|
|
194
|
+
- **Use `@concurrent` only for CPU-intensive work** — image processing, compression, complex computation
|
|
195
|
+
- **Enable MainActor inference mode** for app targets that are mostly single-threaded
|
|
196
|
+
- **Profile before offloading** — use Instruments to find actual bottlenecks
|
|
197
|
+
- **Protect globals with MainActor** — global/static mutable state needs actor isolation
|
|
198
|
+
- **Use isolated conformances** instead of `nonisolated` workarounds or `@Sendable` wrappers
|
|
199
|
+
- **Migrate incrementally** — enable features one at a time in build settings
|
|
200
|
+
|
|
201
|
+
## Anti-Patterns to Avoid
|
|
202
|
+
|
|
203
|
+
- Applying `@concurrent` to every async function (most don't need background execution)
|
|
204
|
+
- Using `nonisolated` to suppress compiler errors without understanding isolation
|
|
205
|
+
- Keeping legacy `DispatchQueue` patterns when actors provide the same safety
|
|
206
|
+
- Skipping `model.availability` checks in concurrency-related Foundation Models code
|
|
207
|
+
- Fighting the compiler — if it reports a data race, the code has a real concurrency issue
|
|
208
|
+
- Assuming all async code runs in the background (Swift 6.2 default: stays on calling actor)
|
|
209
|
+
|
|
210
|
+
## When to Use
|
|
211
|
+
|
|
212
|
+
- All new Swift 6.2+ projects (Approachable Concurrency is the recommended default)
|
|
213
|
+
- Migrating existing apps from Swift 5.x or 6.0/6.1 concurrency
|
|
214
|
+
- Resolving data-race safety compiler errors during Xcode 26 adoption
|
|
215
|
+
- Building MainActor-centric app architectures (most UI apps)
|
|
216
|
+
- Performance optimization — offloading specific heavy computations to background
|
|
@@ -1,6 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: swift-protocol-di-testing
|
|
3
3
|
description: Protocol-based dependency injection for testable Swift code — mock file system, network, and external APIs using focused protocols and Swift Testing.
|
|
4
|
+
origin: ECC
|
|
4
5
|
---
|
|
5
6
|
|
|
6
7
|
# Swift Protocol-Based Dependency Injection for Testing
|