@iowarp/clio-coder 0.4.3 → 0.4.5

package/src/interactive/slash-autocomplete.ts

@@ -69,16 +69,14 @@ export interface SlashCompletionItem extends AutocompleteItem {
 export interface SlashAutocompleteOptions {
 	basePath?: string;
 	fdPath?: string | null;
-	/** Slice 11 replaces the named no-op sources with read-only runtime sources. */
+	/** Read-only runtime values for the named grammar slots. */
 	completionSources?: CompletionSources;
 	/** Replace the default workspace-aware `@` source without adding a second composer provider. */
 	fileReferenceSource?: FileReferenceCompletionSource;
-	/** @deprecated Dynamic skill values move to the named `skills` slot in slice 11. */
-	listSkills?: () => unknown;
 }
 
-/** Complete provider surface now, deliberately inert until slice 11 wiring. */
-function stubCompletionSources(): Record<CompletionSlotName, CompletionSource> {
+/** Slots without an in-app source return no dynamic values. */
+function emptyCompletionSources(): Record<CompletionSlotName, CompletionSource> {
 	const sources = {} as Record<CompletionSlotName, CompletionSource>;
 	for (const slot of COMPLETION_SLOT_NAMES) sources[slot] = async () => [];
 	return sources;
@@ -308,7 +306,7 @@ class ClioAutocompleteProvider implements AutocompleteProvider {
 	) {
 		this.files = new CombinedAutocompleteProvider([], basePath, fdPath);
 		this.fileReferences = fileReferenceSource ?? createFileReferenceCompletionSource({ basePath });
-		this.sourceBySlot = { ...stubCompletionSources(), ...sources };
+		this.sourceBySlot = { ...emptyCompletionSources(), ...sources };
 	}
 
 	async getSuggestions(

package/src/interactive/slash-commands.ts

@@ -1,3 +1,4 @@
+import { acceptanceFromTaskFlags } from "../cli/tasks.js";
 import { BusChannels } from "../core/bus-events.js";
 import { THINKING_LEVELS } from "../core/defaults.js";
 import type { SafeEventBus } from "../core/event-bus.js";
@@ -29,7 +30,11 @@ import type {
 	ResourceList,
 } from "../domains/resources/index.js";
 import { parseSkillCommand, SKILL_SURFACE_CLEAR_ARG } from "../domains/resources/index.js";
+import { activeDecisionRefs } from "../domains/session/decision-board.js";
+import type { DecisionLedgerEntry } from "../domains/session/entries.js";
 import type { ShareImportPlan } from "../domains/share/index.js";
+import type { UserTaskAcceptance } from "../domains/user-tasks/acceptance.js";
+import { formatUserTaskHandoff } from "../domains/user-tasks/handoff.js";
 import type { UserTask } from "../domains/user-tasks/store.js";
 import type { ExtensionReloadOutcome } from "../entry/extension-reload.js";
 import { isToolProfileName, TOOL_PROFILE_NAMES, type ToolProfileName } from "../tools/profiles.js";
@@ -146,7 +151,7 @@ type SlashCommandVariant =
 	| { kind: "context-view" }
 	| { kind: "tasks" }
 	| { kind: "decisions" }
-	| { kind: "tasks-add"; text: string }
+	| { kind: "tasks-add"; text: string; expectedOutputs?: string[]; verification?: string[] }
 	| { kind: "tasks-hand"; id: string }
 	| { kind: "tasks-done"; id: string }
 	| { kind: "tasks-drop"; id: string }
@@ -244,6 +249,8 @@ export interface RunCommandOptions {
 
 export interface HandleRunDeps {
 	dispatch: DispatchContract;
+	/** Live session board, snapshotted before operator run admission. */
+	getDecisionBoard?: () => ReadonlyArray<DecisionLedgerEntry>;
 	/** Strict recipe facts the run's execution role is derived from. */
 	getAgentRoleFacts?: AgentRoleFactsResolver;
 	io: RunIo;
@@ -339,6 +346,7 @@ export async function handleRun(
 	deps: HandleRunDeps,
 	options: RunCommandOptions = {},
 ): Promise<void> {
+	const decisionRefs = activeDecisionRefs(deps.getDecisionBoard?.() ?? []);
 	if (options.target && options.workerProfile) {
 		deps.notice(
 			"warn",
@@ -361,6 +369,7 @@ export async function handleRun(
 				...(deps.getAgentRoleFacts ? { resolveFacts: deps.getAgentRoleFacts } : {}),
 			}),
 			requestOrigin: "user",
+			...(decisionRefs.length > 0 ? { decisionRefs } : {}),
 			...(options.workerProfile ? { workerProfile: options.workerProfile } : {}),
 			...(options.workerRuntime ? { workerRuntime: options.workerRuntime } : {}),
 			...(options.target ? { target: options.target } : {}),
@@ -560,6 +569,7 @@ export interface SlashCommandContext {
 	io: RunIo;
 	notice: (level: NoticeLevel, text: string) => void;
 	dispatch: DispatchContract;
+	getDecisionBoard?: HandleRunDeps["getDecisionBoard"];
 	bus: SafeEventBus;
 	/** Strict recipe facts used to keep interactive /run role-equivalent with the CLI. */
 	getAgentRoleFacts?: AgentRoleFactsResolver;
@@ -690,7 +700,7 @@ export interface SlashCommandContext {
 	openDecisions: () => void;
 	/** Project-scoped operator task inbox backing `/tasks` mutations. */
 	userTasks?: {
-		add(title: string): UserTask;
+		add(title: string, acceptance?: UserTaskAcceptance): UserTask;
 		hand(id: string): UserTask;
 		done(id: string): UserTask;
 		drop(id: string): UserTask;
@@ -789,12 +799,6 @@ function formatReceiptVerificationBlock(runId: string, result: SlashReceiptVerif
 	return lines.join("\n");
 }
 
-/** The one operator-authored turn used by slash and overlay handoff paths. */
-export function formatUserTaskHandoff(task: Pick<UserTask, "id" | "title" | "note">): string {
-	const note = task.note ? ` ${task.note}.` : "";
-	return `Operator task ${task.id}: ${task.title}.${note} Pick it up with tasks action="pick" id="${task.id}" and work it when appropriate.`;
-}
-
 /** The verb a command performs, in the order /help lists the groups. */
 export const SLASH_COMMAND_GROUPS = ["Work", "Inspect", "Configure", "Session"] as const;
 export type SlashCommandGroup = (typeof SLASH_COMMAND_GROUPS)[number];
@@ -1180,6 +1184,7 @@ export const BUILTIN_SLASH_COMMANDS: ReadonlyArray<BuiltinSlashCommand> = [
 					task,
 					{
 						dispatch: ctx.dispatch,
+						...(ctx.getDecisionBoard ? { getDecisionBoard: ctx.getDecisionBoard } : {}),
 						...(ctx.getAgentRoleFacts ? { getAgentRoleFacts: ctx.getAgentRoleFacts } : {}),
 						io: ctx.io,
 						notice: ctx.notice,
@@ -1519,7 +1524,14 @@ export const BUILTIN_SLASH_COMMANDS: ReadonlyArray<BuiltinSlashCommand> = [
 		},
 		args: {
 			subcommands: {
-				add: { positionals: [{ name: "text", required: true, rest: true }] },
+				add: {
+					parseFlagsInRest: true,
+					flags: [
+						{ name: "--expect", takesValue: true, repeatable: true, valueName: "path" },
+						{ name: "--verify", takesValue: true, repeatable: true, valueName: "checkId[:timeoutMs]" },
+					],
+					positionals: [{ name: "text", required: true, rest: true }],
+				},
 				hand: { positionals: [{ name: "id", required: true }] },
 				done: { positionals: [{ name: "id", required: true }] },
 				drop: { positionals: [{ name: "id", required: true }] },
@@ -1530,7 +1542,12 @@ export const BUILTIN_SLASH_COMMANDS: ReadonlyArray<BuiltinSlashCommand> = [
 			switch (parsed.subcommand) {
 				case "add":
 					return parsed.rest
-						? { kind: "tasks-add", text: parsed.rest }
+						? {
+								kind: "tasks-add",
+								text: parsed.rest,
+								...(parsed.flagValues.has("--expect") ? { expectedOutputs: parsed.flagValues.get("--expect") ?? [] } : {}),
+								...(parsed.flagValues.has("--verify") ? { verification: parsed.flagValues.get("--verify") ?? [] } : {}),
+							}
 						: { kind: "usage-error", command: "tasks", reason: "add requires task text" };
 				case "hand":
 				case "done":
@@ -1558,7 +1575,10 @@ export const BUILTIN_SLASH_COMMANDS: ReadonlyArray<BuiltinSlashCommand> = [
 			}
 			try {
 				if (command.kind === "tasks-add") {
-					const task = ctx.userTasks.add(command.text);
+					const task = ctx.userTasks.add(
+						command.text,
+						acceptanceFromTaskFlags(process.cwd(), command.expectedOutputs ?? [], command.verification ?? []),
+					);
 					ctx.notice("success", `logged operator task ${task.id}: ${task.title}`);
 					return;
 				}
@@ -2142,32 +2162,6 @@ const COMMAND_SHAPED_TOKEN = /^[A-Za-z][A-Za-z0-9:-]*$/u;
  */
 const COMMAND_ESCAPE = "\\/";
 
-const RETIRED_COMMAND_HINTS: Readonly<Record<string, string>> = Object.freeze({
-	status: "Use the footer or Alt+U for status; /cost, /context, and /fleet inspect their own domains.",
-	receipts: "Use /view or /view verify <run-id>.",
-	"context-init": "Use /context init.",
-	"context-clear": "Use /context reset for project context or /new for a fresh session.",
-	"context-view": "Use /context.",
-	compact: "Use /context compact [instructions].",
-	skills: "Use /skill or /resources skills.",
-	exit: "Use /quit.",
-	config: "Use /settings.",
-	ctx: "Use /context.",
-	models: "Use /model.",
-	clear: "Use /new for a fresh session or /context reset for project context.",
-	library: "Use /resources library [kind].",
-	prompts: "Use /resources prompts.",
-	extensions: "Use /resources extensions.",
-	interop: "Use /agents connect.",
-	targets: "Use /settings targets.",
-	"scoped-models": "Use /settings chat model-picker.",
-});
-
-function retiredCommandHint(token: string): string | undefined {
-	if (/^skills?:[^\s]+$/u.test(token)) return "Use /skill <name> [task].";
-	return RETIRED_COMMAND_HINTS[token];
-}
-
 /** Pure slash-command parser: no I/O, no side effects. Walks the registry in order. */
 /**
  * One notice line for a reload outcome. Success when the generation committed
@@ -2230,12 +2224,6 @@ export function dispatchSlashCommand(command: SlashCommand, ctx: SlashCommandCon
 		return "accepted";
 	}
 	if (command.kind === "unknown-command") {
-		const retiredHint = retiredCommandHint(command.token);
-		if (retiredHint) {
-			ctx.notice("error", `/${command.token} is retired and did not run. ${retiredHint}`);
-			ctx.render();
-			return "rejected";
-		}
 		// The registry does not own the token, but a prompt template might. The
 		// parser stays pure, so the lookup happens here, and only for a spelling
 		// that matched no command: `/name` is how every other agent invokes the

package/src/interactive/slash-spec.ts

@@ -72,6 +72,8 @@ export interface CommandArgsSpec {
 	 * keep their byte-for-byte parsing.
 	 */
 	parseFlagsBeforeRest?: boolean;
+	/** Recognize flags throughout a rest positional (operator task acceptance). */
+	parseFlagsInRest?: boolean;
 }
 
 export interface ParsedArgs {
@@ -260,6 +262,32 @@ export function parseArgs(spec: CommandArgsSpec, argsLine: string): ParsedArgs {
 	while (index < argsLine.length) {
 		const currentPositionalSpec = positionalSpecs[positionalIndex];
 		if (currentPositionalSpec?.rest) {
+			if (currentSpec.parseFlagsInRest) {
+				const text: string[] = [];
+				while (index < argsLine.length) {
+					const start = index;
+					const read = readToken();
+					if (!read) break;
+					if (read.token === "--") {
+						text.push(argsLine.slice(index).trim());
+						index = argsLine.length;
+						break;
+					}
+					const flag = flagSpecs.find((item) => item.name === read.token);
+					if (flag) {
+						const error = consumeFlag(flag, read.token);
+						if (error) return errorResult(error);
+					} else if (read.token.startsWith("--")) return errorResult(`Unknown flag: ${read.token}`);
+					else text.push(argsLine.slice(start, index));
+					skipWhitespace();
+				}
+				const restVal = text.join(" ").trim();
+				if (!restVal) return errorResult(`Missing required argument: ${currentPositionalSpec.name}`);
+				positionals.push(restVal);
+				rest = restVal;
+				positionalIndex++;
+				break;
+			}
 			if (currentSpec.parseFlagsBeforeRest) {
 				while (index < argsLine.length) {
 					const tokenStart = index;