@computesdk/loops 0.1.12 → 0.1.13

package/README.md

@@ -195,6 +195,11 @@ The default runner uses Pi 1.4.0's own CLI with a narrow, in-memory compatibilit
 before awaiting startup extensions, handle their answers immediately, and hold ordinary commands until setup finishes.
 It verifies the pinned version and exact upstream source before applying the fix; no global Pi files are modified.
 Pi still owns the session, extensions, model configuration, and all decisions.
+
+The bundled Loops extension is a fallback: if Pi's enabled, trusted packages already supply
+`@computesdk/pi-loops`, the default runner uses that installation instead of loading a second copy.
+Other extensions remain enabled, and genuine conflicts still fail. This selection is also SHA-pinned
+to Pi 1.4.0's resource loader; explicit `--extensions` or `--pi` retain normal Pi loading semantics.
 Failed answer submissions remain visible with a retryable error; Loops never accepts or declines an integration for you.
 An explicit `--pi` bypasses this compatibility runner, so that executable must support startup dialogs itself.
 

package/bin/pi-default-extension.mjs

@@ -0,0 +1,47 @@
+// The UI supplies Loops only as a fallback. Let Pi's own enabled-resource and
+// trust resolution choose a workspace/user installation before loading code.
+import { createHash } from 'node:crypto';
+import { readFileSync, realpathSync } from 'node:fs';
+import { join } from 'node:path';
+
+const expected = 'ad50235884ea6d5ee14cdae1efcfe6afef58b22b18ce574e81886d57ca9f4642';
+const canonical = (path) => {
+  try { return realpathSync(path); } catch { return path; }
+};
+
+export function filterDefaultLoopsExtension(paths, metadataByPath, fallback, bootstrap = false) {
+  const fallbackPath = canonical(fallback);
+  // The app's fallback is not a trust provider. Defer it until Pi has made its
+  // trust decision, otherwise its factory can run before a project copy wins.
+  if (bootstrap) return paths.filter((path) => canonical(path) !== fallbackPath);
+  const hasInstalledLoops = paths.some((path) => {
+    if (canonical(path) === fallbackPath) return false;
+    // Do not infer identity from a filename, tool name, or package source string.
+    // Only roots already discovered by Pi as a package are eligible.
+    const root = metadataByPath.get(path)?.packageRoot;
+    if (!root) return false;
+    try {
+      return JSON.parse(readFileSync(join(root, 'package.json'), 'utf8')).name === '@computesdk/pi-loops';
+    } catch { return false; }
+  });
+  return hasInstalledLoops ? paths.filter((path) => canonical(path) !== fallbackPath) : paths;
+}
+
+export function patchDefaultLoopsExtension(source, fallback) {
+  if (createHash('sha256').update(source).digest('hex') !== expected) {
+    throw new Error('Unsupported Pi resource loader source; default Loops compatibility patch requires Pi 1.4.0');
+  }
+  // extensionsOverride runs after extension execution and conflict detection.
+  // Filter only our implicit CLI fallback, before either operation, in both the
+  // untrusted bootstrap pass and the final trusted/untrusted resource pass.
+  const declaration = 'const extensionPaths = (this.noExtensions ? cliEnabledExtensions : this.mergePaths(cliEnabledExtensions, enabledExtensions))';
+  const warnings = '        const packageWarnings = collectExtensionPackageWarnings(extensionPaths, metadataByPath);';
+  const bootstrapMethod = '    async loadCurrentExtensionSet(options) {';
+  if (source.split(declaration).length !== 3 || source.split(warnings).length !== 3 || source.split(bootstrapMethod).length !== 2) {
+    throw new Error('Pi default Loops patch anchor mismatch');
+  }
+  source = source.replaceAll(declaration, declaration.replace('extensionPaths', 'discoveredExtensionPaths'));
+  source = source.split(bootstrapMethod).map((part, index) => part.replace(warnings,
+    `        const extensionPaths = filterDefaultLoopsExtension(discoveredExtensionPaths, metadataByPath, ${JSON.stringify(fallback)}, ${index === 1});\n${warnings}`)).join(bootstrapMethod);
+  return `import { filterDefaultLoopsExtension } from ${JSON.stringify(import.meta.url)};\n${source}`;
+}

package/bin/pi-rpc.mjs

@@ -5,6 +5,7 @@ import { readFileSync, realpathSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { fileURLToPath, pathToFileURL } from 'node:url';
 import { patchRpcStartup } from './pi-rpc-startup.mjs';
+import { patchDefaultLoopsExtension } from './pi-default-extension.mjs';
 
 const main = fileURLToPath(import.meta.resolve('@earendil-works/pi-coding-agent'));
 const root = dirname(dirname(main));
@@ -13,14 +14,27 @@ if (pkg.name !== '@earendil-works/pi-coding-agent' || pkg.version !== '1.4.0') {
   throw new Error('Pi RPC runner requires @earendil-works/pi-coding-agent 1.4.0');
 }
 const target = realpathSync(join(root, 'dist/modes/rpc/rpc-mode.js'));
+// This private runner flag is supplied only for the app's implicit default.
+// Explicit --extensions and custom Pi executables retain normal CLI semantics.
+const defaultIndex = process.argv.indexOf('--loops-default-extension');
+const fallback = defaultIndex < 0 ? undefined : process.argv[defaultIndex + 1];
+if (defaultIndex >= 0) {
+  if (!fallback || fallback.startsWith('-')) throw new Error('--loops-default-extension requires a path');
+  process.argv.splice(defaultIndex, 2, '-e', fallback);
+}
+const resourceTarget = fallback ? realpathSync(join(root, 'dist/core/resource-loader.js')) : undefined;
 // Validate before the CLI starts, including when a future CLI stops loading RPC.
 patchRpcStartup(readFileSync(target, 'utf8'));
+if (resourceTarget) patchDefaultLoopsExtension(readFileSync(resourceTarget, 'utf8'), fallback);
 registerHooks({
   load(url, context, nextLoad) {
     const result = nextLoad(url, context);
     if (url.startsWith('file:') && fileURLToPath(url) === target) {
       return { ...result, source: patchRpcStartup(String(result.source)) };
     }
+    if (resourceTarget && url.startsWith('file:') && fileURLToPath(url) === resourceTarget) {
+      return { ...result, source: patchDefaultLoopsExtension(String(result.source), fallback) };
+    }
     return result;
   },
 });

package/dist/server.mjs

@@ -5308,6 +5308,7 @@ var PI = arg("pi", path10.join(APP_ROOT, "bin", "pi-rpc.mjs"));
 var DATA = path10.resolve(arg("data", process.env.LOOPS_UI_DATA ?? path10.join(process.env.HOME ?? process.cwd(), ".pi", "loops-ui")));
 var MODEL = arg("model", void 0) || null;
 var EXTENSIONS = (arg("extensions", path10.resolve(APP_ROOT, "../pi-loops/extensions/loop.ts")) ?? "").split(",").filter(Boolean);
+var IMPLICIT_BUNDLED_EXTENSION = arg("pi") === void 0 && arg("extensions") === void 0;
 function prerequisites() {
   const out = [];
   const onPath = PI.includes("/") || /^[A-Za-z]:\\/.test(PI) ? fs9.existsSync(PI) : (() => {
@@ -5374,7 +5375,7 @@ var requireChatWorkspace = (id) => {
 };
 var manager = new ChatManager({
   piCommand: PI,
-  args: EXTENSIONS.flatMap((e) => ["-e", e]),
+  args: EXTENSIONS.flatMap((e) => [IMPLICIT_BUNDLED_EXTENSION ? "--loops-default-extension" : "-e", e]),
   model: MODEL,
   // a chat's repo comes from the registry: the browser never hands us a path
   bind: (chat) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@computesdk/loops",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "type": "module",
   "description": "Loops for people who do not live in a terminal: one command starts a localhost UI that runs, watches and steers hill-climbing agents in your repos.",
   "keywords": [
@@ -37,7 +37,7 @@
     "start": "node --experimental-strip-types server/index.ts",
     "check": "tsc --noEmit && tsc --noEmit -p tsconfig.client.json",
     "shots": "node shots.mjs",
-    "test": "node --experimental-strip-types test/ui-compaction.ts && node --experimental-strip-types test/session-client.ts && node --experimental-strip-types test/ui-commands.ts && node --experimental-strip-types test/ui-request.ts && node --experimental-strip-types test/provider-auth.ts && node --experimental-strip-types test/session-actions.ts && node --experimental-strip-types test/navigation.ts && node --experimental-strip-types test/api-client.ts && esbuild test/compare.ts --bundle --platform=node --format=cjs --outfile=node_modules/.tmp/compare.cjs --loader:.tsx=tsx --jsx=automatic --log-level=error && node node_modules/.tmp/compare.cjs && node --experimental-strip-types test/runs.ts && node --experimental-strip-types test/workspaces.ts && node --experimental-strip-types test/chat-lifecycle.ts && node --experimental-strip-types test/workspace-api.ts && node --experimental-strip-types test/api.ts && node --experimental-strip-types test/history-api.ts && node --experimental-strip-types test/stop-api.ts && node --experimental-strip-types test/cli.ts && node --experimental-strip-types test/pi-startup.ts",
+    "test": "node --experimental-strip-types test/ui-compaction.ts && node --experimental-strip-types test/session-client.ts && node --experimental-strip-types test/ui-commands.ts && node --experimental-strip-types test/ui-request.ts && node --experimental-strip-types test/provider-auth.ts && node --experimental-strip-types test/session-actions.ts && node --experimental-strip-types test/navigation.ts && node --experimental-strip-types test/api-client.ts && esbuild test/compare.ts --bundle --platform=node --format=cjs --outfile=node_modules/.tmp/compare.cjs --loader:.tsx=tsx --jsx=automatic --log-level=error && node node_modules/.tmp/compare.cjs && node --experimental-strip-types test/runs.ts && node --experimental-strip-types test/workspaces.ts && node --experimental-strip-types test/chat-lifecycle.ts && node --experimental-strip-types test/workspace-api.ts && node --experimental-strip-types test/api.ts && node --experimental-strip-types test/history-api.ts && node --experimental-strip-types test/stop-api.ts && node --experimental-strip-types test/cli.ts && node --experimental-strip-types test/pi-startup.ts && node test/pi-default-extension.mjs",
     "test:browser": "node test/url-browser.mjs && node test/workspace-browser.mjs && node test/pi-dialog-browser.mjs && node test/stop-browser.mjs && node test/session-actions-browser.mjs"
   },
   "dependencies": {