sisyphi 1.2.25 → 1.2.26

package/dist/cli.js

@@ -842,7 +842,6 @@ var init_ask_store = __esm({
     init_notify();
     init_state();
     ACTIONABLE_KINDS = /* @__PURE__ */ new Set([
-      "validation",
       "decision",
       "context",
       "error",
@@ -6838,7 +6837,7 @@ if [ ! -f "$RESULT_FILE" ]; then printf 'dismiss' > "$RESULT_FILE"; fi
 }
 
 // src/cli/commands/ask.ts
-import { approveDeck, notifyDeck, launchReview, display } from "@crouton-kit/humanloop";
+import { notifyDeck, launchReview, display } from "@crouton-kit/humanloop";
 var ULID_RE = /^[0-9A-HJKMNP-TV-Z]{26}$/;
 function validateAskId(askId) {
   if (!ULID_RE.test(askId)) {
@@ -6849,10 +6848,9 @@ function validateAskId(askId) {
   }
 }
 var ROOT_HELP = `
-ask: user-engagement leaves. deck (option picks) | approve (yes/no gate) | notify (FYI) | review (markdown annotation) | state (recovery) | show (display).
+ask: user-engagement leaves. deck (option picks) | notify (FYI) | review (markdown annotation) | state (recovery) | show (display).
 
-  deck submit FILE     Pose 1+ questions with named alternatives.          | use when picking between concrete named paths or bundling decisions
-  approve TITLE        Yes/no gate on a single concrete action. Blocks.    | use when you need explicit consent before a destructive or irreversible step
+  deck submit FILE     Pose 1+ questions with named alternatives.          | use when picking between concrete named paths, a yes/no gate, or bundling decisions
   notify TITLE         Fire-and-forget acknowledgement. Non-blocking.      | use when the user should know something but no answer is needed
   review {sub}         Line-by-line markdown annotation workflow.          | use when surfacing a document for per-line annotation rather than option picking
   state {sub}          Inspect existing ask state.                         | use on recovery only \u2014 not the normal path
@@ -6878,7 +6876,7 @@ DECK DESIGN
   Each option is a concrete forward path. The user picks an option to commit to a direction; each label must name a real path with its tradeoffs spelled out, grounded in *this* codebase or *this* exploration. Reference specifics \u2014 file names, framework constraints, prior decisions \u2014 not generic descriptions.
 
   Bound option count to 2\u20134 per interaction.
-    < 2: collapses to yes/no \u2014 use \`sis ask approve\` instead.
+    < 2: a bare yes/no gate is a 2-option deck (Yes / No) \u2014 author it that way.
     > 4: too granular for the user to weigh \u2014 split into two interactions or cut weak options.
 
   Options must be mutually exclusive forward paths. Reject feeling-based prompts ("Happy with this?") and fallback-to-freetext options ("Comment", "Maybe"). If "Reject" is a real path, name what rejecting routes to (back to design? abandon? try a different framing?).
@@ -6917,10 +6915,9 @@ DECK JSON SCHEMA
                                                          // $SISYPHUS_SESSION_DIR/context/)
                                                          // and pass a basename like
                                                          // "summary.md".
-    kind?           "decision" | "validation" | "context" | "error" | "notify"
+    kind?           "decision" | "context" | "error" | "notify"
                                                          // display hint for inbox icon/sort:
-                                                         //   decision   \u2014 fork in the road
-                                                         //   validation \u2014 sign-off on work
+                                                         //   decision   \u2014 fork in the road (incl. yes/no sign-off)
                                                          //   context    \u2014 background needing response
                                                          //   error      \u2014 recovery picker
                                                          //   notify     \u2014 use the \`notify\` leaf instead
@@ -6951,34 +6948,12 @@ Effects
 
 Exit codes: 0 ok | 2 usage (invalid file, validation error, missing session)
 `;
-var APPROVE_HELP = `
-approve: yes/no gate on a single concrete action.
-
-Use when the next step is irreversible and proceeding needs explicit user consent. The \`--body\` should describe the concrete action so the user can judge it without a follow-up question. When nuance matters (yes-but or no-because), use \`sis ask deck submit\` instead.
-
-Same blocking semantics as \`sis ask deck submit\`: orchestrator runs foreground, agent uses run_in_background.
-
-Input
-  <title>             required \u2014 action being approved
-  --subtitle <s>      optional \u2014 one-line context shown below the title
-  --body <b>          optional \u2014 markdown body describing what the user is approving
-  --session <id>      optional \u2014 session id (defaults to SISYPHUS_SESSION_ID)
-
-Output (stdout, bare JSON \u2014 NOT the standard {ok, data} envelope)
-  { "askId", "approved": boolean, "completedAt", "responses" }
-  on error: stderr diagnostic + non-zero exit
-
-Effects
-  Blocks until the user picks. Creates an ask in the session store, marks it answered on completion.
-
-Exit codes: 0 ok | 2 usage (missing session)
-`;
 var NOTIFY_HELP = `
 notify: fire-and-forget acknowledgement to the dashboard.
 
 Non-blocking \u2014 returns immediately with { askId } before the user sees it.
 
-Use when the user should know about something (a milestone hit, a finding surfaced, a heads-up before a long-running step) but no answer is needed. When even a minimal answer matters, use \`sis ask approve\`. When picking between paths matters, use \`sis ask deck submit\`. Notifications do not have a return path.
+Use when the user should know about something (a milestone hit, a finding surfaced, a heads-up before a long-running step) but no answer is needed. When even a minimal answer matters (incl. a yes/no gate), use \`sis ask deck submit\`. Notifications do not have a return path.
 
 Input
   <title>             required \u2014 notification title
@@ -6997,7 +6972,7 @@ Exit codes: 0 ok | 2 usage (missing session)
 var REVIEW_BRANCH_HELP = `
 Line-by-line markdown annotation workflow. The agent submits the file; in tmux an editor pane opens next to the agent (also reachable from the dashboard inbox). The user annotates per-line with <Space>c and submits with <Space>s. Multiple open/close cycles are supported \u2014 the draft persists until explicitly submitted.
 
-Use when surfacing a document the user should annotate per-line rather than pick options on. For a global yes/no, use \`sis ask approve\`. For named alternatives, use \`sis ask deck submit\`.
+Use when surfacing a document the user should annotate per-line rather than pick options on. For a global yes/no or named alternatives, use \`sis ask deck submit\`.
 
 Branches
   submit <file>      Submit a .md file for review; blocks until the user submits.   | use when you have a markdown document to send for annotation
@@ -7147,9 +7122,6 @@ function registerAsk(program2) {
   deck.command("submit <file>").description("Submit a deck JSON file; blocks until the user resolves it").option("--session <id>", "Session id (defaults to SISYPHUS_SESSION_ID)").addHelpText("after", DECK_SUBMIT_HELP).action(async (file, opts) => {
     await submit(file, opts);
   });
-  ask.command("approve <title>").description("Yes/no gate on a single concrete action; blocks until answered").option("--subtitle <s>", "Optional one-line context shown below the title").option("--body <b>", "Markdown body describing the action the user is approving").option("--session <id>", "Session id (defaults to SISYPHUS_SESSION_ID)").addHelpText("after", APPROVE_HELP).action(async (title, opts) => {
-    await approve(title, opts);
-  });
   ask.command("notify <title>").description("Fire-and-forget acknowledgement on the dashboard (non-blocking)").option("--body <b>", "Optional markdown body").option("--session <id>", "Session id (defaults to SISYPHUS_SESSION_ID)").addHelpText("after", NOTIFY_HELP).action(async (title, opts) => {
     await notify(title, opts);
   });
@@ -7397,25 +7369,6 @@ async function submit(file, opts) {
   const { output } = await submitDeck(decisions, opts);
   process.stdout.write(JSON.stringify(output) + "\n");
 }
-async function approve(title, opts) {
-  const deck = approveDeck(title, {
-    ...opts.subtitle !== void 0 ? { subtitle: opts.subtitle } : {},
-    ...opts.body !== void 0 ? { body: opts.body } : {}
-  });
-  const { askId, output } = await submitDeck(deck, opts);
-  if (!output) throw new Error("blocking ask returned no output");
-  if (output.kind === "review") throw new Error("approve returned review output \u2014 internal error");
-  const approveResponse = output.responses.find((r) => r.id === "approve");
-  const approved = approveResponse?.selectedOptionId === "yes";
-  process.stdout.write(
-    JSON.stringify({
-      askId,
-      approved,
-      completedAt: output.completedAt,
-      responses: output.responses
-    }) + "\n"
-  );
-}
 async function notify(title, opts) {
   const deck = notifyDeck(title, opts.body !== void 0 ? { body: opts.body } : {});
   const { askId } = await submitDeck(deck, opts, { blocking: false, kindOverride: "notify" });
@@ -7572,10 +7525,9 @@ async function show(path, opts) {
     });
   }
   const windowOpt = rawWindow !== void 0 ? rawWindow : "auto";
-  const watch = opts.watch === true;
   let paneId;
   try {
-    const r = display(path, { watch, window: windowOpt });
+    const r = display(path, { window: windowOpt });
     paneId = r.paneId;
   } catch (err) {
     process.stderr.write(`[sis ask show] display error: ${err.message}