tines 0.0.322 → 0.0.324

package/README.md

@@ -40,6 +40,7 @@ tines issues create <project> --title "…" -d @description.md
 tines issues create <project> --title "Linked" --blocked-by Other/12 --blocks Other/14 --duplicate-of Other/9
 tines issues show <project>/<number>
 tines issues move <project>/<number> <action>        # a workflow transition
+tines issues move <project>/<number> <action> --expect-revision <n>   # refuse unless the issue is still at the decision revision `issues show` printed
 tines issues hold <project>/<number>                 # stop new admission, keep permission
 tines issues release <project>/<number>              # resume eligibility if permission remains on
 tines issues cancel-run <project>/<number> <run-id>   # request bounded cancellation
@@ -81,6 +82,12 @@ every evidence page with no ceiling and takes no `--max-items`.
 When a recurrence is also supplied, the relationships apply only to the first issue; later
 scheduled instances start without copied relationships.
 
+`workflows show` prints a workflow's `revision`, which advances by one on every committed
+save. `workflows edit <workflow> … --expect-revision <n>` (or `"expected_revision"` in the
+JSON body; the flag wins) refuses the update with `workflow_conflict`, writing nothing, when
+the workflow is no longer at that revision. The CLI never fills the revision in for you and
+never retries: re-read with `workflows show`, re-apply your change, and send the new revision.
+
 For a shared project's recurring schedule, `schedules list` and `schedules show` report
 your saved future-instance permission and its epoch. They are read receipts: the CLI and
 API keys cannot turn personal permission on or off. Open the schedule in the browser to

package/dist/index.js

@@ -7631,6 +7631,21 @@ function printIssueLinks(links) {
     table(linkRows(links.duplicated_by));
   }
 }
+var MOVE_REVISION_HELP = `
+Every state change advances the issue's decision revision by one (\`tines
+issues show\` prints it), so leaving a state and returning to it is a different
+revision. Pass --expect-revision <n> to have the move refused with
+decision_refresh_required \u2014 nothing written \u2014 when the issue is no longer on
+the state visit you read. Without it, only a change that commits during this
+command is caught.
+`;
+function parseDecisionRevisionOption(value) {
+  const result = Number(value);
+  if (!/^(0|[1-9]\d*)$/.test(value) || !Number.isSafeInteger(result)) {
+    throw new Error("expect-revision must be a non-negative integer");
+  }
+  return result;
+}
 function printIssueDetail(issue) {
   const shared = issue;
   if (shared.mode === "member") {
@@ -7680,6 +7695,8 @@ ${comment.body}`
       `own state: ${issue.state.name} (${issue.state.category}) \u2014 dormant while this is a duplicate`
     );
   }
+  if (issue.decision_revision !== void 0)
+    console.log(`decision revision: ${issue.decision_revision}`);
   if (issue.project_archived_at !== null) {
     console.log(
       `project archived: ${timestamp(issue.project_archived_at)} \u2014 this issue is read-only`
@@ -7984,7 +8001,11 @@ function register3(program3) {
     }
   );
   withCommon(
-    issues.command("move <ref> <action>").description("Take a current transition (members: no permission change)")
+    issues.command("move <ref> <action>").description("Take a current transition (members: no permission change)").option(
+      "--expect-revision <n>",
+      "refuse the move unless the issue is still at this decision revision (see `tines issues show`)",
+      parseDecisionRevisionOption
+    ).addHelpText("after", MOVE_REVISION_HELP)
   ).action(async (ref, action2, opts) => {
     const api = client(opts);
     const issue = await resolveIssue(api, ref);
@@ -8026,7 +8047,30 @@ function register3(program3) {
         } : { transition_id: transition.transition_id };
       }
     }
-    const moved = await api.transitionIssue(issue.id, request);
+    if (opts.expectRevision !== void 0) {
+      if (request.transition_id === void 0) {
+        const transition = issue.allowed_transitions.find(
+          (candidate) => candidate.name.toLowerCase() === action2.toLowerCase()
+        );
+        if (!transition)
+          throw new Error(`No current transition named "${action2}"; refresh the issue`);
+        request = { transition_id: transition.transition_id };
+      }
+      request = {
+        ...request,
+        expected_state_id: issue.state.id,
+        expected_decision_revision: opts.expectRevision
+      };
+    }
+    const moved = await api.transitionIssue(issue.id, request).catch((error) => {
+      if (opts.expectRevision === void 0 || !(error instanceof ApiError) || error.code !== "decision_refresh_required")
+        throw error;
+      const current = error.details?.current_decision_revision;
+      die(
+        `${error.message} (${error.code})
+hint: re-read with \`tines issues show ${ref}\`, then retry with --expect-revision ${typeof current === "number" ? current : "<decision revision>"}`
+      );
+    });
     if (opts.json) return printJson(moved);
     const movedMember = moved;
     console.log(
@@ -14899,9 +14943,18 @@ function resolveBasePair(lib, pair, stateName) {
   }
   return state.id;
 }
+var WORKFLOW_REVISION_HELP = `
+Every committed update advances the workflow's revision by one (\`tines
+workflows show\` prints it). Pass --expect-revision <n>, or "expected_revision"
+in the JSON body (the flag wins), to have the update refused with
+workflow_conflict \u2014 nothing written \u2014 when the workflow is no longer at the
+revision you read. Without it, only a save that commits during this command is
+caught.
+`;
 function printWorkflowDetail(wf, lib) {
   console.log(`${wf.name}${wf.is_system ? " (standard, read-only)" : ""}  [${wf.id}]`);
   if (wf.description) console.log(wf.description);
+  console.log(`revision: ${wf.revision}`);
   console.log("\nstates:");
   const byId = new Map(wf.states.map((s) => [s.id, s]));
   const rows = wf.states.length === 0 ? [] : formatTable(
@@ -15012,7 +15065,11 @@ see \`tines workflows create --help\` for the expected shape`
     }
   );
   withCommon(
-    workflows.command("edit <id-or-name> [json]").description("Update a workflow from a JSON definition and/or flags").option("-f, --file <path>", 'read the JSON definition from a file ("-" for stdin)').option("-n, --name <name>", "rename the workflow").option("-d, --description <text>", "set the description").option("--initial-state <id-or-name>", "set the initial state").option("--no-prompts", 'allow new states without initial "prompt" instructions').addHelpText("after", WORKFLOW_JSON_HELP)
+    workflows.command("edit <id-or-name> [json]").description("Update a workflow from a JSON definition and/or flags").option("-f, --file <path>", 'read the JSON definition from a file ("-" for stdin)').option("-n, --name <name>", "rename the workflow").option("-d, --description <text>", "set the description").option("--initial-state <id-or-name>", "set the initial state").option("--no-prompts", 'allow new states without initial "prompt" instructions').option(
+      "--expect-revision <n>",
+      "refuse the update unless the workflow is still at this revision (see `tines workflows show`)",
+      (value) => parsePositiveIntegerOption(value, "expect-revision")
+    ).addHelpText("after", WORKFLOW_JSON_HELP + WORKFLOW_REVISION_HELP)
   ).action(
     async (ref, inline, opts) => {
       const api = client(opts);
@@ -15024,10 +15081,18 @@ see \`tines workflows create --help\` for the expected shape`
       if (opts.name !== void 0) body.name = opts.name;
       if (opts.description !== void 0) body.description = opts.description;
       if (opts.initialState !== void 0) body.initial_state = opts.initialState;
-      if (Object.keys(body).length === 0) {
+      if (opts.expectRevision !== void 0) body.expected_revision = opts.expectRevision;
+      if (Object.keys(body).every((key) => key === "expected_revision")) {
         die("nothing to update: pass JSON and/or --name/--description/--initial-state");
       }
-      const updated = await api.updateWorkflow(wf.id, body);
+      const updated = await api.updateWorkflow(wf.id, body).catch((error) => {
+        if (!(error instanceof ApiError) || error.code !== "workflow_conflict") throw error;
+        const current = error.details?.current_revision;
+        die(
+          `${error.message} (${error.code})
+hint: re-read with \`tines workflows show ${wf.id}\`, then retry with --expect-revision ${typeof current === "number" ? current : "<revision>"}`
+        );
+      });
       if (opts.json) return printJson(updated);
       console.log(`updated workflow "${updated.name}" (${updated.id})
 `);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tines",
-  "version": "0.0.322",
+  "version": "0.0.324",
   "description": "CLI for Tines, an orchestration layer for AI agents",
   "repository": {
     "type": "git",