@bytetrue/byspace 0.15.0 → 0.16.1

package/dist/commands/agent/open.js

@@ -1,5 +1,5 @@
 import { buildDaemonConnectionCommandError, connectToDaemon } from "../../utils/client.js";
-import { openDesktopWithAgent } from "../open.js";
+import { openAgentInWebApp } from "../open.js";
 const openAgentSchema = {
     idField: "agentId",
     columns: [
@@ -10,7 +10,7 @@ const openAgentSchema = {
 };
 export function addOpenOptions(command) {
     return command
-        .description("Open an existing agent in BySpace Desktop")
+        .description("Open an existing agent in the BySpace web app")
         .argument("<agent-id>", "Existing agent ID")
         .option("--server <server-id>", "Server ID (defaults to the local daemon)");
 }
@@ -51,7 +51,7 @@ export async function runOpenCommand(agentIdArg, options, _command) {
         throw error;
     }
     const serverId = await resolveServerId(options);
-    await openDesktopWithAgent({ serverId, agentId });
+    await openAgentInWebApp({ serverId, agentId }, { host: options.host });
     return {
         type: "single",
         data: { agentId, serverId, status: "opened" },

package/dist/commands/onboard.js

@@ -106,9 +106,8 @@ function printNextSteps(pairingUrl, byspaceHome, richUi) {
             ? "1. Open BySpace and scan the QR code above, or paste the pairing link."
             : "1. Open BySpace and connect to your daemon.",
         `2. Web app: ${appBaseUrl}`,
-        "3. Desktop app: https://github.com/ByteTrue/byspace/releases/latest",
-        "4. Docs: https://github.com/ByteTrue/byspace",
-        '5. Example: byspace run --output-schema schema.json "extract fields"',
+        "3. Docs: https://github.com/ByteTrue/byspace",
+        '4. Example: byspace run --output-schema schema.json "extract fields"',
     ];
     const quickReferenceLines = [
         "1. byspace --help",

package/dist/commands/open.d.ts

@@ -1,4 +1,19 @@
 import { type AgentDeepLinkTarget } from "@bytetrue/protocol/agent-deep-link";
-export declare function openDesktopWithProject(projectPath: string): Promise<void>;
-export declare function openDesktopWithAgent(target: AgentDeepLinkTarget): Promise<void>;
+/**
+ * The daemon serves its bundled web UI on the same TCP port as `/ws`, so a TCP
+ * daemon host doubles as its HTTP origin. Unix sockets, SSH tunnels, and relay
+ * offers have no address a browser on this machine can reach.
+ */
+export declare function resolveWebUiOrigin(rawHost: string): string | null;
+/**
+ * Mirrors `buildHostWorkspaceRoute` in `packages/app/src/utils/host-routes.ts`.
+ * Workspace IDs are generated as `wks_<hex>` (packages/server/src/server/workspace-registry-model.ts),
+ * so percent-encoding always suffices; the app's base64 path segment form only
+ * exists for pre-v0.1.95 path-shaped IDs, which this fork does not carry.
+ */
+export declare function buildWorkspaceWebRoute(serverId: string, workspaceId: string): string;
+export declare function openProjectInWebApp(projectPath: string): Promise<void>;
+export declare function openAgentInWebApp(target: AgentDeepLinkTarget, options?: {
+    host?: string;
+}): Promise<void>;
 //# sourceMappingURL=open.d.ts.map

package/dist/commands/open.js

@@ -1,89 +1,112 @@
-import { existsSync } from "node:fs";
-import { homedir } from "node:os";
-import path from "node:path";
 import { spawnProcess } from "@bytetrue/server";
-import { buildAgentDeepLink } from "@bytetrue/protocol/agent-deep-link";
-function findDesktopApp() {
-    if (process.platform === "darwin") {
-        const candidates = [
-            "/Applications/BySpace.app",
-            path.join(homedir(), "Applications", "BySpace.app"),
-        ];
-        for (const candidate of candidates) {
-            if (existsSync(candidate)) {
-                return candidate;
-            }
-        }
+import { buildAgentDeepLinkRoute, } from "@bytetrue/protocol/agent-deep-link";
+import { resolveLocalDaemonState, resolveTcpHostFromListen } from "./daemon/local-daemon.js";
+import { buildDaemonConnectionCommandError, connectToDaemon, getDaemonHost, getExplicitDaemonHost, } from "../utils/client.js";
+/**
+ * The daemon serves its bundled web UI on the same TCP port as `/ws`, so a TCP
+ * daemon host doubles as its HTTP origin. Unix sockets, SSH tunnels, and relay
+ * offers have no address a browser on this machine can reach.
+ */
+export function resolveWebUiOrigin(rawHost) {
+    const trimmed = rawHost.trim();
+    if (!trimmed || trimmed.startsWith("ssh://")) {
         return null;
     }
-    if (process.platform === "linux") {
-        const candidates = [
-            "/usr/bin/BySpace",
-            "/opt/BySpace/BySpace",
-            path.join(homedir(), "Applications", "BySpace.AppImage"),
-        ];
-        for (const candidate of candidates) {
-            if (existsSync(candidate)) {
-                return candidate;
-            }
-        }
+    const endpoint = trimmed.startsWith("tcp://") ? trimmed.slice("tcp://".length) : trimmed;
+    const match = /^\[?([^\]/]+?)]?:(\d+)$/.exec(endpoint);
+    if (!match) {
         return null;
     }
-    if (process.platform === "win32") {
-        const localAppData = process.env.LOCALAPPDATA;
-        if (!localAppData) {
-            return null;
-        }
-        const candidate = path.join(localAppData, "Programs", "BySpace", "BySpace.exe");
-        return existsSync(candidate) ? candidate : null;
+    const [, rawHostname, port] = match;
+    const hostname = rawHostname === "0.0.0.0" ? "127.0.0.1" : rawHostname;
+    const hostPart = hostname.includes(":") ? `[${hostname}]` : hostname;
+    return `http://${hostPart}:${port}`;
+}
+function resolveLocalWebUiOrigin() {
+    const listen = resolveLocalDaemonState().listen;
+    const host = resolveTcpHostFromListen(listen) ?? listen;
+    return { origin: resolveWebUiOrigin(host), host };
+}
+function resolveWebUiOriginForHost(host) {
+    const explicitHost = getExplicitDaemonHost(host);
+    if (!explicitHost) {
+        return resolveLocalWebUiOrigin();
     }
-    return null;
+    return { origin: resolveWebUiOrigin(explicitHost), host: explicitHost };
 }
-function cleanEnvForDesktopLaunch() {
-    const env = { ...process.env };
-    // The CLI runs via ELECTRON_RUN_AS_NODE=1. On Linux/Windows the spawned
-    // desktop process inherits the env directly, so we must strip it or the
-    // desktop app would start as a bare Node process instead of Electron.
-    delete env.ELECTRON_RUN_AS_NODE;
-    delete env.ELECTRON_NO_ATTACH_CONSOLE;
-    delete env.BYSPACE_NODE_ENV;
-    return env;
+/**
+ * Mirrors `buildHostWorkspaceRoute` in `packages/app/src/utils/host-routes.ts`.
+ * Workspace IDs are generated as `wks_<hex>` (packages/server/src/server/workspace-registry-model.ts),
+ * so percent-encoding always suffices; the app's base64 path segment form only
+ * exists for pre-v0.1.95 path-shaped IDs, which this fork does not carry.
+ */
+export function buildWorkspaceWebRoute(serverId, workspaceId) {
+    return `/h/${encodeURIComponent(serverId)}/workspace/${encodeURIComponent(workspaceId)}`;
+}
+function openInBrowser(url) {
+    let command;
+    let args;
+    if (process.platform === "darwin") {
+        command = "open";
+        args = [url];
+    }
+    else if (process.platform === "win32") {
+        command = process.env.ComSpec ?? "cmd";
+        args = ["/c", "start", "", url];
+    }
+    else {
+        command = "xdg-open";
+        args = [url];
+    }
+    spawnProcess(command, args, { detached: true, stdio: "ignore", shell: false }).unref();
+    process.stderr.write(`${url}\n`);
 }
-function spawnDetached(command, args) {
-    spawnProcess(command, args, {
-        detached: true,
-        stdio: "ignore",
-        env: cleanEnvForDesktopLaunch(),
-    }).unref();
+function reportWebUiUnavailable(host) {
+    process.stderr.write(`Cannot open the web app for ${host}: the daemon has no HTTP endpoint reachable from here.\n` +
+        "Start it with `byspace daemon start` and open the Web UI URL it prints.\n");
+    process.exitCode = 1;
 }
-function launchDesktop(args) {
-    if (process.env.BYSPACE_DESKTOP_CLI === "1") {
-        throw new Error("Cannot open BySpace Desktop while running in desktop CLI passthrough mode.");
+export async function openProjectInWebApp(projectPath) {
+    const { origin, host } = resolveLocalWebUiOrigin();
+    if (!origin) {
+        reportWebUiUnavailable(host);
+        return;
     }
-    const desktopApp = findDesktopApp();
-    if (!desktopApp) {
-        throw new Error("BySpace desktop app not found. Install it from https://github.com/ByteTrue/byspace/releases");
+    let client;
+    try {
+        client = await connectToDaemon();
     }
-    if (process.platform === "darwin") {
-        // -n forces a new instance even if the app is already running. The new
-        // instance relays its argv to the existing one through Electron's
-        // single-instance lock. -g keeps the terminal in the foreground.
-        spawnDetached("open", ["-n", "-g", "-a", desktopApp, "--args", ...args]);
+    catch (error) {
+        const failure = buildDaemonConnectionCommandError({ error });
+        process.stderr.write(`${failure.message}\n${failure.details}\n`);
+        process.exitCode = 1;
         return;
     }
-    spawnDetached(desktopApp, args);
-}
-export async function openDesktopWithProject(projectPath) {
     try {
-        launchDesktop([projectPath]);
+        const serverId = client.getLastServerInfoMessage()?.serverId.trim();
+        if (!serverId) {
+            throw new Error("The daemon did not report a server ID.");
+        }
+        const result = await client.openProject(projectPath);
+        if (result.error || !result.workspace) {
+            throw new Error(result.error ?? `The daemon did not open a workspace for ${projectPath}`);
+        }
+        openInBrowser(`${origin}${buildWorkspaceWebRoute(serverId, result.workspace.id)}`);
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        process.stderr.write(`${message}\n`);
+        process.stderr.write(`${error instanceof Error ? error.message : String(error)}\n`);
         process.exitCode = 1;
     }
+    finally {
+        await client.close().catch(() => { });
+    }
 }
-export async function openDesktopWithAgent(target) {
-    launchDesktop([buildAgentDeepLink(target)]);
+export async function openAgentInWebApp(target, options = {}) {
+    const { origin, host } = resolveWebUiOriginForHost(options.host);
+    if (!origin) {
+        reportWebUiUnavailable(host || getDaemonHost());
+        return;
+    }
+    openInBrowser(`${origin}${buildAgentDeepLinkRoute(target)}`);
 }
 //# sourceMappingURL=open.js.map

package/dist/run.js

@@ -1,6 +1,6 @@
 import { createCli } from "./cli.js";
 import { classifyInvocation } from "./classify.js";
-import { openDesktopWithProject } from "./commands/open.js";
+import { openProjectInWebApp } from "./commands/open.js";
 export function createCliParseArgv(input) {
     const program = createCli();
     const knownCommands = new Set(program.commands.map((command) => command.name()));
@@ -30,7 +30,7 @@ export async function runCli(argv, options = {}) {
         nodeArgv: options.nodeArgv,
     });
     if (!Array.isArray(parseArgv)) {
-        await openDesktopWithProject(parseArgv.resolvedPath);
+        await openProjectInWebApp(parseArgv.resolvedPath);
         return typeof process.exitCode === "number" ? process.exitCode : 0;
     }
     const program = createCli();

package/node_modules/@bytetrue/client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/client",
-  "version": "0.15.0",
+  "version": "0.16.1",
   "description": "BySpace client SDK package",
   "files": [
     "dist",
@@ -39,8 +39,8 @@
     "test": "vitest run"
   },
   "dependencies": {
-    "@bytetrue/protocol": "0.15.0",
-    "@bytetrue/relay": "0.15.0"
+    "@bytetrue/protocol": "0.16.1",
+    "@bytetrue/relay": "0.16.1"
   },
   "devDependencies": {
     "@types/node": "^20.9.0",

package/node_modules/@bytetrue/highlight/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/highlight",
-  "version": "0.15.0",
+  "version": "0.16.1",
   "files": [
     "dist",
     "!dist/**/*.map",

package/node_modules/@bytetrue/protocol/dist/agent-deep-link.d.ts

@@ -3,6 +3,4 @@ export interface AgentDeepLinkTarget {
     agentId: string;
 }
 export declare function buildAgentDeepLinkRoute(target: AgentDeepLinkTarget): `/h/${string}/agent/${string}`;
-export declare function buildAgentDeepLink(target: AgentDeepLinkTarget): string;
-export declare function parseAgentDeepLink(input: string): AgentDeepLinkTarget | null;
 //# sourceMappingURL=agent-deep-link.d.ts.map

package/node_modules/@bytetrue/protocol/dist/agent-deep-link.js

@@ -13,37 +13,4 @@ export function buildAgentDeepLinkRoute(target) {
     const { serverId, agentId } = normalizeAgentDeepLinkTarget(target);
     return `/h/${encodeURIComponent(serverId)}/agent/${encodeURIComponent(agentId)}`;
 }
-export function buildAgentDeepLink(target) {
-    return `byspace:/${buildAgentDeepLinkRoute(target)}`;
-}
-export function parseAgentDeepLink(input) {
-    let url;
-    try {
-        url = new URL(input);
-    }
-    catch {
-        return null;
-    }
-    if (url.protocol !== "byspace:" ||
-        url.hostname !== "h" ||
-        url.username ||
-        url.password ||
-        url.port ||
-        url.search ||
-        url.hash) {
-        return null;
-    }
-    const segments = url.pathname.split("/").filter(Boolean);
-    if (segments.length !== 3 || segments[1] !== "agent") {
-        return null;
-    }
-    try {
-        const serverId = normalizeSegment(decodeURIComponent(segments[0] ?? ""));
-        const agentId = normalizeSegment(decodeURIComponent(segments[2] ?? ""));
-        return serverId && agentId ? { serverId, agentId } : null;
-    }
-    catch {
-        return null;
-    }
-}
 //# sourceMappingURL=agent-deep-link.js.map

package/node_modules/@bytetrue/protocol/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/protocol",
-  "version": "0.15.0",
+  "version": "0.16.1",
   "description": "BySpace shared protocol schemas and wire types",
   "files": [
     "dist",

package/node_modules/@bytetrue/relay/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytetrue/relay",
-  "version": "0.15.0",
+  "version": "0.16.1",
   "description": "BySpace relay for bridging daemon and client connections",
   "files": [
     "dist",

package/node_modules/@bytetrue/server/dist/server/server/byspace-env.d.ts

@@ -1,3 +1,4 @@
+export declare const PI_TERMINAL_HOOK_OWNER_PID_ENV_KEY = "BYSPACE_PI_TERMINAL_HOOK_OWNER_PID";
 export type BySpaceNodeEnv = "development" | "production" | "test";
 export type ProcessEnvRecord = Record<string, string | undefined>;
 export type ExternalProcessEnv = NodeJS.ProcessEnv & Record<string, string>;

package/node_modules/@bytetrue/server/dist/server/server/byspace-env.js

@@ -1,5 +1,7 @@
 const BYSPACE_NODE_ENV = "BYSPACE_NODE_ENV";
 const ELECTRON_RUN_AS_NODE = "ELECTRON_RUN_AS_NODE";
+// Shared with the installed Pi extension (which cannot import from here).
+export const PI_TERMINAL_HOOK_OWNER_PID_ENV_KEY = "BYSPACE_PI_TERMINAL_HOOK_OWNER_PID";
 const RUNTIME_CONTROL_ENV_KEYS = [
     BYSPACE_NODE_ENV,
     "BYSPACE_DESKTOP_MANAGED",
@@ -7,6 +9,13 @@ const RUNTIME_CONTROL_ENV_KEYS = [
     ELECTRON_RUN_AS_NODE,
     "ELECTRON_NO_ATTACH_CONSOLE",
     "ESBUILD_BINARY_PATH",
+    // Written into `process.env` by the Pi terminal-activity extension so child Pi
+    // processes do not register a second reporter for the same terminal. If it
+    // survives into a fresh terminal or agent, that Pi process sees a marker whose
+    // value is a stale pid, concludes it is an inherited child, and registers no
+    // handlers at all — the terminal then never reports running/idle. It is
+    // per-process state, never inherited. See docs/terminal-activity.md.
+    PI_TERMINAL_HOOK_OWNER_PID_ENV_KEY,
 ];
 function buildInternalProcessEnv(baseEnv) {
     return { ...baseEnv };

package/node_modules/@bytetrue/server/dist/server/terminal/agent-hooks/pi/pi-extension.d.ts

@@ -1,2 +1,2 @@
-export declare const PI_TERMINAL_ACTIVITY_EXTENSION_SOURCE = "// byspace.pi-terminal-activity\nconst ownerPidKey = \"BYSPACE_PI_TERMINAL_HOOK_OWNER_PID\";\n\n// Keep one request in flight and only the latest state waiting behind it.\nexport function createReportQueue(sendReport) {\n  let pendingReport;\n  let reporting;\n\n  async function flushReports() {\n    while (pendingReport) {\n      const next = pendingReport;\n      pendingReport = undefined;\n      try {\n        await sendReport(next);\n      } catch {}\n    }\n    reporting = undefined;\n  }\n\n  return {\n    enqueue(next) {\n      pendingReport = next;\n      reporting ??= Promise.resolve().then(flushReports);\n      return reporting;\n    },\n    wait() {\n      return reporting ?? Promise.resolve();\n    },\n  };\n}\n\nasync function sendReport(next) {\n  await fetch(next.url, {\n    method: \"POST\",\n    headers: { \"content-type\": \"application/json\" },\n    body: JSON.stringify({\n      terminalId: next.terminalId,\n      token: next.token,\n      state: next.state,\n    }),\n    signal: AbortSignal.timeout(1_000),\n  });\n}\n\nconst reportQueue = createReportQueue(sendReport);\n\nfunction report(state) {\n  const url =\n    process.env.BYSPACE_TERMINAL_ACTIVITY_URL;\n  const terminalId = process.env.BYSPACE_TERMINAL_ID;\n  const token = process.env.BYSPACE_ACTIVITY_TOKEN;\n  if (!url || !terminalId || !token) return reportQueue.wait();\n\n  return reportQueue.enqueue({ url, terminalId, token, state });\n}\n\nexport default function byspaceTerminalActivity(pi) {\n  const ownerPid = process.env[ownerPidKey];\n  if (ownerPid && ownerPid !== String(process.pid)) return;\n  process.env[ownerPidKey] = String(process.pid);\n\n  pi.on(\"agent_start\", () => void report(\"running\"));\n  pi.on(\"ui_prompt_start\", () => void report(\"needs-input\"));\n  pi.on(\"ui_prompt_end\", () => void report(\"running\"));\n  pi.on(\"agent_settled\", () => void report(\"idle\"));\n  pi.on(\"session_shutdown\", () => report(\"idle\"));\n}\n";
+export declare const PI_TERMINAL_ACTIVITY_EXTENSION_SOURCE = "// byspace.pi-terminal-activity\nconst ownerPidKey = \"BYSPACE_PI_TERMINAL_HOOK_OWNER_PID\";\n\n// Keep one request in flight and only the latest state waiting behind it.\nexport function createReportQueue(sendReport) {\n  let pendingReport;\n  let reporting;\n\n  async function flushReports() {\n    while (pendingReport) {\n      const next = pendingReport;\n      pendingReport = undefined;\n      try {\n        await sendReport(next);\n      } catch {}\n    }\n    reporting = undefined;\n  }\n\n  return {\n    enqueue(next) {\n      pendingReport = next;\n      reporting ??= Promise.resolve().then(flushReports);\n      return reporting;\n    },\n    wait() {\n      return reporting ?? Promise.resolve();\n    },\n  };\n}\n\nasync function sendReport(next) {\n  await fetch(next.url, {\n    method: \"POST\",\n    headers: { \"content-type\": \"application/json\" },\n    body: JSON.stringify({\n      terminalId: next.terminalId,\n      token: next.token,\n      state: next.state,\n    }),\n    signal: AbortSignal.timeout(1_000),\n  });\n}\n\nconst reportQueue = createReportQueue(sendReport);\n\n// The marker is inherited by Pi child processes so they do not register a second\n// reporter for the same terminal. A marker left by a process that is gone is\n// stale, not an owner: treating it as an owner makes this Pi report nothing at\n// all, which looks exactly like \"the terminal stopped showing running\".\nfunction ownerIsLive(ownerPid) {\n  const pid = Number(ownerPid);\n  if (!Number.isInteger(pid) || pid <= 0) return false;\n  try {\n    process.kill(pid, 0);\n    return true;\n  } catch (error) {\n    return error?.code === \"EPERM\";\n  }\n}\n\nfunction report(state) {\n  const url =\n    process.env.BYSPACE_TERMINAL_ACTIVITY_URL;\n  const terminalId = process.env.BYSPACE_TERMINAL_ID;\n  const token = process.env.BYSPACE_ACTIVITY_TOKEN;\n  if (!url || !terminalId || !token) return reportQueue.wait();\n\n  return reportQueue.enqueue({ url, terminalId, token, state });\n}\n\nexport default function byspaceTerminalActivity(pi) {\n  const ownerPid = process.env[ownerPidKey];\n  if (ownerPid && ownerPid !== String(process.pid) && ownerIsLive(ownerPid)) return;\n  process.env[ownerPidKey] = String(process.pid);\n\n  pi.on(\"agent_start\", () => void report(\"running\"));\n  pi.on(\"ui_prompt_start\", () => void report(\"needs-input\"));\n  pi.on(\"ui_prompt_end\", () => void report(\"running\"));\n  pi.on(\"agent_settled\", () => void report(\"idle\"));\n  pi.on(\"session_shutdown\", () => report(\"idle\"));\n}\n";
 //# sourceMappingURL=pi-extension.d.ts.map

package/node_modules/@bytetrue/server/dist/server/terminal/agent-hooks/pi/pi-extension.js

@@ -44,6 +44,21 @@ async function sendReport(next) {
 
 const reportQueue = createReportQueue(sendReport);
 
+// The marker is inherited by Pi child processes so they do not register a second
+// reporter for the same terminal. A marker left by a process that is gone is
+// stale, not an owner: treating it as an owner makes this Pi report nothing at
+// all, which looks exactly like "the terminal stopped showing running".
+function ownerIsLive(ownerPid) {
+  const pid = Number(ownerPid);
+  if (!Number.isInteger(pid) || pid <= 0) return false;
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch (error) {
+    return error?.code === "EPERM";
+  }
+}
+
 function report(state) {
   const url =
     process.env.BYSPACE_TERMINAL_ACTIVITY_URL;
@@ -56,7 +71,7 @@ function report(state) {
 
 export default function byspaceTerminalActivity(pi) {
   const ownerPid = process.env[ownerPidKey];
-  if (ownerPid && ownerPid !== String(process.pid)) return;
+  if (ownerPid && ownerPid !== String(process.pid) && ownerIsLive(ownerPid)) return;
   process.env[ownerPidKey] = String(process.pid);
 
   pi.on("agent_start", () => void report("running"));

package/node_modules/@bytetrue/server/dist/server/terminal/terminal.js

@@ -786,6 +786,12 @@ export async function createTerminal(options) {
         if (!commandFinished) {
             return true;
         }
+        // The interactive shell only emits this when its foreground job ends. A
+        // terminal agent that exits without reporting idle (SIGKILL, crash, a
+        // suspend that never resumes) leaves `working` set forever, so treat the
+        // shell regaining the prompt as proof that the agent is gone. `interrupt`
+        // only clears `working`; a finished-attention idle state is untouched.
+        activityTracker.interrupt();
         for (const listener of Array.from(commandFinishedListeners)) {
             try {
                 listener(commandFinished);