fix: address Bugbot review — Windows paths, bytesRead, hardcoded paths

- toClaudeProjectPath: handle Windows backslashes and drive letters
- readLastJsonlEntry: use bytesRead to avoid null character corruption
- Remove duplicate next-env.d.ts eslint ignore entry
- Remove hardcoded developer-specific binary paths from integration tests
- Add isProcessing limitation comments with issue references (#17-19)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

eslint.config.js

@@ -13,7 +13,6 @@ export default tseslint.config(
       "packages/web/next-env.d.ts",
       "packages/web/next.config.js",
       "packages/web/postcss.config.mjs",
-      "packages/web/next-env.d.ts",
     ],
   },
 

packages/integration-tests/src/agent-aider.integration.test.ts

@@ -28,10 +28,9 @@ const execFileAsync = promisify(execFile);
 // ---------------------------------------------------------------------------
 
 const SESSION_PREFIX = "ao-inttest-aider-";
-const AIDER_BINARY = "/Users/equinox/Library/Python/3.9/bin/aider";
 
 async function findAiderBinary(): Promise<string | null> {
-  for (const bin of ["aider", AIDER_BINARY]) {
+  for (const bin of ["aider"]) {
     try {
       await execFileAsync("which", [bin], { timeout: 5_000 });
       return bin;

packages/integration-tests/src/agent-claude-code.integration.test.ts

@@ -28,10 +28,9 @@ const execFileAsync = promisify(execFile);
 // ---------------------------------------------------------------------------
 
 const SESSION_PREFIX = "ao-inttest-claude-";
-const CLAUDE_BINARY = "/Users/equinox/.local/bin/claude";
 
 async function findClaudeBinary(): Promise<string | null> {
-  for (const bin of ["claude", CLAUDE_BINARY]) {
+  for (const bin of ["claude"]) {
     try {
       await execFileAsync("which", [bin], { timeout: 5_000 });
       return bin;

packages/integration-tests/src/agent-codex.integration.test.ts

@@ -28,10 +28,9 @@ const execFileAsync = promisify(execFile);
 // ---------------------------------------------------------------------------
 
 const SESSION_PREFIX = "ao-inttest-codex-";
-const CODEX_BINARY = "/opt/homebrew/bin/codex";
 
 async function findCodexBinary(): Promise<string | null> {
-  for (const bin of ["codex", CODEX_BINARY]) {
+  for (const bin of ["codex"]) {
     try {
       await execFileAsync("which", [bin], { timeout: 5_000 });
       return bin;

packages/plugins/agent-aider/src/index.ts

@@ -122,6 +122,8 @@ function createAiderAgent(): Agent {
       }
     },
 
+    // NOTE: Aider lacks introspection to distinguish "processing" from "idle at prompt".
+    // Falling back to process liveness until richer detection is implemented (see #18).
     async isProcessing(session: Session): Promise<boolean> {
       if (!session.runtimeHandle) return false;
       return this.isProcessRunning(session.runtimeHandle);

packages/plugins/agent-claude-code/src/index.ts

@@ -45,7 +45,9 @@ export const manifest = {
  * the resulting directory exists.
  */
 function toClaudeProjectPath(workspacePath: string): string {
-  return workspacePath.replace(/^\//, "").replace(/[/.]/g, "-");
+  // Handle Windows drive letters (C:\Users\... → C-Users-...)
+  const normalized = workspacePath.replace(/\\/g, "/");
+  return normalized.replace(/^\//, "").replace(/:/g, "").replace(/[/.]/g, "-");
 }
 
 /** Find the most recently modified .jsonl session file in a directory */
@@ -112,9 +114,9 @@ async function readLastJsonlEntry(
 
     const readSize = Math.min(TAIL_READ_BYTES, size);
     const buffer = Buffer.alloc(readSize);
-    await fh.read(buffer, 0, readSize, size - readSize);
+    const { bytesRead } = await fh.read(buffer, 0, readSize, size - readSize);
 
-    const chunk = buffer.toString("utf-8");
+    const chunk = buffer.toString("utf-8", 0, bytesRead);
     // Walk backwards through lines to find the last valid JSON object with a type
     const lines = chunk.split("\n");
     for (let i = lines.length - 1; i >= 0; i--) {

packages/plugins/agent-codex/src/index.ts

@@ -126,6 +126,8 @@ function createCodexAgent(): Agent {
       }
     },
 
+    // NOTE: Codex lacks introspection to distinguish "processing" from "idle at prompt".
+    // Falling back to process liveness until richer detection is implemented (see #17).
     async isProcessing(session: Session): Promise<boolean> {
       if (!session.runtimeHandle) return false;
       return this.isProcessRunning(session.runtimeHandle);

packages/plugins/agent-opencode/src/index.ts

@@ -118,6 +118,8 @@ function createOpenCodeAgent(): Agent {
       }
     },
 
+    // NOTE: OpenCode lacks introspection to distinguish "processing" from "idle at prompt".
+    // Falling back to process liveness until richer detection is implemented (see #19).
     async isProcessing(session: Session): Promise<boolean> {
       if (!session.runtimeHandle) return false;
       return this.isProcessRunning(session.runtimeHandle);