akm-cli 0.9.0-rc.1 → 0.9.0-rc.13

package/dist/commands/tasks/tasks-cli.js

@@ -2,36 +2,70 @@
 // License, v. 2.0. If a copy of the MPL was not distributed with this
 // file, You can obtain one at https://mozilla.org/MPL/2.0/.
 /**
- * `akm tasks` command family. Extracted verbatim from src/cli.ts (WS6) so the
- * God Module shrinks; the `main.subCommands.tasks` key and every subcommand's
+ * `akm task` command family. Extracted verbatim from src/cli.ts (WS6) so the
+ * God Module shrinks; the `main.subCommands.task` key and every subcommand's
  * args/output shape are byte-identical. Handlers whose body is a plain
  * `runWithJsonErrors(...) + output(...)` are migrated to `defineJsonCommand`,
  * which emits the same JSON envelope (stdout/stderr/exit-code) as the inline
- * form. `tasks run` keeps a plain `defineCommand` because it forwards the
- * task's own exit code via `process.exit`. The private helper
- * `makeTasksToggleCommand` and the `TASKS_SUBCOMMAND_SET` routing constant move
- * with the family.
+ * form. `task run` keeps a plain `defineCommand` because it forwards the
+ * task's own exit code via `process.exitCode` (F4: not `process.exit()` —
+ * that would skip pending cleanup).
+ *
+ * 0.9 CLI overhaul (S6): the group was renamed from the plural `tasks` to the
+ * singular `task` — hard break, no alias. `init`/`enable`/`disable` are
+ * dropped: the default improve-schedule task set now ships as embedded
+ * templates under src/assets/tasks/improve/ (see src/tasks/embedded.ts),
+ * seeded through the interactive `akm setup` task-review step instead of a
+ * separate CLI command, and toggling a task's enabled state is a file edit +
+ * `task sync` (tasks-sync.test.ts already proves the flip path). Every
+ * subcommand (`add`/`run`/`history`/`sync`) shares one `--bundle <bundle>`
+ * axis (S8.4 ratified `task add`'s `--target` → `--bundle`; a later Gate-1
+ * fix reverted it to match `import`/`proposal accept`, splitting the
+ * write-target axis three-vs-one against `remember`/`clone`/`improve` — the
+ * final review re-applied the ratified rename here).
  */
 import { defineCommand } from "citty";
+import { getParsedInvocation } from "../../cli/invocation.js";
 import { parsePositiveIntFlag } from "../../cli/parse-args.js";
-import { defineGroupCommand, defineJsonCommand, output, runWithJsonErrors } from "../../cli/shared.js";
-import { detectServerDefault, registerDefaultTasks } from "./default-tasks.js";
-import { akmTasksAdd, akmTasksDoctor, akmTasksHistory, akmTasksList, akmTasksRemove, akmTasksRun, akmTasksSetEnabled, akmTasksShow, akmTasksSync, parseTaskRef, } from "./tasks.js";
+import { defineGroupCommand, defineJsonCommand, GLOBAL_OUTPUT_ARGS, output, runWithJsonErrors } from "../../cli/shared.js";
+import { UsageError } from "../../core/errors.js";
+import { akmTasksAdd, akmTasksDoctor, akmTasksHistory, akmTasksRun, akmTasksSync } from "./tasks.js";
+/** Shared `--bundle <bundle>` arg wired onto every task subcommand. */
+const bundleArg = {
+    bundle: {
+        type: "string",
+        description: "Bundle to operate on (defaults to the primary/default bundle)",
+    },
+};
+/**
+ * `--target` was renamed to `--bundle` on `task` in 0.9 (S8.4). citty is
+ * non-strict, so the retired spelling is silently absorbed rather than
+ * rejected — reject it explicitly instead (mirrors improve-cli.ts /
+ * remember-cli.ts).
+ */
+function rejectRetiredTaskTargetFlag() {
+    if (!getParsedInvocation().hasFlag("--target"))
+        return;
+    throw new UsageError("`akm task --target` was renamed to `--bundle` in 0.9. Use `--bundle <name>` instead.", "INVALID_FLAG_VALUE");
+}
 const tasksAddCommand = defineJsonCommand({
     meta: { name: "add", description: "Register a new scheduled task and install it in the OS scheduler" },
     args: {
         id: { type: "positional", description: "Task id (used as filename and scheduler entry)", required: true },
         schedule: { type: "string", description: 'Cron-style schedule, e.g. "0 9 * * *" or "@daily"', required: true },
-        workflow: { type: "string", description: "Workflow ref to invoke (e.g. workflow:my-flow)" },
+        ...bundleArg,
+        workflow: { type: "string", description: "Workflow ref to invoke (e.g. workflows/my-flow)" },
         prompt: {
             type: "string",
-            description: "Prompt for the configured agent harness — inline text, an asset ref like agent:foo, or ./path.md",
+            description: "Prompt for the configured agent harness — inline text, an asset ref like agents/foo, or ./path.md",
         },
         command: {
             type: "string",
-            description: 'Shell command to run on the schedule (no AI agent), e.g. "akm improve --auto-accept safe". Split on whitespace; quote the whole flag value.',
+            description: 'Shell command to run on the schedule (no AI agent), e.g. "akm improve --strategy frequent". Split on whitespace; quote the whole flag value.',
         },
-        profile: { type: "string", description: "Agent profile to use for prompt targets (default: defaults.agent)" },
+        engine: { type: "string", description: "Engine to use for prompt targets (default: defaults.engine)" },
+        model: { type: "string", description: "Model override for prompt targets" },
+        "timeout-ms": { type: "string", description: "Positive timeout in milliseconds for prompt or command targets" },
         params: { type: "string", description: "Workflow params as a JSON object" },
         name: { type: "string", description: "Human-readable name for the task" },
         "when-to-use": { type: "string", description: "Guidance on when this task runs or should be used" },
@@ -39,15 +73,24 @@ const tasksAddCommand = defineJsonCommand({
         tags: { type: "string", description: "Comma-separated tags" },
         disabled: { type: "boolean", description: "Register but leave disabled in the OS scheduler", default: false },
         force: { type: "boolean", description: "Overwrite an existing task with the same id", default: false },
+        rebind: {
+            type: "boolean",
+            description: "Explicitly permit scheduler creation from this ineligible local invocation",
+            default: false,
+        },
     },
     async run({ args }) {
+        rejectRetiredTaskTargetFlag();
         const result = await akmTasksAdd({
             id: args.id,
             schedule: args.schedule,
+            target: args.bundle,
             workflow: args.workflow,
             prompt: args.prompt,
             command: args.command,
-            profile: args.profile,
+            engine: args.engine,
+            model: args.model,
+            timeoutMs: args["timeout-ms"] === undefined ? undefined : parsePositiveIntFlag(args["timeout-ms"]),
             params: args.params,
             name: args.name,
             when_to_use: args["when-to-use"],
@@ -60,86 +103,39 @@ const tasksAddCommand = defineJsonCommand({
                 : undefined,
             disabled: args.disabled === true,
             force: args.force === true,
+            rebind: args.rebind === true,
         });
-        output("tasks-add", result);
-    },
-});
-const tasksInitCommand = defineJsonCommand({
-    meta: {
-        name: "init",
-        description: "Idempotently register the default improve task set (skips when CI=true)",
-    },
-    args: {
-        server: {
-            type: "boolean",
-            description: "Treat this as a server install (enables the nightly sweep). Defaults to platform detection.",
-        },
-        laptop: {
-            type: "boolean",
-            description: "Treat this as a laptop install (leaves the nightly sweep disabled).",
-        },
-    },
-    async run({ args }) {
-        const serverInstall = args.server === true ? true : args.laptop === true ? false : detectServerDefault();
-        const result = await registerDefaultTasks({ serverInstall });
-        output("tasks-init", result);
-    },
-});
-const tasksListCommand = defineJsonCommand({
-    meta: { name: "list", description: "List scheduled tasks in the stash" },
-    async run() {
-        const result = await akmTasksList();
-        output("tasks-list", result);
+        output("task-add", result);
     },
 });
-const tasksShowCommand = defineJsonCommand({
-    meta: { name: "show", description: "Show a parsed task definition" },
-    args: { id: { type: "positional", description: "Task id or task:<id>", required: true } },
-    async run({ args }) {
-        const { id } = parseTaskRef(args.id);
-        const result = await akmTasksShow(id);
-        output("tasks-show", result);
-    },
-});
-const tasksRemoveCommand = defineJsonCommand({
-    meta: { name: "remove", description: "Delete a task file and uninstall it from the OS scheduler" },
-    args: { id: { type: "positional", description: "Task id", required: true } },
-    async run({ args }) {
-        const { id } = parseTaskRef(args.id);
-        const result = await akmTasksRemove(id);
-        output("tasks-remove", result);
-    },
-});
-function makeTasksToggleCommand(enabled) {
-    const verb = enabled ? "enable" : "disable";
-    const description = enabled
-        ? "Enable a previously-disabled task"
-        : "Disable a task in the OS scheduler without removing the file";
-    return defineJsonCommand({
-        meta: { name: verb, description },
-        args: { id: { type: "positional", description: "Task id", required: true } },
-        async run({ args }) {
-            const { id } = parseTaskRef(args.id);
-            const result = await akmTasksSetEnabled(id, enabled);
-            output(`tasks-${verb}`, result);
-        },
-    });
-}
-const tasksEnableCommand = makeTasksToggleCommand(true);
-const tasksDisableCommand = makeTasksToggleCommand(false);
 const tasksRunCommand = defineCommand({
     meta: {
         name: "run",
         description: "Execute a task now (this is what cron / launchd / schtasks invoke at the scheduled time)",
     },
-    args: { id: { type: "positional", description: "Task id", required: true } },
+    // Raw defineCommand (it forwards the task's exit code), so the global output
+    // flags are declared here or `--format md nightly` loses the task id.
+    args: {
+        ...GLOBAL_OUTPUT_ARGS,
+        id: { type: "positional", description: "Task id", required: true },
+        ...bundleArg,
+        scheduled: { type: "boolean", description: "Internal marker for scheduler-generated runs", default: false },
+    },
     async run({ args }) {
         await runWithJsonErrors(async () => {
-            const { id } = parseTaskRef(args.id);
-            const envelope = await akmTasksRun(id);
-            output("tasks-run", envelope);
+            rejectRetiredTaskTargetFlag();
+            const envelope = await akmTasksRun(args.id, {
+                scheduled: args.scheduled === true,
+                ...(args.bundle !== undefined ? { target: args.bundle } : {}),
+            });
+            output("task-run", envelope);
+            // F4: was `process.exit(envelope.exitCode)`, terminating synchronously
+            // and skipping any pending cleanup (this command forwards a run task's
+            // own exit code, so it can be any value, not just 0/1). output() has
+            // already run and this is the last statement, so process.exitCode +
+            // the implicit return is equivalent without the synchronous cutoff.
             if (envelope.exitCode !== 0)
-                process.exit(envelope.exitCode);
+                process.exitCode = envelope.exitCode;
         });
     },
 });
@@ -148,21 +144,32 @@ const tasksHistoryCommand = defineJsonCommand({
     args: {
         id: { type: "string", description: "Filter to one task id" },
         limit: { type: "string", description: "Maximum rows to return (default 50)" },
+        ...bundleArg,
     },
     async run({ args }) {
+        rejectRetiredTaskTargetFlag();
         const limit = parsePositiveIntFlag(args.limit ?? undefined);
-        const result = await akmTasksHistory({ id: args.id, limit });
-        output("tasks-history", result);
+        const result = await akmTasksHistory({ id: args.id, limit, target: args.bundle });
+        output("task-history", result);
     },
 });
 const tasksSyncCommand = defineJsonCommand({
     meta: {
         name: "sync",
-        description: "Reconcile the on-disk task files with the OS scheduler",
+        description: "Reconcile the on-disk task files of a bundle with the OS scheduler",
     },
-    async run() {
-        const result = await akmTasksSync();
-        output("tasks-sync", result);
+    args: {
+        ...bundleArg,
+        rebind: {
+            type: "boolean",
+            description: "Replace installed bindings with the current invocation",
+            default: false,
+        },
+    },
+    async run({ args }) {
+        rejectRetiredTaskTargetFlag();
+        const result = await akmTasksSync({}, args.bundle, { rebind: args.rebind === true });
+        output("task-sync", result);
     },
 });
 const tasksDoctorCommand = defineJsonCommand({
@@ -170,32 +177,27 @@ const tasksDoctorCommand = defineJsonCommand({
         name: "doctor",
         description: "Report the active scheduler backend, akm bin path, log dir, and supported schedule subset",
     },
+    args: {},
     async run() {
         const result = await akmTasksDoctor();
-        output("tasks-doctor", result);
+        output("task-doctor", result);
     },
 });
-export const tasksCommand = defineGroupCommand({
+export const taskCommand = defineGroupCommand({
     meta: {
-        name: "tasks",
-        alias: "task",
-        description: "Schedule workflows or prompts via the OS-native scheduler (cron / launchd / schtasks)",
+        name: "task",
+        description: "Schedule recurring commands, prompts, and workflows through the OS scheduler (cron / launchd / schtasks)",
     },
     subCommands: {
         add: tasksAddCommand,
-        init: tasksInitCommand,
-        list: tasksListCommand,
-        show: tasksShowCommand,
-        remove: tasksRemoveCommand,
-        enable: tasksEnableCommand,
-        disable: tasksDisableCommand,
         run: tasksRunCommand,
         history: tasksHistoryCommand,
         sync: tasksSyncCommand,
         doctor: tasksDoctorCommand,
     },
-    async defaultRun() {
-        const result = await akmTasksList();
-        output("tasks-list", result);
-    },
+    // Bare `akm task` reports scheduler diagnostics. Inspection of individual
+    // tasks moved to the generic `akm search` / `akm show <bundle//tasks/id>`.
+    // No `defaultRun`: bare `akm task` is a usage error (exit 2), the canonical
+    // bare-group behavior — owner ruling 12. Run `akm task doctor` for what the
+    // bare form used to run.
 });