pi-harness-delegate 0.7.0 → 0.7.2

package/extensions/templates.ts

@@ -224,13 +224,32 @@ export function resolveNativePermission(
  *
  * Pure filesystem inspection — no trust logic. The caller supplies the trust decision.
  */
-export function projectTemplatePresence(cwd: string): {
+export function projectTemplatePresence(
+  cwd: string,
+  /** Canonical harness names — the only partitions `loadTemplates` ever reads (callers pass the
+   *  registry's `HARNESS_NAMES`; injected so this module stays free of the harness registry). */
+  harnessNames: readonly string[],
+): {
   /** Template dirs that exist and would load if the project were trusted. */
   dirs: string[];
   /** A leftover `.pi/trusted` file — strong evidence the user relied on the removed mechanism. */
   staleTrustFile: boolean;
 } {
-  const candidates = [projectTemplatesDir(cwd), legacyProjectTemplatesDir(cwd)];
+  // the per-harness partitions (`.pi/delegate/templates/<harness>/`) load too — see loadTemplates —
+  // so they count as skipped content just like the shared root does. Only real harness partitions:
+  // loadTemplates is always called with a canonical name, so `omp/` (an alias), `archive/`,
+  // `shared/` etc. are never read and must not trigger the warning either.
+  const known = new Set(harnessNames);
+  let partitions: string[] = [];
+  try {
+    partitions = readdirSync(projectTemplatesDir(cwd), { withFileTypes: true })
+      .filter(e => e.isDirectory() && known.has(e.name))
+      .map(e => projectTemplatesDir(cwd, e.name))
+      .sort();
+  } catch {
+    // absent or unreadable
+  }
+  const candidates = [projectTemplatesDir(cwd), ...partitions, legacyProjectTemplatesDir(cwd)];
   const dirs: string[] = [];
   for (const dir of candidates) {
     try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-harness-delegate",
-  "version": "0.7.0",
+  "version": "0.7.2",
   "description": "Delegate work to any harness (Claude Code, Muse, OpenCode, Amp, Devin) from the pi coding agent \u2014 code reviews, plans, implementation, security audits, docs, or your own custom templates.",
   "type": "module",
   "packageManager": "bun@1.3.14",
@@ -40,6 +40,7 @@
   "scripts": {
     "typecheck": "tsc --noEmit",
     "test": "bun test",
+    "test:coverage": "bun test --coverage --coverage-reporter=text --coverage-reporter=lcov",
     "lint": "biome check .",
     "lint:fix": "biome check --write .",
     "verify": "bun run lint && bun run typecheck && bun run test",

package/extensions/run-claude.ts

@@ -1,61 +0,0 @@
-/** @deprecated use extensions/runner.ts + harnesses/claude.ts directly */
-
-import { claudeHarness } from './harnesses/claude.ts';
-import type { ActivityEvent, StreamedResult } from './harnesses/types.ts';
-import { runHarness } from './runner.ts';
-
-export interface RunClaudeOptions {
-  prompt: string;
-  cwd: string;
-  permissionMode: string;
-  model?: string;
-  maxBudgetUsd?: number;
-  addDirs?: string[];
-  signal?: AbortSignal;
-  timeoutMs?: number;
-  resumeSessionId?: string;
-  onStream?: (text: string) => void;
-  onActivity?: (ev: ActivityEvent) => void;
-}
-
-export interface ClaudeResult extends StreamedResult {
-  streamedText: string;
-}
-
-export const DEFAULT_TIMEOUT_MS = 600_000;
-
-function mapPerm(mode: string): 'readonly' | 'edit' | 'danger' {
-  if (mode === 'plan') return 'readonly';
-  if (mode === 'bypassPermissions') return 'danger';
-  return 'edit';
-}
-
-export function runClaude(opts: RunClaudeOptions): Promise<ClaudeResult> {
-  const perm = mapPerm(opts.permissionMode);
-  const _nativePerm =
-    perm === 'readonly' && opts.permissionMode !== 'plan'
-      ? opts.permissionMode
-      : perm === 'danger' && opts.permissionMode !== 'bypassPermissions'
-        ? opts.permissionMode
-        : undefined;
-  // normalize: if caller passed a non-standard mode, preserve as native
-  const isStandard =
-    opts.permissionMode === 'plan' ||
-    opts.permissionMode === 'acceptEdits' ||
-    opts.permissionMode === 'bypassPermissions';
-  return runHarness({
-    harness: claudeHarness,
-    prompt: opts.prompt,
-    cwd: opts.cwd,
-    permission: perm,
-    nativePermission: isStandard ? undefined : opts.permissionMode,
-    model: opts.model,
-    maxBudgetUsd: opts.maxBudgetUsd,
-    addDirs: opts.addDirs,
-    signal: opts.signal,
-    timeoutMs: opts.timeoutMs,
-    resumeSessionId: opts.resumeSessionId,
-    onStream: opts.onStream,
-    onActivity: opts.onActivity,
-  }).then(r => ({ ...r, streamedText: r.streamedText }));
-}

package/extensions/stream-parse.ts

@@ -1,37 +0,0 @@
-/** @deprecated use extensions/harnesses/claude.ts parse directly */
-export type {
-  ActivityEvent,
-  ParseOutcome,
-  ParseState,
-  StreamedResult,
-  StreamedUsage,
-  StreamParseOutcome,
-} from './harnesses/types.ts';
-
-import { parseClaudeLine } from './harnesses/claude.ts';
-import type { ParseState, StreamParseOutcome } from './harnesses/types.ts';
-
-export function parseStreamLines(lines: Iterable<string>): StreamParseOutcome {
-  const state: ParseState = { streamedText: '', activities: [], result: null };
-  let streamedText = '';
-  const activities: StreamParseOutcome['activities'] = [];
-  let result: StreamParseOutcome['result'] = null;
-  for (const line of lines) {
-    const out = parseClaudeLine(line, state);
-    if (out.streamedText) {
-      streamedText += out.streamedText;
-      state.streamedText += out.streamedText;
-    }
-    if (out.activities) {
-      for (const a of out.activities) {
-        activities.push(a);
-        state.activities.push(a);
-      }
-    }
-    if (out.result) {
-      result = out.result;
-      state.result = out.result;
-    }
-  }
-  return { streamedText, result, activities };
-}