cueloop 0.1.0-alpha.45 → 0.1.0-alpha.46

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cueloop",
-  "version": "0.1.0-alpha.45",
+  "version": "0.1.0-alpha.46",
   "description": "Terminal review surface for coding agents: annotate plans and diffs, return a verdict, send structured feedback back",
   "homepage": "https://github.com/mmurakaru/cueloop#readme",
   "bugs": {
@@ -27,10 +27,10 @@
     "access": "public"
   },
   "dependencies": {
-    "@cueloop/adapters": "0.1.0-alpha.45",
-    "@cueloop/client": "0.1.0-alpha.45",
-    "@cueloop/daemon": "0.1.0-alpha.45",
-    "@cueloop/schema": "0.1.0-alpha.45"
+    "@cueloop/adapters": "0.1.0-alpha.46",
+    "@cueloop/client": "0.1.0-alpha.46",
+    "@cueloop/daemon": "0.1.0-alpha.46",
+    "@cueloop/schema": "0.1.0-alpha.46"
   },
   "engines": {
     "bun": ">=1.3.0"

package/src/main.ts

@@ -8,10 +8,12 @@
  * observers), `cueloop daemon` runs the daemon in the foreground.
  */
 
+import { basename, resolve } from "node:path";
 import { parseArgs, stringFlag, type ParsedArgs } from "./args";
 import {
   isDiffReview,
   isPlanReview,
+  isPrototypeReview,
   isPrReview,
   isSessionId,
   openTargetMessage,
@@ -47,6 +49,8 @@ async function main(): Promise<number> {
       return planCommand(argv.slice(1));
     case "diff":
       return diffCommand(argv.slice(1));
+    case "prototype":
+      return prototypeCommand(argv.slice(1));
     case "serve": {
       const { positional, flags } = parseArgs(argv.slice(1));
       const port = stringFlag(flags, "port");
@@ -165,6 +169,39 @@ async function planCommand(argv: string[]): Promise<number> {
   return openReviewOfKind(isPlanReview, "plan", openSelector(parseArgs(argv)));
 }
 
+/**
+ * `cueloop prototype <file.html>` creates a review of a rendered HTML file;
+ * a selector or `--open`/`--latest` (or a bare call with no file) opens the
+ * latest pending prototype review instead.
+ */
+async function prototypeCommand(argv: string[]): Promise<number> {
+  const parsed = parseArgs(argv);
+  const selector = openSelector(parsed);
+  const wantsOpen = "open" in parsed.flags || "latest" in parsed.flags;
+  const looksLikeFile =
+    selector !== undefined && !isSessionId(selector) && selector.endsWith(".html");
+  if (wantsOpen || !looksLikeFile)
+    return openReviewOfKind(isPrototypeReview, "prototype", selector);
+
+  const path = resolve(selector);
+  const html = await Bun.file(path)
+    .text()
+    .catch(() => undefined);
+  if (html === undefined) {
+    console.error(`prototype: cannot read ${path}`);
+    return 1;
+  }
+  const client = await DaemonClient.connect({ autostart: true });
+  const review = await openReview(client, {
+    type: "prototype",
+    content: html,
+    prototypePath: path,
+    title: basename(path),
+  });
+  client.close();
+  return runTui(review.id);
+}
+
 /**
  * `cueloop diff` disambiguates create from open by intent:
  *   - a selector (`cueloop diff <id|title>`) or an explicit `--open`/`--latest`
@@ -242,6 +279,7 @@ function printHelp(): void {
       "  cueloop diff [id|title]          review your working tree (untracked files included);",
       "                                   with a clean tree, open the latest pending diff review",
       "  cueloop review <pr>              review a pull request (--no-tui prints the session)",
+      "  cueloop prototype <file.html>    review a rendered HTML prototype (or open the latest by id/title)",
       "",
       "share:",
       "  cueloop serve [session-id]       share over ssh: observers are read-only,",

package/src/open-target.test.ts

@@ -5,6 +5,7 @@ import type { ArtifactType, ReviewSession } from "@cueloop/schema";
 import {
   isDiffReview,
   isPlanReview,
+  isPrototypeReview,
   isPrReview,
   isSessionId,
   openTargetMessage,
@@ -59,6 +60,12 @@ describe("verb scope predicates are disjoint", () => {
     expect(isPrReview(session({ id: "ses_p", type: "plan" }))).toBe(false);
   });
 
+  test("isPrototypeReview matches only prototype artifacts", () => {
+    expect(isPrototypeReview(session({ id: "ses_x", type: "prototype" }))).toBe(true);
+    expect(isPrototypeReview(session({ id: "ses_p", type: "plan" }))).toBe(false);
+    expect(isPrototypeReview(session({ id: "ses_d", type: "diff" }))).toBe(false);
+  });
+
   test("a PR review resolves under exactly one scope - never both diff and review", () => {
     // Arrange
     const prReview = session({ id: "ses_pr", type: "diff", pr: "42" });

package/src/open-target.ts

@@ -28,6 +28,9 @@ import type { ReviewSession } from "@cueloop/schema";
 export function isPlanReview(session: ReviewSession): boolean {
   return session.artifact.type === "plan";
 }
+export function isPrototypeReview(session: ReviewSession): boolean {
+  return session.artifact.type === "prototype";
+}
 export function isDiffReview(session: ReviewSession): boolean {
   return session.artifact.type === "diff" && session.artifact.meta.pr === undefined;
 }

package/src/session-commands.ts

@@ -43,6 +43,7 @@ export async function sessionCommand(argv: string[]): Promise<number> {
           agent: stringFlag(flags, "agent"),
           agentSessionId: stringFlag(flags, "agent-session-id"),
           planPath: stringFlag(flags, "plan-path"),
+          prototypePath: stringFlag(flags, "prototype-path"),
           title: stringFlag(flags, "title"),
           notes,
         });