@specship/specship-darwin-arm64 0.28.0 → 0.30.0

package/lib/dist/workflows/defaults/claude-design-implement.yaml

@@ -53,7 +53,9 @@ systemPromptAppend: |
 
   The fidelity principle: never paraphrase or summarise design content
   when capturing it. The snapshot HTML and tokens.css are the
-  zero-loss reference layer; the spec captures contract only (behaviour,
+  zero-loss reference layer, and design.md states every value from them
+  verbatim in a fixed layout (written by `specship design extract`, never
+  transcribed by hand); the spec captures contract only (behaviour,
   accessibility, responsive, interaction states, data shape).
 
   There are two import shapes — exactly one source input is non-empty:
@@ -73,11 +75,12 @@ nodes:
     allowed_tools:
       - Read
       - Write
+      - Edit
       - Bash
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_search
-      - mcp__specship__designer_snapshot
-      - mcp__specship__designer_handoff
+      - specship_explore
+      - specship_search
+      - designer_snapshot
+      - designer_handoff
     prompt: |
       File label:   $INPUT.FILE_LABEL
       Slug:         $INPUT.SLUG
@@ -128,7 +131,7 @@ nodes:
       ── BRANCH B — Source URL is set (live fetch) ──
 
       B1. Fetch the design content from $INPUT.CONNECTOR_URL. Try in order:
-            a. The `designer` MCP — `mcp__specship__designer_snapshot`
+            a. The `designer` MCP — its `designer_snapshot` tool
                (reads the real rendered HTML over CDP from inside the
                cross-origin preview iframe). This is the only path that
                reliably returns the real design for a claude.ai/design URL.
@@ -160,7 +163,29 @@ nodes:
 
       ── BOTH BRANCHES ──
 
-      4. Extract CSS custom properties + literal token-like values
+      4. Write the pixel-exact design reference (DESIGNMD-DOC). The values
+         come from the extractor, never from you:
+
+           specship design extract specs/$INPUT.SLUG/snapshot.html \
+             --out specs/$INPUT.SLUG/design.md --title "$INPUT.FILE_LABEL"
+
+         (If design.md already exists from an earlier import, use
+         `--update specs/$INPUT.SLUG/design.md` instead of `--out`.)
+         If the generated block reports a "Missing linked stylesheet",
+         copy that file next to the snapshot and re-run with `--update`.
+         Then fill design.md's authored sections — Screens & layout,
+         Components & states, Interactions & motion, Responsive behaviour,
+         Assets — following the design-md skill
+         ($AGENT_SKILLS_DIR/design-md/SKILL.md and its
+         references/authored-sections.md): every value copied verbatim
+         from the generated block with its selector, every styled state
+         listed, no approximate wording. Run
+
+           specship design check specs/$INPUT.SLUG/design.md
+
+         and fix findings until it passes.
+
+      5. Extract CSS custom properties + literal token-like values
          (colors, spacing, font sizes, radii, transitions) from the
          snapshot to `specs/$INPUT.SLUG/tokens.css`. Use
          `specship_explore` first to find the project's existing token
@@ -175,6 +200,7 @@ nodes:
         - Any visible interaction or state cues in the CSS (`:hover`,
           `:focus`, `:disabled`, `[aria-*]` selectors) the spec
           author should be aware of.
+        - Whether `specship design check` passes on design.md.
         - For Branch A: note that source.md carries the verbatim decision
           record — the draft step should mine it for stated behaviour.
 
@@ -184,18 +210,19 @@ nodes:
     output_type: spec_markdown
     allowed_tools:
       - Read
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_search
-      - mcp__specship__specship_node
-      - mcp__specship__specship_spec
+      - specship_explore
+      - specship_search
+      - specship_node
+      - specship_spec
     prompt: |
-      Use the spec-author skill (~/.claude/skills/spec-author/SKILL.md
+      Use the spec-author skill ($AGENT_SKILLS_DIR/spec-author/SKILL.md
       and its references/) to draft a SpecShip-compatible spec for the
       Claude Design import. Inputs available on disk:
 
         - specs/$INPUT.SLUG/snapshot.html  (visual + structural source)
         - specs/$INPUT.SLUG/tokens.css     (design tokens)
         - specs/$INPUT.SLUG/source.md      (audit record)
+        - specs/$INPUT.SLUG/design.md      (pixel-exact design reference)
 
       Spec scope: $INPUT.FILE_LABEL component(s).
       Optional inputs: owner=$INPUT.OWNER  priority=$INPUT.PRIORITY
@@ -236,6 +263,7 @@ nodes:
       source: specs/<SLUG>/source.md
       snapshot: specs/<SLUG>/snapshot.html
       tokens: specs/<SLUG>/tokens.css
+      design: specs/<SLUG>/design.md
       ---
       ```
 
@@ -322,14 +350,15 @@ nodes:
         1. The path of the spec file: `specs/$INPUT.SLUG.md`.
         2. The full list of REQ IDs the spec defines, one per line.
         3. The reference files the implementation phase should read:
+             - specs/$INPUT.SLUG/design.md (read first: every value verbatim)
              - specs/$INPUT.SLUG/snapshot.html
              - specs/$INPUT.SLUG/tokens.css
              - specs/$INPUT.SLUG/source.md
         4. The exact next command:
              /specship:spec implement <first REQ ID>
-        5. A one-line reminder that the implementation step should
-           load the snapshot for visual fidelity — the spec covers
-           contract only.
+        5. A one-line reminder that the implementation step builds the UI
+           from design.md (values verbatim) — the spec covers contract
+           only.
 
       Keep the message ≤200 words. The user is about to switch from
       this workflow to the implement workflow; this is the bridge.

package/lib/dist/workflows/defaults/spec-author.yaml

@@ -35,9 +35,9 @@ nodes:
     kind: prompt
     output_type: code_grounding
     allowed_tools:
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_search
-      - mcp__specship__specship_files
+      - specship_explore
+      - specship_search
+      - specship_files
       - Read
     prompt: |
       Feature: $INPUT.DESCRIPTION
@@ -64,9 +64,9 @@ nodes:
     depends_on: [ground]
     output_type: spec_draft
     allowed_tools:
-      - mcp__specship__specship_spec
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_node
+      - specship_spec
+      - specship_explore
+      - specship_node
       - Read
     prompt: |
       Feature: $INPUT.DESCRIPTION
@@ -146,9 +146,9 @@ nodes:
     depends_on: [incorporate]
     output_type: review_findings
     allowed_tools:
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_node
-      - mcp__specship__specship_search
+      - specship_explore
+      - specship_node
+      - specship_search
       - Read
     prompt: |
       Run the spec-author skill's `references/review-checklist.md` against

package/lib/dist/workflows/defaults/spec-fix.yaml

@@ -24,9 +24,9 @@ nodes:
     kind: prompt
     output_type: diagnosis
     allowed_tools:
-      - mcp__specship__specship_spec
-      - mcp__specship__specship_node
-      - mcp__specship__specship_explore
+      - specship_spec
+      - specship_node
+      - specship_explore
       - Read
     prompt: |
       Spec ID: $INPUT.SPEC_ID
@@ -60,8 +60,8 @@ nodes:
       - Edit
       - Write
       - Bash
-      - mcp__specship__specship_node
-      - mcp__specship__specship_spec
+      - specship_node
+      - specship_spec
     prompt: |
       Apply the fix proposed in the diagnosis. Follow any guidance from
       the reviewer.
@@ -162,9 +162,9 @@ nodes:
     depends_on: [approve_links]
     output_type: link_summary
     allowed_tools:
-      - mcp__specship__specship_spec
-      - mcp__specship__specship_link_assert
-      - mcp__specship__specship_link_verify
+      - specship_spec
+      - specship_link_assert
+      - specship_link_verify
     prompt: |
       The reviewer has approved updating link state for $INPUT.SPEC_ID.
 

package/lib/dist/workflows/defaults/spec-implement.yaml

@@ -18,6 +18,22 @@ systemPromptAppend: |
   `path:Symbol` bullet under the requirement's `implementations:` block in
   its spec file — links in the spec file survive any reindex.
 
+  Design-backed specs (REQ-DESIGNMD-006): if the spec file's frontmatter
+  carries a `design:` key, read that design.md before ANY UI edit and build
+  from it — every color, size, spacing, radius, shadow, font and duration
+  verbatim from its Generated reference, every state under Components &
+  states, every breakpoint under Responsive behaviour. Never round or
+  substitute a "close" value; reuse a project token only when its value is
+  exactly equal. Anything design.md doesn't state is a question, not a guess.
+  Keep it current (REQ-DESIGNMD-007): when your change alters the UI — a new
+  or changed component, state, layout, token or copy — update that design.md
+  in the same change: revise the authored sections that describe what moved,
+  add a dated Change log line naming $INPUT.SPEC_ID, and when the design
+  source itself changed run `specship design extract <sources> --update
+  <design.md>`. Then run `specship design check <design.md>`. The postcheck
+  fails a UI change whose design.md has no new Change log entry, and fails a
+  design.md that no longer passes the check.
+
   JIRA-backed specs (REQ-JIRAPUB-004): if the spec file's frontmatter
   carries a `jira_issue:` key, EVERY git commit message you write for this
   work MUST be prefixed with that key ("PROJ-123: ..."), so JIRA's
@@ -29,8 +45,8 @@ nodes:
     kind: prompt
     output_type: spec_summary
     allowed_tools:
-      - mcp__specship__specship_spec
-      - mcp__specship__specship_explore
+      - specship_spec
+      - specship_explore
     prompt: |
       Spec ID: $INPUT.SPEC_ID
 
@@ -68,9 +84,9 @@ nodes:
     depends_on: [fetch_spec, readiness]
     output_type: plan
     allowed_tools:
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_node
-      - mcp__specship__specship_spec
+      - specship_explore
+      - specship_node
+      - specship_spec
       - Read
     prompt: |
       Based on this spec summary, write a concrete implementation plan:
@@ -189,9 +205,9 @@ nodes:
       - Edit
       - Write
       - Bash
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_node
-      - mcp__specship__specship_spec
+      - specship_explore
+      - specship_node
+      - specship_spec
     prompt: |
       Implement the approved plan. Follow it closely.
 
@@ -285,8 +301,8 @@ nodes:
       - Edit
       - Write
       - Bash
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_node
+      - specship_explore
+      - specship_node
     prompt: |
       Lint / typecheck result for the code you just wrote:
 
@@ -377,7 +393,7 @@ nodes:
       - Edit
       - Write
       - Bash
-      - mcp__specship__specship_spec
+      - specship_spec
     prompt: |
       Mechanical post-implement check for $INPUT.SPEC_ID:
 
@@ -467,12 +483,25 @@ nodes:
       # REQ-AUTHG-004.A1 — the full diff, from git, into a run artifact. The
       # human gate and the adversarial reviewer read THIS; the implement node's
       # prose about what it believes it changed is not the change.
+      # REQ-WFINTEG-002 — diff against the commit the run started from (the
+      # readiness step records it), not HEAD: the implement step may commit,
+      # and a diff against HEAD would then be empty.
       # `git add -A -N` registers new files as intent-to-add so untracked work
-      # appears in `git diff HEAD` without staging any content.
+      # appears in the diff without staging any content.
+      BASE=$(cat "$(git rev-parse --git-dir 2>/dev/null)/specship-run-base" 2>/dev/null)
+      if [ -z "$BASE" ] || ! git cat-file -e "$BASE^{commit}" 2>/dev/null; then
+        echo "HALT_CAUSE=the run's base commit was not recorded — cannot show the change this run made"
+        exit 1
+      fi
       git add -A -N >/dev/null 2>&1 || true
+      DIFF=$(git --no-pager diff "$BASE" | head -c 400000)
+      if [ -z "$DIFF" ] && [ -n "$(git status --porcelain --untracked-files=all 2>/dev/null | grep '^??')" ]; then
+        echo "HALT_CAUSE=the diff is empty but the worktree has files git did not include — refusing to review an empty change"
+        exit 1
+      fi
+      echo "DIFF_BASE=$BASE"
       echo "DIFF_BEGIN"
-      git --no-pager diff HEAD | head -c 400000
-      echo
+      printf '%s\n' "$DIFF"
       echo "DIFF_END"
 
   - id: diff_stat
@@ -484,8 +513,10 @@ nodes:
       # insertions/deletions counted by git. Its own small node so the approval
       # message carries the summary while the full diff stays in the `diff`
       # artifact.
+      # Same base as `diff` (REQ-WFINTEG-002), so the stat counts the whole run.
+      BASE=$(cat "$(git rev-parse --git-dir 2>/dev/null)/specship-run-base" 2>/dev/null)
       echo "DIFF_STAT_BEGIN"
-      git --no-pager diff HEAD --stat
+      git --no-pager diff "${BASE:-HEAD}" --stat
       echo "DIFF_STAT_END"
 
   - id: verify
@@ -573,7 +604,7 @@ nodes:
     allowed_tools:
       - Read
       - Edit
-      - mcp__specship__specship_spec
+      - specship_spec
     prompt: |
       Record spec→code links for the changes made in the implement step —
       durably, in the spec file (REQ-PIPETRUST-006). Do NOT call
@@ -652,10 +683,10 @@ nodes:
     allowed_tools:
       - Read
       - Grep
-      - mcp__specship__specship_spec
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_node
-      - mcp__specship__specship_search
+      - specship_spec
+      - specship_explore
+      - specship_node
+      - specship_search
     prompt: |
       Adversarial review (REQ-AUTHG-005.A3). You are NOT the implementer — your
       job is to find what the implementation missed. This node is read-only:
@@ -698,7 +729,7 @@ nodes:
       - Read
       - Write
       - Bash
-      - mcp__specship__specship_spec
+      - specship_spec
     prompt: |
       Try to BREAK the change for $INPUT.SPEC_ID (REQ-THRU-008). You are not
       the implementer. For EACH acceptance criterion write at least one small
@@ -742,8 +773,8 @@ nodes:
     allowed_tools:
       - Read
       - Write
-      - mcp__specship__specship_spec
-      - mcp__specship__specship_explore
+      - specship_spec
+      - specship_explore
     prompt: |
       Propose knowledge this change relied on, for a HUMAN to confirm
       (REQ-KNOW-004). Domain knowledge is human-confirmed, never

package/lib/dist/workflows/defaults/spec-relink.yaml

@@ -21,7 +21,7 @@ nodes:
     kind: prompt
     output_type: orphan_summary
     allowed_tools:
-      - mcp__specship__specship_spec
+      - specship_spec
     prompt: |
       Spec ID: $INPUT.SPEC_ID
 
@@ -34,9 +34,9 @@ nodes:
     depends_on: [present_orphan]
     output_type: candidates
     allowed_tools:
-      - mcp__specship__specship_search
-      - mcp__specship__specship_explore
-      - mcp__specship__specship_node
+      - specship_search
+      - specship_explore
+      - specship_node
     prompt: |
       The orphan(s):
       $present_orphan.output
@@ -67,7 +67,7 @@ nodes:
     depends_on: [approve_relink]
     output_type: link_summary
     allowed_tools:
-      - mcp__specship__specship_link_assert
+      - specship_link_assert
     prompt: |
       The reviewer chose candidate #$approve_relink.output for spec
       $INPUT.SPEC_ID.

package/lib/dist/workflows/defaults/spec-verify.yaml

@@ -68,9 +68,9 @@ nodes:
     output_type: link_summary
     trigger_rule: all_done
     allowed_tools:
-      - mcp__specship__specship_drifted
-      - mcp__specship__specship_spec
-      - mcp__specship__specship_link_verify
+      - specship_drifted
+      - specship_spec
+      - specship_link_verify
     prompt: |
       Test run completed. Output (truncated):
 

package/lib/dist/workflows/executor.d.ts

@@ -156,4 +156,11 @@ export declare function topoSort(nodes: DagNode[]): string[];
  * they are not re-run. Pure — returns a new metadata object.
  */
 export declare function resetFailedNodes(workflow: WorkflowDefinition, metadata: unknown): Record<string, unknown>;
+/**
+ * Host-profile variables every step can reference (REQ-AGENTSWITCH-001), so a
+ * bundled workflow names the run's agent's paths instead of one host's:
+ * `$AGENT_SKILLS_DIR` is the agent's user skills directory. Caller-supplied
+ * variables win.
+ */
+export declare function withAgentVariables(variables: Record<string, string> | undefined, agentId: string | undefined): Record<string, string>;
 //# sourceMappingURL=executor.d.ts.map

package/lib/dist/workflows/executor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"executor.d.ts","sourceRoot":"","sources":["../../src/workflows/executor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAMH,OAAO,EACL,kBAAkB,EAClB,OAAO,EAOR,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EACL,WAAW,EACX,iBAAiB,EAGlB,MAAM,UAAU,CAAC;AAClB,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AACjD,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AAIzD,OAAO,EACL,UAAU,EAIX,MAAM,uBAAuB,CAAC;AAS/B,MAAM,WAAW,cAAc;IAC7B,sEAAsE;IACtE,WAAW,EAAE,MAAM,CAAC;IACpB,uCAAuC;IACvC,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAChC,6CAA6C;IAC7C,SAAS,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACnC,uBAAuB;IACvB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,yEAAyE;IACzE,KAAK,CAAC,EAAE,MAAM,CAAC;IACf;;;;OAIG;IACH,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB;;;OAGG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACtC;;;;OAIG;IACH,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED;;;;;GAKG;AACH,MAAM,MAAM,gBAAgB,GAAG,CAAC,GAAG,EAAE,WAAW,KAAK,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;AAE1E,MAAM,WAAW,aAAa;IAC5B,GAAG,EAAE,WAAW,CAAC;IACjB,sCAAsC;IACtC,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,iBAAiB,CAAC,CAAC;IAC3C,4CAA4C;IAC5C,OAAO,EAAE,GAAG,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;CAClC;AAED,qBAAa,gBAAgB;IAC3B,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,SAAS,CAAmB;IACpC,OAAO,CAAC,OAAO,CAAmC;IAClD;;;;;OAKG;IACH,OAAO,CAAC,WAAW,CAAC,CAAS;IAC7B;;;;OAIG;IACH,OAAO,CAAC,cAAc,CAAC,CAAmB;gBAGxC,WAAW,EAAE,WAAW,EACxB,SAAS,EAAE,gBAAgB,EAC3B,WAAW,CAAC,EAAE,MAAM,EACpB,cAAc,CAAC,EAAE,gBAAgB;IAmB7B,KAAK,CACT,QAAQ,EAAE,kBAAkB,EAC5B,IAAI,EAAE,cAAc,GACnB,OAAO,CAAC,aAAa,CAAC;IAqFzB;;OAEG;IACG,MAAM,CACV,QAAQ,EAAE,kBAAkB,EAC5B,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,IAAI,CAAC,cAAc,EAAE,OAAO,CAAC,GAClC,OAAO,CAAC,aAAa,CAAC;IA8JzB,MAAM,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,GAAE,MAA4B,GAAG,IAAI;IAwBjE;;;;OAIG;IACH,OAAO,CAAC,KAAK,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI;IA0C9C;;;;;;;;OAQG;IACH,MAAM,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,IAAI;IAyC5C;;;;;OAKG;IACH,KAAK,CAAC,KAAK,EAAE,MAAM,GAAG;QAAE,eAAe,EAAE,OAAO,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAA;KAAE;YAoB1D,cAAc;YA2Jd,YAAY;IA2B1B,OAAO,CAAC,YAAY;IAuBpB;;;;;;;;;;;;;OAaG;IACH,OAAO,CAAC,eAAe;IAwDvB,OAAO,CAAC,uBAAuB;IAa/B;;;;;;OAMG;IACH,OAAO,CAAC,mBAAmB;YAsBb,QAAQ;YAgER,aAAa;IAyB3B,OAAO,CAAC,KAAK;CA4Bd;AAMD;;;GAGG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,EAAE,GAAG,MAAM,EAAE,CA6BnD;AAED;;;;;;GAMG;AACH,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,kBAAkB,EAC5B,QAAQ,EAAE,OAAO,GAChB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAiCzB"}
+{"version":3,"file":"executor.d.ts","sourceRoot":"","sources":["../../src/workflows/executor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAMH,OAAO,EACL,kBAAkB,EAClB,OAAO,EAOR,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EACL,WAAW,EACX,iBAAiB,EAGlB,MAAM,UAAU,CAAC;AAClB,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AACjD,OAAO,EAAE,gBAAgB,EAAE,MAAM,uBAAuB,CAAC;AAIzD,OAAO,EACL,UAAU,EAIX,MAAM,uBAAuB,CAAC;AAU/B,MAAM,WAAW,cAAc;IAC7B,sEAAsE;IACtE,WAAW,EAAE,MAAM,CAAC;IACpB,uCAAuC;IACvC,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAChC,6CAA6C;IAC7C,SAAS,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACnC,uBAAuB;IACvB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,yEAAyE;IACzE,KAAK,CAAC,EAAE,MAAM,CAAC;IACf;;;;OAIG;IACH,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB;;;OAGG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACtC;;;;OAIG;IACH,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED;;;;;GAKG;AACH,MAAM,MAAM,gBAAgB,GAAG,CAAC,GAAG,EAAE,WAAW,KAAK,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;AAE1E,MAAM,WAAW,aAAa;IAC5B,GAAG,EAAE,WAAW,CAAC;IACjB,sCAAsC;IACtC,UAAU,EAAE,GAAG,CAAC,MAAM,EAAE,iBAAiB,CAAC,CAAC;IAC3C,4CAA4C;IAC5C,OAAO,EAAE,GAAG,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;CAClC;AAED,qBAAa,gBAAgB;IAC3B,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,SAAS,CAAmB;IACpC,OAAO,CAAC,OAAO,CAAmC;IAClD;;;;;OAKG;IACH,OAAO,CAAC,WAAW,CAAC,CAAS;IAC7B;;;;OAIG;IACH,OAAO,CAAC,cAAc,CAAC,CAAmB;gBAGxC,WAAW,EAAE,WAAW,EACxB,SAAS,EAAE,gBAAgB,EAC3B,WAAW,CAAC,EAAE,MAAM,EACpB,cAAc,CAAC,EAAE,gBAAgB;IAmB7B,KAAK,CACT,QAAQ,EAAE,kBAAkB,EAC5B,IAAI,EAAE,cAAc,GACnB,OAAO,CAAC,aAAa,CAAC;IAqFzB;;OAEG;IACG,MAAM,CACV,QAAQ,EAAE,kBAAkB,EAC5B,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,IAAI,CAAC,cAAc,EAAE,OAAO,CAAC,GAClC,OAAO,CAAC,aAAa,CAAC;IA+JzB,MAAM,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,GAAE,MAA4B,GAAG,IAAI;IAwBjE;;;;OAIG;IACH,OAAO,CAAC,KAAK,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI;IA0C9C;;;;;;;;OAQG;IACH,MAAM,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,IAAI;IAyC5C;;;;;OAKG;IACH,KAAK,CAAC,KAAK,EAAE,MAAM,GAAG;QAAE,eAAe,EAAE,OAAO,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAA;KAAE;YAoB1D,cAAc;YA4Jd,YAAY;IA2B1B,OAAO,CAAC,YAAY;IAuBpB;;;;;;;;;;;;;OAaG;IACH,OAAO,CAAC,eAAe;IAwDvB,OAAO,CAAC,uBAAuB;IAiB/B;;;;;;OAMG;IACH,OAAO,CAAC,mBAAmB;YAsBb,QAAQ;YAgER,aAAa;IAyB3B,OAAO,CAAC,KAAK;CA4Bd;AAMD;;;GAGG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,EAAE,GAAG,MAAM,EAAE,CA6BnD;AAED;;;;;;GAMG;AACH,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,kBAAkB,EAC5B,QAAQ,EAAE,OAAO,GAChB,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAiCzB;AAUD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAChC,SAAS,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,EAC7C,OAAO,EAAE,MAAM,GAAG,SAAS,GAC1B,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAGxB"}

package/lib/dist/workflows/executor.js

@@ -52,6 +52,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.WorkflowExecutor = void 0;
 exports.topoSort = topoSort;
 exports.resetFailedNodes = resetFailedNodes;
+exports.withAgentVariables = withAgentVariables;
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
 const os = __importStar(require("os"));
@@ -66,6 +67,7 @@ const script_1 = require("./runners/script");
 const approval_1 = require("./runners/approval");
 const cancel_1 = require("./runners/cancel");
 const agents_1 = require("./agents");
+const hook_output_1 = require("../activation/hook-output");
 class WorkflowExecutor {
     specQueries;
     worktrees;
@@ -223,6 +225,7 @@ class WorkflowExecutor {
         const wasRejected = existing.status === 'rejected';
         if (wasRejected) {
             const meta = refreshed.metadata ?? {};
+            const revOpts = { ...opts, variables: withAgentVariables(opts.variables, runAgent(meta)) };
             const rejection = meta.rejection;
             const gate = rejection ? workflow.nodes.find((n) => n.id === rejection.nodeId) : undefined;
             const onReject = gate?.kind === 'approval' ? gate.on_reject : undefined;
@@ -261,7 +264,7 @@ class WorkflowExecutor {
                     // Resolve $nodeId.output refs in the on_reject prompt against the
                     // run's accumulated outputs, same as any other node; workflow-level
                     // model/systemPromptAppend apply too (REQ-AUDFIX-005.A2).
-                    const revise = this.resolveNodeRefs(rawRevise, outputs, opts.inputs, opts.variables, workflow);
+                    const revise = this.resolveNodeRefs(rawRevise, outputs, opts.inputs, revOpts.variables, workflow);
                     this.event(runId, 'step_started', { stepId: reviseId, kind: 'prompt', revision: true });
                     const runner = this.runners.get('prompt');
                     result = runner
@@ -271,7 +274,7 @@ class WorkflowExecutor {
                             logsDir,
                             runId,
                             inputs: opts.inputs,
-                            variables: opts.variables,
+                            variables: revOpts.variables,
                             outputs,
                             verbose: opts.verbose,
                             agent: runAgent(meta),
@@ -455,6 +458,7 @@ class WorkflowExecutor {
         const order = topoSort(workflow.nodes);
         const nodeById = new Map(workflow.nodes.map((n) => [n.id, n]));
         const meta = run.metadata ?? {};
+        opts = { ...opts, variables: withAgentVariables(opts.variables, runAgent(meta)) };
         const nodeStates = new Map(Object.entries(meta.nodeStates ?? {}));
         const outputs = new Map(Object.entries(meta.outputs ?? {}));
         // Initialize states for nodes the resume didn't already cover.
@@ -482,7 +486,7 @@ class WorkflowExecutor {
             }
             // Check when: condition.
             if (node.when !== undefined) {
-                const ctx = this.makeSubstitutionContext(outputs, opts.inputs, opts.variables);
+                const ctx = this.makeSubstitutionContext(outputs, opts.inputs, opts.variables, workflow);
                 const passes = (0, condition_evaluator_1.evaluateCondition)(node.when, ctx);
                 if (!passes) {
                     nodeStates.set(nodeId, 'skipped');
@@ -640,7 +644,7 @@ class WorkflowExecutor {
      * is the workflow value followed by the node value, joined by a blank line.
      */
     resolveNodeRefs(node, outputs, inputs, variables, workflow) {
-        const ctx = this.makeSubstitutionContext(outputs, inputs, variables);
+        const ctx = this.makeSubstitutionContext(outputs, inputs, variables, workflow);
         switch (node.kind) {
             case 'prompt': {
                 const p = node;
@@ -686,12 +690,15 @@ class WorkflowExecutor {
             }
         }
     }
-    makeSubstitutionContext(outputs, inputs, variables) {
+    makeSubstitutionContext(outputs, inputs, variables, workflow) {
         return {
             outputs,
             inputs: inputs ?? {},
             variables: variables ?? {},
             spillDir: os.tmpdir(),
+            // The workflow's step ids, so shell text like `$i.md` is not taken for
+            // a step reference (REQ-WFINTEG-001).
+            ...(workflow?.nodes ? { nodeIds: new Set(workflow.nodes.map((n) => n.id)) } : {}),
         };
     }
     /**
@@ -910,6 +917,16 @@ function sanitizeFilename(s) {
 function sleep(ms) {
     return new Promise((resolve) => setTimeout(resolve, ms));
 }
+/**
+ * Host-profile variables every step can reference (REQ-AGENTSWITCH-001), so a
+ * bundled workflow names the run's agent's paths instead of one host's:
+ * `$AGENT_SKILLS_DIR` is the agent's user skills directory. Caller-supplied
+ * variables win.
+ */
+function withAgentVariables(variables, agentId) {
+    const traits = (0, hook_output_1.hostTraitsFor)(agentId ?? agents_1.DEFAULT_WORKFLOW_AGENT);
+    return { AGENT_SKILLS_DIR: `~/${traits.configDir}/skills`, ...(variables ?? {}) };
+}
 /** The agent stored with a run; undefined for runs created before agents were recorded (→ default). */
 function runAgent(meta) {
     const a = meta[agents_1.RUN_AGENT_KEY];