@mrciphersmith/keryx 0.2.97 → 0.2.98
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/dist/cli.js +569 -235
- package/dist/core.js +1 -1
- package/package.json +1 -1
- package/src/gdskills/bundled/rules/core/api-contracts.mdc +1 -0
- package/src/gdskills/bundled/rules/core/code-style-patterns.mdc +1 -0
- package/src/gdskills/bundled/rules/core/database-patterns.mdc +1 -0
- package/src/gdskills/bundled/rules/core/documentation-management.mdc +33 -38
- package/src/gdskills/bundled/rules/core/error-handling.mdc +1 -11
- package/src/gdskills/bundled/rules/core/execution-metrics.md +1 -2
- package/src/gdskills/bundled/rules/core/frontend-assistant.mdc +1 -0
- package/src/gdskills/bundled/rules/core/git-concurrency.mdc +101 -0
- package/src/gdskills/bundled/rules/core/implementation-plans.mdc +23 -11
- package/src/gdskills/bundled/rules/core/mobx-store-template.mdc +1 -0
- package/src/gdskills/bundled/rules/core/nestjs-dto.mdc +1 -0
- package/src/gdskills/bundled/rules/core/playwright-testing.mdc +1 -0
- package/src/gdskills/bundled/rules/core/requirements-management.mdc +15 -11
- package/src/gdskills/bundled/rules/core/rule-management-workflow.mdc +29 -14
- package/src/gdskills/bundled/rules/core/shared-definitions.mdc +1 -1
- package/src/gdskills/bundled/rules/core/skill-lifecycle.mdc +9 -5
- package/src/gdskills/bundled/rules/core/skills-storage-workflow.mdc +59 -16
- package/src/gdskills/bundled/rules/core/storybook-guidelines.mdc +1 -0
- package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.codex.md +48 -71
- package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.cursor.md +48 -71
- package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.md +48 -71
- package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.opencode.md +48 -71
- package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.zed.md +48 -71
- package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.codex.md +4 -4
- package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.cursor.md +4 -4
- package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.md +4 -4
- package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.opencode.md +4 -4
- package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.zed.md +4 -4
- package/src/gdskills/bundled/skills/orchestration/context-collector/orchestrator-prompt.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.codex.md +12 -22
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.cursor.md +12 -22
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.detail.md +12 -22
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.md +12 -22
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.opencode.md +12 -22
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.zed.md +12 -22
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/analysis-request.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/analysis-request.template.md +1 -1
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/input-contract.schema.json +4 -4
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/orchestrator-prompt.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/SKILL.md +46 -4
- package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.opencode.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.zed.md +2 -2
- package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.codex.md +3 -3
- package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.cursor.md +3 -3
- package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.md +3 -3
- package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.opencode.md +3 -3
- package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.zed.md +3 -3
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.codex.md +65 -23
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.cursor.md +65 -23
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md +65 -23
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.opencode.md +65 -23
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.zed.md +65 -23
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.codex.md +19 -10
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.cursor.md +19 -10
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.md +19 -10
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.opencode.md +19 -10
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.zed.md +19 -10
- package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/planning/interview/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/planning/interview/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/planning/interview/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/planning/interviewer/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/planning/interviewer/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/planning/interviewer/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.codex.md +7 -7
- package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.cursor.md +7 -7
- package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.md +7 -7
- package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.opencode.md +7 -7
- package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.zed.md +7 -7
- package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/platform/hookify/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/platform/hookify/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/platform/hookify/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/changelog/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/quality/changelog/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/quality/changelog/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/commit/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/quality/commit/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/quality/commit/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/deploy/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/quality/deploy/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/quality/deploy/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/perf-check/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/quality/perf-check/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/quality/perf-check/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/pr/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/quality/pr/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/quality/pr/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.opencode.md +2 -2
- package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.zed.md +2 -2
- package/src/gdskills/bundled/skills/quality/push/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/quality/push/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/quality/push/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/quality/test-gen/SKILL.codex.md +7 -2
- package/src/gdskills/bundled/skills/quality/test-gen/SKILL.cursor.md +7 -2
- package/src/gdskills/bundled/skills/quality/test-gen/SKILL.md +7 -2
- package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.codex.md +15 -9
- package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.cursor.md +15 -9
- package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.md +15 -9
- package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.opencode.md +15 -9
- package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.zed.md +15 -9
- package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.opencode.md +2 -2
- package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.zed.md +2 -2
- package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.opencode.md +2 -2
- package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.zed.md +2 -2
- package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.opencode.md +2 -2
- package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.zed.md +2 -2
- package/src/gdskills/bundled/skills/review/code-style-review/SKILL.codex.md +2 -2
- package/src/gdskills/bundled/skills/review/code-style-review/SKILL.cursor.md +2 -2
- package/src/gdskills/bundled/skills/review/code-style-review/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/code-style-review/SKILL.opencode.md +2 -2
- package/src/gdskills/bundled/skills/review/code-style-review/SKILL.zed.md +2 -2
- package/src/gdskills/bundled/skills/review/review-architecture/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-backend/SKILL.md +3 -3
- package/src/gdskills/bundled/skills/review/review-clean-code/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-core-boundaries/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/review/review-flow-graph/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/review/review-frontend/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-frontend-conventions/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/review/review-highload/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-layout/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/review/review-logic/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +40 -35
- package/src/gdskills/bundled/skills/review/review-orchestrator/input-contract.schema.json +1 -2
- package/src/gdskills/bundled/skills/review/review-orchestrator/review-context.schema.json +1 -5
- package/src/gdskills/bundled/skills/review/review-orchestrator/reviewer-input.schema.json +53 -9
- package/src/gdskills/bundled/skills/review/review-performance/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-pr-feedback/SKILL.md +6 -6
- package/src/gdskills/bundled/skills/review/review-security-code/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-style/SKILL.md +3 -3
- package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/review/review-verifier/SKILL.md +1 -1
- package/src/gdskills/bundled/rules/core/review-agent-profile.mdc +0 -49
- package/src/gdskills/bundled/rules/core/review-strict-profile.mdc +0 -48
|
@@ -12,8 +12,8 @@ triggers:
|
|
|
12
12
|
metadata:
|
|
13
13
|
author: "MrCipherSmith"
|
|
14
14
|
version: "2.4.0"
|
|
15
|
-
category: "
|
|
16
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
category: "orchestration"
|
|
16
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
17
17
|
license: "MIT"
|
|
18
18
|
---
|
|
19
19
|
|
|
@@ -275,7 +275,7 @@ When focus specified: boost files matching focus keywords to P0; select ALL focu
|
|
|
275
275
|
1. **Dependency search**: find all target files importing changed DTOs/APIs from source
|
|
276
276
|
2. **Contract divergence**: compare new source contracts with current target implementation
|
|
277
277
|
3. **Target deep dive**: read 2-3 key components that will need changes
|
|
278
|
-
4. **Target rules compliance**: check `.
|
|
278
|
+
4. **Target rules compliance**: check `.metaproject/rules/core/*.mdc` in target repo
|
|
279
279
|
|
|
280
280
|
---
|
|
281
281
|
|
|
@@ -333,24 +333,14 @@ Wait for user confirmation before generating full report.
|
|
|
333
333
|
### Output Structure
|
|
334
334
|
|
|
335
335
|
```
|
|
336
|
-
<DOCS_ROOT>/analysis/<feature-name
|
|
337
|
-
├──
|
|
338
|
-
|
|
339
|
-
│ ├── en/report.md # English for humans
|
|
340
|
-
│ ├── ru/report.md # Russian for humans
|
|
341
|
-
│ └── ai/report.md # Structured for AI agents (EN)
|
|
342
|
-
├── plans/
|
|
343
|
-
│ ├── en/implementation-plan.md
|
|
344
|
-
│ ├── ru/implementation-plan.md
|
|
345
|
-
│ └── ai/implementation-plan.md
|
|
346
|
-
├── contracts/
|
|
347
|
-
│ ├── api-changes.md # API contract diff
|
|
348
|
-
│ └── dto-comparison.md # Before/after DTOs
|
|
349
|
-
└── metrics/
|
|
350
|
-
└── analysis-metrics.md # Analysis metadata
|
|
336
|
+
<DOCS_ROOT>/analysis/<feature-name>/
|
|
337
|
+
├── report.md # Findings, API contract diff, DTO comparison, cross-repo impact, analysis metrics
|
|
338
|
+
└── implementation-plan.md # Actionable implementation plan derived from the analysis
|
|
351
339
|
```
|
|
352
340
|
|
|
353
|
-
|
|
341
|
+
No date-stamped folder: update `report.md`/`implementation-plan.md` in place for a rerun on the same feature rather than creating a parallel copy. Default output is a single English (`en`) pair. Add other language variants only if the user explicitly asks for them, and keep any variants you create synchronized.
|
|
342
|
+
|
|
343
|
+
`report.md` should use Gherkin-style scenarios where they make findings easier for other AI agents to parse.
|
|
354
344
|
> For full Gherkin output format and syntax rules, see `SKILL.detail.md`.
|
|
355
345
|
|
|
356
346
|
---
|
|
@@ -360,7 +350,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
360
350
|
- Every claim MUST reference specific code: `[filename.ts:L123](file:///absolute/path#L123)`
|
|
361
351
|
- Minimum 3 code examples per report
|
|
362
352
|
- Mermaid diagrams for architecture, tables for DTO changes, flowcharts for data flow
|
|
363
|
-
-
|
|
353
|
+
- Default to a single `en` document; add `ru`/`ai` (or other) variants only when the user explicitly asks, keeping them synchronized
|
|
364
354
|
|
|
365
355
|
---
|
|
366
356
|
|
|
@@ -378,7 +368,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
378
368
|
|
|
379
369
|
## Step 13: Analysis Metrics
|
|
380
370
|
|
|
381
|
-
Track and include in `
|
|
371
|
+
Track and include in the `## Analysis Metrics` section of `report.md`:
|
|
382
372
|
- Duration, files analyzed (P0/P1/P2), lines changed
|
|
383
373
|
- Cross-repo dependencies, API endpoints changed, DTOs modified
|
|
384
374
|
- Breaking changes count, test coverage %, risk level
|
|
@@ -415,7 +405,7 @@ Follow `documentation-management.mdc`: update `<DOCS_ROOT>/readme.md`, add entry
|
|
|
415
405
|
4. **Never assume** — ask user when unclear
|
|
416
406
|
5. **Never skip** intermediate review for complex analyses (P0 files > 3)
|
|
417
407
|
6. **Always provide** concrete, actionable recommendations
|
|
418
|
-
7. **Always
|
|
408
|
+
7. **Always default** to a single-language document (`en`); add other language variants only when the user explicitly asks
|
|
419
409
|
|
|
420
410
|
---
|
|
421
411
|
|
|
@@ -224,26 +224,16 @@ grep -r "focus_keyword" --include="*.spec.ts" --include="*.test.ts" src/
|
|
|
224
224
|
|
|
225
225
|
### Output for Mode B
|
|
226
226
|
|
|
227
|
-
Generate **Feature Specification Document** instead of Change Analysis:
|
|
228
|
-
|
|
229
|
-
```
|
|
230
|
-
<DOCS_ROOT>/analysis/<feature
|
|
231
|
-
├──
|
|
232
|
-
|
|
233
|
-
│ ├── en/feature-specification.md
|
|
234
|
-
│ ├── ru/feature-specification.md
|
|
235
|
-
│ └── ai/feature-specification.md
|
|
236
|
-
├── architecture/
|
|
237
|
-
│ ├── data-flow.md
|
|
238
|
-
│ ├── component-diagram.md
|
|
239
|
-
│ └── api-contracts.md
|
|
240
|
-
├── usage/
|
|
241
|
-
│ ├── examples.md
|
|
242
|
-
│ └── patterns.md
|
|
243
|
-
└── tests/
|
|
244
|
-
└── test-coverage.md
|
|
227
|
+
Generate **Feature Specification Document** instead of Change Analysis, using the same flat analysis-category layout as Mode A:
|
|
228
|
+
|
|
229
|
+
```
|
|
230
|
+
<DOCS_ROOT>/analysis/<feature-name>/
|
|
231
|
+
├── report.md # Feature specification: purpose, API contracts, business logic, data flow, architecture, usage examples, test coverage
|
|
232
|
+
└── implementation-plan.md # Only if formalization surfaces follow-up implementation work
|
|
245
233
|
```
|
|
246
234
|
|
|
235
|
+
No date-stamped folder, no `-current-state-<date>` suffix, and no mandatory `en`/`ru`/`ai` split — add language variants only if the user explicitly asks.
|
|
236
|
+
|
|
247
237
|
---
|
|
248
238
|
|
|
249
239
|
## Timeouts and Limits
|
|
@@ -397,7 +387,7 @@ Analyzing only new changes...
|
|
|
397
387
|
"branch": "feature-name",
|
|
398
388
|
"sha": "abc123",
|
|
399
389
|
"base_sha": "def456",
|
|
400
|
-
"path": "feature-name
|
|
390
|
+
"path": "feature-name",
|
|
401
391
|
"created_at": "2024-01-15T10:00:00Z",
|
|
402
392
|
"updated_at": "2024-01-15T10:00:00Z",
|
|
403
393
|
"status": "complete",
|
|
@@ -526,7 +516,7 @@ git log --oneline "${BASE_SHA}..HEAD"
|
|
|
526
516
|
|
|
527
517
|
## Step 10: Gherkin Output Format (Full)
|
|
528
518
|
|
|
529
|
-
|
|
519
|
+
Gherkin-style scenarios in `report.md` (and its `ai` language variant, when one is generated) make analysis results parseable by other AI agents. The same applies to `implementation-plan.md` for Phase scenarios.
|
|
530
520
|
|
|
531
521
|
**Purpose**: Enable other AI agents to parse analysis results programmatically.
|
|
532
522
|
|
|
@@ -633,7 +623,7 @@ Feature: [Concise Feature Name]
|
|
|
633
623
|
And the risk level is "[Low/Medium/High]"
|
|
634
624
|
```
|
|
635
625
|
|
|
636
|
-
### 7. Implementation Plan (in
|
|
626
|
+
### 7. Implementation Plan (in implementation-plan.md)
|
|
637
627
|
```gherkin
|
|
638
628
|
Feature: Implementation Plan for [Feature Name]
|
|
639
629
|
Background:
|
|
@@ -711,7 +701,7 @@ Feature: Implementation Plan for [Feature Name]
|
|
|
711
701
|
|
|
712
702
|
## Step 13: Full Metrics and Complexity Score
|
|
713
703
|
|
|
714
|
-
Track and report in `
|
|
704
|
+
Track and report in the `## Analysis Metrics` section of `report.md`:
|
|
715
705
|
|
|
716
706
|
```markdown
|
|
717
707
|
## Analysis Metrics
|
|
@@ -12,8 +12,8 @@ triggers:
|
|
|
12
12
|
metadata:
|
|
13
13
|
author: "MrCipherSmith"
|
|
14
14
|
version: "2.4.0"
|
|
15
|
-
category: "
|
|
16
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
category: "orchestration"
|
|
16
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
17
17
|
license: "MIT"
|
|
18
18
|
---
|
|
19
19
|
|
|
@@ -275,7 +275,7 @@ When focus specified: boost files matching focus keywords to P0; select ALL focu
|
|
|
275
275
|
1. **Dependency search**: find all target files importing changed DTOs/APIs from source
|
|
276
276
|
2. **Contract divergence**: compare new source contracts with current target implementation
|
|
277
277
|
3. **Target deep dive**: read 2-3 key components that will need changes
|
|
278
|
-
4. **Target rules compliance**: check `.
|
|
278
|
+
4. **Target rules compliance**: check `.metaproject/rules/core/*.mdc` in target repo
|
|
279
279
|
|
|
280
280
|
---
|
|
281
281
|
|
|
@@ -333,24 +333,14 @@ Wait for user confirmation before generating full report.
|
|
|
333
333
|
### Output Structure
|
|
334
334
|
|
|
335
335
|
```
|
|
336
|
-
<DOCS_ROOT>/analysis/<feature-name
|
|
337
|
-
├──
|
|
338
|
-
|
|
339
|
-
│ ├── en/report.md # English for humans
|
|
340
|
-
│ ├── ru/report.md # Russian for humans
|
|
341
|
-
│ └── ai/report.md # Structured for AI agents (EN)
|
|
342
|
-
├── plans/
|
|
343
|
-
│ ├── en/implementation-plan.md
|
|
344
|
-
│ ├── ru/implementation-plan.md
|
|
345
|
-
│ └── ai/implementation-plan.md
|
|
346
|
-
├── contracts/
|
|
347
|
-
│ ├── api-changes.md # API contract diff
|
|
348
|
-
│ └── dto-comparison.md # Before/after DTOs
|
|
349
|
-
└── metrics/
|
|
350
|
-
└── analysis-metrics.md # Analysis metadata
|
|
336
|
+
<DOCS_ROOT>/analysis/<feature-name>/
|
|
337
|
+
├── report.md # Findings, API contract diff, DTO comparison, cross-repo impact, analysis metrics
|
|
338
|
+
└── implementation-plan.md # Actionable implementation plan derived from the analysis
|
|
351
339
|
```
|
|
352
340
|
|
|
353
|
-
|
|
341
|
+
No date-stamped folder: update `report.md`/`implementation-plan.md` in place for a rerun on the same feature rather than creating a parallel copy. Default output is a single English (`en`) pair. Add other language variants only if the user explicitly asks for them, and keep any variants you create synchronized.
|
|
342
|
+
|
|
343
|
+
`report.md` should use Gherkin-style scenarios where they make findings easier for other AI agents to parse.
|
|
354
344
|
> For full Gherkin output format and syntax rules, see `SKILL.detail.md`.
|
|
355
345
|
|
|
356
346
|
---
|
|
@@ -360,7 +350,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
360
350
|
- Every claim MUST reference specific code: `[filename.ts:L123](file:///absolute/path#L123)`
|
|
361
351
|
- Minimum 3 code examples per report
|
|
362
352
|
- Mermaid diagrams for architecture, tables for DTO changes, flowcharts for data flow
|
|
363
|
-
-
|
|
353
|
+
- Default to a single `en` document; add `ru`/`ai` (or other) variants only when the user explicitly asks, keeping them synchronized
|
|
364
354
|
|
|
365
355
|
---
|
|
366
356
|
|
|
@@ -378,7 +368,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
378
368
|
|
|
379
369
|
## Step 13: Analysis Metrics
|
|
380
370
|
|
|
381
|
-
Track and include in `
|
|
371
|
+
Track and include in the `## Analysis Metrics` section of `report.md`:
|
|
382
372
|
- Duration, files analyzed (P0/P1/P2), lines changed
|
|
383
373
|
- Cross-repo dependencies, API endpoints changed, DTOs modified
|
|
384
374
|
- Breaking changes count, test coverage %, risk level
|
|
@@ -415,7 +405,7 @@ Follow `documentation-management.mdc`: update `<DOCS_ROOT>/readme.md`, add entry
|
|
|
415
405
|
4. **Never assume** — ask user when unclear
|
|
416
406
|
5. **Never skip** intermediate review for complex analyses (P0 files > 3)
|
|
417
407
|
6. **Always provide** concrete, actionable recommendations
|
|
418
|
-
7. **Always
|
|
408
|
+
7. **Always default** to a single-language document (`en`); add other language variants only when the user explicitly asks
|
|
419
409
|
|
|
420
410
|
---
|
|
421
411
|
|
|
@@ -12,8 +12,8 @@ triggers:
|
|
|
12
12
|
metadata:
|
|
13
13
|
author: "MrCipherSmith"
|
|
14
14
|
version: "2.4.0"
|
|
15
|
-
category: "
|
|
16
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
category: "orchestration"
|
|
16
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
17
17
|
license: "MIT"
|
|
18
18
|
---
|
|
19
19
|
|
|
@@ -275,7 +275,7 @@ When focus specified: boost files matching focus keywords to P0; select ALL focu
|
|
|
275
275
|
1. **Dependency search**: find all target files importing changed DTOs/APIs from source
|
|
276
276
|
2. **Contract divergence**: compare new source contracts with current target implementation
|
|
277
277
|
3. **Target deep dive**: read 2-3 key components that will need changes
|
|
278
|
-
4. **Target rules compliance**: check `.
|
|
278
|
+
4. **Target rules compliance**: check `.metaproject/rules/core/*.mdc` in target repo
|
|
279
279
|
|
|
280
280
|
---
|
|
281
281
|
|
|
@@ -333,24 +333,14 @@ Wait for user confirmation before generating full report.
|
|
|
333
333
|
### Output Structure
|
|
334
334
|
|
|
335
335
|
```
|
|
336
|
-
<DOCS_ROOT>/analysis/<feature-name
|
|
337
|
-
├──
|
|
338
|
-
|
|
339
|
-
│ ├── en/report.md # English for humans
|
|
340
|
-
│ ├── ru/report.md # Russian for humans
|
|
341
|
-
│ └── ai/report.md # Structured for AI agents (EN)
|
|
342
|
-
├── plans/
|
|
343
|
-
│ ├── en/implementation-plan.md
|
|
344
|
-
│ ├── ru/implementation-plan.md
|
|
345
|
-
│ └── ai/implementation-plan.md
|
|
346
|
-
├── contracts/
|
|
347
|
-
│ ├── api-changes.md # API contract diff
|
|
348
|
-
│ └── dto-comparison.md # Before/after DTOs
|
|
349
|
-
└── metrics/
|
|
350
|
-
└── analysis-metrics.md # Analysis metadata
|
|
336
|
+
<DOCS_ROOT>/analysis/<feature-name>/
|
|
337
|
+
├── report.md # Findings, API contract diff, DTO comparison, cross-repo impact, analysis metrics
|
|
338
|
+
└── implementation-plan.md # Actionable implementation plan derived from the analysis
|
|
351
339
|
```
|
|
352
340
|
|
|
353
|
-
|
|
341
|
+
No date-stamped folder: update `report.md`/`implementation-plan.md` in place for a rerun on the same feature rather than creating a parallel copy. Default output is a single English (`en`) pair. Add other language variants only if the user explicitly asks for them, and keep any variants you create synchronized.
|
|
342
|
+
|
|
343
|
+
`report.md` should use Gherkin-style scenarios where they make findings easier for other AI agents to parse.
|
|
354
344
|
> For full Gherkin output format and syntax rules, see `SKILL.detail.md`.
|
|
355
345
|
|
|
356
346
|
---
|
|
@@ -360,7 +350,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
360
350
|
- Every claim MUST reference specific code: `[filename.ts:L123](file:///absolute/path#L123)`
|
|
361
351
|
- Minimum 3 code examples per report
|
|
362
352
|
- Mermaid diagrams for architecture, tables for DTO changes, flowcharts for data flow
|
|
363
|
-
-
|
|
353
|
+
- Default to a single `en` document; add `ru`/`ai` (or other) variants only when the user explicitly asks, keeping them synchronized
|
|
364
354
|
|
|
365
355
|
---
|
|
366
356
|
|
|
@@ -378,7 +368,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
378
368
|
|
|
379
369
|
## Step 13: Analysis Metrics
|
|
380
370
|
|
|
381
|
-
Track and include in `
|
|
371
|
+
Track and include in the `## Analysis Metrics` section of `report.md`:
|
|
382
372
|
- Duration, files analyzed (P0/P1/P2), lines changed
|
|
383
373
|
- Cross-repo dependencies, API endpoints changed, DTOs modified
|
|
384
374
|
- Breaking changes count, test coverage %, risk level
|
|
@@ -415,7 +405,7 @@ Follow `documentation-management.mdc`: update `<DOCS_ROOT>/readme.md`, add entry
|
|
|
415
405
|
4. **Never assume** — ask user when unclear
|
|
416
406
|
5. **Never skip** intermediate review for complex analyses (P0 files > 3)
|
|
417
407
|
6. **Always provide** concrete, actionable recommendations
|
|
418
|
-
7. **Always
|
|
408
|
+
7. **Always default** to a single-language document (`en`); add other language variants only when the user explicitly asks
|
|
419
409
|
|
|
420
410
|
---
|
|
421
411
|
|
|
@@ -12,8 +12,8 @@ triggers:
|
|
|
12
12
|
metadata:
|
|
13
13
|
author: "MrCipherSmith"
|
|
14
14
|
version: "2.4.0"
|
|
15
|
-
category: "
|
|
16
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
category: "orchestration"
|
|
16
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
17
17
|
license: "MIT"
|
|
18
18
|
---
|
|
19
19
|
|
|
@@ -275,7 +275,7 @@ When focus specified: boost files matching focus keywords to P0; select ALL focu
|
|
|
275
275
|
1. **Dependency search**: find all target files importing changed DTOs/APIs from source
|
|
276
276
|
2. **Contract divergence**: compare new source contracts with current target implementation
|
|
277
277
|
3. **Target deep dive**: read 2-3 key components that will need changes
|
|
278
|
-
4. **Target rules compliance**: check `.
|
|
278
|
+
4. **Target rules compliance**: check `.metaproject/rules/core/*.mdc` in target repo
|
|
279
279
|
|
|
280
280
|
---
|
|
281
281
|
|
|
@@ -333,24 +333,14 @@ Wait for user confirmation before generating full report.
|
|
|
333
333
|
### Output Structure
|
|
334
334
|
|
|
335
335
|
```
|
|
336
|
-
<DOCS_ROOT>/analysis/<feature-name
|
|
337
|
-
├──
|
|
338
|
-
|
|
339
|
-
│ ├── en/report.md # English for humans
|
|
340
|
-
│ ├── ru/report.md # Russian for humans
|
|
341
|
-
│ └── ai/report.md # Structured for AI agents (EN)
|
|
342
|
-
├── plans/
|
|
343
|
-
│ ├── en/implementation-plan.md
|
|
344
|
-
│ ├── ru/implementation-plan.md
|
|
345
|
-
│ └── ai/implementation-plan.md
|
|
346
|
-
├── contracts/
|
|
347
|
-
│ ├── api-changes.md # API contract diff
|
|
348
|
-
│ └── dto-comparison.md # Before/after DTOs
|
|
349
|
-
└── metrics/
|
|
350
|
-
└── analysis-metrics.md # Analysis metadata
|
|
336
|
+
<DOCS_ROOT>/analysis/<feature-name>/
|
|
337
|
+
├── report.md # Findings, API contract diff, DTO comparison, cross-repo impact, analysis metrics
|
|
338
|
+
└── implementation-plan.md # Actionable implementation plan derived from the analysis
|
|
351
339
|
```
|
|
352
340
|
|
|
353
|
-
|
|
341
|
+
No date-stamped folder: update `report.md`/`implementation-plan.md` in place for a rerun on the same feature rather than creating a parallel copy. Default output is a single English (`en`) pair. Add other language variants only if the user explicitly asks for them, and keep any variants you create synchronized.
|
|
342
|
+
|
|
343
|
+
`report.md` should use Gherkin-style scenarios where they make findings easier for other AI agents to parse.
|
|
354
344
|
> For full Gherkin output format and syntax rules, see `SKILL.detail.md`.
|
|
355
345
|
|
|
356
346
|
---
|
|
@@ -360,7 +350,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
360
350
|
- Every claim MUST reference specific code: `[filename.ts:L123](file:///absolute/path#L123)`
|
|
361
351
|
- Minimum 3 code examples per report
|
|
362
352
|
- Mermaid diagrams for architecture, tables for DTO changes, flowcharts for data flow
|
|
363
|
-
-
|
|
353
|
+
- Default to a single `en` document; add `ru`/`ai` (or other) variants only when the user explicitly asks, keeping them synchronized
|
|
364
354
|
|
|
365
355
|
---
|
|
366
356
|
|
|
@@ -378,7 +368,7 @@ The AI-readable format (`report/ai/`, `plans/ai/`) uses Gherkin-style scenarios.
|
|
|
378
368
|
|
|
379
369
|
## Step 13: Analysis Metrics
|
|
380
370
|
|
|
381
|
-
Track and include in `
|
|
371
|
+
Track and include in the `## Analysis Metrics` section of `report.md`:
|
|
382
372
|
- Duration, files analyzed (P0/P1/P2), lines changed
|
|
383
373
|
- Cross-repo dependencies, API endpoints changed, DTOs modified
|
|
384
374
|
- Breaking changes count, test coverage %, risk level
|
|
@@ -415,7 +405,7 @@ Follow `documentation-management.mdc`: update `<DOCS_ROOT>/readme.md`, add entry
|
|
|
415
405
|
4. **Never assume** — ask user when unclear
|
|
416
406
|
5. **Never skip** intermediate review for complex analyses (P0 files > 3)
|
|
417
407
|
6. **Always provide** concrete, actionable recommendations
|
|
418
|
-
7. **Always
|
|
408
|
+
7. **Always default** to a single-language document (`en`); add other language variants only when the user explicitly asks
|
|
419
409
|
|
|
420
410
|
---
|
|
421
411
|
|
|
@@ -57,6 +57,6 @@
|
|
|
57
57
|
| Field | Value |
|
|
58
58
|
|-------|-------|
|
|
59
59
|
| Base Dir | `.metaproject/jobs/<job-name>/ai/analysis` |
|
|
60
|
-
| Folder Name | `async-search
|
|
61
|
-
| Languages | `en
|
|
60
|
+
| Folder Name | `async-search` |
|
|
61
|
+
| Languages | `en` |
|
|
62
62
|
| Include Metrics | `true` |
|
package/src/gdskills/bundled/skills/orchestration/feature-analyzer/input-contract.schema.json
CHANGED
|
@@ -225,7 +225,7 @@
|
|
|
225
225
|
},
|
|
226
226
|
"folder_name": {
|
|
227
227
|
"type": "string",
|
|
228
|
-
"description": "Custom folder name. Defaults to <feature
|
|
228
|
+
"description": "Custom folder name. Defaults to <feature-name> (no date suffix) for both Mode A and Mode B"
|
|
229
229
|
},
|
|
230
230
|
"languages": {
|
|
231
231
|
"type": "array",
|
|
@@ -233,13 +233,13 @@
|
|
|
233
233
|
"type": "string",
|
|
234
234
|
"enum": ["en", "ru", "ai"]
|
|
235
235
|
},
|
|
236
|
-
"default": ["en"
|
|
237
|
-
"description": "Which report languages to generate. ai = Gherkin format for AI agents"
|
|
236
|
+
"default": ["en"],
|
|
237
|
+
"description": "Which report languages to generate. Defaults to a single 'en' document; add 'ru'/'ai' only when the user explicitly asks for them. ai = Gherkin format for AI agents"
|
|
238
238
|
},
|
|
239
239
|
"include_metrics": {
|
|
240
240
|
"type": "boolean",
|
|
241
241
|
"default": true,
|
|
242
|
-
"description": "Include
|
|
242
|
+
"description": "Include an Analysis Metrics section in report.md"
|
|
243
243
|
}
|
|
244
244
|
}
|
|
245
245
|
}
|
|
@@ -19,7 +19,7 @@
|
|
|
19
19
|
↓
|
|
20
20
|
[Субагент] → выполняет SKILL.md feature-analyzer автономно
|
|
21
21
|
↓
|
|
22
|
-
[Результат] → docs/analysis/<feature
|
|
22
|
+
[Результат] → docs/analysis/<feature>/
|
|
23
23
|
-->
|
|
24
24
|
|
|
25
25
|
## Инструкция для оркестратора
|
|
@@ -211,7 +211,7 @@ TICKET REFERENCE:
|
|
|
211
211
|
|
|
212
212
|
Base directory: {{output.base_dir | default("<DOCS_ROOT>/analysis")}}
|
|
213
213
|
Folder name: {{output.folder_name | default("auto-generated per SKILL.md rules")}}
|
|
214
|
-
Languages: {{output.languages | default("en
|
|
214
|
+
Languages: {{output.languages | default("en")}}
|
|
215
215
|
Include metrics: {{output.include_metrics | default("true")}}
|
|
216
216
|
|
|
217
217
|
═══════════════════════════════════════════════
|
|
@@ -10,7 +10,7 @@ triggers:
|
|
|
10
10
|
metadata:
|
|
11
11
|
author: "MrCipherSmith"
|
|
12
12
|
version: "2.0.0"
|
|
13
|
-
category: "
|
|
13
|
+
category: "orchestration"
|
|
14
14
|
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
15
15
|
license: "MIT"
|
|
16
16
|
---
|
|
@@ -97,7 +97,7 @@ End-to-end feature development workflow from idea to merge-ready PR.
|
|
|
97
97
|
Run `code-verifier` on the full diff:
|
|
98
98
|
|
|
99
99
|
```
|
|
100
|
-
Invoke: skills/orchestration/code-verifier/SKILL.md
|
|
100
|
+
Invoke: .metaproject/skills/gdskills/orchestration/code-verifier/SKILL.md
|
|
101
101
|
Input: codebase_path=<project_root>, scope=changed, base_branch=<base>
|
|
102
102
|
```
|
|
103
103
|
|
|
@@ -10,7 +10,7 @@ triggers:
|
|
|
10
10
|
metadata:
|
|
11
11
|
author: "MrCipherSmith"
|
|
12
12
|
version: "2.0.0"
|
|
13
|
-
category: "
|
|
13
|
+
category: "orchestration"
|
|
14
14
|
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
15
15
|
license: "MIT"
|
|
16
16
|
---
|
|
@@ -97,7 +97,7 @@ End-to-end feature development workflow from idea to merge-ready PR.
|
|
|
97
97
|
Run `code-verifier` on the full diff:
|
|
98
98
|
|
|
99
99
|
```
|
|
100
|
-
Invoke: skills/orchestration/code-verifier/SKILL.md
|
|
100
|
+
Invoke: .metaproject/skills/gdskills/orchestration/code-verifier/SKILL.md
|
|
101
101
|
Input: codebase_path=<project_root>, scope=changed, base_branch=<base>
|
|
102
102
|
```
|
|
103
103
|
|
|
@@ -10,7 +10,7 @@ triggers:
|
|
|
10
10
|
metadata:
|
|
11
11
|
author: "MrCipherSmith"
|
|
12
12
|
version: "2.0.0"
|
|
13
|
-
category: "
|
|
13
|
+
category: "orchestration"
|
|
14
14
|
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
15
15
|
license: "MIT"
|
|
16
16
|
---
|
|
@@ -97,7 +97,7 @@ End-to-end feature development workflow from idea to merge-ready PR.
|
|
|
97
97
|
Run `code-verifier` on the full diff:
|
|
98
98
|
|
|
99
99
|
```
|
|
100
|
-
Invoke: skills/orchestration/code-verifier/SKILL.md
|
|
100
|
+
Invoke: .metaproject/skills/gdskills/orchestration/code-verifier/SKILL.md
|
|
101
101
|
Input: codebase_path=<project_root>, scope=changed, base_branch=<base>
|
|
102
102
|
```
|
|
103
103
|
|
|
@@ -231,6 +231,7 @@ Required rules:
|
|
|
231
231
|
- `.metaproject/rules/core/error-handling.mdc`
|
|
232
232
|
- `.metaproject/rules/core/implementation-doc-mandate.mdc`
|
|
233
233
|
- `.metaproject/rules/core/execution-metrics.md`
|
|
234
|
+
- `.metaproject/rules/core/git-concurrency.mdc` — no `git stash`, no unscoped `git add`, pin every dispatch to its worktree
|
|
234
235
|
|
|
235
236
|
Execution metrics (opt-in): when a USER runs this orchestrator directly, at the
|
|
236
237
|
start ask "Collect execution statistics for this run? (yes/no)" per
|
|
@@ -320,6 +321,19 @@ worker reply is a `subagent-result` object
|
|
|
320
321
|
line is `STATUS: <status>`
|
|
321
322
|
(`.metaproject/rules/core/subagent-status-protocol.md`).
|
|
322
323
|
|
|
324
|
+
`<worktree_path>` below is the absolute root of the checkout this flow's work
|
|
325
|
+
happens in: the git worktree the flow runs in, or the project root when it runs
|
|
326
|
+
in the main checkout. Resolve it once, before the first dispatch
|
|
327
|
+
(`git rev-parse --show-toplevel` from that checkout), check
|
|
328
|
+
`git -C <worktree_path> branch --show-current` is the branch the work belongs
|
|
329
|
+
on, and pin every dispatch to it (`rules/core/git-concurrency.mdc`, "Pinning A
|
|
330
|
+
Dispatch To Its Worktree").
|
|
331
|
+
|
|
332
|
+
Workers do not commit: flow-orchestrator owns every commit and dispatches with
|
|
333
|
+
auto-commit off (task-implementer's `automation.auto_commit: false`). A
|
|
334
|
+
`subagent-dispatch` has no `automation` field, so the setting travels as the
|
|
335
|
+
explicit constraint below — never as the worker's default, which is `true`.
|
|
336
|
+
|
|
323
337
|
Dispatch payload, bound to the flow (map `target_skill` from the routing table):
|
|
324
338
|
|
|
325
339
|
```json
|
|
@@ -340,7 +354,10 @@ Dispatch payload, bound to the flow (map `target_skill` from the routing table):
|
|
|
340
354
|
"constraints": [
|
|
341
355
|
"Never edit flow.json.",
|
|
342
356
|
"Never edit frozen acceptance criteria.",
|
|
343
|
-
"Return a subagent-result; first line must be STATUS:."
|
|
357
|
+
"Return a subagent-result; first line must be STATUS:.",
|
|
358
|
+
"Follow rules/core/git-concurrency.mdc: never git stash; never git add -A.",
|
|
359
|
+
"Root: <worktree_path>. Before the first write run `cd <worktree_path> && pwd && git branch --show-current`; run every git command as `git -C <worktree_path>`.",
|
|
360
|
+
"Auto-commit is off (automation.auto_commit: false): do not commit; list every path you changed in changed_files. flow-orchestrator commits them at the task boundary."
|
|
344
361
|
],
|
|
345
362
|
"allowed_actions": ["read", "write", "run-command", "git"],
|
|
346
363
|
"output_contract": { "schema": "subagent-result", "artifact_path": ".metaproject/flows/<dir>/journal.md" },
|
|
@@ -349,12 +366,37 @@ Dispatch payload, bound to the flow (map `target_skill` from the routing table):
|
|
|
349
366
|
}
|
|
350
367
|
```
|
|
351
368
|
|
|
352
|
-
|
|
369
|
+
**Task boundary commit, after every accepted worker result** (`DONE` or
|
|
370
|
+
`DONE_WITH_CONCERNS`), then mark task progress. It commits exactly those
|
|
371
|
+
`changed_files` from the worker's `subagent-result` that git still shows as
|
|
372
|
+
changed, never `-A`/`--all`/`.` (`rules/core/git-concurrency.mdc` rule 4 and
|
|
373
|
+
"Reporting Back"). It skips cleanly when none remain. That is expected, not an
|
|
374
|
+
error, for a worker that reported no files (review, context, docs) or that
|
|
375
|
+
committed on its own (tests-creator's stubs):
|
|
353
376
|
|
|
354
377
|
```bash
|
|
378
|
+
set -- <each changed_files path, single-quoted: 'src/a.ts' 'docs/b c.md'> # may be none; an unquoted path with a space splits and is skipped
|
|
379
|
+
PENDING=()
|
|
380
|
+
for p in "$@"; do
|
|
381
|
+
[ -n "$(git -C <worktree_path> status --porcelain -- "$p")" ] && PENDING+=("$p")
|
|
382
|
+
done
|
|
383
|
+
if [ ${#PENDING[@]} -eq 0 ]; then
|
|
384
|
+
echo "boundary commit: nothing left to commit, skipped"
|
|
385
|
+
else
|
|
386
|
+
git -C <worktree_path> add -- "${PENDING[@]}"
|
|
387
|
+
git -C <worktree_path> commit -m "<type>(<scope>): <Tn title>
|
|
388
|
+
|
|
389
|
+
task: <Tn>" -- "${PENDING[@]}"
|
|
390
|
+
fi
|
|
355
391
|
keryx flow task done <id> <Tn>
|
|
356
392
|
```
|
|
357
393
|
|
|
394
|
+
Why each path is checked on its own: `git status --porcelain --` with no path
|
|
395
|
+
lists the whole tree, `git add` fails on a path that is neither on disk nor
|
|
396
|
+
tracked, and `git commit` with nothing staged exits 1. The trailing
|
|
397
|
+
`-- "${PENDING[@]}"` commits only these paths, so a file another lane staged is
|
|
398
|
+
not swept into this commit.
|
|
399
|
+
|
|
358
400
|
If new work is discovered:
|
|
359
401
|
|
|
360
402
|
```bash
|
|
@@ -369,8 +411,8 @@ properly formatted `subagent-result`.
|
|
|
369
411
|
|
|
370
412
|
| Worker `status` | flow-orchestrator action |
|
|
371
413
|
|---|---|
|
|
372
|
-
| `DONE` | Accept
|
|
373
|
-
| `DONE_WITH_CONCERNS` | Accept, record every concern in `journal.md`, decide continue vs. add a fix task, then `flow task done`. Never silently drop concerns. |
|
|
414
|
+
| `DONE` | Accept, run the task boundary commit (above), `keryx flow task done <id> <Tn>`. Continue. |
|
|
415
|
+
| `DONE_WITH_CONCERNS` | Accept, run the task boundary commit (above), record every concern in `journal.md`, decide continue vs. add a fix task, then `flow task done`. Never silently drop concerns. |
|
|
374
416
|
| `NEEDS_CONTEXT` | Do not fail. Enrich `context_refs`/`files_to_read` from gdgraph/gdctx/wiki/memory, then re-dispatch the same `dispatch_id`. |
|
|
375
417
|
| `BLOCKED` | `keryx flow block <id> --reason "<worker reason>"`; resolve or escalate one concise question, then `flow unblock` and re-dispatch. |
|
|
376
418
|
| `FAILED` | Emitted by harness **child** workers (`src/harness/child/contract.ts`), never by skill workers — `task-implementer` maps its own `failed` onto `BLOCKED`. Retry once with the same dispatch. If it fails again, block the flow and surface the error to the user. |
|
|
@@ -10,9 +10,9 @@ triggers:
|
|
|
10
10
|
metadata:
|
|
11
11
|
author: "MrCipherSmith"
|
|
12
12
|
version: "1.1.0"
|
|
13
|
-
category: "
|
|
13
|
+
category: "orchestration"
|
|
14
14
|
agent_worthy: true
|
|
15
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
16
16
|
license: "MIT"
|
|
17
17
|
---
|
|
18
18
|
|
|
@@ -10,9 +10,9 @@ triggers:
|
|
|
10
10
|
metadata:
|
|
11
11
|
author: "MrCipherSmith"
|
|
12
12
|
version: "1.1.0"
|
|
13
|
-
category: "
|
|
13
|
+
category: "orchestration"
|
|
14
14
|
agent_worthy: true
|
|
15
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
16
16
|
license: "MIT"
|
|
17
17
|
---
|
|
18
18
|
|
|
@@ -10,9 +10,9 @@ triggers:
|
|
|
10
10
|
metadata:
|
|
11
11
|
author: "MrCipherSmith"
|
|
12
12
|
version: "1.1.0"
|
|
13
|
-
category: "
|
|
13
|
+
category: "orchestration"
|
|
14
14
|
agent_worthy: true
|
|
15
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
16
16
|
license: "MIT"
|
|
17
17
|
---
|
|
18
18
|
|
|
@@ -10,9 +10,9 @@ triggers:
|
|
|
10
10
|
metadata:
|
|
11
11
|
author: "MrCipherSmith"
|
|
12
12
|
version: "1.1.0"
|
|
13
|
-
category: "
|
|
13
|
+
category: "orchestration"
|
|
14
14
|
agent_worthy: true
|
|
15
|
-
compatible_harnesses: "cursor,codex,zed,opencode"
|
|
15
|
+
compatible_harnesses: "cursor,codex,zed,opencode,claude"
|
|
16
16
|
license: "MIT"
|
|
17
17
|
---
|
|
18
18
|
|