brl-subagent 2.3.4 → 2.3.6

package/AGENT.md

@@ -0,0 +1,58 @@
+# AGENT.md — brl-subagent capability reference
+
+> The `delegate_task` prompt guidelines are a derived summary of this file; if they disagree, this file wins.
+
+This file is the authoritative reference for how the brl-subagent extension behaves
+and how you (the conductor) should judge a delegation. Read it when planning
+delegation-heavy work.
+
+## Execution models
+
+| Mode | What blocks | What wakes you | Batchable? |
+| --- | --- | --- | --- |
+| single, foreground | blocks until it returns | — (result comes back inline) | no |
+| single, background | returns immediately with an agent id | yes — a `subagent-completion` message on terminal state | no |
+| chain | blocks until the chain finishes or fails | — | yes (foreground) |
+| parallel (`tasks`) | blocks until every task finishes | — | yes (foreground) |
+| graph | blocks; runs in waves via `dependsOn` | — | yes (foreground) |
+
+- Foreground single blocks: you wait for the result inline.
+- Background single returns immediately with an agent id; a completion message wakes you (see below).
+- Chain, parallel, and graph are batch — one call, many subtasks. They are foreground today; background batch is future work.
+- Chain stops at the first failure; max 10 steps.
+- Parallel runs every task regardless of the others — no short-circuit.
+- Graph runs in waves, ordered by `dependsOn` edges.
+
+## The completion contract
+
+- A background run wakes you with a structured `subagent-completion` message when it reaches a terminal state (`completed`, `failed`, or `stopped`). The `details` carry `id`, `status`, `errorCategory` (with `errorMessage` when present), cost/tokens/duration, and `label`; the content carries a tail of output.
+- The wake triggers a turn even when you are idle. Do not poll for it.
+- Delivery mode by status holds at knob `"all"`: `failed`/`stopped` → steering you to act; `completed` → follow-up. The `completionNotify` knob changes both the delivery mode and the wake: under `"failed"` a `completed` run is delivered as a passive `nextTurn` (no wake) while `failed`/`stopped` still steer; under `"off"` everything is a passive `nextTurn` and nothing wakes you. (The knob controls whether an idle conductor is triggered — see below.)
+- Knob dependency: polling is correct only when wakes are disabled (`completionNotify: "off"`). The wake is per-terminal-status and knob-scoped: under `"all"` a terminal run always wakes you; under `"failed"` only a `failed` or `stopped` run wakes you — a `completed` run stays a passive `nextTurn` (no wake); under `"off"` nothing wakes you. So polling is wrong unless wakes are disabled. One status check as a stall check is legitimate; repeated polling is not.
+- One message per run — the extension deduplicates on the first terminal event for a run id, so don't expect multiple messages for a single run.
+- Honest records: trust the message's `category`/abort source over your own guess. The provider/abort origin is authoritative on why the run stopped.
+- Steering is not an abort. Being steered to act on a terminal run is not terminating it. A real `stop` is an abort — and a stopped run still wakes you.
+
+## Delegation judgment
+
+- Foreground vs background: if your next step needs the result inline, run foreground. If the user can keep working or is AFK, run background and act on the wake.
+- Batch vs sequential: if the work is pre-declared, use chain/parallel/graph. If you're steering incrementally with the user, issue sequential calls.
+- Retry taxonomy: `retryRunId` is a retry — it re-runs with the recorded task and params. A re-dispatch (a fresh call with the same task text) is not a retry. Never re-issue an identical spec after a termination without confirming the cause with the user (Rule 18).
+- When to delegate: when the task needs an isolated context, a deep investigation, parallel research, or a long-running analysis.
+- Approval: `approvalMode: "always"` is rejected in background — there is no dialog. Use `auto` (default) or `writes`; `writes` auto-approves in background.
+
+## Canonical shapes
+
+- Batch chain with `{previous}`: `chain: [{ task: "…" }, { task: "Use {previous} …" }]` — each step gets the prior step's output.
+- Graph with `dependsOn`: subtasks declare edges; the scheduler runs them in waves.
+- Background + act on wake: spawn a background run, then act when the completion message arrives. Never poll between spawn and wake.
+- retryRunId: pass a failed run's id to re-run it with its recorded task and params; explicit parameters override the originals.
+
+## Mechanics pointer
+
+Presets and templates are documented in the schema, not here. Judgment only:
+
+- Prefer a preset when the delegation matches a standard shape — review/audit/test/docs; override per-call deliberately.
+- Recurring task shapes → a saved template.
+
+The schema's `delegate_task` parameters, the `templateSummary` and `presetRestrictionSummary` lines in the prompt guidelines, and `get_subagent_result`'s description are the mechanics reference.

package/README.md

@@ -2,7 +2,7 @@
 
 > Multi-agent orchestration for [pi](https://github.com/earendil-works/pi) — chain, parallel, and dependency-graph delegation to isolated subagents with per-step model routing, preset-driven tool scoping, thinking-level control, and background execution with live monitoring, real abort, and per-agent timeouts.
 
-**Version:** 2.3.4 · **Author:** Azmeen Afandi / [Beeroo Labs](https://beeroolabs.com) · **License:** MIT
+**Version:** 2.3.6 · **Author:** Azmeen Afandi / [Beeroo Labs](https://beeroolabs.com) · **License:** MIT
 
 ---
 
@@ -14,7 +14,7 @@
 - **True background execution** — live monitor, real abort (`stop_subagent`), per-agent timeouts, and hard caps. Nothing orphans; nothing leaks.
 - **Preset-driven tool scoping** — every subagent runs with exactly the tools its job needs, restricted by preset or per-call `tools`/`excludeTools`, with auto-route that picks the right preset when you don't.
 - **Templates with slots** — saved, file-backed task templates with `${param}` placeholders for workflows you run again and again.
-- **Safety by default** — task-fence injection protection, sanitized error paths, owner-only persistence, and a 700+ test suite pinning every contract against the real pi SDK.
+- **Safety by default** — task-fence injection protection, sanitized error paths, owner-only persistence, and a 975+ test suite pinning every contract against the real pi SDK.
 
 ---
 
@@ -22,13 +22,18 @@
 
 ```bash
 # Global (all projects)
-cd ~/.pi/agent/extensions && git clone https://github.com/azmeenafandi/brl-subagent.git
+pi install npm:brl-subagent
 
-# Project-local
-cd your-project/.pi/extensions && git clone https://github.com/azmeenafandi/brl-subagent.git
+# Project-local (commit .pi/settings.json to share with your team)
+pi install -l npm:brl-subagent
 ```
 
-Pi auto-discovers extensions in these directories. To update: `git pull` inside the cloned directory.
+Pi installs the package and loads the extension automatically. Updates and removal:
+
+```bash
+pi update --extensions        # update installed packages
+pi remove npm:brl-subagent    # uninstall
+```
 
 ---
 
@@ -55,7 +60,6 @@ Pi auto-discovers extensions in these directories. To update: `git pull` inside
 | `/brl-subagent monitor` | Live monitor running subagents |
 | `/brl-subagent dashboard` | Live observability dashboard |
 | `/brl-subagent retry` | Browse failed runs to retry |
-| `/brl-subagent update-check` | Toggle update check on startup |
 | `/brl-subagent reset` | Reset all configuration |
 
 All settings persist across sessions.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brl-subagent",
-  "version": "2.3.4",
+  "version": "2.3.6",
   "description": "Subagent extension for pi with configurable model and thinking mode via selection UI",
   "keywords": [
     "pi-package"
@@ -16,6 +16,7 @@
     "templates/",
     "README.md",
     "LICENSE",
+    "AGENT.md",
     "!src/__tests__/"
   ],
   "pi": {
@@ -41,7 +42,7 @@
     "jiti": "^2.7.0",
     "typebox": "^1.3.25",
     "typescript": "^5.9.3",
-    "vitest": "^4.1.11"
+    "vitest": "^5.0.0"
   },
   "overrides": {
     "nanoid": "^3.3.17"

package/src/index.ts

@@ -4,7 +4,6 @@
  * See AGENT.md for the conductor-facing capability reference.
  */
 
-import * as fs from "node:fs";
 import * as path from "node:path";
 import type {
 	AgentToolResult,
@@ -52,6 +51,7 @@ import {
 } from "./types";
 import { validateGraph, topologicalSort } from "./scheduler";
 import { resolveTemplate, loadAllTemplates, loadBuiltinTemplates, validateTemplatePresetRefs, extractParamNames } from "./templates";
+import { pkgPath } from "./paths";
 
 import { sanitizeTask, validateCwd, validateOutputFile, stripAnsi, capOutput, getCurrentDepth, sanitizeErrorMessage, buildCrashResult } from "./sanitize";
 import {
@@ -92,7 +92,6 @@ import {
 	showCompletionNotifySelector,
 	showPresetManager,
 	showTemplateManager,
-	showUpdateCheckToggle,
 	showSLAConfig,
 	showSLAStats,
 	showConfigMenu,
@@ -105,8 +104,6 @@ import {
 } from "./tui";
 import { createLogger, type Logger } from "./logging";
 import { Intercom } from "./messaging";
-import { checkForUpdates } from "./update";
-import { UPDATE_CHECK_INTERVAL_MS } from "./types";
 import * as eventBus from "./event-bus";
 import {
 	buildCompletionMessage,
@@ -197,16 +194,6 @@ export function buildTemplateGuideline(templateSummary: string): string {
 export default function (pi: ExtensionAPI) {
 	const log = createLogger("brl-subagent");
 
-	// Read current version from package.json
-	const currentVersion = (() => {
-		try {
-			const pkg = JSON.parse(fs.readFileSync(path.join(__dirname, "..", "package.json"), "utf-8"));
-			return pkg.version || "0.0.0";
-		} catch {
-			return "0.0.0";
-		}
-	})();
-
 	// F7: Session-bound state — initialized per session
 	let state = createSessionState(log);
 
@@ -1931,7 +1918,6 @@ export default function (pi: ExtensionAPI) {
 				preset: () => showPresetManager(ctx, state),
 				templates: () => showTemplateManager(ctx, state),
 				retry: () => showRetryMenu(ctx, state),
-			"update-check": () => showUpdateCheckToggle(ctx, state, applyConfig),
 			sla: () => showSLAConfig(ctx, state, applyConfig),
 			"sla-stats": () => showSLAStats(ctx, state),
 			};
@@ -1954,7 +1940,7 @@ export default function (pi: ExtensionAPI) {
 
 	// B1: Load built-in presets once at registration to expose their tool
 	// restrictions to the conductor before any delegate_task call happens.
-	const registrationPresets = loadBuiltinPresets(path.join(__dirname, "..", "presets"));
+	const registrationPresets = loadBuiltinPresets(pkgPath("presets"));
 	const presetRestrictionSummary = buildInventorySummary(registrationPresets, (p) => {
 		const restricted = p.excludeTools?.length || p.tools?.length;
 		if (!restricted) return p.name;
@@ -1965,9 +1951,14 @@ export default function (pi: ExtensionAPI) {
 	// inventory (name + purpose + ${param} slots) to the conductor before any
 	// delegate_task call. Custom/user templates can't be enumerated statically,
 	// so the summary covers built-ins only and points at /brl-subagent templates.
-	const registrationTemplates = loadBuiltinTemplates(path.join(__dirname, "..", "templates"));
+	const registrationTemplates = loadBuiltinTemplates(pkgPath("templates"));
 	const templateSummary = buildInventorySummary(registrationTemplates, formatTemplateSummaryItem);
 
+	// Issue #158: resolve the shipped AGENT.md from the extension's own directory
+	// rather than a hardcoded sync-copy destination, so npm-installed packages
+	// (installed via `pi install npm:brl-subagent`) can find it.
+	const agentMdPath = pkgPath("AGENT.md");
+
 	pi.registerTool({
 		name: "delegate_task",
 		label: "Delegate Task",
@@ -1983,7 +1974,7 @@ export default function (pi: ExtensionAPI) {
 			"Delegate tasks to a subagent for isolated, parallel or background work",
 		promptGuidelines: [
 			"Use delegate_task when the user asks you to hand off work to a subagent, or when a task would benefit from an isolated context window (e.g., deep investigation, parallel research, long-running analysis).",
-			"The authoritative capability reference for delegation is the extension's AGENT.md — read it at ~/.pi/agent/extensions/brl-subagent/AGENT.md, or ./AGENT.md in the project root when present, when planning delegation-heavy work.",
+			`The authoritative capability reference for delegation is the extension's AGENT.md — read it at ${agentMdPath}, or ./AGENT.md in the project root when present, when planning delegation-heavy work.`,
 			"The subagent inherits your system prompt and runs with its own model (configurable via /brl-subagent). It reports what it did when done.",
 			"You can customize per-call via inheritSystemPrompt and systemPrompt: set inheritSystemPrompt: false to save context, provide a systemPrompt for custom instructions, or use both to add instructions on top of inheritance.",
 			"Set thinkingLevel per call to match task complexity. The level is capped at the user's configured maximum. Map tasks to levels using this heuristic: off = file listing, grep, simple read. minimal = file diff, syntax check, find-and-replace. low = refactoring, test generation, documentation. medium = default — code review, debugging, moderate analysis. high = security audit, architecture review, complex debugging. xhigh = multi-step causal reasoning, research, novel problem solving. Default to 'off' or 'minimal' for trivial tasks — do not waste the user's budget.",
@@ -3691,11 +3682,11 @@ export default function (pi: ExtensionAPI) {
 		// Issue #147: capture the session context for the completion-push subscriber.
 		sessionCtx = ctx;
 		// Load built-in presets
-		const presetsDir = path.join(__dirname, "..", "presets");
+		const presetsDir = pkgPath("presets");
 		state.builtinPresets = loadBuiltinPresets(presetsDir, log);
 		state.customPresets = loadCustomPresets(ctx.cwd, log);
 		// Builtin templates + merged full stack (custom overrides builtin)
-		const templatesDir = path.join(__dirname, "..", "templates");
+		const templatesDir = pkgPath("templates");
 		state.builtinTemplates = loadBuiltinTemplates(templatesDir, log);
 		state.config.templates = loadAllTemplates(ctx.cwd, log, templatesDir);
 
@@ -3734,23 +3725,6 @@ export default function (pi: ExtensionAPI) {
 		// F5/F9: Safe state restoration with type guards
 		state.restoreFromSession(ctx);
 
-		// Check for updates (non-blocking, once per 24h)
-		if (state.config.updateCheckEnabled) {
-			const now = Date.now();
-			if (now - state.config.lastUpdateCheck > UPDATE_CHECK_INTERVAL_MS) {
-				state.config.lastUpdateCheck = now;
-				checkForUpdates(currentVersion, log).then((result) => {
-					if (result?.available) {
-						ctx.ui.notify(
-							"brl-subagent " + result.version + " available (current: " + currentVersion + "). Visit " + result.url + " to update. /brl-subagent update-check to disable.",
-							"info"
-						);
-						log.info("Update available", { current: currentVersion, latest: result.version });
-					}
-				}).catch(() => {}); // silently ignore errors
-			}
-		}
-
 		updateStatus(state, ctx);
 	});
 

package/src/paths.ts

@@ -0,0 +1,25 @@
+/**
+ * brl-subagent — Package-root path resolution
+ *
+ * The extension is loaded from `<pkg>/src/` (package.json `pi.extensions`
+ * points at `./src/index.ts`), so every module in this package lives directly
+ * in `src/` and the package root is exactly one level up. `pkgPath()` is the
+ * single helper for resolving bundled assets against that root — the same base
+ * used for `presets/`, `templates/`, `package.json`, and `AGENT.md`.
+ */
+
+import * as path from "node:path";
+
+/**
+ * Resolve `segments` against the package root.
+ *
+ * Because the extension loads from `<pkg>/src/`, the package root is one level
+ * up from this module's directory. A wrong depth here would resolve inside
+ * `src/` and fail to find the bundled asset.
+ *
+ * @param segments - Path segments below the package root (e.g. `"presets"`, `"AGENT.md"`)
+ * @returns The absolute path to the bundled asset
+ */
+export function pkgPath(...segments: string[]): string {
+	return path.join(__dirname, "..", ...segments);
+}

package/src/state.ts

@@ -142,8 +142,6 @@ export class SessionState {
 			circuitBreaker: this.defaultCircuitBreaker(),
 			slaTrackingEnabled: false,
 			slaWindowSize: 50,
-			updateCheckEnabled: true,
-			lastUpdateCheck: 0,
 		};
 	}
 
@@ -168,8 +166,6 @@ export class SessionState {
 		slaTrackingEnabled: this.config.slaTrackingEnabled,
 		slaWindowSize: this.config.slaWindowSize,
 		lastSLAMetrics: this.config.lastSLAMetrics,
-		updateCheckEnabled: this.config.updateCheckEnabled,
-		lastUpdateCheck: this.config.lastUpdateCheck,
 		});
 	}
 
@@ -258,10 +254,6 @@ export class SessionState {
 			this.config.lastSLAMetrics = data.lastSLAMetrics as import("./types").SLAMetrics;
 		}
 
-		// Restore update check fields
-		if (typeof data.updateCheckEnabled === "boolean") this.config.updateCheckEnabled = data.updateCheckEnabled;
-		if (typeof data.lastUpdateCheck === "number" && data.lastUpdateCheck >= 0) this.config.lastUpdateCheck = data.lastUpdateCheck;
-
 		this.log?.info("State restored from session", {
 			model: data.model ? `${data.model.provider}/${data.model.id}` : "none",
 			thinkingLevel: data.maxThinkingLevel,
@@ -508,8 +500,6 @@ export class SessionState {
 		this.config.slaTrackingEnabled = false;
 		this.config.slaWindowSize = 50;
 		this.config.lastSLAMetrics = undefined;
-		this.config.updateCheckEnabled = true;
-		this.config.lastUpdateCheck = 0;
 	}
 }
 

package/src/templates.ts

@@ -27,6 +27,7 @@ import type { TaskTemplate, ThinkingLevel, SubagentPreset } from "./types";
 import { TEMPLATE_PARAM_RE, THINKING_LEVELS } from "./types";
 import type { Logger } from "./logging";
 import { parseFrontmatter, sanitizeFileName } from "./presets";
+import { pkgPath } from "./paths";
 
 // ---------------------------------------------------------------------------
 // Param extraction
@@ -373,7 +374,7 @@ export function getAllTemplates(
  * the extension's presets/ dir.
  */
 export function loadAllTemplates(cwd: string, log?: Logger, templatesDir?: string): TaskTemplate[] {
-	const builtinDir = templatesDir ?? path.join(__dirname, "..", "templates");
+	const builtinDir = templatesDir ?? pkgPath("templates");
 	return getAllTemplates(loadBuiltinTemplates(builtinDir, log), loadCustomTemplates(cwd, log));
 }
 

package/src/tui.ts

@@ -337,28 +337,6 @@ export async function showCompletionNotifySelector(
 	onConfigChanged(ctx, `Completion notify mode set to ${result}`);
 }
 
-// ---------------------------------------------------------------------------
-// Update check toggle
-// ---------------------------------------------------------------------------
-
-export async function showUpdateCheckToggle(
-	ctx: ExtensionContext,
-	state: SessionState,
-	onConfigChanged: (ctx: ExtensionContext, msg: string) => void,
-): Promise<void> {
-	const items: SelectItem[] = [
-		{ value: "enabled", label: "Enabled", description: "Check for updates on session start" },
-		{ value: "disabled", label: "Disabled", description: "Skip update check" },
-	];
-
-	const result = await showSelectList(ctx, "Update Check", items, 5);
-	if (!result) return;
-
-	state.config.updateCheckEnabled = result === "enabled";
-	onConfigChanged(ctx, `Update check ${result}`);
-}
-
-// ---------------------------------------------------------------------------
 // ---------------------------------------------------------------------------
 // E4: SLA tracking configuration
 // ---------------------------------------------------------------------------
@@ -1199,11 +1177,6 @@ export function getConfigMenuItems(state: SessionState): SelectItem[] {
 				? `Enabled (window: ${state.config.slaWindowSize})`
 				: "Disabled",
 		},
-		{
-			value: "update-check",
-			label: "Update Check",
-			description: state.config.updateCheckEnabled ? "Enabled" : "Disabled",
-		},
 	];
 }
 

package/src/types.ts

@@ -209,8 +209,6 @@ export interface SubagentState {
 	slaTrackingEnabled: boolean; // E4: SLA tracking toggle
 	slaWindowSize: number; // E4: number of recent runs to analyze (10-500)
 	lastSLAMetrics?: SLAMetrics; // E4: persisted baseline for degradation comparison
-	updateCheckEnabled: boolean; // Version update notifier toggle
-	lastUpdateCheck: number; // Epoch ms of last update check
 }
 
 /**
@@ -556,8 +554,6 @@ export const TEMPLATE_PARAM_RE = /\$\{(\w+)\}/g;
 
 export const DEFAULT_SESSION_COST_LIMIT = 0;
 
-export const UPDATE_CHECK_INTERVAL_MS = 24 * 60 * 60 * 1000; // 24 hours
-
 export const CUSTOM_ENTRY_TYPES = {
 	run: "brl-subagent-run",
 	state: "brl-subagent-state",
@@ -589,7 +585,7 @@ export const RESERVED_NAME_PATTERN = /^__.*__$/;
 export const RESERVED_COMMAND_NAMES = new Set([
 	"model", "thinking", "concurrency", "depth", "history", "monitor",
 	"preset", "retry", "reset", "priority", "templates", "dashboard", "approval",
- "costlimit", "historyentries", "sla", "update-check", "completionnotify",
+ "costlimit", "historyentries", "sla", "completionnotify",
 	"graph", "sla-stats",
 ]);
 
@@ -750,14 +746,6 @@ export function isSubagentStateShape(value: unknown): value is SubagentState {
 		if (typeof v.slaWindowSize !== "number" || v.slaWindowSize < 10 || v.slaWindowSize > 500) return false;
 	}
 
-	// updateCheckEnabled must be a boolean if present
-	if (v.updateCheckEnabled !== undefined && typeof v.updateCheckEnabled !== "boolean") return false;
-
-	// lastUpdateCheck must be a non-negative number if present
-	if (v.lastUpdateCheck !== undefined) {
-		if (typeof v.lastUpdateCheck !== "number" || v.lastUpdateCheck < 0) return false;
-	}
-
 	return true;
 }
 

package/src/update.ts

@@ -1,109 +0,0 @@
-/**
- * brl-subagent — Version Update Notifier
- *
- * Checks GitHub releases for newer versions of brl-subagent.
- * Runs non-blocking with a 5-second timeout on session start.
- */
-
-import type { Logger } from "./logging";
-
-// ---------------------------------------------------------------------------
-// Types
-// ---------------------------------------------------------------------------
-
-export interface UpdateCheckResult {
-	available: boolean;
-	version?: string;
-	url?: string;
-}
-
-// ---------------------------------------------------------------------------
-// Semver comparison
-// ---------------------------------------------------------------------------
-
-/**
- * Compare two semver strings (e.g., "2.1.0" vs "2.0.1").
- * Returns 1 if a > b, -1 if a < b, 0 if equal.
- */
-function compareSemver(a: string, b: string): number {
-	const aParts = a.split(".").map(Number);
-	const bParts = b.split(".").map(Number);
-	const len = Math.max(aParts.length, bParts.length);
-
-	for (let i = 0; i < len; i++) {
-		const aVal = aParts[i] ?? 0;
-		const bVal = bParts[i] ?? 0;
-		if (aVal > bVal) return 1;
-		if (aVal < bVal) return -1;
-	}
-	return 0;
-}
-
-// ---------------------------------------------------------------------------
-// Main export
-// ---------------------------------------------------------------------------
-
-/**
- * Check GitHub for a newer release of brl-subagent.
- *
- * @param currentVersion - The current version string (e.g., "2.0.1")
- * @param log - Optional logger for debug output
- * @returns Update check result, or null on any error (never throws)
- */
-export async function checkForUpdates(
-	currentVersion: string,
-	log?: Logger,
-): Promise<UpdateCheckResult | null> {
-	try {
-		const controller = new AbortController();
-		const timeoutId = setTimeout(() => controller.abort(), 5_000);
-
-		const response = await fetch(
-			"https://api.github.com/repos/azmeenafandi/brl-subagent/releases/latest",
-			{
-				headers: {
-					Accept: "application/vnd.github+json",
-					"User-Agent": "brl-subagent",
-				},
-				signal: controller.signal,
-			},
-		);
-
-		clearTimeout(timeoutId);
-
-		if (!response.ok) {
-			log?.debug("Update check: non-OK response", { status: response.status });
-			return null;
-		}
-
-		const data = (await response.json()) as {
-			tag_name?: string;
-			html_url?: string;
-		};
-
-		if (!data.tag_name || !data.html_url) {
-			log?.debug("Update check: missing tag_name or html_url in response");
-			return null;
-		}
-
-		// Strip "v" prefix from tag_name (e.g., "v2.1.0" → "2.1.0")
-		const latestVersion = data.tag_name.replace(/^v/, "");
-
-		const cmp = compareSemver(latestVersion, currentVersion);
-
-		if (cmp > 0) {
-			return {
-				available: true,
-				version: latestVersion,
-				url: data.html_url,
-			};
-		}
-
-		return { available: false };
-	} catch (err) {
-		log?.debug("Update check failed", {
-			error: err instanceof Error ? err.message : String(err),
-		});
-		return null;
-	}
-}