@sayknow-cli/coding-agent 0.5.15 → 0.5.17

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## [Unreleased]
 
+## [0.5.17] - 2026-09-16
+
+### Fixed
+
+- Compute the bundled UI-skill directive before workflow-state lookups in `UserPromptSubmit`, and isolate each lookup so a Linux CI recovery/activation failure cannot return empty context for `animate` / `ralplan` routing.
+
+## [0.5.16] - 2026-09-16
+
+### Fixed
+
+- Native skill hooks never throw out of `UserPromptSubmit` / `Stop`. A failed install-root or state lookup returns empty context instead of aborting the turn, which is what collapsed the bundled UI-skill directive in Linux CI.
+
 ## [0.5.15] - 2026-09-16
 
 ### Fixed

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@sayknow-cli/coding-agent",
-	"version": "0.5.15",
+	"version": "0.5.17",
 	"description": "Sayknow-CLI CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://sayknow-cli.com",
 	"author": "jaybeyond",
@@ -54,12 +54,12 @@
 		"@agentclientprotocol/sdk": "1.3.0",
 		"@babel/parser": "^7.29.3",
 		"@mozilla/readability": "^0.6.0",
-		"@sayknow-cli/stats": "0.5.15",
-		"@sayknow-cli/agent-core": "0.5.15",
-		"@sayknow-cli/ai": "0.5.15",
-		"@sayknow-cli/natives": "0.5.15",
-		"@sayknow-cli/tui": "0.5.15",
-		"@sayknow-cli/utils": "0.5.15",
+		"@sayknow-cli/stats": "0.5.17",
+		"@sayknow-cli/agent-core": "0.5.17",
+		"@sayknow-cli/ai": "0.5.17",
+		"@sayknow-cli/natives": "0.5.17",
+		"@sayknow-cli/tui": "0.5.17",
+		"@sayknow-cli/utils": "0.5.17",
 		"@puppeteer/browsers": "^2.13.0",
 		"@types/turndown": "5.0.6",
 		"@xterm/headless": "^6.0.0",

package/src/hooks/native-skill-hook.ts

@@ -288,47 +288,98 @@ function readSessionFile(payload: HookPayload): string | undefined {
 export async function dispatchSkcNativeSkillHook(
 	payload: HookPayload,
 	options: SkcNativeHookDispatchOptions = {},
+): Promise<SkcNativeHookDispatchResult> {
+	try {
+		return await dispatchSkcNativeSkillHookInner(payload, options);
+	} catch {
+		// A native hook must never fail the turn. Surface nothing rather than
+		// aborting UserPromptSubmit / Stop over an install-root or state lookup.
+		const hookEventName = readHookEventName(payload);
+		return { hookEventName, outputJson: null };
+	}
+}
+
+async function dispatchSkcNativeSkillHookInner(
+	payload: HookPayload,
+	options: SkcNativeHookDispatchOptions = {},
 ): Promise<SkcNativeHookDispatchResult> {
 	const hookEventName = readHookEventName(payload);
 	const cwd = (options.cwd ?? safeString(payload.cwd).trim()) || process.cwd();
 	if (hookEventName === "UserPromptSubmit") {
-		const recoveryDiagnostics = await collectUserPromptStateRecoveryDiagnostics({
-			cwd,
-			sessionId: readSessionId(payload),
-			threadId: readThreadId(payload),
-			stateDir: options.stateDir,
-			prompt: readPromptText(payload),
-			sessionFile: readSessionFile(payload),
-		});
-		const recoveryContext = buildStateRecoveryDiagnosticsContext(recoveryDiagnostics);
 		const prompt = readPromptText(payload);
-		const skillState = prompt
-			? await recordSkillActivation({
-					cwd,
-					text: prompt,
-					sessionId: readSessionId(payload),
-					threadId: readThreadId(payload),
-					turnId: readTurnId(payload),
-					stateDir: options.stateDir,
-				})
-			: null;
-		const effectiveSkillConfig = skillState
-			? await resolveEffectiveSkillConfig(cwd, options.effectiveSkillConfig, options.configPaths, {
-					sessionId: readSessionId(payload),
-					threadId: readThreadId(payload),
-					stateDir: options.stateDir,
-				})
-			: undefined;
-		const activeUltragoalContext = skillState
-			? null
-			: await buildActiveUltragoalPromptContext({
-					cwd,
-					sessionId: readSessionId(payload),
-					threadId: readThreadId(payload),
-					stateDir: options.stateDir,
-					prompt,
-					sessionFile: readSessionFile(payload),
-				});
+		// Compute the bundled UI-skill directive first so a later state/recovery
+		// lookup cannot abort the turn without it.
+		const uiSkillContext = buildUiSkillActivationContext(prompt);
+		let externalUiContext: string | null = null;
+		try {
+			externalUiContext = await buildExternalUiSkillContext({
+				cwd,
+				home: options.home ?? os.homedir(),
+				text: prompt,
+			});
+		} catch {
+			externalUiContext = null;
+		}
+
+		let recoveryContext: string | null = null;
+		try {
+			const recoveryDiagnostics = await collectUserPromptStateRecoveryDiagnostics({
+				cwd,
+				sessionId: readSessionId(payload),
+				threadId: readThreadId(payload),
+				stateDir: options.stateDir,
+				prompt,
+				sessionFile: readSessionFile(payload),
+			});
+			recoveryContext = buildStateRecoveryDiagnosticsContext(recoveryDiagnostics);
+		} catch {
+			recoveryContext = null;
+		}
+
+		let skillState = null;
+		try {
+			skillState = prompt
+				? await recordSkillActivation({
+						cwd,
+						text: prompt,
+						sessionId: readSessionId(payload),
+						threadId: readThreadId(payload),
+						turnId: readTurnId(payload),
+						stateDir: options.stateDir,
+					})
+				: null;
+		} catch {
+			skillState = null;
+		}
+
+		let effectiveSkillConfig: EffectiveSkillConfigInput | undefined;
+		try {
+			effectiveSkillConfig = skillState
+				? await resolveEffectiveSkillConfig(cwd, options.effectiveSkillConfig, options.configPaths, {
+						sessionId: readSessionId(payload),
+						threadId: readThreadId(payload),
+						stateDir: options.stateDir,
+					})
+				: undefined;
+		} catch {
+			effectiveSkillConfig = undefined;
+		}
+
+		let activeUltragoalContext: string | null = null;
+		try {
+			activeUltragoalContext = skillState
+				? null
+				: await buildActiveUltragoalPromptContext({
+						cwd,
+						sessionId: readSessionId(payload),
+						threadId: readThreadId(payload),
+						stateDir: options.stateDir,
+						prompt,
+						sessionFile: readSessionFile(payload),
+					});
+		} catch {
+			activeUltragoalContext = null;
+		}
 		if (activeUltragoalContext?.startsWith("BLOCK_ULTRAGOAL_COMPLETION:")) {
 			return {
 				hookEventName,
@@ -342,25 +393,9 @@ export async function dispatchSkcNativeSkillHook(
 				},
 			};
 		}
-		// Frontend skills SKC routes to but does not vendor (license-restricted
-		// collections). Resolved against the user's own install roots. A lookup
-		// failure must not swallow the bundled UI-skill directive.
-		let externalUiContext: string | null = null;
-		try {
-			externalUiContext = await buildExternalUiSkillContext({
-				cwd,
-				home: options.home ?? os.homedir(),
-				text: prompt,
-			});
-		} catch {
-			externalUiContext = null;
-		}
 		const additionalContext = [
 			skillState ? buildSkillActivationAdditionalContext(skillState, effectiveSkillConfig) : activeUltragoalContext,
-			// Bundled frontend UI skills are advertised independently of workflow
-			// activation: they have no mode state and must fire even when a
-			// workflow keyword already matched.
-			buildUiSkillActivationContext(prompt),
+			uiSkillContext,
 			externalUiContext,
 			recoveryContext,
 			classifyQuestionOnlyPrompt(prompt),