@qvac/harness 0.4.0 → 0.6.0

package/worker/harness-sidecar.mjs

@@ -1,10 +1,4 @@
-// bare-stow entry: serves the Harness over the ipc duplex bare-stow wires.
-// argv (empty = absent): [2] skillsDir, [3] SDK storage JSON (required: the fs deny list fences it), [4] embedding config JSON,
-// [5] imageModel JSON, [6] imageClamps JSON,
-// [7] '1' to enable harness logging (written to `harness.log`, beside assistant.log),
-// [8] llm config JSON (llmDevice/llmGpuLayers/contextSize/kvCacheK/kvCacheV/completionModelCap),
-// [9] musicModel JSON, [10] musicClamps JSON, [11] vendored python wheels dir,
-// [12] host-supplied GPU stats JSON.
+// bare-stow entry: argv slots are the ones spawnHarness forwards in lib/spawn.ts; empty is absent
 import '../dist-lib/lib/hardware/gpu-probe-entry.js'
 import FileLog from 'bare-file-logger'
 import fs from 'fs'
@@ -41,10 +35,7 @@ const SCOPE = '[harness]'
 // when the file will not open, everything still has somewhere to go that is not the pipe
 const DISCARD = { info() {}, debug() {}, warn() {}, error() {} }
 
-// On Windows the host does not read our stdout, and writing past the OS buffer takes the
-// Bare thread with it: the event loop stops, nothing raises, and a download never produces
-// a byte. Elsewhere the same write only queues in memory (measured), which is why it only
-// showed up there. Every line goes to a file instead, via the FileLog the assistant uses.
+// on Windows unread stdout past the OS buffer silently halts the Bare thread, so log to a file
 function openHarnessLog(path) {
   try {
     fs.mkdirSync(dirname(path), { recursive: true })
@@ -54,8 +45,7 @@ function openHarnessLog(path) {
   }
 }
 
-// Ungated by the host's log flag: the SDK and its addons log through `console`, and a
-// chatty dependency must not be able to wedge this process.
+// ungated by the log flag: the SDK and addons log via console, and chatter must not wedge us
 function captureConsole(log) {
   console.log = console.info = (...args) => log.info(SCOPE, ...args)
   console.debug = (...args) => log.debug(SCOPE, ...args)
@@ -73,17 +63,16 @@ async function alignSdkConsoleWithHost(hostWantsLogs) {
   }
 }
 
+// lunte-disable-next-line qvac/no-default-export -- the stow entry calls this module's default
 export default async function start(ipc, ready) {
   let harness
   let dying = false
-  // close() cancels in-flight SDK model loads (see Harness._close) so exit is not
-  // parked behind a multi-GB load; a wedged native call still ends at the host's
-  // kill deadline (bare-supervisor/stow reap)
+  // close() cancels in-flight SDK loads so exit is not parked; the host's kill deadline backstops
   const die = () => {
     if (dying) return
     dying = true
     Promise.resolve(harness?.close())
-      .catch(() => {})
+      .catch(noop)
       .then(() => Bare.exit(0))
   }
   Bare.IPC.on('end', die).on('close', die)
@@ -92,9 +81,7 @@ export default async function start(ipc, ready) {
   const deniedRoots = deniedRootsFor(sdkStorage.home)
   const hostWantsLogs = Bare.argv[7] === '1'
   setHarnessLog(hostWantsLogs)
-  // before the first await, so nothing logs ahead of the sink. The SDK home sits in
-  // the storage root, which is where the host writes assistant.log — harness.log
-  // belongs next to it.
+  // before the first await so nothing logs ahead of the sink; the host writes assistant.log beside it
   const log = openHarnessLog(join(dirname(sdkStorage.home), 'harness.log')) ?? DISCARD
   setHarnessLogSink((line) => log.info(SCOPE, line))
   captureConsole(log)
@@ -109,7 +96,9 @@ export default async function start(ipc, ready) {
   const musicClamps = Bare.argv[10] ? JSON.parse(Bare.argv[10]) : undefined
   const gpuStats = Bare.argv[12] ? JSON.parse(Bare.argv[12]) : undefined
   // Only the host knows where the shipped wheels landed.
-  if (Bare.argv[11]) process.env.QVAC_PYTHON_WHEELS_DIR = Bare.argv[11]
+  if (Bare.argv[11]) {
+    process.env.QVAC_PYTHON_WHEELS_DIR = Bare.argv[11]
+  }
 
   configureSdkStorage(sdkStorage)
   process.env.QVAC_PYODIDE_CACHE_DIR = `${sdkStorage.home}/pyodide-cache`
@@ -148,14 +137,16 @@ export default async function start(ipc, ready) {
   if (skillCatalog) {
     tools.push(...compileOperationTools(skillCatalog, { httpRequestTool, mcpCallTool }))
   }
+
   if (skillsDir) {
     tools.push(
       createSkillTool({
         skillsDir,
-        ...(credentialed ? { credentialStore: credentialed.credentialStore } : {})
+        credentialStore: credentialed?.credentialStore
       })
     )
   }
+
   harness = new Harness({
     gpuStats,
     tools,
@@ -180,3 +171,5 @@ export default async function start(ipc, ready) {
 
   return () => {}
 }
+
+function noop() {}

package/worker/stow-entry.mjs

@@ -1,6 +1,5 @@
 /* global Bare */
-// bare-stow's generated bundle shim (lib/shim.js) as a plain file, so the
-// sidecar can be spawned without stowing one.
+// bare-stow's generated shim (lib/shim.js) as a plain file, so the sidecar spawns unstowed
 import '../dist-lib/lib/hardware/gpu-probe-entry.js'
 import protocol from 'bare-stow/protocol'
 
@@ -22,12 +21,16 @@ async function shutdown(code, err = null) {
   exited = true
 
   try {
-    if (stop) await stop()
+    if (stop) {
+      await stop()
+    }
   } catch (e) {
     err = err || e
   }
 
-  if (err) ipc.send('error', { message: err.message, stack: err.stack })
+  if (err) {
+    ipc.send('error', { message: err.message, stack: err.stack })
+  }
 
   await ipc.send('exit', { code })
 

package/dist-lib/lib/tools/mcp/utils.d.ts

@@ -1,47 +0,0 @@
-import type { AbortSignal as BareAbortSignal } from 'bare-abort-controller';
-import type { ToolExecuteContext } from '../../capabilities/completion.ts';
-import { type FetchInit } from '../http-request/utils.ts';
-export declare const MCP_PROTOCOL_VERSION = "2025-03-26";
-export declare const TIMEOUT_MS = 30000;
-export type McpFetchFn = (url: URL, init: FetchInit) => Promise<{
-    status: number;
-    statusText?: string;
-    headers: {
-        get(name: string): string | null;
-    };
-    text(): Promise<string> | string;
-}>;
-export declare function isNotification(method: string): boolean;
-export declare function isMetadataMethod(method: unknown): boolean;
-export declare function normalizeMethod(method: string, params: Record<string, unknown> | undefined): {
-    method: string;
-    params: Record<string, unknown> | undefined;
-};
-export declare function flattenMcpResponse(body: string): {
-    error: string;
-} | {
-    body: string;
-};
-export declare function sessionKey(url: string, ctx: ToolExecuteContext | undefined, credentialKey?: string): string;
-export declare function getSession(key: string): string;
-export declare function deleteSession(key: string): void;
-export declare function cacheSession(key: string, sessionId: string): void;
-export declare function post(fetch: McpFetchFn, url: URL, method: string, params: Record<string, unknown> | undefined, authHeader: string | null, sessionId: string | undefined, signal: BareAbortSignal): Promise<{
-    status: number;
-    statusText?: string;
-    headers: {
-        get(name: string): string | null;
-    };
-    text(): Promise<string> | string;
-}>;
-export declare function readBody(response: {
-    text(): Promise<string> | string;
-    headers: {
-        get(name: string): string | null;
-    };
-}): Promise<string>;
-export declare function needsSession(status: number, cached: string): boolean;
-export declare function establishSession(fetch: McpFetchFn, url: URL, authHeader: string | null, signal: BareAbortSignal): Promise<string | {
-    error: string;
-    stateless?: true;
-}>;