@sayknow-cli/coding-agent 0.3.10 → 0.3.12

package/src/slash-commands/builtin-registry.ts

@@ -811,12 +811,12 @@ const BUILTIN_SLASH_COMMAND_REGISTRY: ReadonlyArray<SlashCommandSpec> = [
 	{
 		name: "session",
 		priority: 88,
-		description: "Show current session info or delete current session",
+		description: "Show session info or delete the current session transcript/artifacts",
 		acpDescription: "Show session information",
 		acpInputHint: "info|delete",
 		subcommands: [
 			{ name: "info", description: "Show current session id, title, and workspace" },
-			{ name: "delete", description: "Delete current session and return to selector" },
+			{ name: "delete", description: "Delete current session transcript and artifacts" },
 		],
 		allowArgs: true,
 		handle: async (command, runtime) => {
@@ -845,7 +845,10 @@ const BUILTIN_SLASH_COMMAND_REGISTRY: ReadonlyArray<SlashCommandSpec> = [
 					return usage(`Failed to delete session: ${errorMessage(err)}`, runtime);
 				}
 				await runtime.output(
-					`Session deleted: ${sessionFile}. Use ACP \`session/load\` to switch to another session.`,
+					[
+						`Deleted current session transcript and artifacts: ${sessionFile}`,
+						"Other sessions and topic/history metadata were not deleted.",
+					].join("\n"),
 				);
 				return commandConsumed();
 			}
@@ -1021,8 +1024,10 @@ const BUILTIN_SLASH_COMMAND_REGISTRY: ReadonlyArray<SlashCommandSpec> = [
 				return commandConsumed();
 			}
 			if (args === "login" || args.startsWith("login ")) {
+				const providerId = args.slice("login".length).trim();
+				const loginCommand = providerId ? `/login ${providerId}` : "/login [provider-id]";
 				await runtime.output(
-					"Use the terminal UI /login selector for browser, device-code, or manual callback provider login.",
+					`Open the terminal UI and run ${loginCommand} for OAuth/subscription account login. Paste callbacks with /login <redirect URL or code>.`,
 				);
 				return commandConsumed();
 			}

package/src/tools/bash-interactive.ts

@@ -12,7 +12,6 @@ import {
 } from "@sayknow-cli/tui";
 import { sanitizeText } from "@sayknow-cli/utils";
 import type { Terminal as XtermTerminalType } from "@xterm/headless";
-import xterm from "@xterm/headless";
 import { Settings } from "../config/settings";
 import { NON_INTERACTIVE_ENV } from "../exec/non-interactive-env";
 import type { Theme } from "../modes/theme/theme";
@@ -32,8 +31,6 @@ function normalizeCaptureChunk(chunk: string): string {
 	return sanitizeWithOptionalSixelPassthrough(normalized, sanitizeText);
 }
 
-const XtermTerminal = xterm.Terminal;
-
 function normalizeInputForPty(data: string, applicationCursorKeysMode: boolean): string {
 	const kitty = parseKittySequence(data);
 	if (kitty?.eventType === 3) {
@@ -113,6 +110,7 @@ class BashInteractiveOverlayComponent implements Component {
 		private readonly command: string,
 		private readonly uiTheme: Theme,
 		private readonly getTerminalRows: () => number,
+		XtermTerminal: typeof XtermTerminalType,
 	) {
 		this.#terminal = new XtermTerminal({
 			cols: 120,
@@ -305,10 +303,17 @@ export async function runInteractiveBashPty(
 		headBytes: resolveOutputSinkHeadBytes(settings),
 		maxColumns: resolveOutputMaxColumns(settings),
 	});
+	const { default: xterm } = await import("@xterm/headless");
+	const XtermTerminal = xterm.Terminal;
 	const result = await ui.custom<BashInteractiveResult>(
 		(tui, uiTheme, _keybindings, done) => {
 			const session = new PtySession();
-			const component = new BashInteractiveOverlayComponent(options.command, uiTheme, () => tui.terminal.rows);
+			const component = new BashInteractiveOverlayComponent(
+				options.command,
+				uiTheme,
+				() => tui.terminal.rows,
+				XtermTerminal,
+			);
 			component.setSession(session);
 			let finished = false;
 			const finalize = (run: PtyRunResult) => {

package/src/tools/browser/readable.ts

@@ -1,5 +1,5 @@
 import { Readability } from "@mozilla/readability";
-import { parseHTML } from "linkedom";
+import { parseHtmlLazy } from "../../utils/linkedom";
 import { htmlToBasicMarkdown } from "../../web/scrapers/types";
 
 export type ReadableFormat = "text" | "markdown";
@@ -31,7 +31,7 @@ export async function extractReadableFromHtml(
 	url: string,
 	format: ReadableFormat,
 ): Promise<ReadableResult | null> {
-	const { document } = parseHTML(html);
+	const { document } = await parseHtmlLazy(html);
 
 	// --- Primary: Readability article extraction ---
 	const article = new Readability(document).parse();

package/src/tools/fetch.ts

@@ -5,7 +5,6 @@ import type { ImageContent, TextContent } from "@sayknow-cli/ai";
 import { htmlToMarkdown } from "@sayknow-cli/natives";
 import { type Component, Text } from "@sayknow-cli/tui";
 import { $which, ptree, truncate } from "@sayknow-cli/utils";
-import { parseHTML } from "linkedom";
 import type { Settings } from "../config/settings";
 import type { RenderResultOptions } from "../extensibility/custom-tools/types";
 import { type Theme, theme } from "../modes/theme/theme";
@@ -15,6 +14,7 @@ import { DEFAULT_MAX_BYTES, truncateHead } from "../session/streaming-output";
 import { renderStatusLine } from "../tui";
 import { CachedOutputBlock } from "../tui/output-block";
 import { formatDimensionNote, resizeImage } from "../utils/image-resize";
+import { parseHtmlLazy } from "../utils/linkedom";
 import { ensureTool } from "../utils/tools-manager";
 import { INSANE_NOTES, tryInsaneFetch } from "../web/insane/bridge";
 import { validatePublicHttpUrl, validatePublicHttpUrlForInsane } from "../web/insane/url-guard";
@@ -483,9 +483,9 @@ function cleanFeedText(text: string): string {
 /**
  * Parse RSS/Atom feed to markdown
  */
-function parseFeedToMarkdown(content: string, maxItems = 10): string {
+async function parseFeedToMarkdown(content: string, maxItems = 10): Promise<string> {
 	try {
-		const doc = parseHTML(content).document;
+		const doc = (await parseHtmlLazy(content)).document;
 
 		// Try RSS
 		const channel = doc.querySelector("channel");
@@ -1026,7 +1026,7 @@ async function renderUrl(
 	}
 
 	if (isFeed || (isXml && (rawContent.includes("<rss") || rawContent.includes("<feed")))) {
-		const parsed = parseFeedToMarkdown(rawContent);
+		const parsed = await parseFeedToMarkdown(rawContent);
 		const output = finalizeOutput(parsed);
 		return {
 			url,
@@ -1119,7 +1119,7 @@ async function renderUrl(
 			const altResult = await loadPage(resolved, { timeout, signal });
 			if (altResult.ok && altResult.content.trim().length > 200) {
 				notes.push(`Used feed alternate: ${resolved}`);
-				const parsed = parseFeedToMarkdown(altResult.content);
+				const parsed = await parseFeedToMarkdown(altResult.content);
 				const output = finalizeOutput(parsed);
 				return {
 					url,

package/src/tools/skill.ts

@@ -4,15 +4,16 @@
  * Lets the agent hand off to another available skill in the current turn. The
  * callee's SKILL.md is dispatched through the same custom-message path used by
  * `/skill:<name>` typing, as a user-attribution message delivered same-turn
- * (without `deliverAs: "nextTurn"`). Before dispatch, the tool calls
- * `skc state <caller> handoff --to <callee>` in-process via the state-runtime
- * function so caller and callee mode-states plus `skill-active-state.json`
- * transition atomically.
+ * (without `deliverAs: "nextTurn"`). When the caller is one of the canonical
+ * workflow skills, the tool first runs the native state handoff so workflow
+ * mode-state and `skill-active-state.json` transition atomically. Runtime
+ * project/user skills have no native workflow mode-state, so they are dispatched
+ * directly and tracked by the prompt observer instead.
  *
- * Chaining is refused unless the caller's `current_phase` is in
- * `{complete, completed, handoff, failed, cancelled, canceled, inactive}`. The
- * agent declares readiness either by writing `current_phase: "handoff"` to its
- * mode-state or by running the handoff verb directly.
+ * Canonical workflow chaining is refused unless the caller's `current_phase` is
+ * in `{complete, completed, handoff, failed, cancelled, canceled, inactive}`.
+ * The agent declares readiness either by writing `current_phase: "handoff"` to
+ * its mode-state or by running the handoff verb directly.
  */
 
 import type { AgentTool, AgentToolResult } from "@sayknow-cli/agent-core";
@@ -24,6 +25,7 @@ import { buildSkillPromptMessage } from "../extensibility/skills";
 import skillDescription from "../prompts/tools/skill.md" with { type: "text" };
 import { SKILL_PROMPT_MESSAGE_TYPE } from "../session/messages";
 import { runNativeStateCommand } from "../skc-runtime/state-runtime";
+import { isCanonicalSkcWorkflowSkill } from "../skill-state/active-state";
 import type { ToolSession } from ".";
 import { ToolError } from "./tool-errors";
 
@@ -130,9 +132,10 @@ export class SkillTool implements AgentTool<typeof skillSchema, SkillToolDetails
 				throw new ToolError(`skill tool: unknown skill "${requestedName}".${hint}`);
 			}
 
-			// Phase guard + atomic handoff. Only runs when transitioning between
-			// distinct skills (same-skill recursion was already refused above).
-			if (activeSkill) {
+			// Phase guard + atomic native handoff only apply to canonical workflow
+			// skills. Runtime project/user skills do not have a native mode-state,
+			// so there is no `skc state <skill>` command to run for them.
+			if (activeSkill && isCanonicalSkcWorkflowSkill(activeSkill)) {
 				const phase = (this.#session.getActiveSkillPhase?.() ?? "running").trim().toLowerCase();
 				if (!TERMINAL_PHASES.has(phase)) {
 					throw new ToolError(

package/src/utils/linkedom.ts

@@ -0,0 +1,14 @@
+type LinkedomModule = typeof import("linkedom");
+
+type ParseHtml = LinkedomModule["parseHTML"];
+
+let parseHtml: ParseHtml | undefined;
+let linkedomImport: Promise<LinkedomModule> | undefined;
+
+export async function parseHtmlLazy(markup: string): Promise<ReturnType<ParseHtml>> {
+	if (!parseHtml) {
+		linkedomImport ??= import("linkedom");
+		parseHtml = (await linkedomImport).parseHTML;
+	}
+	return parseHtml(markup);
+}

package/src/utils/pasted-image-path.ts

@@ -1,12 +1,12 @@
 /**
  * Resolve pasted editor text to an image file path.
  *
- * Terminals insert a shell-escaped filesystem path when a file is drag-dropped
- * onto them (e.g. macOS `Screenshot\ 2026-07-07\ at\ 11.06.38 PM.png`, where
- * the visible "space" before AM/PM is U+202F and stays unescaped). When the
- * entire paste is a single path to an existing image file, the interactive
- * editor attaches the image and inserts an `[image N]` placeholder instead of
- * leaving the raw path in the prompt.
+ * Some terminal/clipboard integrations paste a temporary filesystem path after a
+ * copied image is pasted into the editor (for example `/tmp/clipboard-...png`).
+ * Only those recognized clipboard temp files are auto-attached and replaced with
+ * an `[image N] source="/path"` reference so the model receives both the image
+ * payload and the raw temp file path. Ordinary image paths remain literal prompt
+ * text; users attach saved files explicitly with `@path/to/image.png`.
  */
 import * as fs from "node:fs";
 import * as os from "node:os";
@@ -14,6 +14,7 @@ import * as path from "node:path";
 import { fileURLToPath } from "node:url";
 
 const IMAGE_FILE_EXTENSION_PATTERN = /\.(?:png|jpe?g|gif|webp)$/i;
+const CLIPBOARD_TEMP_BASENAME_PATTERN = /^clipboard-\d{4}-\d{2}-\d{2}-\d{6}-[A-Za-z0-9-]+\.(?:png|jpe?g|gif|webp)$/i;
 
 export interface DecodePastedPathOptions {
 	/**
@@ -150,16 +151,25 @@ function hasSupportedImageMagic(filePath: string): boolean {
 	}
 }
 
+function isRecognizedClipboardTempPath(filePath: string): boolean {
+	const resolved = path.resolve(filePath);
+	const tempRoot = path.resolve(os.tmpdir());
+	if (resolved !== tempRoot && !resolved.startsWith(`${tempRoot}${path.sep}`)) return false;
+	return CLIPBOARD_TEMP_BASENAME_PATTERN.test(path.basename(resolved));
+}
+
 /**
- * Returns the resolved path when the whole pasted text is a single path to an
- * existing image file (verified by extension AND content signature),
- * otherwise `undefined` (the paste is inserted as text).
+ * Returns the resolved path when the whole pasted text is a recognized clipboard
+ * temp path to an existing image file, otherwise `undefined` (the paste is
+ * inserted as text). Arbitrary saved image paths are not auto-attached here;
+ * users attach them explicitly with `@path/to/image.png`.
  */
 export function resolvePastedImagePath(text: string, options?: ResolvePastedImagePathOptions): string | undefined {
 	const candidate = decodePastedPathCandidate(text, options);
 	if (!candidate || !IMAGE_FILE_EXTENSION_PATTERN.test(candidate)) return undefined;
 
 	const resolved = path.resolve(options?.cwd ?? process.cwd(), candidate);
+	if (!isRecognizedClipboardTempPath(resolved)) return undefined;
 	try {
 		if (!fs.statSync(resolved).isFile()) return undefined;
 	} catch {
@@ -168,3 +178,7 @@ export function resolvePastedImagePath(text: string, options?: ResolvePastedImag
 	if (!hasSupportedImageMagic(resolved)) return undefined;
 	return resolved;
 }
+
+export function formatPastedImageReference(placeholder: string, imagePath: string): string {
+	return `${placeholder} source=${JSON.stringify(imagePath)}`;
+}

package/src/web/scrapers/arxiv.ts

@@ -1,4 +1,4 @@
-import { parseHTML } from "linkedom";
+import { parseHtmlLazy } from "../../utils/linkedom";
 import type { RenderResult, SpecialHandler } from "./types";
 import { buildResult, loadPage } from "./types";
 import { convertWithMarkit, fetchBinary } from "./utils";
@@ -31,7 +31,7 @@ export const handleArxiv: SpecialHandler = async (
 		if (!result.ok) return null;
 
 		// Parse the Atom feed response
-		const doc = parseHTML(result.content).document;
+		const doc = (await parseHtmlLazy(result.content)).document;
 		const entry = doc.querySelector("entry");
 
 		if (!entry) return null;

package/src/web/scrapers/go-pkg.ts

@@ -1,5 +1,5 @@
 import { tryParseJson } from "@sayknow-cli/utils";
-import { parseHTML } from "linkedom";
+import { parseHtmlLazy } from "../../utils/linkedom";
 import type { RenderResult, SpecialHandler } from "./types";
 import { buildResult, htmlToBasicMarkdown, loadPage } from "./types";
 
@@ -97,7 +97,7 @@ export const handleGoPkg: SpecialHandler = async (
 			});
 		}
 
-		const doc = parseHTML(pageResult.content).document;
+		const doc = (await parseHtmlLazy(pageResult.content)).document;
 
 		// Extract actual module path from breadcrumb or header
 		const breadcrumb = doc.querySelector(".go-Breadcrumb");

package/src/web/scrapers/iacr.ts

@@ -1,4 +1,4 @@
-import { parseHTML } from "linkedom";
+import { parseHtmlLazy } from "../../utils/linkedom";
 import type { RenderResult, SpecialHandler } from "./types";
 import { buildResult, loadPage } from "./types";
 import { convertWithMarkit, fetchBinary } from "./utils";
@@ -30,7 +30,7 @@ export const handleIacr: SpecialHandler = async (
 
 		if (!result.ok) return null;
 
-		const doc = parseHTML(result.content).document;
+		const doc = (await parseHtmlLazy(result.content)).document;
 
 		// Extract metadata from the page
 		const title =

package/src/web/scrapers/readthedocs.ts

@@ -1,7 +1,7 @@
 /**
  * Read the Docs handler for web-fetch
  */
-import { parseHTML } from "linkedom";
+import { parseHtmlLazy } from "../../utils/linkedom";
 import { buildResult, htmlToBasicMarkdown, loadPage, type RenderResult, type SpecialHandler } from "./types";
 
 export const handleReadTheDocs: SpecialHandler = async (
@@ -39,7 +39,7 @@ export const handleReadTheDocs: SpecialHandler = async (
 	}
 
 	// Parse HTML
-	const root = parseHTML(result.content).document;
+	const root = (await parseHtmlLazy(result.content)).document;
 
 	// Extract main content from common Read the Docs selectors
 	let mainContent =

package/src/web/scrapers/twitter.ts

@@ -1,5 +1,6 @@
-import { type HTMLElement, parseHTML } from "linkedom";
+import type { HTMLElement } from "linkedom";
 import { ToolAbortError } from "../../tools/tool-errors";
+import { parseHtmlLazy } from "../../utils/linkedom";
 import type { RenderResult, SpecialHandler } from "./types";
 import { buildResult, loadPage } from "./types";
 
@@ -33,7 +34,7 @@ export const handleTwitter: SpecialHandler = async (
 
 			if (result.ok && result.content.length > 500) {
 				// Parse the Nitter HTML
-				const doc = parseHTML(result.content).document;
+				const doc = (await parseHtmlLazy(result.content)).document;
 
 				// Extract tweet content
 				const tweetContent = doc.querySelector(".tweet-content")?.textContent?.trim();

package/src/web/scrapers/wikipedia.ts

@@ -1,4 +1,4 @@
-import { parseHTML } from "linkedom";
+import { parseHtmlLazy } from "../../utils/linkedom";
 import type { RenderResult, SpecialHandler } from "./types";
 import { buildResult, loadPage } from "./types";
 
@@ -45,7 +45,7 @@ export const handleWikipedia: SpecialHandler = async (
 		const contentResult = await loadPage(contentUrl, { timeout, signal });
 
 		if (contentResult.ok) {
-			const doc = parseHTML(contentResult.content).document;
+			const doc = (await parseHtmlLazy(contentResult.content)).document;
 
 			// Extract main content sections
 			const sections = doc.querySelectorAll("section");