takt 0.45.0 → 0.47.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 +55 -2
- package/builtins/en/config.yaml +37 -12
- package/builtins/en/facets/instructions/findings-manager.md +13 -0
- package/builtins/en/facets/instructions/fix.md +9 -4
- package/builtins/en/facets/instructions/implement-maintenance.md +5 -32
- package/builtins/en/facets/instructions/implement.md +3 -2
- package/builtins/en/facets/instructions/loop-monitor-reviewers-fix.md +5 -1
- package/builtins/en/facets/instructions/plan-test.md +4 -3
- package/builtins/en/facets/instructions/review-arch.md +1 -1
- package/builtins/en/facets/instructions/review-coding.md +11 -5
- package/builtins/en/facets/instructions/review-cqrs-es.md +1 -1
- package/builtins/en/facets/instructions/review-pure.md +9 -0
- package/builtins/en/facets/instructions/review-qa.md +2 -0
- package/builtins/en/facets/instructions/review-terraform.md +1 -1
- package/builtins/en/facets/instructions/review-test.md +3 -0
- package/builtins/en/facets/instructions/supervise-maintenance.md +7 -77
- package/builtins/en/facets/instructions/supervise.md +12 -65
- package/builtins/en/facets/instructions/write-tests-first.md +1 -0
- package/builtins/en/facets/instructions/write-tests-maintenance.md +1 -0
- package/builtins/en/facets/knowledge/architecture.md +31 -0
- package/builtins/en/facets/knowledge/backend.md +25 -1
- package/builtins/en/facets/knowledge/cqrs-es.md +57 -0
- package/builtins/en/facets/knowledge/e2e-testing.md +11 -1
- package/builtins/en/facets/knowledge/unit-testing.md +11 -0
- package/builtins/en/facets/output-contracts/ai-antipattern-review-finding-contract.md +31 -0
- package/builtins/en/facets/output-contracts/architecture-review-finding-contract.md +35 -0
- package/builtins/en/facets/output-contracts/coding-review-finding-contract.md +25 -0
- package/builtins/en/facets/output-contracts/coding-review.md +11 -4
- package/builtins/en/facets/output-contracts/cqrs-es-review-finding-contract.md +31 -0
- package/builtins/en/facets/output-contracts/findings-manager.md +45 -0
- package/builtins/en/facets/output-contracts/frontend-review-finding-contract.md +30 -0
- package/builtins/en/facets/output-contracts/maintenance-scope.md +5 -0
- package/builtins/en/facets/output-contracts/pure-review-finding-contract.md +45 -0
- package/builtins/en/facets/output-contracts/pure-review.md +61 -0
- package/builtins/en/facets/output-contracts/qa-review-finding-contract.md +35 -0
- package/builtins/en/facets/output-contracts/qa-review.md +15 -3
- package/builtins/en/facets/output-contracts/security-review-finding-contract.md +36 -0
- package/builtins/en/facets/output-contracts/supervisor-validation.md +35 -11
- package/builtins/en/facets/output-contracts/terraform-review-finding-contract.md +31 -0
- package/builtins/en/facets/output-contracts/testing-review-finding-contract.md +41 -0
- package/builtins/en/facets/output-contracts/testing-review.md +6 -1
- package/builtins/en/facets/personas/ai-antipattern-reviewer.md +2 -2
- package/builtins/en/facets/personas/architecture-reviewer.md +1 -1
- package/builtins/en/facets/personas/findings-manager.md +18 -0
- package/builtins/en/facets/personas/pr-commenter.md +2 -2
- package/builtins/en/facets/personas/pure-reviewer.md +22 -0
- package/builtins/en/facets/personas/supervisor.md +1 -1
- package/builtins/en/facets/personas/terraform-reviewer.md +3 -3
- package/builtins/en/facets/policies/ai-antipattern.md +33 -0
- package/builtins/en/facets/policies/coding.md +36 -0
- package/builtins/en/facets/policies/review.md +90 -17
- package/builtins/en/facets/policies/testing.md +49 -0
- package/builtins/en/provider-options/edit.yaml +21 -0
- package/builtins/en/provider-options/review-files.yaml +10 -0
- package/builtins/en/provider-options/review-readonly.yaml +16 -0
- package/builtins/en/provider-options/review-web.yaml +14 -0
- package/builtins/en/workflow-categories.yaml +1 -0
- package/builtins/en/workflows/audit-architecture-backend.yaml +8 -0
- package/builtins/en/workflows/audit-architecture-dual.yaml +8 -0
- package/builtins/en/workflows/audit-architecture-frontend.yaml +8 -0
- package/builtins/en/workflows/audit-architecture.yaml +8 -0
- package/builtins/en/workflows/audit-e2e.yaml +8 -0
- package/builtins/en/workflows/audit-security.yaml +8 -0
- package/builtins/en/workflows/audit-unit.yaml +8 -0
- package/builtins/en/workflows/auto-improvement-loop.yaml +6 -0
- package/builtins/en/workflows/backend-cqrs-mini.yaml +20 -0
- package/builtins/en/workflows/backend-cqrs.yaml +60 -1
- package/builtins/en/workflows/backend-mini.yaml +20 -0
- package/builtins/en/workflows/backend.yaml +60 -1
- package/builtins/en/workflows/compound-eye.yaml +8 -0
- package/builtins/en/workflows/deep-research.yaml +8 -0
- package/builtins/en/workflows/default-draft.yaml +6 -0
- package/builtins/en/workflows/default-high.yaml +7 -1
- package/builtins/en/workflows/default-mini.yaml +2 -0
- package/builtins/en/workflows/default-peer-review.yaml +41 -4
- package/builtins/en/workflows/default.yaml +4 -0
- package/builtins/en/workflows/draft.yaml +12 -32
- package/builtins/en/workflows/dual-cqrs-mini.yaml +20 -0
- package/builtins/en/workflows/dual-cqrs.yaml +56 -1
- package/builtins/en/workflows/dual-mini.yaml +20 -0
- package/builtins/en/workflows/dual.yaml +44 -6
- package/builtins/en/workflows/frontend-maintenance.yaml +62 -1
- package/builtins/en/workflows/frontend-mini.yaml +20 -0
- package/builtins/en/workflows/frontend.yaml +62 -1
- package/builtins/en/workflows/magi.yaml +6 -0
- package/builtins/en/workflows/peer-review-with-fc.yaml +206 -0
- package/builtins/en/workflows/peer-review.yaml +35 -72
- package/builtins/en/workflows/research.yaml +6 -0
- package/builtins/en/workflows/review-backend-cqrs.yaml +42 -1
- package/builtins/en/workflows/review-backend.yaml +40 -1
- package/builtins/en/workflows/review-default.yaml +27 -7
- package/builtins/en/workflows/review-dual-cqrs.yaml +44 -1
- package/builtins/en/workflows/review-dual.yaml +42 -1
- package/builtins/en/workflows/review-fix-backend-cqrs.yaml +46 -1
- package/builtins/en/workflows/review-fix-backend.yaml +44 -1
- package/builtins/en/workflows/review-fix-default.yaml +31 -7
- package/builtins/en/workflows/review-fix-dual-cqrs.yaml +48 -1
- package/builtins/en/workflows/review-fix-dual.yaml +46 -1
- package/builtins/en/workflows/review-fix-frontend.yaml +46 -1
- package/builtins/en/workflows/review-fix-takt-default.yaml +38 -6
- package/builtins/en/workflows/review-frontend.yaml +42 -1
- package/builtins/en/workflows/review-takt-default.yaml +29 -7
- package/builtins/en/workflows/takt-default-refresh-all.yaml +36 -6
- package/builtins/en/workflows/takt-default-refresh-fast.yaml +35 -5
- package/builtins/en/workflows/takt-default-with-fc.yaml +110 -0
- package/builtins/en/workflows/takt-default.yaml +10 -27
- package/builtins/en/workflows/terraform.yaml +52 -3
- package/builtins/ja/config.yaml +37 -12
- package/builtins/ja/facets/instructions/findings-manager.md +13 -0
- package/builtins/ja/facets/instructions/fix.md +9 -4
- package/builtins/ja/facets/instructions/implement-maintenance.md +5 -32
- package/builtins/ja/facets/instructions/implement.md +3 -2
- package/builtins/ja/facets/instructions/loop-monitor-reviewers-fix.md +5 -1
- package/builtins/ja/facets/instructions/plan-test.md +4 -3
- package/builtins/ja/facets/instructions/review-arch.md +1 -1
- package/builtins/ja/facets/instructions/review-coding.md +11 -5
- package/builtins/ja/facets/instructions/review-cqrs-es.md +1 -1
- package/builtins/ja/facets/instructions/review-pure.md +9 -0
- package/builtins/ja/facets/instructions/review-qa.md +2 -0
- package/builtins/ja/facets/instructions/review-terraform.md +1 -1
- package/builtins/ja/facets/instructions/review-test.md +3 -0
- package/builtins/ja/facets/instructions/supervise-maintenance.md +7 -77
- package/builtins/ja/facets/instructions/supervise.md +12 -66
- package/builtins/ja/facets/instructions/write-tests-first.md +1 -0
- package/builtins/ja/facets/instructions/write-tests-maintenance.md +1 -0
- package/builtins/ja/facets/knowledge/architecture.md +31 -0
- package/builtins/ja/facets/knowledge/backend.md +25 -1
- package/builtins/ja/facets/knowledge/cqrs-es.md +31 -0
- package/builtins/ja/facets/knowledge/e2e-testing.md +11 -1
- package/builtins/ja/facets/knowledge/unit-testing.md +11 -0
- package/builtins/ja/facets/output-contracts/ai-antipattern-review-finding-contract.md +31 -0
- package/builtins/ja/facets/output-contracts/architecture-review-finding-contract.md +35 -0
- package/builtins/ja/facets/output-contracts/coding-review-finding-contract.md +25 -0
- package/builtins/ja/facets/output-contracts/coding-review.md +11 -4
- package/builtins/ja/facets/output-contracts/cqrs-es-review-finding-contract.md +31 -0
- package/builtins/ja/facets/output-contracts/findings-manager.md +45 -0
- package/builtins/ja/facets/output-contracts/frontend-review-finding-contract.md +30 -0
- package/builtins/ja/facets/output-contracts/maintenance-scope.md +5 -0
- package/builtins/ja/facets/output-contracts/pure-review-finding-contract.md +45 -0
- package/builtins/ja/facets/output-contracts/pure-review.md +61 -0
- package/builtins/ja/facets/output-contracts/qa-review-finding-contract.md +35 -0
- package/builtins/ja/facets/output-contracts/qa-review.md +15 -3
- package/builtins/ja/facets/output-contracts/security-review-finding-contract.md +36 -0
- package/builtins/ja/facets/output-contracts/supervisor-validation.md +35 -11
- package/builtins/ja/facets/output-contracts/terraform-review-finding-contract.md +31 -0
- package/builtins/ja/facets/output-contracts/testing-review-finding-contract.md +41 -0
- package/builtins/ja/facets/output-contracts/testing-review.md +6 -1
- package/builtins/ja/facets/personas/ai-antipattern-reviewer.md +2 -2
- package/builtins/ja/facets/personas/architecture-reviewer.md +2 -2
- package/builtins/ja/facets/personas/findings-manager.md +18 -0
- package/builtins/ja/facets/personas/pr-commenter.md +3 -3
- package/builtins/ja/facets/personas/pure-reviewer.md +22 -0
- package/builtins/ja/facets/personas/supervisor.md +1 -1
- package/builtins/ja/facets/personas/terraform-reviewer.md +3 -3
- package/builtins/ja/facets/policies/ai-antipattern.md +17 -0
- package/builtins/ja/facets/policies/coding.md +36 -0
- package/builtins/ja/facets/policies/review.md +88 -17
- package/builtins/ja/facets/policies/testing.md +49 -0
- package/builtins/ja/provider-options/edit.yaml +21 -0
- package/builtins/ja/provider-options/review-files.yaml +10 -0
- package/builtins/ja/provider-options/review-readonly.yaml +16 -0
- package/builtins/ja/provider-options/review-web.yaml +14 -0
- package/builtins/ja/workflow-categories.yaml +1 -0
- package/builtins/ja/workflows/audit-architecture-backend.yaml +8 -0
- package/builtins/ja/workflows/audit-architecture-dual.yaml +8 -0
- package/builtins/ja/workflows/audit-architecture-frontend.yaml +8 -0
- package/builtins/ja/workflows/audit-architecture.yaml +8 -0
- package/builtins/ja/workflows/audit-e2e.yaml +8 -0
- package/builtins/ja/workflows/audit-security.yaml +8 -0
- package/builtins/ja/workflows/audit-unit.yaml +8 -0
- package/builtins/ja/workflows/auto-improvement-loop.yaml +6 -0
- package/builtins/ja/workflows/backend-cqrs-mini.yaml +20 -0
- package/builtins/ja/workflows/backend-cqrs.yaml +60 -1
- package/builtins/ja/workflows/backend-mini.yaml +20 -0
- package/builtins/ja/workflows/backend.yaml +60 -1
- package/builtins/ja/workflows/compound-eye.yaml +8 -0
- package/builtins/ja/workflows/deep-research.yaml +8 -0
- package/builtins/ja/workflows/default-draft.yaml +6 -0
- package/builtins/ja/workflows/default-high.yaml +7 -1
- package/builtins/ja/workflows/default-mini.yaml +2 -0
- package/builtins/ja/workflows/default-peer-review.yaml +41 -4
- package/builtins/ja/workflows/default.yaml +4 -0
- package/builtins/ja/workflows/draft.yaml +12 -32
- package/builtins/ja/workflows/dual-cqrs-mini.yaml +20 -0
- package/builtins/ja/workflows/dual-cqrs.yaml +56 -1
- package/builtins/ja/workflows/dual-mini.yaml +20 -0
- package/builtins/ja/workflows/dual.yaml +44 -6
- package/builtins/ja/workflows/frontend-maintenance.yaml +62 -1
- package/builtins/ja/workflows/frontend-mini.yaml +20 -0
- package/builtins/ja/workflows/frontend.yaml +62 -1
- package/builtins/ja/workflows/magi.yaml +6 -0
- package/builtins/ja/workflows/peer-review-with-fc.yaml +206 -0
- package/builtins/ja/workflows/peer-review.yaml +35 -72
- package/builtins/ja/workflows/research.yaml +6 -0
- package/builtins/ja/workflows/review-backend-cqrs.yaml +42 -1
- package/builtins/ja/workflows/review-backend.yaml +40 -1
- package/builtins/ja/workflows/review-default.yaml +28 -8
- package/builtins/ja/workflows/review-dual-cqrs.yaml +44 -1
- package/builtins/ja/workflows/review-dual.yaml +42 -1
- package/builtins/ja/workflows/review-fix-backend-cqrs.yaml +46 -1
- package/builtins/ja/workflows/review-fix-backend.yaml +44 -1
- package/builtins/ja/workflows/review-fix-default.yaml +31 -7
- package/builtins/ja/workflows/review-fix-dual-cqrs.yaml +48 -1
- package/builtins/ja/workflows/review-fix-dual.yaml +46 -1
- package/builtins/ja/workflows/review-fix-frontend.yaml +46 -1
- package/builtins/ja/workflows/review-fix-takt-default.yaml +39 -7
- package/builtins/ja/workflows/review-frontend.yaml +42 -1
- package/builtins/ja/workflows/review-takt-default.yaml +30 -8
- package/builtins/ja/workflows/takt-default-refresh-all.yaml +36 -6
- package/builtins/ja/workflows/takt-default-refresh-fast.yaml +35 -5
- package/builtins/ja/workflows/takt-default-with-fc.yaml +110 -0
- package/builtins/ja/workflows/takt-default.yaml +10 -27
- package/builtins/ja/workflows/terraform.yaml +51 -2
- package/builtins/skill/references/yaml-schema.md +4 -0
- package/builtins/skill-codex/references/yaml-schema.md +4 -0
- package/dist/agents/decompose-task-usecase.d.ts +2 -1
- package/dist/agents/decompose-task-usecase.d.ts.map +1 -1
- package/dist/agents/decompose-task-usecase.js +7 -5
- package/dist/agents/decompose-task-usecase.js.map +1 -1
- package/dist/agents/judge-status-usecase.d.ts +4 -1
- package/dist/agents/judge-status-usecase.d.ts.map +1 -1
- package/dist/agents/judge-status-usecase.js +5 -0
- package/dist/agents/judge-status-usecase.js.map +1 -1
- package/dist/agents/runner-prompt.d.ts +3 -1
- package/dist/agents/runner-prompt.d.ts.map +1 -1
- package/dist/agents/runner-prompt.js +11 -4
- package/dist/agents/runner-prompt.js.map +1 -1
- package/dist/agents/runner.d.ts.map +1 -1
- package/dist/agents/runner.js +23 -5
- package/dist/agents/runner.js.map +1 -1
- package/dist/agents/structured-caller/capability-aware-structured-caller.d.ts +1 -1
- package/dist/agents/structured-caller/capability-aware-structured-caller.d.ts.map +1 -1
- package/dist/agents/structured-caller/capability-aware-structured-caller.js +3 -3
- package/dist/agents/structured-caller/capability-aware-structured-caller.js.map +1 -1
- package/dist/agents/structured-caller/contracts.d.ts +1 -1
- package/dist/agents/structured-caller/contracts.d.ts.map +1 -1
- package/dist/agents/structured-caller/default-structured-caller.d.ts +1 -1
- package/dist/agents/structured-caller/default-structured-caller.d.ts.map +1 -1
- package/dist/agents/structured-caller/default-structured-caller.js +2 -2
- package/dist/agents/structured-caller/default-structured-caller.js.map +1 -1
- package/dist/agents/structured-caller/prompt-based-structured-caller.d.ts +1 -1
- package/dist/agents/structured-caller/prompt-based-structured-caller.d.ts.map +1 -1
- package/dist/agents/structured-caller/prompt-based-structured-caller.js +9 -3
- package/dist/agents/structured-caller/prompt-based-structured-caller.js.map +1 -1
- package/dist/agents/team-leader-structured-output.d.ts +3 -3
- package/dist/agents/team-leader-structured-output.d.ts.map +1 -1
- package/dist/agents/team-leader-structured-output.js +14 -10
- package/dist/agents/team-leader-structured-output.js.map +1 -1
- package/dist/agents/types.d.ts +1 -0
- package/dist/agents/types.d.ts.map +1 -1
- package/dist/app/cli/routing-inputs.d.ts +1 -0
- package/dist/app/cli/routing-inputs.d.ts.map +1 -1
- package/dist/app/cli/routing-inputs.js +5 -2
- package/dist/app/cli/routing-inputs.js.map +1 -1
- package/dist/app/cli/routing.d.ts.map +1 -1
- package/dist/app/cli/routing.js +14 -0
- package/dist/app/cli/routing.js.map +1 -1
- package/dist/commands/repertoire/add.d.ts.map +1 -1
- package/dist/commands/repertoire/add.js +45 -4
- package/dist/commands/repertoire/add.js.map +1 -1
- package/dist/commands/repertoire/remove.d.ts.map +1 -1
- package/dist/commands/repertoire/remove.js +19 -3
- package/dist/commands/repertoire/remove.js.map +1 -1
- package/dist/core/models/config-schemas.d.ts +787 -0
- package/dist/core/models/config-schemas.d.ts.map +1 -1
- package/dist/core/models/config-schemas.js +6 -0
- package/dist/core/models/config-schemas.js.map +1 -1
- package/dist/core/models/config-types.d.ts +8 -0
- package/dist/core/models/config-types.d.ts.map +1 -1
- package/dist/core/models/finding-schemas.d.ts +447 -0
- package/dist/core/models/finding-schemas.d.ts.map +1 -0
- package/dist/core/models/finding-schemas.js +234 -0
- package/dist/core/models/finding-schemas.js.map +1 -0
- package/dist/core/models/finding-types.d.ts +146 -0
- package/dist/core/models/finding-types.d.ts.map +1 -0
- package/dist/core/models/finding-types.js +5 -0
- package/dist/core/models/finding-types.js.map +1 -0
- package/dist/core/models/index.d.ts +1 -1
- package/dist/core/models/index.d.ts.map +1 -1
- package/dist/core/models/index.js.map +1 -1
- package/dist/core/models/part.d.ts +2 -1
- package/dist/core/models/part.d.ts.map +1 -1
- package/dist/core/models/schema-base.d.ts +66 -0
- package/dist/core/models/schema-base.d.ts.map +1 -1
- package/dist/core/models/schema-base.js +7 -2
- package/dist/core/models/schema-base.js.map +1 -1
- package/dist/core/models/types.d.ts +2 -2
- package/dist/core/models/types.d.ts.map +1 -1
- package/dist/core/models/workflow-condition-expression.d.ts +3 -0
- package/dist/core/models/workflow-condition-expression.d.ts.map +1 -1
- package/dist/core/models/workflow-condition-expression.js +223 -4
- package/dist/core/models/workflow-condition-expression.js.map +1 -1
- package/dist/core/models/workflow-provider-options.d.ts +5 -0
- package/dist/core/models/workflow-provider-options.d.ts.map +1 -1
- package/dist/core/models/workflow-provider-options.js.map +1 -1
- package/dist/core/models/workflow-schemas.d.ts +197 -31
- package/dist/core/models/workflow-schemas.d.ts.map +1 -1
- package/dist/core/models/workflow-schemas.js +67 -18
- package/dist/core/models/workflow-schemas.js.map +1 -1
- package/dist/core/models/workflow-system-schemas.d.ts.map +1 -1
- package/dist/core/models/workflow-system-schemas.js +1 -0
- package/dist/core/models/workflow-system-schemas.js.map +1 -1
- package/dist/core/models/workflow-types.d.ts +14 -1
- package/dist/core/models/workflow-types.d.ts.map +1 -1
- package/dist/core/models/workflow-types.js.map +1 -1
- package/dist/core/workflow/engine/ArpeggioRunner.d.ts +2 -0
- package/dist/core/workflow/engine/ArpeggioRunner.d.ts.map +1 -1
- package/dist/core/workflow/engine/ArpeggioRunner.js +1 -0
- package/dist/core/workflow/engine/ArpeggioRunner.js.map +1 -1
- package/dist/core/workflow/engine/OptionsBuilder.d.ts +4 -1
- package/dist/core/workflow/engine/OptionsBuilder.d.ts.map +1 -1
- package/dist/core/workflow/engine/OptionsBuilder.js +23 -8
- package/dist/core/workflow/engine/OptionsBuilder.js.map +1 -1
- package/dist/core/workflow/engine/ParallelRunner.d.ts +15 -0
- package/dist/core/workflow/engine/ParallelRunner.d.ts.map +1 -1
- package/dist/core/workflow/engine/ParallelRunner.js +155 -18
- package/dist/core/workflow/engine/ParallelRunner.js.map +1 -1
- package/dist/core/workflow/engine/StepExecutor.d.ts +4 -2
- package/dist/core/workflow/engine/StepExecutor.d.ts.map +1 -1
- package/dist/core/workflow/engine/StepExecutor.js +11 -3
- package/dist/core/workflow/engine/StepExecutor.js.map +1 -1
- package/dist/core/workflow/engine/TeamLeaderRunner.d.ts.map +1 -1
- package/dist/core/workflow/engine/TeamLeaderRunner.js +9 -4
- package/dist/core/workflow/engine/TeamLeaderRunner.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowCallExecutor.d.ts +3 -1
- package/dist/core/workflow/engine/WorkflowCallExecutor.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowCallExecutor.js +23 -5
- package/dist/core/workflow/engine/WorkflowCallExecutor.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowCallRunner.d.ts +1 -0
- package/dist/core/workflow/engine/WorkflowCallRunner.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowCallRunner.js +5 -1
- package/dist/core/workflow/engine/WorkflowCallRunner.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngine.d.ts +7 -1
- package/dist/core/workflow/engine/WorkflowEngine.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngine.js +63 -9
- package/dist/core/workflow/engine/WorkflowEngine.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngineSetup.d.ts +3 -0
- package/dist/core/workflow/engine/WorkflowEngineSetup.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowEngineSetup.js +29 -1
- package/dist/core/workflow/engine/WorkflowEngineSetup.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowRunLoop.d.ts +11 -7
- package/dist/core/workflow/engine/WorkflowRunLoop.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowRunLoop.js +66 -33
- package/dist/core/workflow/engine/WorkflowRunLoop.js.map +1 -1
- package/dist/core/workflow/engine/WorkflowValidator.d.ts.map +1 -1
- package/dist/core/workflow/engine/WorkflowValidator.js +78 -1
- package/dist/core/workflow/engine/WorkflowValidator.js.map +1 -1
- package/dist/core/workflow/engine/engine-provider-options.d.ts.map +1 -1
- package/dist/core/workflow/engine/engine-provider-options.js +3 -3
- package/dist/core/workflow/engine/engine-provider-options.js.map +1 -1
- package/dist/core/workflow/engine/parallel-logger.d.ts +1 -1
- package/dist/core/workflow/engine/parallel-logger.d.ts.map +1 -1
- package/dist/core/workflow/engine/parallel-logger.js +3 -1
- package/dist/core/workflow/engine/parallel-logger.js.map +1 -1
- package/dist/core/workflow/engine/task-decomposer.d.ts +1 -1
- package/dist/core/workflow/engine/task-decomposer.d.ts.map +1 -1
- package/dist/core/workflow/engine/task-decomposer.js +3 -3
- package/dist/core/workflow/engine/task-decomposer.js.map +1 -1
- package/dist/core/workflow/engine/team-leader-budget-errors.d.ts +2 -0
- package/dist/core/workflow/engine/team-leader-budget-errors.d.ts.map +1 -0
- package/dist/core/workflow/engine/team-leader-budget-errors.js +12 -0
- package/dist/core/workflow/engine/team-leader-budget-errors.js.map +1 -0
- package/dist/core/workflow/engine/team-leader-common.d.ts.map +1 -1
- package/dist/core/workflow/engine/team-leader-common.js +11 -0
- package/dist/core/workflow/engine/team-leader-common.js.map +1 -1
- package/dist/core/workflow/engine/team-leader-execution.d.ts.map +1 -1
- package/dist/core/workflow/engine/team-leader-execution.js +16 -2
- package/dist/core/workflow/engine/team-leader-execution.js.map +1 -1
- package/dist/core/workflow/engine/team-leader-part-runner.d.ts.map +1 -1
- package/dist/core/workflow/engine/team-leader-part-runner.js +22 -1
- package/dist/core/workflow/engine/team-leader-part-runner.js.map +1 -1
- package/dist/core/workflow/evaluation/AggregateEvaluator.d.ts.map +1 -1
- package/dist/core/workflow/evaluation/AggregateEvaluator.js +7 -4
- package/dist/core/workflow/evaluation/AggregateEvaluator.js.map +1 -1
- package/dist/core/workflow/evaluation/RuleEvaluator.d.ts +3 -0
- package/dist/core/workflow/evaluation/RuleEvaluator.d.ts.map +1 -1
- package/dist/core/workflow/evaluation/RuleEvaluator.js +16 -1
- package/dist/core/workflow/evaluation/RuleEvaluator.js.map +1 -1
- package/dist/core/workflow/evaluation/rule-utils.d.ts +5 -1
- package/dist/core/workflow/evaluation/rule-utils.d.ts.map +1 -1
- package/dist/core/workflow/evaluation/rule-utils.js +34 -1
- package/dist/core/workflow/evaluation/rule-utils.js.map +1 -1
- package/dist/core/workflow/evaluation/when-evaluator.d.ts.map +1 -1
- package/dist/core/workflow/evaluation/when-evaluator.js +4 -1
- package/dist/core/workflow/evaluation/when-evaluator.js.map +1 -1
- package/dist/core/workflow/findings/context.d.ts +5 -0
- package/dist/core/workflow/findings/context.d.ts.map +1 -0
- package/dist/core/workflow/findings/context.js +82 -0
- package/dist/core/workflow/findings/context.js.map +1 -0
- package/dist/core/workflow/findings/ledger-validation.d.ts +3 -0
- package/dist/core/workflow/findings/ledger-validation.d.ts.map +1 -0
- package/dist/core/workflow/findings/ledger-validation.js +26 -0
- package/dist/core/workflow/findings/ledger-validation.js.map +1 -0
- package/dist/core/workflow/findings/manager-output-validation.d.ts +15 -0
- package/dist/core/workflow/findings/manager-output-validation.d.ts.map +1 -0
- package/dist/core/workflow/findings/manager-output-validation.js +221 -0
- package/dist/core/workflow/findings/manager-output-validation.js.map +1 -0
- package/dist/core/workflow/findings/manager-runner.d.ts +89 -0
- package/dist/core/workflow/findings/manager-runner.d.ts.map +1 -0
- package/dist/core/workflow/findings/manager-runner.js +291 -0
- package/dist/core/workflow/findings/manager-runner.js.map +1 -0
- package/dist/core/workflow/findings/reconciler.d.ts +10 -0
- package/dist/core/workflow/findings/reconciler.d.ts.map +1 -0
- package/dist/core/workflow/findings/reconciler.js +413 -0
- package/dist/core/workflow/findings/reconciler.js.map +1 -0
- package/dist/core/workflow/findings/schemas.d.ts +2 -0
- package/dist/core/workflow/findings/schemas.d.ts.map +1 -0
- package/dist/core/workflow/findings/schemas.js +2 -0
- package/dist/core/workflow/findings/schemas.js.map +1 -0
- package/dist/core/workflow/findings/store.d.ts +33 -0
- package/dist/core/workflow/findings/store.d.ts.map +1 -0
- package/dist/core/workflow/findings/store.js +169 -0
- package/dist/core/workflow/findings/store.js.map +1 -0
- package/dist/core/workflow/findings/types.d.ts +3 -0
- package/dist/core/workflow/findings/types.d.ts.map +1 -0
- package/dist/core/workflow/findings/types.js +2 -0
- package/dist/core/workflow/findings/types.js.map +1 -0
- package/dist/core/workflow/instruction/InstructionBuilder.d.ts +1 -0
- package/dist/core/workflow/instruction/InstructionBuilder.d.ts.map +1 -1
- package/dist/core/workflow/instruction/InstructionBuilder.js +35 -11
- package/dist/core/workflow/instruction/InstructionBuilder.js.map +1 -1
- package/dist/core/workflow/instruction/ReportInstructionBuilder.d.ts +4 -0
- package/dist/core/workflow/instruction/ReportInstructionBuilder.d.ts.map +1 -1
- package/dist/core/workflow/instruction/ReportInstructionBuilder.js +22 -1
- package/dist/core/workflow/instruction/ReportInstructionBuilder.js.map +1 -1
- package/dist/core/workflow/instruction/fenced-json.d.ts +2 -0
- package/dist/core/workflow/instruction/fenced-json.d.ts.map +1 -0
- package/dist/core/workflow/instruction/fenced-json.js +11 -0
- package/dist/core/workflow/instruction/fenced-json.js.map +1 -0
- package/dist/core/workflow/instruction/instruction-context.d.ts +8 -0
- package/dist/core/workflow/instruction/instruction-context.d.ts.map +1 -1
- package/dist/core/workflow/instruction/instruction-context.js.map +1 -1
- package/dist/core/workflow/observability/traceDiscovery.d.ts +33 -0
- package/dist/core/workflow/observability/traceDiscovery.d.ts.map +1 -0
- package/dist/core/workflow/observability/traceDiscovery.js +104 -0
- package/dist/core/workflow/observability/traceDiscovery.js.map +1 -0
- package/dist/core/workflow/observability/workflowSpans.d.ts +5 -2
- package/dist/core/workflow/observability/workflowSpans.d.ts.map +1 -1
- package/dist/core/workflow/observability/workflowSpans.js +60 -13
- package/dist/core/workflow/observability/workflowSpans.js.map +1 -1
- package/dist/core/workflow/phase-runner.d.ts +4 -0
- package/dist/core/workflow/phase-runner.d.ts.map +1 -1
- package/dist/core/workflow/phase-runner.js.map +1 -1
- package/dist/core/workflow/promotion/PromotionEvaluator.d.ts +2 -0
- package/dist/core/workflow/promotion/PromotionEvaluator.d.ts.map +1 -1
- package/dist/core/workflow/promotion/PromotionEvaluator.js +1 -0
- package/dist/core/workflow/promotion/PromotionEvaluator.js.map +1 -1
- package/dist/core/workflow/promotion/promotion-runtime.d.ts +2 -0
- package/dist/core/workflow/promotion/promotion-runtime.d.ts.map +1 -1
- package/dist/core/workflow/promotion/promotion-runtime.js +1 -0
- package/dist/core/workflow/promotion/promotion-runtime.js.map +1 -1
- package/dist/core/workflow/provider-options-trace.d.ts +3 -2
- package/dist/core/workflow/provider-options-trace.d.ts.map +1 -1
- package/dist/core/workflow/provider-resolution.d.ts +11 -3
- package/dist/core/workflow/provider-resolution.d.ts.map +1 -1
- package/dist/core/workflow/provider-resolution.js +68 -2
- package/dist/core/workflow/provider-resolution.js.map +1 -1
- package/dist/core/workflow/quality-gates/commandGateRunner.d.ts +1 -1
- package/dist/core/workflow/quality-gates/commandGateRunner.d.ts.map +1 -1
- package/dist/core/workflow/quality-gates/commandGateRunner.js +11 -6
- package/dist/core/workflow/quality-gates/commandGateRunner.js.map +1 -1
- package/dist/core/workflow/quality-gates/qualityGateRunner.d.ts +1 -1
- package/dist/core/workflow/quality-gates/qualityGateRunner.d.ts.map +1 -1
- package/dist/core/workflow/quality-gates/qualityGateRunner.js +2 -2
- package/dist/core/workflow/quality-gates/qualityGateRunner.js.map +1 -1
- package/dist/core/workflow/quality-gates/types.d.ts +2 -0
- package/dist/core/workflow/quality-gates/types.d.ts.map +1 -1
- package/dist/core/workflow/report-phase-runner.d.ts.map +1 -1
- package/dist/core/workflow/report-phase-runner.js +59 -44
- package/dist/core/workflow/report-phase-runner.js.map +1 -1
- package/dist/core/workflow/report-writer.d.ts +2 -0
- package/dist/core/workflow/report-writer.d.ts.map +1 -0
- package/dist/core/workflow/report-writer.js +42 -0
- package/dist/core/workflow/report-writer.js.map +1 -0
- package/dist/core/workflow/run/run-meta.d.ts +5 -0
- package/dist/core/workflow/run/run-meta.d.ts.map +1 -1
- package/dist/core/workflow/run/run-meta.js.map +1 -1
- package/dist/core/workflow/state/workflow-state-access.d.ts +2 -2
- package/dist/core/workflow/state/workflow-state-access.d.ts.map +1 -1
- package/dist/core/workflow/state/workflow-state-access.js +17 -2
- package/dist/core/workflow/state/workflow-state-access.js.map +1 -1
- package/dist/core/workflow/status-judgment-phase.d.ts.map +1 -1
- package/dist/core/workflow/status-judgment-phase.js +1 -0
- package/dist/core/workflow/status-judgment-phase.js.map +1 -1
- package/dist/core/workflow/types.d.ts +27 -1
- package/dist/core/workflow/types.d.ts.map +1 -1
- package/dist/features/analytics/events.d.ts +1 -1
- package/dist/features/analytics/events.d.ts.map +1 -1
- package/dist/features/analytics/index.d.ts +1 -1
- package/dist/features/analytics/index.d.ts.map +1 -1
- package/dist/features/analytics/index.js +1 -1
- package/dist/features/analytics/index.js.map +1 -1
- package/dist/features/analytics/report-parser.d.ts +5 -3
- package/dist/features/analytics/report-parser.d.ts.map +1 -1
- package/dist/features/analytics/report-parser.js +42 -7
- package/dist/features/analytics/report-parser.js.map +1 -1
- package/dist/features/interactive/aiCaller.d.ts.map +1 -1
- package/dist/features/interactive/aiCaller.js +4 -2
- package/dist/features/interactive/aiCaller.js.map +1 -1
- package/dist/features/pipeline/execute.d.ts.map +1 -1
- package/dist/features/pipeline/execute.js +2 -3
- package/dist/features/pipeline/execute.js.map +1 -1
- package/dist/features/pipeline/steps.d.ts +4 -3
- package/dist/features/pipeline/steps.d.ts.map +1 -1
- package/dist/features/pipeline/steps.js +46 -8
- package/dist/features/pipeline/steps.js.map +1 -1
- package/dist/features/repertoire/file-filter.d.ts +3 -3
- package/dist/features/repertoire/file-filter.d.ts.map +1 -1
- package/dist/features/repertoire/file-filter.js +3 -3
- package/dist/features/repertoire/file-filter.js.map +1 -1
- package/dist/features/repertoire/pack-summary.d.ts +21 -7
- package/dist/features/repertoire/pack-summary.d.ts.map +1 -1
- package/dist/features/repertoire/pack-summary.js +156 -24
- package/dist/features/repertoire/pack-summary.js.map +1 -1
- package/dist/features/repertoire/remove.d.ts +5 -2
- package/dist/features/repertoire/remove.d.ts.map +1 -1
- package/dist/features/repertoire/remove.js +9 -2
- package/dist/features/repertoire/remove.js.map +1 -1
- package/dist/features/repertoire/takt-repertoire-config.d.ts +3 -3
- package/dist/features/repertoire/takt-repertoire-config.d.ts.map +1 -1
- package/dist/features/repertoire/takt-repertoire-config.js +31 -16
- package/dist/features/repertoire/takt-repertoire-config.js.map +1 -1
- package/dist/features/tasks/add/index.d.ts +1 -0
- package/dist/features/tasks/add/index.d.ts.map +1 -1
- package/dist/features/tasks/add/index.js +1 -0
- package/dist/features/tasks/add/index.js.map +1 -1
- package/dist/features/tasks/execute/analyticsEmitter.d.ts +4 -1
- package/dist/features/tasks/execute/analyticsEmitter.d.ts.map +1 -1
- package/dist/features/tasks/execute/analyticsEmitter.js +30 -3
- package/dist/features/tasks/execute/analyticsEmitter.js.map +1 -1
- package/dist/features/tasks/execute/runAllTasks.d.ts.map +1 -1
- package/dist/features/tasks/execute/runAllTasks.js +2 -2
- package/dist/features/tasks/execute/runAllTasks.js.map +1 -1
- package/dist/features/tasks/execute/runMeta.d.ts +5 -1
- package/dist/features/tasks/execute/runMeta.d.ts.map +1 -1
- package/dist/features/tasks/execute/runMeta.js +6 -1
- package/dist/features/tasks/execute/runMeta.js.map +1 -1
- package/dist/features/tasks/execute/selectAndExecute.d.ts.map +1 -1
- package/dist/features/tasks/execute/selectAndExecute.js +12 -0
- package/dist/features/tasks/execute/selectAndExecute.js.map +1 -1
- package/dist/features/tasks/execute/slackSummaryAdapter.d.ts +0 -1
- package/dist/features/tasks/execute/slackSummaryAdapter.d.ts.map +1 -1
- package/dist/features/tasks/execute/slackSummaryAdapter.js +0 -5
- package/dist/features/tasks/execute/slackSummaryAdapter.js.map +1 -1
- package/dist/features/tasks/execute/taskExecution.d.ts.map +1 -1
- package/dist/features/tasks/execute/taskExecution.js +9 -0
- package/dist/features/tasks/execute/taskExecution.js.map +1 -1
- package/dist/features/tasks/execute/taskWorkflowExecution.d.ts.map +1 -1
- package/dist/features/tasks/execute/taskWorkflowExecution.js +20 -1
- package/dist/features/tasks/execute/taskWorkflowExecution.js.map +1 -1
- package/dist/features/tasks/execute/traceTaskMetadata.d.ts +20 -0
- package/dist/features/tasks/execute/traceTaskMetadata.d.ts.map +1 -0
- package/dist/features/tasks/execute/traceTaskMetadata.js +54 -0
- package/dist/features/tasks/execute/traceTaskMetadata.js.map +1 -0
- package/dist/features/tasks/execute/types.d.ts +13 -1
- package/dist/features/tasks/execute/types.d.ts.map +1 -1
- package/dist/features/tasks/execute/workflowExecution.d.ts.map +1 -1
- package/dist/features/tasks/execute/workflowExecution.js +44 -1
- package/dist/features/tasks/execute/workflowExecution.js.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionBootstrap.d.ts +2 -0
- package/dist/features/tasks/execute/workflowExecutionBootstrap.d.ts.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionBootstrap.js +16 -2
- package/dist/features/tasks/execute/workflowExecutionBootstrap.js.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionEvents.d.ts +2 -0
- package/dist/features/tasks/execute/workflowExecutionEvents.d.ts.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionEvents.js +16 -2
- package/dist/features/tasks/execute/workflowExecutionEvents.js.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionReporting.d.ts +3 -2
- package/dist/features/tasks/execute/workflowExecutionReporting.d.ts.map +1 -1
- package/dist/features/tasks/execute/workflowExecutionReporting.js +14 -2
- package/dist/features/tasks/execute/workflowExecutionReporting.js.map +1 -1
- package/dist/features/tasks/index.d.ts +1 -1
- package/dist/features/tasks/index.d.ts.map +1 -1
- package/dist/features/tasks/index.js.map +1 -1
- package/dist/features/tasks/resume/index.d.ts.map +1 -1
- package/dist/features/tasks/resume/index.js +5 -0
- package/dist/features/tasks/resume/index.js.map +1 -1
- package/dist/infra/claude/client.d.ts.map +1 -1
- package/dist/infra/claude/client.js +1 -0
- package/dist/infra/claude/client.js.map +1 -1
- package/dist/infra/claude/options-builder.d.ts.map +1 -1
- package/dist/infra/claude/options-builder.js +2 -3
- package/dist/infra/claude/options-builder.js.map +1 -1
- package/dist/infra/claude/types.d.ts +2 -0
- package/dist/infra/claude/types.d.ts.map +1 -1
- package/dist/infra/claude-headless/headless-spawn.d.ts.map +1 -1
- package/dist/infra/claude-headless/headless-spawn.js +2 -1
- package/dist/infra/claude-headless/headless-spawn.js.map +1 -1
- package/dist/infra/claude-headless/types.d.ts +1 -0
- package/dist/infra/claude-headless/types.d.ts.map +1 -1
- package/dist/infra/claude-terminal/client.d.ts.map +1 -1
- package/dist/infra/claude-terminal/client.js +1 -0
- package/dist/infra/claude-terminal/client.js.map +1 -1
- package/dist/infra/claude-terminal/response-normalizer.js.map +1 -1
- package/dist/infra/claude-terminal/tmux-backend.d.ts.map +1 -1
- package/dist/infra/claude-terminal/tmux-backend.js +15 -1
- package/dist/infra/claude-terminal/tmux-backend.js.map +1 -1
- package/dist/infra/claude-terminal/types.d.ts +2 -0
- package/dist/infra/claude-terminal/types.d.ts.map +1 -1
- package/dist/infra/codex/client.d.ts.map +1 -1
- package/dist/infra/codex/client.js +3 -1
- package/dist/infra/codex/client.js.map +1 -1
- package/dist/infra/codex/types.d.ts +1 -0
- package/dist/infra/codex/types.d.ts.map +1 -1
- package/dist/infra/config/configNormalizers.d.ts +18 -7
- package/dist/infra/config/configNormalizers.d.ts.map +1 -1
- package/dist/infra/config/configNormalizers.js +55 -14
- package/dist/infra/config/configNormalizers.js.map +1 -1
- package/dist/infra/config/env/global-current-env-specs.d.ts.map +1 -1
- package/dist/infra/config/env/global-current-env-specs.js +1 -0
- package/dist/infra/config/env/global-current-env-specs.js.map +1 -1
- package/dist/infra/config/env/project-current-env-specs.d.ts.map +1 -1
- package/dist/infra/config/env/project-current-env-specs.js +1 -0
- package/dist/infra/config/env/project-current-env-specs.js.map +1 -1
- package/dist/infra/config/global/globalConfigCore.d.ts.map +1 -1
- package/dist/infra/config/global/globalConfigCore.js +2 -1
- package/dist/infra/config/global/globalConfigCore.js.map +1 -1
- package/dist/infra/config/global/globalConfigSerializer.d.ts.map +1 -1
- package/dist/infra/config/global/globalConfigSerializer.js +5 -1
- package/dist/infra/config/global/globalConfigSerializer.js.map +1 -1
- package/dist/infra/config/loaders/namedResourceResolver.d.ts +18 -0
- package/dist/infra/config/loaders/namedResourceResolver.d.ts.map +1 -0
- package/dist/infra/config/loaders/namedResourceResolver.js +49 -0
- package/dist/infra/config/loaders/namedResourceResolver.js.map +1 -0
- package/dist/infra/config/loaders/providerOptionsLookupDirectories.d.ts +13 -0
- package/dist/infra/config/loaders/providerOptionsLookupDirectories.d.ts.map +1 -0
- package/dist/infra/config/loaders/providerOptionsLookupDirectories.js +61 -0
- package/dist/infra/config/loaders/providerOptionsLookupDirectories.js.map +1 -0
- package/dist/infra/config/loaders/workflowDiscovery.js +1 -1
- package/dist/infra/config/loaders/workflowDiscovery.js.map +1 -1
- package/dist/infra/config/loaders/workflowLoopMonitorNormalizer.d.ts.map +1 -1
- package/dist/infra/config/loaders/workflowLoopMonitorNormalizer.js +1 -1
- package/dist/infra/config/loaders/workflowLoopMonitorNormalizer.js.map +1 -1
- package/dist/infra/config/loaders/workflowParser.d.ts.map +1 -1
- package/dist/infra/config/loaders/workflowParser.js +64 -3
- package/dist/infra/config/loaders/workflowParser.js.map +1 -1
- package/dist/infra/config/loaders/workflowPreview.d.ts.map +1 -1
- package/dist/infra/config/loaders/workflowPreview.js +8 -3
- package/dist/infra/config/loaders/workflowPreview.js.map +1 -1
- package/dist/infra/config/loaders/workflowProviderOptionsResolver.d.ts +25 -0
- package/dist/infra/config/loaders/workflowProviderOptionsResolver.d.ts.map +1 -0
- package/dist/infra/config/loaders/workflowProviderOptionsResolver.js +150 -0
- package/dist/infra/config/loaders/workflowProviderOptionsResolver.js.map +1 -0
- package/dist/infra/config/loaders/workflowRuleNormalizer.d.ts.map +1 -1
- package/dist/infra/config/loaders/workflowRuleNormalizer.js +5 -16
- package/dist/infra/config/loaders/workflowRuleNormalizer.js.map +1 -1
- package/dist/infra/config/loaders/workflowStepFeaturesNormalizer.d.ts.map +1 -1
- package/dist/infra/config/loaders/workflowStepFeaturesNormalizer.js +5 -3
- package/dist/infra/config/loaders/workflowStepFeaturesNormalizer.js.map +1 -1
- package/dist/infra/config/loaders/workflowStepNormalizer.d.ts +3 -2
- package/dist/infra/config/loaders/workflowStepNormalizer.d.ts.map +1 -1
- package/dist/infra/config/loaders/workflowStepNormalizer.js +65 -13
- package/dist/infra/config/loaders/workflowStepNormalizer.js.map +1 -1
- package/dist/infra/config/paths.d.ts +4 -0
- package/dist/infra/config/paths.d.ts.map +1 -1
- package/dist/infra/config/paths.js +13 -0
- package/dist/infra/config/paths.js.map +1 -1
- package/dist/infra/config/project/projectConfig.d.ts.map +1 -1
- package/dist/infra/config/project/projectConfig.js +12 -3
- package/dist/infra/config/project/projectConfig.js.map +1 -1
- package/dist/infra/config/providerOptions.d.ts +24 -7
- package/dist/infra/config/providerOptions.d.ts.map +1 -1
- package/dist/infra/config/providerOptions.js +85 -11
- package/dist/infra/config/providerOptions.js.map +1 -1
- package/dist/infra/config/providerOptionsContract.d.ts +3 -3
- package/dist/infra/config/providerOptionsContract.d.ts.map +1 -1
- package/dist/infra/config/providerOptionsContract.js +8 -0
- package/dist/infra/config/providerOptionsContract.js.map +1 -1
- package/dist/infra/config/traced/tracedConfigSchema.d.ts.map +1 -1
- package/dist/infra/config/traced/tracedConfigSchema.js +8 -0
- package/dist/infra/config/traced/tracedConfigSchema.js.map +1 -1
- package/dist/infra/copilot/client.d.ts.map +1 -1
- package/dist/infra/copilot/client.js +7 -8
- package/dist/infra/copilot/client.js.map +1 -1
- package/dist/infra/copilot/types.d.ts +1 -0
- package/dist/infra/copilot/types.d.ts.map +1 -1
- package/dist/infra/cursor/client.d.ts.map +1 -1
- package/dist/infra/cursor/client.js +120 -55
- package/dist/infra/cursor/client.js.map +1 -1
- package/dist/infra/cursor/types.d.ts +1 -0
- package/dist/infra/cursor/types.d.ts.map +1 -1
- package/dist/infra/kiro/client.d.ts.map +1 -1
- package/dist/infra/kiro/client.js +10 -0
- package/dist/infra/kiro/client.js.map +1 -1
- package/dist/infra/kiro/process.d.ts.map +1 -1
- package/dist/infra/kiro/process.js +4 -2
- package/dist/infra/kiro/process.js.map +1 -1
- package/dist/infra/kiro/types.d.ts +2 -0
- package/dist/infra/kiro/types.d.ts.map +1 -1
- package/dist/infra/observability/otelFoundation.d.ts.map +1 -1
- package/dist/infra/observability/otelFoundation.js +93 -20
- package/dist/infra/observability/otelFoundation.js.map +1 -1
- package/dist/infra/opencode/OpenCodeStreamHandler.d.ts +19 -0
- package/dist/infra/opencode/OpenCodeStreamHandler.d.ts.map +1 -1
- package/dist/infra/opencode/OpenCodeStreamHandler.js +10 -0
- package/dist/infra/opencode/OpenCodeStreamHandler.js.map +1 -1
- package/dist/infra/opencode/allowedTools.d.ts +2 -0
- package/dist/infra/opencode/allowedTools.d.ts.map +1 -0
- package/dist/infra/opencode/allowedTools.js +10 -0
- package/dist/infra/opencode/allowedTools.js.map +1 -0
- package/dist/infra/opencode/client.d.ts +4 -2
- package/dist/infra/opencode/client.d.ts.map +1 -1
- package/dist/infra/opencode/client.js +304 -124
- package/dist/infra/opencode/client.js.map +1 -1
- package/dist/infra/opencode/index.d.ts +1 -1
- package/dist/infra/opencode/index.d.ts.map +1 -1
- package/dist/infra/opencode/index.js +1 -1
- package/dist/infra/opencode/index.js.map +1 -1
- package/dist/infra/opencode/types.d.ts +12 -10
- package/dist/infra/opencode/types.d.ts.map +1 -1
- package/dist/infra/opencode/types.js +101 -36
- package/dist/infra/opencode/types.js.map +1 -1
- package/dist/infra/opencode/unavailable-tool-loop.d.ts +7 -0
- package/dist/infra/opencode/unavailable-tool-loop.d.ts.map +1 -0
- package/dist/infra/opencode/unavailable-tool-loop.js +33 -0
- package/dist/infra/opencode/unavailable-tool-loop.js.map +1 -0
- package/dist/infra/providers/allowed-tool-edit-policy.d.ts +3 -0
- package/dist/infra/providers/allowed-tool-edit-policy.d.ts.map +1 -0
- package/dist/infra/providers/allowed-tool-edit-policy.js +10 -0
- package/dist/infra/providers/allowed-tool-edit-policy.js.map +1 -0
- package/dist/infra/providers/claude-headless.d.ts +2 -0
- package/dist/infra/providers/claude-headless.d.ts.map +1 -1
- package/dist/infra/providers/claude-headless.js +8 -0
- package/dist/infra/providers/claude-headless.js.map +1 -1
- package/dist/infra/providers/claude-terminal.d.ts +2 -0
- package/dist/infra/providers/claude-terminal.d.ts.map +1 -1
- package/dist/infra/providers/claude-terminal.js +8 -0
- package/dist/infra/providers/claude-terminal.js.map +1 -1
- package/dist/infra/providers/claude.d.ts +2 -0
- package/dist/infra/providers/claude.d.ts.map +1 -1
- package/dist/infra/providers/claude.js +8 -0
- package/dist/infra/providers/claude.js.map +1 -1
- package/dist/infra/providers/codex.d.ts +2 -0
- package/dist/infra/providers/codex.d.ts.map +1 -1
- package/dist/infra/providers/codex.js +7 -0
- package/dist/infra/providers/codex.js.map +1 -1
- package/dist/infra/providers/copilot.d.ts +2 -0
- package/dist/infra/providers/copilot.d.ts.map +1 -1
- package/dist/infra/providers/copilot.js +7 -0
- package/dist/infra/providers/copilot.js.map +1 -1
- package/dist/infra/providers/cursor.d.ts +2 -0
- package/dist/infra/providers/cursor.d.ts.map +1 -1
- package/dist/infra/providers/cursor.js +7 -0
- package/dist/infra/providers/cursor.js.map +1 -1
- package/dist/infra/providers/kiro.d.ts +2 -0
- package/dist/infra/providers/kiro.d.ts.map +1 -1
- package/dist/infra/providers/kiro.js +8 -0
- package/dist/infra/providers/kiro.js.map +1 -1
- package/dist/infra/providers/mock.d.ts +2 -0
- package/dist/infra/providers/mock.d.ts.map +1 -1
- package/dist/infra/providers/mock.js +7 -0
- package/dist/infra/providers/mock.js.map +1 -1
- package/dist/infra/providers/opencode.d.ts +3 -1
- package/dist/infra/providers/opencode.d.ts.map +1 -1
- package/dist/infra/providers/opencode.js +16 -3
- package/dist/infra/providers/opencode.js.map +1 -1
- package/dist/infra/providers/provider-capabilities.d.ts +2 -0
- package/dist/infra/providers/provider-capabilities.d.ts.map +1 -1
- package/dist/infra/providers/provider-capabilities.js +13 -1
- package/dist/infra/providers/provider-capabilities.js.map +1 -1
- package/dist/infra/providers/runtimeSystemPrompt.d.ts +2 -0
- package/dist/infra/providers/runtimeSystemPrompt.d.ts.map +1 -0
- package/dist/infra/providers/runtimeSystemPrompt.js +11 -0
- package/dist/infra/providers/runtimeSystemPrompt.js.map +1 -0
- package/dist/infra/providers/types.d.ts +3 -0
- package/dist/infra/providers/types.d.ts.map +1 -1
- package/dist/infra/rate-limit/detection.d.ts +1 -1
- package/dist/infra/rate-limit/detection.d.ts.map +1 -1
- package/dist/infra/rate-limit/detection.js +6 -7
- package/dist/infra/rate-limit/detection.js.map +1 -1
- package/dist/infra/resources/schema-loader.d.ts +1 -1
- package/dist/infra/resources/schema-loader.d.ts.map +1 -1
- package/dist/infra/resources/schema-loader.js +8 -4
- package/dist/infra/resources/schema-loader.js.map +1 -1
- package/dist/infra/service/runSyncConflictResolver.d.ts.map +1 -1
- package/dist/infra/service/runSyncConflictResolver.js +3 -1
- package/dist/infra/service/runSyncConflictResolver.js.map +1 -1
- package/dist/infra/task/clone-base-branch.d.ts +1 -0
- package/dist/infra/task/clone-base-branch.d.ts.map +1 -1
- package/dist/infra/task/clone-base-branch.js +22 -14
- package/dist/infra/task/clone-base-branch.js.map +1 -1
- package/dist/infra/task/clone.d.ts +2 -0
- package/dist/infra/task/clone.d.ts.map +1 -1
- package/dist/infra/task/clone.js +7 -1
- package/dist/infra/task/clone.js.map +1 -1
- package/dist/infra/task/index.d.ts +1 -1
- package/dist/infra/task/index.d.ts.map +1 -1
- package/dist/infra/task/index.js +1 -1
- package/dist/infra/task/index.js.map +1 -1
- package/dist/infra/task/mapper.d.ts.map +1 -1
- package/dist/infra/task/mapper.js +1 -0
- package/dist/infra/task/mapper.js.map +1 -1
- package/dist/infra/task/summarize.d.ts.map +1 -1
- package/dist/infra/task/summarize.js +3 -1
- package/dist/infra/task/summarize.js.map +1 -1
- package/dist/infra/task/taskExecutionSchemas.d.ts +3 -3
- package/dist/infra/task/taskRecordSchemas.d.ts +2 -2
- package/dist/infra/task/types.d.ts +1 -0
- package/dist/infra/task/types.d.ts.map +1 -1
- package/dist/shared/constants.d.ts +2 -0
- package/dist/shared/constants.d.ts.map +1 -1
- package/dist/shared/constants.js +2 -0
- package/dist/shared/constants.js.map +1 -1
- package/dist/shared/prompts/en/perform_agent_system_prompt.md +9 -1
- package/dist/shared/prompts/en/perform_phase2_message.md +1 -0
- package/dist/shared/prompts/en/provider_runtime_system_prompt.md +18 -0
- package/dist/shared/prompts/index.d.ts +1 -1
- package/dist/shared/prompts/index.d.ts.map +1 -1
- package/dist/shared/prompts/index.js +7 -1
- package/dist/shared/prompts/index.js.map +1 -1
- package/dist/shared/prompts/ja/perform_agent_system_prompt.md +9 -1
- package/dist/shared/prompts/ja/perform_phase2_message.md +1 -0
- package/dist/shared/prompts/ja/provider_runtime_system_prompt.md +18 -0
- package/dist/shared/telemetry/childProcessEnv.d.ts +7 -0
- package/dist/shared/telemetry/childProcessEnv.d.ts.map +1 -0
- package/dist/shared/telemetry/childProcessEnv.js +176 -0
- package/dist/shared/telemetry/childProcessEnv.js.map +1 -0
- package/dist/shared/telemetry/index.d.ts +3 -0
- package/dist/shared/telemetry/index.d.ts.map +1 -0
- package/dist/shared/telemetry/index.js +3 -0
- package/dist/shared/telemetry/index.js.map +1 -0
- package/dist/shared/telemetry/otlp.d.ts +40 -0
- package/dist/shared/telemetry/otlp.d.ts.map +1 -0
- package/dist/shared/telemetry/otlp.js +72 -0
- package/dist/shared/telemetry/otlp.js.map +1 -0
- package/dist/shared/types/provider.d.ts +25 -0
- package/dist/shared/types/provider.d.ts.map +1 -1
- package/dist/shared/ui/StreamDisplay.d.ts.map +1 -1
- package/dist/shared/ui/StreamDisplay.js +2 -1
- package/dist/shared/ui/StreamDisplay.js.map +1 -1
- package/dist/shared/utils/index.d.ts +1 -0
- package/dist/shared/utils/index.d.ts.map +1 -1
- package/dist/shared/utils/index.js +1 -0
- package/dist/shared/utils/index.js.map +1 -1
- package/dist/shared/utils/runId.d.ts +2 -0
- package/dist/shared/utils/runId.d.ts.map +1 -0
- package/dist/shared/utils/runId.js +18 -0
- package/dist/shared/utils/runId.js.map +1 -0
- package/package.json +25 -10
- package/builtins/en/facets/instructions/review-requirements.md +0 -17
- package/builtins/en/facets/output-contracts/requirements-review.md +0 -56
- package/builtins/en/facets/personas/requirements-reviewer.md +0 -33
- package/builtins/ja/facets/instructions/review-requirements.md +0 -17
- package/builtins/ja/facets/output-contracts/requirements-review.md +0 -56
- package/builtins/ja/facets/personas/requirements-reviewer.md +0 -33
|
@@ -168,6 +168,37 @@ class AgentRunner {
|
|
|
168
168
|
| 外部形式から内部形式への変換が1箇所に集約されている | OK |
|
|
169
169
|
| 同じ正規化ロジックが複数箇所にコピーされている | REJECT |
|
|
170
170
|
|
|
171
|
+
### 候補解決と値合成の分離
|
|
172
|
+
|
|
173
|
+
複数の候補から参照先を選ぶ処理と、選ばれた値を合成する処理は別の契約として扱う。探索順、上書き規則、参照種別を混ぜると、表示・検証・実行で別の結果になりやすい。
|
|
174
|
+
|
|
175
|
+
| 基準 | 判定 |
|
|
176
|
+
|------|------|
|
|
177
|
+
| 候補探索が first-match なのに、値合成の deep merge と混同して複数候補を暗黙に合成する | REJECT |
|
|
178
|
+
| 近いスコープの候補が遠いスコープの候補より後に探索される | REJECT |
|
|
179
|
+
| 参照文字列を「区切り文字を含むか」だけで分類し、特殊参照と明示パスを混同する | REJECT |
|
|
180
|
+
| 候補探索、参照種別の分類、値合成の責務が別関数として読める | OK |
|
|
181
|
+
|
|
182
|
+
```typescript
|
|
183
|
+
// REJECT - 参照種別と探索基準が1つの条件に混ざっている
|
|
184
|
+
const root = ref.includes('/') ? currentRoot : ownerRoot
|
|
185
|
+
|
|
186
|
+
// OK - 種別を先に分類し、種別ごとの探索契約を分ける
|
|
187
|
+
const kind = classifyReference(ref)
|
|
188
|
+
const root = resolveRootForReference(kind, resolvedPath)
|
|
189
|
+
```
|
|
190
|
+
|
|
191
|
+
### Raw入力の正規化
|
|
192
|
+
|
|
193
|
+
外部ファイルや設定から読む値は、構文上 valid でも期待する shape とは限らない。境界で unknown として受け、配列・record・scalar へ正規化してから内部処理へ渡す。
|
|
194
|
+
|
|
195
|
+
| 基準 | 判定 |
|
|
196
|
+
|------|------|
|
|
197
|
+
| parse 直後の unknown 値に配列メソッドやプロパティアクセスを直接行う | REJECT |
|
|
198
|
+
| 「存在する」だけでファイル種別やディレクトリ要件を満たしたと扱う | REJECT |
|
|
199
|
+
| unknown を境界で内部型へ正規化し、契約外shapeを無視・正規化・明示エラーのいずれかに固定する | OK |
|
|
200
|
+
| ファイルとディレクトリの要件を実体種別まで確認する | OK |
|
|
201
|
+
|
|
171
202
|
### フェーズ分離
|
|
172
203
|
|
|
173
204
|
入力、解釈、実行、出力を段階で分ける。反復処理は、できる限り「解釈済みの入力をまとめて受け取り、実行だけを繰り返す」構造にする。
|
|
@@ -65,7 +65,7 @@ data class Order(
|
|
|
65
65
|
|
|
66
66
|
## API層設計(Controller)
|
|
67
67
|
|
|
68
|
-
Controller
|
|
68
|
+
Controller は薄く保つ。リクエスト受信、DTO変換、認証・認可境界の解決、UseCaseまたは問い合わせ層への委譲、レスポンス返却に集中する。
|
|
69
69
|
|
|
70
70
|
```kotlin
|
|
71
71
|
// CORRECT - Controller は薄い
|
|
@@ -208,6 +208,19 @@ fun confirm(confirmedBy: String): OrderConfirmedEvent {
|
|
|
208
208
|
| 構造バリデーション(@NotBlank等)がドメインにある | REJECT。API層で |
|
|
209
209
|
| UseCase層のバリデーションがAggregate内にある | REJECT。Read Model参照はUseCase層 |
|
|
210
210
|
|
|
211
|
+
### 読み取りと書き込みの入口
|
|
212
|
+
|
|
213
|
+
読み取りと書き込みは入口で分離する。読み取り用の問い合わせ層は副作用を持たず、書き込みはコマンドまたはUseCaseで扱う。
|
|
214
|
+
|
|
215
|
+
| 基準 | 判定 |
|
|
216
|
+
|------|------|
|
|
217
|
+
| 問い合わせ層が保存・削除・外部呼び出し・コマンド送信を行う | REJECT |
|
|
218
|
+
| 読み取り用のクラス名やメソッド名なのに副作用を持つ | REJECT |
|
|
219
|
+
| 単純な参照APIが問い合わせ層を呼び、レスポンスDTOに変換するだけ | OK |
|
|
220
|
+
| 単純な状態変更APIが構造検証と認可境界の解決後にコマンドを1つ送るだけ | OK |
|
|
221
|
+
| 複数のRead Model参照、外部連携、複数コマンド、結果待機をControllerに置く | REJECT。UseCase層に分離 |
|
|
222
|
+
| UseCaseが別サービスへの薄い委譲だけでドメイン上の判断や調整を持たない | 削除を検討 |
|
|
223
|
+
|
|
211
224
|
## エラーハンドリング
|
|
212
225
|
|
|
213
226
|
### 例外階層設計
|
|
@@ -245,6 +258,17 @@ class OrderExceptionHandler {
|
|
|
245
258
|
| try-catch の空 catch | REJECT |
|
|
246
259
|
| Controller 内で例外を握りつぶして 200 を返す | REJECT |
|
|
247
260
|
|
|
261
|
+
### 例外変換のスコープ
|
|
262
|
+
|
|
263
|
+
HTTPステータスへの例外変換は、HTTP adapter 境界の例外変換レイヤに分離する。グローバルな変換は認証・入力検証・共通エラー形状など真に横断的なものに限り、特定 API やリソース固有の変換は、その API スコープに閉じた境界で扱う。
|
|
264
|
+
|
|
265
|
+
| 基準 | 判定 |
|
|
266
|
+
|------|------|
|
|
267
|
+
| 各 endpoint が同じ try-catch や wrapper で例外を HTTP 表現に変換している | REJECT。HTTP adapter 境界の例外変換レイヤに分離 |
|
|
268
|
+
| 特定 API 固有の例外変換を global handler に追加する | スコープ過大。対象 API の境界へ閉じる |
|
|
269
|
+
| 認証失敗、入力検証、共通エラー形状など全 API 共通の変換 | OK。global な境界で扱う |
|
|
270
|
+
| 例外型から HTTP 表現への変換が application/domain 層にある | REJECT。HTTP adapter 境界で扱う |
|
|
271
|
+
|
|
248
272
|
## ドメインモデル設計
|
|
249
273
|
|
|
250
274
|
### イミュータブル + require
|
|
@@ -361,6 +361,8 @@ UseCaseが必要なケース:
|
|
|
361
361
|
|
|
362
362
|
UseCaseが不要なケース:
|
|
363
363
|
- Controllerからコマンドを1つ送るだけで完結する単純な操作
|
|
364
|
+
- ControllerからQuery側へ問い合わせてレスポンスへ変換するだけの単純な参照
|
|
365
|
+
- 既存リソースの存在確認・スコープ確認後にコマンドを1つ送るだけの操作
|
|
364
366
|
|
|
365
367
|
| 基準 | 判定 |
|
|
366
368
|
|------|------|
|
|
@@ -368,6 +370,7 @@ UseCaseが不要なケース:
|
|
|
368
370
|
| UseCaseがHTTPリクエスト/レスポンスに依存 | REJECT。UseCaseはプロトコル非依存 |
|
|
369
371
|
| UseCaseがAggregate内部状態を直接変更 | REJECT。CommandGateway経由 |
|
|
370
372
|
| UseCaseがSubscription Queryで結果を待機 | REJECT。分散環境で動作しない。リアクティブポーリングを使う |
|
|
373
|
+
| UseCaseが別の問い合わせ層やコマンド送信への薄い委譲だけで終わる | 削除を検討 |
|
|
371
374
|
|
|
372
375
|
## プロジェクション設計
|
|
373
376
|
|
|
@@ -435,6 +438,19 @@ class InventoryReleaseHandler(private val commandGateway: CommandGateway) {
|
|
|
435
438
|
| EventHandler 内で Repository に save | REJECT。Projection に分離 |
|
|
436
439
|
| 1クラスに Projection と EventHandler の責務が混在 | REJECT。クラスを分離 |
|
|
437
440
|
|
|
441
|
+
### 外部処理の起動
|
|
442
|
+
|
|
443
|
+
外部ワーカーや非同期処理の起動は、Aggregate が確定したドメインイベントを起点にする。Application Service や Coordinator が、コマンド送信と外部副作用を同じ制御フローで束ねない。
|
|
444
|
+
|
|
445
|
+
| 基準 | 判定 |
|
|
446
|
+
|------|------|
|
|
447
|
+
| Application Service や Coordinator がコマンド送信直後に同じ状態遷移の外部処理を起動する | REJECT。確定済みイベントの EventHandler に分離 |
|
|
448
|
+
| Aggregate が生成開始・処理開始を表すイベントを発行し、EventHandler が外部処理を起動する | OK |
|
|
449
|
+
| 外部処理の起動失敗を EventHandler が失敗コマンドとして Aggregate に戻す | OK |
|
|
450
|
+
| 外部処理に必要な入力がイベントまたは安定したIDから再取得できるデータで表現されている | OK |
|
|
451
|
+
| 外部処理の入力がコマンド処理中のローカル変数にしか存在しない | REJECT。イベントまたは再取得可能な参照へ移す |
|
|
452
|
+
| 競合や補償を持たない単純な外部処理起動に Saga を使う | REJECT。EventHandler で十分 |
|
|
453
|
+
|
|
438
454
|
## Query側の設計
|
|
439
455
|
|
|
440
456
|
Query側はイベント駆動のPubSubモデルで動作する。Projection が EventHandler でRead Modelを更新し、Query側はRead Modelを参照する。
|
|
@@ -451,6 +467,9 @@ Query側はイベント駆動のPubSubモデルで動作する。Projection が
|
|
|
451
467
|
| Controller から Repository を直接参照 | REJECT。UseCase層を経由 |
|
|
452
468
|
| Query側が Command Model を参照 | REJECT |
|
|
453
469
|
| QueryHandler がコマンドを発行 | REJECT |
|
|
470
|
+
| Query側のサービスやハンドラが保存・削除・外部API呼び出しを行う | REJECT |
|
|
471
|
+
| Command と Query を同じサービスに混在させる | REJECT。責務と命名を分離 |
|
|
472
|
+
| Query側で存在確認やスコープ確認を行い、呼び出し元がコマンドを送る | OK |
|
|
454
473
|
|
|
455
474
|
レイヤー間の型:
|
|
456
475
|
- `application/query/` - Query結果の型(例: `OrderDetail`)
|
|
@@ -503,6 +522,18 @@ Aggregate → Event Bus → Projection(@EventHandler) → Repository(Read Model)
|
|
|
503
522
|
QueryHandler がここを参照
|
|
504
523
|
```
|
|
505
524
|
|
|
525
|
+
### 非同期コールバックと並行制御
|
|
526
|
+
|
|
527
|
+
非同期処理の完了通知は重複・遅延・順序逆転を前提に設計する。Controller や単一プロセス内のロックではなく、Aggregate の状態遷移とコマンドの冪等性で守る。
|
|
528
|
+
|
|
529
|
+
| 基準 | 判定 |
|
|
530
|
+
|------|------|
|
|
531
|
+
| Controllerやアプリケーションプロセス内のロックで重複callbackを防ぐ | REJECT。複数インスタンスで効かない |
|
|
532
|
+
| 処理中かどうかをAggregate状態で判断する | OK |
|
|
533
|
+
| callbackの試行IDや世代をAggregateが検証する | OK |
|
|
534
|
+
| 古いcallbackや重複callbackを状態遷移で冪等に無視する | OK |
|
|
535
|
+
| 並行制御がController、UseCase、Aggregateに重複して散らばる | REJECT |
|
|
536
|
+
|
|
506
537
|
## 結果整合性
|
|
507
538
|
|
|
508
539
|
コマンド発行後に同期的なレスポンスが必要な場合、リアクティブポーリングで Projection の更新を待機する。
|
|
@@ -17,6 +17,17 @@ E2Eテストはユーザーの操作フロー全体を検証する。ユニッ
|
|
|
17
17
|
| 複数コマンド/ページにまたがるシナリオ | E2Eテストが適切 |
|
|
18
18
|
| エラーメッセージの表示確認 | E2Eテストが適切 |
|
|
19
19
|
|
|
20
|
+
## 振る舞い観測
|
|
21
|
+
|
|
22
|
+
E2Eテストはユーザーから見える振る舞いを観測する。設定値、ログ、スナップショットの確認だけでは、実際の拒否、許可、隔離、回復が起きたことを保証しない。
|
|
23
|
+
|
|
24
|
+
| 基準 | 判定 |
|
|
25
|
+
|------|------|
|
|
26
|
+
| ユーザー操作や外部入力に対する結果が観測されている | OK |
|
|
27
|
+
| 拒否・エラー・リカバリーの経路で期待する結果が確認されている | OK |
|
|
28
|
+
| 設定や内部状態だけを確認してユーザー-visible な結果を見ていない | REJECT |
|
|
29
|
+
| 実外部環境に依存する確認しかなく、主要境界の deterministic test がない | 警告またはREJECT |
|
|
30
|
+
|
|
20
31
|
## UXルートの洗い出し
|
|
21
32
|
|
|
22
33
|
E2Eテストの網羅性はユーザー操作ルートの洗い出しに依存する。ドキュメントではなくコードから起点を特定する。
|
|
@@ -86,4 +97,3 @@ E2Eテストの網羅性を保証するため、テストケースをリスト
|
|
|
86
97
|
| 起点ごとに分類 | コマンド/ページ/エンドポイント単位でグループ化する |
|
|
87
98
|
| 優先度付け | ユーザー影響度 × 未テストのリスクで優先度を決定する |
|
|
88
99
|
| 既存テストとの突き合わせ | 新規追加前に既存テストのカバー範囲を確認する |
|
|
89
|
-
|
|
@@ -54,6 +54,17 @@ test('validates age at boundaries', () => {
|
|
|
54
54
|
})
|
|
55
55
|
```
|
|
56
56
|
|
|
57
|
+
## 振る舞い保証
|
|
58
|
+
|
|
59
|
+
ユニットテストは設定値や内部状態のスナップショットだけでなく、公開された契約が期待どおりに振る舞うことを検証する。拒否、許可、隔離、解放のような境界変更は、主要な成功/失敗ケースを deterministic に確認する。
|
|
60
|
+
|
|
61
|
+
| 基準 | 判定 |
|
|
62
|
+
|------|------|
|
|
63
|
+
| 期待する戻り値・例外・副作用が直接検証されている | OK |
|
|
64
|
+
| 境界変更の成功/失敗、許可/拒否の両側が検証されている | OK |
|
|
65
|
+
| 設定値や最後の内部状態だけを確認している | REJECT |
|
|
66
|
+
| 外部環境がないと主要な境界条件を再現できない | Fake や Stub による deterministic test を検討 |
|
|
67
|
+
|
|
57
68
|
## テストフィクスチャ設計
|
|
58
69
|
|
|
59
70
|
テストデータはファクトリ関数で管理する。
|
|
@@ -0,0 +1,31 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# AI生成コードレビュー
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{1文で結果を要約}
|
|
8
|
+
|
|
9
|
+
## 検証した項目
|
|
10
|
+
| 観点 | 結果 | 備考 |
|
|
11
|
+
|------|------|------|
|
|
12
|
+
| 仮定の妥当性 | ✅ | - |
|
|
13
|
+
| API/ライブラリの実在 | ✅ | - |
|
|
14
|
+
| コンテキスト適合 | ✅ | - |
|
|
15
|
+
| スコープ | ✅ | - |
|
|
16
|
+
|
|
17
|
+
期待される `family_tag` 値: `hallucination`, `unvalidated-assumption`, `off-by-one`, `api-mismatch`, `missing-edge-case`, `logic-error`, `scope-creep`.
|
|
18
|
+
structured raw findings を要求された場合は、この表の `family_tag` 値を structured output の `familyTag` フィールドへコピーする。
|
|
19
|
+
|
|
20
|
+
## 観測した指摘
|
|
21
|
+
| # | family_tag | カテゴリ | 重大度 | 場所 | 問題 | 修正案 |
|
|
22
|
+
|---|------------|---------|--------|------|------|--------|
|
|
23
|
+
| 1 | hallucination | 幻覚API | high / medium / low | `src/file.ts:23` | 存在しないメソッド | 実在APIへ置換 |
|
|
24
|
+
|
|
25
|
+
## REJECT判定条件
|
|
26
|
+
- ブロッキング指摘が1件以上ある場合のみ REJECT 可
|
|
27
|
+
```
|
|
28
|
+
|
|
29
|
+
**認知負荷軽減ルール:**
|
|
30
|
+
- 問題なし → サマリー + チェック表 + 空の指摘セクション(10行以内)
|
|
31
|
+
- 問題あり → 該当セクションのみ行追加(30行以内)
|
|
@@ -0,0 +1,35 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# アーキテクチャレビュー
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / IMPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{1-2文で結果を要約}
|
|
8
|
+
|
|
9
|
+
## 確認した観点
|
|
10
|
+
- [x] 構造・設計
|
|
11
|
+
- [x] コード品質
|
|
12
|
+
- [x] 変更スコープ
|
|
13
|
+
- [x] テストカバレッジ
|
|
14
|
+
- [x] デッドコード
|
|
15
|
+
- [x] 呼び出しチェーン検証
|
|
16
|
+
|
|
17
|
+
## 観測した指摘
|
|
18
|
+
| # | family_tag | スコープ | 重大度 | 場所 | 問題 | 修正案 |
|
|
19
|
+
|---|------------|---------|--------|------|------|--------|
|
|
20
|
+
| 1 | design-violation | スコープ内 | high / medium / low | `src/file.ts:42` | 問題の説明 | 修正方法 |
|
|
21
|
+
|
|
22
|
+
スコープ: 「スコープ内」(今回修正可能)/ 「スコープ外」(既存問題・非ブロッキング)
|
|
23
|
+
|
|
24
|
+
## 検証証跡
|
|
25
|
+
- ビルド: {確認対象・確認内容・結果。未確認ならその旨}
|
|
26
|
+
- テスト: {確認対象・確認内容・結果。未確認ならその旨}
|
|
27
|
+
- 動作確認: {確認対象・確認内容・結果。未確認ならその旨}
|
|
28
|
+
|
|
29
|
+
## REJECT判定条件
|
|
30
|
+
- スコープ内のブロッキング指摘が1件以上ある場合のみ REJECT 可
|
|
31
|
+
```
|
|
32
|
+
|
|
33
|
+
**認知負荷軽減ルール:**
|
|
34
|
+
- APPROVE → サマリーのみ(5行以内)
|
|
35
|
+
- REJECT → 該当指摘のみ表で記載(30行以内)
|
|
@@ -0,0 +1,25 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# コーディングレビュー
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{1-2文でレビュー結果を要約}
|
|
8
|
+
|
|
9
|
+
## 観測した指摘
|
|
10
|
+
| # | family_tag | 重大度 | 場所 | 問題 | 影響 | 修正案 |
|
|
11
|
+
|---|------------|--------|------|------|------|--------|
|
|
12
|
+
| 1 | bug | High / Medium / Low | `src/file.ts:42` | {問題} | {影響} | {修正案} |
|
|
13
|
+
|
|
14
|
+
## 検証証跡
|
|
15
|
+
- 差分確認: {確認内容}
|
|
16
|
+
- ビルド: {結果。未確認ならその旨}
|
|
17
|
+
- テスト: {結果。未確認ならその旨}
|
|
18
|
+
|
|
19
|
+
## REJECT判定条件
|
|
20
|
+
- ブロッキング指摘が1件以上ある場合のみ REJECT
|
|
21
|
+
```
|
|
22
|
+
|
|
23
|
+
**認知負荷軽減ルール:**
|
|
24
|
+
- APPROVE → サマリーのみ(5行以内)
|
|
25
|
+
- REJECT → 該当指摘のみ表で記載(30行以内)
|
|
@@ -6,6 +6,13 @@
|
|
|
6
6
|
## サマリー
|
|
7
7
|
{1-2文でレビュー結果を要約}
|
|
8
8
|
|
|
9
|
+
## 契約入口チェック
|
|
10
|
+
ID、名前、メタデータ、設定、環境変数、出力契約の追加・変更がある場合は記載する。
|
|
11
|
+
|
|
12
|
+
| 入口・経路 | 元要件 | 実装根拠 | テスト根拠 | 判定 | 例外・未確認の根拠 |
|
|
13
|
+
|-----------|--------|----------|------------|------|-------------------|
|
|
14
|
+
| {通常入口 / 派生条件 / 検証 / 評価 / 出力 / 再注入など} | {要件} | `src/file.ts:42` | `src/file.test.ts:10` | ✅/❌/⚠️ | {なし / 根拠} |
|
|
15
|
+
|
|
9
16
|
## 今回の指摘(new)
|
|
10
17
|
| # | finding_id | family_tag | 重大度 | 場所 | 問題 | 影響 | 修正案 |
|
|
11
18
|
|---|------------|------------|--------|------|------|------|--------|
|
|
@@ -17,9 +24,9 @@
|
|
|
17
24
|
| 1 | CODE-PERSIST-src-file-L77 | regression | `src/file.ts:77` | `src/file.ts:77` | {未解消の問題} | {修正案} |
|
|
18
25
|
|
|
19
26
|
## 解消済み(resolved)
|
|
20
|
-
| finding_id | 解消根拠 |
|
|
21
|
-
|
|
22
|
-
| CODE-RESOLVED-src-file-L10 | `src/file.ts:10` で解消 |
|
|
27
|
+
| finding_id | 元の期待結果 | 解消根拠 |
|
|
28
|
+
|------------|--------------|----------|
|
|
29
|
+
| CODE-RESOLVED-src-file-L10 | {元 finding の受入条件} | `src/file.ts:10` で解消 |
|
|
23
30
|
|
|
24
31
|
## 再開指摘(reopened)
|
|
25
32
|
| # | finding_id | family_tag | 解消根拠(前回) | 再発根拠 | 問題 | 修正案 |
|
|
@@ -37,5 +44,5 @@
|
|
|
37
44
|
```
|
|
38
45
|
|
|
39
46
|
**認知負荷軽減ルール:**
|
|
40
|
-
- APPROVE →
|
|
47
|
+
- APPROVE → サマリーと、必要な場合のみ契約入口チェック(合計10行以内)
|
|
41
48
|
- REJECT → 該当指摘のみ表で記載(30行以内)
|
|
@@ -0,0 +1,31 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# CQRS+ESレビュー
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{1-2文で結果を要約}
|
|
8
|
+
|
|
9
|
+
## 確認した観点
|
|
10
|
+
| 観点 | 結果 | 備考 |
|
|
11
|
+
|------|------|------|
|
|
12
|
+
| Aggregate設計 | ✅ | - |
|
|
13
|
+
| イベント設計 | ✅ | - |
|
|
14
|
+
| Command/Query分離 | ✅ | - |
|
|
15
|
+
| プロジェクション | ✅ | - |
|
|
16
|
+
| 結果整合性 | ✅ | - |
|
|
17
|
+
|
|
18
|
+
## 観測した指摘
|
|
19
|
+
| # | family_tag | 重大度 | スコープ | 場所 | 問題 | 影響 | 修正案 |
|
|
20
|
+
|---|------------|--------|---------|------|------|------|--------|
|
|
21
|
+
| 1 | cqrs-violation | High / Medium / Low | スコープ内 | `src/file.ts:42` | 問題の説明 | ドメイン整合性・保守性への影響 | 修正方法 |
|
|
22
|
+
|
|
23
|
+
スコープ: 「スコープ内」(今回修正可能)/ 「スコープ外」(既存問題・非ブロッキング)
|
|
24
|
+
|
|
25
|
+
## REJECT判定条件
|
|
26
|
+
- ブロッキング指摘が1件以上ある場合のみ REJECT
|
|
27
|
+
```
|
|
28
|
+
|
|
29
|
+
**認知負荷軽減ルール:**
|
|
30
|
+
- APPROVE → サマリーのみ(5行以内)
|
|
31
|
+
- REJECT → 該当指摘のみ表で記載(30行以内)
|
|
@@ -0,0 +1,45 @@
|
|
|
1
|
+
```json
|
|
2
|
+
{
|
|
3
|
+
"matches": [
|
|
4
|
+
{
|
|
5
|
+
"findingId": "F-0001",
|
|
6
|
+
"rawFindingIds": ["raw-1"],
|
|
7
|
+
"evidence": "同一問題と判断した根拠"
|
|
8
|
+
}
|
|
9
|
+
],
|
|
10
|
+
"newFindings": [
|
|
11
|
+
{
|
|
12
|
+
"rawFindingIds": ["raw-2"],
|
|
13
|
+
"title": "新規指摘の短い題名",
|
|
14
|
+
"severity": "high"
|
|
15
|
+
}
|
|
16
|
+
],
|
|
17
|
+
"resolvedFindings": [
|
|
18
|
+
{
|
|
19
|
+
"findingId": "F-0002",
|
|
20
|
+
"rawFindingIds": ["raw-previous-1"],
|
|
21
|
+
"evidence": "解消を確認した根拠"
|
|
22
|
+
}
|
|
23
|
+
],
|
|
24
|
+
"reopenedFindings": [
|
|
25
|
+
{
|
|
26
|
+
"findingId": "F-0003",
|
|
27
|
+
"rawFindingIds": ["raw-3"],
|
|
28
|
+
"evidence": "再発と判断した根拠"
|
|
29
|
+
}
|
|
30
|
+
],
|
|
31
|
+
"conflicts": [
|
|
32
|
+
{
|
|
33
|
+
"findingIds": [],
|
|
34
|
+
"rawFindingIds": ["raw-4", "raw-5"],
|
|
35
|
+
"description": "レビュワー間の矛盾"
|
|
36
|
+
}
|
|
37
|
+
],
|
|
38
|
+
"resolvedConflicts": [
|
|
39
|
+
{
|
|
40
|
+
"conflictId": "C-012345ABCDEF",
|
|
41
|
+
"evidence": "調停済みと判断した根拠"
|
|
42
|
+
}
|
|
43
|
+
]
|
|
44
|
+
}
|
|
45
|
+
```
|
|
@@ -0,0 +1,30 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# フロントエンドレビュー
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{1-2文で結果を要約}
|
|
8
|
+
|
|
9
|
+
## 確認した観点
|
|
10
|
+
| 観点 | 結果 | 備考 |
|
|
11
|
+
|------|------|------|
|
|
12
|
+
| コンポーネント設計 | ✅ | - |
|
|
13
|
+
| 状態管理 | ✅ | - |
|
|
14
|
+
| 正規状態と派生状態 | ✅ | - |
|
|
15
|
+
| パフォーマンス | ✅ | - |
|
|
16
|
+
| アクセシビリティ | ✅ | - |
|
|
17
|
+
| 型安全性 | ✅ | - |
|
|
18
|
+
|
|
19
|
+
## 観測した指摘
|
|
20
|
+
| # | family_tag | 重大度 | 場所 | 問題 | 影響 | 修正案 |
|
|
21
|
+
|---|------------|--------|------|------|------|--------|
|
|
22
|
+
| 1 | component-design | High / Medium / Low | `src/file.tsx:42` | 問題の説明 | ユーザー体験・保守性への影響 | 修正方法 |
|
|
23
|
+
|
|
24
|
+
## REJECT判定条件
|
|
25
|
+
- ブロッキング指摘が1件以上ある場合のみ REJECT
|
|
26
|
+
```
|
|
27
|
+
|
|
28
|
+
**認知負荷軽減ルール:**
|
|
29
|
+
- APPROVE → サマリーのみ(5行以内)
|
|
30
|
+
- REJECT → 該当指摘のみ表で記載(30行以内)
|
|
@@ -19,6 +19,11 @@
|
|
|
19
19
|
|------|------|----------|
|
|
20
20
|
| {契約種別} | {対象} | {守る内容} |
|
|
21
21
|
|
|
22
|
+
## 置換された旧仕様
|
|
23
|
+
| 旧仕様 | 新仕様 | 削除対象 | テスト方針 |
|
|
24
|
+
|--------|--------|----------|------------|
|
|
25
|
+
| {旧仕様} | {新仕様} | {削除する実装/テスト} | {新仕様を検証する場所} |
|
|
26
|
+
|
|
22
27
|
## 不要変更の確認
|
|
23
28
|
| 確認対象 | 結果 | 備考 |
|
|
24
29
|
|----------|------|------|
|
|
@@ -0,0 +1,45 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# Pure Review
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{この修正がマージ可能な品質かを1-2文で要約。REJECT の場合は最大のブロッカーを先に書く}
|
|
8
|
+
|
|
9
|
+
## マージ可否チェック
|
|
10
|
+
| # | 観点 | 状態 | 根拠(ファイル:行 / テスト / 実行証跡) | コメント |
|
|
11
|
+
|---|------|------|---------------------------------------|----------|
|
|
12
|
+
| 1 | 要求充足 | 充足 / 未充足 / 未確認 | `src/file.ts:42` | {補足} |
|
|
13
|
+
| 2 | 既存契約・既存フローへの影響 | 問題なし / 問題あり / 未確認 | `src/file.ts:42` | {補足} |
|
|
14
|
+
| 3 | テスト・検証 | 十分 / 不足 / 未確認 | `npm test` | {補足} |
|
|
15
|
+
| 4 | 要求外変更・スコープクリープ | 問題なし / 問題あり / 未確認 | `src/file.ts:42` | {補足} |
|
|
16
|
+
| 5 | 明白なセキュリティ・データ保護・運用リスク | 問題なし / 問題あり / 未確認 | `src/file.ts:42` | {補足} |
|
|
17
|
+
|
|
18
|
+
## 要求照合
|
|
19
|
+
| # | 要求(タスクから抽出) | 状態 | 根拠(ファイル:行) | コメント |
|
|
20
|
+
|---|-------------------|------|-------------------|----------|
|
|
21
|
+
| 1 | {要件1} | 充足 / 未充足 / 未確認 | `src/file.ts:42` | {補足} |
|
|
22
|
+
|
|
23
|
+
## 要求外変更・既存影響
|
|
24
|
+
| # | 変更 | ファイル | 判定 | コメント |
|
|
25
|
+
|---|------|---------|------|----------|
|
|
26
|
+
| 1 | {要求にない変更または既存影響} | `src/file.ts` | 妥当 / 要再確認 / 不要 / 問題あり | {理由} |
|
|
27
|
+
|
|
28
|
+
## 観測した指摘
|
|
29
|
+
| # | family_tag | カテゴリ | 重大度 | 場所 | 問題 | 修正案 |
|
|
30
|
+
|---|------------|---------|--------|------|------|--------|
|
|
31
|
+
| 1 | mergeability | 回帰 / 要求未充足 / テスト不足 / 契約破壊 / スコープクリープ | high / medium / low | `src/file.ts:42` | 問題の説明 | 修正方法 |
|
|
32
|
+
|
|
33
|
+
## 検証証跡
|
|
34
|
+
- ビルド: {確認対象・確認内容・結果。未確認ならその旨}
|
|
35
|
+
- テスト: {確認対象・確認内容・結果。未確認ならその旨}
|
|
36
|
+
- 動作確認: {確認対象・確認内容・結果。未確認ならその旨}
|
|
37
|
+
|
|
38
|
+
## REJECT判定条件
|
|
39
|
+
- マージを止めるべき観測指摘が1件以上ある場合は REJECT
|
|
40
|
+
- 未確認項目は、それがマージ可否に影響する場合のみ REJECT 根拠にする
|
|
41
|
+
```
|
|
42
|
+
|
|
43
|
+
**認知負荷軽減ルール:**
|
|
44
|
+
- APPROVE → サマリー + マージ可否チェックのみ(10行以内)
|
|
45
|
+
- REJECT → ブロッカー指摘を優先(40行以内)
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# Pure Review
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{この修正がマージ可能な品質かを1-2文で要約。REJECT の場合は最大のブロッカーを先に書く}
|
|
8
|
+
|
|
9
|
+
## マージ可否チェック
|
|
10
|
+
| # | 観点 | 状態 | 根拠(ファイル:行 / テスト / 実行証跡) | コメント |
|
|
11
|
+
|---|------|------|---------------------------------------|----------|
|
|
12
|
+
| 1 | 要求充足 | 充足 / 未充足 / 未確認 | `src/file.ts:42` | {補足} |
|
|
13
|
+
| 2 | 既存契約・既存フローへの影響 | 問題なし / 問題あり / 未確認 | `src/file.ts:42` | {補足} |
|
|
14
|
+
| 3 | テスト・検証 | 十分 / 不足 / 未確認 | `npm test` | {補足} |
|
|
15
|
+
| 4 | 要求外変更・スコープクリープ | 問題なし / 問題あり / 未確認 | `src/file.ts:42` | {補足} |
|
|
16
|
+
| 5 | 明白なセキュリティ・データ保護・運用リスク | 問題なし / 問題あり / 未確認 | `src/file.ts:42` | {補足} |
|
|
17
|
+
|
|
18
|
+
## 要求照合
|
|
19
|
+
| # | 要求(タスクから抽出) | 元要件の出典 | 状態 | 根拠(ファイル:行) | 例外・未確認の根拠 |
|
|
20
|
+
|---|-------------------|--------------|------|-------------------|-------------------|
|
|
21
|
+
| 1 | {要件1} | `order.md:10` | 充足 / 未充足 / 未確認 | `src/file.ts:42` | {なし / 根拠} |
|
|
22
|
+
|
|
23
|
+
## 要求外変更・既存影響
|
|
24
|
+
| # | 変更 | ファイル | 判定 | コメント |
|
|
25
|
+
|---|------|---------|------|----------|
|
|
26
|
+
| 1 | {要求にない変更または既存影響} | `src/file.ts` | 妥当 / 要再確認 / 不要 / 問題あり | {理由} |
|
|
27
|
+
|
|
28
|
+
## 今回の指摘(new)
|
|
29
|
+
| # | finding_id | family_tag | カテゴリ | 場所 | 問題 | 修正案 |
|
|
30
|
+
|---|------------|------------|---------|------|------|--------|
|
|
31
|
+
| 1 | PURE-NEW-src-file-L42 | mergeability | 回帰 / 要求未充足 / テスト不足 / 契約破壊 / スコープクリープ | `src/file.ts:42` | 問題の説明 | 修正方法 |
|
|
32
|
+
|
|
33
|
+
## 継続指摘(persists)
|
|
34
|
+
| # | finding_id | family_tag | 前回根拠 | 今回根拠 | 問題 | 修正案 |
|
|
35
|
+
|---|------------|------------|----------|----------|------|--------|
|
|
36
|
+
| 1 | PURE-PERSIST-src-file-L77 | mergeability | `file:line` | `file:line` | 未解消 | 修正方法 |
|
|
37
|
+
|
|
38
|
+
## 解消済み(resolved)
|
|
39
|
+
| finding_id | 元の期待結果 | 解消根拠 |
|
|
40
|
+
|------------|--------------|----------|
|
|
41
|
+
| PURE-RESOLVED-src-file-L10 | {元 finding の受入条件} | `file:line` で解消 |
|
|
42
|
+
|
|
43
|
+
## 再開指摘(reopened)
|
|
44
|
+
| # | finding_id | family_tag | 解消根拠(前回) | 再発根拠 | 問題 | 修正案 |
|
|
45
|
+
|---|------------|------------|----------------|---------|------|--------|
|
|
46
|
+
| 1 | PURE-REOPENED-src-file-L55 | mergeability | `前回: file:line で修正済み` | `file:line で再発` | 問題の説明 | 修正方法 |
|
|
47
|
+
|
|
48
|
+
## 検証証跡
|
|
49
|
+
- ビルド: {確認対象・確認内容・結果。未確認ならその旨}
|
|
50
|
+
- テスト: {確認対象・確認内容・結果。未確認ならその旨}
|
|
51
|
+
- 動作確認: {確認対象・確認内容・結果。未確認ならその旨}
|
|
52
|
+
|
|
53
|
+
## REJECT判定条件
|
|
54
|
+
- マージを止めるべき問題が `new`、`persists`、または `reopened` に1件以上ある場合は REJECT
|
|
55
|
+
- `finding_id` なしの指摘は無効
|
|
56
|
+
- 未確認項目は、それがマージ可否に影響する場合のみ REJECT 根拠にする
|
|
57
|
+
```
|
|
58
|
+
|
|
59
|
+
**認知負荷軽減ルール:**
|
|
60
|
+
- APPROVE → サマリー + マージ可否チェックのみ(10行以内)
|
|
61
|
+
- REJECT → ブロッカー指摘を優先(40行以内)
|
|
@@ -0,0 +1,35 @@
|
|
|
1
|
+
```markdown
|
|
2
|
+
# QAレビュー
|
|
3
|
+
|
|
4
|
+
## 結果: APPROVE / REJECT
|
|
5
|
+
|
|
6
|
+
## サマリー
|
|
7
|
+
{1-2文で結果を要約}
|
|
8
|
+
|
|
9
|
+
## 確認した観点
|
|
10
|
+
| 観点 | 結果 | 備考 |
|
|
11
|
+
|------|------|------|
|
|
12
|
+
| テストカバレッジ | ✅ | - |
|
|
13
|
+
| テスト品質 | ✅ | - |
|
|
14
|
+
| エラーハンドリング | ✅ | - |
|
|
15
|
+
| ドキュメント | ✅ | - |
|
|
16
|
+
| 保守性 | ✅ | - |
|
|
17
|
+
|
|
18
|
+
## 観測した指摘
|
|
19
|
+
| # | family_tag | カテゴリ | 重大度 | 場所 | 問題 | 修正案 |
|
|
20
|
+
|---|------------|---------|--------|------|------|--------|
|
|
21
|
+
| 1 | test-coverage | テスト | high / medium / low | `src/test.ts:42` | 異常系テスト不足 | 失敗系ケースを追加 |
|
|
22
|
+
|
|
23
|
+
## 検証証跡
|
|
24
|
+
- ビルド: {確認対象・確認内容・結果。未確認ならその旨}
|
|
25
|
+
- テスト: {確認対象・確認内容・結果。未確認ならその旨}
|
|
26
|
+
- 動作確認: {確認対象・確認内容・結果。未確認ならその旨}
|
|
27
|
+
|
|
28
|
+
## 未確認範囲
|
|
29
|
+
| 項目 | 理由 | 判定への影響 |
|
|
30
|
+
|------|------|--------------|
|
|
31
|
+
| {未確認の範囲。なければ「なし」} | {未確認の理由} | {APPROVE可 / REJECT理由} |
|
|
32
|
+
|
|
33
|
+
## REJECT判定条件
|
|
34
|
+
- ブロッキング指摘が1件以上ある場合のみ REJECT 可
|
|
35
|
+
```
|
|
@@ -15,6 +15,13 @@
|
|
|
15
15
|
| ドキュメント | ✅ | - |
|
|
16
16
|
| 保守性 | ✅ | - |
|
|
17
17
|
|
|
18
|
+
## 契約入口チェック
|
|
19
|
+
ID、名前、メタデータ、設定、環境変数、出力契約の追加・変更がある場合は記載する。
|
|
20
|
+
|
|
21
|
+
| 入口・経路 | 元要件 | 実装根拠 | テスト根拠 | 判定 | 例外・未確認の根拠 |
|
|
22
|
+
|-----------|--------|----------|------------|------|-------------------|
|
|
23
|
+
| {通常入口 / 派生条件 / 検証 / 評価 / 出力 / 再注入など} | {要件} | `src/file.ts:42` | `src/file.test.ts:10` | ✅/❌/⚠️ | {なし / 根拠} |
|
|
24
|
+
|
|
18
25
|
## 今回の指摘(new)
|
|
19
26
|
| # | finding_id | family_tag | カテゴリ | 場所 | 問題 | 修正案 |
|
|
20
27
|
|---|------------|------------|---------|------|------|--------|
|
|
@@ -26,9 +33,9 @@
|
|
|
26
33
|
| 1 | QA-PERSIST-src-test-L77 | test-coverage | `src/test.ts:77` | `src/test.ts:77` | 不安定なまま | アサーションとセットアップを安定化 |
|
|
27
34
|
|
|
28
35
|
## 解消済み(resolved)
|
|
29
|
-
| finding_id | 解消根拠 |
|
|
30
|
-
|
|
31
|
-
| QA-RESOLVED-src-test-L10 | `src/test.ts:10` で異常系が網羅済み |
|
|
36
|
+
| finding_id | 元の期待結果 | 解消根拠 |
|
|
37
|
+
|------------|--------------|----------|
|
|
38
|
+
| QA-RESOLVED-src-test-L10 | {元 finding の受入条件} | `src/test.ts:10` で異常系が網羅済み |
|
|
32
39
|
|
|
33
40
|
## 再開指摘(reopened)
|
|
34
41
|
| # | finding_id | family_tag | 解消根拠(前回) | 再発根拠 | 問題 | 修正案 |
|
|
@@ -40,6 +47,11 @@
|
|
|
40
47
|
- テスト: {確認対象・確認内容・結果。未確認ならその旨}
|
|
41
48
|
- 動作確認: {確認対象・確認内容・結果。未確認ならその旨}
|
|
42
49
|
|
|
50
|
+
## 未確認範囲
|
|
51
|
+
| 項目 | 理由 | 判定への影響 |
|
|
52
|
+
|------|------|--------------|
|
|
53
|
+
| {未確認の範囲。なければ「なし」} | {未確認の理由} | {APPROVE可 / REJECT理由} |
|
|
54
|
+
|
|
43
55
|
## REJECT判定条件
|
|
44
56
|
- `new`、`persists`、または `reopened` が1件以上ある場合のみ REJECT 可
|
|
45
57
|
- `finding_id` なしの指摘は無効
|