@oh-my-pi/pi-coding-agent 15.12.0 → 15.12.2

package/dist/types/collab/protocol.d.ts

@@ -92,11 +92,17 @@ export declare function rewriteEnvelopePeer(data: Uint8Array, peerId: number): v
 export declare function generateRoomId(): string;
 /**
  * Render the shareable link. Compact forms: the default relay collapses to
- * `<roomId>#<key>`, other wss relays drop the scheme (`host[:port]/r/…`);
+ * `<roomId>.<key>`, other wss relays drop the scheme (`host[:port]/r/…`);
  * only localhost ws:// links keep their full URL so parsing cannot
  * mis-infer wss.
  *
- * Full links append the write token to the key in the fragment
+ * The room secret is dot-joined (`<roomId>.<key>`) rather than `#`-joined:
+ * RFC 3986 forbids a raw `#` inside a fragment, so strict URL stacks (macOS
+ * Foundation behind terminal click-to-open) percent-encode a second `#` to
+ * `%23` and break the link. Parsers still accept the legacy `#` form and the
+ * mangled `%23` form.
+ *
+ * Full links append the write token to the key
  * (`base64url(key ∥ writeToken)`); read-only (view) links carry the bare
  * 32-byte key, which is also the pre-token link format.
  */

package/dist/types/config/settings-schema.d.ts

@@ -1184,7 +1184,7 @@ export declare const SETTINGS_SCHEMA: {
     readonly doubleEscapeAction: {
         readonly type: "enum";
         readonly values: readonly ["branch", "tree", "none"];
-        readonly default: "tree";
+        readonly default: "branch";
         readonly ui: {
             readonly tab: "interaction";
             readonly group: "Input";
@@ -1401,7 +1401,7 @@ export declare const SETTINGS_SCHEMA: {
             readonly tab: "interaction";
             readonly group: "Collab";
             readonly label: "Relay URL";
-            readonly description: "Relay used by /collab (wss://host[:port]; self-host with the omp-collab-relay service)";
+            readonly description: "Relay used by /collab (wss://host[:port])";
         };
     };
     readonly "collab.displayName": {
@@ -1777,6 +1777,16 @@ export declare const SETTINGS_SCHEMA: {
             readonly description: "Prune older read results when the same file is read again (cache-aware, runs every turn)";
         };
     };
+    readonly "compaction.dropUseless": {
+        readonly type: "boolean";
+        readonly default: true;
+        readonly ui: {
+            readonly tab: "context";
+            readonly group: "Compaction";
+            readonly label: "Elide Uneventful Results";
+            readonly description: "Prune tool results flagged contextually useless (no matches, timed-out waits) once consumed (cache-aware)";
+        };
+    };
     readonly "snapcompact.systemPrompt": {
         readonly type: "enum";
         readonly values: readonly ["none", "agents-md", "all"];
@@ -4509,6 +4519,7 @@ export interface CompactionSettings {
     idleThresholdTokens: number;
     idleTimeoutSeconds: number;
     supersedeReads: boolean;
+    dropUseless: boolean;
 }
 export interface ContextPromotionSettings {
     enabled: boolean;

package/dist/types/tools/tool-result.d.ts

@@ -26,6 +26,8 @@ export declare class ToolResultBuilder<TDetails extends DetailsWithMeta> {
     diagnostics(summary: string, messages: string[]): this;
     /** Flag the result as a non-throwing failure (agent-loop surfaces it as a tool error). */
     error(value?: boolean): this;
+    /** Marks the result contextually useless — compaction may elide it once consumed. */
+    useless(value?: boolean): this;
     done(): AgentToolResult<TDetails>;
 }
 export declare function toolResult<TDetails extends DetailsWithMeta>(details?: TDetails): ToolResultBuilder<TDetails>;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "15.12.0",
+	"version": "15.12.2",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -47,17 +47,17 @@
 		"@agentclientprotocol/sdk": "0.22.1",
 		"@babel/parser": "^7.29.7",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/hashline": "15.12.0",
-		"@oh-my-pi/omp-stats": "15.12.0",
-		"@oh-my-pi/pi-agent-core": "15.12.0",
-		"@oh-my-pi/pi-ai": "15.12.0",
-		"@oh-my-pi/pi-catalog": "15.12.0",
-		"@oh-my-pi/pi-mnemopi": "15.12.0",
-		"@oh-my-pi/pi-natives": "15.12.0",
-		"@oh-my-pi/pi-tui": "15.12.0",
-		"@oh-my-pi/pi-utils": "15.12.0",
-		"@oh-my-pi/pi-wire": "15.12.0",
-		"@oh-my-pi/snapcompact": "15.12.0",
+		"@oh-my-pi/hashline": "15.12.2",
+		"@oh-my-pi/omp-stats": "15.12.2",
+		"@oh-my-pi/pi-agent-core": "15.12.2",
+		"@oh-my-pi/pi-ai": "15.12.2",
+		"@oh-my-pi/pi-catalog": "15.12.2",
+		"@oh-my-pi/pi-mnemopi": "15.12.2",
+		"@oh-my-pi/pi-natives": "15.12.2",
+		"@oh-my-pi/pi-tui": "15.12.2",
+		"@oh-my-pi/pi-utils": "15.12.2",
+		"@oh-my-pi/pi-wire": "15.12.2",
+		"@oh-my-pi/snapcompact": "15.12.2",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/context-async-hooks": "^2.7.1",
 		"@opentelemetry/exporter-trace-otlp-proto": "^0.218.0",

package/src/collab/host.ts

@@ -29,6 +29,7 @@ import {
 	COLLAB_PROTO,
 	type CollabFrame,
 	type CollabParticipant,
+	type CollabPromptDetails,
 	type CollabSessionState,
 	formatCollabLink,
 	formatCollabWebLink,
@@ -364,13 +365,24 @@ export class CollabHost {
 		const name = peer.name;
 		const content: string | (TextContent | ImageContent)[] =
 			images && images.length > 0 ? [{ type: "text", text }, ...images] : text;
+		const details: CollabPromptDetails & { __pendingDisplayTag?: string } = { from: name };
+		if (this.#ctx.session.isStreaming) {
+			// Mid-turn guest prompts are steered: register the pending-display twin
+			// so queuedMessageCount reflects the queued steer (host pending bar +
+			// guests' "queued ×N" badge). The tag dequeues the entry when the agent
+			// consumes the message (mirrors the skill-prompt path).
+			details.__pendingDisplayTag = this.#ctx.session.enqueueCustomMessageDisplay(text, "steer");
+			this.#ctx.updatePendingMessagesDisplay();
+			this.#ctx.ui.requestRender();
+			this.#scheduleStateBroadcast();
+		}
 		this.#ctx.session
 			.promptCustomMessage(
 				{
 					customType: COLLAB_PROMPT_MESSAGE_TYPE,
 					content,
 					display: true,
-					details: { from: name },
+					details,
 					attribution: "user",
 				},
 				{ streamingBehavior: "steer" },

package/src/collab/protocol.ts

@@ -108,11 +108,11 @@ export function rewriteEnvelopePeer(data: Uint8Array, peerId: number): void {
 }
 
 // ═══════════════════════════════════════════════════════════════════════════
-// Link format: wss://<host[:port]>/r/<roomId>#<base64url-32-byte-key>
+// Link format: wss://<host[:port]>/r/<roomId>.<base64url-32-byte-key>
 // ═══════════════════════════════════════════════════════════════════════════
 
-const ROOM_PATH_RE = /^\/r\/([A-Za-z0-9_-]{10,64})$/;
-const BARE_LINK_RE = /^([A-Za-z0-9_-]{10,64})#([A-Za-z0-9_-]+)$/;
+const ROOM_PATH_RE = /^\/r\/([A-Za-z0-9_-]{10,64})(?:\.([A-Za-z0-9_-]+))?$/;
+const BARE_LINK_RE = /^([A-Za-z0-9_-]{10,64})[#.]([A-Za-z0-9_-]+)$/;
 const B64URL_RE = /^[A-Za-z0-9_-]+$/;
 const LOCAL_HOSTNAMES: Record<string, true> = { localhost: true, "127.0.0.1": true, "::1": true, "[::1]": true };
 
@@ -152,11 +152,17 @@ function normalizeRelayOrigin(relayUrl: string): { origin: string } | { error: s
 
 /**
  * Render the shareable link. Compact forms: the default relay collapses to
- * `<roomId>#<key>`, other wss relays drop the scheme (`host[:port]/r/…`);
+ * `<roomId>.<key>`, other wss relays drop the scheme (`host[:port]/r/…`);
  * only localhost ws:// links keep their full URL so parsing cannot
  * mis-infer wss.
  *
- * Full links append the write token to the key in the fragment
+ * The room secret is dot-joined (`<roomId>.<key>`) rather than `#`-joined:
+ * RFC 3986 forbids a raw `#` inside a fragment, so strict URL stacks (macOS
+ * Foundation behind terminal click-to-open) percent-encode a second `#` to
+ * `%23` and break the link. Parsers still accept the legacy `#` form and the
+ * mangled `%23` form.
+ *
+ * Full links append the write token to the key
  * (`base64url(key ∥ writeToken)`); read-only (view) links carry the bare
  * 32-byte key, which is also the pre-token link format.
  */
@@ -165,11 +171,11 @@ export function formatCollabLink(relayUrl: string, roomId: string, key: Uint8Arr
 	if ("error" in normalized) throw new Error(normalized.error);
 	const secret = writeToken ? Buffer.concat([key, writeToken]) : Buffer.from(key);
 	const keyText = secret.toString("base64url");
-	if (normalized.origin === DEFAULT_RELAY_URL) return `${roomId}#${keyText}`;
+	if (normalized.origin === DEFAULT_RELAY_URL) return `${roomId}.${keyText}`;
 	const compact = normalized.origin.startsWith("wss://")
 		? normalized.origin.slice("wss://".length)
 		: normalized.origin;
-	return `${compact}/r/${roomId}#${keyText}`;
+	return `${compact}/r/${roomId}.${keyText}`;
 }
 
 /**
@@ -193,10 +199,12 @@ export function formatCollabWebLink(
 }
 
 export function parseCollabLink(link: string): ParsedCollabLink | { error: string } {
-	let text = link.trim();
-	// Bare `<roomId>#<key>` → default relay.
+	// Lenient input: terminals that open OSC 8 links through strict URL stacks
+	// (macOS Foundation) percent-encode the legacy second `#` to `%23`.
+	let text = link.trim().replace(/%23/gi, "#");
+	// Bare `<roomId>.<key>` (legacy `<roomId>#<key>`) → default relay.
 	const bare = BARE_LINK_RE.exec(text);
-	if (bare) text = `${DEFAULT_RELAY_URL}/r/${bare[1]}#${bare[2]}`;
+	if (bare) text = `${DEFAULT_RELAY_URL}/r/${bare[1]}.${bare[2]}`;
 	// Scheme-less `host[:port]/r/…` → wss.
 	else if (!text.includes("://")) text = `wss://${text}`;
 	let url: URL;
@@ -210,17 +218,18 @@ export function parseCollabLink(link: string): ParsedCollabLink | { error: strin
 	const match = ROOM_PATH_RE.exec(url.pathname);
 	if (!match) {
 		// Web deep link: `http(s)://<relay>/#<collab-link>` — the fragment holds
-		// the whole link (which itself contains another `#`). URL.hash spans to
-		// the end of the string, so recurse on it; the inner fragment is just
-		// the key (no `#`), which bounds the recursion.
+		// the whole link, so recurse on it. The recursion terminates because
+		// the inner text is a strict suffix of the input.
 		const inner = url.hash.startsWith("#") ? url.hash.slice(1) : url.hash;
-		if (inner.includes("#")) return parseCollabLink(inner);
+		if (inner) return parseCollabLink(inner);
 		return { error: "Collab link must contain a /r/<roomId> path" };
 	}
 	const roomId = match[1]!;
-	const fragment = url.hash.startsWith("#") ? url.hash.slice(1) : url.hash;
+	// Key rides dot-joined in the path (`/r/<roomId>.<key>`); legacy links
+	// carry it in the fragment (`/r/<roomId>#<key>`).
+	const fragment = match[2] ?? (url.hash.startsWith("#") ? url.hash.slice(1) : url.hash);
 	if (!fragment) {
-		return { error: "Collab link is missing the #<key> fragment" };
+		return { error: "Collab link is missing the <key> part" };
 	}
 	const secret = B64URL_RE.test(fragment) ? new Uint8Array(Buffer.from(fragment, "base64url")) : null;
 	if (!secret || (secret.byteLength !== ROOM_KEY_BYTES && secret.byteLength !== ROOM_KEY_BYTES + WRITE_TOKEN_BYTES)) {

package/src/config/settings-schema.ts

@@ -1132,7 +1132,7 @@ export const SETTINGS_SCHEMA = {
 	doubleEscapeAction: {
 		type: "enum",
 		values: ["branch", "tree", "none"] as const,
-		default: "tree",
+		default: "branch",
 		ui: {
 			tab: "interaction",
 			group: "Input",
@@ -1339,7 +1339,7 @@ export const SETTINGS_SCHEMA = {
 			tab: "interaction",
 			group: "Collab",
 			label: "Relay URL",
-			description: "Relay used by /collab (wss://host[:port]; self-host with the omp-collab-relay service)",
+			description: "Relay used by /collab (wss://host[:port])",
 		},
 	},
 
@@ -1621,6 +1621,18 @@ export const SETTINGS_SCHEMA = {
 		},
 	},
 
+	"compaction.dropUseless": {
+		type: "boolean",
+		default: true,
+		ui: {
+			tab: "context",
+			group: "Compaction",
+			label: "Elide Uneventful Results",
+			description:
+				"Prune tool results flagged contextually useless (no matches, timed-out waits) once consumed (cache-aware)",
+		},
+	},
+
 	// Experimental: snapcompact inline imaging (transient, per-request; never persisted)
 	"snapcompact.systemPrompt": {
 		type: "enum",
@@ -3936,9 +3948,6 @@ export const SETTINGS_SCHEMA = {
 
 	"dev.autoqaPush.endpoint": {
 		type: "string",
-		// Bundled QA collector — runs `/work/pi-www/autoqa` behind qa.omp.sh.
-		// Override via `PI_AUTO_QA_PUSH_URL` or `dev.autoqaPush.endpoint`
-		// in `config.yml` to point at a self-hosted instance.
 		default: "https://qa.omp.sh/v1/grievances" as const,
 		ui: {
 			tab: "tools",
@@ -4079,6 +4088,7 @@ export interface CompactionSettings {
 	idleThresholdTokens: number;
 	idleTimeoutSeconds: number;
 	supersedeReads: boolean;
+	dropUseless: boolean;
 }
 
 export interface ContextPromotionSettings {