@brainervirus/workit-core 2.4.0 → 2.6.0

package/src/hooks/handle.ts

@@ -12,6 +12,20 @@ const NONE: HookDecision = { kind: "none" };
 /** Sessions already offered unfinished tasks in this process. */
 const offered = new Set<string>();
 
+/** The Workit plugin's worktree-isolated implementer is the one Claude Code
+ * subagent that writes. Claude namespaces plugin agents, so only the exact
+ * `workit:implementer` is ours: a bare or other-plugin `implementer` is not. */
+const CLAUDE_WORKTREE_IMPLEMENTER = "workit:implementer";
+
+const subagentStartText = (
+  host: HookInput["host"],
+  descriptor: HostDescriptor,
+  event: Extract<HookInput["event"], { kind: "subagent.start" }>,
+): string =>
+  host === "claude_code" && event.agentType === CLAUDE_WORKTREE_IMPLEMENTER
+    ? `Workit observed ${descriptor.label} subagent ${event.agentId} (${event.agentType}) working in its own git worktree: it may edit and commit there, within its brief's scope. Before the first commit, switch to a policy-compliant branch (\`git switch -c <type>/<slug>\`, e.g. feature/<slug>); branch policy hooks still deny protected or non-compliant branches. Never push, open a PR, or merge unless the brief asks for it.`
+    : `Workit observed ${descriptor.label} subagent ${event.agentId} (${event.agentType}) as read-only/agent-guided; writer delegation is unavailable.`;
+
 export function handleHook(input: HookInput, deps: HookDeps): HookDecision {
   const { descriptor } = deps;
   const event = input.event;
@@ -36,10 +50,7 @@ export function handleHook(input: HookInput, deps: HookDeps): HookDecision {
     case "shell.pre":
       return usable(descriptor.shellPolicy.deny) ? shellPolicy(input.cwd, event.command) : NONE;
     case "subagent.start":
-      return {
-        kind: "context",
-        text: `Workit observed ${descriptor.label} subagent ${event.agentId} (${event.agentType}) as read-only/agent-guided; writer delegation is unavailable.`,
-      };
+      return { kind: "context", text: subagentStartText(input.host, descriptor, event) };
     case "compact.pre":
       return {
         kind: "notice",

package/src/ledger.ts

@@ -62,7 +62,7 @@ export type VerdictSurface = (typeof VERDICT_SURFACES)[number];
 export type Observer = "agent_asserted" | "workit_cli";
 
 /** Row types written by the CLI's own PR verbs (S10/S11); the only PR→branch source. */
-export const PR_ROW_TYPES: ReadonlySet<string> = new Set(["pr.created", "pr.status"]);
+export const PR_ROW_TYPES: ReadonlySet<string> = new Set(["pr.created", "pr.status", "pr.merged"]);
 
 /** Who wrote a row. Unverified: the environment says so (D18). */
 export type LedgerActor = { host: string; session: string | null; agentId: string | null };
@@ -1156,6 +1156,22 @@ export function summarizeRow(row: ReadRow): RowSummary {
     case "handoff":
       summary = `next: ${text("next")}${text("note") ? ` (note: ${text("note")})` : ""}`;
       break;
+    case "commit.recorded":
+      summary = `${(row.head ?? "?").slice(0, 12)} ${text("subject")}${row.actor.session ? "" : " (no session)"}`;
+      break;
+    case "push.verified":
+    case "push.noop":
+      summary = `${text("remote")} ${(str(row.previous) ?? "(new)").slice(0, 12)} -> ${(row.head ?? "?").slice(0, 12)}${row.forced === true ? " (force-with-lease)" : ""}`;
+      break;
+    case "pr.created":
+      summary = `#${row.pr ?? "?"} -> ${text("base")} at ${(row.head ?? "?").slice(0, 12)}${row.created === false ? " (existing)" : ""}`;
+      break;
+    case "pr.merged":
+      summary = `#${row.pr ?? "?"} ${text("method")} at ${(row.head ?? "?").slice(0, 12)}${text("mergeSha") ? ` -> ${text("mergeSha").slice(0, 12)}` : ""}`;
+      break;
+    case "delivery.verified":
+      summary = `${text("expect")} delivered at ${(row.head ?? "?").slice(0, 12)}`;
+      break;
     default:
       summary = text("what") || text("name") || row.type;
   }