@sasasin/fetch-url 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.1
+
+- `tests/` is no longer shipped in the npm tarball. Node refuses to strip TypeScript types for
+  files under `node_modules`, so `node tests/selftest.mjs` could never work from an `npm:`-installed
+  copy (`ERR_UNSUPPORTED_NODE_MODULES_TYPE_STRIPPING`), and `tests/test-load.mjs` needs jiti, which
+  lives in `devDependencies`. The scripts stay in the repository and run from a git clone
+  (`~/.pi/agent/git/github.com/sasasin/fetch_url`) or the source tree.
+- README: the self-test section now says where the scripts can run, and notes that the extension
+  itself is unaffected because pi loads it through jiti.
+
 ## 0.1.0
 
 Initial public release.

package/README.md

@@ -126,8 +126,13 @@ in-extension self-update command:
 
 ## Self-test without pi
 
-The core is importable on its own (**Node 22.19+**, the same floor as pi itself — `tests/selftest.mjs`
-imports `.ts` directly and relies on native type stripping):
+Run these from a git clone (the git source lives at `~/.pi/agent/git/github.com/sasasin/fetch_url`)
+or from the source tree — **not** from the `npm:` installed copy: Node refuses to strip types for
+files under `node_modules`, so `selftest.mjs` cannot import `fetch-core.ts` there. The extension
+itself is unaffected, because pi loads it through jiti. Hence `tests/` is kept out of the npm
+tarball.
+
+Needs Node 22.19+ (the same floor as pi itself — `selftest.mjs` relies on native type stripping):
 
 ```bash
 npm install --omit=dev
@@ -135,7 +140,7 @@ node tests/selftest.mjs --concurrency 4 --mode text https://news.ycombinator.com
 node tests/selftest.mjs --abort-after 1500 --concurrency 2 https://httpbin.org/delay/8 https://example.com/
 ```
 
-To check the extension the way pi loads it (through jiti, no browser started):
+To check tool registration without starting a browser (uses jiti from `devDependencies`):
 
 ```bash
 npm install

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sasasin/fetch-url",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Pi extension that fetches many web pages at once by driving Playwright directly: one Chromium, one BrowserContext per URL, concurrent navigation, no MCP and no API billing.",
   "type": "module",
   "license": "MIT",
@@ -36,7 +36,6 @@
   "files": [
     "index.ts",
     "fetch-core.ts",
-    "tests",
     "README.md",
     "CHANGELOG.md",
     "LICENSE"

package/tests/selftest.mjs

@@ -1,57 +0,0 @@
-#!/usr/bin/env node
-/**
- * 自己テスト: pi の拡張 runtime を使わず fetch-core.ts を直接叩く。
- *   node tests/selftest.mjs [--concurrency N] [--mode aria|text|html] [--max-chars N] <url...>
- * 並列度 1（read_url 相当の直列）と 4 を同じ URL 群で比べるため、
- * 実行のたびに --concurrency を変えて 2 回流すと差が見える。
- */
-import { fetchPages, closeBrowser } from "../fetch-core.ts";
-
-const args = process.argv.slice(2);
-let concurrency = 4;
-let mode = "aria";
-let maxChars = 8000;
-let timeoutMs;
-let abortAfter;
-const urls = [];
-for (let i = 0; i < args.length; i++) {
-	if (args[i] === "--concurrency") concurrency = Number(args[++i]);
-	else if (args[i] === "--mode") mode = args[++i];
-	else if (args[i] === "--max-chars") maxChars = Number(args[++i]);
-	else if (args[i] === "--timeout") timeoutMs = Number(args[++i]);
-	else if (args[i] === "--abort-after") abortAfter = Number(args[++i]);
-	else urls.push(args[i]);
-}
-
-const controller = new AbortController();
-if (abortAfter) setTimeout(() => controller.abort(), abortAfter).unref?.();
-
-const started = Date.now();
-const results = await fetchPages(urls, {
-	concurrency,
-	mode,
-	maxCharsPerPage: maxChars,
-	...(timeoutMs ? { navTimeoutMs: timeoutMs } : {}),
-},{ signal: controller.signal });
-const total = Date.now() - started;
-
-for (const result of results) {
-	console.log(
-		[
-			result.ok ? "ok   " : "fail ",
-			result.url,
-			`attempt=${result.attempts}`,
-			`ms=${result.ms}`,
-			result.status ? `status=${result.status}` : "",
-			result.finalUrl && result.finalUrl !== result.url ? `final=${result.finalUrl}` : "",
-			`chars=${result.characters}`,
-			result.title ? `title=${JSON.stringify(result.title)}` : "",
-			result.note ? `note=${result.note}` : "",
-		]
-			.filter(Boolean)
-			.join(" ")
-	);
-	console.log(`  head: ${result.body.slice(0, 120).replace(/\s+/g, " ")}`);
-}
-console.log(`\nconcurrency=${concurrency} mode=${mode} urls=${urls.length} wall=${total}ms sum=${results.reduce((a, r) => a + r.ms, 0)}ms`);
-await closeBrowser();

package/tests/test-load.mjs

@@ -1,56 +0,0 @@
-#!/usr/bin/env node
-/**
- * pi と同じ jiti 経由で拡張をロードし、登録内容が生きていることを確認する。
- *   node tests/test-load.mjs
- * pi と同じ jiti 経路で読むので、ブラウザを起動せずに登録内容だけ確認できる
- * （selftest.mjs は .ts を直接 import するため型ストリッピング可能な Node 22.19+ が要る）。
- */
-import { dirname, resolve } from "node:path";
-import { fileURLToPath } from "node:url";
-import { createJiti } from "jiti";
-
-// このスクリプトは tests/ にあるので、パッケージルートは親ディレクトリ
-const here = dirname(fileURLToPath(import.meta.url));
-const root = resolve(here, "..");
-const jiti = createJiti(import.meta.url, { moduleCache: false });
-
-const mod = await jiti.import(resolve(root, "index.ts"));
-if (typeof mod.default !== "function") {
-	console.error("index.ts must default-export an extension factory");
-	process.exit(1);
-}
-
-const tools = [];
-const commands = [];
-const events = [];
-mod.default({
-	registerTool: (tool) => tools.push(tool?.name),
-	registerCommand: (name) => commands.push(name),
-	on: (event) => events.push(event),
-	exec: async () => ({ code: 0, stdout: "", stderr: "" }),
-});
-
-const core = await jiti.import(resolve(root, "fetch-core.ts"));
-
-// 拡張が実際に依存を解決できるか（インストール状態の妥当性チェック）
-let playwrightResolvable = false;
-try {
-	const pw = await jiti.import("playwright-core");
-	playwrightResolvable = Boolean(pw?.chromium);
-} catch {}
-
-const checks = [
-	["fetch_url tool registered", tools.includes("fetch_url")],
-	["registers no slash commands (updates go through pi update --extensions)", commands.length === 0],
-	["session_shutdown handler registered", events.includes("session_shutdown")],
-	["packageDir() resolves to this folder", resolve(core.packageDir()) === root],
-	["playwright-core is resolvable from the extension", playwrightResolvable],
-];
-
-let failed = false;
-for (const [label, ok] of checks) {
-	if (!ok) failed = true;
-	console.log(`${ok ? "ok  " : "FAIL"} ${label}`);
-}
-console.log(`node=${process.version} tools=${JSON.stringify(tools)} commands=${JSON.stringify(commands)}`);
-process.exit(failed ? 1 : 0);