@cydm/pie 1.0.7 → 1.0.9

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@cydm/pie",
-	"version": "1.0.7",
+	"version": "1.0.9",
 	"description": "Pie AI Agent CLI",
 	"type": "module",
 	"bin": {
@@ -16,7 +16,7 @@
 		"validate:models": "tsx scripts/validate-models-config.ts",
 		"test": "npm run test:offline",
 		"pretest:offline": "npm run build --workspace @pie/agent-framework && npm run build --workspace @pie/shared-headless-capabilities && npm run build",
-		"test:offline": "node --test --import tsx test/core-services.test.ts test/cli.test.ts test/execution-state-manager.test.ts test/queued-steer-execution-state.test.ts test/architecture-boundaries.test.ts test/attachment-resolver.test.ts test/kimi-attachment-extension.test.ts test/document-attachment-extension.test.ts test/auto-compact-overflow.test.ts test/hierarchical-command-system.test.ts test/message-queue-unit.test.ts test/evals-harness.test.ts test/session-trace.test.ts test/runtime-mode-context.test.ts test/tool-policy.test.ts test/yolo-command.test.ts test/source-size-guard.test.ts test/runtime-logger.test.ts test/release-readiness.test.ts test/code-intel-typescript-provider.test.ts test/mention-file-select.terminal.test.ts test/tool-partial-display.terminal.test.ts test/tool-name-rendering.test.ts test/tools-registry.test.ts test/system-prompt-contract.test.ts test/daily-agent-smoke.terminal.test.ts test/viewport-position-final.test.ts",
+		"test:offline": "node --test --import tsx test/core-services.test.ts test/cli.test.ts test/execution-state-manager.test.ts test/queued-steer-execution-state.test.ts test/architecture-boundaries.test.ts test/attachment-resolver.test.ts test/kimi-attachment-extension.test.ts test/document-attachment-extension.test.ts test/auto-compact-overflow.test.ts test/hierarchical-command-system.test.ts test/message-queue-unit.test.ts test/evals-harness.test.ts test/session-trace.test.ts test/runtime-mode-context.test.ts test/tool-policy.test.ts test/yolo-command.test.ts test/footer-lifecycle.test.ts test/files-utils.test.ts test/source-size-guard.test.ts test/runtime-logger.test.ts test/release-readiness.test.ts test/code-intel-typescript-provider.test.ts test/init-extension-contract.test.ts test/skills-runtime.test.ts test/mention-file-select.terminal.test.ts test/tool-partial-display.terminal.test.ts test/tool-name-rendering.test.ts test/tools-registry.test.ts test/system-prompt-contract.test.ts test/daily-agent-smoke.terminal.test.ts test/viewport-position-final.test.ts",
 		"test:todo-tui": "PIE_REAL_TODO_TUI=1 node --test --import tsx test/todo-closure.terminal.test.ts",
 		"test:daily-tui": "npm run test:daily-tui:quick",
 		"test:daily-tui:quick": "PIE_REAL_DAILY_TUI=1 PIE_DAILY_TUI_LEVEL=quick node --test --import tsx test/daily-agent-smoke.terminal.test.ts",

package/dist/builtin/skills/pie-unity-rpc/SKILL.md

@@ -1,121 +0,0 @@
----
-name: pie-unity-rpc
-description: Connect to a local pie-unity instance for Unity project discovery, health checks, manifest lookup, and a small set of stable RPC/tool calls. Use when working on a Unity project that imports pie-unity.
----
-
-# Pie Unity RPC
-
-Use the helper script `pie-unity-rpc.js` to talk to local `pie-unity` instances. Prefer the shell-style host workflow:
-
-1. Discover instances
-2. Select the target project or instance
-3. Check health
-4. Read only the manifest slice you need
-5. Query or inspect Unity state
-6. Edit the scene, read logs, refresh Unity after file changes, or run a frame-scheduled Unity script task
-7. Verify the result
-
-Do not dump the full manifest unless the user explicitly asks for it.
-
-Script resource: `pie-unity-rpc.js`
-
-Command templates below use `<script>` as a placeholder for the resolved script path in the current host.
-
-## Discover
-
-```bash
-node <script> instances
-node <script> instances --project "/abs/path/to/project"
-```
-
-Selection behavior:
-
-- if there is only one active instance, the helper uses it automatically
-- `--project` is treated as a matching hint, not only as an exact path
-- if multiple candidates remain, the helper returns a structured candidate list; show it and ask the user which instance to use
-- prefer `--instance` when the user already knows the target instance ID
-
-## Health
-
-```bash
-node <script> health --project "/abs/path/to/project"
-node <script> health --instance "<instance-id>"
-```
-
-## Manifest
-
-Read only what you need:
-
-```bash
-node <script> manifest --project "/abs/path/to/project"
-node <script> manifest --project "/abs/path/to/project" --namespace chat
-node <script> manifest --project "/abs/path/to/project" --name unity_project_inspect
-```
-
-## Host Workflow
-
-Prefer the host commands over raw `tool` names:
-
-```bash
-node <script> query --project "/abs/path/to/project" --data '{"scope":"scene_object","name":"Main Camera","limit":1}'
-node <script> inspect --project "/abs/path/to/project" --data '{"target":{"path":"Main Camera"}}'
-node <script> edit --project "/abs/path/to/project" --data '{"action":"create_scene_object","name":"CubeA","primitiveType":"Cube","x":0,"y":1,"z":0}'
-node <script> log-read --project "/abs/path/to/project" --data '{"source":"active","tailLines":120,"contains":"error"}'
-```
-
-For Unity project files, use normal Pie file tools rooted at the Unity project. The RPC helper deliberately does not provide separate project file read/write commands.
-
-## Script Run
-
-Use `script-run` only for JavaScript generator tasks that should execute inside Unity. The script must define a generator entrypoint. For multi-frame work, yield `ctx.nextFrame()`, `ctx.waitFrames(n)`, or `ctx.waitSeconds(s)`. The command returns after the task completes, fails, or times out.
-
-```bash
-node <script> script-run --project "/abs/path/to/project" --data '{"script":"export function* run(ctx, args) { return ctx.query({ scope: \"scene_object\", limit: 3 }); }"}'
-node <script> script-run --project "/abs/path/to/project" --data '{"script":"export function* run(ctx, args) { for (let i = 0; i < 60; i++) { ctx.log(\"frame\", i); yield ctx.nextFrame(); } return { ok: true }; }","totalTimeoutMs":10000,"maxFrames":120}'
-```
-
-Do not pass C#, shader source, or raw file contents to `script-run`. Use normal Pie file tools for Unity project files, then call `unity_refresh` through `tool` if Unity must import changed assets. Do not write long synchronous loops. Express long goals as short generator steps that yield between frames.
-
-## Raw Tool / RPC
-
-Use raw `tool` or `rpc` only when the shell-style commands do not cover what you need.
-
-## Retry Behavior
-
-The helper automatically retries when `pie-unity` reports temporary unavailability during compilation or domain reload. Use `--wait-ms` to override the default polling delay.
-
-When using `rpc --method pie_chat.set_config`, the helper waits for the Unity-side `configAppliedVersion` ack when available, then adds a short settle window before returning. Use `--config-settle-ms` to override that delay if needed.
-
-## Guidance
-
-- Start with `instances` if the target Unity project is not yet confirmed.
-- Start with `health` if the project is known.
-- Read this skill contract first. Do not start by scanning the local repo or helper source code unless the task is explicitly about the repo/tooling itself.
-- If a command returns an ambiguous instance error, present the candidate list and ask the user to choose one instance.
-- Read `manifest --namespace unity` or `manifest --namespace unity.script` before using unfamiliar host commands.
-- Once the target `projectPath` is known, keep filesystem exploration scoped to that Unity project. Do not search unrelated repo directories to rediscover the project.
-- For scene setup with named objects such as `Player`, `Goal`, or `Main Camera`, start with `query` for exact names before creating anything.
-- If `query` finds exactly one match, reuse and update it instead of creating a duplicate.
-- If `query` finds multiple matches, do not create another duplicate; inspect/select one explicit result or ask the user.
-- Prefer `query -> inspect -> edit -> inspect` over ad-hoc menu execution. Use `edit` actions `add_component`, `remove_component`, `set_component_enabled`, and `set_component_property` for common scene object component changes instead of writing one-off script-run code.
-- Use normal file tools with the Unity project root for project text files; do not route file access through Unity RPC.
-- Read helper source code only when command behavior is unclear after reading this skill contract or manifest output.
-- After editing C# scripts under `Assets`, call `tool --tool unity_refresh --data {}` if needed, then expect a brief compile/domain-reload window. Re-check `health` or retry the next host command instead of assuming Unity is stuck. If compile/runtime errors appear, use `log-read` instead of guessing.
-- Prefer `script-run` when the task needs filtering, derived values, or multi-frame composition inside Unity.
-
-## Smoke Test
-
-Recommended minimum verification sequence:
-
-```bash
-node <script> health --project "/abs/path/to/project"
-node <script> manifest --project "/abs/path/to/project" --namespace unity
-node <script> tool --project "/abs/path/to/project" --tool chat_send --data '{"text":"respond with exactly pong"}'
-node <script> tool --project "/abs/path/to/project" --tool chat_get_state
-node <script> tool --project "/abs/path/to/project" --tool chat_get_messages
-```
-
-Expected result:
-
-- `chat_get_state` eventually returns `isBusy: false`
-- the final assistant message contains the expected response

package/dist/builtin/skills/pie-unity-rpc/pie-unity-rpc.js

@@ -1,417 +0,0 @@
-#!/usr/bin/env node
-
-import { readFileSync, existsSync } from "node:fs";
-import { homedir } from "node:os";
-import path from "node:path";
-import { pathToFileURL } from "node:url";
-
-const REGISTRY_FILE = path.join(homedir(), ".pie-unity", "registry.json");
-const DEFAULT_WAIT_MS = 600;
-const DEFAULT_RETRIES = 30;
-const ACTIVE_INSTANCE_MAX_AGE_SEC = 120;
-
-class InstanceSelectionError extends Error {
-	constructor(code, message, details = {}) {
-		super(message);
-		this.name = "InstanceSelectionError";
-		this.code = code;
-		this.details = details;
-	}
-}
-
-export function parseArgs(argv) {
-	const args = argv.slice(2);
-	const command = args[0] || "instances";
-	const flags = {};
-	for (let i = 1; i < args.length; i += 1) {
-		const item = args[i];
-		if (!item.startsWith("--")) continue;
-		const key = item.slice(2);
-		const next = args[i + 1];
-		if (!next || next.startsWith("--")) {
-			flags[key] = "true";
-			continue;
-		}
-		flags[key] = next;
-		i += 1;
-	}
-	return { command, flags };
-}
-
-export function loadRegistry(filePath = REGISTRY_FILE) {
-	if (!existsSync(filePath)) {
-		return [];
-	}
-	try {
-		const raw = readFileSync(filePath, "utf8").replace(/^\uFEFF/, "");
-		const json = JSON.parse(raw);
-		const items = Array.isArray(json?.instances) ? json.instances : [];
-		return dedupeInstances(items.filter(Boolean));
-	} catch {
-		return [];
-	}
-}
-
-export function getActiveInstances(instances, nowUnix = Math.floor(Date.now() / 1000)) {
-	const recent = instances.filter((item) => {
-		const lastSeenUnix = Number(item?.lastSeenUnix || 0);
-		return lastSeenUnix > 0 && nowUnix - lastSeenUnix <= ACTIVE_INSTANCE_MAX_AGE_SEC;
-	});
-	return recent.length > 0 ? recent : instances;
-}
-
-function basenameMatch(left, right) {
-	return path.basename(left || "") !== ""
-		&& path.basename(left || "") === path.basename(right || "");
-}
-
-function computeProjectMatchScore(requestedProject, candidateProject) {
-	const requested = normalizePath(requestedProject || "");
-	const candidate = normalizePath(candidateProject || "");
-	if (!requested || !candidate) return 0;
-	if (requested === candidate) return 1000;
-	if (basenameMatch(requested, candidate)) return 700;
-
-	const requestedWithSlash = requested.endsWith("/") ? requested : `${requested}/`;
-	const candidateWithSlash = candidate.endsWith("/") ? candidate : `${candidate}/`;
-	if (requestedWithSlash.startsWith(candidateWithSlash) || candidateWithSlash.startsWith(requestedWithSlash)) {
-		const distance = Math.abs(requested.length - candidate.length);
-		return 500 - Math.min(distance, 400);
-	}
-
-	return 0;
-}
-
-function formatCandidate(item) {
-	return {
-		instanceId: String(item?.instanceId || ""),
-		projectPath: normalizePath(item?.projectPath || ""),
-		projectName: String(item?.projectName || ""),
-		mode: String(item?.mode || ""),
-		port: Number(item?.port || 0),
-		lastSeenUnix: Number(item?.lastSeenUnix || 0),
-	};
-}
-
-function buildSelectionError(code, message, flags, candidates) {
-	return new InstanceSelectionError(code, message, {
-		reason: code === "PIE_UNITY_INSTANCE_AMBIGUOUS" ? "ambiguous" : "no_match",
-		requestedProject: normalizePath(flags.project || ""),
-		requestedInstance: String(flags.instance || "").trim(),
-		requestedPort: Number(flags.port || 0) || 0,
-		candidates: candidates.map(formatCandidate),
-	});
-}
-
-export function selectInstance(instances, flags) {
-	const activeInstances = getActiveInstances(instances);
-	const project = normalizePath(flags.project || "");
-	const instanceId = String(flags.instance || "").trim();
-	const explicitPort = Number(flags.port || 0);
-
-	if (explicitPort > 0) {
-		return {
-			instanceId: instanceId || `port_${explicitPort}`,
-			projectPath: project,
-			port: explicitPort,
-			mode: "unknown",
-		};
-	}
-
-	if (instanceId) {
-		const match = activeInstances.find((item) => String(item.instanceId || "") === instanceId);
-		if (match) return match;
-		throw buildSelectionError(
-			"PIE_UNITY_INSTANCE_NOT_FOUND",
-			`No pie-unity instance found for --instance ${instanceId}.`,
-			flags,
-			activeInstances,
-		);
-	}
-
-	if (project && activeInstances.length > 0) {
-		const scored = activeInstances
-			.map((item) => ({ item, score: computeProjectMatchScore(project, item.projectPath || "") }))
-			.filter((entry) => entry.score > 0)
-			.sort((left, right) => right.score - left.score || Number(right.item.lastSeenUnix || 0) - Number(left.item.lastSeenUnix || 0));
-
-		if (scored.length === 1) {
-			return scored[0].item;
-		}
-
-		if (scored.length > 1) {
-			const bestScore = scored[0].score;
-			const best = scored.filter((entry) => entry.score === bestScore).map((entry) => entry.item);
-			if (best.length === 1) {
-				return best[0];
-			}
-			throw buildSelectionError(
-				"PIE_UNITY_INSTANCE_AMBIGUOUS",
-				`Multiple pie-unity instances match project hint ${project}. Pass --instance or --port.`,
-				flags,
-				best,
-			);
-		}
-	}
-
-	if (activeInstances.length === 1) {
-		return activeInstances[0];
-	}
-
-	if (activeInstances.length > 1) {
-		throw buildSelectionError(
-			"PIE_UNITY_INSTANCE_AMBIGUOUS",
-			"Multiple pie-unity instances found. Pass --instance, --port, or a more specific --project.",
-			flags,
-			activeInstances,
-		);
-	}
-
-	throw buildSelectionError(
-		"PIE_UNITY_INSTANCE_NOT_FOUND",
-		project
-			? `No matching pie-unity instance found for project hint ${project}.`
-			: "No matching pie-unity instance found.",
-		flags,
-		activeInstances,
-	);
-}
-
-export async function requestJson(url, init = {}, options = {}) {
-	const waitMs = Number(options.waitMs || DEFAULT_WAIT_MS);
-	const retries = Number(options.retries || DEFAULT_RETRIES);
-	let lastError;
-	for (let attempt = 0; attempt < retries; attempt += 1) {
-		try {
-			const response = await fetch(url, init);
-			const text = await response.text();
-			const json = text ? JSON.parse(text) : {};
-
-			if (!response.ok) {
-				throw new Error(`HTTP ${response.status}: ${text}`);
-			}
-
-			const unavailable = extractAvailability(json);
-			if (unavailable) {
-				lastError = new Error(unavailable.message || "pie-unity temporarily unavailable");
-				if (attempt < retries - 1) {
-					await sleep(waitMs);
-					continue;
-				}
-			}
-
-			return json;
-		} catch (error) {
-			lastError = error;
-			if (attempt >= retries - 1) break;
-			await sleep(waitMs);
-		}
-	}
-	throw lastError || new Error("Request failed");
-}
-
-export async function runCli(argv = process.argv) {
-	const { command, flags } = parseArgs(argv);
-	if (command === "instances") {
-		const instances = getActiveInstances(loadRegistry());
-		const filtered = flags.project
-			? instances.filter((item) => computeProjectMatchScore(flags.project, item.projectPath || "") > 0)
-			: instances;
-		writeJson({ service: "pie-unity", instances: filtered });
-		return;
-	}
-
-	const instance = selectInstance(loadRegistry(), flags);
-	const baseUrl = `http://127.0.0.1:${instance.port}`;
-
-	switch (command) {
-		case "health": {
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/health`, {}, flags)));
-			return;
-		}
-		case "manifest": {
-			const query = new URLSearchParams();
-			if (flags.namespace) query.set("namespace", flags.namespace);
-			if (flags.name) query.set("name", flags.name);
-			const suffix = query.toString() ? `?${query.toString()}` : "";
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/manifest${suffix}`, {}, flags)));
-			return;
-		}
-		case "query": {
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/tool/${encodeURIComponent("unity_scene_query")}`, postJson(parseData(flags.data)), flags)));
-			return;
-		}
-		case "inspect": {
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/tool/${encodeURIComponent("unity_scene_object_inspect")}`, postJson(parseData(flags.data)), flags)));
-			return;
-		}
-		case "edit": {
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/tool/${encodeURIComponent("unity_scene_object_edit")}`, postJson(parseData(flags.data)), flags)));
-			return;
-		}
-		case "log-read": {
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/tool/${encodeURIComponent("unity_log_read")}`, postJson(parseData(flags.data)), flags)));
-			return;
-		}
-		case "script-run": {
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/tool/${encodeURIComponent("unity_script_run")}`, postJson(parseData(flags.data)), flags)));
-			return;
-		}
-		case "tool": {
-			const name = String(flags.tool || "").trim();
-			if (!name) throw new Error("--tool is required");
-			const body = parseData(flags.data);
-			writeJson(normalizeEnvelope(await requestJson(`${baseUrl}/tool/${encodeURIComponent(name)}`, postJson(body), flags)));
-			return;
-		}
-		case "rpc": {
-			const name = String(flags.method || "").trim();
-			if (!name) throw new Error("--method is required");
-			const body = parseData(flags.data);
-			const response = normalizeEnvelope(await requestJson(`${baseUrl}/rpc/${encodeURIComponent(name)}`, postJson(body), flags));
-			if (name === "pie_chat.set_config") {
-				await settleChatConfig(baseUrl, flags, Number(response?.result?.configAppliedVersion || 0));
-			}
-			writeJson(response);
-			return;
-		}
-		default:
-			throw new Error(`Unknown command: ${command}`);
-	}
-}
-
-function extractAvailability(json) {
-	const raw = json?.serverAvailability || json?.availability || "";
-	if (!raw) return null;
-	if (typeof raw === "string") {
-		try {
-			return JSON.parse(raw);
-		} catch {
-			return { message: raw };
-		}
-	}
-	return raw;
-}
-
-function normalizeEnvelope(json) {
-	if (!json || typeof json !== "object") return json;
-	if (!("result" in json) && !("serverAvailability" in json)) return json;
-
-	const next = { ...json };
-	if (typeof next.result === "string") {
-		const trimmed = next.result.trim();
-		if (trimmed === "null") {
-			next.result = null;
-		} else if ((trimmed.startsWith("{") && trimmed.endsWith("}")) || (trimmed.startsWith("[") && trimmed.endsWith("]"))) {
-			try {
-				next.result = JSON.parse(trimmed);
-			} catch {
-				// Keep original string.
-			}
-		}
-	}
-
-	if (typeof next.serverAvailability === "string" && next.serverAvailability.trim()) {
-		try {
-			next.serverAvailability = JSON.parse(next.serverAvailability);
-		} catch {
-			// Keep original string.
-		}
-	}
-
-	return next;
-}
-
-function parseData(input) {
-	if (!input) return {};
-	return JSON.parse(input);
-}
-
-function postJson(body) {
-	return {
-		method: "POST",
-		headers: { "Content-Type": "application/json" },
-		body: JSON.stringify(body || {}),
-	};
-}
-
-async function settleChatConfig(baseUrl, flags, expectedVersion = 0) {
-	const settleMs = Math.max(0, Number(flags.configSettleMs || 500));
-	const pollMs = Math.max(100, Math.min(settleMs || 200, 250));
-	const deadline = Date.now() + Math.max(1500, settleMs * 4 || 1500);
-
-	while (Date.now() < deadline) {
-		try {
-			const state = normalizeEnvelope(
-				await requestJson(`${baseUrl}/tool/${encodeURIComponent("chat_get_state")}`, {}, {
-					waitMs: pollMs,
-					retries: 1,
-				}),
-			);
-			const appliedVersion = Number(state?.result?.configAppliedVersion || 0);
-			const versionApplied = expectedVersion > 0 ? appliedVersion >= expectedVersion : false;
-			if (versionApplied || !state?.result?.isBusy) {
-				if (settleMs > 0) {
-					await sleep(settleMs);
-				}
-				return;
-			}
-		} catch {
-			// Ignore transient polling failures while the bridge settles.
-		}
-		await sleep(pollMs);
-	}
-
-	if (settleMs > 0) {
-		await sleep(settleMs);
-	}
-}
-
-function sleep(ms) {
-	return new Promise((resolve) => setTimeout(resolve, ms));
-}
-
-function normalizePath(value) {
-	return String(value || "").replace(/\\/g, "/");
-}
-
-function writeJson(payload) {
-	process.stdout.write(`${JSON.stringify(payload, null, 2)}\n`);
-}
-
-function dedupeInstances(items) {
-	const sorted = [...items].sort((a, b) => {
-		const timeA = Number(a?.lastSeenUnix || 0);
-		const timeB = Number(b?.lastSeenUnix || 0);
-		return timeB - timeA;
-	});
-	const seen = new Set();
-	const next = [];
-	for (const item of sorted) {
-		const key = [
-			String(item?.instanceId || ""),
-			normalizePath(item?.projectPath || ""),
-			String(item?.mode || ""),
-		].join("|");
-		if (seen.has(key)) continue;
-		seen.add(key);
-		next.push(item);
-	}
-	return next;
-}
-
-if (process.argv[1] && import.meta.url === pathToFileURL(process.argv[1]).href) {
-	runCli().catch((error) => {
-		if (error instanceof InstanceSelectionError) {
-			process.stderr.write(`${JSON.stringify({
-				error: error.message,
-				code: error.code,
-				...error.details,
-			}, null, 2)}\n`);
-			process.exitCode = 2;
-			return;
-		}
-		process.stderr.write(`${error?.message || error}\n`);
-		process.exitCode = 1;
-	});
-}