neoctl 0.2.14 → 0.2.16

package/dist/web/index.d.ts

@@ -52,17 +52,19 @@ interface UiLine {
     title?: string;
     bodyTitle?: string;
     titleStatus?: "success" | "failure";
+    toolKind?: string;
     format?: "markdown" | "ansi" | "plain" | "diff";
     previewStyle?: "summary";
     summaryMaxLines?: number;
     live?: boolean;
     collapsible?: boolean;
     image?: UiLineImage;
+    artifact?: unknown;
 }
 interface UiActiveTool {
     id: string;
     name: string;
-    input: unknown;
+    kind: string;
     startedAt: number;
 }
 interface UiStatus {
@@ -96,6 +98,10 @@ export interface CreateWebRuntimeOptions {
     cwd?: string;
     /** Additional tools to register in the web runtime. */
     externalTools?: readonly Tool[];
+    /** Additional shared skill roots. Defaults can also be provided with NEO_SKILL_ROOTS. */
+    skillRoots?: readonly string[];
+    /** Writable root for skill_create/skill_update. Defaults to NEO_SKILL_CREATE_ROOT or ~/.neoctl/skills. */
+    skillCreateRoot?: string;
 }
 export interface WebRuntimeScope {
     /** Browser-tab or client-instance identifier. Omit for the legacy singleton runtime. */
@@ -271,6 +277,7 @@ export declare class WebRepl {
     private handleCommandOrPrompt;
     private runCompaction;
     private send;
+    private sendSerialized;
     private broadcastSync;
 }
 export {};

package/dist/web/index.js

@@ -2,7 +2,7 @@
 import http from "node:http";
 import fs from "node:fs/promises";
 import path from "node:path";
-import { existsSync, readFileSync } from "node:fs";
+import { createReadStream, existsSync, readFileSync } from "node:fs";
 import { fileURLToPath } from "node:url";
 import { createRequire } from "node:module";
 import { QueryEngine } from "../core/query-engine.js";
@@ -28,9 +28,14 @@ import { createTaskTools } from "../tasks/task-tools.js";
 import { TaskStore } from "../tasks/task-store.js";
 import { parseReplCommand, helpText, replCommandDefinitions } from "../repl/commands.js";
 import { writeSessionMarkdownExport } from "../session/session-export.js";
+import { DefaultContextManager } from "../context/context-manager.js";
+import { buildEffectiveSystemPrompt } from "../context/prompts.js";
 import { WEB_HTML } from "./html.js";
 import { openDirectory } from "../open-directory.js";
-import { resolveImageBlockDataSync } from "../core/image-storage.js";
+import { getNeoctlHome } from "../paths.js";
+import { FileSystemSkillCatalog } from "../skills/skill-filesystem.js";
+import { createSkillTool } from "../skills/skill-tool.js";
+import { createSkillManagementTools } from "../skills/skill-management-tools.js";
 const require = createRequire(import.meta.url);
 const markedPackageDir = path.dirname(require.resolve("marked/package.json"));
 const highlightPackageDir = path.dirname(require.resolve("@highlightjs/cdn-assets/package.json"));
@@ -109,6 +114,85 @@ function parseWebArgs(argv) {
         port = 3000;
     return { host, port: Math.round(port) };
 }
+class SkillCatalogContextManager {
+    catalog;
+    base;
+    constructor(catalog, base = new DefaultContextManager()) {
+        this.catalog = catalog;
+        this.base = base;
+    }
+    async build(input) {
+        const runtimeContext = await this.base.build(input);
+        const skillSection = await buildSkillCatalogPromptSection(this.catalog);
+        if (!skillSection)
+            return runtimeContext;
+        const promptSections = [...runtimeContext.promptSections, skillSection];
+        return {
+            ...runtimeContext,
+            promptSections,
+            systemPrompt: buildEffectiveSystemPrompt(promptSections, input),
+        };
+    }
+}
+async function buildSkillCatalogPromptSection(catalog) {
+    const skills = await catalog.list();
+    if (skills.length === 0)
+        return undefined;
+    const visible = skills.slice(0, 80);
+    const lines = visible.map((skill) => {
+        const tags = skill.tags?.length ? `; tags=${skill.tags.join(",")}` : "";
+        const tools = skill.allowedTools?.length ? `; allowedTools=${skill.allowedTools.join(",")}` : "";
+        return `- ${skill.name}: ${skill.description} (execution=${skill.execution}${tags}${tools})`;
+    });
+    if (skills.length > visible.length)
+        lines.push(`- ... ${skills.length - visible.length} more skills available; use skill_list for the full catalog.`);
+    return {
+        name: "Available Skills",
+        cacheStable: false,
+        content: [
+            "Reusable skills are available through the `skill` tool and the /skill UI command.",
+            "When the user's task matches a skill name, description, tags, or domain capability, proactively call the `skill` tool before doing the work directly.",
+            "Do not wait for the user to explicitly say 'use skill'. Use skill_list/skill_read if you need to inspect details.",
+            "Available skill catalog:",
+            ...lines,
+        ].join("\n"),
+    };
+}
+function resolveSkillCatalogRoots(cwd, extraRoots = [], createRoot) {
+    const userRoot = path.resolve(process.env.NEO_SKILL_CREATE_ROOT || createRoot || path.join(getNeoctlHome(), "skills"));
+    const configuredRoots = splitPathList(process.env.NEO_SKILL_ROOTS);
+    const workspaceRoot = path.resolve(cwd, ".neo", "skills");
+    const roots = uniquePaths([
+        userRoot,
+        ...extraRoots,
+        ...configuredRoots,
+        workspaceRoot,
+    ]).map((root) => ({
+        root,
+        kind: path.resolve(root) === userRoot ? "user" : "workspace",
+    }));
+    return { roots, createRoot: userRoot };
+}
+function splitPathList(value) {
+    return String(value || "")
+        .split(path.delimiter)
+        .map((item) => item.trim())
+        .filter(Boolean)
+        .map((item) => path.resolve(item));
+}
+function uniquePaths(values) {
+    const seen = new Set();
+    const result = [];
+    for (const value of values) {
+        const resolved = path.resolve(value);
+        const key = process.platform === "win32" ? resolved.toLowerCase() : resolved;
+        if (seen.has(key))
+            continue;
+        seen.add(key);
+        result.push(resolved);
+    }
+    return result;
+}
 export async function createWebRuntime(options = {}) {
     const envLoad = loadDefaultDotEnvFiles({ override: true });
     const cwd = options.cwd ? path.resolve(options.cwd) : process.cwd();
@@ -117,6 +201,11 @@ export async function createWebRuntime(options = {}) {
     const modelGateway = new LoggingModelGateway(createModelGatewayFromProcessEnv(process.env), communicationLogger);
     const taskStore = new TaskStore();
     const tools = new ToolRegistry();
+    const { roots: skillRoots, createRoot: skillCreateRoot } = resolveSkillCatalogRoots(cwd, options.skillRoots, options.skillCreateRoot);
+    const skills = new FileSystemSkillCatalog({
+        roots: skillRoots,
+        createRoot: skillCreateRoot,
+    });
     tools.register(editTool);
     tools.register(writeTool);
     tools.register(createExecTool({ taskStore }));
@@ -129,6 +218,9 @@ export async function createWebRuntime(options = {}) {
     if (modelConfig?.provider === "openai")
         tools.register(createOpenAIImageGenerationTool());
     tools.register(planTool);
+    tools.register(createSkillTool(skills));
+    for (const tool of createSkillManagementTools(skills, { requireApproval: true, allowDelete: false }))
+        tools.register(tool);
     for (const tool of options.externalTools ?? [])
         tools.register(tool);
     const agentRuntime = { modelGateway, tools, taskStore };
@@ -155,6 +247,8 @@ export async function createWebRuntime(options = {}) {
         modelGateway,
         tools,
         appPromptStore,
+        contextManager: new SkillCatalogContextManager(skills),
+        skills: (await skills.list()).map((skill) => skill.name),
         taskNotificationSource: createTaskNotificationSource(taskStore),
         commands: replCommandDefinitions.map((command) => command.usage),
         session: {
@@ -266,6 +360,7 @@ export class WebRepl {
     queuedAttachments;
     foregroundRun;
     foregroundRunToken = 0;
+    pendingUserImageEchoLabels;
     backgroundSessionRuns = new Map();
     suppressReattachedStreaming = new Set();
     backgroundTaskCount;
@@ -508,7 +603,7 @@ export class WebRepl {
         this.broadcastSync();
     }
     setStatus(next) {
-        this.status = next;
+        this.status = next.phase === "running_tools" ? next : { ...next, currentTool: undefined };
         this.broadcastSync();
     }
     finalizeForegroundView() {
@@ -611,6 +706,11 @@ export class WebRepl {
     }
     handleEvent(event) {
         this.reduce(event);
+        if (event.type === "message" && this.matchesPendingUserImageEcho(event.message)) {
+            this.pendingUserImageEchoLabels = undefined;
+            this.broadcastSync();
+            return;
+        }
         if (event.type === "state" || event.type === "context.metrics" || event.type === "usage" || event.type === "tool_call.delta" || event.type === "retrying") {
             this.broadcastSync();
             return;
@@ -695,6 +795,15 @@ export class WebRepl {
         if (event.type === "error")
             this.append({ kind: "error", text: event.error.message });
     }
+    matchesPendingUserImageEcho(message) {
+        const pending = this.pendingUserImageEchoLabels;
+        if (!pending?.length || message.role !== "user")
+            return false;
+        const labels = message.blocks
+            .filter((block) => block.type === "image" && typeof block.label === "string")
+            .map((block) => block.label);
+        return labels.length === pending.length && labels.every((label, index) => label === pending[index]);
+    }
     async handleCommandOrPrompt(text, attachments = []) {
         const command = parseReplCommand(text);
         if (command.type === "exit") {
@@ -794,6 +903,10 @@ export class WebRepl {
         this.append({ kind: "user", text: promptPayload.displayText });
         for (const line of imageLinesForBlocks("user", promptPayload.blocks))
             this.append(line);
+        const optimisticImageLabels = (promptPayload.blocks ?? [])
+            .filter((block) => block.type === "image" && typeof block.label === "string")
+            .map((block) => block.label);
+        this.pendingUserImageEchoLabels = optimisticImageLabels.length ? optimisticImageLabels : undefined;
         const runToken = ++this.foregroundRunToken;
         const abortController = new AbortController();
         this.activeAbortController = abortController;
@@ -833,6 +946,7 @@ export class WebRepl {
                 this.activeAbortController = undefined;
             this.interruptArmed = false;
             this.finalizeForegroundView();
+            this.pendingUserImageEchoLabels = undefined;
             const queuedText = this.queuedInput;
             const queuedAttach = this.queuedAttachments;
             this.queuedInput = undefined;
@@ -885,10 +999,14 @@ export class WebRepl {
         res.write(`event: ${event}\n`);
         res.write(`data: ${JSON.stringify(data)}\n\n`);
     }
+    sendSerialized(res, event, data) {
+        res.write(`event: ${event}\n`);
+        res.write(`data: ${data}\n\n`);
+    }
     broadcastSync() {
-        const payload = this.snapshot(false);
+        const payload = JSON.stringify(this.snapshot(false));
         for (const res of this.subscribers)
-            this.send(res, "sync", payload);
+            this.sendSerialized(res, "sync", payload);
     }
 }
 function reqKeepAlive(res) {
@@ -907,6 +1025,8 @@ async function route(req, res, router) {
             return sendFile(res, highlightAssetPath, "text/javascript; charset=utf-8");
         if (req.method === "GET" && url.pathname === "/vendor/highlight-theme.css")
             return sendFile(res, highlightThemeAssetPath, "text/css; charset=utf-8");
+        if (req.method === "GET" && url.pathname === "/api/images")
+            return sendImage(res, url.searchParams.get("path"), url.searchParams.get("mime"));
         const scope = webRuntimeScopeFromUrl(url);
         const repl = await router.get(scope);
         if (req.method === "GET" && url.pathname === "/events")
@@ -970,6 +1090,23 @@ async function sendFile(res, filepath, contentType) {
     res.writeHead(200, { "Content-Type": contentType, "Cache-Control": "public, max-age=3600" });
     res.end(body);
 }
+async function sendImage(res, encodedPath, mimeType) {
+    const filepath = decodeImagePath(encodedPath);
+    if (!filepath)
+        return sendJson(res, { error: "missing image path" }, 400);
+    const contentType = safeImageContentType(mimeType);
+    const binaryPath = filepath.endsWith(".base64.txt") ? filepath.slice(0, -".base64.txt".length) : filepath;
+    if (existsSync(binaryPath)) {
+        res.writeHead(200, { "Content-Type": contentType, "Cache-Control": "no-store" });
+        createReadStream(binaryPath).pipe(res);
+        return;
+    }
+    if (!existsSync(filepath))
+        return sendJson(res, { error: "image not found" }, 404);
+    const base64 = (await fs.readFile(filepath, "utf8")).trim();
+    res.writeHead(200, { "Content-Type": contentType, "Cache-Control": "no-store" });
+    res.end(Buffer.from(stripDataUrlPrefix(base64), "base64"));
+}
 function sendJson(res, value, status = 200) {
     res.writeHead(status, { "Content-Type": "application/json; charset=utf-8", "Cache-Control": "no-store" });
     res.end(JSON.stringify(value));
@@ -1074,7 +1211,7 @@ function reduceStatus(status, event) {
     if (event.type === "state")
         return { ...status, phase: event.phase, detail: event.detail, currentTool: event.phase === "preparing" || event.phase === "calling_model" || event.phase === "ready" ? undefined : status.currentTool, usage: event.phase === "preparing" ? undefined : status.usage, streamedOutputTokens: event.phase === "preparing" ? 0 : status.streamedOutputTokens, inputTokenUpdatedAt: event.phase === "preparing" ? undefined : status.inputTokenUpdatedAt, outputTokenUpdatedAt: event.phase === "preparing" ? undefined : status.outputTokenUpdatedAt, retryCooldownUntil: event.phase === "preparing" ? undefined : status.retryCooldownUntil, activityTick: status.activityTick + 1 };
     if (event.type === "tool.started")
-        return { ...status, phase: "running_tools", detail: event.toolUse.name, currentTool: { id: event.toolUse.id, name: event.toolUse.name, input: event.toolUse.input, startedAt: Date.now() }, activityTick: status.activityTick + 1 };
+        return { ...status, phase: "running_tools", detail: event.toolUse.name, currentTool: { id: event.toolUse.id, name: event.toolUse.name, kind: toolKindForToolUse(event.toolUse.name, event.toolUse.input), startedAt: Date.now() }, activityTick: status.activityTick + 1 };
     if (event.type === "tool.finished")
         return { ...status, currentTool: status.currentTool?.id === event.toolUse.id ? undefined : status.currentTool, activityTick: status.activityTick + 1 };
     if (event.type === "context.metrics")
@@ -1095,6 +1232,21 @@ function reduceStatus(status, event) {
         return { ...status, activityTick: status.activityTick + 1 };
     return status;
 }
+function toolKindForToolUse(toolName, input) {
+    if (toolName === "image2")
+        return isRecord(input) && input.mode === "edit" ? "修图" : "作图";
+    if (toolName === "edit" || toolName === "write" || toolName.includes("artifact_editor") || toolName.includes("apply_patch"))
+        return "编辑";
+    if (toolName === "exec" || toolName.includes("shell") || toolName.includes("command"))
+        return "执行";
+    if (toolName.includes("download"))
+        return "下载";
+    if (toolName === "read" || toolName === "list" || toolName === "grep" || toolName === "search" || toolName.includes("query") || toolName.includes("load"))
+        return "查询";
+    if (toolName === "plan")
+        return "计划";
+    return "工具";
+}
 async function handleExportCommand(outputPath, runtime) {
     const snapshot = runtime.engine.snapshot();
     if (!snapshot.session)
@@ -1495,20 +1647,43 @@ function imageLineForBlock(role, block) {
         kind,
         text: block.label ?? `[image ${block.mimeType}]`,
         image: {
-            src: imageBlockToDataUrl(block),
+            src: imageBlockToSrc(block),
             label: block.label,
             mimeType: block.mimeType,
         },
     };
 }
-function imageBlockToDataUrl(block) {
-    const data = resolveImageBlockDataSync(block);
+function imageBlockToSrc(block) {
+    if (block.storage?.path) {
+        return `/api/images?path=${encodeImagePath(block.storage.path)}&mime=${encodeURIComponent(block.mimeType)}`;
+    }
+    const data = block.data.trim();
     if (!data)
         return "";
     if (data.startsWith("data:"))
         return data;
     return `data:${block.mimeType};base64,${data}`;
 }
+function encodeImagePath(filepath) {
+    return Buffer.from(filepath, "utf8").toString("base64url");
+}
+function decodeImagePath(value) {
+    if (!value)
+        return undefined;
+    try {
+        return Buffer.from(value, "base64url").toString("utf8");
+    }
+    catch {
+        return undefined;
+    }
+}
+function safeImageContentType(value) {
+    return value && /^image\/[a-z0-9.+-]+$/i.test(value) ? value : "application/octet-stream";
+}
+function stripDataUrlPrefix(value) {
+    const match = /^data:[^;]+;base64,(.*)$/is.exec(value);
+    return match ? match[1] : value;
+}
 function assistantText(message) {
     const text = message.blocks.filter((block) => block.type === "text").map((block) => block.text).join("");
     return text.length > 0 ? text : undefined;
@@ -1561,14 +1736,21 @@ function thinkingLine(text, live = false) {
     return { kind: "thinking", title: titleForKind("thinking"), text, previewStyle: "summary", summaryMaxLines: THINKING_SUMMARY_MAX_LINES, live };
 }
 function formatToolUse(toolUse) {
+    const toolKind = toolKindForToolUse(toolUse.name, toolUse.input);
     if (toolUse.name === "plan" && isPlanToolPayload(toolUse.input))
-        return { kind: "tool", title: toolTitle(toolUse.name, "running"), bodyTitle: planToolBodyTitle(toolUse.input), text: formatPlanToolPayload(toolUse.input), collapsible: true };
+        return { kind: "tool", title: toolTitle(toolUse.name, "running"), bodyTitle: planToolBodyTitle(toolUse.input), toolKind, text: formatPlanToolPayload(toolUse.input), collapsible: true };
     const description = toolUse.name === "exec" ? execDescriptionFromInput(toolUse.input) : undefined;
-    return { kind: "tool", title: toolTitle(toolUse.name, "running"), bodyTitle: description, text: formatReplData(toolUse.input, 1200), previewStyle: "summary", collapsible: true };
+    return { kind: "tool", title: toolTitle(toolUse.name, "running"), bodyTitle: description, toolKind, text: formatReplData(toolUse.input, 1200), previewStyle: "summary", collapsible: true };
 }
 function formatToolResultLine(toolName, output, ok) {
     const formatted = formatToolResult(toolName, output, ok);
-    return { kind: ok ? "tool" : "error", title: toolTitle(toolName, "finished"), bodyTitle: formatted.bodyTitle, titleStatus: ok ? "success" : "failure", text: formatted.text, format: formatted.format, live: false, previewStyle: formatted.full ? undefined : "summary", summaryMaxLines: formatted.summaryMaxLines, collapsible: true };
+    return { kind: ok ? "tool" : "error", title: toolTitle(toolName, "finished"), bodyTitle: formatted.bodyTitle, titleStatus: ok ? "success" : "failure", toolKind: toolKindForToolUse(toolName, undefined), text: formatted.text, format: formatted.format, live: false, previewStyle: formatted.full ? undefined : "summary", summaryMaxLines: formatted.summaryMaxLines, collapsible: true, artifact: xhsArtifactFromToolOutput(toolName, output) };
+}
+function xhsArtifactFromToolOutput(toolName, output) {
+    if (toolName !== "open_xhs_artifact_editor" || !isRecord(output))
+        return undefined;
+    const artifact = output.artifact;
+    return isRecord(artifact) && typeof artifact.id === "string" ? artifact : undefined;
 }
 function toolTitle(toolName, _phase) {
     return toolName;
@@ -1695,6 +1877,17 @@ function formatImageGenerationToolResult(output, ok) {
     const quality = typeof output.quality === "string" ? output.quality : undefined;
     const format = typeof output.outputFormat === "string" ? output.outputFormat : undefined;
     const sourceImages = typeof output.sourceImages === "number" ? output.sourceImages : undefined;
+    const prompt = typeof output.prompt === "string" ? output.prompt.trim() : "";
+    const semanticName = typeof output.semanticName === "string" ? output.semanticName.trim() : "";
+    const background = typeof output.background === "string" ? output.background : undefined;
+    const startedAtIso = typeof output.startedAtIso === "string" ? output.startedAtIso : undefined;
+    const finishedAtIso = typeof output.finishedAtIso === "string" ? output.finishedAtIso : undefined;
+    const revisedPrompt = Array.isArray(output.images)
+        ? output.images
+            .filter(isRecord)
+            .map((image) => typeof image.revisedPrompt === "string" ? image.revisedPrompt.trim() : "")
+            .find((value) => Boolean(value))
+        : undefined;
     const lines = [`${mode === "edit" ? "edited" : "generated"} ${returnedImages ?? 0} image${returnedImages === 1 ? "" : "s"}`];
     const details = [provider, model, size, quality && quality !== "auto" ? quality : undefined, format].filter((value) => Boolean(value));
     if (details.length > 0)
@@ -1704,6 +1897,19 @@ function formatImageGenerationToolResult(output, ok) {
     const duration = imageGenerationDuration(output);
     if (duration !== undefined)
         lines.push(`duration: ${duration}ms`);
+    if (prompt)
+        lines.push(`prompt: ${prompt}`);
+    if (revisedPrompt && revisedPrompt !== prompt)
+        lines.push(`revisedPrompt: ${revisedPrompt}`);
+    lines.push(`mode: ${mode}`);
+    if (semanticName)
+        lines.push(`semanticName: ${semanticName}`);
+    if (background)
+        lines.push(`background: ${background}`);
+    if (startedAtIso)
+        lines.push(`startedAt: ${startedAtIso}`);
+    if (finishedAtIso)
+        lines.push(`finishedAt: ${finishedAtIso}`);
     return lines.join("\n");
 }
 function imageGenerationDuration(output) {
@@ -1916,4 +2122,4 @@ if (process.argv[1] && path.resolve(process.argv[1]) === fileURLToPath(import.me
         process.exitCode = 1;
     });
 }
-//# sourceMappingURL=index.js.map
+//# sourceMappingURL=index.js.map