takt 0.32.0 → 0.32.2
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/builtins/en/facets/instructions/e2e-coverage-implement.md +26 -0
- package/builtins/en/facets/instructions/e2e-coverage-plan.md +38 -0
- package/builtins/en/facets/instructions/e2e-coverage-supervise.md +21 -0
- package/builtins/en/facets/instructions/fix.md +4 -0
- package/builtins/en/facets/instructions/loop-monitor-ai-fix.md +4 -3
- package/builtins/en/facets/instructions/loop-monitor-reviewers-fix.md +4 -2
- package/builtins/en/facets/instructions/plan.md +2 -0
- package/builtins/en/facets/instructions/review-frontend.md +9 -0
- package/builtins/en/facets/instructions/security-audit-plan.md +12 -0
- package/builtins/en/facets/instructions/security-audit-review.md +22 -0
- package/builtins/en/facets/instructions/security-audit-supervise.md +20 -0
- package/builtins/en/facets/instructions/security-audit-team-leader.md +27 -0
- package/builtins/en/facets/knowledge/e2e-testing.md +89 -0
- package/builtins/en/facets/knowledge/frontend.md +46 -0
- package/builtins/en/facets/knowledge/react.md +90 -0
- package/builtins/en/facets/knowledge/unit-testing.md +108 -0
- package/builtins/en/facets/output-contracts/e2e-coverage-plan.md +33 -0
- package/builtins/en/facets/output-contracts/plan-frontend.md +41 -0
- package/builtins/en/facets/output-contracts/plan.md +8 -0
- package/builtins/en/facets/output-contracts/security-audit.md +31 -0
- package/builtins/en/facets/personas/coder.md +1 -0
- package/builtins/en/facets/personas/frontend-reviewer.md +4 -0
- package/builtins/en/facets/policies/ai-antipattern.md +43 -0
- package/builtins/en/facets/policies/coding.md +90 -1
- package/builtins/en/facets/policies/design-fidelity.md +51 -0
- package/builtins/en/facets/policies/design-planning.md +52 -0
- package/builtins/en/facets/policies/qa.md +15 -0
- package/builtins/en/facets/policies/testing.md +54 -1
- package/builtins/en/piece-categories.yaml +3 -2
- package/builtins/en/pieces/backend-cqrs.yaml +5 -0
- package/builtins/en/pieces/backend.yaml +5 -0
- package/builtins/en/pieces/default.yaml +2 -0
- package/builtins/en/pieces/dual-cqrs-mini.yaml +8 -1
- package/builtins/en/pieces/dual-cqrs.yaml +10 -2
- package/builtins/en/pieces/dual-mini.yaml +8 -1
- package/builtins/en/pieces/dual.yaml +14 -2
- package/builtins/en/pieces/{e2e-test.yaml → fill-e2e.yaml} +41 -61
- package/builtins/en/pieces/{unit-test.yaml → fill-unit.yaml} +12 -2
- package/builtins/en/pieces/frontend-mini.yaml +8 -1
- package/builtins/en/pieces/frontend.yaml +25 -3
- package/builtins/en/pieces/review-default.yaml +3 -0
- package/builtins/en/pieces/review-dual-cqrs.yaml +3 -1
- package/builtins/en/pieces/review-dual.yaml +3 -1
- package/builtins/en/pieces/review-fix-default.yaml +3 -0
- package/builtins/en/pieces/review-fix-dual-cqrs.yaml +5 -1
- package/builtins/en/pieces/review-fix-dual.yaml +5 -1
- package/builtins/en/pieces/review-fix-frontend.yaml +5 -1
- package/builtins/en/pieces/review-fix-takt-default.yaml +5 -2
- package/builtins/en/pieces/review-frontend.yaml +3 -1
- package/builtins/en/pieces/review-takt-default.yaml +3 -0
- package/builtins/en/pieces/security-audit.yaml +68 -0
- package/builtins/en/pieces/takt-default.yaml +7 -2
- package/builtins/en/pieces/terraform.yaml +0 -5
- package/builtins/ja/INSTRUCTION_STYLE_GUIDE.md +9 -10
- package/builtins/ja/KNOWLEDGE_STYLE_GUIDE.md +4 -4
- package/builtins/ja/OUTPUT_CONTRACT_STYLE_GUIDE.md +4 -4
- package/builtins/ja/PERSONA_STYLE_GUIDE.md +8 -8
- package/builtins/ja/POLICY_STYLE_GUIDE.md +5 -5
- package/builtins/ja/STYLE_GUIDE.md +8 -26
- package/builtins/ja/facets/instructions/e2e-coverage-implement.md +26 -0
- package/builtins/ja/facets/instructions/e2e-coverage-plan.md +38 -0
- package/builtins/ja/facets/instructions/e2e-coverage-supervise.md +21 -0
- package/builtins/ja/facets/instructions/fix.md +4 -0
- package/builtins/ja/facets/instructions/loop-monitor-ai-fix.md +4 -3
- package/builtins/ja/facets/instructions/loop-monitor-reviewers-fix.md +4 -2
- package/builtins/ja/facets/instructions/plan.md +2 -0
- package/builtins/ja/facets/instructions/review-frontend.md +9 -0
- package/builtins/ja/facets/instructions/security-audit-plan.md +12 -0
- package/builtins/ja/facets/instructions/security-audit-review.md +22 -0
- package/builtins/ja/facets/instructions/security-audit-supervise.md +20 -0
- package/builtins/ja/facets/instructions/security-audit-team-leader.md +27 -0
- package/builtins/ja/facets/knowledge/e2e-testing.md +89 -0
- package/builtins/ja/facets/knowledge/frontend.md +46 -0
- package/builtins/ja/facets/knowledge/react.md +90 -0
- package/builtins/ja/facets/knowledge/unit-testing.md +108 -0
- package/builtins/ja/facets/output-contracts/e2e-coverage-plan.md +33 -0
- package/builtins/ja/facets/output-contracts/plan-frontend.md +41 -0
- package/builtins/ja/facets/output-contracts/plan.md +8 -0
- package/builtins/ja/facets/output-contracts/security-audit.md +31 -0
- package/builtins/ja/facets/personas/coder.md +1 -0
- package/builtins/ja/facets/personas/frontend-reviewer.md +2 -0
- package/builtins/ja/facets/policies/ai-antipattern.md +43 -0
- package/builtins/ja/facets/policies/coding.md +90 -1
- package/builtins/ja/facets/policies/design-fidelity.md +51 -0
- package/builtins/ja/facets/policies/design-planning.md +52 -0
- package/builtins/ja/facets/policies/qa.md +15 -0
- package/builtins/ja/facets/policies/testing.md +54 -1
- package/builtins/ja/piece-categories.yaml +3 -2
- package/builtins/ja/pieces/backend-cqrs.yaml +5 -0
- package/builtins/ja/pieces/backend.yaml +5 -0
- package/builtins/ja/pieces/default.yaml +2 -0
- package/builtins/ja/pieces/dual-cqrs-mini.yaml +8 -1
- package/builtins/ja/pieces/dual-cqrs.yaml +10 -2
- package/builtins/ja/pieces/dual-mini.yaml +8 -1
- package/builtins/ja/pieces/dual.yaml +14 -2
- package/builtins/ja/pieces/{e2e-test.yaml → fill-e2e.yaml} +40 -60
- package/builtins/ja/pieces/{unit-test.yaml → fill-unit.yaml} +12 -2
- package/builtins/ja/pieces/frontend-mini.yaml +8 -1
- package/builtins/ja/pieces/frontend.yaml +25 -3
- package/builtins/ja/pieces/review-default.yaml +3 -0
- package/builtins/ja/pieces/review-dual-cqrs.yaml +3 -1
- package/builtins/ja/pieces/review-dual.yaml +3 -1
- package/builtins/ja/pieces/review-fix-default.yaml +3 -0
- package/builtins/ja/pieces/review-fix-dual-cqrs.yaml +5 -1
- package/builtins/ja/pieces/review-fix-dual.yaml +5 -1
- package/builtins/ja/pieces/review-fix-frontend.yaml +5 -1
- package/builtins/ja/pieces/review-fix-takt-default.yaml +5 -2
- package/builtins/ja/pieces/review-frontend.yaml +3 -1
- package/builtins/ja/pieces/review-takt-default.yaml +3 -0
- package/builtins/ja/pieces/security-audit.yaml +68 -0
- package/builtins/ja/pieces/takt-default.yaml +7 -2
- package/builtins/ja/pieces/terraform.yaml +0 -5
- package/dist/app/cli/routing.js +1 -1
- package/dist/app/cli/routing.js.map +1 -1
- package/dist/core/models/config-types.d.ts +4 -0
- package/dist/core/models/config-types.d.ts.map +1 -1
- package/dist/core/models/schemas.d.ts +4 -0
- package/dist/core/models/schemas.d.ts.map +1 -1
- package/dist/core/models/schemas.js +4 -0
- package/dist/core/models/schemas.js.map +1 -1
- package/dist/core/piece/engine/MovementExecutor.d.ts +1 -0
- package/dist/core/piece/engine/MovementExecutor.d.ts.map +1 -1
- package/dist/core/piece/engine/MovementExecutor.js +8 -4
- package/dist/core/piece/engine/MovementExecutor.js.map +1 -1
- package/dist/core/piece/engine/OptionsBuilder.d.ts.map +1 -1
- package/dist/core/piece/engine/OptionsBuilder.js +4 -1
- package/dist/core/piece/engine/OptionsBuilder.js.map +1 -1
- package/dist/features/config/deploySkillInternal.d.ts.map +1 -1
- package/dist/features/config/deploySkillInternal.js +2 -6
- package/dist/features/config/deploySkillInternal.js.map +1 -1
- package/dist/features/interactive/conversationLoop.d.ts.map +1 -1
- package/dist/features/interactive/conversationLoop.js +4 -15
- package/dist/features/interactive/conversationLoop.js.map +1 -1
- package/dist/features/pipeline/steps.d.ts.map +1 -1
- package/dist/features/pipeline/steps.js +5 -1
- package/dist/features/pipeline/steps.js.map +1 -1
- package/dist/features/tasks/execute/resolveTask.d.ts.map +1 -1
- package/dist/features/tasks/execute/resolveTask.js +11 -3
- package/dist/features/tasks/execute/resolveTask.js.map +1 -1
- package/dist/infra/config/global/globalConfigCore.d.ts.map +1 -1
- package/dist/infra/config/global/globalConfigCore.js +11 -8
- 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 +6 -0
- package/dist/infra/config/global/globalConfigSerializer.js.map +1 -1
- package/dist/infra/config/pathExpansion.d.ts +3 -0
- package/dist/infra/config/pathExpansion.d.ts.map +1 -0
- package/dist/infra/config/pathExpansion.js +15 -0
- package/dist/infra/config/pathExpansion.js.map +1 -0
- package/dist/infra/config/project/projectConfig.d.ts.map +1 -1
- package/dist/infra/config/project/projectConfig.js +15 -2
- package/dist/infra/config/project/projectConfig.js.map +1 -1
- package/dist/infra/config/resolveConfigValue.d.ts.map +1 -1
- package/dist/infra/config/resolveConfigValue.js +4 -1
- package/dist/infra/config/resolveConfigValue.js.map +1 -1
- package/dist/infra/cursor/client.js +1 -1
- package/dist/infra/cursor/client.js.map +1 -1
- package/dist/infra/github/pr.d.ts.map +1 -1
- package/dist/infra/github/pr.js +36 -8
- package/dist/infra/github/pr.js.map +1 -1
- package/dist/infra/resources/index.d.ts +5 -6
- package/dist/infra/resources/index.d.ts.map +1 -1
- package/dist/infra/resources/index.js +5 -6
- package/dist/infra/resources/index.js.map +1 -1
- package/dist/infra/task/autoCommit.d.ts.map +1 -1
- package/dist/infra/task/autoCommit.js +5 -1
- package/dist/infra/task/autoCommit.js.map +1 -1
- package/dist/infra/task/clone.d.ts +2 -1
- package/dist/infra/task/clone.d.ts.map +1 -1
- package/dist/infra/task/clone.js +5 -2
- package/dist/infra/task/clone.js.map +1 -1
- package/dist/infra/task/git.d.ts +5 -1
- package/dist/infra/task/git.d.ts.map +1 -1
- package/dist/infra/task/git.js +51 -3
- package/dist/infra/task/git.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/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/pathBoundary.d.ts +2 -0
- package/dist/shared/utils/pathBoundary.d.ts.map +1 -0
- package/dist/shared/utils/pathBoundary.js +10 -0
- package/dist/shared/utils/pathBoundary.js.map +1 -0
- package/package.json +2 -2
- package/builtins/en/facets/instructions/implement-e2e-test.md +0 -51
- package/builtins/en/facets/instructions/plan-e2e-test.md +0 -11
- package/builtins/en/templates/instructions/ai-fix.md +0 -74
- package/builtins/en/templates/instructions/ai-review-standalone.md +0 -47
- package/builtins/en/templates/instructions/arbitrate.md +0 -45
- package/builtins/en/templates/instructions/architect.md +0 -48
- package/builtins/en/templates/instructions/fix.md +0 -86
- package/builtins/en/templates/instructions/implement.md +0 -102
- package/builtins/en/templates/instructions/plan.md +0 -55
- package/builtins/en/templates/instructions/review.md +0 -101
- package/builtins/en/templates/instructions/supervise.md +0 -106
- package/builtins/en/templates/personas/character.md +0 -45
- package/builtins/en/templates/personas/expert.md +0 -68
- package/builtins/en/templates/personas/simple.md +0 -22
- package/builtins/en/templates/policies/policy.md +0 -49
- package/builtins/en/templates/reports/architecture-design.md +0 -31
- package/builtins/en/templates/reports/plan.md +0 -70
- package/builtins/en/templates/reports/review.md +0 -143
- package/builtins/en/templates/reports/security-review.md +0 -43
- package/builtins/en/templates/reports/summary.md +0 -52
- package/builtins/en/templates/reports/validation.md +0 -31
- package/builtins/ja/facets/instructions/implement-e2e-test.md +0 -51
- package/builtins/ja/facets/instructions/plan-e2e-test.md +0 -11
- package/builtins/ja/templates/instructions/ai-fix.md +0 -74
- package/builtins/ja/templates/instructions/ai-review-standalone.md +0 -47
- package/builtins/ja/templates/instructions/arbitrate.md +0 -45
- package/builtins/ja/templates/instructions/architect.md +0 -48
- package/builtins/ja/templates/instructions/fix.md +0 -86
- package/builtins/ja/templates/instructions/implement.md +0 -102
- package/builtins/ja/templates/instructions/plan.md +0 -55
- package/builtins/ja/templates/instructions/review.md +0 -101
- package/builtins/ja/templates/instructions/supervise.md +0 -106
- package/builtins/ja/templates/knowledge/knowledge.md +0 -39
- package/builtins/ja/templates/output-contracts/architecture-design.md +0 -31
- package/builtins/ja/templates/output-contracts/plan.md +0 -70
- package/builtins/ja/templates/output-contracts/review.md +0 -143
- package/builtins/ja/templates/output-contracts/security-review.md +0 -43
- package/builtins/ja/templates/output-contracts/summary.md +0 -52
- package/builtins/ja/templates/output-contracts/validation.md +0 -31
- package/builtins/ja/templates/personas/character.md +0 -43
- package/builtins/ja/templates/personas/expert.md +0 -21
- package/builtins/ja/templates/personas/simple.md +0 -22
- package/builtins/ja/templates/policies/policy.md +0 -49
|
@@ -12,7 +12,7 @@ loop_monitors:
|
|
|
12
12
|
- cycle:
|
|
13
13
|
- ai_review
|
|
14
14
|
- ai_fix
|
|
15
|
-
threshold:
|
|
15
|
+
threshold: 5
|
|
16
16
|
judge:
|
|
17
17
|
persona: supervisor
|
|
18
18
|
instruction: loop-monitor-ai-fix
|
|
@@ -24,7 +24,7 @@ loop_monitors:
|
|
|
24
24
|
- cycle:
|
|
25
25
|
- reviewers
|
|
26
26
|
- fix
|
|
27
|
-
threshold:
|
|
27
|
+
threshold: 5
|
|
28
28
|
judge:
|
|
29
29
|
persona: supervisor
|
|
30
30
|
instruction: loop-monitor-reviewers-fix
|
|
@@ -142,6 +142,9 @@ movements:
|
|
|
142
142
|
policy:
|
|
143
143
|
- review
|
|
144
144
|
- testing
|
|
145
|
+
knowledge:
|
|
146
|
+
- unit-testing
|
|
147
|
+
- e2e-testing
|
|
145
148
|
provider_options:
|
|
146
149
|
claude:
|
|
147
150
|
allowed_tools:
|
|
@@ -0,0 +1,68 @@
|
|
|
1
|
+
name: security-audit
|
|
2
|
+
description: Full security audit. Reads every project file one by one for security review
|
|
3
|
+
max_movements: 15
|
|
4
|
+
initial_movement: plan
|
|
5
|
+
|
|
6
|
+
loop_monitors:
|
|
7
|
+
- cycle: [supervise, review]
|
|
8
|
+
threshold: 5
|
|
9
|
+
judge:
|
|
10
|
+
persona: supervisor
|
|
11
|
+
instruction: |
|
|
12
|
+
The supervise → review cycle for the security audit has repeated {cycle_count} times.
|
|
13
|
+
Judge the overall quality and completeness of the audit, and decide whether to accept the current results.
|
|
14
|
+
rules:
|
|
15
|
+
- condition: Quality is sufficient
|
|
16
|
+
next: COMPLETE
|
|
17
|
+
- condition: Quality is insufficient with no prospect of improvement
|
|
18
|
+
next: ABORT
|
|
19
|
+
|
|
20
|
+
movements:
|
|
21
|
+
- name: plan
|
|
22
|
+
persona: planner
|
|
23
|
+
instruction: security-audit-plan
|
|
24
|
+
edit: false
|
|
25
|
+
output_contracts:
|
|
26
|
+
report:
|
|
27
|
+
- name: 01-plan.md
|
|
28
|
+
format: plan
|
|
29
|
+
rules:
|
|
30
|
+
- condition: Project structure understood
|
|
31
|
+
next: audit
|
|
32
|
+
|
|
33
|
+
- name: audit
|
|
34
|
+
persona: security-reviewer
|
|
35
|
+
knowledge: security
|
|
36
|
+
instruction: security-audit-team-leader
|
|
37
|
+
edit: false
|
|
38
|
+
team_leader:
|
|
39
|
+
max_parts: 3
|
|
40
|
+
part_persona: security-reviewer
|
|
41
|
+
part_edit: false
|
|
42
|
+
output_contracts:
|
|
43
|
+
report:
|
|
44
|
+
- name: 02-security-audit.md
|
|
45
|
+
format: security-audit
|
|
46
|
+
rules:
|
|
47
|
+
- condition: Audit complete
|
|
48
|
+
next: supervise
|
|
49
|
+
|
|
50
|
+
- name: supervise
|
|
51
|
+
persona: supervisor
|
|
52
|
+
knowledge: security
|
|
53
|
+
instruction: security-audit-supervise
|
|
54
|
+
edit: false
|
|
55
|
+
rules:
|
|
56
|
+
- condition: All files audited with sufficient quality
|
|
57
|
+
next: COMPLETE
|
|
58
|
+
- condition: Unaudited files exist or audit is insufficient
|
|
59
|
+
next: review
|
|
60
|
+
|
|
61
|
+
- name: review
|
|
62
|
+
persona: security-reviewer
|
|
63
|
+
knowledge: security
|
|
64
|
+
instruction: security-audit-review
|
|
65
|
+
edit: false
|
|
66
|
+
rules:
|
|
67
|
+
- condition: Re-audit complete
|
|
68
|
+
next: supervise
|
|
@@ -12,7 +12,7 @@ loop_monitors:
|
|
|
12
12
|
- cycle:
|
|
13
13
|
- ai_review
|
|
14
14
|
- ai_fix
|
|
15
|
-
threshold:
|
|
15
|
+
threshold: 5
|
|
16
16
|
judge:
|
|
17
17
|
persona: supervisor
|
|
18
18
|
instruction: loop-monitor-ai-fix
|
|
@@ -24,7 +24,7 @@ loop_monitors:
|
|
|
24
24
|
- cycle:
|
|
25
25
|
- reviewers
|
|
26
26
|
- fix
|
|
27
|
-
threshold:
|
|
27
|
+
threshold: 5
|
|
28
28
|
judge:
|
|
29
29
|
persona: supervisor
|
|
30
30
|
instruction: loop-monitor-reviewers-fix
|
|
@@ -72,6 +72,8 @@ movements:
|
|
|
72
72
|
knowledge:
|
|
73
73
|
- takt
|
|
74
74
|
- architecture
|
|
75
|
+
- unit-testing
|
|
76
|
+
- e2e-testing
|
|
75
77
|
provider_options:
|
|
76
78
|
claude:
|
|
77
79
|
allowed_tools:
|
|
@@ -305,6 +307,9 @@ movements:
|
|
|
305
307
|
policy:
|
|
306
308
|
- review
|
|
307
309
|
- testing
|
|
310
|
+
knowledge:
|
|
311
|
+
- unit-testing
|
|
312
|
+
- e2e-testing
|
|
308
313
|
provider_options:
|
|
309
314
|
claude:
|
|
310
315
|
allowed_tools:
|
|
@@ -51,7 +51,6 @@ movements:
|
|
|
51
51
|
- Bash
|
|
52
52
|
- WebSearch
|
|
53
53
|
- WebFetch
|
|
54
|
-
required_permission_mode: full
|
|
55
54
|
quality_gates:
|
|
56
55
|
- terraform fmt -check passes without errors
|
|
57
56
|
- terraform validate succeeds
|
|
@@ -176,7 +175,6 @@ movements:
|
|
|
176
175
|
- Bash
|
|
177
176
|
- WebSearch
|
|
178
177
|
- WebFetch
|
|
179
|
-
required_permission_mode: full
|
|
180
178
|
rules:
|
|
181
179
|
- condition: AI issue fix complete
|
|
182
180
|
- condition: No fix needed (target file/spec verified)
|
|
@@ -198,7 +196,6 @@ movements:
|
|
|
198
196
|
- Bash
|
|
199
197
|
- WebSearch
|
|
200
198
|
- WebFetch
|
|
201
|
-
required_permission_mode: full
|
|
202
199
|
rules:
|
|
203
200
|
- condition: Supervisor's findings have been fixed
|
|
204
201
|
- condition: Cannot proceed with fix
|
|
@@ -225,7 +222,6 @@ movements:
|
|
|
225
222
|
- Bash
|
|
226
223
|
- WebSearch
|
|
227
224
|
- WebFetch
|
|
228
|
-
required_permission_mode: full
|
|
229
225
|
pass_previous_response: false
|
|
230
226
|
rules:
|
|
231
227
|
- condition: AI issue fix complete
|
|
@@ -252,7 +248,6 @@ movements:
|
|
|
252
248
|
- Bash
|
|
253
249
|
- WebSearch
|
|
254
250
|
- WebFetch
|
|
255
|
-
required_permission_mode: full
|
|
256
251
|
pass_previous_response: false
|
|
257
252
|
rules:
|
|
258
253
|
- condition: Supervisor's findings have been fixed
|
|
@@ -2,17 +2,16 @@
|
|
|
2
2
|
|
|
3
3
|
このガイドは `instructions/` のファイルを作成・編集する際のルールを定義する。
|
|
4
4
|
|
|
5
|
-
##
|
|
5
|
+
## 参照元
|
|
6
6
|
|
|
7
|
-
`
|
|
7
|
+
`facets/instructions/` を参照元として使う。新規作成時は既存ファイルを参照して使う。
|
|
8
8
|
|
|
9
|
-
|
|
|
9
|
+
| 参照ファイル | 用途 |
|
|
10
10
|
|------------|------|
|
|
11
11
|
| `plan.md` | 計画(タスク分析・要件整理) |
|
|
12
12
|
| `architect.md` | 設計(アーキテクチャ設計) |
|
|
13
13
|
| `implement.md` | 実装(コーディング + レポート埋め込み) |
|
|
14
|
-
| `review.md` | レビュー(parallel sub-step 汎用) |
|
|
15
|
-
| `ai-review-standalone.md` | AIレビュー(standalone、iteration tracking) |
|
|
14
|
+
| `review-ai.md` | AIレビュー(parallel sub-step 汎用) |
|
|
16
15
|
| `ai-fix.md` | AI指摘修正(全ピース共通) |
|
|
17
16
|
| `fix.md` | レビュー指摘修正(汎用 fix / supervise fix) |
|
|
18
17
|
| `arbitrate.md` | 裁定(レビュアー vs コーダー) |
|
|
@@ -22,7 +21,7 @@
|
|
|
22
21
|
|
|
23
22
|
## インストラクションとは
|
|
24
23
|
|
|
25
|
-
ピースのムーブメントで実行する具体的な手順。`
|
|
24
|
+
ピースのムーブメントで実行する具体的な手順。`instruction` フィールドでファイル参照するか、インラインで直接記述する。
|
|
26
25
|
|
|
27
26
|
| 項目 | 内容 |
|
|
28
27
|
|------|------|
|
|
@@ -65,7 +64,7 @@
|
|
|
65
64
|
|
|
66
65
|
### セクション注入(`## Instructions` の前に別セクションとして挿入)
|
|
67
66
|
|
|
68
|
-
`perform_phase1_message.md`
|
|
67
|
+
`perform_phase1_message.md` 側で処理される。`instruction` の外に配置される。
|
|
69
68
|
|
|
70
69
|
| セクション | 内容 | デフォルト |
|
|
71
70
|
|-----------|------|-----------|
|
|
@@ -73,11 +72,11 @@
|
|
|
73
72
|
| `## Previous Response` | 前ムーブメントの出力 | **初回以外は常に表示**(`pass_previous_response` デフォルト: `true`) |
|
|
74
73
|
| `## Additional User Inputs` | 蓄積されたユーザー入力 | **常に表示** |
|
|
75
74
|
|
|
76
|
-
抑制したい場合は
|
|
75
|
+
抑制したい場合は `instruction` 内にプレースホルダー(`{task}`, `{previous_response}`, `{user_inputs}`)を含めるか、`pass_previous_response: false` を設定する。通常は不要。
|
|
77
76
|
|
|
78
|
-
###
|
|
77
|
+
### テンプレート変数展開(`instruction` 内のプレースホルダーを置換)
|
|
79
78
|
|
|
80
|
-
InstructionBuilder が
|
|
79
|
+
InstructionBuilder が `instruction` 内の `{変数名}` を展開する。インストラクション内で使用可能。
|
|
81
80
|
|
|
82
81
|
| 変数 | 内容 |
|
|
83
82
|
|------|------|
|
|
@@ -2,13 +2,13 @@
|
|
|
2
2
|
|
|
3
3
|
このガイドは `knowledge/` のファイルを作成・編集する際のルールを定義する。
|
|
4
4
|
|
|
5
|
-
##
|
|
5
|
+
## 参照元
|
|
6
6
|
|
|
7
|
-
`
|
|
7
|
+
`facets/knowledge/` を参照元として使う。新規作成時は既存ファイルをコピーして使う。
|
|
8
8
|
|
|
9
|
-
|
|
|
9
|
+
| 参照ファイル | 用途 | 使用例 |
|
|
10
10
|
|------------|------|--------|
|
|
11
|
-
| `
|
|
11
|
+
| `takt.md` | ドメイン知識 | frontend, backend, security |
|
|
12
12
|
|
|
13
13
|
---
|
|
14
14
|
|
|
@@ -2,15 +2,15 @@
|
|
|
2
2
|
|
|
3
3
|
このガイドは `output-contracts/` のファイルを作成・編集する際のルールを定義する。
|
|
4
4
|
|
|
5
|
-
##
|
|
5
|
+
## 参照元
|
|
6
6
|
|
|
7
|
-
`
|
|
7
|
+
`facets/output-contracts/` を参照元として使う。新規作成時は既存ファイルを参照して使う。
|
|
8
8
|
|
|
9
|
-
|
|
|
9
|
+
| 参照ファイル | 用途 |
|
|
10
10
|
|------------|------|
|
|
11
11
|
| `plan.md` | タスク計画(標準 / 拡張) |
|
|
12
12
|
| `architecture-design.md` | アーキテクチャ設計 |
|
|
13
|
-
| `review.md` | 汎用レビュー結果 |
|
|
13
|
+
| `review-gather.md` | 汎用レビュー結果 |
|
|
14
14
|
| `security-review.md` | セキュリティレビュー(重大度付き) |
|
|
15
15
|
| `validation.md` | 最終検証結果 |
|
|
16
16
|
| `summary.md` | タスク完了サマリー |
|
|
@@ -2,15 +2,15 @@
|
|
|
2
2
|
|
|
3
3
|
このガイドは `personas/` のファイルを作成・編集する際のルールを定義する。
|
|
4
4
|
|
|
5
|
-
##
|
|
5
|
+
## 参照元
|
|
6
6
|
|
|
7
|
-
`
|
|
7
|
+
`facets/personas/` を参照元として使う。新規作成時は既存ファイルをコピーして使う。
|
|
8
8
|
|
|
9
|
-
|
|
|
9
|
+
| 参照ファイル | 用途 | 使用例 |
|
|
10
10
|
|------------|------|--------|
|
|
11
|
-
| `
|
|
12
|
-
| `
|
|
13
|
-
| `
|
|
11
|
+
| `coder.md` | ドメイン知識を持たない実行系エージェント | coder, planner, conductor, research-digger |
|
|
12
|
+
| `architecture-reviewer.md` | ドメイン知識を持つ専門エージェント | architecture-reviewer, security-reviewer, cqrs-es-reviewer |
|
|
13
|
+
| `melchior.md` | 固有の人格・口調を持つキャラクター型 | melchior, balthasar, casper |
|
|
14
14
|
|
|
15
15
|
---
|
|
16
16
|
|
|
@@ -48,7 +48,7 @@
|
|
|
48
48
|
このルール/知識は…
|
|
49
49
|
├── 特定のエージェントだけが必要 → ペルソナ(ドメイン知識)
|
|
50
50
|
├── 複数のエージェントが共有 → ポリシー
|
|
51
|
-
└── 特定のピースの実行手順 →
|
|
51
|
+
└── 特定のピースの実行手順 → instruction(ペルソナに書かない)
|
|
52
52
|
```
|
|
53
53
|
|
|
54
54
|
---
|
|
@@ -158,7 +158,7 @@
|
|
|
158
158
|
1. **ポリシーの詳細ルール**: コード例・判定基準・例外リスト等の詳細はポリシーの責務(1行の行動指針は行動姿勢に記載してよい)
|
|
159
159
|
2. **ピース固有の概念**: ムーブメント名、レポートファイル名、ステップ間ルーティング
|
|
160
160
|
3. **ツール固有の環境情報**: `.takt/runs/` 等のディレクトリパス、テンプレート変数(`{report_dir}` 等)
|
|
161
|
-
4. **実行手順**: 「まず〜を読み、次に〜を実行」のような手順は
|
|
161
|
+
4. **実行手順**: 「まず〜を読み、次に〜を実行」のような手順は instruction の責務
|
|
162
162
|
|
|
163
163
|
### 例外: ドメイン知識としての重複
|
|
164
164
|
|
|
@@ -2,13 +2,13 @@
|
|
|
2
2
|
|
|
3
3
|
このガイドは `policies/` のファイルを作成・編集する際のルールを定義する。
|
|
4
4
|
|
|
5
|
-
##
|
|
5
|
+
## 参照元
|
|
6
6
|
|
|
7
|
-
`
|
|
7
|
+
`facets/policies/` を参照元として使う。新規作成時は既存ファイルをコピーして使う。
|
|
8
8
|
|
|
9
|
-
|
|
|
9
|
+
| 参照ファイル | 用途 | 使用例 |
|
|
10
10
|
|------------|------|--------|
|
|
11
|
-
| `
|
|
11
|
+
| `coding.md` | 共有行動規範 | coding, review, testing |
|
|
12
12
|
|
|
13
13
|
---
|
|
14
14
|
|
|
@@ -29,7 +29,7 @@
|
|
|
29
29
|
このルール/知識は…
|
|
30
30
|
├── 特定のエージェントだけが必要 → ペルソナ(ドメイン知識)
|
|
31
31
|
├── 複数のエージェントが共有 → ポリシー
|
|
32
|
-
└── 特定のピースの実行手順 →
|
|
32
|
+
└── 特定のピースの実行手順 → instruction(ポリシーに書かない)
|
|
33
33
|
```
|
|
34
34
|
|
|
35
35
|
---
|
|
@@ -10,35 +10,17 @@
|
|
|
10
10
|
| インストラクション | [INSTRUCTION_STYLE_GUIDE.md](INSTRUCTION_STYLE_GUIDE.md) | Phase 1 メッセージ(`{{instructions}}`) |
|
|
11
11
|
| 出力契約 | [OUTPUT_CONTRACT_STYLE_GUIDE.md](OUTPUT_CONTRACT_STYLE_GUIDE.md) | `report.format` |
|
|
12
12
|
|
|
13
|
-
##
|
|
13
|
+
## 参照元
|
|
14
14
|
|
|
15
|
-
`
|
|
15
|
+
`facets/` のファイルを参照元として使う。新規作成時はコピーまたは参照して使う。
|
|
16
16
|
|
|
17
17
|
```
|
|
18
|
-
|
|
19
|
-
├── personas/ #
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
│ └── policy.md
|
|
25
|
-
├── instructions/ # インストラクションテンプレート
|
|
26
|
-
│ ├── plan.md
|
|
27
|
-
│ ├── architect.md
|
|
28
|
-
│ ├── implement.md
|
|
29
|
-
│ ├── review.md
|
|
30
|
-
│ ├── ai-review-standalone.md
|
|
31
|
-
│ ├── ai-fix.md
|
|
32
|
-
│ ├── fix.md
|
|
33
|
-
│ ├── arbitrate.md
|
|
34
|
-
│ └── supervise.md
|
|
35
|
-
└── reports/ # 出力契約テンプレート
|
|
36
|
-
├── plan.md
|
|
37
|
-
├── architecture-design.md
|
|
38
|
-
├── review.md
|
|
39
|
-
├── security-review.md
|
|
40
|
-
├── validation.md
|
|
41
|
-
└── summary.md
|
|
18
|
+
facets/
|
|
19
|
+
├── personas/ # ペルソナ
|
|
20
|
+
├── policies/ # ポリシー
|
|
21
|
+
├── instructions/ # インストラクション
|
|
22
|
+
├── knowledge/ # ナレッジ
|
|
23
|
+
└── output-contracts/ # 出力契約
|
|
42
24
|
```
|
|
43
25
|
|
|
44
26
|
## 3層プロンプトアーキテクチャ
|
|
@@ -0,0 +1,26 @@
|
|
|
1
|
+
テストケースリストに基づき、不足しているE2Eテストを実装してください。
|
|
2
|
+
|
|
3
|
+
**重要:** テスト計画レポートを参照してください: {report:01-e2e-coverage-plan.md}
|
|
4
|
+
|
|
5
|
+
**注意:** Previous Responseがある場合は差し戻しのため、
|
|
6
|
+
未実装と指摘されたテストケースを確認し、そのケースの実装を行ってください。
|
|
7
|
+
|
|
8
|
+
**やること:**
|
|
9
|
+
1. テスト計画のテストケースリスト(番号付き)を確認する
|
|
10
|
+
2. 既存E2Eテストのパターン(ファイル構成、ヘルパー、フィクスチャ、モック方針)に合わせて実装する
|
|
11
|
+
3. テストケースリストの全件を実装する(一部だけ実装して終わりにしない)
|
|
12
|
+
4. E2Eテストを実行し、全テストがパスすることを確認する
|
|
13
|
+
5. 既存のE2Eテストが壊れていないことを確認する
|
|
14
|
+
|
|
15
|
+
**実装の制約:**
|
|
16
|
+
- 既存のE2Eテストフレームワークを変更しない
|
|
17
|
+
- テストは1シナリオ1関心で記述し、期待結果を明確にする
|
|
18
|
+
- 外部依存があるケースは既存のフィクスチャ・ヘルパー・モック方針に合わせる
|
|
19
|
+
|
|
20
|
+
**必須出力(見出しを含める)**
|
|
21
|
+
## 実装したテストケース
|
|
22
|
+
- {テストケースリストの番号と対応するテストファイル・テスト名}
|
|
23
|
+
## 未実装のテストケース(ある場合)
|
|
24
|
+
- {番号と未実装の理由}
|
|
25
|
+
## テスト結果
|
|
26
|
+
- {実行コマンドと結果}
|
|
@@ -0,0 +1,38 @@
|
|
|
1
|
+
アプリケーションのユーザー操作ルートを網羅的に洗い出し、不足しているE2Eテストケースをリスト化してください。
|
|
2
|
+
|
|
3
|
+
**注意:** Previous Responseがある場合は差し戻しのため、
|
|
4
|
+
その内容を踏まえてリストを見直してください。
|
|
5
|
+
|
|
6
|
+
**やること:**
|
|
7
|
+
|
|
8
|
+
1. **E2Eテストインフラの把握**
|
|
9
|
+
- 既存E2Eテストのディレクトリ構成、テストランナー、ヘルパー、フィクスチャ、モック方針を確認する
|
|
10
|
+
- テストの実行コマンドと実行方法を把握する
|
|
11
|
+
|
|
12
|
+
2. **ユーザー操作の起点を特定する**(ドキュメントではなくコードを読む)
|
|
13
|
+
- CLIの場合: コマンド定義・サブコマンド・オプション一覧をコードから抽出する
|
|
14
|
+
- Webの場合: ルーティング定義・ページ遷移・APIエンドポイントをコードから抽出する
|
|
15
|
+
- 各起点のハンドラ・処理フローを追跡し、分岐・状態遷移を把握する
|
|
16
|
+
|
|
17
|
+
3. **UXバリエーションを深掘りする**
|
|
18
|
+
- 各起点について、ユーザーが辿りうる全ルートを列挙する
|
|
19
|
+
- オプション・フラグの組み合わせによる分岐(例: `--pipeline` あり/なし、`--auto-pr` あり/なし)
|
|
20
|
+
- 状態による分岐(初回実行 vs 既存データあり、設定ファイルあり vs なし)
|
|
21
|
+
- 正常フローだけでなく、途中で失敗した場合のエラーハンドリング・リカバリールート
|
|
22
|
+
- 権限・ロール別のルート
|
|
23
|
+
- 外部依存の状態による分岐(接続成功 vs 失敗、レスポンス正常 vs 異常)
|
|
24
|
+
|
|
25
|
+
4. **既存E2Eテストとの突き合わせ**
|
|
26
|
+
- 既存テストが何をカバーしているかをファイル単位で分析する
|
|
27
|
+
- 洗い出したルートのうち、既存テストでカバー済みのものを特定する
|
|
28
|
+
- 未カバーのルートを「不足テストケース」としてリスト化する
|
|
29
|
+
|
|
30
|
+
5. **テストケースリストの作成**
|
|
31
|
+
- 全テストケースに一意の番号を付ける(supervisor が突き合わせに使う台帳)
|
|
32
|
+
- 各ケースに優先度を付ける(ユーザー影響度 × 未テストのリスク)
|
|
33
|
+
- **省略しない。** 1-2件で終わらせず、洗い出した全ルートを網羅すること
|
|
34
|
+
|
|
35
|
+
**絶対禁止:**
|
|
36
|
+
- ドキュメントやREADMEだけ読んでテストケースを推測する → 禁止。コードを読む
|
|
37
|
+
- 「他にもあるかもしれない」で列挙を打ち切る → 禁止。全件洗い出す
|
|
38
|
+
- 既存テストと重複するケースをリストに含める → 禁止。突き合わせ済みのもののみ
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
テスト計画のテストケースリストと実装結果を突き合わせ、全件実装されたか検証してください。
|
|
2
|
+
|
|
3
|
+
**重要:** テスト計画レポートを参照してください: {report:01-e2e-coverage-plan.md}
|
|
4
|
+
|
|
5
|
+
**検証手順:**
|
|
6
|
+
|
|
7
|
+
1. **テストケースリストとの突き合わせ(最重要)**
|
|
8
|
+
- 計画レポートのテストケースリスト(番号付き)を1件ずつ確認する
|
|
9
|
+
- 各ケースに対応するテストファイル・テスト名を特定する
|
|
10
|
+
- テストファイルを Read して、実際にそのケースがテストされているか確認する
|
|
11
|
+
- 対応するテストがないケースを「未実装」として列挙する
|
|
12
|
+
- 未実装が1件でもあれば REJECT する
|
|
13
|
+
|
|
14
|
+
2. **テストの品質検証**
|
|
15
|
+
- 各テストがテストケースの意図を正しく検証しているか
|
|
16
|
+
- アサーションが適切か(存在確認だけでなく、期待値の検証)
|
|
17
|
+
- モック・フィクスチャの使い方が既存パターンに合っているか
|
|
18
|
+
|
|
19
|
+
3. **テスト実行の確認**
|
|
20
|
+
- E2Eテストを実行し、全件パスすることを確認する
|
|
21
|
+
- 既存テストが壊れていないことを確認する
|
|
@@ -1,5 +1,9 @@
|
|
|
1
1
|
Report Directory内のレビューレポートを確認し、指摘事項を修正してください。
|
|
2
2
|
|
|
3
|
+
**修正の原則:**
|
|
4
|
+
- 指摘の「修正案」が示されている場合はそれに沿った対応を優先し、独自の迂回策を取らない
|
|
5
|
+
- 指摘の対象コードを直接修正すること。テストやドキュメントの追加で指摘を回避しない
|
|
6
|
+
|
|
3
7
|
**レポート参照方針:**
|
|
4
8
|
- Report Directory内の最新レビューレポートを一次情報として参照してください。
|
|
5
9
|
- 過去イテレーションのレポートは `{ファイル名}.{タイムスタンプ}` 形式で同ディレクトリに保存されています(例: `architect-review.md.20260304T123456Z`)。各レポートについて `{レポート名}.*` パターンで Glob を実行し、タイムスタンプ降順で最大2件まで読み、persists / reopened の傾向を把握してから修正に入ること。
|
|
@@ -7,6 +7,7 @@ ai_review と ai_fix のループが {cycle_count} 回繰り返されました
|
|
|
7
7
|
- AIレビュー結果: {report:ai-review.md}
|
|
8
8
|
|
|
9
9
|
**判断基準:**
|
|
10
|
-
-
|
|
11
|
-
-
|
|
12
|
-
-
|
|
10
|
+
- 同一 finding_id が複数サイクルにわたって persists しているか
|
|
11
|
+
- 同一 finding_id が繰り返し persists → 非生産的(スタックしている)
|
|
12
|
+
- 前回の finding が resolved され、新しい finding が new → 健全(進捗あり)
|
|
13
|
+
- 修正が実際にコードに反映されているか
|
|
@@ -4,6 +4,8 @@ Report Directory 内の最新レビューレポートを確認し、
|
|
|
4
4
|
このループが健全(収束傾向)か非生産的(発散・振動)かを判断してください。
|
|
5
5
|
|
|
6
6
|
**判断基準:**
|
|
7
|
-
-
|
|
8
|
-
-
|
|
7
|
+
- 同一 finding_id が複数サイクルにわたって persists しているか
|
|
8
|
+
- 同一 finding_id が繰り返し persists → 非生産的(スタックしている)
|
|
9
|
+
- 前回の finding が resolved され、新しい finding が new → 健全(収束傾向)
|
|
9
10
|
- 修正が実際にコードに反映されているか
|
|
11
|
+
- new / reopened の指摘件数が全体として減少傾向にあるか
|
|
@@ -25,7 +25,9 @@
|
|
|
25
25
|
5. ファイル構成・設計パターンを決定する(必要な場合)
|
|
26
26
|
6. 実装アプローチを決める
|
|
27
27
|
- 実装アプローチがナレッジ・ポリシーの制約に違反しないか照合する
|
|
28
|
+
- 利用者向け機能の追加や変更がある場合、利用者がその機能へ到達する条件・入口・起動経路を固定する
|
|
28
29
|
7. Coder向けの実装ガイドラインに以下を含めること:
|
|
29
30
|
- 参照すべき既存実装パターン(ファイル:行)。同種の処理が既にある場合は必ず示す
|
|
30
31
|
- 変更の影響範囲。特に新しいパラメータを追加する場合、配線が必要な全箇所を列挙する
|
|
31
32
|
- このタスクで特に注意すべきアンチパターン(該当するものがあれば)
|
|
33
|
+
- 利用者向け機能の追加や変更がある場合、到達経路・呼び出し元・起動条件に関する変更箇所
|
|
@@ -1,13 +1,21 @@
|
|
|
1
1
|
フロントエンド開発の観点から変更をレビューしてください。
|
|
2
2
|
|
|
3
3
|
**レビュー観点:**
|
|
4
|
+
- デザイン忠実度(デザイン参照が指定されている場合、最優先で確認)
|
|
4
5
|
- コンポーネント設計(責務分離、粒度)
|
|
5
6
|
- 状態管理(ローカル/グローバルの判断)
|
|
6
7
|
- パフォーマンス(再レンダリング、メモ化)
|
|
7
8
|
- アクセシビリティ(キーボード操作、ARIA)
|
|
8
9
|
- データフェッチパターン
|
|
10
|
+
- 利用者が機能へ到達できる配線(route、導線、起動条件)
|
|
9
11
|
- TypeScript型安全性
|
|
10
12
|
|
|
13
|
+
**デザイン忠実度チェック(デザイン参照がある場合):**
|
|
14
|
+
1. タスク指示書の参照資料からデザイン参照を特定する
|
|
15
|
+
2. デザインの要素(レイアウト、文言、色、間隔)を実装と要素単位で比較する
|
|
16
|
+
3. 差分がある場合、決定ログで意図的かどうかを確認する
|
|
17
|
+
4. 意図的でない差分はブロッキング問題として報告する
|
|
18
|
+
|
|
11
19
|
**注意**: このプロジェクトがフロントエンドを含まない場合は、
|
|
12
20
|
問題なしとして次に進んでください。
|
|
13
21
|
|
|
@@ -21,5 +29,6 @@
|
|
|
21
29
|
|
|
22
30
|
1. 変更差分を確認し、フロントエンド開発の観点に基づいて問題を検出する
|
|
23
31
|
- ナレッジの判定基準テーブル(REJECT条件)と変更内容を照合する
|
|
32
|
+
- 新規の画面・機能ファイルが追加されている場合、利用者が到達する入口や呼び出し元の更新有無を確認する
|
|
24
33
|
2. 検出した問題ごとに、Policyのスコープ判定表と判定ルールに基づいてブロッキング/非ブロッキングを分類する
|
|
25
34
|
3. ブロッキング問題が1件でもあればREJECTと判定する
|
|
@@ -0,0 +1,12 @@
|
|
|
1
|
+
プロジェクトの全体像を把握し、セキュリティ監査の対象ファイル一覧を作成してください。
|
|
2
|
+
|
|
3
|
+
**やること:**
|
|
4
|
+
1. プロジェクトのソースコードディレクトリを特定し、全ファイルを Glob で一覧化する
|
|
5
|
+
2. プロジェクトの技術スタック、フレームワーク、主要な依存関係を把握する
|
|
6
|
+
3. 各ファイルの役割を簡潔に分類する(API層、ドメイン層、インフラ層、ユーティリティ等)
|
|
7
|
+
4. セキュリティリスクの高いファイルを特定する(認証、入力処理、外部通信、ファイル操作、設定等)
|
|
8
|
+
|
|
9
|
+
**重要:**
|
|
10
|
+
- 全ファイルを漏れなく一覧化すること。省略しない
|
|
11
|
+
- 設定ファイル、テストファイルも含める
|
|
12
|
+
- ファイル数が多い場合でも省略せず全件列挙する
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
前回の監査で不十分と判断されたファイルを再監査してください。
|
|
2
|
+
|
|
3
|
+
**重要:** supervisor の検証結果を確認し、以下を把握してください:
|
|
4
|
+
- 未監査ファイルの一覧
|
|
5
|
+
- 監査が不十分と指摘されたファイルの一覧
|
|
6
|
+
- 具体的な指摘内容
|
|
7
|
+
|
|
8
|
+
**やること:**
|
|
9
|
+
1. 指摘された全ファイルを **Read tool で1つずつ全文を読む**
|
|
10
|
+
2. 各ファイルについてセキュリティ観点でレビューする
|
|
11
|
+
3. 発見した問題を重大度付きで報告する
|
|
12
|
+
|
|
13
|
+
**絶対禁止事項:**
|
|
14
|
+
- Grep でキーワード検索してヒットしたファイルだけ見る → 禁止
|
|
15
|
+
- ファイルの一部だけ読んで判断する → 禁止
|
|
16
|
+
- 「問題なさそう」で読み飛ばす → 禁止
|
|
17
|
+
|
|
18
|
+
**必須出力(見出しを含める):**
|
|
19
|
+
## 再監査結果
|
|
20
|
+
- {各ファイルごとの監査結果}
|
|
21
|
+
## 検出された問題
|
|
22
|
+
- {問題の詳細(重大度、場所、修正案)}
|
|
@@ -0,0 +1,20 @@
|
|
|
1
|
+
セキュリティ監査の網羅性と品質を検証してください。
|
|
2
|
+
|
|
3
|
+
**重要:** 計画レポートを参照してください: {report:01-plan.md}
|
|
4
|
+
|
|
5
|
+
**検証手順:**
|
|
6
|
+
|
|
7
|
+
1. **網羅性の検証(最重要)**
|
|
8
|
+
- 計画レポートのファイル一覧と、監査結果で言及されたファイルを突き合わせる
|
|
9
|
+
- 監査結果に言及がないファイルを「未監査ファイル」として列挙する
|
|
10
|
+
- 未監査ファイルが1件でもあれば REJECT する
|
|
11
|
+
|
|
12
|
+
2. **手法の検証**
|
|
13
|
+
- 各ファイルの監査結果が具体的なコード内容に言及しているか確認する
|
|
14
|
+
- 「問題なし」のみで具体的な確認内容が書かれていない場合、実際に Read していない可能性がある → REJECT
|
|
15
|
+
- Grep のキーワードマッチだけで判断している形跡がないか確認する
|
|
16
|
+
|
|
17
|
+
3. **品質の検証**
|
|
18
|
+
- 検出された問題の重大度分類が妥当か
|
|
19
|
+
- 見落としがないか、自分でもセキュリティリスクの高いファイルを数件 Read して確認する
|
|
20
|
+
- 誤検知(FP)が多すぎないか
|