skydive-cli 0.5.0-beta.14 → 0.5.0-beta.15

package/dist/js/bin.mjs

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
-import { C as ensureActiveOrganization, D as setActiveWorkspace, E as listWorkspaces, O as name, T as getSessionIdentity, a as installCrashHandler, k as version, t as maybeStartProfiling, u as brandHelpArt, w as getActiveWorkspaceId, x as themes } from "./profiler-COGO4LN8.mjs";
+import { C as ensureActiveOrganization, D as setActiveWorkspace, E as listWorkspaces, O as name, T as getSessionIdentity, a as installCrashHandler, k as version, t as maybeStartProfiling, u as brandHelpArt, w as getActiveWorkspaceId, x as themes } from "./profiler-B97UpfHm.mjs";
 import { A as getStoredApiKeyId, B as saveConfig, C as getDefaultAgent, E as getPromptHistoryPath, F as resolveChatAuth, I as resolveConfig, L as resolveManagementAuth, M as getUpdateCheckDisabled, P as resolveAppUrl, R as resolveSession, S as getConfigPath, T as getPreference, U as setLastSeenVersion, V as saveSession, _ as API_KEY_PREFIX, b as PREFERENCES, g as API_KEY_FAMILY_PREFIX, h as API_KEYS_URL, i as resolveAgent, j as getStoredApiKeyWorkspaceName, k as getShareMachineDefault, v as DEFAULT_API_URL, w as getLastSeenVersion, x as deleteConfig } from "./print-riiY5Gly.mjs";
 import { n as printError, r as printTable, t as output } from "./output-DYzzdXYV.mjs";
 import { t as HttpError } from "./http-error-DzyrsLAZ.mjs";
 import { t as createRestClient } from "./rest-CLrHVkxh.mjs";
 import "./billing-blocked-2wju4gC_.mjs";
 import { r as resolveMachineIdentity } from "./client-DbqRBquD.mjs";
-import { i as queryDaemonStatus, l as PORTAL_DAEMON_FLAG, n as ensureDaemonRunning, s as stopDaemon, u as daemonPaths } from "./daemon-CL25CwjW.mjs";
+import { i as queryDaemonStatus, l as PORTAL_DAEMON_FLAG, n as ensureDaemonRunning, s as stopDaemon, u as daemonPaths } from "./daemon-eb9dm6Ih.mjs";
 import { i as grantPortalAccess, n as fetchPortalDevices, o as registerPortalDevice, r as findThisDevice, s as revokePortalAccess } from "./api-DG5W6iwx.mjs";
 import { t as SandboxStream } from "./client-c4c5MmgN.mjs";
 import { hideBin } from "yargs/helpers";
@@ -1517,7 +1517,7 @@ const importCommand = {
 				process.exit(1);
 			}
 		}
-		const { runChat } = await import("./boot-CJKq8rvL.mjs");
+		const { runChat } = await import("./boot-DZKl8JNI.mjs");
 		await runChat({
 			appUrl,
 			sessionToken: session.value.sessionToken,
@@ -1930,7 +1930,7 @@ const chatCommand = {
 			sessionToken: auth.value.token,
 			agentSelector: argv.agent ?? null
 		});
-		const { runChat } = await import("./boot-CJKq8rvL.mjs");
+		const { runChat } = await import("./boot-DZKl8JNI.mjs");
 		await runChat({
 			appUrl: auth.value.appUrl,
 			sessionToken: auth.value.token,
@@ -2290,7 +2290,7 @@ const switchCommand = {
 				printError("The workspace picker needs the Bun runtime and it could not be set up automatically. Pass a workspace slug instead, or install Bun and retry.");
 				process.exit(1);
 			}
-			const { runWorkspacePicker } = await import("./boot-CJKq8rvL.mjs");
+			const { runWorkspacePicker } = await import("./boot-DZKl8JNI.mjs");
 			await runWorkspacePicker(session);
 			return;
 		}
@@ -2369,7 +2369,7 @@ const openCommand = {
 		const agent = argv.agent ? resolveAgent((await fetchPortalDevices(session)).agents, argv.agent) : null;
 		const cwd = argv.cwd ? path.resolve(argv.cwd) : process.cwd();
 		const { machineName } = await resolveMachineIdentity(null);
-		const { PortalDaemonClient } = await import("./daemon-client-DEyOdfdA.mjs");
+		const { PortalDaemonClient } = await import("./daemon-client-Bemq-pLd.mjs");
 		let lastLine = "";
 		let signalConnected;
 		const connected = new Promise((resolve) => {
@@ -4215,7 +4215,7 @@ if (process.argv.includes(UPDATE_WORKER_FLAG)) {
 	process.exit(0);
 }
 if (process.argv.includes(PORTAL_DAEMON_FLAG)) {
-	const { runPortalDaemon } = await import("./daemon-DjsUMczp.mjs");
+	const { runPortalDaemon } = await import("./daemon-Djei8xaW.mjs");
 	runPortalDaemon(process.argv);
 } else runCli();
 function runCli() {

package/dist/js/{boot-CJKq8rvL.mjs → boot-DZKl8JNI.mjs}

@@ -1,14 +1,14 @@
 #!/usr/bin/env node
-import { D as setActiveWorkspace, E as listWorkspaces, S as themesForMode, _ as themeForMode, a as installCrashHandler, b as themeVersion, c as MARK_CELLS, d as DEFAULT_THEME_ID, f as applyTheme, g as theme, h as noColorRequested, i as writeArtifact, l as WORDMARK, m as monoTheme, n as profilingEnabled, o as buildCrashReport, p as findTheme, r as record, s as writeCrashReport, v as themeMode, w as getActiveWorkspaceId, y as themeModeFromColorFgBg } from "./profiler-COGO4LN8.mjs";
+import { D as setActiveWorkspace, E as listWorkspaces, S as themesForMode, _ as themeForMode, a as installCrashHandler, b as themeVersion, c as MARK_CELLS, d as DEFAULT_THEME_ID, f as applyTheme, g as theme, h as noColorRequested, i as writeArtifact, l as WORDMARK, m as monoTheme, n as profilingEnabled, o as buildCrashReport, p as findTheme, r as record, s as writeCrashReport, v as themeMode, w as getActiveWorkspaceId, y as themeModeFromColorFgBg } from "./profiler-B97UpfHm.mjs";
 import { D as getReviewStateDir, H as saveTheme, N as recordDefaultAgent, O as getSavedTheme, S as getConfigPath, c as cardActionErrorMessage, d as reconcileMaskedInput, f as resolveConnectUrl, i as resolveAgent, l as parseExternalOauthConnectParams, m as specKeyFor, p as parseConnectCard, s as MASK_CHAR, u as parseOauthConnectParams, v as DEFAULT_API_URL, y as DEFAULT_APP_URL, z as resolveWebUrl } from "./print-riiY5Gly.mjs";
 import { t as HttpError } from "./http-error-DzyrsLAZ.mjs";
 import { a as isRecord, i as errorMessage, n as errorDetail, r as sendErrorMessage, t as createRestClient } from "./rest-CLrHVkxh.mjs";
 import { i as billingBlockedOutcomeFromSendResponse } from "./billing-blocked-2wju4gC_.mjs";
 import { t as PortalClient } from "./client-DbqRBquD.mjs";
-import "./daemon-CL25CwjW.mjs";
+import "./daemon-eb9dm6Ih.mjs";
 import "./api-DG5W6iwx.mjs";
 import { t as SandboxStream } from "./client-c4c5MmgN.mjs";
-import { t as PortalDaemonClient } from "./daemon-client-k8ICifN2.mjs";
+import { t as PortalDaemonClient } from "./daemon-client-N8rvGusv.mjs";
 import { t as runRawPtyPassthrough } from "./raw-pty-DY4KelZW.mjs";
 import * as os$1 from "node:os";
 import { homedir, platform, release, tmpdir } from "node:os";
@@ -1080,42 +1080,133 @@ function createAgentHost({ renderer, notifications }) {
 * take the one host we can identify unambiguously (github.com, incl. `www.`),
 * which covers the common product case; GHES stays the native badge's job.
 *
-* Only the `/pull/<n>` form (not `/issues/`, not a bare `#123` — ambiguous with
-* issues without local repo context we don't have). A trailing path/query/
-* fragment (e.g. `/files`, `#discussion`) is allowed but not captured into the
-* canonical URL. Also matches inside markdown links, backticks, and trailing
-* punctuation.
-*/
-const PR_URL_RE = /https?:\/\/(?:www\.)?github\.com\/([\w.-]+)\/([\w.-]+)\/pull\/(\d+)/gi;
-/**
-* Extracts every distinct GitHub PR referenced in a block of text, in first-seen
-* order, de-duplicated by **repo-qualified identity** (`owner/repo#number`) —
-* NOT by number, since `#42` can exist in many repos. Each ref carries a
-* canonicalized URL (scheme + host + `/owner/repo/pull/number`, no trailing
-* path/query/fragment) so the same PR linked twice with different fragments
-* collapses to one pill.
-*/
-function extractPullRequests(text, options) {
-	const requireCompleteNumber = options?.requireCompleteNumber ?? true;
-	const byKey = /* @__PURE__ */ new Map();
-	for (const match of text.matchAll(PR_URL_RE)) {
+* Only the `/pull/<n>` form (not `/issues/`). A trailing path/query/fragment
+* (e.g. `/files`, `#discussion`) is allowed but not captured into the canonical
+* URL. Also matches inside markdown links, backticks, and trailing punctuation.
+*
+* The `https://` (and `www.`) prefix is optional: agents and people routinely
+* write a bare `github.com/o/r/pull/7`, and that should link a PR the same way
+* the full URL does. To keep a bare host from matching inside a longer hostname
+* (`mygithub.com/...`, `notgithub.com/...`), a protocol-less match must be
+* preceded by a start-of-string or a non-host character (anything but a word
+* char or `.`) — a leading boundary the `(?:^|[^\w.])` group asserts and
+* consumes. The consumed boundary char is outside the captured groups, so it
+* never leaks into owner/repo. `matchAll` won't overlap, but the boundary is a
+* single delimiter (space, `(`, `[`, etc.) between PRs in practice, so distinct
+* back-to-back URLs still each match.
+*/
+const PR_URL_RE = /(?:^|[^\w.])(?:https?:\/\/)?(?:www\.)?github\.com\/([\w.-]+)\/([\w.-]+)\/pull\/(\d+)/gi;
+/**
+* Matches a bare, human-style PR reference `#<n>` — the way people (and LLMs)
+* name a PR in prose ("opened #20858", "landed #42"). Requires a non-word
+* boundary before the `#` so a fragment tail like `/pull/7#discussion_r5` or a
+* hex color `#20858` embedded in a word doesn't count, and captures the digits
+* up to a word boundary. We only harvest these from **assistant prose**, never
+* from tool output, because `#N` is exactly the relevance signal we're after:
+* the agent chose to talk about this PR.
+*/
+const PR_HASH_RE = /(?:^|[^\w#/])#(\d+)\b/g;
+/**
+* Every full `/pull/<n>` github.com URL in `urlText`, keyed by number to the
+* candidate refs for that number. A number can map to more than one repo, so we
+* keep all candidates; a bare `#N` label can't say which repo, but the pill's
+* own URL does. Deduped by repo-qualified identity (`owner/repo#number`) so the
+* same PR linked twice — even with a different trailing path/fragment —
+* collapses to one ref.
+*
+* `deferTrailingNumber` guards the streaming truncation case: when true, a URL
+* whose number runs to the very end of `urlText` is skipped because the next
+* delta may extend those digits (`.../pull/19` now, `.../pull/19739` later).
+* Only the still-growing PROSE buffer sets this; settled tool output is always
+* complete, so its URLs are harvested with the guard off.
+*/
+function urlsByNumber(urlText, deferTrailingNumber, byNumber) {
+	for (const match of urlText.matchAll(PR_URL_RE)) {
 		const owner = match[1];
 		const repo = match[2];
 		const numberText = match[3];
 		if (!owner || !repo || !numberText) continue;
-		if (requireCompleteNumber) {
-			if (match.index + match[0].length >= text.length) continue;
+		if (deferTrailingNumber) {
+			if (match.index + match[0].length >= urlText.length) continue;
 		}
 		const number = Number.parseInt(numberText, 10);
 		if (!Number.isFinite(number) || number <= 0) continue;
-		const key = `${owner.toLowerCase()}/${repo.toLowerCase()}#${number}`;
-		if (byKey.has(key)) continue;
-		byKey.set(key, {
+		const ref = {
 			number,
-			key,
+			key: `${owner.toLowerCase()}/${repo.toLowerCase()}#${number}`,
 			url: `https://github.com/${owner}/${repo}/pull/${number}`
-		});
+		};
+		const existing = byNumber.get(number);
+		if (existing) {
+			if (!existing.some((r) => r.key === ref.key)) existing.push(ref);
+		} else byNumber.set(number, [ref]);
 	}
+}
+/** Every human-style `#N` named in `proseText` — the relevance signal. */
+function hashNumbers(proseText) {
+	const numbers = /* @__PURE__ */ new Set();
+	for (const match of proseText.matchAll(PR_HASH_RE)) {
+		const numberText = match[1];
+		if (!numberText) continue;
+		const number = Number.parseInt(numberText, 10);
+		if (Number.isFinite(number) && number > 0) numbers.add(number);
+	}
+	return numbers;
+}
+/**
+* The relevance contract, in one place.
+*
+* A GitHub PR URL is cheap to emit and shows up in bulk any time a tool dumps
+* PRs the conversation isn't actually about — `gh pr list`, `gh pr status`, a
+* `gh search` — printing dozens of `/pull/N` URLs the agent never discusses.
+* Pinning a sidebar pill for every one of those is noise.
+*
+* So a PR is surfaced only when BOTH hold:
+*   1. a fully-qualified `github.com/<o>/<r>/pull/<n>` URL exists — this may
+*      come from the assistant's prose OR from tool output (a `gh pr create`
+*      that printed the URL the agent then referenced by number), and
+*   2. the **assistant's prose** names the number as a human-style `#<n>`.
+*
+* The `#N` in prose is the intent signal — "the agent is talking about this
+* PR", as opposed to "a tool happened to list it". The full URL (from wherever)
+* proves the PR is real and pins down its repo + canonical link. A URL with no
+* `#N` in prose (the whole of a `gh pr list` dump) is dropped; a `#N` with no
+* matching URL anywhere (a bare issue-style ref) is dropped.
+*
+* `proseText` is the assistant's prose alone (the only source of the `#N`
+* relevance signal); `urlOnlyText` is settled tool output that may carry full
+* PR URLs but no `#N` intent. URLs are harvested from both; `#N` from prose
+* only.
+*/
+function collectRelevant(proseText, urlOnlyText, options, byKey) {
+	const byNumber = /* @__PURE__ */ new Map();
+	urlsByNumber(proseText, options.requireCompleteNumber, byNumber);
+	if (urlOnlyText) urlsByNumber(urlOnlyText, false, byNumber);
+	if (byNumber.size === 0) return;
+	const named = hashNumbers(proseText);
+	if (named.size === 0) return;
+	for (const [number, refs] of byNumber) {
+		if (!named.has(number)) continue;
+		for (const ref of refs) if (!byKey.has(ref.key)) byKey.set(ref.key, ref);
+	}
+}
+/**
+* Extracts every distinct GitHub PR that is both linked (a full `/pull/<n>`
+* URL) AND named (`#<n>`), de-duplicated by **repo-qualified identity**
+* (`owner/repo#number`) — NOT by number, since `#42` can exist in many repos.
+*
+* The two-argument form takes the URL pool and the prose separately, so a URL
+* from tool output can satisfy the link requirement while the `#N` stays a
+* prose-only signal. The one-argument form (prose only) is the common case: it
+* requires the URL to be in the same prose that names it.
+*
+* See {@link collectRelevant} for the contract.
+*/
+function extractPullRequests(proseText, options) {
+	const requireCompleteNumber = options?.requireCompleteNumber ?? true;
+	const urlOnly = options?.urlOnlyText ?? "";
+	const byKey = /* @__PURE__ */ new Map();
+	collectRelevant(proseText, urlOnly, { requireCompleteNumber }, byKey);
 	return [...byKey.values()];
 }
 /**
@@ -1154,28 +1245,34 @@ function stringifyToolOutput(value) {
 }
 /**
 * A streaming PR scanner: fed run chunks as they stream, it reports each
-* distinct PR the moment its URL is seen, exactly once, without ever rescanning
-* the transcript. Two sources feed it:
+* distinct PR the moment BOTH conditions hold — a full `/pull/<n>` URL has been
+* seen (in prose OR tool output) AND the assistant's prose names it `#<n>` —
+* exactly once, without ever rescanning the transcript.
 *
-* - **Assistant text** (`text-delta`): accumulated per part id (a URL can split
-*   across two deltas) and only the current part's buffer is scanned, so cost
-*   is bounded by one text block, not the whole scrollback.
-* - **Tool output** (`tool-output-available`): the settled output of a tool
-*   call (e.g. `gh pr create` stdout) arrives whole, not delta-streamed, so we
-*   flatten and scan it in one pass. This catches PRs a tool printed that the
-*   agent never restated in prose — the common case for an opened PR.
+* Two sources feed it, with different roles:
+* - **Assistant text** (`text-delta`): the ONLY source of the `#N` relevance
+*   signal, and also a source of URLs. Accumulated per part id (a URL or `#N`
+*   can split across two deltas); only the current part's buffer is scanned.
+* - **Tool output** (`tool-output-available`): a URL-only source. A
+*   `gh pr create` prints the URL the agent then references by number, so the
+*   URL for a #N mention often lives here, not in the prose. Tool output can
+*   NEVER originate a pill on its own — a `gh pr list` dump (URLs, no `#N`)
+*   stays silent — but it satisfies the URL half for a PR the prose names.
 *
 * `onPullRequest` fires once per never-before-seen PR (keyed by
-* `owner/repo#number`) for the life of the scanner, regardless of which source
-* it came from. Construct one per run (or per conversation) and feed it the
-* same chunks the transcript reducer sees.
+* `owner/repo#number`). Construct one per run (or per conversation) and feed it
+* the same chunks the transcript reducer sees. See {@link collectRelevant}.
 */
 function createPullRequestScanner(onPullRequest) {
 	let text = "";
 	let lastPartId;
+	let toolUrlText = "";
 	const seen = /* @__PURE__ */ new Set();
-	const report = (buffer, final) => {
-		for (const pr of extractPullRequests(buffer, { requireCompleteNumber: !final })) {
+	const report = (final) => {
+		for (const pr of extractPullRequests(text, {
+			requireCompleteNumber: !final,
+			urlOnlyText: toolUrlText
+		})) {
 			if (seen.has(pr.key)) continue;
 			seen.add(pr.key);
 			onPullRequest(pr);
@@ -1185,7 +1282,8 @@ function createPullRequestScanner(onPullRequest) {
 		if (chunk["type"] === "tool-output-available") {
 			const outputText = stringifyToolOutput(chunk["output"]);
 			if (!outputText.includes("/pull/")) return;
-			report(outputText, true);
+			toolUrlText += `\n${outputText}`;
+			report(true);
 			return;
 		}
 		if (chunk["type"] !== "text-delta") return;
@@ -1193,13 +1291,12 @@ function createPullRequestScanner(onPullRequest) {
 		if (!delta) return;
 		const partId = typeof chunk["id"] === "string" ? chunk["id"] : void 0;
 		if (partId && lastPartId && partId !== lastPartId) {
-			if (text.includes("/pull/")) report(text, true);
+			report(true);
 			text = "";
 		}
 		text += delta;
 		lastPartId = partId ?? lastPartId;
-		if (!text.includes("/pull/")) return;
-		report(text, false);
+		report(false);
 	} };
 }
 
@@ -10886,10 +10983,13 @@ function ChatScreen({ agent, conversation }) {
 					text: recapText
 				});
 				setItems(loaded);
+				const toolUrlText = loaded.map((item) => item.kind === "tool" && item.output?.kind === "ok" ? stringifyToolOutput(item.output.value) : "").join("\n");
 				for (const item of loaded) {
-					const text = item.kind === "assistant-text" ? item.text : item.kind === "tool" && item.output?.kind === "ok" ? stringifyToolOutput(item.output.value) : null;
-					if (text === null) continue;
-					for (const pr of extractPullRequests(text, { requireCompleteNumber: false })) {
+					if (item.kind !== "assistant-text") continue;
+					for (const pr of extractPullRequests(item.text, {
+						requireCompleteNumber: false,
+						urlOnlyText: toolUrlText
+					})) {
 						if (reportedPrsRef.current.has(pr.key)) continue;
 						reportedPrsRef.current.set(pr.key, pr);
 						setPrLinks((prev) => [...prev, pr]);

package/dist/js/{daemon-DjsUMczp.mjs → daemon-Djei8xaW.mjs}

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import "./client-DbqRBquD.mjs";
-import { a as runPortalDaemon, i as queryDaemonStatus, n as ensureDaemonRunning, o as startPortalDaemon, r as isDaemonListening, s as stopDaemon, t as PortalDaemon } from "./daemon-CL25CwjW.mjs";
+import { a as runPortalDaemon, i as queryDaemonStatus, n as ensureDaemonRunning, o as startPortalDaemon, r as isDaemonListening, s as stopDaemon, t as PortalDaemon } from "./daemon-eb9dm6Ih.mjs";
 import "./api-DG5W6iwx.mjs";
 
 export { runPortalDaemon };

package/dist/js/daemon-client-Bemq-pLd.mjs

@@ -0,0 +1,7 @@
+#!/usr/bin/env node
+import "./client-DbqRBquD.mjs";
+import "./daemon-eb9dm6Ih.mjs";
+import "./api-DG5W6iwx.mjs";
+import { t as PortalDaemonClient } from "./daemon-client-N8rvGusv.mjs";
+
+export { PortalDaemonClient };

package/dist/js/{daemon-client-k8ICifN2.mjs → daemon-client-N8rvGusv.mjs}

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { c as LOCAL_PROTOCOL_VERSION, d as encodeLine, f as makeLineParser, n as ensureDaemonRunning, p as parseDaemonMessage, u as daemonPaths } from "./daemon-CL25CwjW.mjs";
+import { c as LOCAL_PROTOCOL_VERSION, d as encodeLine, f as makeLineParser, n as ensureDaemonRunning, p as parseDaemonMessage, u as daemonPaths } from "./daemon-eb9dm6Ih.mjs";
 import { randomUUID } from "node:crypto";
 import { connect } from "node:net";
 

package/dist/js/{daemon-CL25CwjW.mjs → daemon-eb9dm6Ih.mjs}

@@ -20,7 +20,7 @@ import { access, appendFile, mkdir, readFile, unlink, writeFile } from "node:fs/
 * the commit time of the built tree is one monotonic clock they share.
 */
 function portalDaemonBuild() {
-	return "1786604432";
+	return "1786605017";
 }
 /**
 * Whether a client carrying `mine` should replace a running daemon carrying

package/dist/js/{profiler-COGO4LN8.mjs → profiler-B97UpfHm.mjs}

@@ -8,7 +8,7 @@ import fs from "node:fs";
 
 //#region package.json
 var name = "skydive-cli";
-var version$1 = "0.5.0-beta.14";
+var version$1 = "0.5.0-beta.15";
 
 //#endregion
 //#region src/auth/organization.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skydive-cli",
-  "version": "0.5.0-beta.14",
+  "version": "0.5.0-beta.15",
   "description": "Skydive CLI — cloud agents from the command line",
   "homepage": "https://skydive.com",
   "license": "MIT",

package/dist/js/daemon-client-DEyOdfdA.mjs

@@ -1,7 +0,0 @@
-#!/usr/bin/env node
-import "./client-DbqRBquD.mjs";
-import "./daemon-CL25CwjW.mjs";
-import "./api-DG5W6iwx.mjs";
-import { t as PortalDaemonClient } from "./daemon-client-k8ICifN2.mjs";
-
-export { PortalDaemonClient };