pi-chrome 0.15.44 → 0.15.46

package/extensions/chrome-profile-bridge/browser-extension/manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "Pi Chrome Connector",
-  "version": "0.15.44",
+  "version": "0.15.46",
   "description": "Lets Pi control tabs in Chrome via a local connector at 127.0.0.1.",
   "permissions": [
     "tabs",

package/extensions/chrome-profile-bridge/index.ts

@@ -701,6 +701,7 @@ export default function (pi: ExtensionAPI): void {
 		else globalState[PI_CHROME_AUTH_KEY] = { until: chromeAuthorizedUntil };
 	};
 	let chromeToolsRegistered = false;
+	let chromeToolsUsable = false;
 	let authExpiryTimer: NodeJS.Timeout | undefined;
 	let countdownInterval: NodeJS.Timeout | undefined;
 	// Remembered so bridge sends can tag tabs with this session's group even when ctx isn't handy.
@@ -734,13 +735,17 @@ export default function (pi: ExtensionAPI): void {
 		return chromeToolsActive(before);
 	};
 
-	const logChromeToolChange = (action: "authorized" | "revoked" | "expired", label?: string): void => {
-		const enabled = action === "authorized";
-		const content = enabled
-			? `Chrome tools enabled by /chrome authorize${label ? ` (${label})` : ""}.`
-			: action === "expired"
-				? "Chrome tools disabled because /chrome authorize grant expired."
-				: "Chrome tools disabled by /chrome revoke.";
+	const logChromeToolChange = (
+		action: "authorized" | "reauthorized" | "revoked" | "expired",
+		options: { label?: string; authorizedUntil?: number | "indefinite" } = {},
+	): void => {
+		const content = action === "authorized"
+			? `Chrome tools enabled by /chrome authorize${options.label ? ` (${options.label})` : ""}.`
+			: action === "reauthorized"
+				? `Chrome tool authorization updated by /chrome authorize${options.label ? ` (${options.label})` : ""}.`
+				: action === "expired"
+					? "Chrome tools disabled because /chrome authorize grant expired."
+					: "Chrome tools disabled by /chrome revoke.";
 		pi.sendMessage({
 			customType: "pi-chrome-tool-change",
 			content,
@@ -748,7 +753,7 @@ export default function (pi: ExtensionAPI): void {
 			details: {
 				action,
 				tools: [...CHROME_TOOL_NAMES],
-				authorizedUntil: chromeAuthorizedUntil,
+				authorizedUntil: options.authorizedUntil,
 				at: Date.now(),
 			},
 		}, { triggerTurn: false });
@@ -766,8 +771,10 @@ export default function (pi: ExtensionAPI): void {
 	const lockChromeControl = (logAction?: "revoked" | "expired"): void => {
 		clearAuthExpiryTimer();
 		clearCountdownInterval();
-		const hadChromeTools = deactivateChromeTools();
-		if (logAction && hadChromeTools) logChromeToolChange(logAction);
+		const wasUsable = chromeToolsUsable;
+		deactivateChromeTools();
+		chromeToolsUsable = false;
+		if (logAction && wasUsable) logChromeToolChange(logAction, { authorizedUntil: undefined });
 		chromeAuthorizedUntil = undefined;
 		persistAuth();
 		// Revoking control ends pi-chrome's automation for this session; tidy up the target we own.
@@ -908,8 +915,12 @@ export default function (pi: ExtensionAPI): void {
 		// Reestablish in-memory state after a /reload restored chromeAuthorizedUntil from globalThis.
 		if (chromeControlAuthorized()) {
 			activateChromeTools();
+			chromeToolsUsable = true;
 			if (typeof chromeAuthorizedUntil === "number") scheduleAuthExpiry(ctx, chromeAuthorizedUntil);
 			else if (chromeAuthorizedUntil === "indefinite") startCountdownTicker(ctx);
+		} else {
+			deactivateChromeTools();
+			chromeToolsUsable = false;
 		}
 		updateChromeStatus(ctx);
 	});
@@ -1063,10 +1074,12 @@ Usage rules:
 			ctx.ui.notify("Chrome control remains locked.", "info");
 			return;
 		}
+		const wasUsable = chromeToolsUsable;
 		chromeAuthorizedUntil = until;
 		persistAuth();
 		activateChromeTools();
-		logChromeToolChange("authorized", label);
+		chromeToolsUsable = true;
+		logChromeToolChange(wasUsable ? "reauthorized" : "authorized", { label, authorizedUntil: until });
 		scheduleAuthExpiry(ctx, until);
 		ctx.ui.notify(`Chrome control authorized for ${label}.`, "info");
 		updateChromeStatus(ctx);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-chrome",
-	"version": "0.15.44",
+	"version": "0.15.46",
 	"scripts": {
 		"test": "node test-suite/unit/csp-eval.test.mjs && node test-suite/unit/automation-target.test.mjs",
 		"version": "node scripts/sync-manifest-version.js",