@asterxsk/kiln 0.3.0 → 0.4.2

package/agent/extensions/shared/dashboard-state.ts

@@ -1,99 +1,99 @@
-export const MODEL_INFO_CHANNEL = "dashboard:model-info";
-export const GIT_INFO_CHANNEL = "dashboard:git-info";
-export const REFRESH_CHANNEL = "dashboard:refresh";
-
-export interface ModelInfoState {
-  provider: string;
-  modelId: string;
-  modelName: string;
-  thinking: string;
-  contextTokens: number | null;
-  contextWindow: number;
-  contextPercent: number | null;
-  cost: number;
-  tokensPerSecond: number | null;
-  generating: boolean;
-}
-
-export interface PullRequestInfo {
-  number: number;
-  url: string;
-  isDraft: boolean;
-}
-
-export interface GitInfoState {
-  isRepository: boolean;
-  branch: string | null;
-  changedFiles: number;
-  pullRequest: PullRequestInfo | null;
-}
-
-export function emptyModelInfoState(): ModelInfoState {
-  return {
-    provider: "",
-    modelId: "no-model",
-    modelName: "No model",
-    thinking: "off",
-    contextTokens: null,
-    contextWindow: 0,
-    contextPercent: null,
-    cost: 0,
-    tokensPerSecond: null,
-    generating: false,
-  };
-}
-
-export function emptyGitInfoState(): GitInfoState {
-  return {
-    isRepository: false,
-    branch: null,
-    changedFiles: 0,
-    pullRequest: null,
-  };
-}
-
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null;
-}
-
-function isNullableNumber(value: unknown) {
-  return value === null || typeof value === "number";
-}
-
-export function isModelInfoState(value: unknown): value is ModelInfoState {
-  if (!isRecord(value)) return false;
-
-  return (
-    typeof value.provider === "string" &&
-    typeof value.modelId === "string" &&
-    typeof value.modelName === "string" &&
-    typeof value.thinking === "string" &&
-    isNullableNumber(value.contextTokens) &&
-    typeof value.contextWindow === "number" &&
-    isNullableNumber(value.contextPercent) &&
-    typeof value.cost === "number" &&
-    isNullableNumber(value.tokensPerSecond) &&
-    typeof value.generating === "boolean"
-  );
-}
-
-function isPullRequestInfo(value: unknown): value is PullRequestInfo {
-  if (!isRecord(value)) return false;
-
-  return (
-    typeof value.number === "number" &&
-    typeof value.url === "string" &&
-    typeof value.isDraft === "boolean"
-  );
-}
-
-export function isGitInfoState(value: unknown): value is GitInfoState {
-  if (!isRecord(value)) return false;
-
-  return (
-    typeof value.isRepository === "boolean" &&
-    (value.branch === null || typeof value.branch === "string") &&
-    typeof value.changedFiles === "number" &&
-    (value.pullRequest === null || isPullRequestInfo(value.pullRequest))
-  );
-}
+export const MODEL_INFO_CHANNEL = "dashboard:model-info";
+export const GIT_INFO_CHANNEL = "dashboard:git-info";
+export const REFRESH_CHANNEL = "dashboard:refresh";
+
+export interface ModelInfoState {
+  provider: string;
+  modelId: string;
+  modelName: string;
+  thinking: string;
+  contextTokens: number | null;
+  contextWindow: number;
+  contextPercent: number | null;
+  cost: number;
+  tokensPerSecond: number | null;
+  generating: boolean;
+}
+
+export interface PullRequestInfo {
+  number: number;
+  url: string;
+  isDraft: boolean;
+}
+
+export interface GitInfoState {
+  isRepository: boolean;
+  branch: string | null;
+  changedFiles: number;
+  pullRequest: PullRequestInfo | null;
+}
+
+export function emptyModelInfoState(): ModelInfoState {
+  return {
+    provider: "",
+    modelId: "no-model",
+    modelName: "No model",
+    thinking: "off",
+    contextTokens: null,
+    contextWindow: 0,
+    contextPercent: null,
+    cost: 0,
+    tokensPerSecond: null,
+    generating: false,
+  };
+}
+
+export function emptyGitInfoState(): GitInfoState {
+  return {
+    isRepository: false,
+    branch: null,
+    changedFiles: 0,
+    pullRequest: null,
+  };
+}
+
+function isRecord(value: unknown): value is Record<string, unknown> {
+  return typeof value === "object" && value !== null;
+}
+
+function isNullableNumber(value: unknown) {
+  return value === null || typeof value === "number";
+}
+
+export function isModelInfoState(value: unknown): value is ModelInfoState {
+  if (!isRecord(value)) return false;
+
+  return (
+    typeof value.provider === "string" &&
+    typeof value.modelId === "string" &&
+    typeof value.modelName === "string" &&
+    typeof value.thinking === "string" &&
+    isNullableNumber(value.contextTokens) &&
+    typeof value.contextWindow === "number" &&
+    isNullableNumber(value.contextPercent) &&
+    typeof value.cost === "number" &&
+    isNullableNumber(value.tokensPerSecond) &&
+    typeof value.generating === "boolean"
+  );
+}
+
+function isPullRequestInfo(value: unknown): value is PullRequestInfo {
+  if (!isRecord(value)) return false;
+
+  return (
+    typeof value.number === "number" &&
+    typeof value.url === "string" &&
+    typeof value.isDraft === "boolean"
+  );
+}
+
+export function isGitInfoState(value: unknown): value is GitInfoState {
+  if (!isRecord(value)) return false;
+
+  return (
+    typeof value.isRepository === "boolean" &&
+    (value.branch === null || typeof value.branch === "string") &&
+    typeof value.changedFiles === "number" &&
+    (value.pullRequest === null || isPullRequestInfo(value.pullRequest))
+  );
+}

package/agent/extensions/shared/install.ps1

@@ -1,27 +1,27 @@
-#!/usr/bin/env pwsh
-# Per-extension installer — Windows / PowerShell
-# Installs npm dependencies for this extension. No-op if no package.json.
-$ErrorActionPreference = "Stop"
-# Drop npm_config_allow_scripts inherited from a parent npm/npx process:
-# npm >=11 rejects it for project installs (EALLOWSCRIPTS). File config still applies.
-Remove-Item Env:\npm_config_allow_scripts -ErrorAction SilentlyContinue
-Remove-Item Env:\NPM_CONFIG_ALLOW_SCRIPTS -ErrorAction SilentlyContinue
-$extDir = $PSScriptRoot
-if (-not $extDir) { $extDir = Split-Path -Parent $MyInvocation.MyCommand.Path }
-if (-not (Test-Path "$extDir/package.json")) {
-  Write-Host "  - no package.json — skipping" -ForegroundColor DarkGray
-  exit 0
-}
-if (-not (Get-Command npm -ErrorAction SilentlyContinue)) {
-  Write-Host "  x npm not found — install Node.js 22+" -ForegroundColor Red
-  exit 1
-}
-if (Test-Path "$extDir/package-lock.json") {
-  Write-Host "  - npm ci" -ForegroundColor DarkGray
-  npm --prefix $extDir ci --no-audit --no-fund
-} else {
-  Write-Host "  - npm install" -ForegroundColor DarkGray
-  npm --prefix $extDir install --no-audit --no-fund
-}
-if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE }
-Write-Host "  + done" -ForegroundColor Green
+#!/usr/bin/env pwsh
+# Per-extension installer — Windows / PowerShell
+# Installs npm dependencies for this extension. No-op if no package.json.
+$ErrorActionPreference = "Stop"
+# Drop npm_config_allow_scripts inherited from a parent npm/npx process:
+# npm >=11 rejects it for project installs (EALLOWSCRIPTS). File config still applies.
+Remove-Item Env:\npm_config_allow_scripts -ErrorAction SilentlyContinue
+Remove-Item Env:\NPM_CONFIG_ALLOW_SCRIPTS -ErrorAction SilentlyContinue
+$extDir = $PSScriptRoot
+if (-not $extDir) { $extDir = Split-Path -Parent $MyInvocation.MyCommand.Path }
+if (-not (Test-Path "$extDir/package.json")) {
+  Write-Host "  - no package.json — skipping" -ForegroundColor DarkGray
+  exit 0
+}
+if (-not (Get-Command npm -ErrorAction SilentlyContinue)) {
+  Write-Host "  x npm not found — install Node.js 22+" -ForegroundColor Red
+  exit 1
+}
+if (Test-Path "$extDir/package-lock.json") {
+  Write-Host "  - npm ci" -ForegroundColor DarkGray
+  npm --prefix $extDir ci --no-audit --no-fund
+} else {
+  Write-Host "  - npm install" -ForegroundColor DarkGray
+  npm --prefix $extDir install --no-audit --no-fund
+}
+if ($LASTEXITCODE -ne 0) { exit $LASTEXITCODE }
+Write-Host "  + done" -ForegroundColor Green

package/agent/extensions/shared/install.sh

@@ -1,24 +1,24 @@
-#!/bin/sh
-# Per-extension installer — macOS / Linux
-# Installs npm dependencies for this extension. No-op if no package.json.
-set -eu
-# Drop npm_config_allow_scripts inherited from a parent npm/npx process:
-# npm >=11 rejects it for project installs (EALLOWSCRIPTS). File config still applies.
-unset npm_config_allow_scripts NPM_CONFIG_ALLOW_SCRIPTS 2>/dev/null || true
-EXT_DIR="$(cd "$(dirname "$0")" && pwd)"
-if [ ! -f "$EXT_DIR/package.json" ]; then
-  echo "  · no package.json — skipping"
-  exit 0
-fi
-if ! command -v npm >/dev/null 2>&1; then
-  echo "  ✖ npm not found — install Node.js 22+" >&2
-  exit 1
-fi
-if [ -f "$EXT_DIR/package-lock.json" ]; then
-  echo "  · npm ci"
-  npm --prefix "$EXT_DIR" ci --no-audit --no-fund
-else
-  echo "  · npm install"
-  npm --prefix "$EXT_DIR" install --no-audit --no-fund
-fi
-echo "  ✓ done"
+#!/bin/sh
+# Per-extension installer — macOS / Linux
+# Installs npm dependencies for this extension. No-op if no package.json.
+set -eu
+# Drop npm_config_allow_scripts inherited from a parent npm/npx process:
+# npm >=11 rejects it for project installs (EALLOWSCRIPTS). File config still applies.
+unset npm_config_allow_scripts NPM_CONFIG_ALLOW_SCRIPTS 2>/dev/null || true
+EXT_DIR="$(cd "$(dirname "$0")" && pwd)"
+if [ ! -f "$EXT_DIR/package.json" ]; then
+  echo "  · no package.json — skipping"
+  exit 0
+fi
+if ! command -v npm >/dev/null 2>&1; then
+  echo "  ✖ npm not found — install Node.js 22+" >&2
+  exit 1
+fi
+if [ -f "$EXT_DIR/package-lock.json" ]; then
+  echo "  · npm ci"
+  npm --prefix "$EXT_DIR" ci --no-audit --no-fund
+else
+  echo "  · npm install"
+  npm --prefix "$EXT_DIR" install --no-audit --no-fund
+fi
+echo "  ✓ done"

package/agent/extensions/shared/tool-call-timeout.test.ts

@@ -1,117 +1,117 @@
-import assert from "node:assert/strict";
-import test from "node:test";
-import type { ToolDefinition } from "@earendil-works/pi-coding-agent";
-import { Type } from "typebox";
-import {
-  createToolCallTimeoutGuard,
-  runWithToolCallTimeout,
-  CHILD_TOOL_CALL_TIMEOUT_MS,
-  ToolCallTimeoutError,
-} from "./tool-call-timeout.ts";
-
-test("the production timeout error names the tool and three-minute limit", () => {
-  assert.equal(
-    new ToolCallTimeoutError("fixture_tool", CHILD_TOOL_CALL_TIMEOUT_MS)
-      .message,
-    'Tool call "fixture_tool" timed out after 3 minutes.',
-  );
-});
-
-test("a hung tool call fails clearly and receives an abort signal", async () => {
-  let executionSignal: AbortSignal | undefined;
-
-  await assert.rejects(
-    runWithToolCallTimeout("hung_fixture", 10, undefined, (signal) => {
-      executionSignal = signal;
-      return new Promise(() => {});
-    }),
-    (error: unknown) => {
-      assert.equal(error instanceof ToolCallTimeoutError, true);
-      assert.equal(
-        error instanceof Error ? error.message : "",
-        'Tool call "hung_fixture" timed out after 10 ms.',
-      );
-      return true;
-    },
-  );
-
-  assert.equal(executionSignal?.aborted, true);
-  assert.equal(executionSignal?.reason instanceof ToolCallTimeoutError, true);
-});
-
-test("parent cancellation still stops the timeout wrapper immediately", async () => {
-  const controller = new AbortController();
-  const reason = new Error("cancelled fixture");
-  const pending = runWithToolCallTimeout(
-    "hung_fixture",
-    60_000,
-    controller.signal,
-    () => new Promise(() => {}),
-  );
-
-  controller.abort(reason);
-
-  await assert.rejects(pending, (error: unknown) => error === reason);
-});
-
-test("the guard wraps each definition once and can discover later tools", () => {
-  const definitions = new Map<string, ToolDefinition>();
-  const createDefinition = (name: string): ToolDefinition => ({
-    name,
-    label: name,
-    description: name,
-    parameters: Type.Object({}),
-    async execute() {
-      return { content: [{ type: "text", text: "done" }], details: {} };
-    },
-  });
-  const first = createDefinition("first");
-  definitions.set(first.name, first);
-  const registry = {
-    getAllTools: () => [...definitions.keys()].map((name) => ({ name })),
-    getToolDefinition: (name: string) => definitions.get(name),
-  };
-  const guard = createToolCallTimeoutGuard(10);
-
-  const firstExecute = first.execute;
-  guard.apply(registry);
-  const firstWrappedExecute = first.execute;
-  assert.notEqual(firstWrappedExecute, firstExecute);
-
-  const second = createDefinition("second");
-  const secondExecute = second.execute;
-  definitions.set(second.name, second);
-  guard.apply(registry);
-
-  assert.equal(first.execute, firstWrappedExecute);
-  assert.notEqual(second.execute, secondExecute);
-});
-
-test("successful and terminating tool results pass through unchanged", async () => {
-  const result = {
-    content: [{ type: "text" as const, text: "recorded" }],
-    details: { value: "fixture" },
-    terminate: true,
-  };
-
-  assert.equal(
-    await runWithToolCallTimeout(
-      "structured_output",
-      10,
-      undefined,
-      async () => result,
-    ),
-    result,
-  );
-});
-
-test("the timeout is fresh for each tool call, not shared across calls", async () => {
-  const execute = () =>
-    runWithToolCallTimeout("slow_fixture", 100, undefined, async () => {
-      await new Promise((resolve) => setTimeout(resolve, 60));
-      return "done";
-    });
-
-  assert.equal(await execute(), "done");
-  assert.equal(await execute(), "done");
-});
+import assert from "node:assert/strict";
+import test from "node:test";
+import type { ToolDefinition } from "@earendil-works/pi-coding-agent";
+import { Type } from "typebox";
+import {
+  createToolCallTimeoutGuard,
+  runWithToolCallTimeout,
+  CHILD_TOOL_CALL_TIMEOUT_MS,
+  ToolCallTimeoutError,
+} from "./tool-call-timeout.ts";
+
+test("the production timeout error names the tool and three-minute limit", () => {
+  assert.equal(
+    new ToolCallTimeoutError("fixture_tool", CHILD_TOOL_CALL_TIMEOUT_MS)
+      .message,
+    'Tool call "fixture_tool" timed out after 3 minutes.',
+  );
+});
+
+test("a hung tool call fails clearly and receives an abort signal", async () => {
+  let executionSignal: AbortSignal | undefined;
+
+  await assert.rejects(
+    runWithToolCallTimeout("hung_fixture", 10, undefined, (signal) => {
+      executionSignal = signal;
+      return new Promise(() => {});
+    }),
+    (error: unknown) => {
+      assert.equal(error instanceof ToolCallTimeoutError, true);
+      assert.equal(
+        error instanceof Error ? error.message : "",
+        'Tool call "hung_fixture" timed out after 10 ms.',
+      );
+      return true;
+    },
+  );
+
+  assert.equal(executionSignal?.aborted, true);
+  assert.equal(executionSignal?.reason instanceof ToolCallTimeoutError, true);
+});
+
+test("parent cancellation still stops the timeout wrapper immediately", async () => {
+  const controller = new AbortController();
+  const reason = new Error("cancelled fixture");
+  const pending = runWithToolCallTimeout(
+    "hung_fixture",
+    60_000,
+    controller.signal,
+    () => new Promise(() => {}),
+  );
+
+  controller.abort(reason);
+
+  await assert.rejects(pending, (error: unknown) => error === reason);
+});
+
+test("the guard wraps each definition once and can discover later tools", () => {
+  const definitions = new Map<string, ToolDefinition>();
+  const createDefinition = (name: string): ToolDefinition => ({
+    name,
+    label: name,
+    description: name,
+    parameters: Type.Object({}),
+    async execute() {
+      return { content: [{ type: "text", text: "done" }], details: {} };
+    },
+  });
+  const first = createDefinition("first");
+  definitions.set(first.name, first);
+  const registry = {
+    getAllTools: () => [...definitions.keys()].map((name) => ({ name })),
+    getToolDefinition: (name: string) => definitions.get(name),
+  };
+  const guard = createToolCallTimeoutGuard(10);
+
+  const firstExecute = first.execute;
+  guard.apply(registry);
+  const firstWrappedExecute = first.execute;
+  assert.notEqual(firstWrappedExecute, firstExecute);
+
+  const second = createDefinition("second");
+  const secondExecute = second.execute;
+  definitions.set(second.name, second);
+  guard.apply(registry);
+
+  assert.equal(first.execute, firstWrappedExecute);
+  assert.notEqual(second.execute, secondExecute);
+});
+
+test("successful and terminating tool results pass through unchanged", async () => {
+  const result = {
+    content: [{ type: "text" as const, text: "recorded" }],
+    details: { value: "fixture" },
+    terminate: true,
+  };
+
+  assert.equal(
+    await runWithToolCallTimeout(
+      "structured_output",
+      10,
+      undefined,
+      async () => result,
+    ),
+    result,
+  );
+});
+
+test("the timeout is fresh for each tool call, not shared across calls", async () => {
+  const execute = () =>
+    runWithToolCallTimeout("slow_fixture", 100, undefined, async () => {
+      await new Promise((resolve) => setTimeout(resolve, 60));
+      return "done";
+    });
+
+  assert.equal(await execute(), "done");
+  assert.equal(await execute(), "done");
+});