@harness-engineering/cli 3.0.0 → 3.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.
Files changed (58) hide show
  1. package/dist/agents/skills/claude-code/harness-execution/SKILL.md +35 -0
  2. package/dist/agents/skills/claude-code/harness-roadmap-pilot/SKILL.md +33 -44
  3. package/dist/agents/skills/codex/harness-execution/SKILL.md +35 -0
  4. package/dist/agents/skills/codex/harness-roadmap-pilot/SKILL.md +33 -44
  5. package/dist/agents/skills/cursor/harness-execution/SKILL.md +35 -0
  6. package/dist/agents/skills/cursor/harness-roadmap-pilot/SKILL.md +33 -44
  7. package/dist/agents/skills/gemini-cli/harness-execution/SKILL.md +35 -0
  8. package/dist/agents/skills/gemini-cli/harness-roadmap-pilot/SKILL.md +33 -44
  9. package/dist/{agents-md-XOJE5ETA.js → agents-md-YYCLGMBF.js} +2 -2
  10. package/dist/{architecture-JEZPJ725.js → architecture-5KPP4ID4.js} +3 -3
  11. package/dist/{assess-project-RHXOQNMM.js → assess-project-B6ENHUAU.js} +1 -1
  12. package/dist/bin/harness-mcp.js +13 -13
  13. package/dist/bin/harness.js +22 -22
  14. package/dist/{check-phase-gate-GNDOMS35.js → check-phase-gate-REG3HWYC.js} +3 -3
  15. package/dist/{chunk-N24HCQA4.js → chunk-A65QO5KN.js} +132 -46
  16. package/dist/{chunk-UX3PQKVZ.js → chunk-BXKNWSG4.js} +1 -1
  17. package/dist/{chunk-XX4OLNWQ.js → chunk-C2ZGWMTN.js} +2 -2
  18. package/dist/{chunk-HEIMJJI4.js → chunk-DNX35Q5K.js} +2 -2
  19. package/dist/{chunk-BYVT5LVO.js → chunk-ERZCC5VH.js} +3 -3
  20. package/dist/{chunk-XFU6SB2Q.js → chunk-FL5SLUKZ.js} +62 -27
  21. package/dist/{chunk-NWLGL3IR.js → chunk-FUH4J43G.js} +1 -1
  22. package/dist/{chunk-MGUPSE6D.js → chunk-HIPBU2Q4.js} +5 -5
  23. package/dist/{chunk-3ISHDWO7.js → chunk-IF3JK3G5.js} +1 -1
  24. package/dist/{chunk-KAJOS3BA.js → chunk-LMZ2JTNP.js} +2 -2
  25. package/dist/{chunk-527G27VI.js → chunk-OQFEFVIN.js} +2 -2
  26. package/dist/{chunk-Z4AWEIBY.js → chunk-PZX2Y3RV.js} +3 -3
  27. package/dist/{chunk-ONCPJGHY.js → chunk-RGQXHCCH.js} +6 -6
  28. package/dist/{chunk-QBC7DI4Q.js → chunk-TCVQVHEF.js} +1 -1
  29. package/dist/{chunk-PC2R5RUJ.js → chunk-TE5YHHHZ.js} +1 -1
  30. package/dist/{chunk-HWTUUQOW.js → chunk-UBTMPO5H.js} +3 -3
  31. package/dist/{chunk-GSP2XIVS.js → chunk-XEOSSKIB.js} +1 -1
  32. package/dist/{chunk-DE5U6KOL.js → chunk-XZ2Q4UTW.js} +1 -1
  33. package/dist/{chunk-YQSYCBAH.js → chunk-YF3AY6KS.js} +6 -6
  34. package/dist/{chunk-GGKRA7A7.js → chunk-YQAFKOOL.js} +1 -1
  35. package/dist/{chunk-WWEVS7RO.js → chunk-ZOUJP3EY.js} +106 -80
  36. package/dist/{chunk-6WQCLCBO.js → chunk-ZRKFDE6M.js} +1 -1
  37. package/dist/{ci-workflow-ORZ4F45F.js → ci-workflow-JEYYCBOC.js} +2 -2
  38. package/dist/{dist-LHINSVK4.js → dist-3HW4LCHE.js} +15 -1
  39. package/dist/{docs-AU3DXFFO.js → docs-YRMW7UED.js} +3 -3
  40. package/dist/{engine-6GPWOYQH.js → engine-E7H5U3AC.js} +2 -2
  41. package/dist/{entropy-VLKDRMRT.js → entropy-3JBKCGJJ.js} +2 -2
  42. package/dist/{feedback-YXYY44ZC.js → feedback-U6VJ6AUJ.js} +1 -1
  43. package/dist/{generate-agent-definitions-XIHQPKG3.js → generate-agent-definitions-S2FRZF4Y.js} +3 -3
  44. package/dist/hooks/format-check.js +202 -0
  45. package/dist/hooks/quality-warner.js +32 -0
  46. package/dist/hooks/strict-quality-gate.js +48 -0
  47. package/dist/index.d.ts +8 -8
  48. package/dist/index.js +22 -22
  49. package/dist/{loader-YAN56MT3.js → loader-RGQJYERV.js} +2 -2
  50. package/dist/{mcp-RK3SFVYB.js → mcp-NDTFZXWF.js} +13 -13
  51. package/dist/{performance-64DPTLLQ.js → performance-FPWKC4CN.js} +3 -3
  52. package/dist/{review-pipeline-NENG6LW7.js → review-pipeline-WUHG7PHO.js} +2 -2
  53. package/dist/{runtime-FFIIRT2T.js → runtime-Q3ORD7LW.js} +2 -2
  54. package/dist/{security-3PIBN35B.js → security-JDHTRBGC.js} +1 -1
  55. package/dist/{validate-7IAF2ES3.js → validate-G3LKBOYA.js} +3 -3
  56. package/dist/{validate-cross-check-QPSAOZKD.js → validate-cross-check-4ITA72DF.js} +2 -2
  57. package/package.json +8 -8
  58. package/dist/hooks/quality-gate.js +0 -131
@@ -1,6 +1,6 @@
1
1
  # Harness Roadmap Pilot
2
2
 
3
- > AI-assisted selection of the next highest-impact unblocked roadmap item. Scores candidates, recommends one, assigns it, and transitions to the appropriate next skill.
3
+ > AI-assisted selection of the next highest-impact unblocked roadmap item. Scores candidates, recommends one, and transitions to the appropriate next skill. Selection does NOT assign — the `assignee` field names who is _executing_, and is written at execution start (harness-execution), not at selection.
4
4
 
5
5
  ## When to Use
6
6
 
@@ -14,9 +14,9 @@
14
14
 
15
15
  ### Iron Law
16
16
 
17
- **Never assign or transition without the human confirming the recommendation first.**
17
+ **Never transition without the human confirming the recommendation first. Selection never writes `assignee`.**
18
18
 
19
- Present the ranked candidates, the AI reasoning, and the recommended pick. Wait for explicit confirmation before making any changes.
19
+ Present the ranked candidates, the AI reasoning, and the recommended pick. Wait for explicit confirmation before transitioning. The `assignee` field means _who is executing_ and is claimed at execution start by harness-execution — picking an item must NOT mark it assigned (an early assignee makes the item look human-claimed and the orchestrator silently skips it).
20
20
 
21
21
  ---
22
22
 
@@ -133,28 +133,19 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
133
133
  - If **pick another**: ask which candidate number, then proceed with that pick.
134
134
  - If **no**: stop. No changes made.
135
135
 
136
- ### Phase 4: ASSIGN -- Execute Assignment and Transition
136
+ ### Phase 4: TRANSITION -- Hand Off to the Next Skill
137
137
 
138
- 1. Call `manage_roadmap` with action `update` to assign the feature:
138
+ **Do NOT write the `assignee` field here.** Selection picks the item; it does not
139
+ claim it. harness-execution claims the item (`status=in-progress` + `assignee`) at
140
+ execution start. Writing an assignee at selection makes the orchestrator treat the
141
+ item as already-claimed and silently skip it (the bug this skill must not reintroduce).
142
+ The item stays `planned`/`backlog` with `Assignee: —` and remains orchestrator-eligible.
139
143
 
140
- ```json
141
- manage_roadmap({
142
- path: "<project-root>",
143
- action: "update",
144
- feature: "<feature-name>",
145
- assignee: "<currentUser>"
146
- })
147
- ```
148
-
149
- - This updates the feature's `Assignee` field with assignment history tracking
150
- - Automatically triggers external sync (GitHub Issues) if tracker config exists in `harness.config.json`
151
- - External sync is fire-and-forget — errors are logged but do not block the assignment
152
-
153
- 2. Determine the transition target:
144
+ 1. Determine the transition target:
154
145
  - If the feature has a `spec` field (non-null): transition to `harness:autopilot`
155
146
  - If the feature has no `spec`: transition to `harness:brainstorming`
156
147
 
157
- 3. Present the transition to the human via `emit_interaction`:
148
+ 2. Present the transition to the human via `emit_interaction`:
158
149
 
159
150
  ```json
160
151
  emit_interaction({
@@ -163,16 +154,16 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
163
154
  transition: {
164
155
  completedPhase: "roadmap-pilot",
165
156
  suggestedNext: "<brainstorming|autopilot>",
166
- reason: "Feature '<name>' assigned and ready for <brainstorming|execution>",
157
+ reason: "Feature '<name>' selected and ready for <brainstorming|execution>",
167
158
  artifacts: ["docs/roadmap.md"],
168
159
  requiresConfirmation: true,
169
- summary: "Assigned '<name>' to <user>. <Spec exists -- ready for autopilot|No spec -- needs brainstorming first>.",
160
+ summary: "Selected '<name>'. <Spec exists -- ready for autopilot|No spec -- needs brainstorming first>. (Not assigned — harness-execution claims it at execution start.)",
170
161
  qualityGate: {
171
162
  checks: [
172
163
  { "name": "roadmap-parsed", "passed": true },
173
164
  { "name": "candidate-scored", "passed": true },
174
165
  { "name": "human-confirmed", "passed": true },
175
- { "name": "assignment-written", "passed": true }
166
+ { "name": "no-assignment-at-selection", "passed": true }
176
167
  ],
177
168
  allPassed: true
178
169
  }
@@ -180,7 +171,7 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
180
171
  })
181
172
  ```
182
173
 
183
- 4. Run `harness validate`.
174
+ 3. Run `harness validate`.
184
175
 
185
176
  ---
186
177
 
@@ -190,19 +181,18 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
190
181
  - **`loadProjectRoadmapMode` / `loadTrackerClientConfigFromProject` / `createTrackerClient`** -- Resolve `roadmap.mode` and obtain a `RoadmapTrackerClient` for file-less mode. Import from `@harness-engineering/core`.
191
182
  - **`scoreRoadmapCandidatesForMode`** -- Mode-aware scoring entry point. Import from `@harness-engineering/core`. In file-backed mode delegates to `scoreRoadmapCandidates`; in file-less mode routes through `scoreRoadmapCandidatesFileLess` (priority + createdAt sort, FR-S3).
192
183
  - **`scoreRoadmapCandidates`** -- Underlying file-backed scoring algorithm. Prefer `scoreRoadmapCandidatesForMode` from the skill; direct callers in file-backed-only code paths can still use this.
193
- - **`manage_roadmap update`** -- Used for assignment. Supports `assignee` field which delegates to `assignFeature` internally, handles history tracking, and automatically triggers external sync (GitHub Issues). In file-less mode, `manage_roadmap` dispatches through the tracker; the skill flow is unchanged.
194
- - **`emit_interaction`** -- Used for the skill transition at the end. Transitions to `harness:brainstorming` (no spec) or `harness:autopilot` (spec exists).
184
+ - **`emit_interaction`** -- Used for the skill transition at the end. Transitions to `harness:brainstorming` (no spec) or `harness:autopilot` (spec exists). This skill does NOT call `manage_roadmap update` to assign the `assignee` field is owned by harness-execution (claim at execution start), enforced by RMH005 (`assignee ≠ null ⟺ in-progress`).
195
185
  - **`STRATEGY.md` alignment (Phase 2 step 1a)** -- When present at repo root and valid, loaded via `validateStrategy` + `parseStrategyDoc` + `asStrategyDoc` from `@harness-engineering/core`. Applied as a bounded tiebreaker bonus (max `+0.75`) only when candidates score within `0.05` on the base formula. Boundary: roadmap-pilot READS; `harness-strategy` WRITES. Never modify `STRATEGY.md` from this skill.
196
- - **`harness validate`** -- Run after assignment is written.
186
+ - **`harness validate`** -- Run after the transition is presented.
197
187
 
198
188
  ## Success Criteria
199
189
 
200
190
  1. Roadmap is parsed and unblocked planned/backlog items are scored
201
191
  2. Scoring uses two-tier sort: explicit priority first, then weighted score
202
192
  3. AI reads top candidates' specs and provides recommendation with reasoning
203
- 4. Human confirms before any changes are made
204
- 5. Assignment updates feature field, appends history records, and syncs externally
205
- 6. Reassignment produces two history records (unassigned + assigned)
193
+ 4. Human confirms before the transition is made
194
+ 5. Selection does NOT write the `assignee` field (no assignment, no history record, no external assignee sync) — the pick stays `planned`/`backlog` and orchestrator-eligible
195
+ 6. The `assignee` field is owned by harness-execution, which claims at execution start; RMH005 enforces `assignee ≠ null ⟺ in-progress`
206
196
  7. Transition routes to brainstorming (no spec) or autopilot (spec exists)
207
197
  8. When a pulse report exists, the recommendation rationale cites pulse signal for any top-3 candidate whose area is referenced in the pulse Headlines or Followups.
208
198
  9. When `STRATEGY.md` is present and valid AND two candidates score within `0.05` on the base formula, the recommendation rationale cites strategy-alignment as the tiebreaker. The bonus never overrides a meaningful base-score difference.
@@ -211,12 +201,13 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
211
201
 
212
202
  ## Rationalizations to Reject
213
203
 
214
- | Rationalization | Reality |
215
- | ----------------------------------------------------------------------------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
216
- | "The top-scored candidate is obviously correct, so I can assign it without asking the human" | The Iron Law: never assign or transition without the human confirming the recommendation first. |
217
- | "Affinity data is not available so the scoring is degraded -- I should just pick the first planned item" | Proceed without affinity scoring by zeroing out the affinity weight. Position and dependents signals still produce meaningful rankings. |
218
- | "The feature has no spec, but I can skip brainstorming and jump straight to planning since the summary is clear enough" | No spec routes to brainstorming, spec exists routes to autopilot. A one-line roadmap summary is not a spec. |
219
- | "STRATEGY.md exists, so I should let it override the top-scored candidate when alignment is clear" | The alignment bonus is bounded (max `+0.75`) and only fires when base scores are within `0.05`. It is a tiebreaker, not a hard filter — a clearly higher-scored item still wins. |
204
+ | Rationalization | Reality |
205
+ | ----------------------------------------------------------------------------------------------------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
206
+ | "The top-scored candidate is obviously correct, so I can transition without asking the human" | The Iron Law: never transition without the human confirming the recommendation first. |
207
+ | "I'll write the assignee now so the pick is recorded / the user is credited" | Selection never writes `assignee`. An assignee means _in-progress_ (set by harness-execution at execution start). Writing it at selection makes the orchestrator silently skip the item the exact bug this skill must not reintroduce. |
208
+ | "Affinity data is not available so the scoring is degraded -- I should just pick the first planned item" | Proceed without affinity scoring by zeroing out the affinity weight. Position and dependents signals still produce meaningful rankings. |
209
+ | "The feature has no spec, but I can skip brainstorming and jump straight to planning since the summary is clear enough" | No spec routes to brainstorming, spec exists routes to autopilot. A one-line roadmap summary is not a spec. |
210
+ | "STRATEGY.md exists, so I should let it override the top-scored candidate when alignment is clear" | The alignment bonus is bounded (max `+0.75`) and only fires when base scores are within `0.05`. It is a tiebreaker, not a hard filter — a clearly higher-scored item still wins. |
220
211
 
221
212
  ## Examples
222
213
 
@@ -252,25 +243,23 @@ Proceed? (y/n/pick another)
252
243
 
253
244
  Human confirms **y**.
254
245
 
255
- **Phase 4: ASSIGN**
246
+ **Phase 4: TRANSITION**
256
247
 
257
248
  ```
258
- manage_roadmap update: Graph Connector assignee -> @cwarner
259
- History: +1 record (assigned, 2026-04-02)
260
- Roadmap updated: docs/roadmap.md
261
- External sync: github:harness-eng/harness#43 assigned (automatic)
249
+ Graph Connector stays planned, Assignee: (not assigned at selection).
250
+ harness-execution will claim it (status=in-progress + assignee) at execution start.
262
251
 
263
252
  Transitioning to harness:autopilot (spec exists)...
264
253
  ```
265
254
 
266
255
  ## Gates
267
256
 
268
- - **No assignment without human confirmation.** The CONFIRM phase must complete with explicit approval. Never auto-assign.
269
- - **No transition without assignment.** The skill must write the assignment before transitioning to the next skill.
257
+ - **No transition without human confirmation.** The CONFIRM phase must complete with explicit approval before transitioning.
258
+ - **No assignment at selection.** This skill never writes the `assignee` field that field means _who is executing_ and is claimed at execution start by harness-execution. Writing it here would make the orchestrator silently skip the item.
270
259
  - **No scoring without a parsed roadmap.** If `docs/roadmap.md` does not exist or fails to parse, stop with an error.
271
260
 
272
261
  ## Escalation
273
262
 
274
263
  - **When no unblocked candidates exist:** Inform the human. Suggest reviewing blocked items to see if blockers can be resolved, or adding new features via `harness-roadmap --add`.
275
264
  - **When affinity data is unavailable:** Proceed without affinity scoring (weight falls to 0 for all candidates). Note this in the output.
276
- - **When external sync fails:** Log the error, complete the local assignment, and note that external sync can be retried with `harness-roadmap --sync`.
265
+ - **When the human asks you to assign the item to them at selection:** Explain that the `assignee` field means _who is executing_ and is claimed at execution start by harness-execution; assigning at selection makes the orchestrator silently skip the item (enforced by RMH005). Proceed with the transition instead.
@@ -98,6 +98,39 @@ Do not improvise past unknowns. An assumption that turns out wrong is cheaper th
98
98
 
99
99
  Report progress with: `**[Phase N/M]** Task N — <description>`
100
100
 
101
+ #### Step 0: Claim the roadmap item (execution-start claim)
102
+
103
+ Execution start is the one true "who is working on this now" moment — so this is
104
+ where the `assignee` is written, **not** at selection (roadmap-pilot no longer
105
+ assigns). The invariant is `assignee ≠ null ⟺ status == in-progress`, enforced by
106
+ RMH005 (`harness validate`).
107
+
108
+ 1. **Resolve `currentUser`** the same way roadmap-pilot does: the `--user` argument
109
+ if provided, else `git config user.name` (fall back to `git config user.email`).
110
+ 2. **If `docs/roadmap.md` exists and a row matches this plan** (match by the row's
111
+ `Spec`/`Plan` path or the feature name from the handoff/spec):
112
+ - **If `currentUser` cannot be resolved, STOP** with an actionable message —
113
+ "Set your identity via `git config user.name`/`user.email` (or pass `--user`)
114
+ before starting execution" — rather than claiming with a null owner (S4-002).
115
+ Leave the row `planned` and unassigned.
116
+ - Otherwise claim it:
117
+
118
+ ```json
119
+ manage_roadmap({
120
+ path: "<project-root>",
121
+ action: "update",
122
+ feature: "<feature-name>",
123
+ status: "in-progress",
124
+ assignee: "<currentUser>"
125
+ })
126
+ ```
127
+
128
+ This routes through the assignee-lifecycle `claim()` chokepoint
129
+ (first-claim-wins: if the row is already `in-progress` under a different
130
+ owner, the claim is a no-op and you should treat the item as taken).
131
+
132
+ - If no roadmap row matches this plan (e.g. ad-hoc execution), skip the claim.
133
+
101
134
  For each task, starting from current position:
102
135
 
103
136
  1. **Read task instructions completely** before writing any code.
@@ -344,11 +377,13 @@ Claims about task completion, test results, or code behavior MUST cite evidence:
344
377
  - **`harness state show`** — View current position and progress.
345
378
  - **`harness state learn "<message>"`** — Append a learning from CLI.
346
379
  - **State/Learnings files** — Session-scoped when session known, otherwise `.harness/`. State updated after every task; learnings append-only.
380
+ - **Roadmap claim** — Phase 2 Step 0: `manage_roadmap update` with `status: in-progress` + `assignee: <currentUser>` claims the item at execution start (the assignee = "who is executing" invariant). Stop if `currentUser` is unresolvable. Marking the row `done` later auto-clears the assignee via the lifecycle `setStatus` chokepoint — never leave a non-in-progress row assigned (RMH005).
347
381
  - **Roadmap sync** — After plan completion, `manage_roadmap sync` with `apply: true`. Mandatory when roadmap exists. No `force_sync: true`.
348
382
  - **`emit_interaction`** — Auto-transition to harness-verification at plan completion.
349
383
 
350
384
  ## Success Criteria
351
385
 
386
+ - The roadmap item is claimed at execution start (`status=in-progress` + `assignee=<currentUser>`) when a matching row exists; execution stops with an actionable message if `currentUser` is unresolvable
352
387
  - Every task executed in order, atomically, one commit per task
353
388
  - `.harness/state.json` accurately reflects position and progress
354
389
  - `.harness/learnings.md` has entries for sessions with non-trivial discoveries
@@ -1,6 +1,6 @@
1
1
  # Harness Roadmap Pilot
2
2
 
3
- > AI-assisted selection of the next highest-impact unblocked roadmap item. Scores candidates, recommends one, assigns it, and transitions to the appropriate next skill.
3
+ > AI-assisted selection of the next highest-impact unblocked roadmap item. Scores candidates, recommends one, and transitions to the appropriate next skill. Selection does NOT assign — the `assignee` field names who is _executing_, and is written at execution start (harness-execution), not at selection.
4
4
 
5
5
  ## When to Use
6
6
 
@@ -14,9 +14,9 @@
14
14
 
15
15
  ### Iron Law
16
16
 
17
- **Never assign or transition without the human confirming the recommendation first.**
17
+ **Never transition without the human confirming the recommendation first. Selection never writes `assignee`.**
18
18
 
19
- Present the ranked candidates, the AI reasoning, and the recommended pick. Wait for explicit confirmation before making any changes.
19
+ Present the ranked candidates, the AI reasoning, and the recommended pick. Wait for explicit confirmation before transitioning. The `assignee` field means _who is executing_ and is claimed at execution start by harness-execution — picking an item must NOT mark it assigned (an early assignee makes the item look human-claimed and the orchestrator silently skips it).
20
20
 
21
21
  ---
22
22
 
@@ -133,28 +133,19 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
133
133
  - If **pick another**: ask which candidate number, then proceed with that pick.
134
134
  - If **no**: stop. No changes made.
135
135
 
136
- ### Phase 4: ASSIGN -- Execute Assignment and Transition
136
+ ### Phase 4: TRANSITION -- Hand Off to the Next Skill
137
137
 
138
- 1. Call `manage_roadmap` with action `update` to assign the feature:
138
+ **Do NOT write the `assignee` field here.** Selection picks the item; it does not
139
+ claim it. harness-execution claims the item (`status=in-progress` + `assignee`) at
140
+ execution start. Writing an assignee at selection makes the orchestrator treat the
141
+ item as already-claimed and silently skip it (the bug this skill must not reintroduce).
142
+ The item stays `planned`/`backlog` with `Assignee: —` and remains orchestrator-eligible.
139
143
 
140
- ```json
141
- manage_roadmap({
142
- path: "<project-root>",
143
- action: "update",
144
- feature: "<feature-name>",
145
- assignee: "<currentUser>"
146
- })
147
- ```
148
-
149
- - This updates the feature's `Assignee` field with assignment history tracking
150
- - Automatically triggers external sync (GitHub Issues) if tracker config exists in `harness.config.json`
151
- - External sync is fire-and-forget — errors are logged but do not block the assignment
152
-
153
- 2. Determine the transition target:
144
+ 1. Determine the transition target:
154
145
  - If the feature has a `spec` field (non-null): transition to `harness:autopilot`
155
146
  - If the feature has no `spec`: transition to `harness:brainstorming`
156
147
 
157
- 3. Present the transition to the human via `emit_interaction`:
148
+ 2. Present the transition to the human via `emit_interaction`:
158
149
 
159
150
  ```json
160
151
  emit_interaction({
@@ -163,16 +154,16 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
163
154
  transition: {
164
155
  completedPhase: "roadmap-pilot",
165
156
  suggestedNext: "<brainstorming|autopilot>",
166
- reason: "Feature '<name>' assigned and ready for <brainstorming|execution>",
157
+ reason: "Feature '<name>' selected and ready for <brainstorming|execution>",
167
158
  artifacts: ["docs/roadmap.md"],
168
159
  requiresConfirmation: true,
169
- summary: "Assigned '<name>' to <user>. <Spec exists -- ready for autopilot|No spec -- needs brainstorming first>.",
160
+ summary: "Selected '<name>'. <Spec exists -- ready for autopilot|No spec -- needs brainstorming first>. (Not assigned — harness-execution claims it at execution start.)",
170
161
  qualityGate: {
171
162
  checks: [
172
163
  { "name": "roadmap-parsed", "passed": true },
173
164
  { "name": "candidate-scored", "passed": true },
174
165
  { "name": "human-confirmed", "passed": true },
175
- { "name": "assignment-written", "passed": true }
166
+ { "name": "no-assignment-at-selection", "passed": true }
176
167
  ],
177
168
  allPassed: true
178
169
  }
@@ -180,7 +171,7 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
180
171
  })
181
172
  ```
182
173
 
183
- 4. Run `harness validate`.
174
+ 3. Run `harness validate`.
184
175
 
185
176
  ---
186
177
 
@@ -190,19 +181,18 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
190
181
  - **`loadProjectRoadmapMode` / `loadTrackerClientConfigFromProject` / `createTrackerClient`** -- Resolve `roadmap.mode` and obtain a `RoadmapTrackerClient` for file-less mode. Import from `@harness-engineering/core`.
191
182
  - **`scoreRoadmapCandidatesForMode`** -- Mode-aware scoring entry point. Import from `@harness-engineering/core`. In file-backed mode delegates to `scoreRoadmapCandidates`; in file-less mode routes through `scoreRoadmapCandidatesFileLess` (priority + createdAt sort, FR-S3).
192
183
  - **`scoreRoadmapCandidates`** -- Underlying file-backed scoring algorithm. Prefer `scoreRoadmapCandidatesForMode` from the skill; direct callers in file-backed-only code paths can still use this.
193
- - **`manage_roadmap update`** -- Used for assignment. Supports `assignee` field which delegates to `assignFeature` internally, handles history tracking, and automatically triggers external sync (GitHub Issues). In file-less mode, `manage_roadmap` dispatches through the tracker; the skill flow is unchanged.
194
- - **`emit_interaction`** -- Used for the skill transition at the end. Transitions to `harness:brainstorming` (no spec) or `harness:autopilot` (spec exists).
184
+ - **`emit_interaction`** -- Used for the skill transition at the end. Transitions to `harness:brainstorming` (no spec) or `harness:autopilot` (spec exists). This skill does NOT call `manage_roadmap update` to assign the `assignee` field is owned by harness-execution (claim at execution start), enforced by RMH005 (`assignee ≠ null ⟺ in-progress`).
195
185
  - **`STRATEGY.md` alignment (Phase 2 step 1a)** -- When present at repo root and valid, loaded via `validateStrategy` + `parseStrategyDoc` + `asStrategyDoc` from `@harness-engineering/core`. Applied as a bounded tiebreaker bonus (max `+0.75`) only when candidates score within `0.05` on the base formula. Boundary: roadmap-pilot READS; `harness-strategy` WRITES. Never modify `STRATEGY.md` from this skill.
196
- - **`harness validate`** -- Run after assignment is written.
186
+ - **`harness validate`** -- Run after the transition is presented.
197
187
 
198
188
  ## Success Criteria
199
189
 
200
190
  1. Roadmap is parsed and unblocked planned/backlog items are scored
201
191
  2. Scoring uses two-tier sort: explicit priority first, then weighted score
202
192
  3. AI reads top candidates' specs and provides recommendation with reasoning
203
- 4. Human confirms before any changes are made
204
- 5. Assignment updates feature field, appends history records, and syncs externally
205
- 6. Reassignment produces two history records (unassigned + assigned)
193
+ 4. Human confirms before the transition is made
194
+ 5. Selection does NOT write the `assignee` field (no assignment, no history record, no external assignee sync) — the pick stays `planned`/`backlog` and orchestrator-eligible
195
+ 6. The `assignee` field is owned by harness-execution, which claims at execution start; RMH005 enforces `assignee ≠ null ⟺ in-progress`
206
196
  7. Transition routes to brainstorming (no spec) or autopilot (spec exists)
207
197
  8. When a pulse report exists, the recommendation rationale cites pulse signal for any top-3 candidate whose area is referenced in the pulse Headlines or Followups.
208
198
  9. When `STRATEGY.md` is present and valid AND two candidates score within `0.05` on the base formula, the recommendation rationale cites strategy-alignment as the tiebreaker. The bonus never overrides a meaningful base-score difference.
@@ -211,12 +201,13 @@ Ask the human in plain text (matching the `y/n/pick another` example above). Do
211
201
 
212
202
  ## Rationalizations to Reject
213
203
 
214
- | Rationalization | Reality |
215
- | ----------------------------------------------------------------------------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
216
- | "The top-scored candidate is obviously correct, so I can assign it without asking the human" | The Iron Law: never assign or transition without the human confirming the recommendation first. |
217
- | "Affinity data is not available so the scoring is degraded -- I should just pick the first planned item" | Proceed without affinity scoring by zeroing out the affinity weight. Position and dependents signals still produce meaningful rankings. |
218
- | "The feature has no spec, but I can skip brainstorming and jump straight to planning since the summary is clear enough" | No spec routes to brainstorming, spec exists routes to autopilot. A one-line roadmap summary is not a spec. |
219
- | "STRATEGY.md exists, so I should let it override the top-scored candidate when alignment is clear" | The alignment bonus is bounded (max `+0.75`) and only fires when base scores are within `0.05`. It is a tiebreaker, not a hard filter — a clearly higher-scored item still wins. |
204
+ | Rationalization | Reality |
205
+ | ----------------------------------------------------------------------------------------------------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
206
+ | "The top-scored candidate is obviously correct, so I can transition without asking the human" | The Iron Law: never transition without the human confirming the recommendation first. |
207
+ | "I'll write the assignee now so the pick is recorded / the user is credited" | Selection never writes `assignee`. An assignee means _in-progress_ (set by harness-execution at execution start). Writing it at selection makes the orchestrator silently skip the item the exact bug this skill must not reintroduce. |
208
+ | "Affinity data is not available so the scoring is degraded -- I should just pick the first planned item" | Proceed without affinity scoring by zeroing out the affinity weight. Position and dependents signals still produce meaningful rankings. |
209
+ | "The feature has no spec, but I can skip brainstorming and jump straight to planning since the summary is clear enough" | No spec routes to brainstorming, spec exists routes to autopilot. A one-line roadmap summary is not a spec. |
210
+ | "STRATEGY.md exists, so I should let it override the top-scored candidate when alignment is clear" | The alignment bonus is bounded (max `+0.75`) and only fires when base scores are within `0.05`. It is a tiebreaker, not a hard filter — a clearly higher-scored item still wins. |
220
211
 
221
212
  ## Examples
222
213
 
@@ -252,25 +243,23 @@ Proceed? (y/n/pick another)
252
243
 
253
244
  Human confirms **y**.
254
245
 
255
- **Phase 4: ASSIGN**
246
+ **Phase 4: TRANSITION**
256
247
 
257
248
  ```
258
- manage_roadmap update: Graph Connector assignee -> @cwarner
259
- History: +1 record (assigned, 2026-04-02)
260
- Roadmap updated: docs/roadmap.md
261
- External sync: github:harness-eng/harness#43 assigned (automatic)
249
+ Graph Connector stays planned, Assignee: (not assigned at selection).
250
+ harness-execution will claim it (status=in-progress + assignee) at execution start.
262
251
 
263
252
  Transitioning to harness:autopilot (spec exists)...
264
253
  ```
265
254
 
266
255
  ## Gates
267
256
 
268
- - **No assignment without human confirmation.** The CONFIRM phase must complete with explicit approval. Never auto-assign.
269
- - **No transition without assignment.** The skill must write the assignment before transitioning to the next skill.
257
+ - **No transition without human confirmation.** The CONFIRM phase must complete with explicit approval before transitioning.
258
+ - **No assignment at selection.** This skill never writes the `assignee` field that field means _who is executing_ and is claimed at execution start by harness-execution. Writing it here would make the orchestrator silently skip the item.
270
259
  - **No scoring without a parsed roadmap.** If `docs/roadmap.md` does not exist or fails to parse, stop with an error.
271
260
 
272
261
  ## Escalation
273
262
 
274
263
  - **When no unblocked candidates exist:** Inform the human. Suggest reviewing blocked items to see if blockers can be resolved, or adding new features via `harness-roadmap --add`.
275
264
  - **When affinity data is unavailable:** Proceed without affinity scoring (weight falls to 0 for all candidates). Note this in the output.
276
- - **When external sync fails:** Log the error, complete the local assignment, and note that external sync can be retried with `harness-roadmap --sync`.
265
+ - **When the human asks you to assign the item to them at selection:** Explain that the `assignee` field means _who is executing_ and is claimed at execution start by harness-execution; assigning at selection makes the orchestrator silently skip the item (enforced by RMH005). Proceed with the transition instead.
@@ -1,7 +1,7 @@
1
1
  import {
2
2
  generateAgentsMd
3
- } from "./chunk-QBC7DI4Q.js";
4
- import "./chunk-N24HCQA4.js";
3
+ } from "./chunk-TCVQVHEF.js";
4
+ import "./chunk-A65QO5KN.js";
5
5
  import "./chunk-N55WOGN3.js";
6
6
  import "./chunk-BDGTZRZ6.js";
7
7
  import "./chunk-PP6ZRL5T.js";
@@ -1,11 +1,11 @@
1
1
  import {
2
2
  checkDependenciesDefinition,
3
3
  handleCheckDependencies
4
- } from "./chunk-527G27VI.js";
5
- import "./chunk-DE5U6KOL.js";
4
+ } from "./chunk-OQFEFVIN.js";
5
+ import "./chunk-XZ2Q4UTW.js";
6
6
  import "./chunk-RC3OI5NK.js";
7
7
  import "./chunk-W6Y7ZW3Y.js";
8
- import "./chunk-N24HCQA4.js";
8
+ import "./chunk-A65QO5KN.js";
9
9
  import "./chunk-N55WOGN3.js";
10
10
  import "./chunk-BDGTZRZ6.js";
11
11
  import "./chunk-PP6ZRL5T.js";
@@ -2,7 +2,7 @@ import {
2
2
  assessProjectDefinition,
3
3
  handleAssessProject,
4
4
  parseToolResponse
5
- } from "./chunk-YQSYCBAH.js";
5
+ } from "./chunk-YF3AY6KS.js";
6
6
  import "./chunk-RC3OI5NK.js";
7
7
  import "./chunk-W6Y7ZW3Y.js";
8
8
  import "./chunk-KFQGP6VL.js";
@@ -1,30 +1,30 @@
1
1
  #!/usr/bin/env node
2
2
  import {
3
3
  startServer
4
- } from "../chunk-WWEVS7RO.js";
5
- import "../chunk-HEIMJJI4.js";
6
- import "../chunk-XX4OLNWQ.js";
7
- import "../chunk-YQSYCBAH.js";
8
- import "../chunk-Z4AWEIBY.js";
9
- import "../chunk-ONCPJGHY.js";
4
+ } from "../chunk-ZOUJP3EY.js";
5
+ import "../chunk-DNX35Q5K.js";
6
+ import "../chunk-C2ZGWMTN.js";
7
+ import "../chunk-YF3AY6KS.js";
8
+ import "../chunk-PZX2Y3RV.js";
9
+ import "../chunk-RGQXHCCH.js";
10
10
  import "../chunk-IOW3MW2K.js";
11
- import "../chunk-BYVT5LVO.js";
11
+ import "../chunk-ERZCC5VH.js";
12
12
  import "../chunk-3ISINLYT.js";
13
13
  import "../chunk-CLI4K2CH.js";
14
- import "../chunk-HWTUUQOW.js";
15
- import "../chunk-527G27VI.js";
16
- import "../chunk-DE5U6KOL.js";
17
- import "../chunk-MGUPSE6D.js";
14
+ import "../chunk-UBTMPO5H.js";
15
+ import "../chunk-OQFEFVIN.js";
16
+ import "../chunk-XZ2Q4UTW.js";
17
+ import "../chunk-HIPBU2Q4.js";
18
18
  import "../chunk-RC3OI5NK.js";
19
19
  import "../chunk-W6Y7ZW3Y.js";
20
20
  import "../chunk-27AJKSQY.js";
21
21
  import "../chunk-6B6UN6SG.js";
22
22
  import "../chunk-HIK6OEUF.js";
23
- import "../chunk-GGKRA7A7.js";
23
+ import "../chunk-YQAFKOOL.js";
24
24
  import "../chunk-ZCZD7FHU.js";
25
25
  import "../chunk-EBJQ6N4M.js";
26
26
  import "../chunk-3WGJMBKH.js";
27
- import "../chunk-N24HCQA4.js";
27
+ import "../chunk-A65QO5KN.js";
28
28
  import "../chunk-N55WOGN3.js";
29
29
  import "../chunk-BDGTZRZ6.js";
30
30
  import "../chunk-PP6ZRL5T.js";
@@ -3,38 +3,38 @@ import {
3
3
  createProgram,
4
4
  installCommandTelemetry,
5
5
  printFirstRunWelcome
6
- } from "../chunk-XFU6SB2Q.js";
7
- import "../chunk-6WQCLCBO.js";
8
- import "../chunk-QBC7DI4Q.js";
9
- import "../chunk-GSP2XIVS.js";
10
- import "../chunk-3ISHDWO7.js";
11
- import "../chunk-NWLGL3IR.js";
6
+ } from "../chunk-FL5SLUKZ.js";
7
+ import "../chunk-ZRKFDE6M.js";
8
+ import "../chunk-TCVQVHEF.js";
9
+ import "../chunk-XEOSSKIB.js";
10
+ import "../chunk-IF3JK3G5.js";
11
+ import "../chunk-FUH4J43G.js";
12
12
  import "../chunk-KET4QQZB.js";
13
13
  import "../chunk-2N4OENGE.js";
14
- import "../chunk-UX3PQKVZ.js";
14
+ import "../chunk-BXKNWSG4.js";
15
15
  import "../chunk-CQ553GZO.js";
16
16
  import "../chunk-ZP5E7YET.js";
17
- import "../chunk-PC2R5RUJ.js";
17
+ import "../chunk-TE5YHHHZ.js";
18
18
  import "../chunk-GISMXMVL.js";
19
19
  import "../chunk-WWACIIBA.js";
20
20
  import "../chunk-TEFCFC4H.js";
21
- import "../chunk-KAJOS3BA.js";
21
+ import "../chunk-LMZ2JTNP.js";
22
22
  import {
23
23
  dispatchSkillsFromGit
24
- } from "../chunk-WWEVS7RO.js";
25
- import "../chunk-HEIMJJI4.js";
26
- import "../chunk-XX4OLNWQ.js";
27
- import "../chunk-YQSYCBAH.js";
28
- import "../chunk-Z4AWEIBY.js";
29
- import "../chunk-ONCPJGHY.js";
24
+ } from "../chunk-ZOUJP3EY.js";
25
+ import "../chunk-DNX35Q5K.js";
26
+ import "../chunk-C2ZGWMTN.js";
27
+ import "../chunk-YF3AY6KS.js";
28
+ import "../chunk-PZX2Y3RV.js";
29
+ import "../chunk-RGQXHCCH.js";
30
30
  import "../chunk-IOW3MW2K.js";
31
- import "../chunk-BYVT5LVO.js";
31
+ import "../chunk-ERZCC5VH.js";
32
32
  import "../chunk-3ISINLYT.js";
33
33
  import "../chunk-CLI4K2CH.js";
34
- import "../chunk-HWTUUQOW.js";
35
- import "../chunk-527G27VI.js";
36
- import "../chunk-DE5U6KOL.js";
37
- import "../chunk-MGUPSE6D.js";
34
+ import "../chunk-UBTMPO5H.js";
35
+ import "../chunk-OQFEFVIN.js";
36
+ import "../chunk-XZ2Q4UTW.js";
37
+ import "../chunk-HIPBU2Q4.js";
38
38
  import "../chunk-RC3OI5NK.js";
39
39
  import "../chunk-W6Y7ZW3Y.js";
40
40
  import "../chunk-27AJKSQY.js";
@@ -43,7 +43,7 @@ import "../chunk-HIK6OEUF.js";
43
43
  import {
44
44
  findConfigFile,
45
45
  loadConfig
46
- } from "../chunk-GGKRA7A7.js";
46
+ } from "../chunk-YQAFKOOL.js";
47
47
  import "../chunk-ZCZD7FHU.js";
48
48
  import "../chunk-EBJQ6N4M.js";
49
49
  import {
@@ -56,7 +56,7 @@ import {
56
56
  readCheckState,
57
57
  shouldRunCheck,
58
58
  spawnBackgroundCheck
59
- } from "../chunk-N24HCQA4.js";
59
+ } from "../chunk-A65QO5KN.js";
60
60
  import {
61
61
  CLI_VERSION
62
62
  } from "../chunk-BM3PWGXQ.js";
@@ -1,12 +1,12 @@
1
1
  import {
2
2
  createCheckPhaseGateCommand,
3
3
  runCheckPhaseGate
4
- } from "./chunk-KAJOS3BA.js";
5
- import "./chunk-GGKRA7A7.js";
4
+ } from "./chunk-LMZ2JTNP.js";
5
+ import "./chunk-YQAFKOOL.js";
6
6
  import "./chunk-ZCZD7FHU.js";
7
7
  import "./chunk-EBJQ6N4M.js";
8
8
  import "./chunk-3WGJMBKH.js";
9
- import "./chunk-N24HCQA4.js";
9
+ import "./chunk-A65QO5KN.js";
10
10
  import "./chunk-N55WOGN3.js";
11
11
  import "./chunk-BDGTZRZ6.js";
12
12
  import "./chunk-PP6ZRL5T.js";