cueloop 0.1.0-alpha.43 → 0.1.0-alpha.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cueloop",
-  "version": "0.1.0-alpha.43",
+  "version": "0.1.0-alpha.44",
   "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.43",
-    "@cueloop/client": "0.1.0-alpha.43",
-    "@cueloop/daemon": "0.1.0-alpha.43",
-    "@cueloop/schema": "0.1.0-alpha.43"
+    "@cueloop/adapters": "0.1.0-alpha.44",
+    "@cueloop/client": "0.1.0-alpha.44",
+    "@cueloop/daemon": "0.1.0-alpha.44",
+    "@cueloop/schema": "0.1.0-alpha.44"
   },
   "engines": {
     "bun": ">=1.3.0"

package/src/main.ts

@@ -19,8 +19,8 @@ import {
 } from "./open-target";
 import { sessionCommand } from "./session-commands";
 import { CLI_VERSION } from "./version";
-import { workingTreeDiff } from "./working-tree";
 import { DaemonClient } from "@cueloop/daemon/client";
+import { workingTreeDiff } from "@cueloop/daemon/working-tree";
 import type { ReviewSession } from "@cueloop/schema";
 import { openReview, resolveWorkspace } from "@cueloop/daemon/review";
 

package/src/working-tree.ts

@@ -1,100 +0,0 @@
-/** Working-tree diff capture for `cueloop diff`, untracked files included. */
-
-import { join } from "node:path";
-import type { DiffFileContents } from "@cueloop/schema";
-
-async function git(args: string[], cwd: string): Promise<string | null> {
-  const gitProcess = Bun.spawn(["git", ...args], { cwd, stdout: "pipe", stderr: "ignore" });
-  const stdout = await new Response(gitProcess.stdout).text();
-  if ((await gitProcess.exited) !== 0) return null;
-  return stdout.trim();
-}
-
-/** The blob at HEAD for a path, or "" when the path is not in HEAD (new file). */
-async function headContents(path: string, cwd: string): Promise<string> {
-  const gitProcess = Bun.spawn(["git", "show", `HEAD:${path}`], {
-    cwd,
-    stdout: "pipe",
-    stderr: "ignore",
-  });
-  const stdout = await new Response(gitProcess.stdout).text();
-  return (await gitProcess.exited) === 0 ? stdout : "";
-}
-
-/** The working-tree file, or "" when it is gone (deleted). */
-async function workingContents(path: string, cwd: string): Promise<string> {
-  try {
-    return await Bun.file(join(cwd, path)).text();
-  } catch {
-    return "";
-  }
-}
-
-/** A NUL byte marks content git would treat as binary - not text-curatable. */
-function looksBinary(text: string): boolean {
-  for (let index = 0; index < text.length; index++) {
-    if (text.charCodeAt(index) === 0) return true;
-  }
-  return false;
-}
-
-/**
- * Full old/new contents for every tracked change against HEAD, classified by
- * git's own status letter. Renames/copies and binary files are omitted: a
- * path+text pair cannot express a rename header or a binary blob, so curation
- * would emit a patch that drops the old path or corrupts the file - leaving them
- * out keeps those changes in the patch but simply not curatable. The
- * NUL-delimited stream survives paths with spaces or unicode.
- */
-async function trackedFileContents(cwd: string): Promise<DiffFileContents[]> {
-  const nameStatus = (await git(["diff", "HEAD", "--name-status", "-z"], cwd)) ?? "";
-  const tokens = nameStatus.split("\0").filter((token) => token.length > 0);
-  const files: DiffFileContents[] = [];
-  let cursor = 0;
-  while (cursor < tokens.length) {
-    const code = tokens[cursor++]![0];
-    if (code === "R" || code === "C") {
-      cursor += 2;
-      continue;
-    }
-    const path = tokens[cursor++]!;
-    const oldContents = code === "A" ? "" : await headContents(path, cwd);
-    const newContents = code === "D" ? "" : await workingContents(path, cwd);
-    if (looksBinary(oldContents) || looksBinary(newContents)) continue;
-    const status = code === "A" ? "added" : code === "D" ? "deleted" : "modified";
-    files.push({ path, oldContents, newContents, status });
-  }
-  return files;
-}
-
-export interface WorkingTreeDiff {
-  patch: string;
-  /** Full file contents per curatable changed file, keeping curation applyable. */
-  files: DiffFileContents[];
-}
-
-export async function workingTreeDiff(cwd = process.cwd()): Promise<WorkingTreeDiff> {
-  const tracked = (await git(["diff", "HEAD"], cwd)) ?? "";
-  const files = await trackedFileContents(cwd);
-  // 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)) {
-    // `git diff --no-index` exits 1 when the files differ - that is success here
-    const gitProcess = Bun.spawn(["git", "diff", "--no-index", "--", "/dev/null", file], {
-      cwd,
-      stdout: "pipe",
-      stderr: "ignore",
-    });
-    const stdout = await new Response(gitProcess.stdout).text();
-    const code = await gitProcess.exited;
-    if ((code === 0 || code === 1) && stdout.trim()) {
-      untracked += stdout.trim() + "\n";
-      const newContents = await workingContents(file, cwd);
-      if (!looksBinary(newContents)) {
-        files.push({ path: file, oldContents: "", newContents, status: "added" });
-      }
-    }
-  }
-  return { patch: [tracked, untracked].filter(Boolean).join("\n"), files };
-}