@cueloop/daemon 0.1.0-alpha.102 → 0.1.0-alpha.104

package/src/daemon-sdk.ts

@@ -81,6 +81,7 @@ export class ReviewDaemonSdk {
 
       return () => {};
     }
+
     const dispose = subscribeThreadState({
       connect: connectThreadObserver(this.connectionOptions),
       matches: (event) => event.sessionId === input.threadId,
@@ -128,6 +129,7 @@ export class ReviewDaemonSdk {
 
       if (!comment)
         throw new DaemonClientError("not_found", "SDK comment reply target does not exist");
+
       const rootId = comment.replyTo ?? comment.id;
       const root = thread.annotations.find((entry) => entry.id === rootId);
 
@@ -277,6 +279,7 @@ export class OwnerDaemonSdk extends ReviewDaemonSdk {
       return Promise.reject(
         new DaemonTransportError("connection", "SDK connection is closed", "not_sent"),
       );
+
     if (!accepted.submissionIds.length)
       return Promise.reject(
         new DaemonClientError("invalid_params", "SDK wait requires accepted submissions"),
@@ -294,11 +297,13 @@ export class OwnerDaemonSdk extends ReviewDaemonSdk {
       let detach: (() => void) | undefined;
       const finish = (result?: AgentCompletion, error?: DaemonSdkError) => {
         if (done) return;
+
         done = true;
         clearTimeout(timer);
         signal.removeEventListener("abort", abort);
         this.waits.delete(controller);
         detach?.();
+
         if (result) resolve(result);
         else reject(error);
       };
@@ -321,11 +326,13 @@ export class OwnerDaemonSdk extends ReviewDaemonSdk {
       );
 
       signal.addEventListener("abort", abort, { once: true });
+
       if (signal.aborted) {
         abort();
 
         return;
       }
+
       detach = subscribeThreadState({
         connect: connectThreadObserver(this.connectionOptions),
         matches: (event) =>
@@ -347,6 +354,7 @@ export class OwnerDaemonSdk extends ReviewDaemonSdk {
 
             return;
           }
+
           if (receipt.outcome) finish({ ...accepted, outcome: receipt.outcome, state });
         },
         onError: (error) => {

package/src/diff-hot-reload.test.ts

@@ -59,6 +59,7 @@ describe("session.refreshDiff", () => {
     });
 
     if (init.exitCode !== 0) throw new Error(init.stderr.toString());
+
     writeFileSync(join(repo, "a.ts"), "export const a = 2;\n");
     const first = await core.repoDiff(repo);
     const workspace = await resolveWorkspace(repo);
@@ -81,6 +82,7 @@ describe("session.refreshDiff", () => {
     });
 
     if (describe.exitCode !== 0) throw new Error(describe.stderr.toString());
+
     const result = await core.sessionRefreshDiff(session.id);
     const current = core.sessionGet(session.id);
 
@@ -100,6 +102,7 @@ describe("session.refreshDiff", () => {
     });
 
     if (init.exitCode !== 0) throw new Error(init.stderr.toString());
+
     writeFileSync(join(repo, "a.ts"), "export const a = 2;\n");
     const first = await core.repoDiff(repo);
     const workspace = await resolveWorkspace(repo);
@@ -339,6 +342,7 @@ describe("the fs watcher drives hot-reload", () => {
     });
 
     if (init.exitCode !== 0) throw new Error(init.stderr.toString());
+
     const session = await core.workbenchSession(repo);
 
     expect(session.artifact.meta.vcs).toBe("jj");

package/src/diff-watcher.test.ts

@@ -18,6 +18,7 @@ test("startup reconciliation watches a directory whose creation event was missed
   const change = Promise.withResolvers<void>();
   const watcher = new DiffWatcher(() => {
     refreshes++;
+
     if (refreshes === 1) startup.resolve();
     else change.resolve();
   });
@@ -91,6 +92,7 @@ test("replacing a watch during reconciliation stops the old directory traversal"
   const watcher = new DiffWatcher(() => {});
   const spy = spyOn(directoryReads, "readdir").mockImplementation((path) => {
     scanned.push(String(path));
+
     if (path === first) {
       scanStarted.resolve();
 

package/src/diff-watcher.ts

@@ -38,6 +38,7 @@ function ignoredDirectories(repoRoot: string): Set<string> {
   );
 
   if (!result || result.exitCode !== 0) return ignored;
+
   for (const line of result.stdout.toString().split("\n")) {
     const relative = line.trim().replace(/\/$/, "");
 
@@ -77,9 +78,11 @@ function resolveGitDirs(repoRoot: string): { gitDir: string; commonDir: string }
   const result = spawnGit(["rev-parse", "--absolute-git-dir", "--git-common-dir"], repoRoot);
 
   if (!result || result.exitCode !== 0) return null;
+
   const [gitDir, commonRaw] = result.stdout.toString().trim().split("\n");
 
   if (!gitDir) return null;
+
   const commonDir = commonRaw
     ? isAbsolute(commonRaw)
       ? commonRaw
@@ -117,10 +120,12 @@ export class DiffWatcher {
 
     if (existing) {
       existing.sessionIds.add(sessionId);
+
       if (vcs === "jj") this.startJjPoll(existing, repoRoot, sessionId);
 
       return;
     }
+
     const repoWatch: RepoWatch = {
       handles: [],
       sessionIds: new Set([sessionId]),
@@ -133,6 +138,7 @@ export class DiffWatcher {
     };
 
     this.repoWatches.set(repoRoot, repoWatch);
+
     if (vcs === "jj") this.startJjPoll(repoWatch, repoRoot, sessionId);
 
     // one watcher per non-ignored working-tree directory; a tracked-file change re-captures
@@ -157,6 +163,7 @@ export class DiffWatcher {
       this.watchInto(repoWatch.handles, join(gitDirs.commonDir, "refs"), { recursive: true }, () =>
         this.scheduleRepoRefresh(repoRoot),
       );
+
       if (gitDirs.commonDir !== gitDirs.gitDir)
         this.watchInto(repoWatch.handles, gitDirs.commonDir, { recursive: false }, (filename) => {
           if (isRefChange(filename)) this.scheduleRepoRefresh(repoRoot);
@@ -169,7 +176,9 @@ export class DiffWatcher {
 
   private startJjPoll(repoWatch: RepoWatch, repoRoot: string, sessionId: string): void {
     repoWatch.jjSessionIds.add(sessionId);
+
     if (repoWatch.jjPoll !== null) return;
+
     repoWatch.jjPoll = setInterval(() => this.scheduleRepoRefresh(repoRoot), 2000);
   }
 
@@ -183,7 +192,9 @@ export class DiffWatcher {
     const repoWatch = this.repoWatches.get(repoRoot);
 
     if (!repoWatch) return false;
+
     if (repoWatch.watchedDirs.has(dir)) return true;
+
     if (ALWAYS_IGNORED_DIRS.has(basename(dir)) || repoWatch.ignored.has(dir)) return true;
 
     const opened = this.watchInto(repoWatch.handles, dir, { recursive: false }, (filename) => {
@@ -193,6 +204,7 @@ export class DiffWatcher {
     });
 
     if (!opened) return false;
+
     repoWatch.watchedDirs.add(dir);
 
     let entries: Dirent[];
@@ -223,8 +235,11 @@ export class DiffWatcher {
       // the path is already gone (a transient temp dir); nothing to watch
       return;
     }
+
     if (ALWAYS_IGNORED_DIRS.has(basename(path))) return;
+
     if (isGitIgnored(repoRoot, path)) return;
+
     this.watchTree(repoRoot, path);
   }
 
@@ -233,11 +248,13 @@ export class DiffWatcher {
     const repoWatch = this.repoWatches.get(repoRoot);
 
     if (!repoWatch) return;
+
     if (!repoWatch.watchedDirs.has(dir)) {
       this.extendWatch(repoRoot, dir);
 
       return;
     }
+
     let entries: Dirent[];
 
     try {
@@ -245,9 +262,12 @@ export class DiffWatcher {
     } catch {
       return;
     }
+
     if (this.repoWatches.get(repoRoot) !== repoWatch) return;
+
     for (const entry of entries) {
       if (this.repoWatches.get(repoRoot) !== repoWatch) return;
+
       if (entry.isDirectory() && !entry.isSymbolicLink())
         // eslint-disable-next-line no-await-in-loop
         await this.reconcileWatchTree(repoRoot, join(dir, entry.name));
@@ -285,15 +305,21 @@ export class DiffWatcher {
     const repoWatch = this.repoWatches.get(repoRoot);
 
     if (!repoWatch) return;
+
     repoWatch.sessionIds.delete(sessionId);
     repoWatch.jjSessionIds.delete(sessionId);
+
     if (repoWatch.jjSessionIds.size === 0 && repoWatch.jjPoll !== null) {
       clearInterval(repoWatch.jjPoll);
       repoWatch.jjPoll = null;
     }
+
     if (repoWatch.sessionIds.size > 0) return;
+
     if (repoWatch.debounce !== null) clearTimeout(repoWatch.debounce);
+
     if (repoWatch.jjPoll !== null) clearInterval(repoWatch.jjPoll);
+
     for (const handle of repoWatch.handles) handle.close();
     this.repoWatches.delete(repoRoot);
   }
@@ -302,14 +328,19 @@ export class DiffWatcher {
     const repoWatch = this.repoWatches.get(repoRoot);
 
     if (!repoWatch) return;
+
     if (repoWatch.debounce !== null) clearTimeout(repoWatch.debounce);
+
     repoWatch.debounce = setTimeout(async () => {
       repoWatch.debounce = null;
+
       if (repoWatch.reconcilePending) {
         repoWatch.reconcilePending = false;
         await this.reconcileWatchTree(repoRoot, repoRoot);
       }
+
       if (this.repoWatches.get(repoRoot) !== repoWatch) return;
+
       this.onRepoChange(repoRoot);
     }, DIFF_REFRESH_DEBOUNCE_MS);
   }
@@ -318,7 +349,9 @@ export class DiffWatcher {
   close(): void {
     for (const repoWatch of this.repoWatches.values()) {
       if (repoWatch.debounce !== null) clearTimeout(repoWatch.debounce);
+
       if (repoWatch.jjPoll !== null) clearInterval(repoWatch.jjPoll);
+
       for (const handle of repoWatch.handles) handle.close();
     }
     this.repoWatches.clear();

package/src/examples/daemon-sdk.ts

@@ -28,6 +28,7 @@ const reviewConversation = Effect.gen(function* () {
 
   if (completed.outcome !== "completed")
     throw new Error(`SDK example agent finished: ${completed.outcome}`);
+
   const comments = yield* sdk.comments.list(thread.id);
 
   if (comments[0])
@@ -36,6 +37,7 @@ const reviewConversation = Effect.gen(function* () {
       commentId: comments[0].id,
       body: "The explanation is clear.",
     });
+
   yield* sdk.sessions.sendMessage({
     threadId: thread.id,
     operationId: sdk.ids.operation(randomUUID()),

package/src/gh.ts

@@ -26,6 +26,7 @@ export async function prRefs(pr: string): Promise<PullRequestRefs | null> {
   const { code, stdout } = await gh(["pr", "view", pr, "--json", "baseRefOid,headRefOid"]);
 
   if (code !== 0) return null;
+
   let input: unknown;
 
   try {
@@ -60,12 +61,15 @@ export async function prSnapshot(
     const before = await readRefs(pr);
 
     if (before === null) return null;
+
     const patch = await readDiff(pr);
 
     if (patch === null) return null;
+
     const after = await readRefs(pr);
 
     if (after === null) return null;
+
     if (before.baseSha === after.baseSha && before.headSha === after.headSha)
       return { patch, ...after };
   }

package/src/ghostty-thread-surface-store.ts

@@ -65,6 +65,7 @@ export class GhosttyThreadSurfaceStore {
     const previous = this.surfaces.get(threadId);
 
     if (previous?.opening) return false;
+
     this.surfaces.set(threadId, { ...previous, opening: true });
 
     try {
@@ -83,8 +84,10 @@ export class GhosttyThreadSurfaceStore {
     const record = this.surfaces.get(threadId);
 
     if (!record?.opening) return;
+
     if (record.terminalId) this.surfaces.set(threadId, { terminalId: record.terminalId });
     else this.surfaces.delete(threadId);
+
     this.persist();
   }
 

package/src/ghostty-thread-surface.test.ts

@@ -54,6 +54,7 @@ function persistence(initial: GhosttyThreadSurfaceHandle | null = null) {
   const port: GhosttyThreadSurfacePersistence = {
     ghosttyClaimThreadSurface: async () => {
       if (claimed) return false;
+
       claimed = true;
 
       return true;
@@ -316,6 +317,7 @@ describe("openGhosttyThreadSurface", () => {
     const store: GhosttyThreadSurfacePersistence = {
       ghosttyClaimThreadSurface: async () => {
         if (claimed) return false;
+
         claimed = true;
 
         return true;

package/src/ghostty-thread-surface.ts

@@ -139,10 +139,12 @@ export function resolveCueloopLaunchCommand(
 
   for (const directory of path?.split(delimiter) ?? []) {
     if (!isAbsolute(directory)) continue;
+
     const candidate = join(directory, "cueloop");
 
     try {
       accessSync(candidate, constants.X_OK);
+
       if (statSync(candidate).isFile()) return shellQuote(candidate);
     } catch {
       // A PATH entry without an executable cannot launch the Thread.
@@ -161,6 +163,7 @@ export function supportsGhosttyAppleScript(version: string | null): boolean {
   const parsed = version?.match(/^(\d+)\.(\d+)\.(\d+)/);
 
   if (!parsed) return false;
+
   const major = Number(parsed[1]);
   const minor = Number(parsed[2]);
 
@@ -183,7 +186,9 @@ export async function openGhosttyThreadSurface(
   cueloopBinPath?: string | null,
 ): Promise<ThreadSurfaceOpenStatus> {
   if (platform !== "darwin" || !insideGhostty(env)) return "unavailable";
+
   if (mode === "none") return "disabled";
+
   if (!supportsGhosttyAppleScript(runAppleScript(binPath, VERSION_SCRIPT))) return "failed";
 
   try {
@@ -236,6 +241,7 @@ async function openClaimedGhosttyThreadSurface(
     const result = runAppleScript(binPath, FOCUS_SCRIPT, [recorded.terminalId]);
 
     if (result === "focused") return { status: "focused", retainClaim: false };
+
     if (result !== "closed") return { status: "failed", retainClaim: false };
   }
 

package/src/harness-diagnostics.ts

@@ -20,6 +20,7 @@ export function writeHarnessDiagnostic(
 
   try {
     mkdirSync(directory, { recursive: true, mode: 0o700 });
+
     if (existsSync(path) && statSync(path).size + Buffer.byteLength(record) > 256 * 1024)
       writeFileSync(path, record, { mode: 0o600 });
     else appendFileSync(path, record, { mode: 0o600 });