@coderook/cli 0.31.5 → 0.31.6

package/.claude-plugin/plugin.json

@@ -2,7 +2,7 @@
   "name": "coderook",
   "displayName": "CodeRook",
   "description": "Save, browse and restore whole-snapshot versions of a project on CodeRook, from Claude Code.",
-  "version": "0.31.5",
+  "version": "0.31.6",
   "author": {
     "name": "ACCA Gaming Productions",
     "url": "https://coderook.com"

package/dist/cli/src/api.js

@@ -10,6 +10,7 @@ exports.projects = projects;
 exports.findPublicProject = findPublicProject;
 exports.findProject = findProject;
 exports.health = health;
+exports.requestReviews = requestReviews;
 exports.openProposal = openProposal;
 exports.proposals = proposals;
 exports.mergeTracks = mergeTracks;
@@ -267,6 +268,11 @@ async function health() {
     });
     return (await response.json());
 }
+/** Ask people, by username, to review a merge. */
+async function requestReviews(mergeTrackId, usernames) {
+    const body = await call(`/v1/merge-tracks/${mergeTrackId}/review-requests`, { method: "POST", body: { reviewers: usernames } });
+    return body.asked;
+}
 /** Offer a line of work to another, usually main. */
 async function openProposal(repositoryId, input) {
     return call(`/v1/repositories/${repositoryId}/proposals`, {

package/dist/cli/src/cli.js

@@ -1667,6 +1667,17 @@ async function commandMerge(parsed) {
     const folder = folderFor({ ...parsed, positional: parsed.positional.slice(1) });
     const summary = await findMerge(folder, reference);
     const detail = await (0, api_js_1.mergeTrack)(summary.id);
+    /*
+      Asking people to look. Named the way people name each other, and said
+      back by name, so a typo in a username is visible rather than silent.
+    */
+    const ask = flagText(parsed, "ask");
+    if (ask) {
+        const asked = await (0, api_js_1.requestReviews)(summary.id, ask.split(",").map((name) => name.trim()).filter(Boolean));
+        console.log(asked.length
+            ? `Asked ${asked.map((person) => person.displayName).join(", ")} to review ${summary.reference}.`
+            : "Nobody new was asked.");
+    }
     if (hasFlag(parsed, "cancel")) {
         await (0, api_js_1.cancelMerge)(summary.id);
         console.log(`${summary.reference} cancelled. Your upload is still stored and nothing published was touched.`);
@@ -1714,6 +1725,20 @@ async function commandMerge(parsed) {
         for (const blocked of now.provisional.blockedByChecks ?? []) {
             console.log(`  ${red("check")}  ${blocked}`);
         }
+        for (const one of now.changeRequest?.reviews ?? []) {
+            if (one.verdict === "comment" && !one.body)
+                continue;
+            const said = one.verdict === "approve"
+                ? accent("approved")
+                : one.verdict === "request_changes"
+                    ? red("asked for changes")
+                    : dim("commented");
+            console.log(`  ${one.reviewerName ?? "A reviewer"} ${said}${one.body ? dim(`: ${one.body}`) : ""}`);
+        }
+        const waiting = (now.changeRequest?.requested ?? []).filter((person) => !person.reviewed);
+        if (waiting.length) {
+            console.log(dim(`  waiting on ${waiting.map((person) => person.displayName).join(", ")}`));
+        }
     }
     console.log(`
 ${accent(now.mergeTrack.reference)} · ${now.provisional.fileCount} files · ` +
@@ -2556,12 +2581,16 @@ const SPECS = [
         options: [
             { flags: "--ship <action>", description: "its builds become downloads on the version" },
             { flags: "--no-ship <action>", description: "keep its files without attaching them" },
+            { flags: "--on-save <action>", description: "run it on every save" },
+            { flags: "--on-proposal <action>", description: "run it on proposals; its result becomes a check" },
+            { flags: "--by-hand <action>", description: "run it only when somebody starts it" },
             { flags: "--project <name>", description: "which project" },
         ],
         examples: [
             "cbx actions",
             'cbx actions --ship "Windows installer"',
             'cbx actions --no-ship "Windows installer"',
+            'cbx actions --on-proposal Tests',
         ],
         run: service_commands_js_1.commandWorkflows,
     },
@@ -2686,6 +2715,7 @@ const SPECS = [
             { flags: "--theirs", description: "keep theirs" },
             { flags: "--both", description: "keep both" },
             { flags: "--drop", description: "abandon the upload" },
+            { flags: "--ask <usernames>", description: "ask people to review it, comma separated" },
         ],
         run: commandMerge,
     },

package/dist/cli/src/service_commands.js

@@ -314,6 +314,46 @@ async function commandWorkflows(parsed) {
         }
         console.log("");
     }
+    /*
+      When an action runs. A trigger used to be a label nothing acted on; now a
+      save or a proposal really does start the actions that asked for it, so
+      choosing is worth a flag.
+    */
+    const WHEN = [
+        ["on-save", "version_saved", "runs on every save"],
+        ["on-proposal", "merge_opened", "runs when a line is proposed, and again on each save to it"],
+        ["by-hand", "manual", "runs when somebody starts it"],
+    ];
+    for (const [flag, trigger, meaning] of WHEN) {
+        const named = parsed.flags.get(flag);
+        if (named === undefined)
+            continue;
+        if (typeof named !== "string" || !named.trim()) {
+            console.error(red("Which action?") + dim(`  cbx actions --${flag} "Tests"`));
+            return 1;
+        }
+        const target = found.find((workflow) => workflow.name.toLowerCase() === named.trim().toLowerCase());
+        if (!target) {
+            console.error(red(`${project.name} has no action called ${named.trim()}.`));
+            if (found.length) {
+                console.error(dim(`  It has: ${found.map((one) => one.name).join(", ")}`));
+            }
+            return 1;
+        }
+        try {
+            await (0, api_js_1.changeWorkflow)(project.id, target.id, { trigger });
+        }
+        catch (error) {
+            console.error(red(error instanceof Error ? error.message : String(error)));
+            return 1;
+        }
+        target.trigger = trigger;
+        console.log(green(`${target.name} ${meaning}.`));
+        if (trigger === "merge_opened") {
+            console.log(dim("  Its result becomes a required check on the proposal."));
+        }
+        console.log("");
+    }
     if (!found.length) {
         console.log(dim("No actions on this project."));
         return 0;

package/dist/desktop-app/src/main/tracks.js

@@ -256,8 +256,36 @@ class Tracks {
             ready: body.provisional?.ready ?? false,
             unresolvedPaths: body.provisional?.unresolvedPaths ?? [],
             blockedByChecks: body.provisional?.blockedByChecks ?? [],
+            proposal: body.proposal ?? null,
+            request: body.changeRequest
+                ? { ...body.changeRequest, requested: body.changeRequest.requested ?? [] }
+                : null,
         };
     }
+    /**
+     * Offer a line to another, usually main.
+     *
+     * The service reads both lines, combines what it can, and answers with the
+     * merge it opened — which the window then shows like any other.
+     */
+    async propose(repositoryId, input) {
+        const body = await this.call(`/v1/repositories/${repositoryId}/proposals`, {
+            method: "POST",
+            body: JSON.stringify(input.body ? input : { from: input.from, into: input.into, title: input.title }),
+        });
+        return {
+            mergeTrackId: body.mergeTrack.id,
+            reference: body.mergeTrack.reference,
+            unresolved: body.conflicts.unresolved,
+        };
+    }
+    /** Say where you stand on a proposal. */
+    async review(mergeTrackId, verdict, note) {
+        await this.call(`/v1/merge-tracks/${mergeTrackId}/review`, {
+            method: "PUT",
+            body: JSON.stringify(note ? { verdict, body: note } : { verdict }),
+        });
+    }
     /**
      * One side of a conflict, as text.
      *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@coderook/cli",
-  "version": "0.31.5",
+  "version": "0.31.6",
   "description": "CodeRook from the command line, on any operating system",
   "license": "SEE LICENSE IN LICENSE.txt",
   "homepage": "https://coderook.com",