@sayknow-cli/coding-agent 0.3.5 → 0.3.7

package/dist/types/skc-runtime/session-state-sidecar.d.ts

@@ -1,5 +1,7 @@
+import { postmortem } from "@sayknow-cli/utils";
 export declare const SKC_COORDINATOR_SESSION_STATE_FILE_ENV = "SKC_COORDINATOR_SESSION_STATE_FILE";
 export declare const SKC_COORDINATOR_SESSION_ID_ENV = "SKC_COORDINATOR_SESSION_ID";
+export declare const SKC_COORDINATOR_SESSION_BRANCH_ENV = "SKC_COORDINATOR_SESSION_BRANCH";
 export type RuntimeState = "ready_for_input" | "running" | "needs_user_input" | "completed" | "errored";
 interface RuntimeStateEvent {
     type: string;
@@ -9,6 +11,7 @@ interface RuntimeStateContext {
     sessionId: string;
     cwd: string;
     sessionFile?: string | null;
+    branch?: string | null;
 }
 export type TerminalRuntimeStateStatus = {
     terminal: true;
@@ -24,4 +27,6 @@ export declare function readTerminalRuntimeStateMarker(input: {
     sessionFile?: string | null;
 }): Promise<TerminalRuntimeStateStatus>;
 export declare function persistCoordinatorRuntimeStateFromEvent(event: RuntimeStateEvent, context: RuntimeStateContext): Promise<void>;
+export declare function persistCoordinatorRuntimeStateFromPostmortem(reason: postmortem.Reason, context: RuntimeStateContext): void;
+export declare function registerCoordinatorRuntimeStateFinalizer(context: RuntimeStateContext): () => void;
 export {};

package/dist/types/skc-runtime/state-writer.d.ts

@@ -67,6 +67,7 @@ export type GuardedWriteResult = {
     path: string;
     written: true;
     revision: number;
+    stamped: unknown;
 } | {
     path: string;
     written: false;

package/dist/types/skc-runtime/team-runtime.d.ts

@@ -280,7 +280,7 @@ export declare function pruneTeamWorkerGcRecord(record: GcRecord, probe: GcPidPr
 export declare function persistSkcTeamModeStateSummary(snapshot: SkcTeamSnapshot, cwd?: string): Promise<void>;
 export declare function recoverSkcTeamStaleClaims(teamName: string, cwd?: string, env?: NodeJS.ProcessEnv): Promise<SkcTeamLivenessRecoveryResult>;
 type SkcTeamTaskMetadataInput = Partial<Pick<SkcTeamTask, "owner" | "lane" | "required_role" | "allowed_roles" | "depends_on" | "blocked_by">>;
-export declare function resolveSkcWorkerCommand(cwd?: string, env?: NodeJS.ProcessEnv): string;
+export declare function resolveSkcWorkerCommand(cwd?: string, env?: NodeJS.ProcessEnv, platform?: NodeJS.Platform, argv?: string[], execPath?: string): string;
 /** @internal Exported for unit tests. */
 export declare function buildWorkerCommand(config: SkcTeamConfig, worker: SkcTeamWorker, platform?: NodeJS.Platform): string;
 export type SkcWorkerCheckpointClassification = {

package/dist/types/skc-runtime/tmux-common.d.ts

@@ -60,6 +60,18 @@ export declare function buildSkcTmuxExactOptionTarget(sessionName: string, opts?
     platform?: NodeJS.Platform;
     binary?: ResolvedTmuxBinary;
 }): string;
+/**
+ * Build the exact-session target for tmux *session-scoped* commands such as
+ * `attach-session` and `kill-session`. Native tmux accepts `=NAME` for an
+ * exact session match, but Windows psmux 3.3.x rejects that target form for
+ * session commands even though the bare `NAME` resolves. Keep native tmux on
+ * exact targets and intentionally use the bare session name for psmux.
+ */
+export declare function buildSkcTmuxExactSessionTarget(sessionName: string, opts?: {
+    env?: NodeJS.ProcessEnv;
+    platform?: NodeJS.Platform;
+    binary?: ResolvedTmuxBinary;
+}): string;
 export declare const SKC_TMUX_UNTAGGED_REASON = "skc_tmux_session_untagged";
 export declare function buildSkcTmuxUntaggedSessionHint(tmuxCommand: string): string;
 export declare function buildSkcTmuxUntaggedSessionError(sessionName: string, tmuxCommand: string): string;

package/dist/types/skc-runtime/ultragoal-guard.d.ts

@@ -29,7 +29,13 @@ export declare function readUltragoalVerificationState(input: {
     currentGoal?: CurrentGoalLike | null;
     sessionId?: string | null;
 }): Promise<UltragoalGuardDiagnostic>;
-export declare function isUltragoalAskBlocked(cwd: string): Promise<UltragoalAskBlockDiagnostic>;
+export declare function verifyUltragoalDurableCompletionState(input: {
+    cwd: string;
+    sessionId?: string | null;
+}): Promise<UltragoalGuardDiagnostic>;
+export declare function isUltragoalAskBlocked(cwd: string, options?: {
+    sessionId?: string | null;
+}): Promise<UltragoalAskBlockDiagnostic>;
 /**
  * Escalating per-attempt refusal text. Deliberately avoids every
  * `isUltragoalBypassPrompt` trigger: no `update_goal(`, no "skip/weaken verification",

package/dist/types/skc-runtime/ultragoal-runtime.d.ts

@@ -36,7 +36,6 @@ export interface UltragoalCompletionVerification {
     skcGoalMode: UltragoalSkcGoalMode;
     skcObjective: string;
     qualityGateHash: string;
-    skcGoalSnapshotHash: string;
     planGeneration: string;
     basis: {
         planHashBeforeCheckpoint: string;
@@ -237,7 +236,6 @@ export declare function checkpointUltragoalGoal(input: {
     goalId: string;
     status: UltragoalGoalStatus;
     evidence: string;
-    skcGoalJson?: string;
     qualityGateJson?: string;
 }): Promise<UltragoalPlan>;
 export interface UltragoalCheckpointContinuation {
@@ -253,7 +251,6 @@ export declare function checkpointAndContinueUltragoalGoal(input: {
     goalId: string;
     status: UltragoalGoalStatus;
     evidence: string;
-    skcGoalJson?: string;
     qualityGateJson?: string;
     advanceNext?: boolean;
     retryFailed?: boolean;
@@ -271,7 +268,6 @@ export declare function recordUltragoalReviewBlockers(input: {
     title: string;
     objective: string;
     evidence: string;
-    skcGoalJson?: string;
 }): Promise<UltragoalPlan>;
 export type UltragoalBlockerClassification = "human_blocked" | "resolvable";
 /**

package/dist/types/slash-commands/types.d.ts

@@ -17,6 +17,8 @@ export interface BuiltinSlashCommand {
     subcommands?: SubcommandDef[];
     /** Static inline hint when command takes a simple argument (no subcommands). */
     inlineHint?: string;
+    /** Higher values sort earlier in TUI slash-command autocomplete. */
+    priority?: number;
 }
 /** Parsed slash-command text after stripping the leading "/". */
 export interface ParsedSlashCommand {

package/dist/types/system-prompt.d.ts

@@ -76,6 +76,13 @@ export interface BuildSystemPromptOptions {
     mcpDiscoveryMode?: boolean;
     /** Discoverable MCP server summaries to advertise when discovery mode is active. */
     mcpDiscoveryServerSummaries?: string[];
+    /** Whether built-in tool discovery ("all" mode) is active; enables the tool-discovery prompt block. */
+    toolDiscoveryActive?: boolean;
+    /** Hidden discoverable tools to advertise (name + one-line summary) when tool discovery is active. */
+    discoverableTools?: Array<{
+        name: string;
+        summary: string;
+    }>;
     /** Encourage the agent to delegate via tasks unless changes are trivial. */
     eagerTasks?: boolean;
     /** Rules with alwaysApply=true — their full content is injected into the prompt. */

package/dist/types/tools/skill.d.ts

@@ -33,7 +33,7 @@ export declare class SkillTool implements AgentTool<typeof skillSchema, SkillToo
     readonly name = "skill";
     readonly label = "Skill";
     readonly summary = "Chain into another available skill in the current turn";
-    readonly loadMode = "discoverable";
+    readonly loadMode = "essential";
     readonly description: string;
     readonly parameters: z.ZodObject<{
         name: z.ZodString;

package/dist/types/utils/cmux-workspace.d.ts

@@ -0,0 +1,52 @@
+export interface CmuxWorkspaceRenameCommand {
+    command: string;
+    args: string[];
+}
+/** Current ownership state of a cmux workspace, read back from `cmux workspace list`. */
+export interface CmuxWorkspaceOwnership {
+    /** cmux marks a workspace `has_custom_title` once an explicit title is set (by the user or by SKC). */
+    hasCustomTitle: boolean;
+    /** The workspace's current display title. */
+    title: string;
+}
+export interface CmuxWorkspaceRenameProcess {
+    exited: Promise<number>;
+    kill(): void;
+    unref(): void;
+}
+export interface CmuxWorkspaceTitleSyncOptions {
+    env?: NodeJS.ProcessEnv;
+    isTty?: boolean;
+    which?: (command: string) => string | null;
+    spawn?: (command: string[], options: {
+        env: NodeJS.ProcessEnv;
+        stdin: "ignore";
+        stdout: "ignore";
+        stderr: "ignore";
+    }) => CmuxWorkspaceRenameProcess;
+    /** Reads the current ownership state of `workspaceId`. Injectable for tests. */
+    readOwnership?: (cmuxCommand: string, workspaceId: string, env: NodeJS.ProcessEnv) => Promise<CmuxWorkspaceOwnership | null>;
+}
+export declare function sanitizeCmuxWorkspaceTitle(title: string | undefined): string | undefined;
+export declare function formatCmuxWorkspaceTitle(title: string | undefined): string | undefined;
+export declare function buildCmuxWorkspaceRenameCommand(sessionName: string | undefined, env?: NodeJS.ProcessEnv): CmuxWorkspaceRenameCommand | null;
+/** Parse `cmux workspace list --json --id-format both` output and return the
+ * ownership state of the workspace matching `workspaceId` (by UUID `id` or `ref`). */
+export declare function parseCmuxWorkspaceOwnership(jsonText: string, workspaceId: string): CmuxWorkspaceOwnership | null;
+/** Only rename when SKC owns the name:
+ * - unknown ownership (read failed) → skip (fail safe, never clobber)
+ * - already the desired title → skip (no-op)
+ * - workspace still on its default title → rename
+ * - any custom title (user- or peer-set) → skip
+ * This makes SKC name a fresh workspace once and then leave it alone, so it
+ * never overwrites a user-pinned name and multiple sessions sharing one
+ * CMUX_WORKSPACE_ID do not thrash the workspace title. */
+export declare function shouldRenameCmuxWorkspace(ownership: CmuxWorkspaceOwnership | null, desiredTitle: string): boolean;
+/**
+ * Best-effort sync of the containing cmux workspace title to the current SKC
+ * session name. Ownership-guarded: SKC reads the current workspace title and
+ * only renames a workspace that still has its default title, so it never
+ * overwrites a name the user pinned or a name a peer session (sharing the same
+ * CMUX_WORKSPACE_ID) set. Opt out with SKC_NO_CMUX_RENAME.
+ */
+export declare function syncCmuxWorkspaceTitle(sessionName: string | undefined, options?: CmuxWorkspaceTitleSyncOptions): Promise<void>;

package/dist/types/web/search/index.d.ts

@@ -44,6 +44,8 @@ export type SearchToolParams = z.infer<typeof webSearchSchema>;
 export interface SearchQueryParams extends SearchToolParams {
     provider?: SearchProviderId | "auto";
 }
+/** Override the hedge delay (tests). Non-finite/non-positive resets to default. */
+export declare function setDdgHedgeDelayMs(ms: number | undefined): void;
 /**
  * Execute a web search query for CLI/testing workflows.
  *
@@ -110,7 +112,7 @@ export declare class WebSearchTool implements AgentTool<typeof webSearchSchema,
 /** Web search tool as CustomTool (for TUI rendering support) */
 export declare const webSearchCustomTool: CustomTool<typeof webSearchSchema, SearchRenderDetails>;
 export declare function getSearchTools(): CustomTool<any, any>[];
-export { getSearchProvider, setPreferredSearchProvider, setSearchFallbackProviders } from "./provider";
-export { setSearchHardTimeoutMs } from "./providers/utils";
+export { getConfiguredSearchProviderPreference, getSearchProvider, setPreferredSearchProvider, setSearchFallbackProviders, } from "./provider";
+export { applyConfiguredSearchTimeout, setSearchHardTimeoutMs } from "./providers/utils";
 export type { SearchProviderId as SearchProvider, SearchResponse } from "./types";
 export { isConfigurableSearchProviderId, isSearchProviderPreference } from "./types";

package/dist/types/web/search/provider.d.ts

@@ -6,8 +6,22 @@ export { SearchProvider } from "./providers/base";
 export declare function getSearchProviderLabel(id: SearchProviderId): string;
 export declare function getSearchProvider(id: SearchProviderId): Promise<SearchProvider>;
 export declare const SEARCH_PROVIDER_ORDER: SearchProviderId[];
+/** Drop every cached resolved chain (settings changed, tests). */
+export declare function clearResolvedChainCache(): void;
 export declare function setPreferredSearchProvider(provider: SearchProviderId | "auto"): void;
 export declare function setSearchFallbackProviders(ids: readonly string[]): void;
+/**
+ * Fire-and-forget warm-up: resolve the provider chain once so the provider
+ * modules are imported and the chain cache is primed before the first
+ * user-visible search. Errors are intentionally swallowed — prewarming must
+ * never surface a failure the real search path would handle itself.
+ */
+export declare function prewarmSearchProviders(options: ResolveProviderChainOptions): void;
+export interface SearchProviderPreferenceSource {
+    get(path: "providers.webSearch" | "web_search.provider"): SearchProviderId | "auto";
+    has(path: "providers.webSearch" | "web_search.provider"): boolean;
+}
+export declare function getConfiguredSearchProviderPreference(settings: SearchProviderPreferenceSource): SearchProviderId | "auto";
 export interface ResolveProviderChainOptions {
     authStorage: AuthStorage;
     sessionId?: string;

package/dist/types/web/search/providers/utils.d.ts

@@ -15,27 +15,53 @@ import { SearchProviderError, type SearchProviderId, type SearchSource } from ".
  */
 export declare function findCredential(storage: AgentStorage | null | undefined, envKey: string | null | undefined, ...storageProviders: string[]): string | null;
 /**
- * Default hard ceiling for a single web-search round-trip. 300s tolerates
- * legitimate slow LLM-mediated responses (anthropic web_search_20250305,
- * perplexity, gemini, OpenAI code backend) while still guaranteeing the session unfreezes
- * if Bun's `AbortSignal` fails to propagate on Windows.
- *
- * Pure search APIs (brave, exa, jina, tavily, searxng, synthetic, zai)
- * settle far faster in practice; reusing the same ceiling keeps the wiring
- * uniform without compromising correctness.
+ * Legacy hard ceiling for a single web-search round-trip, retained as the
+ * fallback for call sites that do not declare a timeout class. 300s tolerates
+ * pathological cases while still guaranteeing the session unfreezes if Bun's
+ * `AbortSignal` fails to propagate on Windows.
  */
 export declare const SEARCH_HARD_TIMEOUT_MS = 300000;
+/**
+ * Hard ceiling for pure search APIs (brave, exa, jina, tavily, kagi,
+ * parallel, searxng, synthetic, zai, duckduckgo). These settle in ~1-3s in
+ * practice; a hung TCP/TLS connection should fall through to the next
+ * provider in seconds, not minutes. Kimi is the exception: it declares an
+ * explicit ceiling aligned with its upstream 30s request budget.
+ */
+export declare const SEARCH_API_TIMEOUT_MS = 15000;
+/**
+ * Hard ceiling for LLM-mediated search providers (anthropic, codex, gemini,
+ * perplexity, xai, openai-compatible). These legitimately take tens of
+ * seconds because a model performs the search and synthesizes an answer.
+ */
+export declare const SEARCH_LLM_TIMEOUT_MS = 120000;
+/** Timeout class a provider declares for its outbound round-trips. */
+export type SearchTimeoutClass = "api" | "llm";
 /**
  * Override the hard timeout applied to every web-search round-trip.
  *
  * @param ms - Hard timeout in milliseconds. Non-finite or non-positive
- *   values reset the timeout to {@link SEARCH_HARD_TIMEOUT_MS}.
+ *   values clear the override so per-class defaults apply.
  */
 export declare function setSearchHardTimeoutMs(ms: number | undefined): void;
+/** Structural settings source for {@link applyConfiguredSearchTimeout}. */
+export interface SearchTimeoutSettingSource {
+    get(path: "web_search.timeout"): unknown;
+    has(path: "web_search.timeout"): boolean;
+}
+/**
+ * Install the `web_search.timeout` override from settings — but only when the
+ * user explicitly configured it. The settings schema ships a default value
+ * (300), and consuming that default via `get()` alone would reinstall a
+ * uniform 300s ceiling on every session, silently disabling the per-class
+ * timeout defaults. `has()` distinguishes loaded/overridden settings from
+ * schema defaults; unset clears any previous override so class defaults apply.
+ */
+export declare function applyConfiguredSearchTimeout(settings: SearchTimeoutSettingSource): void;
 /**
  * Compose a caller-supplied {@link AbortSignal} with a hard timeout so an
- * outbound `fetch()` is guaranteed to settle within `ms` even when the
- * runtime fails to propagate cancellation to the underlying transport.
+ * outbound `fetch()` is guaranteed to settle even when the runtime fails to
+ * propagate cancellation to the underlying transport.
  *
  * Bun's WinHTTP backend on Windows is known to ignore `AbortSignal` once a
  * TCP/TLS connection stalls (oven-sh/bun#15275, oven-sh/bun#18536); without
@@ -43,9 +69,12 @@ export declare function setSearchHardTimeoutMs(ms: number | undefined): void;
  * because the user's Esc is never delivered to the native layer.
  *
  * @param signal - Caller cancellation signal, if any.
- * @param ms - Hard timeout in milliseconds. Defaults to the configured value.
+ * @param msOrClass - Explicit timeout in milliseconds, or a
+ *   {@link SearchTimeoutClass} resolved to its class default. Omitted falls
+ *   back to the legacy {@link SEARCH_HARD_TIMEOUT_MS} ceiling. A user-set
+ *   `web_search.timeout` overrides class defaults (but not explicit ms).
  */
-export declare function withHardTimeout(signal: AbortSignal | undefined, ms?: number): AbortSignal;
+export declare function withHardTimeout(signal: AbortSignal | undefined, msOrClass?: number | SearchTimeoutClass): AbortSignal;
 /**
  * Map a provider's raw source list to the unified SearchSource shape,
  * clamped to the requested result count and annotated with ageSeconds.

package/dist/types/web/search/render.d.ts

@@ -10,6 +10,7 @@ import type { SearchResponse } from "./types";
 export interface SearchRenderDetails {
     response: SearchResponse;
     error?: string;
+    warning?: string;
 }
 /** Render web search result with tree-based layout */
 export declare function renderSearchResult(result: {

package/dist/types/workflow/workflow-intent-diff.d.ts

@@ -0,0 +1,19 @@
+export declare const WORKFLOW_INTENT_DIFF_CUSTOM_TYPE = "workflow-intent-diff";
+export type WorkflowIntentRoute = "direct" | "deep-interview" | "ralplan" | "ultragoal" | "team";
+export type DirectTrackingMode = "custom-entry-only" | "not-direct";
+export type RootCausePhaseStatus = "active" | "inactive";
+export interface WorkflowIntentDiff {
+    readonly version: 1;
+    readonly route: WorkflowIntentRoute;
+    readonly reason: string;
+    readonly directTracking: DirectTrackingMode;
+    readonly recommendedSkill?: Exclude<WorkflowIntentRoute, "direct">;
+    readonly recommendedInvocation?: string;
+    readonly triggers: readonly string[];
+    readonly rootCausePhase: {
+        readonly status: RootCausePhaseStatus;
+        readonly triggers: readonly string[];
+    };
+    readonly promptPreview: string;
+}
+export declare function buildWorkflowIntentDiff(text: string): WorkflowIntentDiff | null;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@sayknow-cli/coding-agent",
-	"version": "0.3.5",
+	"version": "0.3.7",
 	"description": "Sayknow-CLI CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/jaybeyond/Sayknow_CLI",
 	"author": "jaybeyond",
@@ -35,7 +35,7 @@
 		"check": "biome check . && bun run check:types",
 		"check:types": "tsgo -p tsconfig.json --noEmit",
 		"lint": "biome lint .",
-		"test": "bun test",
+		"test": "bun test --isolate",
 		"bench:context": "bun run bench/context-optimization.bench.ts",
 		"generate-schemas": "bun ../../scripts/generate-json-schemas.ts",
 		"check:schemas": "bun ../../scripts/generate-json-schemas.ts --check",
@@ -52,12 +52,12 @@
 		"@agentclientprotocol/sdk": "0.21.0",
 		"@babel/parser": "^7.29.3",
 		"@mozilla/readability": "^0.6.0",
-		"@sayknow-cli/stats": "0.3.5",
-		"@sayknow-cli/agent-core": "0.3.5",
-		"@sayknow-cli/ai": "0.3.5",
-		"@sayknow-cli/natives": "0.3.5",
-		"@sayknow-cli/tui": "0.3.5",
-		"@sayknow-cli/utils": "0.3.5",
+		"@sayknow-cli/stats": "0.3.7",
+		"@sayknow-cli/agent-core": "0.3.7",
+		"@sayknow-cli/ai": "0.3.7",
+		"@sayknow-cli/natives": "0.3.7",
+		"@sayknow-cli/tui": "0.3.7",
+		"@sayknow-cli/utils": "0.3.7",
 		"@puppeteer/browsers": "^2.13.0",
 		"@types/turndown": "5.0.6",
 		"@xterm/headless": "^6.0.0",

package/scripts/build-binary.ts

@@ -69,6 +69,9 @@ async function main(): Promise<void> {
 					// Hidden notify daemon CLI (loaded via dynamic import from notify-cli); list it
 					// explicitly so Bun standalone keeps `skc notify daemon-internal` usable.
 					"./src/notifications/telegram-daemon-cli.ts",
+					// Hidden Telegram Remote gateway entrypoint. `skc telegram start` self-spawns
+					// `skc telegram __gateway`, which dynamic-imports this bundled entry.
+					"../telegram-remote/src/index.ts",
 					"--outfile",
 					"dist/skc",
 				],

package/scripts/verify-insane-vendor.ts

@@ -28,11 +28,6 @@ function fail(msg: string): void {
 function walk(dir: string): string[] {
 	const out: string[] = [];
 	for (const entry of readdirSync(dir)) {
-		// node_modules holds locally-installed Phase 3 runtime deps (gitignored,
-		// never vendored and never packed by npm). Vendor checks apply only to
-		// vendored upstream source; third-party package files legitimately carry
-		// paths like ".../references/" that would otherwise false-positive.
-		if (entry === "node_modules") continue;
 		const full = join(dir, entry);
 		if (statSync(full).isDirectory()) out.push(...walk(full));
 		else out.push(full);

package/src/capability/mcp.ts

@@ -15,6 +15,8 @@ export interface MCPServer {
 	name: string;
 	/** Whether this server is enabled (default: true) */
 	enabled?: boolean;
+	/** Whether to connect automatically at session startup (default: true) */
+	autoload?: boolean;
 	/** Connection timeout in milliseconds */
 	timeout?: number;
 	/** Command to run (for stdio transport) */