brl-subagent 2.3.5 → 2.3.6

package/AGENT.md

@@ -1,4 +1,4 @@
-# AGENT.md — brl-subagent capability reference (v2.3.4+)
+# AGENT.md — brl-subagent capability reference
 
 > The `delegate_task` prompt guidelines are a derived summary of this file; if they disagree, this file wins.
 
@@ -56,33 +56,3 @@ Presets and templates are documented in the schema, not here. Judgment only:
 - 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.
-
-## Local development
-
-This repo is the package source. Daily use runs the **published npm package**;
-development runs the **local checkout**. Both are `pi install` entries and cannot
-coexist — two loaded copies register the same tools — so remove one first.
-
-```bash
-# Daily: the published artifact — the same thing users install
-pi remove <local-repo-path>
-pi install npm:brl-subagent
-
-# Develop: load the checkout in place — no copying, no syncing
-pi remove npm:brl-subagent
-pi install <local-repo-path>        # or a worktree: …/brl-subagent-wt-<n>
-# then /reload pi after edits
-
-# Back to the published artifact
-pi remove <local-repo-path>
-pi install npm:brl-subagent
-```
-
-A local-path install loads the directory in place and can point at an **unmerged
-worktree**, so it doubles as live verification for a PR. `pi update --extensions`
-advances the npm install and leaves a local-path install alone.
-
-(The former `sync-extension.sh` — rsync into `~/.pi/agent/extensions/brl-subagent`
-— was retired once the package was published: the local-path install supersedes it
-and does not fight `pi update --extensions` by mutating a package-manager-owned
-directory.)

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
 
 ---
 
@@ -35,16 +35,6 @@ pi update --extensions        # update installed packages
 pi remove npm:brl-subagent    # uninstall
 ```
 
-<details>
-<summary>Alternative: install from git</summary>
-
-```bash
-pi install git:github.com/azmeenafandi/brl-subagent@v2.3.4
-```
-
-Git installs are **pinned** to the ref you specify. `pi update --extensions` reconciles to that ref but never advances it — upgrading means re-running `pi install` with a newer `@ref`.
-</details>
-
 ---
 
 ## Quick reference
@@ -70,7 +60,6 @@ Git installs are **pinned** to the ref you specify. `pi update --extensions` rec
 | `/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.5",
+  "version": "2.3.6",
   "description": "Subagent extension for pi with configurable model and thinking mode via selection UI",
   "keywords": [
     "pi-package"
@@ -42,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,
@@ -93,7 +92,6 @@ import {
 	showCompletionNotifySelector,
 	showPresetManager,
 	showTemplateManager,
-	showUpdateCheckToggle,
 	showSLAConfig,
 	showSLAStats,
 	showConfigMenu,
@@ -106,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,
@@ -198,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(pkgPath("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);
 
@@ -1932,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),
 			};
@@ -3740,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/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/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;
-	}
-}