@gobing-ai/knowledge-kit 0.1.11 → 0.1.12

package/plugins/kk/workflows/kk-itc.yaml

@@ -19,6 +19,8 @@ vars:
   playbook: ""
   rubric: "tech-accuracy"
   outline: ""
+  outline_bullets_only: ""
+  plan_only: "false"
   research: ""
   judge: ""
   auto: ""
@@ -88,11 +90,22 @@ nodes:
         command: |
           cid="${vars.__runId}-author"
           v=$(jq -c --arg wd "${vars.work_dir}" --arg agent "${vars.agent}" --arg rubric "${vars.rubric}" --arg outline "${vars.outline}" \
+            --arg ob "${vars.outline_bullets_only}" \
             --arg run "${vars.__runId}" --arg dd "${vars.__definitionDigest}" \
             '{work_dir:$wd, agent:$agent, rubric:$rubric, outline:$outline, writer:.writer, playbook:.playbook,
+              outline_bullets_only:(if $ob == "true" then "true" else "false" end),
               auto:(.auto|tostring), unslop:(.unslop|tostring), judge:(.judge|tostring), max_revisions:(.maxRevisions|tostring),
               parent_run_id:$run, child_run_id:($run + "-author"), definition_digest:$dd, options_digest:.optionsDigest}' \
             "${vars.work_dir}/itc-options.json") || exit 1
+          # Child definition: the running tree first (a project checkout shadows the install),
+          # else the installed kk package root via its real path (the run's cwd may be anywhere).
+          wf="plugins/kk/workflows/kk-itc-author.yaml"
+          if [ ! -f "$wf" ]; then
+            test -x "$(command -v kk)" || { echo "kk-itc-author.yaml not found from the cwd and no kk binary for fallback" >&2; exit 1; }
+            kk_pkg="$(dirname "$(dirname "$(python3 -c 'import os,sys;print(os.path.realpath(sys.argv[1]))' "$(command -v kk)")")")"
+            wf="$kk_pkg/plugins/kk/workflows/kk-itc-author.yaml"
+          fi
+          test -f "$wf" || { echo "child workflow not found at $wf (cwd tree or kk package root)" >&2; exit 1; }
           owned() {
             env -u SPUR_WORKFLOW_RUN_ACTIVE -u SPUR_EXPECTED_DEFINITION_DIGEST "$@" &
             pid=$!
@@ -105,7 +118,7 @@ nodes:
           case "$st" in
             done) exit 0 ;;
             paused) owned spur workflow continue "$cid" --yes ;;
-            "") owned spur workflow run plugins/kk/workflows/kk-itc-author.yaml --run-id "$cid" --vars "$v" ;;
+            "") owned spur workflow run "$wf" --run-id "$cid" --vars "$v" ;;
             *) echo "child $cid is $st; refusing to relaunch it" >&2; exit 1 ;;
           esac
           st=$(spur workflow trace "$cid" --json 2>/dev/null | jq -r '.run.status // empty')
@@ -191,8 +204,10 @@ nodes:
 
   - id: send-confirm
     description: >-
-      Operator approval of a LIVE send: `pause: true` stops the run and RESUME is the approval
-      (no action); cancelling is the rejection. Skipped for draft-only or empty plans.
+      Operator approval of a send: `pause: true` stops the run and RESUME is the approval
+      (no action); cancelling is the rejection. Entered for any plan with targets — a draft
+      still creates real remote drafts, not only a LIVE send — and skipped for empty plans
+      and plan_only runs (the plan preview is the stop point).
     dependsOn:
       - adapt-verify
       - check-targets
@@ -200,12 +215,19 @@ nodes:
     condition:
       kind: shell
       options:
-        command: 'jq -e ".live == true and (.targets // [] | length) > 0" "${vars.work_dir}/4-publish/publish-plan.json" >/dev/null 2>&1'
+        # Item 5: draft-only plans with targets also confirm (draft saves are real remote
+        # artifacts). plan_only runs stop at the plan instead of pausing for a no-op confirm.
+        command: 'jq -e "((.targets // []) | length) > 0" "${vars.work_dir}/4-publish/publish-plan.json" >/dev/null 2>&1 && test "${vars.plan_only}" != "true"'
 
   - id: fanout-send
     description: "Dispatch target payloads via executor fan-out and record receipts"
     dependsOn:
       - send-confirm
+    # plan_only stops after the confirmed plan: no sends, no recovery, no settle.
+    condition:
+      kind: shell
+      options:
+        command: 'test "${vars.plan_only}" != "true"'
     action:
       kind: shell
       options:
@@ -217,6 +239,10 @@ nodes:
     description: "Retry safe effect-none target failures once within budget"
     dependsOn:
       - fanout-send
+    condition:
+      kind: shell
+      options:
+        command: 'test "${vars.plan_only}" != "true"'
     action:
       kind: shell
       options:
@@ -228,6 +254,12 @@ nodes:
     description: "Reconcile target receipts and settle store selection"
     dependsOn:
       - fanout-recovery
+    # plan_only left no receipts to reconcile; with targets present the stage would fail on
+    # their absence, so the plan preview ends the run before settle (report still runs).
+    condition:
+      kind: shell
+      options:
+        command: 'test "${vars.plan_only}" != "true"'
     action:
       kind: shell
       options: