gsd-pi 2.78.0-dev.aeeb2ca00 → 2.78.1-dev.0fdacd524

package/dist/resources/extensions/browser-tools/tests/capture-sharp-optional.test.cjs

@@ -1,91 +0,0 @@
-/**
- * Regression tests for the optional sharp dependency in capture.ts.
- *
- * Behaviour:
- *   - constrainScreenshot must fall back to returning the raw buffer
- *     unchanged when sharp is unavailable, rather than throwing.
- *   - When sharp IS available, oversized screenshots get resized.
- *
- * No source-grep. The test drives the real constrainScreenshot function
- * after seeding the module-private `_sharp` cache via the test-only
- * `__setSharpForTesting` export.
- */
-
-const { describe, it, afterEach } = require("node:test");
-const assert = require("node:assert/strict");
-const jiti = require("jiti")(__filename, { interopDefault: true, debug: false });
-
-const { constrainScreenshot, __setSharpForTesting } = jiti("../capture.ts");
-
-describe("constrainScreenshot — sharp unavailable (null)", () => {
-	afterEach(() => {
-		// Clear the test override so later tests don't inherit a null sharp.
-		__setSharpForTesting(undefined);
-	});
-
-	it("returns the raw buffer unchanged when sharp is null", async () => {
-		__setSharpForTesting(null);
-
-		const rawBuffer = Buffer.from([0x89, 0x50, 0x4e, 0x47, 0x0d, 0x0a, 0x1a, 0x0a]); // PNG magic bytes
-		const result = await constrainScreenshot(null, rawBuffer, "image/png", 80);
-
-		assert.strictEqual(
-			result,
-			rawBuffer,
-			"constrainScreenshot must return the exact same buffer instance when sharp is null",
-		);
-	});
-
-	it("returns the raw buffer unchanged for JPEG input when sharp is null", async () => {
-		__setSharpForTesting(null);
-
-		const rawBuffer = Buffer.from([0xff, 0xd8, 0xff, 0xe0]); // JPEG magic bytes
-		const result = await constrainScreenshot(null, rawBuffer, "image/jpeg", 80);
-
-		assert.strictEqual(result, rawBuffer);
-	});
-});
-
-describe("constrainScreenshot — sharp available", () => {
-	afterEach(() => {
-		__setSharpForTesting(undefined);
-	});
-
-	it("passes through a small image unchanged (below cap)", async () => {
-		const sharp = require("sharp");
-		const small = await sharp({
-			create: {
-				width: 400,
-				height: 300,
-				channels: 3,
-				background: { r: 128, g: 128, b: 128 },
-			},
-		})
-			.jpeg({ quality: 80 })
-			.toBuffer();
-
-		const result = await constrainScreenshot(null, small, "image/jpeg", 80);
-		const meta = await sharp(result).metadata();
-		assert.equal(meta.width, 400, "small images must not be resized");
-		assert.equal(meta.height, 300);
-	});
-
-	it("resizes an oversized image to within 1568px", async () => {
-		const sharp = require("sharp");
-		const big = await sharp({
-			create: {
-				width: 3000,
-				height: 2000,
-				channels: 3,
-				background: { r: 128, g: 128, b: 128 },
-			},
-		})
-			.jpeg({ quality: 80 })
-			.toBuffer();
-
-		const result = await constrainScreenshot(null, big, "image/jpeg", 80);
-		const meta = await sharp(result).metadata();
-		assert.ok(meta.width <= 1568, `width ${meta.width} must be <= 1568`);
-		assert.ok(meta.height <= 1568, `height ${meta.height} must be <= 1568`);
-	});
-});

package/dist/resources/extensions/gsd/tests/auto-supervisor.test.mjs

@@ -1,53 +0,0 @@
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, readFileSync } from 'node:fs';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { writeUnitRuntimeRecord, readUnitRuntimeRecord } from '../unit-runtime.ts';
-import { resolveAutoSupervisorConfig } from '../preferences.ts';
-
-test('resolveAutoSupervisorConfig provides safe timeout defaults', () => {
-  const supervisor = resolveAutoSupervisorConfig();
-  assert.equal(supervisor.soft_timeout_minutes, 20);
-  assert.equal(supervisor.idle_timeout_minutes, 10);
-  assert.equal(supervisor.hard_timeout_minutes, 30);
-});
-
-test('writeUnitRuntimeRecord persists progress and recovery metadata defaults', () => {
-  const base = mkdtempSync(join(tmpdir(), 'gsd-auto-supervisor-'));
-  const startedAt = 1234567890;
-
-  writeUnitRuntimeRecord(base, 'plan-milestone', 'M010', startedAt, {
-    phase: 'dispatched',
-    lastProgressAt: startedAt,
-    progressCount: 1,
-    lastProgressKind: 'dispatch',
-  });
-
-  const runtime = readUnitRuntimeRecord(base, 'plan-milestone', 'M010');
-  assert.ok(runtime);
-  assert.equal(runtime.phase, 'dispatched');
-  assert.equal(runtime.lastProgressAt, startedAt);
-  assert.equal(runtime.progressCount, 1);
-  assert.equal(runtime.lastProgressKind, 'dispatch');
-  assert.equal(runtime.recoveryAttempts, 0);
-});
-
-test('writeUnitRuntimeRecord keeps explicit recovery attempt fields', () => {
-  const base = mkdtempSync(join(tmpdir(), 'gsd-auto-supervisor-'));
-  const startedAt = 2234567890;
-
-  writeUnitRuntimeRecord(base, 'research-milestone', 'M011', startedAt, {
-    phase: 'timeout',
-    recoveryAttempts: 2,
-    lastRecoveryReason: 'idle',
-    lastProgressAt: startedAt + 50,
-    progressCount: 3,
-    lastProgressKind: 'recovery-retry',
-  });
-
-  const runtime = JSON.parse(readFileSync(join(base, '.gsd/runtime/units/research-milestone-M011.json'), 'utf8'));
-  assert.equal(runtime.recoveryAttempts, 2);
-  assert.equal(runtime.lastRecoveryReason, 'idle');
-  assert.equal(runtime.lastProgressKind, 'recovery-retry');
-});

package/dist/resources/extensions/gsd/tests/dist-redirect.mjs

@@ -1,112 +0,0 @@
-import { existsSync, readFileSync } from 'node:fs';
-import { createRequire } from 'node:module';
-import { fileURLToPath } from 'node:url';
-
-const require = createRequire(import.meta.url);
-
-const ROOT = new URL("../../../../../", import.meta.url);
-
-export function resolve(specifier, context, nextResolve) {
-  // 1. Redirect all workspace package bare imports to source.
-  //    CI portability runs don't build any packages/ dist artifacts, so every
-  //    @gsd/* specifier (including transitive ones pulled in by pi-coding-agent
-  //    source itself) must resolve to the TypeScript source entrypoint.
-  if (specifier === "../../packages/pi-coding-agent/src/index.js") {
-    specifier = new URL("packages/pi-coding-agent/src/index.ts", ROOT).href;
-  } else if (specifier === "@gsd/pi-coding-agent") {
-    specifier = new URL("packages/pi-coding-agent/src/index.ts", ROOT).href;
-  } else if (specifier === "@gsd/pi-ai/oauth") {
-    specifier = new URL("packages/pi-ai/src/utils/oauth/index.ts", ROOT).href;
-  } else if (specifier === "@gsd/pi-ai") {
-    specifier = new URL("packages/pi-ai/src/index.ts", ROOT).href;
-  } else if (specifier === "@gsd/pi-agent-core") {
-    specifier = new URL("packages/pi-agent-core/src/index.ts", ROOT).href;
-  } else if (specifier === "@gsd/pi-tui") {
-    specifier = new URL("packages/pi-tui/src/index.ts", ROOT).href;
-  } else if (specifier === "@gsd/native") {
-    specifier = new URL("packages/native/src/index.ts", ROOT).href;
-  } else if (specifier.startsWith("@gsd/native/")) {
-    // Sub-path imports like @gsd/native/fd, @gsd/native/text, etc.
-    const subpath = specifier.slice("@gsd/native/".length);
-    specifier = new URL(`packages/native/src/${subpath}/index.ts`, ROOT).href;
-  }
-  // 2. Redirect packages/*/dist/ → packages/*/src/ with .js→.ts for strip-types
-  //    Also handles local imports — skip rewrite for dist/ paths that are real compiled artifacts.
-
-  else if (specifier.endsWith('.js') && (specifier.startsWith('./') || specifier.startsWith('../'))) {
-    if (context.parentURL && context.parentURL.includes('/src/')) {
-      if (specifier.includes('/dist/')) {
-        specifier = specifier.replace('/dist/', '/src/').replace(/\.js$/, '.ts');
-      } else {
-        specifier = specifier.replace(/\.js$/, '.ts');
-      }
-    }
-  }
-  // 3. Extensionless relative imports from web/ (Next.js convention).
-  //    Transpiled .tsx files emit extensionless imports — try .ts then .tsx.
-  else if (
-    (specifier.startsWith('./') || specifier.startsWith('../')) &&
-    !specifier.match(/\.\w+$/) &&
-    context.parentURL &&
-    context.parentURL.includes('/web/')
-  ) {
-    const baseUrl = new URL(specifier, context.parentURL);
-    for (const ext of ['.ts', '.tsx']) {
-      const candidate = fileURLToPath(baseUrl) + ext;
-      if (existsSync(candidate)) {
-        specifier = baseUrl.href + ext;
-        break;
-      }
-
-    }
-  }
-
-  return nextResolve(specifier, context);
-}
-
-export function load(url, context, nextLoad) {
-  // Node's --experimental-strip-types handles plain .ts but not .tsx and not
-  // all TypeScript syntax used by workspace packages (parameter properties,
-  // decorators, etc.). Transpile all workspace package source files and .tsx
-  // files through TypeScript's transpileModule to avoid those crashes.
-  const shouldTranspileWithTypeScript =
-    url.endsWith('.tsx') ||
-    (url.endsWith('.ts') && url.includes('/packages/') && url.includes('/src/'));
-
-  if (shouldTranspileWithTypeScript) {
-    const ts = require('typescript');
-    const source = readFileSync(fileURLToPath(url), 'utf-8');
-    const { outputText } = ts.transpileModule(source, {
-      fileName: fileURLToPath(url),
-      compilerOptions: {
-        jsx: ts.JsxEmit.ReactJSX,
-        module: ts.ModuleKind.ESNext,
-        target: ts.ScriptTarget.ESNext,
-        esModuleInterop: true,
-        experimentalDecorators: true,
-        emitDecoratorMetadata: true,
-      },
-    });
-    // Inject CJS-compatible globals (__dirname, __filename, require) so that
-    // workspace packages compiled as ESM can still use them.  This avoids the
-    // need for import.meta.url behind indirect invocation patterns that fail in
-    // CJS and in dynamically-created scopes.
-    // Only inject globals that the source file doesn't already declare itself.
-    const preambleLines = [
-      'import { fileURLToPath as __preamble_fUTP } from "node:url";',
-      'import { dirname as __preamble_dn } from "node:path";',
-      'import { createRequire as __preamble_cR } from "node:module";',
-    ];
-    if (!outputText.includes('const __filename') && !outputText.includes('let __filename')) {
-      preambleLines.push('const __filename = __preamble_fUTP(import.meta.url);');
-    }
-    if (!outputText.includes('const __dirname') && !outputText.includes('let __dirname')) {
-      preambleLines.push('const __dirname = __preamble_dn(__preamble_fUTP(import.meta.url));');
-    }
-    if (!outputText.includes('const require') && !outputText.includes('let require')) {
-      preambleLines.push('const require = __preamble_cR(import.meta.url);');
-    }
-    return { format: 'module', source: preambleLines.join('\n') + '\n' + outputText, shortCircuit: true };
-  }
-  return nextLoad(url, context);
-}

package/dist/resources/extensions/gsd/tests/resolve-ts-hooks.mjs

@@ -1,23 +0,0 @@
-import { fileURLToPath } from 'node:url';
-
-const ROOT = new URL("../../../../../", import.meta.url);
-const PACKAGES_ROOT = fileURLToPath(new URL("packages/", ROOT));
-
-export function resolve(specifier, context, nextResolve) {
-  let tsSpecifier = specifier;
-  if (specifier.includes('@gsd/')) {
-    tsSpecifier = specifier.replace('@gsd/', PACKAGES_ROOT).replace('/dist/', '/src/');
-    if (tsSpecifier.includes('/packages/pi-ai') && !tsSpecifier.endsWith('.ts')) {
-        tsSpecifier = tsSpecifier.replace(/\/packages\/pi-ai$/, '/packages/pi-ai/src/index.ts');
-    } else if (!tsSpecifier.includes('/src/') && !tsSpecifier.endsWith('.ts')) {
-        // Fallback for other gsd packages like pi-coding-agent, pi-tui, pi-agent-core
-        tsSpecifier = tsSpecifier.replace(/\/packages\/([^\/]+)$/, '/packages/$1/src/index.ts');
-    } else if (!tsSpecifier.endsWith('.ts') && !tsSpecifier.endsWith('.js') && !tsSpecifier.endsWith('.mjs')) {
-        tsSpecifier += '/index.ts';
-    }
-  } else if (specifier.endsWith('.js')) {
-    tsSpecifier = specifier.replace(/\.js$/, '.ts');
-  }
-
-  return nextResolve(tsSpecifier, context);
-}

package/dist/resources/extensions/gsd/tests/resolve-ts.mjs

@@ -1,5 +0,0 @@
-import { register } from 'node:module';
-import { pathToFileURL } from 'node:url';
-
-// Register hook to redirect imports to the dist directory
-register(new URL('./dist-redirect.mjs', import.meta.url), pathToFileURL('./'));