cueloop 0.1.0-alpha.16 → 0.1.0-alpha.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cueloop",
-  "version": "0.1.0-alpha.16",
+  "version": "0.1.0-alpha.18",
   "license": "Apache-2.0",
   "type": "module",
   "bin": {
@@ -10,9 +10,9 @@
     ".": "./src/main.ts"
   },
   "dependencies": {
-    "@cueloop/schema": "0.1.0-alpha.16",
-    "@cueloop/daemon": "0.1.0-alpha.16",
-    "@cueloop/client": "0.1.0-alpha.16"
+    "@cueloop/schema": "0.1.0-alpha.18",
+    "@cueloop/daemon": "0.1.0-alpha.18",
+    "@cueloop/client": "0.1.0-alpha.18"
   },
   "publishConfig": {
     "access": "public"

package/src/args.ts

@@ -9,16 +9,16 @@ export function parseArgs(argv: string[]): ParsedArgs {
   const positional: string[] = [];
   const flags: Record<string, string | boolean> = {};
   for (let i = 0; i < argv.length; i++) {
-    const a = argv[i]!;
-    if (!a.startsWith("--")) {
-      positional.push(a);
+    const argument = argv[i]!;
+    if (!argument.startsWith("--")) {
+      positional.push(argument);
       continue;
     }
-    const eq = a.indexOf("=");
-    if (eq !== -1) {
-      flags[a.slice(2, eq)] = a.slice(eq + 1);
+    const equalsIndex = argument.indexOf("=");
+    if (equalsIndex !== -1) {
+      flags[argument.slice(2, equalsIndex)] = argument.slice(equalsIndex + 1);
     } else {
-      const key = a.slice(2);
+      const key = argument.slice(2);
       const next = argv[i + 1];
       if (next !== undefined && !next.startsWith("--")) {
         flags[key] = next;
@@ -31,7 +31,7 @@ export function parseArgs(argv: string[]): ParsedArgs {
   return { positional, flags };
 }
 
-export function flagStr(flags: Record<string, string | boolean>, key: string): string | undefined {
-  const v = flags[key];
-  return typeof v === "string" ? v : undefined;
+export function stringFlag(flags: Record<string, string | boolean>, key: string): string | undefined {
+  const value = flags[key];
+  return typeof value === "string" ? value : undefined;
 }

package/src/main.ts

@@ -1,14 +1,14 @@
 #!/usr/bin/env bun
 /**
- * cueloop entry points (#18): `cueloop` opens the TUI on the inbox,
+ * cueloop entry points: `cueloop` opens the TUI on the inbox,
  * `cueloop diff` reviews the working tree, `cueloop review <pr>` a PR,
  * `cueloop session *` mirrors the daemon API for agents and scripts,
  * `cueloop serve` shares a session over ssh (read-only observers),
  * `cueloop daemon` runs the daemon in the foreground.
  */
 
-import { parseArgs, flagStr } from "./args";
-import { sessionCommand } from "./session-cmds";
+import { parseArgs, stringFlag } from "./args";
+import { sessionCommand } from "./session-commands";
 import { workingTreeDiff } from "./working-tree";
 import { DaemonClient } from "@cueloop/daemon/client";
 import { openReview, resolveWorkspace } from "@cueloop/daemon/review";
@@ -51,11 +51,11 @@ async function main(): Promise<number> {
     }
     case "serve": {
       const { positional, flags } = parseArgs(argv.slice(1));
-      const port = flagStr(flags, "port");
+      const port = stringFlag(flags, "port");
       const { serveClient } = await import("@cueloop/client");
       const handle = await serveClient({
         port: port !== undefined ? Number(port) : undefined,
-        host: flagStr(flags, "host"),
+        host: stringFlag(flags, "host"),
         sessionId: positional[0],
       });
       console.log(

package/src/pr.ts

@@ -1,5 +1,5 @@
 /**
- * PR review entry (#18): `cueloop review <pr>` fetches the PR diff through the
+ * PR review entry: `cueloop review <pr>` fetches the PR diff through the
  * user's `gh` CLI (auth fully delegated), opens a diff session, and posts the
  * verdict back to the PR as a real review when the session resolves.
  * `cueloop review-post <session-id> <pr>` is the non-interactive post-back

package/src/{session-cmds.ts → session-commands.ts}

@@ -1,5 +1,5 @@
 /**
- * `cueloop session *` - mirrors the daemon socket API 1:1 (#14). One surface,
+ * `cueloop session *` - mirrors the daemon socket API 1:1. One surface,
  * three consumers: agent adapters, the dev loop, and integrations. Output is
  * JSON on stdout; exit code 0 unless the daemon returned an error.
  */
@@ -7,7 +7,7 @@
 import { newAnnotationId, type ArtifactType, type VerdictKind } from "@cueloop/schema";
 import { DaemonClient } from "@cueloop/daemon/client";
 import { openReview, verdictResponse, type ReviewNote } from "@cueloop/daemon/review";
-import { parseArgs, flagStr } from "./args";
+import { parseArgs, stringFlag } from "./args";
 
 async function readStdin(): Promise<string> {
   return await new Response(Bun.stdin.stream()).text();
@@ -24,19 +24,19 @@ export async function sessionCommand(argv: string[]): Promise<number> {
   try {
     switch (verb) {
       case "create": {
-        const contentFile = flagStr(flags, "content-file");
+        const contentFile = stringFlag(flags, "content-file");
         const content = contentFile ? await Bun.file(contentFile).text() : await readStdin();
         // per-file agent notes for diff sessions: a JSON array of { path, body }
-        const notesFile = flagStr(flags, "notes-file");
+        const notesFile = stringFlag(flags, "notes-file");
         const notes = notesFile ? (JSON.parse(await Bun.file(notesFile).text()) as ReviewNote[]) : undefined;
         const review = await openReview(client, {
-          type: (flagStr(flags, "type") ?? "plan") as ArtifactType,
+          type: (stringFlag(flags, "type") ?? "plan") as ArtifactType,
           content,
-          cwd: flagStr(flags, "cwd"),
-          agent: flagStr(flags, "agent"),
-          agentSessionId: flagStr(flags, "agent-session-id"),
-          planPath: flagStr(flags, "plan-path"),
-          title: flagStr(flags, "title"),
+          cwd: stringFlag(flags, "cwd"),
+          agent: stringFlag(flags, "agent"),
+          agentSessionId: stringFlag(flags, "agent-session-id"),
+          planPath: stringFlag(flags, "plan-path"),
+          title: stringFlag(flags, "title"),
           notes,
         });
         out(review.session);
@@ -47,13 +47,13 @@ export async function sessionCommand(argv: string[]): Promise<number> {
         return 0;
       }
       case "list": {
-        const status = flagStr(flags, "status") as "pending" | "resolved" | undefined;
+        const status = stringFlag(flags, "status") as "pending" | "resolved" | undefined;
         out(await client.sessionList(status ? { status } : undefined));
         return 0;
       }
       case "wait": {
         const id = required(positional[1], "session id");
-        const timeoutMs = Number(flagStr(flags, "timeout-ms") ?? "60000");
+        const timeoutMs = Number(stringFlag(flags, "timeout-ms") ?? "60000");
         const session = await client.sessionWait(id, timeoutMs);
         if (session === null) {
           out({ status: "pending" });
@@ -65,16 +65,16 @@ export async function sessionCommand(argv: string[]): Promise<number> {
       }
       case "annotate": {
         const id = required(positional[1], "session id");
-        const body = flagStr(flags, "body") ?? "";
-        const quote = required(flagStr(flags, "quote"), "--quote");
+        const body = stringFlag(flags, "body") ?? "";
+        const quote = required(stringFlag(flags, "quote"), "--quote");
         out(
           await client.sessionAnnotate(id, {
-            id: flagStr(flags, "annotation-id") ?? newAnnotationId(),
-            kind: flagStr(flags, "kind") ?? "comment",
+            id: stringFlag(flags, "annotation-id") ?? newAnnotationId(),
+            kind: stringFlag(flags, "kind") ?? "comment",
             anchor: {
               quote,
-              prefix: flagStr(flags, "prefix") ?? "",
-              suffix: flagStr(flags, "suffix") ?? "",
+              prefix: stringFlag(flags, "prefix") ?? "",
+              suffix: stringFlag(flags, "suffix") ?? "",
             },
             body,
           }),
@@ -83,13 +83,13 @@ export async function sessionCommand(argv: string[]): Promise<number> {
       }
       case "resolve": {
         const id = required(positional[1], "session id");
-        const kind = required(flagStr(flags, "verdict"), "--verdict") as VerdictKind;
-        out(await client.sessionResolve(id, kind, flagStr(flags, "summary") ?? ""));
+        const kind = required(stringFlag(flags, "verdict"), "--verdict") as VerdictKind;
+        out(await client.sessionResolve(id, kind, stringFlag(flags, "summary") ?? ""));
         return 0;
       }
       case "submit-revision": {
         const id = required(positional[1], "session id");
-        const contentFile = flagStr(flags, "content-file");
+        const contentFile = stringFlag(flags, "content-file");
         const content = contentFile ? await Bun.file(contentFile).text() : await readStdin();
         out(await client.sessionSubmitRevision(id, content));
         return 0;
@@ -105,7 +105,7 @@ export async function sessionCommand(argv: string[]): Promise<number> {
   }
 }
 
-function required<T>(v: T | undefined, what: string): T {
-  if (v === undefined) throw new Error(`missing ${what}`);
-  return v;
+function required<T>(value: T | undefined, description: string): T {
+  if (value === undefined) throw new Error(`missing ${description}`);
+  return value;
 }

package/src/working-tree.ts

@@ -9,7 +9,7 @@ async function git(args: string[], cwd: string): Promise<string | null> {
 
 export async function workingTreeDiff(cwd = process.cwd()): Promise<string> {
   const tracked = (await git(["diff", "HEAD"], cwd)) ?? "";
-  // untracked files included by default (map note: changed-first)
+  // untracked files included by default
   const untrackedList = (await git(["ls-files", "--others", "--exclude-standard"], cwd)) ?? "";
   let untracked = "";
   for (const file of untrackedList.split("\n").filter(Boolean)) {