mandrel 2.0.0 → 2.1.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/.agents/README.md +59 -28
- package/.agents/agents/acceptance-critic.md +9 -7
- package/.agents/agents/story-worker.md +41 -46
- package/.agents/audit-checklists/performance.md +1 -1
- package/.agents/docs/SDLC.md +51 -44
- package/.agents/docs/agentrc-reference.json +8 -13
- package/.agents/docs/configuration.md +32 -56
- package/.agents/docs/execution-reference.md +39 -10
- package/.agents/docs/quality-gates.md +14 -16
- package/.agents/docs/workflows.md +6 -6
- package/.agents/instructions.md +64 -79
- package/.agents/rules/ci-remediation.md +3 -3
- package/.agents/rules/git-conventions-reference.md +42 -51
- package/.agents/schemas/agentrc.schema.json +34 -45
- package/.agents/schemas/audit-rules.json +59 -1
- package/.agents/schemas/audit-rules.schema.json +33 -1
- package/.agents/schemas/lifecycle/README.md +1 -2
- package/.agents/schemas/lifecycle/ledger-record.schema.json +1 -1
- package/.agents/schemas/lifecycle/merge.flip-failed.schema.json +33 -0
- package/.agents/schemas/lifecycle/merge.unlanded.schema.json +1 -0
- package/.agents/schemas/lifecycle/story.merged.schema.json +1 -1
- package/.agents/schemas/signal-event.schema.json +3 -3
- package/.agents/schemas/story-deliver-terminal.schema.json +152 -0
- package/.agents/schemas/validation-evidence.schema.json +1 -1
- package/.agents/scripts/acceptance-eval.js +22 -66
- package/.agents/scripts/agents-bootstrap-github.js +1 -1
- package/.agents/scripts/bootstrap.js +3 -3
- package/.agents/scripts/check-dead-exports.js +43 -104
- package/.agents/scripts/check-doc-links.js +2 -2
- package/.agents/scripts/check-lifecycle-lint.js +1 -1
- package/.agents/scripts/check-workflow-cli-lint.js +91 -0
- package/.agents/scripts/deliver-recover.js +122 -0
- package/.agents/scripts/drain-pending-cleanup.js +1 -1
- package/.agents/scripts/evidence-gate.js +20 -50
- package/.agents/scripts/generate-skills-index.js +17 -1
- package/.agents/scripts/generate-workflows-doc.js +4 -4
- package/.agents/scripts/lib/ITicketingProvider.js +1 -19
- package/.agents/scripts/lib/audit-suite/selector.js +323 -23
- package/.agents/scripts/lib/baselines/kinds/maintainability.js +0 -11
- package/.agents/scripts/lib/bootstrap/ci-workflow-template.js +28 -33
- package/.agents/scripts/lib/bootstrap/manifest.js +8 -11
- package/.agents/scripts/lib/bootstrap/project-bootstrap.js +30 -53
- package/.agents/scripts/lib/bootstrap/quality-bootstrap.js +0 -2
- package/.agents/scripts/lib/checks/core-bare-clean.js +4 -1
- package/.agents/scripts/lib/checks/index.js +1 -1
- package/.agents/scripts/lib/checks/loop-health.js +12 -11
- package/.agents/scripts/lib/checks/state.js +17 -248
- package/.agents/scripts/lib/checks/story-init-not-backgrounded.js +3 -3
- package/.agents/scripts/lib/checks/subagent-agent-tool-required.js +3 -4
- package/.agents/scripts/lib/checks/worktree-bootstrap-env.js +2 -2
- package/.agents/scripts/lib/checks/worktree-residue-biome.js +3 -3
- package/.agents/scripts/lib/cli-args.js +23 -2
- package/.agents/scripts/lib/close-validation/gates.js +13 -13
- package/.agents/scripts/lib/close-validation/projections/inputs.js +7 -7
- package/.agents/scripts/lib/close-validation/projections/maintainability.js +12 -12
- package/.agents/scripts/lib/close-validation/runner.js +13 -21
- package/.agents/scripts/lib/close-validation/telemetry.js +17 -8
- package/.agents/scripts/lib/config/delivery-routing.js +7 -6
- package/.agents/scripts/lib/config/explain.js +10 -16
- package/.agents/scripts/lib/config/github.js +7 -5
- package/.agents/scripts/lib/config/limits.js +15 -25
- package/.agents/scripts/lib/config/quality.js +11 -14
- package/.agents/scripts/lib/config/runners.js +8 -21
- package/.agents/scripts/lib/config/temp-paths.js +18 -56
- package/.agents/scripts/lib/config-settings-schema-delivery.js +31 -13
- package/.agents/scripts/lib/config-settings-schema-quality.js +9 -2
- package/.agents/scripts/lib/config-settings-schema.js +48 -22
- package/.agents/scripts/lib/dead-exports-knip.js +105 -0
- package/.agents/scripts/lib/dead-exports-mode.js +51 -0
- package/.agents/scripts/lib/duplicate-search.js +38 -7
- package/.agents/scripts/lib/findings/promote-finding.js +23 -14
- package/.agents/scripts/lib/format-generated-json.js +97 -0
- package/.agents/scripts/lib/framework-version.js +19 -189
- package/.agents/scripts/lib/gh-exec.js +8 -0
- package/.agents/scripts/lib/git-branch-lifecycle.js +0 -158
- package/.agents/scripts/lib/git-utils.js +0 -14
- package/.agents/scripts/lib/json-utils.js +1 -2
- package/.agents/scripts/lib/label-constants.js +0 -15
- package/.agents/scripts/lib/label-taxonomy.js +1 -12
- package/.agents/scripts/lib/observability/active-story-env.js +42 -163
- package/.agents/scripts/lib/observability/runtime-friction.js +243 -0
- package/.agents/scripts/lib/observability/signal-validator.js +4 -4
- package/.agents/scripts/lib/observability/signals-writer.js +6 -82
- package/.agents/scripts/lib/observability/source-classifier.js +2 -2
- package/.agents/scripts/lib/observability/tool-trace-hook.js +2 -12
- package/.agents/scripts/lib/orchestration/acceptance-clusters.js +1 -1
- package/.agents/scripts/lib/orchestration/acceptance-eval-decision.js +2 -2
- package/.agents/scripts/lib/orchestration/ceremony-routing.js +43 -45
- package/.agents/scripts/lib/orchestration/code-review.js +58 -168
- package/.agents/scripts/lib/orchestration/consolidation-precondition.js +3 -3
- package/.agents/scripts/lib/orchestration/deliver-recover.js +328 -0
- package/.agents/scripts/lib/orchestration/detectors-phase.js +12 -6
- package/.agents/scripts/lib/orchestration/git-cleanup/phases/fast-forward.js +34 -0
- package/.agents/scripts/lib/orchestration/lease-guard-shared.js +3 -2
- package/.agents/scripts/lib/orchestration/lifecycle/emit-ledger-event.js +142 -0
- package/.agents/scripts/lib/orchestration/lifecycle/emit-loop-tick.js +9 -11
- package/.agents/scripts/lib/orchestration/lifecycle/emit-merge-flip-failed.js +86 -0
- package/.agents/scripts/lib/orchestration/lifecycle/emit-merge-unlanded.js +37 -103
- package/.agents/scripts/lib/orchestration/lifecycle/listeners/README.md +7 -3
- package/.agents/scripts/lib/orchestration/lifecycle/listeners/watcher.js +50 -85
- package/.agents/scripts/lib/orchestration/lifecycle/trace-logger.js +3 -14
- package/.agents/scripts/lib/orchestration/merge-block-class.js +76 -20
- package/.agents/scripts/lib/orchestration/merge-poll.js +104 -0
- package/.agents/scripts/lib/orchestration/plan-context.js +114 -24
- package/.agents/scripts/lib/orchestration/plan-critic-conditions.js +11 -22
- package/.agents/scripts/lib/orchestration/plan-critics-evaluate.js +3 -7
- package/.agents/scripts/lib/orchestration/plan-metrics.js +38 -6
- package/.agents/scripts/lib/orchestration/plan-persist/fan-out-gate.js +16 -6
- package/.agents/scripts/lib/orchestration/plan-persist/persist-helpers.js +173 -25
- package/.agents/scripts/lib/orchestration/plan-persist/plan-context-source.js +116 -0
- package/.agents/scripts/lib/orchestration/plan-persist/run-plan-persist.js +280 -75
- package/.agents/scripts/lib/orchestration/plan-persist/story-ops.js +472 -55
- package/.agents/scripts/lib/orchestration/plan-persist/summary.js +21 -16
- package/.agents/scripts/lib/orchestration/plan-persist/supersede-ops.js +509 -0
- package/.agents/scripts/lib/orchestration/planning/authoring-context.js +41 -40
- package/.agents/scripts/lib/orchestration/planning/spec-authoring-grounding.js +1 -1
- package/.agents/scripts/lib/orchestration/resolve-stories.js +344 -0
- package/.agents/scripts/lib/orchestration/retro-proposals.js +7 -7
- package/.agents/scripts/lib/orchestration/review-depth.js +105 -40
- package/.agents/scripts/lib/orchestration/review-providers/findings-renderer.js +3 -13
- package/.agents/scripts/lib/orchestration/review-providers/native.js +1 -154
- package/.agents/scripts/lib/orchestration/review-providers/review-depth.js +3 -2
- package/.agents/scripts/lib/orchestration/review-providers/security-review.js +1 -1
- package/.agents/scripts/lib/orchestration/review-providers/types.js +5 -4
- package/.agents/scripts/lib/orchestration/review-providers/ultrareview.js +1 -1
- package/.agents/scripts/lib/orchestration/run-epilogue.js +374 -16
- package/.agents/scripts/lib/orchestration/single-story-close/phases/auto-merge.js +24 -0
- package/.agents/scripts/lib/orchestration/single-story-close/phases/base-sync.js +11 -9
- package/.agents/scripts/lib/orchestration/single-story-close/phases/close-validation.js +4 -4
- package/.agents/scripts/lib/orchestration/single-story-close/phases/code-review.js +3 -12
- package/.agents/scripts/lib/orchestration/single-story-close/phases/confirm-merge.js +608 -152
- package/.agents/scripts/lib/orchestration/single-story-close/phases/options.js +72 -30
- package/.agents/scripts/lib/orchestration/single-story-close/phases/post-land.js +305 -0
- package/.agents/scripts/lib/orchestration/single-story-close/phases/pull-request.js +1 -1
- package/.agents/scripts/lib/orchestration/single-story-close/phases/review-block.js +12 -8
- package/.agents/scripts/lib/orchestration/single-story-close/phases/worktree-reap.js +37 -4
- package/.agents/scripts/lib/orchestration/single-story-close/phases/wrong-tree-guard.js +2 -2
- package/.agents/scripts/lib/orchestration/single-story-close/runner.js +264 -43
- package/.agents/scripts/lib/orchestration/single-story-lease-guard.js +1 -1
- package/.agents/scripts/lib/orchestration/story-close/format-autofix.js +10 -10
- package/.agents/scripts/lib/orchestration/story-close/phases/code-review.js +15 -32
- package/.agents/scripts/lib/orchestration/story-deliver-terminal.js +360 -0
- package/.agents/scripts/lib/orchestration/story-follow-ups.js +75 -14
- package/.agents/scripts/lib/orchestration/story-init-remote.js +12 -8
- package/.agents/scripts/lib/orchestration/story-plan-state.js +14 -29
- package/.agents/scripts/lib/orchestration/task-body-validator.js +52 -7
- package/.agents/scripts/lib/orchestration/ticket-lease.js +27 -74
- package/.agents/scripts/lib/orchestration/ticket-validator-conflicts.js +119 -14
- package/.agents/scripts/lib/orchestration/ticket-validator-sizing.js +3 -4
- package/.agents/scripts/lib/orchestration/ticket-validator.js +121 -18
- package/.agents/scripts/lib/orchestration/ticketing/bulk.js +14 -47
- package/.agents/scripts/lib/orchestration/ticketing/reads.js +19 -32
- package/.agents/scripts/lib/orchestration/ticketing/transition.js +61 -1
- package/.agents/scripts/lib/orchestration/ticketing.js +0 -1
- package/.agents/scripts/lib/plan-phase-cleanup.js +12 -14
- package/.agents/scripts/lib/planning-corpus.js +12 -286
- package/.agents/scripts/lib/preflight-runner.js +2 -2
- package/.agents/scripts/lib/qa/qa-context-hydrator.js +5 -5
- package/.agents/scripts/lib/signals/index.js +4 -17
- package/.agents/scripts/lib/signals/read.js +35 -35
- package/.agents/scripts/lib/signals/schema.js +8 -11
- package/.agents/scripts/lib/signals/span-tree.js +7 -7
- package/.agents/scripts/lib/signals/write.js +0 -1
- package/.agents/scripts/lib/single-story/story-merged-notify.js +13 -2
- package/.agents/scripts/lib/skills/parse-skill.js +16 -3
- package/.agents/scripts/lib/story-adjacency.js +8 -7
- package/.agents/scripts/lib/story-body/story-body.js +6 -5
- package/.agents/scripts/lib/templates/decomposer-prompts.js +7 -3
- package/.agents/scripts/lib/test-env.js +14 -1
- package/.agents/scripts/lib/test-tiers.js +0 -3
- package/.agents/scripts/lib/ticket-body-sections.js +0 -14
- package/.agents/scripts/lib/validation-evidence.js +31 -59
- package/.agents/scripts/lib/wave-runner/ready-set.js +32 -6
- package/.agents/scripts/lib/worktree/lifecycle/pending-cleanup.js +1 -1
- package/.agents/scripts/lib/worktree/lifecycle/reap.js +68 -19
- package/.agents/scripts/lib/worktree/lifecycle-manager.js +1 -2
- package/.agents/scripts/plan-context.js +38 -6
- package/.agents/scripts/plan-persist.js +145 -35
- package/.agents/scripts/plan-run-epilogue.js +83 -38
- package/.agents/scripts/post-structured-comment.js +0 -38
- package/.agents/scripts/pr-watch-with-update.js +43 -22
- package/.agents/scripts/providers/github/compose.js +0 -1
- package/.agents/scripts/providers/github/errors.js +0 -19
- package/.agents/scripts/providers/github/issues.js +1 -11
- package/.agents/scripts/providers/github/mappers.js +5 -0
- package/.agents/scripts/providers/github/sub-issues.js +0 -47
- package/.agents/scripts/providers/github/tickets.js +33 -153
- package/.agents/scripts/providers/github.js +17 -6
- package/.agents/scripts/resolve-stories.js +236 -0
- package/.agents/scripts/run-coverage.js +4 -1
- package/.agents/scripts/run-lint.js +2 -2
- package/.agents/scripts/run-verify.js +31 -2
- package/.agents/scripts/signals-view.js +9 -10
- package/.agents/scripts/single-story-close.js +173 -18
- package/.agents/scripts/single-story-confirm-merge.js +288 -15
- package/.agents/scripts/single-story-init.js +6 -10
- package/.agents/scripts/stories-wave-tick.js +79 -4
- package/.agents/scripts/story-plan.js +3 -3
- package/.agents/scripts/update-ticket-state.js +8 -50
- package/.agents/skills/core/code-review-and-quality/SKILL.md +28 -450
- package/.agents/skills/core/code-review-and-quality/reference.md +458 -0
- package/.agents/skills/core/debugging-and-error-recovery/SKILL.md +22 -315
- package/.agents/skills/core/debugging-and-error-recovery/reference.md +323 -0
- package/.agents/skills/core/diagnose-friction/SKILL.md +14 -18
- package/.agents/skills/core/documentation-and-adrs/SKILL.md +25 -397
- package/.agents/skills/core/documentation-and-adrs/reference.md +403 -0
- package/.agents/skills/core/gates-and-baselines/SKILL.md +12 -12
- package/.agents/skills/core/idea-refinement/SKILL.md +3 -3
- package/.agents/skills/core/scope-triage/SKILL.md +3 -0
- package/.agents/skills/core/security-and-hardening/SKILL.md +22 -367
- package/.agents/skills/core/security-and-hardening/reference.md +375 -0
- package/.agents/skills/skills.index.json +2 -12
- package/.agents/skills/stack/qa/playwright-bdd/SKILL.md +2 -4
- package/.agents/skills/stack/qa/qa-explore-driving/SKILL.md +1 -1
- package/.agents/skills/stack/qa/qa-harness/SKILL.md +1 -3
- package/.agents/workflows/audit-architecture.md +3 -4
- package/.agents/workflows/audit-clean-code.md +4 -4
- package/.agents/workflows/audit-documentation.md +4 -5
- package/.agents/workflows/audit-lighthouse.md +8 -0
- package/.agents/workflows/audit-navigability.md +10 -0
- package/.agents/workflows/audit-performance.md +2 -3
- package/.agents/workflows/audit-quality.md +8 -9
- package/.agents/workflows/audit-security.md +1 -2
- package/.agents/workflows/audit-seo.md +10 -0
- package/.agents/workflows/audit-ux-ui.md +7 -0
- package/.agents/workflows/deliver.md +98 -45
- package/.agents/workflows/git-cleanup.md +2 -2
- package/.agents/workflows/git-deliver.md +1 -1
- package/.agents/workflows/helpers/acceptance-self-eval.md +21 -13
- package/.agents/workflows/helpers/code-quality-guardrails.md +7 -7
- package/.agents/workflows/helpers/code-review.md +12 -10
- package/.agents/workflows/helpers/deliver-story-reference.md +73 -32
- package/.agents/workflows/helpers/deliver-story.md +193 -118
- package/.agents/workflows/helpers/parallel-tooling.md +2 -2
- package/.agents/workflows/helpers/worktree-lifecycle.md +28 -32
- package/.agents/workflows/plan.md +184 -19
- package/.agents/workflows/qa-assist.md +6 -6
- package/.agents/workflows/qa-explore.md +3 -3
- package/.agents/workflows/qa-run.md +1 -5
- package/bin/mandrel.js +12 -1
- package/docs/CHANGELOG.md +40 -0
- package/lib/cli/registry.js +262 -19
- package/lib/cli/sync-agents.js +157 -0
- package/lib/cli/sync-commands.js +115 -6
- package/lib/cli/sync.js +168 -6
- package/lib/cli/update.js +105 -8
- package/lib/cli/version-helpers.js +131 -0
- package/lib/migrations/README.md +7 -5
- package/lib/migrations/index.js +12 -9
- package/lib/migrations/steps/2.1.0-retire-mi-drop-knobs.js +100 -0
- package/lib/migrations/steps/2.1.0-retire-verify-concurrency-cap.js +101 -0
- package/package.json +1 -1
- package/.agents/schemas/epic-perf-report.schema.json +0 -89
- package/.agents/schemas/lifecycle/acceptance.reconcile.failed.schema.json +0 -13
- package/.agents/schemas/lifecycle/acceptance.reconcile.ok.schema.json +0 -13
- package/.agents/schemas/lifecycle/acceptance.reconcile.skipped.schema.json +0 -13
- package/.agents/schemas/lifecycle/acceptance.reconcile.start.schema.json +0 -12
- package/.agents/schemas/lifecycle/acceptance.reconcile.waived.schema.json +0 -13
- package/.agents/schemas/lifecycle/epic.automerge.end.schema.json +0 -15
- package/.agents/schemas/lifecycle/epic.automerge.start.schema.json +0 -13
- package/.agents/schemas/lifecycle/epic.blocked.schema.json +0 -13
- package/.agents/schemas/lifecycle/epic.cleanup.end.schema.json +0 -12
- package/.agents/schemas/lifecycle/epic.cleanup.start.schema.json +0 -12
- package/.agents/schemas/lifecycle/epic.close.end.schema.json +0 -12
- package/.agents/schemas/lifecycle/epic.complete.schema.json +0 -13
- package/.agents/schemas/lifecycle/epic.finalize.end.schema.json +0 -13
- package/.agents/schemas/lifecycle/epic.finalize.start.schema.json +0 -12
- package/.agents/schemas/lifecycle/epic.merge.armed.schema.json +0 -13
- package/.agents/schemas/lifecycle/epic.merge.blocked.schema.json +0 -14
- package/.agents/schemas/lifecycle/epic.merge.confirmed.schema.json +0 -17
- package/.agents/schemas/lifecycle/epic.merge.ready.schema.json +0 -15
- package/.agents/schemas/lifecycle/epic.plan.end.schema.json +0 -18
- package/.agents/schemas/lifecycle/epic.plan.start.schema.json +0 -12
- package/.agents/schemas/lifecycle/epic.snapshot.end.schema.json +0 -16
- package/.agents/schemas/lifecycle/epic.snapshot.start.schema.json +0 -12
- package/.agents/schemas/lifecycle/epic.watch.end.schema.json +0 -29
- package/.agents/schemas/lifecycle/epic.watch.start.schema.json +0 -16
- package/.agents/schemas/lifecycle/story.heartbeat.schema.json +0 -20
- package/.agents/schemas/risk-verdict.schema.json +0 -53
- package/.agents/schemas/story-perf-summary.schema.json +0 -73
- package/.agents/scripts/analyze-execution.js +0 -444
- package/.agents/scripts/check-prepush-recovery.js +0 -90
- package/.agents/scripts/lib/git-merge-orchestrator.js +0 -261
- package/.agents/scripts/lib/observability/baseline-refresh-rate.js +0 -221
- package/.agents/scripts/lib/observability/hook-heartbeat.js +0 -187
- package/.agents/scripts/lib/observability/perf-aggregator.js +0 -813
- package/.agents/scripts/lib/observability/perf-report-readers.js +0 -328
- package/.agents/scripts/lib/observability/perf-report-render.js +0 -182
- package/.agents/scripts/lib/orchestration/audit-lens-routing.js +0 -128
- package/.agents/scripts/lib/orchestration/bookkeeping-outbox.js +0 -273
- package/.agents/scripts/lib/orchestration/error-journal.js +0 -139
- package/.agents/scripts/lib/orchestration/lifecycle/emit-story-heartbeat.js +0 -155
- package/.agents/scripts/lib/orchestration/lifecycle/ledger-diff.js +0 -140
- package/.agents/scripts/lib/orchestration/lifecycle/listeners/merge-watcher.js +0 -665
- package/.agents/scripts/lib/orchestration/plan-review-routing.js +0 -63
- package/.agents/scripts/lib/orchestration/planning/risk-verdict.js +0 -104
- package/.agents/scripts/lib/orchestration/planning-context-budget.js +0 -213
- package/.agents/scripts/lib/orchestration/planning-risk.js +0 -194
- package/.agents/scripts/lib/orchestration/post-merge/phases/branch-cleanup.js +0 -56
- package/.agents/scripts/lib/orchestration/post-merge/phases/dashboard-refresh.js +0 -21
- package/.agents/scripts/lib/orchestration/post-merge/phases/notification.js +0 -78
- package/.agents/scripts/lib/orchestration/post-merge/phases/temp-cleanup.js +0 -68
- package/.agents/scripts/lib/orchestration/post-merge/phases/ticket-closure.js +0 -118
- package/.agents/scripts/lib/orchestration/post-merge/phases/worktree-reap.js +0 -397
- package/.agents/scripts/lib/orchestration/preflight-cache.js +0 -187
- package/.agents/scripts/lib/orchestration/resolve-plan-run.js +0 -155
- package/.agents/scripts/lib/orchestration/retro-perf-heuristics.js +0 -275
- package/.agents/scripts/lib/orchestration/story-progress/story-run-progress-writer.js +0 -400
- package/.agents/scripts/lib/single-story/confirm-merge-follow-ups.js +0 -36
- package/.agents/scripts/resolve-plan-run.js +0 -117
- package/.agents/skills/core/analyze-execution/SKILL.md +0 -101
|
@@ -2,7 +2,7 @@
|
|
|
2
2
|
* phases/authoring-context.js — emit-context phase.
|
|
3
3
|
*
|
|
4
4
|
* Builds the authoring context the host LLM (or the
|
|
5
|
-
*
|
|
5
|
+
* `/plan` author step) needs to write the Tech Spec.
|
|
6
6
|
* Returns a plain JSON-serialisable object; never hits the network beyond
|
|
7
7
|
* the provider call needed to load the Epic.
|
|
8
8
|
*/
|
|
@@ -15,13 +15,12 @@ import {
|
|
|
15
15
|
} from '../../bdd-runner-detect.js';
|
|
16
16
|
import { scanBddScenarios } from '../../bdd-scenario-scanner.js';
|
|
17
17
|
import { buildCodebaseSnapshot } from '../../codebase-snapshot.js';
|
|
18
|
-
import {
|
|
18
|
+
import { getPaths, PROJECT_ROOT } from '../../config-resolver.js';
|
|
19
19
|
import { scanMemoryFreshness } from '../../feedback-loop/memory-freshness.js';
|
|
20
20
|
import { fetchPriorFeedback } from '../../feedback-loop/prior-feedback-fetcher.js';
|
|
21
21
|
import { Logger } from '../../Logger.js';
|
|
22
22
|
import { hasTicketSection } from '../../ticket-body-sections.js';
|
|
23
23
|
import { ensureDocsDigest } from '../docs-digest.js';
|
|
24
|
-
import { applyBudget } from '../planning-context-budget.js';
|
|
25
24
|
import { collectReferences, hasNewFileCue } from '../spec-freshness.js';
|
|
26
25
|
import { buildAuthoringGrounding } from './spec-authoring-grounding.js';
|
|
27
26
|
|
|
@@ -60,11 +59,11 @@ function resolveMemoryDir({ github } = {}) {
|
|
|
60
59
|
*
|
|
61
60
|
* Ensures a docs digest exists at the **same** per-Epic temp path the
|
|
62
61
|
* `/deliver` story sub-agents already consume
|
|
63
|
-
* (`<tempRoot>/epic-<
|
|
62
|
+
* (`<tempRoot>/epic-<seedIssueId>/docs-digest.md`) via the shared
|
|
64
63
|
* `ensureDocsDigest` export in `docs-digest.js` — one generator, one file,
|
|
65
64
|
* reused across the planning and delivery surfaces for the same Epic. The
|
|
66
65
|
* envelope carries only the digest path, not embedded doc content; the
|
|
67
|
-
* planner (host LLM
|
|
66
|
+
* planner (host LLM driving the `/plan` author step) reads the digest
|
|
68
67
|
* and pulls a full file/section on demand when it bears on the decision.
|
|
69
68
|
*
|
|
70
69
|
* Returns `null` — a silent no-op — when `project.docsContextFiles` is not
|
|
@@ -72,10 +71,10 @@ function resolveMemoryDir({ github } = {}) {
|
|
|
72
71
|
* planning; that fallback remains a `doc-reader.js` primitive used
|
|
73
72
|
* elsewhere, not part of this envelope.
|
|
74
73
|
*
|
|
75
|
-
* @param {{
|
|
74
|
+
* @param {{ seedIssueId: number, settings: object, cwd: string }} args
|
|
76
75
|
* @returns {Promise<{ mode: 'digest', digestPath: string } | null>}
|
|
77
76
|
*/
|
|
78
|
-
async function buildPlanningDocsContext({
|
|
77
|
+
async function buildPlanningDocsContext({ seedIssueId, settings, cwd }) {
|
|
79
78
|
const docsContextFiles = Array.isArray(settings?.docsContextFiles)
|
|
80
79
|
? settings.docsContextFiles
|
|
81
80
|
: [];
|
|
@@ -83,7 +82,11 @@ async function buildPlanningDocsContext({ epicId, settings, cwd }) {
|
|
|
83
82
|
|
|
84
83
|
const paths = getPaths({ project: { paths: settings?.paths } });
|
|
85
84
|
const docsRoot = path.resolve(cwd, paths.docsRoot);
|
|
86
|
-
const relPath = path.join(
|
|
85
|
+
const relPath = path.join(
|
|
86
|
+
paths.tempRoot,
|
|
87
|
+
`epic-${seedIssueId}`,
|
|
88
|
+
'docs-digest.md',
|
|
89
|
+
);
|
|
87
90
|
const absPath = path.resolve(cwd, relPath);
|
|
88
91
|
|
|
89
92
|
const result = await ensureDocsDigest({
|
|
@@ -97,18 +100,23 @@ async function buildPlanningDocsContext({ epicId, settings, cwd }) {
|
|
|
97
100
|
|
|
98
101
|
/**
|
|
99
102
|
* Build the authoring context the host LLM (or the
|
|
100
|
-
*
|
|
103
|
+
* `/plan` author step) needs to write the Tech Spec.
|
|
101
104
|
*
|
|
102
105
|
* `docsContext` is digest-first (Story #4433): a pointer at the per-Epic
|
|
103
106
|
* docs digest rather than embedded doc content, `null` when
|
|
104
|
-
* `project.docsContextFiles` is unset.
|
|
105
|
-
*
|
|
106
|
-
*
|
|
107
|
-
*
|
|
108
|
-
*
|
|
107
|
+
* `project.docsContextFiles` is unset.
|
|
108
|
+
*
|
|
109
|
+
* The body carried here is **unbounded**. Story #4541 removed the
|
|
110
|
+
* `applyBudget` pass (and the `--full-context` flag that toggled it): both
|
|
111
|
+
* `plan-context.js` envelope builders discard this `epic` field entirely and
|
|
112
|
+
* ship the raw seed on `seed.content` instead, so the budget bounded nothing
|
|
113
|
+
* that shipped and the flag was a no-op. The raw-seed passthrough is
|
|
114
|
+
* deliberate and test-pinned; the budget pass was the orphan. Envelope size
|
|
115
|
+
* is guarded where it is actually decided — `PLAN_CONTEXT_ENVELOPE_BYTE_CEILING`
|
|
116
|
+
* in `lib/orchestration/plan-context.js`.
|
|
109
117
|
*/
|
|
110
118
|
export async function buildAuthoringContext(
|
|
111
|
-
|
|
119
|
+
seedIssueId,
|
|
112
120
|
provider,
|
|
113
121
|
settings = {},
|
|
114
122
|
opts = {},
|
|
@@ -118,26 +126,19 @@ export async function buildAuthoringContext(
|
|
|
118
126
|
// envelope build, which also needs the raw body for clarity scoring and
|
|
119
127
|
// re-plan detection) does not pay a second provider fetch. Absent, the
|
|
120
128
|
// fetch behaviour is unchanged.
|
|
121
|
-
const epic = opts.epic ?? (await provider.getEpic(
|
|
129
|
+
const epic = opts.epic ?? (await provider.getEpic(seedIssueId));
|
|
122
130
|
if (!epic) {
|
|
123
|
-
throw new Error(`Epic #${
|
|
131
|
+
throw new Error(`Epic #${seedIssueId} not found.`);
|
|
124
132
|
}
|
|
125
133
|
|
|
126
|
-
const
|
|
127
|
-
const { fullContext = false, cwd = PROJECT_ROOT } = opts;
|
|
134
|
+
const { cwd = PROJECT_ROOT } = opts;
|
|
128
135
|
|
|
129
136
|
const docsContext = await buildPlanningDocsContext({
|
|
130
|
-
|
|
137
|
+
seedIssueId: epic.id,
|
|
131
138
|
settings,
|
|
132
139
|
cwd,
|
|
133
140
|
});
|
|
134
141
|
|
|
135
|
-
const epicBody = applyBudget(
|
|
136
|
-
[{ path: `epic-${epic.id}.md`, content: epic.body ?? '' }],
|
|
137
|
-
planningLimits,
|
|
138
|
-
{ fullContext },
|
|
139
|
-
);
|
|
140
|
-
|
|
141
142
|
// Story #2094 Task #2103 — verify the project's BDD runner pending-tag
|
|
142
143
|
// support so the acceptance-spec body can record either the verified tag
|
|
143
144
|
// (features-first ordering) or "fallback: dependencies-first ordering"
|
|
@@ -153,7 +154,7 @@ export async function buildAuthoringContext(
|
|
|
153
154
|
const featureRoots = resolveFeatureRoots({ cwd: PROJECT_ROOT });
|
|
154
155
|
bddScenarios = scanBddScenarios({ featureRoots });
|
|
155
156
|
} catch (err) {
|
|
156
|
-
Logger.warn(`[
|
|
157
|
+
Logger.warn(`[plan-context] BDD scenario scan skipped: ${err.message}`);
|
|
157
158
|
}
|
|
158
159
|
|
|
159
160
|
// Story #2557 — memory-freshness pre-flight runs BEFORE the prior-feedback
|
|
@@ -220,37 +221,37 @@ export async function buildAuthoringContext(
|
|
|
220
221
|
if (grounding.truncation) {
|
|
221
222
|
const { dropped, matched, tier } = grounding.truncation;
|
|
222
223
|
Logger.warn(
|
|
223
|
-
`[
|
|
224
|
+
`[plan-context] codebase snapshot truncated: ${dropped} of ` +
|
|
224
225
|
`${matched} matched file(s) dropped from the ${tier}-tier view. ` +
|
|
225
|
-
`The
|
|
226
|
+
`The /plan authoring context is partial. To restore full ` +
|
|
227
|
+
`grounding, ` +
|
|
226
228
|
`set planning.codebaseSnapshot.tier: "medium" and/or narrow ` +
|
|
227
229
|
`planning.codebaseSnapshot.include in .agentrc.json.`,
|
|
228
230
|
);
|
|
229
231
|
}
|
|
230
232
|
if (grounding.citedButAbsent.length > 0) {
|
|
231
233
|
Logger.warn(
|
|
232
|
-
`[
|
|
233
|
-
`in the
|
|
234
|
-
`${grounding.citedButAbsent.join(', ')}.
|
|
235
|
-
`
|
|
234
|
+
`[plan-context] ${grounding.citedButAbsent.length} path(s) cited ` +
|
|
235
|
+
`in the authored Spec are absent from the codebase snapshot: ` +
|
|
236
|
+
`${grounding.citedButAbsent.join(', ')}. /plan will flag these ` +
|
|
237
|
+
`as drift unless they are net-new.`,
|
|
236
238
|
);
|
|
237
239
|
}
|
|
238
240
|
}
|
|
239
241
|
} catch (err) {
|
|
240
|
-
Logger.warn(`[
|
|
242
|
+
Logger.warn(`[plan-context] codebase snapshot skipped: ${err.message}`);
|
|
241
243
|
}
|
|
242
244
|
|
|
243
|
-
//
|
|
244
|
-
//
|
|
245
|
-
// (`
|
|
246
|
-
//
|
|
245
|
+
// Story #4542 — planning authors no risk artifact at all. Review depth and
|
|
246
|
+
// the acceptance-critic mode are derived from the diff at close time
|
|
247
|
+
// (`review-depth.js#deriveChangeLevel`), so there is nothing to classify
|
|
248
|
+
// here and nothing for the author step to judge about itself.
|
|
247
249
|
|
|
248
250
|
return {
|
|
249
251
|
epic: {
|
|
250
252
|
id: epic.id,
|
|
251
253
|
title: epic.title,
|
|
252
|
-
body:
|
|
253
|
-
bodySummary: epicBody.mode === 'summary' ? epicBody.items[0] : null,
|
|
254
|
+
body: epic.body ?? null,
|
|
254
255
|
// Story #4324: the context-ticket classes are retired. A re-planned
|
|
255
256
|
// Epic's previous Tech Spec / Acceptance Table content rides along
|
|
256
257
|
// inside `body` (managed sections), which is how the author keeps
|
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
/**
|
|
2
|
-
*
|
|
2
|
+
* planning/spec-authoring-grounding.js — F10 Spec code-grounding.
|
|
3
3
|
*
|
|
4
4
|
* Story #4139 (Epic #4131). `/plan` Phase 7 authors the Tech Spec
|
|
5
5
|
* from the Epic body, the scraped project docs, and the
|
|
@@ -0,0 +1,344 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* lib/orchestration/resolve-stories.js — resolve a set of Story ids into the
|
|
3
|
+
* `{ stories, dag, done }` envelope the delivery scheduler consumes.
|
|
4
|
+
*
|
|
5
|
+
* This is the ONE resolution step for `/deliver`. It generalizes the
|
|
6
|
+
* envelope shaping proven by the retired `resolve-plan-run.js` and fixes the
|
|
7
|
+
* two defects that shipped with it:
|
|
8
|
+
*
|
|
9
|
+
* - it fetched with `state: 'open'`, so an already-landed sibling vanished
|
|
10
|
+
* from the envelope and `done[]` could never be populated;
|
|
11
|
+
* - `done[]` was computed only over label-fetched issues, so a dependency
|
|
12
|
+
* outside the fetched set could never be satisfied.
|
|
13
|
+
*
|
|
14
|
+
* Both made cross-run, over-time delivery structurally impossible. Here every
|
|
15
|
+
* dependency — in-set or foreign — is resolved against live issue state, so a
|
|
16
|
+
* Story whose blocker landed weeks ago in another plan run is simply ready.
|
|
17
|
+
*
|
|
18
|
+
* Two contracts differ deliberately from the label-scoped ancestor:
|
|
19
|
+
*
|
|
20
|
+
* 1. **Id-scoped fetch means a named non-Story is an ERROR, not a filter.**
|
|
21
|
+
* `toStoryRecord` used to return `null` for a non-Story, which is right
|
|
22
|
+
* when a label query returns incidental noise and wrong when an operator
|
|
23
|
+
* names an id explicitly: silently dropping it yields a partial envelope
|
|
24
|
+
* that under-delivers without saying so.
|
|
25
|
+
* 2. **The native-edge read fails loud.** See {@link readNativeBlockedBy}.
|
|
26
|
+
*
|
|
27
|
+
* @module lib/orchestration/resolve-stories
|
|
28
|
+
*/
|
|
29
|
+
|
|
30
|
+
import { extractEpicIdFromBody, parseBlockedBy } from '../dependency-parser.js';
|
|
31
|
+
import { TYPE_LABELS } from '../label-constants.js';
|
|
32
|
+
import { buildStoryAdjacency } from '../story-adjacency.js';
|
|
33
|
+
import {
|
|
34
|
+
extractChangePaths,
|
|
35
|
+
parse as parseStoryBody,
|
|
36
|
+
} from '../story-body/story-body.js';
|
|
37
|
+
|
|
38
|
+
/** Labels/state that mean a blocker no longer gates its dependents. */
|
|
39
|
+
const DONE_LABEL = 'agent::done';
|
|
40
|
+
|
|
41
|
+
/**
|
|
42
|
+
* Module-private: `toStoryRecord` and `isSatisfiedBlocker` are its only
|
|
43
|
+
* callers. The ancestor exported it with no external consumer, which is how
|
|
44
|
+
* a symbol ends up baselined as a dead export.
|
|
45
|
+
*
|
|
46
|
+
* @param {object} issue
|
|
47
|
+
* @returns {string[]}
|
|
48
|
+
*/
|
|
49
|
+
function normalizeIssueLabels(issue) {
|
|
50
|
+
const raw = issue?.labels;
|
|
51
|
+
if (!Array.isArray(raw)) return [];
|
|
52
|
+
return raw
|
|
53
|
+
.map((l) => (typeof l === 'string' ? l : l?.name))
|
|
54
|
+
.filter((n) => typeof n === 'string' && n.length > 0);
|
|
55
|
+
}
|
|
56
|
+
|
|
57
|
+
/**
|
|
58
|
+
* Map one fetched issue into a Story record, or throw naming the id and the
|
|
59
|
+
* remedy. Unlike the label-scoped ancestor this **never** returns `null`:
|
|
60
|
+
* under `--ids` the operator named this issue, so dropping it silently would
|
|
61
|
+
* emit a partial envelope.
|
|
62
|
+
*
|
|
63
|
+
* @param {object} issue
|
|
64
|
+
* @param {number} [requestedId] The id the operator asked for, for error text.
|
|
65
|
+
* @returns {{ id, title, body, url, labels, state }}
|
|
66
|
+
*/
|
|
67
|
+
export function toStoryRecord(issue, requestedId) {
|
|
68
|
+
const id = Number(issue?.number ?? issue?.id ?? requestedId);
|
|
69
|
+
if (!Number.isInteger(id) || id <= 0) {
|
|
70
|
+
throw new Error(
|
|
71
|
+
`[resolve-stories] #${requestedId ?? '?'} did not resolve to an issue number.`,
|
|
72
|
+
);
|
|
73
|
+
}
|
|
74
|
+
const labels = normalizeIssueLabels(issue);
|
|
75
|
+
if (!labels.includes(TYPE_LABELS.STORY)) {
|
|
76
|
+
throw new Error(
|
|
77
|
+
`[resolve-stories] Issue #${id} is not a Story (labels: ${labels.join(', ') || 'none'}). ` +
|
|
78
|
+
`/deliver accepts ${TYPE_LABELS.STORY} tickets only — close it or re-plan it as a v2 Story.`,
|
|
79
|
+
);
|
|
80
|
+
}
|
|
81
|
+
const body = String(issue?.body ?? '');
|
|
82
|
+
const epicId = extractEpicIdFromBody(body);
|
|
83
|
+
if (epicId !== null) {
|
|
84
|
+
throw new Error(
|
|
85
|
+
`[resolve-stories] Issue #${id} still carries an "Epic: #${epicId}" footer. ` +
|
|
86
|
+
`v2 is Story-only — re-plan it as a v2 Story or finish it on a pre-v2 checkout.`,
|
|
87
|
+
);
|
|
88
|
+
}
|
|
89
|
+
return {
|
|
90
|
+
id,
|
|
91
|
+
title: String(issue?.title ?? ''),
|
|
92
|
+
body,
|
|
93
|
+
url: issue?.html_url ?? issue?.url ?? null,
|
|
94
|
+
labels,
|
|
95
|
+
state: String(issue?.state ?? 'open').toLowerCase(),
|
|
96
|
+
};
|
|
97
|
+
}
|
|
98
|
+
|
|
99
|
+
/**
|
|
100
|
+
* A blocker stops gating once its issue is closed or carries `agent::done`.
|
|
101
|
+
*
|
|
102
|
+
* @param {{ state?: string, labels?: string[] }} issue
|
|
103
|
+
* @returns {boolean}
|
|
104
|
+
*/
|
|
105
|
+
export function isSatisfiedBlocker(issue) {
|
|
106
|
+
const state = String(issue?.state ?? '').toLowerCase();
|
|
107
|
+
if (state === 'closed') return true;
|
|
108
|
+
return normalizeIssueLabels(issue).includes(DONE_LABEL);
|
|
109
|
+
}
|
|
110
|
+
|
|
111
|
+
/**
|
|
112
|
+
* Footprint emitted when a Story's declared changes cannot be READ. It is a
|
|
113
|
+
* glob, so `storiesOverlap` (`lib/wave-runner/ready-set.js`) treats it as
|
|
114
|
+
* overlapping every other **declared** footprint and the Story takes its
|
|
115
|
+
* beat alone. (A Story declaring nothing still overlaps nothing — the guard
|
|
116
|
+
* short-circuits on an empty footprint either side, which is the deliberate
|
|
117
|
+
* permissive escape hatch that keeps undeclared work parallel.)
|
|
118
|
+
*
|
|
119
|
+
* Deliberately NOT `[]`. An empty footprint means "declares nothing", which
|
|
120
|
+
* the guard reads as "overlaps nothing" and never withholds — correct for a
|
|
121
|
+
* Story that genuinely declares no changes, and wrong for one whose changes
|
|
122
|
+
* we failed to parse. Those are different facts: the second is *unknown*
|
|
123
|
+
* width, and the same argument that makes a glob overlap everything
|
|
124
|
+
* ("unknown width is not no width") applies to a body we could not read.
|
|
125
|
+
*/
|
|
126
|
+
const UNKNOWN_FOOTPRINT = Object.freeze(['**']);
|
|
127
|
+
|
|
128
|
+
/**
|
|
129
|
+
* Extract a Story's declared file footprint as **plain path strings**.
|
|
130
|
+
*
|
|
131
|
+
* The shape matters: `stories-wave-tick.js`'s `parseDag` rejects any `files`
|
|
132
|
+
* entry that is not a string, while `extractChangePaths` returns
|
|
133
|
+
* `{ path, isGlob }` objects — so forwarding its output verbatim fails every
|
|
134
|
+
* multi-Story run with an input error. Map to `.path`.
|
|
135
|
+
*
|
|
136
|
+
* Never throws — these are live, human-editable issue bodies, and one
|
|
137
|
+
* malformed body must not take the whole resolution down. It fails **safe**
|
|
138
|
+
* rather than open: an unreadable footprint yields {@link UNKNOWN_FOOTPRINT},
|
|
139
|
+
* serializing that Story instead of silently letting it race.
|
|
140
|
+
*
|
|
141
|
+
* @param {string} body
|
|
142
|
+
* @param {number} [id] Story id, for the warning.
|
|
143
|
+
* @param {(msg: string) => void} [warn]
|
|
144
|
+
* @returns {string[]}
|
|
145
|
+
*/
|
|
146
|
+
export function storyFootprintPaths(body, id, warn) {
|
|
147
|
+
let parsed;
|
|
148
|
+
try {
|
|
149
|
+
parsed = parseStoryBody(String(body ?? '')).body;
|
|
150
|
+
} catch (err) {
|
|
151
|
+
warn?.(
|
|
152
|
+
`[resolve-stories] #${id}: body is unparseable, so its file footprint is unknown ` +
|
|
153
|
+
`(${err?.message ?? err}). Treating it as overlapping every other Story, so it is ` +
|
|
154
|
+
`never co-dispatched — fix the body to restore parallelism.`,
|
|
155
|
+
);
|
|
156
|
+
return [...UNKNOWN_FOOTPRINT];
|
|
157
|
+
}
|
|
158
|
+
try {
|
|
159
|
+
// An empty `changes` is a real declaration of "no files", not a read
|
|
160
|
+
// failure — it keeps the permissive empty footprint.
|
|
161
|
+
return extractChangePaths(parsed?.changes ?? [])
|
|
162
|
+
.map((entry) => entry?.path)
|
|
163
|
+
.filter((p) => typeof p === 'string' && p.trim().length > 0)
|
|
164
|
+
.map((p) => p.trim());
|
|
165
|
+
} catch (err) {
|
|
166
|
+
warn?.(
|
|
167
|
+
`[resolve-stories] #${id}: malformed changes entry, so its file footprint is unknown ` +
|
|
168
|
+
`(${err?.message ?? err}). Treating it as overlapping every other Story.`,
|
|
169
|
+
);
|
|
170
|
+
return [...UNKNOWN_FOOTPRINT];
|
|
171
|
+
}
|
|
172
|
+
}
|
|
173
|
+
|
|
174
|
+
/**
|
|
175
|
+
* Build the DAG nodes. `dependsOn` is the union the adjacency builder already
|
|
176
|
+
* computes (body-parsed `blocked by #N` + explicit fields), plus any native
|
|
177
|
+
* edges threaded in via `nativeEdges`. `files` is a plain `string[]`.
|
|
178
|
+
*
|
|
179
|
+
* @param {object[]} stories
|
|
180
|
+
* @param {Map<number, number[]>} [nativeEdges]
|
|
181
|
+
* @param {(msg: string) => void} [warn]
|
|
182
|
+
* @returns {{ id: number, dependsOn: number[], files: string[] }[]}
|
|
183
|
+
*/
|
|
184
|
+
export function storiesToDag(stories, nativeEdges = new Map(), warn) {
|
|
185
|
+
const withNative = stories.map((s) => ({
|
|
186
|
+
...s,
|
|
187
|
+
dependsOn: [
|
|
188
|
+
...new Set([
|
|
189
|
+
...parseBlockedBy(s.body ?? ''),
|
|
190
|
+
...(nativeEdges.get(s.id) ?? []),
|
|
191
|
+
]),
|
|
192
|
+
],
|
|
193
|
+
}));
|
|
194
|
+
// dropForeign:false — a dependency outside the requested set is a real
|
|
195
|
+
// gate, not noise. What changes here is that such a gate is now
|
|
196
|
+
// *satisfiable*: `done[]` carries foreign blockers resolved from live state.
|
|
197
|
+
const adjacency = buildStoryAdjacency(withNative, { dropForeign: false });
|
|
198
|
+
return stories.map((s) => ({
|
|
199
|
+
id: s.id,
|
|
200
|
+
dependsOn: adjacency.get(s.id) ?? [],
|
|
201
|
+
files: storyFootprintPaths(s.body, s.id, warn),
|
|
202
|
+
}));
|
|
203
|
+
}
|
|
204
|
+
|
|
205
|
+
/**
|
|
206
|
+
* Project the dependencies API response onto issue **numbers**.
|
|
207
|
+
*
|
|
208
|
+
* The API returns both `id` (database id) and `number` (issue number); the
|
|
209
|
+
* write path (`providers/github/blocked-by-add.js`) reads `id` because its
|
|
210
|
+
* POST body needs `issue_id`. Reusing that projection here would build
|
|
211
|
+
* `dependsOn: [4902374986]` for a blocker whose issue number is 4530 — an id
|
|
212
|
+
* matching no Story, foreign to the set, never satisfiable, and (because
|
|
213
|
+
* foreign edges are real gates) a silent permanent wedge.
|
|
214
|
+
*
|
|
215
|
+
* Cross-repo blockers are rejected rather than matched: another repo's #4530
|
|
216
|
+
* is not this repo's #4530, and treating it as one could satisfy a gate that
|
|
217
|
+
* is still open.
|
|
218
|
+
*
|
|
219
|
+
* @param {unknown} data Parsed API response.
|
|
220
|
+
* @param {{ owner: string, repo: string, issueNumber: number }} ctx
|
|
221
|
+
* @returns {number[]}
|
|
222
|
+
*/
|
|
223
|
+
export function nativeBlockedByNumbers(data, { owner, repo, issueNumber }) {
|
|
224
|
+
if (!Array.isArray(data)) return [];
|
|
225
|
+
const out = [];
|
|
226
|
+
for (const item of data) {
|
|
227
|
+
const repoUrl = item?.repository_url ?? item?.repository?.url ?? null;
|
|
228
|
+
if (typeof repoUrl === 'string' && repoUrl.length > 0) {
|
|
229
|
+
const expected = `/repos/${owner}/${repo}`;
|
|
230
|
+
if (!repoUrl.endsWith(expected)) {
|
|
231
|
+
throw new Error(
|
|
232
|
+
`[resolve-stories] #${issueNumber} is blocked by an issue in another repository ` +
|
|
233
|
+
`(${repoUrl}). Cross-repo dependency edges are not supported — its number cannot ` +
|
|
234
|
+
`be matched against this repo's Stories without risking a false match.`,
|
|
235
|
+
);
|
|
236
|
+
}
|
|
237
|
+
}
|
|
238
|
+
const number = Number(item?.number);
|
|
239
|
+
if (Number.isInteger(number) && number > 0) out.push(number);
|
|
240
|
+
}
|
|
241
|
+
return [...new Set(out)];
|
|
242
|
+
}
|
|
243
|
+
|
|
244
|
+
/**
|
|
245
|
+
* Read an issue's native `blocked_by` edges as issue numbers.
|
|
246
|
+
*
|
|
247
|
+
* **Fails loud**, deliberately inverting the write path's non-fatal contract.
|
|
248
|
+
* A dropped write-side edge is cosmetic (the ordering still lives in the
|
|
249
|
+
* `blocked by #N` body footer); a dropped READ-side edge silently removes a
|
|
250
|
+
* dispatch gate, so a 403 (dependencies API disabled, or a token without the
|
|
251
|
+
* scope) would erase every native edge at once and co-dispatch the whole run
|
|
252
|
+
* against unlanded blockers. A 404 means "no dependencies on this issue" and
|
|
253
|
+
* is a legitimate empty result.
|
|
254
|
+
*
|
|
255
|
+
* @param {{ gh: object, owner: string, repo: string, issueNumber: number, parseJson: Function }} opts
|
|
256
|
+
* @returns {Promise<number[]>}
|
|
257
|
+
*/
|
|
258
|
+
export async function readNativeBlockedBy({
|
|
259
|
+
gh,
|
|
260
|
+
owner,
|
|
261
|
+
repo,
|
|
262
|
+
issueNumber,
|
|
263
|
+
parseJson,
|
|
264
|
+
}) {
|
|
265
|
+
let result;
|
|
266
|
+
try {
|
|
267
|
+
result = await gh.api({
|
|
268
|
+
method: 'GET',
|
|
269
|
+
endpoint: `/repos/${owner}/${repo}/issues/${issueNumber}/dependencies/blocked_by`,
|
|
270
|
+
});
|
|
271
|
+
} catch (err) {
|
|
272
|
+
const detail = String(err?.message ?? err);
|
|
273
|
+
if (/404|not found/i.test(detail)) return [];
|
|
274
|
+
throw new Error(
|
|
275
|
+
`[resolve-stories] Could not read native blocked_by edges for #${issueNumber}: ${detail}. ` +
|
|
276
|
+
`Refusing to continue: a dropped dependency edge would silently remove a dispatch gate ` +
|
|
277
|
+
`and co-dispatch this Story against an unlanded blocker.`,
|
|
278
|
+
);
|
|
279
|
+
}
|
|
280
|
+
return nativeBlockedByNumbers(parseJson(result), {
|
|
281
|
+
owner,
|
|
282
|
+
repo,
|
|
283
|
+
issueNumber,
|
|
284
|
+
});
|
|
285
|
+
}
|
|
286
|
+
|
|
287
|
+
/**
|
|
288
|
+
* Assemble the envelope from resolved records.
|
|
289
|
+
*
|
|
290
|
+
* @param {object[]} stories
|
|
291
|
+
* @param {Map<number, number[]>} nativeEdges
|
|
292
|
+
* @param {number[]} foreignDone Ids outside the set already satisfied.
|
|
293
|
+
* @param {(msg: string) => void} [warn]
|
|
294
|
+
* @returns {{ kind: string, stories: object[], dag: object[], done: number[] }}
|
|
295
|
+
*/
|
|
296
|
+
export function buildStoriesEnvelope({
|
|
297
|
+
stories,
|
|
298
|
+
nativeEdges = new Map(),
|
|
299
|
+
foreignDone = [],
|
|
300
|
+
warn,
|
|
301
|
+
}) {
|
|
302
|
+
const sorted = [...stories].sort((a, b) => a.id - b.id);
|
|
303
|
+
const inSetDone = sorted.filter(isSatisfiedBlocker).map((s) => s.id);
|
|
304
|
+
return {
|
|
305
|
+
kind: 'stories',
|
|
306
|
+
stories: sorted.map(({ id, title, url, labels, state }) => ({
|
|
307
|
+
id,
|
|
308
|
+
title,
|
|
309
|
+
url,
|
|
310
|
+
labels,
|
|
311
|
+
state,
|
|
312
|
+
})),
|
|
313
|
+
dag: storiesToDag(sorted, nativeEdges, warn),
|
|
314
|
+
done: [...new Set([...inSetDone, ...foreignDone])].sort((a, b) => a - b),
|
|
315
|
+
};
|
|
316
|
+
}
|
|
317
|
+
|
|
318
|
+
/**
|
|
319
|
+
* Parse and validate the `--ids` list.
|
|
320
|
+
*
|
|
321
|
+
* @param {string|undefined} raw
|
|
322
|
+
* @returns {number[]}
|
|
323
|
+
*/
|
|
324
|
+
export function parseIds(raw) {
|
|
325
|
+
const ids = String(raw ?? '')
|
|
326
|
+
.split(',')
|
|
327
|
+
.map((s) => s.trim())
|
|
328
|
+
.filter(Boolean)
|
|
329
|
+
.map((s) => {
|
|
330
|
+
const n = Number.parseInt(s, 10);
|
|
331
|
+
if (!Number.isInteger(n) || n <= 0 || String(n) !== s) {
|
|
332
|
+
throw new Error(
|
|
333
|
+
`[resolve-stories] --ids must be a comma-separated list of positive issue numbers (got "${s}").`,
|
|
334
|
+
);
|
|
335
|
+
}
|
|
336
|
+
return n;
|
|
337
|
+
});
|
|
338
|
+
if (ids.length === 0) {
|
|
339
|
+
throw new Error(
|
|
340
|
+
'[resolve-stories] --ids is required: node resolve-stories.js --ids 101,102',
|
|
341
|
+
);
|
|
342
|
+
}
|
|
343
|
+
return [...new Set(ids)];
|
|
344
|
+
}
|
|
@@ -43,7 +43,7 @@
|
|
|
43
43
|
*
|
|
44
44
|
* @typedef {Object} RoutedProposalsInput
|
|
45
45
|
* @property {number} anchorId Story or run/Epic id in titles.
|
|
46
|
-
* @property {'
|
|
46
|
+
* @property {'story'|'run'} [anchorKind] Wording in titles/bodies (default `story`).
|
|
47
47
|
* @property {string} frameworkRepo `"<owner>/<repo>"`.
|
|
48
48
|
* @property {string} consumerRepo `"<owner>/<repo>"`.
|
|
49
49
|
* @property {FrictionSignal[]} [signals]
|
|
@@ -144,7 +144,7 @@ function dominantSource(entry) {
|
|
|
144
144
|
}
|
|
145
145
|
|
|
146
146
|
/**
|
|
147
|
-
* @param {'
|
|
147
|
+
* @param {'story'|'run'} kind
|
|
148
148
|
* @param {number} id
|
|
149
149
|
* @returns {string}
|
|
150
150
|
*/
|
|
@@ -154,7 +154,7 @@ function formatAnchor(kind, id) {
|
|
|
154
154
|
return `Epic #${id}`;
|
|
155
155
|
}
|
|
156
156
|
|
|
157
|
-
/** Story scope promotes single-occurrence friction;
|
|
157
|
+
/** Story scope promotes single-occurrence friction; run keeps ≥2. */
|
|
158
158
|
function isActionableFriction(total, force, anchorKind) {
|
|
159
159
|
const threshold = anchorKind === 'story' ? 1 : 2;
|
|
160
160
|
return total >= threshold || Boolean(force);
|
|
@@ -167,7 +167,7 @@ function isActionableFriction(total, force, anchorKind) {
|
|
|
167
167
|
*
|
|
168
168
|
* @param {{
|
|
169
169
|
* anchorId: number,
|
|
170
|
-
* anchorKind: '
|
|
170
|
+
* anchorKind: 'story'|'run',
|
|
171
171
|
* category: string,
|
|
172
172
|
* occurrences: number,
|
|
173
173
|
* source: "framework"|"consumer",
|
|
@@ -225,7 +225,7 @@ function renderIssueCommand({ repo, title, metaLabel, category, body }) {
|
|
|
225
225
|
*
|
|
226
226
|
* @param {{
|
|
227
227
|
* anchorId: number,
|
|
228
|
-
* anchorKind: '
|
|
228
|
+
* anchorKind: 'story'|'run',
|
|
229
229
|
* category: string,
|
|
230
230
|
* occurrences: number,
|
|
231
231
|
* source: "framework"|"consumer",
|
|
@@ -272,7 +272,7 @@ function buildRoutedItem({
|
|
|
272
272
|
* @param {unknown} input
|
|
273
273
|
* @returns {{
|
|
274
274
|
* anchorId: number,
|
|
275
|
-
* anchorKind: '
|
|
275
|
+
* anchorKind: 'story'|'run',
|
|
276
276
|
* frameworkRepo: string,
|
|
277
277
|
* consumerRepo: string,
|
|
278
278
|
* signals: FrictionSignal[],
|
|
@@ -280,7 +280,7 @@ function buildRoutedItem({
|
|
|
280
280
|
* } | null}
|
|
281
281
|
*/
|
|
282
282
|
function normalizeAnchorKind(kind) {
|
|
283
|
-
return kind === 'story' || kind === 'run'
|
|
283
|
+
return kind === 'story' || kind === 'run' ? kind : 'story';
|
|
284
284
|
}
|
|
285
285
|
|
|
286
286
|
function normaliseInput(input) {
|