@blastin-dev/clocktopus-cli 0.2.1 → 0.3.1

package/dist/src/lib/git-remotes.js

@@ -0,0 +1,50 @@
+// Every remote a checkout has, for attributing the session to a repository.
+//
+// `origin` is a naming convention, not evidence. On a fork it names the developer's
+// own copy while the webhooks — and so every commit row — arrive under the upstream,
+// which left those sessions attributed to a repository nothing else in the system
+// ever mentions (BLA-598). The receiver decides which one wins; the hook's job is to
+// report them all.
+/** Ceiling matching `AgentSessionHookSchema`. A checkout with more is not a real one. */
+const MAX_REMOTES = 20;
+/**
+ * Parses `git remote -v` into a deduplicated URL list, `origin` first.
+ *
+ * Order is load-bearing in one narrow way: the first entry is what a receiver older
+ * than BLA-598 reads as the only remote, so keeping `origin` there preserves the
+ * previous behaviour exactly.
+ */
+export function parseGitRemotes(output) {
+    if (!output?.trim())
+        return [];
+    const byName = new Map();
+    for (const line of output.split("\n")) {
+        // `name\turl (fetch|push)` — push URLs can differ from fetch ones, and both are
+        // worth reporting: a fork often fetches upstream and pushes to itself.
+        const match = line.trim().match(/^(\S+)\s+(\S+)\s+\((fetch|push)\)$/);
+        if (!match)
+            continue;
+        const [, name, url, direction] = match;
+        if (!name || !url)
+            continue;
+        const key = `${name}:${direction}`;
+        if (!byName.has(key))
+            byName.set(key, url);
+    }
+    const entries = [...byName.entries()];
+    const ordered = [
+        ...entries.filter(([key]) => key.startsWith("origin:")),
+        ...entries.filter(([key]) => !key.startsWith("origin:")),
+    ];
+    const seen = new Set();
+    const urls = [];
+    for (const [, url] of ordered) {
+        if (seen.has(url))
+            continue;
+        seen.add(url);
+        urls.push(url);
+        if (urls.length === MAX_REMOTES)
+            break;
+    }
+    return urls;
+}

package/dist/src/lib/git-remotes.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=git-remotes.test.d.ts.map

package/dist/src/lib/git-remotes.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"git-remotes.test.d.ts","sourceRoot":"","sources":["../../../src/lib/git-remotes.test.ts"],"names":[],"mappings":""}

package/dist/src/lib/git-remotes.test.js

@@ -0,0 +1,52 @@
+import { describe, expect, it } from "vitest";
+import { parseGitRemotes } from "./git-remotes.js";
+describe("parseGitRemotes", () => {
+    it("keeps origin first, so an older receiver reading only the first entry is unchanged", () => {
+        const urls = parseGitRemotes([
+            "upstream\tgit@github.com:acme-co/portal.git (fetch)",
+            "upstream\tgit@github.com:acme-co/portal.git (push)",
+            "origin\tgit@github.com:jordan/portal.git (fetch)",
+            "origin\tgit@github.com:jordan/portal.git (push)",
+        ].join("\n"));
+        expect(urls[0]).toBe("git@github.com:jordan/portal.git");
+        expect(urls).toContain("git@github.com:acme-co/portal.git");
+    });
+    it("collapses the fetch and push lines of one remote", () => {
+        const urls = parseGitRemotes([
+            "origin\tgit@github.com:acme-co/portal.git (fetch)",
+            "origin\tgit@github.com:acme-co/portal.git (push)",
+        ].join("\n"));
+        expect(urls).toEqual(["git@github.com:acme-co/portal.git"]);
+    });
+    /** A fork often fetches from upstream and pushes to itself under one remote name. */
+    it("keeps both URLs when a remote's push target differs from its fetch", () => {
+        const urls = parseGitRemotes([
+            "origin\tgit@github.com:acme-co/portal.git (fetch)",
+            "origin\tgit@github.com:jordan/portal.git (push)",
+        ].join("\n"));
+        expect(urls).toEqual([
+            "git@github.com:acme-co/portal.git",
+            "git@github.com:jordan/portal.git",
+        ]);
+    });
+    it("returns nothing for a checkout with no remotes", () => {
+        expect(parseGitRemotes("")).toEqual([]);
+        expect(parseGitRemotes(undefined)).toEqual([]);
+    });
+    it("works when no remote is called origin", () => {
+        const urls = parseGitRemotes("github\tgit@github.com:acme-co/portal.git (fetch)");
+        expect(urls).toEqual(["git@github.com:acme-co/portal.git"]);
+    });
+    it("ignores lines that are not remote entries", () => {
+        const urls = parseGitRemotes([
+            "origin\tgit@github.com:acme-co/portal.git (fetch)",
+            "warning: something else entirely",
+            "",
+        ].join("\n"));
+        expect(urls).toEqual(["git@github.com:acme-co/portal.git"]);
+    });
+    it("caps a pathological remote list", () => {
+        const lines = Array.from({ length: 40 }, (_, i) => `r${i}\tgit@github.com:acme-co/repo-${i}.git (fetch)`);
+        expect(parseGitRemotes(lines.join("\n"))).toHaveLength(20);
+    });
+});

package/dist/src/lib/opencode-config.d.ts

@@ -1,104 +1,19 @@
-/**
- * Installs and reads back the OpenCode plugin that reports to Clocktopus.
- *
- * OpenCode is the odd one out. Claude Code and Codex both expose a *hook*
- * — a command they run with JSON on stdin — so wiring them up is a matter
- * of editing config. OpenCode instead exposes a plugin API: JavaScript
- * loaded into its own process, handed a stream of typed events. So the
- * integration is a file of ours rather than a config entry pointing at the
- * CLI.
- *
- * The install is a single file. OpenCode auto-loads everything in
- * `<config dir>/plugin/`, verified against 1.18.18, so nothing has to be
- * added to `opencode.json` — which is worth having: that file is the
- * user's model, provider and permission configuration, and not touching it
- * removes a whole class of ways to break their setup. Uninstalling is
- * deleting the file.
- *
- * ## Why a plugin and not OpenCode's own OpenTelemetry
- *
- * OpenCode has `experimental.openTelemetry`, which exports OTLP/JSON traces
- * to `OTEL_EXPORTER_OTLP_ENDPOINT` — on the face of it exactly what we
- * want, and a one-line install. It was measured and rejected:
- *
- * - **It ships the conversation.** The AI SDK spans carry `ai.prompt`,
- *   `ai.prompt.messages` and `ai.response.text` — the system prompt, every
- *   user message and the model's replies, verbatim, with no switch to turn
- *   them off. Codex's log stream leaks tool output; this leaks everything.
- * - **It is enormous.** One prompt produced 230KB, almost all of it
- *   OpenCode's internal spans — SQLite queries, file reads, lock
- *   acquisitions — with the AI spans a rounding error inside it.
- * - **It carries no cost and no repository**, so it could not do the one
- *   job the feature exists for.
- *
- * The plugin sees a better source than the traces do: `AssistantMessage`
- * carries `cost`, a full token breakdown, the model, and `path.cwd`. It
- * sends numbers and identifiers, and nothing a person wrote.
- */
 export declare const OPENCODE_PLUGIN_FILENAME = "clocktopus.js";
-/**
- * Bumped whenever `buildOpencodePlugin` changes what it emits.
- *
- * OpenCode is the only agent whose integration is *generated source* rather
- * than a command string in a config file. Claude Code and Codex hold
- * `clocktopus agent hook …`, which means whatever the installed CLI means,
- * so upgrading the CLI upgrades them. This plugin does not: the file on
- * disk stays exactly as the CLI that wrote it left it, and a fix shipped to
- * the plugin body reaches nobody until they re-run `agent setup`.
- *
- * Stamping the generation is what makes that visible — `agent doctor`
- * compares this number against the file and says so when they differ.
- * Nothing auto-rewrites the file: it holds a token, and a command that
- * quietly rewrites credentials is worse than one that tells you to.
- *
- * A plugin written before the stamp existed parses as `null`, which reads
- * as stale — correct, since it predates every version that has one.
- */
 export declare const OPENCODE_PLUGIN_VERSION = 1;
 export declare function opencodeConfigDir(): string;
 export declare function opencodePluginPath(): string;
 export type OpencodePluginConfig = {
     endpoint: string;
     token: string;
-    /**
-     * The hook as argv, not as a command line.
-     *
-     * `resolveHookCommand` returns a shell-quoted string because that is what
-     * `settings.json` and `hooks.json` want — their hosts run it through a
-     * shell. This plugin spawns it directly, so it needs the arguments
-     * already separated: splitting the string on spaces would tear a quoted
-     * path in two the moment anyone installs Node somewhere with a space in
-     * it, and the failure would be silent.
-     */
     hookArgv: string[];
 };
-/**
- * Splits a shell-quoted command line into argv.
- *
- * Only as clever as `resolveHookCommand` is: double quotes around
- * whitespace, nothing else. It is fed that function's output, never a
- * user's shell.
- */
 export declare function splitCommandLine(command: string): string[];
-/**
- * The plugin source, with this machine's configuration baked into one line.
- *
- * Generated rather than shipped as a file because the token has to be in
- * it: OpenCode gives a plugin no way to read our environment, so the
- * credentials live where every other agent's do — in that agent's config,
- * written 0600.
- */
 export declare function buildOpencodePlugin(config: OpencodePluginConfig): string;
 export declare function readOpencodePlugin(path?: string): {
     path: string;
     exists: boolean;
     modifiedAt: Date | null;
     config: OpencodePluginConfig | null;
-    /**
-     * The generation that wrote this file, or null when it carries no stamp —
-     * either a plugin from before stamping existed, or one hand-edited past
-     * recognition. Both mean the same thing to the caller: not current.
-     */
     version: number | null;
 };
 export declare function writeOpencodePlugin(source: string, path?: string): {

package/dist/src/lib/opencode-config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"opencode-config.d.ts","sourceRoot":"","sources":["../../../src/lib/opencode-config.ts"],"names":[],"mappings":"AAaA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAoCG;AAEH,eAAO,MAAM,wBAAwB,kBAAkB,CAAC;AAQxD;;;;;;;;;;;;;;;;;GAiBG;AACH,eAAO,MAAM,uBAAuB,IAAI,CAAC;AAEzC,wBAAgB,iBAAiB,IAAI,MAAM,CAK1C;AAED,wBAAgB,kBAAkB,IAAI,MAAM,CAE3C;AAED,MAAM,MAAM,oBAAoB,GAAG;IACjC,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,MAAM,CAAC;IACd;;;;;;;;;OASG;IACH,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB,CAAC;AAEF;;;;;;GAMG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAI1D;AAED;;;;;;;GAOG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,oBAAoB,GAAG,MAAM,CAqLxE;AAED,wBAAgB,kBAAkB,CAAC,IAAI,SAAuB,GAAG;IAC/D,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,OAAO,CAAC;IAChB,UAAU,EAAE,IAAI,GAAG,IAAI,CAAC;IACxB,MAAM,EAAE,oBAAoB,GAAG,IAAI,CAAC;IACpC;;;;OAIG;IACH,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;CACxB,CA+CA;AAED,wBAAgB,mBAAmB,CACjC,MAAM,EAAE,MAAM,EACd,IAAI,SAAuB,GAC1B;IAAE,UAAU,EAAE,MAAM,GAAG,IAAI,CAAA;CAAE,CAiB/B;AAED,wBAAgB,oBAAoB,CAAC,IAAI,SAAuB,GAAG,OAAO,CAIzE"}
+{"version":3,"file":"opencode-config.d.ts","sourceRoot":"","sources":["../../../src/lib/opencode-config.ts"],"names":[],"mappings":"AAsCA,eAAO,MAAM,wBAAwB,kBAAkB,CAAC;AAoBxD,eAAO,MAAM,uBAAuB,IAAI,CAAC;AAEzC,wBAAgB,iBAAiB,IAAI,MAAM,CAK1C;AAED,wBAAgB,kBAAkB,IAAI,MAAM,CAE3C;AAED,MAAM,MAAM,oBAAoB,GAAG;IACjC,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,MAAM,CAAC;IAOd,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB,CAAC;AAIF,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAI1D;AAMD,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,oBAAoB,GAAG,MAAM,CA0KxE;AAED,wBAAgB,kBAAkB,CAAC,IAAI,SAAuB,GAAG;IAC/D,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,OAAO,CAAC;IAChB,UAAU,EAAE,IAAI,GAAG,IAAI,CAAC;IACxB,MAAM,EAAE,oBAAoB,GAAG,IAAI,CAAC;IAIpC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;CACxB,CA8CA;AAED,wBAAgB,mBAAmB,CACjC,MAAM,EAAE,MAAM,EACd,IAAI,SAAuB,GAC1B;IAAE,UAAU,EAAE,MAAM,GAAG,IAAI,CAAA;CAAE,CAiB/B;AAED,wBAAgB,oBAAoB,CAAC,IAAI,SAAuB,GAAG,OAAO,CAIzE"}

package/dist/src/lib/opencode-config.js

@@ -1,66 +1,47 @@
 import { copyFileSync, existsSync, mkdirSync, readFileSync, renameSync, rmSync, statSync, writeFileSync, } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
-/**
- * Installs and reads back the OpenCode plugin that reports to Clocktopus.
- *
- * OpenCode is the odd one out. Claude Code and Codex both expose a *hook*
- * — a command they run with JSON on stdin — so wiring them up is a matter
- * of editing config. OpenCode instead exposes a plugin API: JavaScript
- * loaded into its own process, handed a stream of typed events. So the
- * integration is a file of ours rather than a config entry pointing at the
- * CLI.
- *
- * The install is a single file. OpenCode auto-loads everything in
- * `<config dir>/plugin/`, verified against 1.18.18, so nothing has to be
- * added to `opencode.json` — which is worth having: that file is the
- * user's model, provider and permission configuration, and not touching it
- * removes a whole class of ways to break their setup. Uninstalling is
- * deleting the file.
- *
- * ## Why a plugin and not OpenCode's own OpenTelemetry
- *
- * OpenCode has `experimental.openTelemetry`, which exports OTLP/JSON traces
- * to `OTEL_EXPORTER_OTLP_ENDPOINT` — on the face of it exactly what we
- * want, and a one-line install. It was measured and rejected:
- *
- * - **It ships the conversation.** The AI SDK spans carry `ai.prompt`,
- *   `ai.prompt.messages` and `ai.response.text` — the system prompt, every
- *   user message and the model's replies, verbatim, with no switch to turn
- *   them off. Codex's log stream leaks tool output; this leaks everything.
- * - **It is enormous.** One prompt produced 230KB, almost all of it
- *   OpenCode's internal spans — SQLite queries, file reads, lock
- *   acquisitions — with the AI spans a rounding error inside it.
- * - **It carries no cost and no repository**, so it could not do the one
- *   job the feature exists for.
- *
- * The plugin sees a better source than the traces do: `AssistantMessage`
- * carries `cost`, a full token breakdown, the model, and `path.cwd`. It
- * sends numbers and identifiers, and nothing a person wrote.
- */
+// Installs and reads back the OpenCode plugin that reports to Clocktopus.
+//
+// OpenCode is the odd one out: Claude Code and Codex expose a *hook* — a command run
+// with JSON on stdin — while OpenCode exposes a plugin API, JavaScript loaded into
+// its own process. So the integration is a file of ours rather than a config entry.
+//
+// The install is a single file. OpenCode auto-loads everything in `<config
+// dir>/plugin/` (verified against 1.18.18), so `opencode.json` is never touched —
+// worth having, since that file is the user's model, provider and permission config.
+// Uninstalling is deleting the file.
+//
+// Not OpenCode's own `experimental.openTelemetry`, which exports OTLP/JSON traces and
+// looks like a one-line install. It was measured and rejected:
+//
+// - It ships the conversation. The AI SDK spans carry `ai.prompt`,
+//   `ai.prompt.messages` and `ai.response.text` verbatim, with no switch to turn
+//   them off.
+// - It is enormous: one prompt produced 230KB, almost all OpenCode's internal spans.
+// - It carries no cost and no repository, so it could not do the one job the feature
+//   exists for.
+//
+// The plugin sees a better source: `AssistantMessage` carries `cost`, a full token
+// breakdown, the model, and `path.cwd`. It sends numbers and identifiers, and
+// nothing a person wrote.
 export const OPENCODE_PLUGIN_FILENAME = "clocktopus.js";
 /** The line `readOpencodeTelemetry` parses back out of the plugin. */
 const CONFIG_MARKER = "const CLOCKTOPUS = ";
 /** Prefix of the line carrying the generation stamp. */
 const VERSION_MARKER = "// clocktopus-plugin-version: ";
-/**
- * Bumped whenever `buildOpencodePlugin` changes what it emits.
- *
- * OpenCode is the only agent whose integration is *generated source* rather
- * than a command string in a config file. Claude Code and Codex hold
- * `clocktopus agent hook …`, which means whatever the installed CLI means,
- * so upgrading the CLI upgrades them. This plugin does not: the file on
- * disk stays exactly as the CLI that wrote it left it, and a fix shipped to
- * the plugin body reaches nobody until they re-run `agent setup`.
- *
- * Stamping the generation is what makes that visible — `agent doctor`
- * compares this number against the file and says so when they differ.
- * Nothing auto-rewrites the file: it holds a token, and a command that
- * quietly rewrites credentials is worse than one that tells you to.
- *
- * A plugin written before the stamp existed parses as `null`, which reads
- * as stale — correct, since it predates every version that has one.
- */
+// Bumped whenever `buildOpencodePlugin` changes what it emits.
+//
+// OpenCode is the only agent whose integration is generated source rather than a
+// command string. Claude Code and Codex hold `clocktopus agent hook …`, which means
+// whatever the installed CLI means, so upgrading the CLI upgrades them. This plugin
+// does not: a fix reaches nobody until they re-run `agent setup`.
+//
+// Stamping the generation makes that visible — `agent doctor` compares this number
+// against the file. Nothing auto-rewrites it: it holds a token, and a command that
+// quietly rewrites credentials is worse than one that tells you to.
+//
+// A plugin written before the stamp parses as `null`, which reads as stale.
 export const OPENCODE_PLUGIN_VERSION = 1;
 export function opencodeConfigDir() {
     if (process.env.OPENCODE_CONFIG_DIR)
@@ -72,42 +53,28 @@ export function opencodeConfigDir() {
 export function opencodePluginPath() {
     return join(opencodeConfigDir(), "plugin", OPENCODE_PLUGIN_FILENAME);
 }
-/**
- * Splits a shell-quoted command line into argv.
- *
- * Only as clever as `resolveHookCommand` is: double quotes around
- * whitespace, nothing else. It is fed that function's output, never a
- * user's shell.
- */
+// Only as clever as `resolveHookCommand` is: double quotes around whitespace,
+// nothing else. It is fed that function's output, never a user's shell.
 export function splitCommandLine(command) {
     return (command.match(/"[^"]*"|\S+/g) ?? []).map((part) => part.startsWith('"') && part.endsWith('"') ? part.slice(1, -1) : part);
 }
-/**
- * The plugin source, with this machine's configuration baked into one line.
- *
- * Generated rather than shipped as a file because the token has to be in
- * it: OpenCode gives a plugin no way to read our environment, so the
- * credentials live where every other agent's do — in that agent's config,
- * written 0600.
- */
+// The plugin source, with this machine's configuration baked into one line.
+// Generated rather than shipped because the token has to be in it: OpenCode gives a
+// plugin no way to read our environment, so the credentials live where every other
+// agent's do — in that agent's config, written 0600.
 export function buildOpencodePlugin(config) {
     return `// Generated by 'clocktopus agent setup'. Edits will be overwritten.
 ${VERSION_MARKER}${OPENCODE_PLUGIN_VERSION}
 //
-// Reports what OpenCode sessions cost to Clocktopus. Two channels, for the
-// same reason every other agent has two: spend and repository context come
-// from different places and neither is much use alone.
+// Reports what OpenCode sessions cost to Clocktopus. Two channels, because spend and
+// repository context come from different places and neither is much use alone:
 //
-//   spend    POSTed straight to the receiver as OTel GenAI spans, because
-//            it happens per assistant message and spawning a process each
-//            time would be absurd.
-//   context  handed to 'clocktopus agent hook', which already knows how to
-//            resolve a git remote, diff a session's commit range and sweep
-//            sessions that died — none of which is worth reimplementing in
-//            here.
+//   spend    POSTed straight to the receiver as OTel GenAI spans, because it happens
+//            per assistant message and spawning a process each time would be absurd.
+//   context  handed to 'clocktopus agent hook', which already resolves git remotes,
+//            diffs a session's commit range and sweeps dead sessions.
 //
-// Nothing a person wrote is read: not the prompt, not the model's reply,
-// not tool output. Only counts, identifiers and timings.
+// Nothing a person wrote is read: not the prompt, not the reply, not tool output.
 import { spawn } from "node:child_process";
 
 ${CONFIG_MARKER}${JSON.stringify(config)};
@@ -132,10 +99,9 @@ function spans(message, version) {
 
   return [
     {
-      // Carries the turn's duration and nothing else. Clocktopus takes
-      // active time from this span alone, so emitting one per assistant
-      // message is what makes the total "time the agent was working"
-      // rather than "time the window was open".
+      // Carries the turn's duration and nothing else. Clocktopus takes active time from
+      // this span alone, so one per assistant message is what makes the total "time the
+      // agent was working" rather than "time the window was open".
       name: "gen_ai.client.session",
       startTimeUnixNano: nano(started),
       endTimeUnixNano: nano(ended),
@@ -149,16 +115,13 @@ function spans(message, version) {
         ...shared,
         attr("gen_ai.request.model", message.modelID),
         attr("gen_ai.provider.name", message.providerID),
-        // OpenCode reports input already net of the cached prefix, so these
-        // buckets add up rather than overlap — the opposite of Codex, which
-        // reports input inclusive and has to be subtracted.
+        // OpenCode reports input already net of the cached prefix, so these buckets add up
+        // rather than overlap — the opposite of Codex, which reports input inclusive.
         attr("gen_ai.usage.input_tokens", message.tokens.input),
-        // Reasoning tokens are output tokens that were not shown. OpenCode
-        // reports them separately and Clocktopus has no bucket for them, so
-        // they are folded in here rather than dropped: its own totals treat
-        // them this way (input + cache.read + output + reasoning == total)
-        // and it prices them at the output rate, so leaving them out would
-        // under-report tokens against a cost that already includes them.
+        // Reasoning tokens are output tokens that were not shown. OpenCode reports them
+        // separately and Clocktopus has no bucket for them, so they are folded in: its own
+        // totals treat them this way and it prices them at the output rate, so leaving them
+        // out would under-report tokens against a cost that already includes them.
         attr(
           "gen_ai.usage.output_tokens",
           message.tokens.output + (message.tokens.reasoning ?? 0),
@@ -168,8 +131,8 @@ function spans(message, version) {
           "gen_ai.usage.cache_creation_input_tokens",
           message.tokens.cache.write,
         ),
-        // Priced by OpenCode from the models.dev rate card, so Clocktopus
-        // records it as an estimate, never as a settled bill.
+        // Priced by OpenCode from the models.dev rate card, so Clocktopus records it as an
+        // estimate, never as a settled bill.
         attr("gen_ai.usage.cost", message.cost),
       ],
     },
@@ -249,11 +212,10 @@ export const ClocktopusPlugin = async ({ directory, worktree }) => {
         return;
       }
 
-      // OpenCode has no "session ended" event — 'idle' is what it emits
-      // when the agent stops working and waits for a human. Treating that
-      // as the end keeps the session's end time tracking the last moment it
-      // was actually busy, which is the bound commit attribution needs, and
-      // re-sending it on a later turn is harmless: the receiver merges.
+      // OpenCode has no "session ended" event — 'idle' is what it emits when the agent
+      // stops and waits for a human. Treating that as the end keeps the session's end time
+      // tracking the last moment it was busy, which is the bound commit attribution needs;
+      // re-sending on a later turn is harmless, since the receiver merges.
       if (event.type === "session.idle") {
         notify("SessionEnd", event.properties.sessionID, root);
         return;
@@ -261,9 +223,8 @@ export const ClocktopusPlugin = async ({ directory, worktree }) => {
 
       if (event.type === "message.updated") {
         const message = event.properties.info;
-        // Assistant messages only, and only once finished — an in-flight
-        // message is republished on every token, with counts that are not
-        // final.
+        // Assistant messages only, and only once finished — an in-flight message is
+        // republished on every token, with counts that are not final.
         if (message?.role !== "assistant" || !message.time?.completed) return;
         await report(message, version);
       }
@@ -300,10 +261,9 @@ export function readOpencodePlugin(path = opencodePluginPath()) {
         }
     }
     catch {
-        // A plugin we cannot read the configuration out of is reported as
-        // unconfigured, which sends the user to `setup` — the right answer
-        // whether it was hand-edited or written by a version that has since
-        // changed shape.
+        // A plugin we cannot read the configuration out of is reported as unconfigured,
+        // which sends the user to `setup` — the right answer whether it was hand-edited or
+        // written by a version that has since changed shape.
     }
     return { path, exists: true, modifiedAt, config, version };
 }
@@ -314,9 +274,9 @@ export function writeOpencodePlugin(source, path = opencodePluginPath()) {
         backupPath = `${path}.clocktopus-backup`;
         copyFileSync(path, backupPath);
     }
-    // Rename rather than write in place: OpenCode loads every file in this
-    // directory at startup, and a half-written one would be a syntax error
-    // that takes the whole plugin system down with it.
+    // Rename rather than write in place: OpenCode loads every file in this directory at
+    // startup, and a half-written one would be a syntax error that takes the whole
+    // plugin system down with it.
     const temporaryPath = `${path}.clocktopus-tmp`;
     writeFileSync(temporaryPath, source, { encoding: "utf8", mode: 0o600 });
     renameSync(temporaryPath, path);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blastin-dev/clocktopus-cli",
-  "version": "0.2.1",
+  "version": "0.3.1",
   "type": "module",
   "bin": {
     "clocktopus": "./dist/bin/clocktopus.js"
@@ -16,14 +16,14 @@
     "zod": "4.4.3"
   },
   "devDependencies": {
+    "@repo/eslint-config": "0.0.0",
+    "@repo/prettier-config": "0.1.0",
+    "@repo/typescript-config": "0.0.0",
     "@types/node": "22.15.3",
     "eslint": "9.37.0",
     "eslint-plugin-import-x": "^4.16.1",
     "typescript": "5.9.2",
-    "vitest": "^4.1.5",
-    "@repo/eslint-config": "0.0.0",
-    "@repo/prettier-config": "0.1.0",
-    "@repo/typescript-config": "0.0.0"
+    "vitest": "^4.1.5"
   },
   "prettier": "@repo/prettier-config",
   "scripts": {