dsh-deeptutor 0.1.11 → 0.2.0

package/src/index.ts

@@ -1,19 +1,18 @@
 /**
  * DeepTutor bridge plugin for DeepSeek Harness (dsh).
  *
- * Migrated from the pi coding-agent extension (TecFancy/pi-extensions,
- * extensions/deeptutor + skills/deeptutor). Registers three model-facing tools:
- *   - deeptutor_run  — execute a DeepTutor learning capability (HTTP/WS first, CLI fallback)
+ * Registers three model-facing tools:
+ *   - deeptutor_run  — execute a DeepTutor learning capability
  *   - deeptutor_kb   — list / search / info knowledge bases
  *   - deeptutor_note — archive a Markdown learning note to a server notebook
  *
- * Deployment auto-detection:
- *   - Local: deeptutor serve runs on this machine → API probe succeeds directly; if serve is
- *     not running, falls back to the local CLI (DEEPTUTOR_LOCAL_BIN).
- *   - Remote: serve runs on a server (listening on 127.0.0.1 only) → an SSH tunnel is started
- *     automatically to reach the API; if the tunnel is unavailable, falls back to the SSH CLI.
+ * Transport: every tool drives the deeptutor CLI.
+ *   - Local: DEEPTUTOR_SSH_HOST unset → runs the local CLI (DEEPTUTOR_LOCAL_BIN).
+ *   - Remote: DEEPTUTOR_SSH_HOST set → runs the server's CLI over SSH.
+ * The HTTP/WS API was dropped: DeepTutor 1.7 moved it off /api/v1 and put it behind
+ * a login, so the old probe always 404'd and cost ~30s per call before the CLI ran anyway.
  *
- * Env config (agent-agnostic): DEEPTUTOR_SSH_HOST / DEEPTUTOR_API_BASE / DEEPTUTOR_LOCAL_BIN /
+ * Env config (agent-agnostic): DEEPTUTOR_SSH_HOST / DEEPTUTOR_LOCAL_BIN /
  * DEEPTUTOR_REMOTE_BIN / DEEPTUTOR_REMOTE_HOME — see src/config.ts and
  * .agents/skills/deeptutor/SKILL.md.
  */
@@ -21,9 +20,8 @@ import type { Context } from "@deepseek-ai/cordis";
 import { defineTool } from "@deepseek-ai/dsh-tools";
 import { CAPABILITIES, configGuide, configHint, SSH_HOST, truncate } from "./config.ts";
 import { cliNoteLocal, cliNoteRemote, runCli } from "./cli-exec.ts";
-import { apiJson, ensureApi, killTunnel } from "./http-api.ts";
 import { renderHtml } from "./html-render.ts";
-import { cliRunTurn, fmtTurnResult, wsRunTurn } from "./turn.ts";
+import { cliRunTurn, fmtTurnResult } from "./turn.ts";
 import { HTML_DOC_SCRIPT } from "./html-render.ts";
 
 export const name = "dsh-deeptutor";
@@ -41,10 +39,6 @@ function json<T extends Record<string, unknown>>(o: T): Jsonify<T> {
 
 export function apply(ctx: Context) {
   console.log(`[dsh-deeptutor] plugin loaded (html script: ${HTML_DOC_SCRIPT})`);
-  // Kill the SSH tunnel process when the plugin unloads.
-  ctx.effect(() => {
-    return () => killTunnel();
-  });
 
   // ---------- Tool 1: run a DeepTutor learning capability ----------
   ctx.tools.register(
@@ -130,27 +124,6 @@ export function apply(ctx: Context) {
       async execute(args, exec) {
         const timeoutMs = (args.timeout ?? 600) * 1000;
         const maxChars = args.max_chars ?? 30000;
-        try {
-          if (await ensureApi()) {
-            const r = await wsRunTurn(args, timeoutMs, exec.signal);
-            const htmlNote = await renderHtml(args, r.answer);
-            return json({
-              mode: "http",
-              sessionId: r.sessionId,
-              toolCalls: r.toolCalls,
-              errors: r.errors,
-              text:
-                configHint() + fmtTurnResult(args.capability, r, maxChars) + htmlNote,
-            });
-          }
-        } catch (err: any) {
-          if (exec.signal.aborted) {
-            // user cancelled mid-flight: don't start a fresh CLI run behind their back
-            throw new Error("[deeptutor_run cancelled]");
-          }
-          console.log(`[deeptutor] HTTP/WS failed, falling back to CLI: ${err?.message}`);
-        }
-        // fallback: local/SSH CLI
         try {
           const r = await cliRunTurn(args, timeoutMs, exec.signal);
           const htmlNote = await renderHtml(args, r.answer);
@@ -169,7 +142,7 @@ export function apply(ctx: Context) {
           }
           throw new Error(
             configGuide() +
-              `[deeptutor_run failed (HTTP and CLI both unavailable)] ${err?.message ?? String(err)}`,
+              `[deeptutor_run failed] ${err?.message ?? String(err)}`,
           );
         }
       },
@@ -230,50 +203,7 @@ export function apply(ctx: Context) {
       },
       async execute(args, exec) {
         const maxChars = args.max_chars ?? 30000;
-        // search goes through the CLI (REST has no dedicated search route);
-        // list/info prefer HTTP, fall back to CLI
-        if (args.action !== "search") {
-          try {
-            if (await ensureApi()) {
-              if (args.action === "list") {
-                const arr = await apiJson("/api/v1/knowledge/list", undefined, {
-                  signal: exec.signal,
-                });
-                const body = Array.isArray(arr)
-                  ? arr
-                      .map(
-                        (kb: any) =>
-                          `- ${kb.name} (status=${kb.status ?? "?"}, ${kb.statistics ? JSON.stringify(kb.statistics) : "no stats"}${kb.is_default ? ", default" : ""})`,
-                      )
-                      .join("\n")
-                  : JSON.stringify(arr);
-                return json({
-                  action: "list",
-                  mode: "http",
-                  text:
-                    configHint() +
-                    `Knowledge bases (${Array.isArray(arr) ? arr.length : "?"}):\n${body}`,
-                });
-              }
-              const kb = await apiJson(
-                `/api/v1/knowledge/${encodeURIComponent(args.kb ?? "")}`,
-                undefined,
-                { signal: exec.signal },
-              );
-              return json({
-                action: "info",
-                mode: "http",
-                kb: args.kb,
-                text: truncate(JSON.stringify(kb, null, 2), maxChars),
-              });
-            }
-          } catch (err: any) {
-            console.log(
-              `[deeptutor] kb ${args.action} HTTP failed, falling back to CLI: ${err?.message}`,
-            );
-          }
-        }
-        // CLI fallback (local or SSH). NOTE: `kb search` has no --top-k flag
+        // Local or SSH CLI. NOTE: `kb search` has no --top-k flag
         // (verified on deeptutor CLI 1.5.x); top_k is applied locally below.
         let argv: string[];
         if (args.action === "list") {
@@ -392,60 +322,7 @@ export function apply(ctx: Context) {
       async execute(args, exec) {
         const recordType = (args.type ?? "chat") as string;
         const nbName = args.notebook.trim();
-        // HTTP first
-        try {
-          if (await ensureApi()) {
-            const list = await apiJson("/api/v1/notebook/list", undefined, {
-              signal: exec.signal,
-            });
-            const notebooks: any[] = list?.notebooks ?? [];
-            let nb = notebooks.find((n) => n.name?.trim() === nbName);
-            if (!nb) {
-              const created = await apiJson(
-                "/api/v1/notebook/create",
-                {
-                  method: "POST",
-                  headers: { "Content-Type": "application/json" },
-                  body: JSON.stringify({
-                    name: nbName,
-                    description: `Archived by dsh agent (${new Date().toISOString().slice(0, 10)})`,
-                  }),
-                },
-                { signal: exec.signal },
-              );
-              nb = created?.notebook ?? created;
-            }
-            const nbId = nb?.id ?? (typeof nb === "string" ? nb : undefined);
-            if (!nbId) throw new Error("cannot resolve notebook id");
-            const added = await apiJson(
-              "/api/v1/notebook/add_record",
-              {
-                method: "POST",
-                headers: { "Content-Type": "application/json" },
-                body: JSON.stringify({
-                  notebook_ids: [nbId],
-                  record_type: recordType,
-                  title: args.title,
-                  user_query: args.title,
-                  output: args.content,
-                }),
-              },
-              { signal: exec.signal },
-            );
-            return json({
-              notebook: nbName,
-              nbId: String(nbId),
-              title: args.title,
-              mode: "http",
-              text:
-                configHint() +
-                `[note archived (HTTP)] notebook=${nbName} (${nbId}), title=${args.title}, type=${recordType}\n${truncate(JSON.stringify(added ?? {}), 800)}`,
-            });
-          }
-        } catch (err: any) {
-          console.log(`[deeptutor] note HTTP failed, falling back to CLI: ${err?.message}`);
-        }
-        // CLI fallback: remote deployment uses the remote PY_NOTE script; local deployment runs locally
+        // Remote deployment runs the PY_NOTE script on the server; local deployment runs locally
         const { stdout, stderr, code } = SSH_HOST
           ? await cliNoteRemote(nbName, args.title, recordType, args.content, exec.signal)
           : await cliNoteLocal(nbName, args.title, recordType, args.content, exec.signal);
@@ -461,7 +338,7 @@ export function apply(ctx: Context) {
           mode: "cli",
           text:
             configHint() +
-            `[note archived (CLI)] notebook=${nbName}, title=${args.title}, type=${recordType}\n${truncate((stdout ?? "").trim(), 2000)}`,
+            `[note archived] notebook=${nbName}, title=${args.title}, type=${recordType}\n${truncate((stdout ?? "").trim(), 2000)}`,
         });
       },
     }),

package/src/turn.ts

@@ -1,8 +1,8 @@
 /**
- * DeepTutor bridge: one learning "turn" — running a capability over WebSocket (HTTP mode)
- * or the CLI (fallback), and shared JSONL/WS event parsing + result formatting.
+ * DeepTutor bridge: one learning "turn" — running a capability through the deeptutor CLI
+ * (local or over SSH), JSONL event parsing and result formatting.
  */
-import { API_BASE, truncate } from "./config.ts";
+import { truncate } from "./config.ts";
 import { runCli } from "./cli-exec.ts";
 
 export interface TurnResult {
@@ -13,7 +13,7 @@ export interface TurnResult {
   title?: string;
 }
 
-/** Fields both wsRunTurn and cliRunTurn read off the deeptutor_run tool params. */
+/** Fields cliRunTurn reads off the deeptutor_run tool params. */
 export interface RunParams {
   prompt: string;
   capability: string;
@@ -24,7 +24,7 @@ export interface RunParams {
   config?: Record<string, unknown>;
 }
 
-/** Fold one event (WS or CLI JSONL) into the running turn state. Exported for unit tests. */
+/** Fold one CLI JSONL event into the running turn state. Exported for unit tests. */
 export function collectEvent(state: TurnResult, e: any) {
   if (e.type === "content" && typeof e.content === "string") state.answer += e.content;
   else if (e.type === "result" && typeof e.metadata?.response === "string")
@@ -39,95 +39,6 @@ export function collectEvent(state: TurnResult, e: any) {
   if (!state.sessionId && typeof e.session_id === "string") state.sessionId = e.session_id;
 }
 
-/** Execute one turn over WebSocket (streaming). */
-export function wsRunTurn(
-  params: RunParams,
-  timeoutMs: number,
-  signal?: AbortSignal,
-): Promise<TurnResult> {
-  return new Promise((resolve, reject) => {
-    if (signal?.aborted) return reject(new Error("Aborted"));
-    const WSImpl: any = (globalThis as any).WebSocket;
-    if (!WSImpl) return reject(new Error("WebSocket is not available in this environment"));
-    const ws = new WSImpl(`${API_BASE.replace(/^http/, "ws")}/api/v1/ws`);
-    let settled = false;
-    const state: TurnResult = { answer: "", errors: [], toolCalls: [] };
-    let timer: ReturnType<typeof setTimeout> | undefined;
-    const finish = (fn: () => void) => {
-      if (settled) return;
-      settled = true;
-      if (timer) clearTimeout(timer);
-      signal?.removeEventListener("abort", onAbort);
-      fn();
-    };
-    timer = setTimeout(() => {
-      finish(() => {
-        try {
-          ws.close();
-        } catch {}
-        reject(new Error(`WebSocket timeout (${Math.round(timeoutMs / 1000)}s)`));
-      });
-    }, timeoutMs);
-    const onAbort = () => {
-      finish(() => {
-        try {
-          ws.close();
-        } catch {}
-        reject(new Error("Aborted"));
-      });
-    };
-    signal?.addEventListener("abort", onAbort, { once: true });
-    ws.onopen = () => {
-      ws.send(
-        JSON.stringify({
-          type: "start_turn",
-          content: params.prompt,
-          capability: params.capability,
-          session_id: params.session_id ?? null,
-          tools: params.tools ?? [],
-          knowledge_bases: params.kbs ?? [],
-          language: params.language ?? "en",
-          config: params.config ?? {},
-          notebook_references: [],
-          history_references: [],
-          attachments: [],
-          skills: [],
-        }),
-      );
-    };
-    ws.onmessage = (ev: any) => {
-      let e: any;
-      try {
-        e = JSON.parse(ev.data);
-      } catch {
-        return;
-      }
-      collectEvent(state, e);
-      if (e.type === "done") {
-        finish(() => {
-          try {
-            ws.close();
-          } catch {}
-          resolve(state);
-        });
-      }
-    };
-    ws.onerror = () => {
-      finish(() => {
-        try {
-          ws.close();
-        } catch {}
-        reject(new Error("WebSocket connection error (server not running or tunnel down)"));
-      });
-    };
-    ws.onclose = () => {
-      finish(() => {
-        reject(new Error("WebSocket closed unexpectedly"));
-      });
-    };
-  });
-}
-
 /**
  * Build the `deeptutor run` argv for a capability turn (pure, unit-tested).
  * Scalar config values go as repeatable --config k=v; object/array values are

package/src/win-cmd.ts

@@ -0,0 +1,129 @@
+/**
+ * DeepTutor bridge: launching the local deeptutor CLI on Windows.
+ *
+ * Node's execFile cannot start a .cmd/.bat file directly on Windows (spawn EINVAL, see
+ * issue #8), and it does not search PATHEXT for a bare command name. resolveLocalCommand
+ * turns DEEPTUTOR_LOCAL_BIN into something execFile can run:
+ *   1. a bare name is looked up on PATH using PATHEXT;
+ *   2. a .cmd/.bat that only forwards its arguments to an .exe (`"C:\...\deeptutor.exe" %*`)
+ *      is replaced by that .exe, so no shell is involved;
+ *   3. any other .cmd/.bat runs through `cmd.exe /d /s /c` with every argument quoted and
+ *      its metacharacters escaped (the same scheme as cross-spawn).
+ * cmd.exe ends a command line at a newline, so case 3 cannot pass multi-line arguments;
+ * resolveLocalCommand reports that instead of silently truncating the prompt.
+ */
+import { existsSync, readFileSync } from "node:fs";
+import { extname, win32 } from "node:path";
+
+export interface LocalCommand {
+  file: string;
+  args: string[];
+  /** Pass the args to CreateProcess unchanged (needed for the cmd.exe command line). */
+  windowsVerbatimArguments?: boolean;
+  /** Set when the command cannot be run as requested; nothing should be spawned. */
+  error?: string;
+}
+
+const META_CHARS = /([()\][%!^"`<>&|;, *?])/g;
+
+/** Quote one argument for a cmd.exe command line (cross-spawn's escapeArgument). */
+export function escapeCmdArgument(arg: string, doubleEscapeMetaChars: boolean): string {
+  let out = arg.replace(/(\\*)"/g, '$1$1\\"');
+  out = out.replace(/(\\*)$/, "$1$1");
+  out = `"${out}"`;
+  out = out.replace(META_CHARS, "^$1");
+  if (doubleEscapeMetaChars) out = out.replace(META_CHARS, "^$1");
+  return out;
+}
+
+/** Escape the command path itself for a cmd.exe command line. */
+export function escapeCmdCommand(command: string): string {
+  return command.replace(META_CHARS, "^$1");
+}
+
+/**
+ * Build the `cmd.exe /d /s /c "..."` invocation for a batch file. Arguments are escaped
+ * twice because a batch file that forwards `%*` makes cmd.exe parse them a second time.
+ */
+export function buildCmdInvocation(batchFile: string, args: string[], comSpec = "cmd.exe"): LocalCommand {
+  const line = [escapeCmdCommand(batchFile), ...args.map((a) => escapeCmdArgument(a, true))].join(" ");
+  return { file: comSpec, args: ["/d", "/s", "/c", `"${line}"`], windowsVerbatimArguments: true };
+}
+
+/**
+ * If a .cmd/.bat wrapper does nothing but forward its arguments to an executable, return
+ * that executable's path. Accepts `@echo off`, `rem`/`::` comments and `setlocal`/`endlocal`
+ * around a single `["]<path>.exe["] %*` line; `%~dp0` is expanded to the wrapper's folder.
+ */
+export function parseCmdWrapper(text: string, wrapperPath: string): string | undefined {
+  const lines = text
+    .split(/\r?\n/)
+    .map((l) => l.trim())
+    .filter(
+      (l) =>
+        l !== "" &&
+        !/^@?echo\s+off$/i.test(l) &&
+        !/^@?(rem(\s|$)|::)/i.test(l) &&
+        !/^@?(setlocal|endlocal)(\s|$)/i.test(l),
+    );
+  if (lines.length !== 1) return undefined;
+  const m = /^@?(?:"([^"]+\.(?:exe|com))"|([^\s"]+\.(?:exe|com)))\s+%\*$/i.exec(lines[0]);
+  if (!m) return undefined;
+  // Windows path rules regardless of the host, so this also behaves the same in Linux CI.
+  const dir = win32.dirname(wrapperPath);
+  const target = (m[1] ?? m[2]).replace(/%~dp0/gi, dir + "\\");
+  if (/%/.test(target)) return undefined; // other variables: let cmd.exe expand them
+  return win32.resolve(dir, target);
+}
+
+/** Find a bare command name on PATH, trying each PATHEXT extension (Windows search order). */
+export function findOnPath(
+  name: string,
+  env: NodeJS.ProcessEnv = process.env,
+  exists: (p: string) => boolean = existsSync,
+): string | undefined {
+  const exts = (env.PATHEXT ?? ".COM;.EXE;.BAT;.CMD").split(";").filter(Boolean);
+  // Windows path rules regardless of the host, so this also behaves the same in Linux CI.
+  const dirs = (env.PATH ?? env.Path ?? "").split(win32.delimiter).filter(Boolean);
+  for (const dir of dirs) {
+    for (const ext of exts) {
+      const candidate = win32.join(dir.replace(/^"|"$/g, ""), name + ext.toLowerCase());
+      if (exists(candidate)) return candidate;
+    }
+  }
+  return undefined;
+}
+
+/** Turn a local CLI path plus argv into something execFile can start on this platform. */
+export function resolveLocalCommand(
+  file: string,
+  args: string[],
+  platform: NodeJS.Platform = process.platform,
+  env: NodeJS.ProcessEnv = process.env,
+): LocalCommand {
+  if (platform !== "win32") return { file, args };
+  let target = file;
+  const isBareName = !/[\\/]/.test(target) && extname(target) === "";
+  if (isBareName) target = findOnPath(target, env) ?? target;
+  if (!/\.(cmd|bat)$/i.test(target)) return { file: target, args };
+
+  let wrapped: string | undefined;
+  try {
+    wrapped = parseCmdWrapper(readFileSync(target, "utf8"), target);
+  } catch {
+    wrapped = undefined;
+  }
+  if (wrapped && existsSync(wrapped)) return { file: wrapped, args };
+
+  if (args.some((a) => /[\r\n]/.test(a))) {
+    return {
+      file: target,
+      args,
+      error:
+        `${target} is a batch file, and cmd.exe cannot pass arguments that contain line breaks. ` +
+        "Point DEEPTUTOR_LOCAL_BIN at deeptutor.exe instead (in a pip/venv install it is " +
+        "<venv>\\Scripts\\deeptutor.exe).",
+    };
+  }
+  return buildCmdInvocation(target, args, env.ComSpec ?? env.COMSPEC ?? "cmd.exe");
+}

package/lib/http-api.d.ts

@@ -1,10 +0,0 @@
-/** Kill the SSH tunnel process, if one is running (call on plugin disposal). */
-export declare function killTunnel(): void;
-export declare function apiProbe(timeoutMs?: number): Promise<boolean>;
-/** Ensure the local API is reachable: probe → start SSH tunnel if needed (remote only) → wait. */
-export declare function ensureApi(): Promise<boolean>;
-/** JSON fetch against the DeepTutor API. Combines a fixed timeout with an optional caller-supplied abort signal. */
-export declare function apiJson(path: string, init?: RequestInit, opts?: {
-    timeoutMs?: number;
-    signal?: AbortSignal;
-}): Promise<any>;

package/lib/http-api.js

@@ -1,94 +0,0 @@
-/**
- * DeepTutor bridge: HTTP/WS transport (API probing, SSH tunnel management, JSON calls).
- */
-import { spawn } from "node:child_process";
-import { API_BASE, SSH_HOST, TUNNEL_PORT } from "./config.js";
-let tunnelProc = null;
-/** Kill the SSH tunnel process, if one is running (call on plugin disposal). */
-export function killTunnel() {
-    if (tunnelProc && tunnelProc.exitCode === null) {
-        try {
-            tunnelProc.kill();
-        }
-        catch { }
-        tunnelProc = null;
-    }
-}
-export async function apiProbe(timeoutMs = 1500) {
-    try {
-        const res = await fetch(`${API_BASE}/api/v1/system/status`, {
-            signal: AbortSignal.timeout(timeoutMs),
-        });
-        return res.ok;
-    }
-    catch {
-        return false;
-    }
-}
-/** Ensure the local API is reachable: probe → start SSH tunnel if needed (remote only) → wait. */
-export async function ensureApi() {
-    if (await apiProbe(1200))
-        return true;
-    if (!SSH_HOST)
-        return false; // Local deployment: fall back to local CLI when serve is down, no tunnel needed
-    let tunnelDead = false;
-    if (!tunnelProc || tunnelProc.exitCode !== null) {
-        tunnelProc = spawn("ssh", [
-            "-N",
-            "-L",
-            `${TUNNEL_PORT}:127.0.0.1:${TUNNEL_PORT}`,
-            "-o",
-            "BatchMode=yes",
-            "-o",
-            "ExitOnForwardFailure=yes",
-            "-o",
-            "ServerAliveInterval=30",
-            "-o",
-            "ServerAliveCountMax=3",
-            "-o",
-            "ConnectTimeout=10",
-            SSH_HOST,
-        ], { stdio: "ignore", windowsHide: true });
-        // Fast-fail: a tunnel that exits immediately (bad alias, port in use,
-        // refused connection) is not going to recover — stop waiting for it.
-        tunnelProc.once("exit", () => {
-            tunnelDead = true;
-            tunnelProc = null;
-        });
-        tunnelProc.unref();
-    }
-    for (let i = 0; i < 20; i++) {
-        if (tunnelDead)
-            return false;
-        if (await apiProbe(1000))
-            return true;
-        await new Promise((r) => setTimeout(r, 500));
-    }
-    return false;
-}
-/** JSON fetch against the DeepTutor API. Combines a fixed timeout with an optional caller-supplied abort signal. */
-export async function apiJson(path, init, opts = {}) {
-    const controller = new AbortController();
-    const timer = setTimeout(() => controller.abort(), opts.timeoutMs ?? 60_000);
-    const onAbort = () => controller.abort();
-    // An already-aborted signal never fires its "abort" event again — check the
-    // flag first so a pre-aborted call fails immediately instead of running to
-    // completion against a still-open controller.
-    if (opts.signal?.aborted) {
-        controller.abort();
-    }
-    else {
-        opts.signal?.addEventListener("abort", onAbort, { once: true });
-    }
-    try {
-        const res = await fetch(`${API_BASE}${path}`, { ...init, signal: controller.signal });
-        if (!res.ok)
-            throw new Error(`HTTP ${res.status} ${res.statusText}`);
-        const text = await res.text();
-        return text ? JSON.parse(text) : null;
-    }
-    finally {
-        clearTimeout(timer);
-        opts.signal?.removeEventListener("abort", onAbort);
-    }
-}

package/scripts/sync-skills.mjs

@@ -1,47 +0,0 @@
-#!/usr/bin/env node
-/**
- * Copy the shared skill files from a pi-extensions checkout into this repo.
- * The pi repo is the single source of truth; the files must be agent-neutral.
- *
- * Usage:
- *   node scripts/sync-skills.mjs [path-to-pi-extensions]
- * Default source: ../pi-extensions
- */
-import { copyFileSync, existsSync, mkdirSync } from "node:fs";
-import { dirname, join, resolve } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const here = resolve(dirname(fileURLToPath(import.meta.url)), "..");
-const piRoot = resolve(process.argv[2] ?? join(here, "..", "pi-extensions"));
-
-// [pi source, this-repo destination]
-// The bundle uses the root scripts/ copy at runtime (src/html-render.ts);
-// the skill copy ships inside the package under .agents/skills/. Both must
-// stay identical.
-const FILES = [
-  ["skills/deeptutor/SKILL.md", ".agents/skills/deeptutor/SKILL.md"],
-  ["skills/deeptutor/README.md", ".agents/skills/deeptutor/README.md"],
-  ["skills/html-doc/SKILL.md", ".agents/skills/html-doc/SKILL.md"],
-  ["skills/html-doc/assets/template.html", ".agents/skills/html-doc/assets/template.html"],
-  ["skills/html-doc/scripts/md-to-html.js", "scripts/md-to-html.js"],
-  ["skills/html-doc/scripts/md-to-html.js", ".agents/skills/html-doc/scripts/md-to-html.js"],
-];
-
-let failures = 0;
-for (const [src, dst] of FILES) {
-  const from = join(piRoot, src);
-  const to = join(here, dst);
-  if (!existsSync(from)) {
-    console.error(`[sync-skills] missing source: ${from}`);
-    failures++;
-    continue;
-  }
-  mkdirSync(dirname(to), { recursive: true });
-  copyFileSync(from, to);
-  console.log(`[sync-skills] ${src} -> ${dst}`);
-}
-if (failures > 0) {
-  console.error("[sync-skills] some sources missing — is the pi-extensions path correct?");
-  process.exit(1);
-}
-console.log("[sync-skills] done — review the diff, then commit.");