@tablation/crew 0.117.0 → 0.118.0

package/dist/cli.js

@@ -6932,6 +6932,69 @@ var PERISCOPE_COLUMNS = [
   "expires_at"
 ];
 
+// src/tool-classify.ts
+var INTEL_SERVER_NAME2 = "code";
+var SEARCH_TOOLS = /* @__PURE__ */ new Set([
+  "Grep",
+  "Glob",
+  "LS",
+  // Claude
+  "search_file_content",
+  "glob",
+  "list_directory",
+  // Gemini
+  "grep_search",
+  "file_search"
+]);
+var EDIT_TOOLS = /* @__PURE__ */ new Set([
+  "Edit",
+  "Write",
+  "MultiEdit",
+  "NotebookEdit",
+  // Claude
+  "file_change",
+  // Codex
+  "replace",
+  "write_file"
+  // Gemini
+]);
+var SHELL_TOOLS = /* @__PURE__ */ new Set(["Bash", "command", "run_shell_command"]);
+var SEARCH_COMMANDS = /* @__PURE__ */ new Set(["grep", "egrep", "fgrep", "rg", "find", "ls", "fd", "ag"]);
+function classifyTool(tool, target) {
+  if (!tool) return "other";
+  if (tool.startsWith(`mcp__${INTEL_SERVER_NAME2}__`)) return "symbol";
+  if (EDIT_TOOLS.has(tool)) return "edit";
+  if (SEARCH_TOOLS.has(tool)) return "search";
+  if (SHELL_TOOLS.has(tool)) {
+    const first = (target ?? "").split("/").pop();
+    return first && SEARCH_COMMANDS.has(first) ? "search" : "other";
+  }
+  return "other";
+}
+var BeforeFirstEditTracker = class {
+  calls = { search: 0, symbol: 0, other: 0 };
+  edited = false;
+  seconds;
+  /** `elapsedMs` is the time since session start, when known (not on a replayed stream). */
+  note(tool, target, elapsedMs) {
+    if (this.edited) return;
+    const kind = classifyTool(tool, target);
+    if (kind === "edit") {
+      this.edited = true;
+      if (elapsedMs !== void 0) this.seconds = Math.max(0, Math.round(elapsedMs / 1e3));
+    } else {
+      this.calls[kind]++;
+    }
+  }
+  result() {
+    return {
+      calls: { ...this.calls },
+      edited: this.edited,
+      ...this.seconds !== void 0 ? { secondsToFirstEdit: this.seconds } : {}
+    };
+  }
+};
+
 // src/agent-log.ts
 import { existsSync as existsSync17, mkdirSync as mkdirSync12, readdirSync as readdirSync6, readFileSync as readFileSync17, rmSync as rmSync10, writeFileSync as writeFileSync13 } from "fs";
 import { join as join18 } from "path";
@@ -7006,6 +7069,36 @@ function counterIncrements(run3, cols, have) {
   }
   return out;
 }
+var BEFORE_EDIT_COLUMNS = {
+  search: "calls_before_edit_search",
+  symbol: "calls_before_edit_symbol",
+  other: "calls_before_edit_other",
+  seconds: "seconds_to_first_edit",
+  edited: "edited"
+};
+function beforeFirstEditBody(b) {
+  return {
+    [BEFORE_EDIT_COLUMNS.search]: b.calls.search,
+    [BEFORE_EDIT_COLUMNS.symbol]: b.calls.symbol,
+    [BEFORE_EDIT_COLUMNS.other]: b.calls.other,
+    [BEFORE_EDIT_COLUMNS.edited]: b.edited,
+    ...b.secondsToFirstEdit !== void 0 ? { [BEFORE_EDIT_COLUMNS.seconds]: b.secondsToFirstEdit } : {}
+  };
+}
+async function writeBeforeFirstEdit(target, rowId, b) {
+  try {
+    const model = await request(
+      target,
+      "GET",
+      `/data-models/agent_log?workspaceId=${encodeURIComponent(target.workspaceId)}`
+    );
+    const have = await columnsOf(target, model.id);
+    const body = Object.fromEntries(Object.entries(beforeFirstEditBody(b)).filter(([column]) => have.has(column)));
+    if (Object.keys(body).length === 0) return;
+    await request(target, "PATCH", `/data-models/${model.id}/records/${rowId}`, body);
+  } catch {
+  }
+}
 async function post(target, path, body) {
   const url = `${target.baseUrl.replace(/\/+$/, "")}/api/workspaces/${target.workspaceId}${path}`;
   const res = await fetch(url, {
@@ -7074,6 +7167,7 @@ async function reportAgentRun(target, run3) {
       ...cycle.role ? { role: cycle.role } : {}
     });
   }
+  if (run3.beforeFirstEdit) await writeBeforeFirstEdit(target, entry.id, run3.beforeFirstEdit);
   const counters = target.tickets && ticketRecordId ? await addRunToTicket(target, target.tickets, ticketRecordId, run3) : void 0;
   return {
     agentLogId: entry.id,
@@ -10588,6 +10682,8 @@ var RunObserver = class {
   result;
   /** Prefix tokens of the first assistant call that carried usage; kept once. */
   prefixTokens;
+  /** Tool calls before the run's first edit (CREW-2879). */
+  beforeFirstEdit = new BeforeFirstEditTracker();
   /** Set once the stream carries the harness's final result (turn count or cost). */
   sawResult = false;
   muted = false;
@@ -10599,6 +10695,7 @@ var RunObserver = class {
   /** Called after `liveTicket` changes, so the sidecar can be rewritten. */
   onClaim;
   buf = "";
+  startMs = Date.now();
   mapper;
   id;
   adapter;
@@ -10654,6 +10751,7 @@ var RunObserver = class {
     for (const ev of events) if ((ev.kind === "thought" || ev.kind === "text") && ev.text) this.attribution.note(ev.text);
     for (const ev of events) {
       if (ev.kind === "tool") this.attribution.noteWrite(ev.tool, ev.input);
+      if (ev.kind === "tool") this.beforeFirstEdit.note(ev.tool, ev.target, this.muted ? void 0 : Date.now() - this.startMs);
       if (ev.kind === "tool" && isClaimWrite(ev.tool, ev.input)) {
         const claimed = this.attribution.current;
         if (claimed && claimed !== this.liveTicket) {
@@ -10768,6 +10866,7 @@ async function finishRun(id, obs, emit, startedAt, end, agentLogKey, hintMoved)
       emit.warn(`could not save the pending hand-off: ${e.message}`, { step: "agent" });
     }
   }
+  const before = obs.beforeFirstEdit.result();
   try {
     writeRunMeta(id, startedAt, obs.liveTicket, {
       attributedTicket: attributedTicket ?? null,
@@ -10779,6 +10878,7 @@ async function finishRun(id, obs, emit, startedAt, end, agentLogKey, hintMoved)
       sessionId: result?.sessionId ?? null,
       numTurns: result?.numTurns ?? null,
       prefixTokens: obs.prefixTokens ?? null,
+      beforeFirstEdit: before,
       totalCostUsd: result?.costUsd ?? null
     });
   } catch {
@@ -10794,6 +10894,9 @@ async function finishRun(id, obs, emit, startedAt, end, agentLogKey, hintMoved)
       attributedTicket: attributedTicket ?? null,
       attributedBy,
       ...result?.sessionId ? { sessionId: result.sessionId } : {},
+      callsBeforeFirstEdit: before.calls,
+      edited: before.edited,
+      ...before.secondsToFirstEdit !== void 0 ? { secondsToFirstEdit: before.secondsToFirstEdit } : {},
       ...result?.numTurns !== void 0 ? { numTurns: result.numTurns } : {},
       ...result?.costUsd !== void 0 ? { totalCostUsd: result.costUsd } : {}
     }
@@ -10814,6 +10917,7 @@ async function finishRun(id, obs, emit, startedAt, end, agentLogKey, hintMoved)
       // Both come from a finished run: without the result line (interrupted) neither is trustworthy.
       ...result?.numTurns !== void 0 ? { numTurns: result.numTurns } : {},
       ...result?.numTurns !== void 0 && obs.prefixTokens !== void 0 ? { prefixTokens: obs.prefixTokens } : {},
+      beforeFirstEdit: before,
       harness: identity.harness,
       provider: identity.provider,
       promptVersion: id.promptVersion,
@@ -19591,7 +19695,20 @@ async function listLogEntries(route2, opts = {}) {
     limit: opts.limit ?? 20,
     ...filters.length > 0 ? { filters: JSON.stringify(filters) } : {}
   });
-  return [...entries].sort((a, b) => String(b.started_at ?? "").localeCompare(String(a.started_at ?? "")));
+  const newestFirst = [...entries].sort((a, b) => String(b.started_at ?? "").localeCompare(String(a.started_at ?? "")));
+  if (!opts.beforeEdit) return newestFirst;
+  const searches = (e) => e.calls_before_edit_search ?? -1;
+  return newestFirst.sort((a, b) => searches(b) - searches(a));
+}
+function formatBeforeEdit(e) {
+  if (e.calls_before_edit_search == null && e.calls_before_edit_symbol == null && e.calls_before_edit_other == null) return void 0;
+  const parts = [
+    `search ${e.calls_before_edit_search ?? 0}`,
+    `symbol ${e.calls_before_edit_symbol ?? 0}`,
+    `other ${e.calls_before_edit_other ?? 0}`
+  ];
+  const when = e.edited === false ? "no edit" : e.seconds_to_first_edit != null ? `${e.seconds_to_first_edit}s to first edit` : void 0;
+  return `${parts.join(", ")}${when ? ` (${when})` : ""}`;
 }
 async function showLogEntry(route2, entryId, opts = {}) {
   requireWorkspace(route2);
@@ -23526,7 +23643,7 @@ function usage(problem, requested = false) {
   crew repos clone [ROUTE] NAME  clone a Repos row's remote into the route directory and attach it (--dir PATH, --dry-run, --json)
   crew repos hooks [ROUTE] NAME suggest test/build hooks for a checkout (--agent, --write route|repo, --recipes, --json)
   crew repos list ROUTE         the checkouts a route has, with the tracker's Repos row each matches (--json: every Repos row too)
-  crew logbook list [route]     recent Agent Log entries, filterable by --role/--ticket
+  crew logbook list [route]     recent Agent Log entries, filterable by --role/--ticket; --before-edit sorts by searches before the first edit
   crew logbook show [route] ID [--prompt]   one entry; --prompt reconstructs and verifies its prompt
   crew passengers [route]       sync Host Passengers containers/tunnels once; what the passengers unit invokes
   crew install                  write and load this platform's scheduler unit
@@ -26179,7 +26296,7 @@ ${cu.tail.trimEnd()}
       const sub = positional[1];
       if (sub !== "list" && sub !== "show") {
         process.stderr.write(
-          "crew logbook list [route] [--role R] [--ticket T] [--limit N]\ncrew logbook show [route] <entryId> [--prompt]\n"
+          "crew logbook list [route] [--role R] [--ticket T] [--limit N] [--before-edit]\ncrew logbook show [route] <entryId> [--prompt]\n"
         );
         process.exit(2);
       }
@@ -26206,7 +26323,8 @@ ${cu.tail.trimEnd()}
             userAgent: cfg.ship.userAgent,
             role: value("role"),
             ticket: value("ticket"),
-            limit: value("limit") ? Number(value("limit")) : void 0
+            limit: value("limit") ? Number(value("limit")) : void 0,
+            beforeEdit: flag("before-edit")
           });
           if (entries.length === 0) {
             process.stdout.write("(no Agent Log entries match)\n");
@@ -26214,7 +26332,7 @@ ${cu.tail.trimEnd()}
           }
           for (const e of entries) {
             process.stdout.write(
-              `${e.id}  ${e.started_at ?? "?"}  ${e.outcome ?? "?"}${e.ticket_reference ? `  ${e.ticket_reference}` : ""}${e.model ? `  ${e.model}` : ""}${e.harness || e.provider ? `  (${[e.harness, e.provider].filter(Boolean).join(" \xB7 ")})` : ""}${e.num_turns != null ? `  turns ${e.num_turns}` : ""}${e.prefix_tokens != null ? `  prefix ${e.prefix_tokens}` : ""}
+              `${e.id}  ${e.started_at ?? "?"}  ${e.outcome ?? "?"}${e.ticket_reference ? `  ${e.ticket_reference}` : ""}${e.model ? `  ${e.model}` : ""}${e.harness || e.provider ? `  (${[e.harness, e.provider].filter(Boolean).join(" \xB7 ")})` : ""}${e.num_turns != null ? `  turns ${e.num_turns}` : ""}${e.prefix_tokens != null ? `  prefix ${e.prefix_tokens}` : ""}${formatBeforeEdit(e) ? `  before edit: ${formatBeforeEdit(e)}` : ""}
 `
             );
           }
@@ -26234,6 +26352,7 @@ harness:    ${entry.harness ?? "?"}
 provider:   ${entry.provider ?? "?"}
 ` + (entry.num_turns != null ? `turns:      ${entry.num_turns}
 ` : "") + (entry.prefix_tokens != null ? `prefix:     ${entry.prefix_tokens} tokens
+` : "") + (formatBeforeEdit(entry) ? `before edit: ${formatBeforeEdit(entry)}
 ` : "")
           );
           if (flag("prompt")) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.117.0",
+  "version": "0.118.0",
   "description": "A standing team of headless agents that picks work off a Tablation board, does it on a machine you control, and reports back.",
   "type": "module",
   "license": "MIT",

package/prompts/default/personas/lane-pair.md

@@ -93,6 +93,11 @@ available only in a repository whose hooks table declares `preview`.
   the URL. Tell them where it is once, early: the URL always, and, when you
   are the operator's own session on this ship (not a Passenger's chat), the
   worktree path as well, so they can open the files directly.
+- **When the person refers to something on the page** ("this", "here",
+  "that heading"), read the newest line of `.crew/pointed-at.jsonl` in the
+  periscope's worktree before searching; it names the file and line they
+  clicked. If the file is empty, ask them to point at it with the inspector
+  or paste the location.
 - **When a dev lane is mid-flight on the same files** (visible from the
   roster and the board), say so, so the person knows a merge is coming.
 - **Before keep,** read back the ticket title and a one-paragraph summary of

package/prompts/default/skills/crew-lane-agent.md

@@ -424,6 +424,26 @@ before going further, and never change a ticket's `report_type` yourself.
    sets the ticket back to `in_progress` instead of passing it to QA. A ticket
    with no Acceptance section is unaffected.
 
+### Finding where something lives
+
+Before you read or edit, locate the code once, in this order:
+
+1. **The map.** Read the "Repository map" section of this brief (when the
+   repository has one) and the repository's own instructions file, which may
+   also say where things live.
+2. **The symbol tools.** When the Environment section lists code tools, use
+   them for code questions before searching text: find a symbol by name, list
+   its references, read its definition, outline a file. Text search is for
+   strings that are not symbols.
+3. **One broad search.** To find a visible string, search for it once across
+   the whole relevant tree, excluding tests, and prefer registries over
+   components. A search that found only a test file is widened, not
+   narrowed.
+
+Count the calls: if more than three searches have not found the location, say
+so in your progress comment and ask for the map or the person's pointer
+rather than continuing to search.
+
 ## Step 4 — you never merge, and you never deploy
 
 **Nothing you do touches the base branch.** The release phase takes every

package/prompts/dev-qa/personas/lane-pair.md

@@ -93,6 +93,11 @@ available only in a repository whose hooks table declares `preview`.
   the URL. Tell them where it is once, early: the URL always, and, when you
   are the operator's own session on this ship (not a Passenger's chat), the
   worktree path as well, so they can open the files directly.
+- **When the person refers to something on the page** ("this", "here",
+  "that heading"), read the newest line of `.crew/pointed-at.jsonl` in the
+  periscope's worktree before searching; it names the file and line they
+  clicked. If the file is empty, ask them to point at it with the inspector
+  or paste the location.
 - **When a dev lane is mid-flight on the same files** (visible from the
   roster and the board), say so, so the person knows a merge is coming.
 - **Before keep,** read back the ticket title and a one-paragraph summary of

package/prompts/dev-qa/skills/crew-lane-agent.md

@@ -419,6 +419,26 @@ before going further, and never change a ticket's `report_type` yourself.
    sets the ticket back to `in_progress` instead of passing it to QA. A ticket
    with no Acceptance section is unaffected.
 
+### Finding where something lives
+
+Before you read or edit, locate the code once, in this order:
+
+1. **The map.** Read the "Repository map" section of this brief (when the
+   repository has one) and the repository's own instructions file, which may
+   also say where things live.
+2. **The symbol tools.** When the Environment section lists code tools, use
+   them for code questions before searching text: find a symbol by name, list
+   its references, read its definition, outline a file. Text search is for
+   strings that are not symbols.
+3. **One broad search.** To find a visible string, search for it once across
+   the whole relevant tree, excluding tests, and prefer registries over
+   components. A search that found only a test file is widened, not
+   narrowed.
+
+Count the calls: if more than three searches have not found the location, say
+so in your progress comment and ask for the map or the person's pointer
+rather than continuing to search.
+
 ## Step 4 — you never merge, and you never deploy
 
 **Nothing you do touches the base branch.** The release phase takes every