@skrr-ai/cli 0.1.21 → 0.1.23

package/dist/commands/tasks/complete.d.ts

@@ -1,144 +1,10 @@
-import { dataService } from '@skrr-ai/data-provider';
-import type { TaskReport } from '@skrr-ai/data-provider';
 import { BaseCommand } from '../../base-command';
-import { type TaskDeliveryReceipt } from '../../lib/git-task';
-type CompletionStatus = 'completed' | 'failed';
-type TaskCommentResult = Awaited<ReturnType<typeof dataService.addTaskComment>>;
-type MovedTaskResult = Awaited<ReturnType<typeof dataService.moveTask>>;
-/** Wire shape of POST /api/tasks/:taskId/complete (OSK-1969). */
-/** One thing a completion did not have. Never a refusal — `done` has none. */
-export interface CompletionAdvisory {
-    code: string;
-    message: string;
-}
-export interface CompleteEndpointResponse {
-    task: MovedTaskResult | null;
-    comment: TaskCommentResult | null;
-    commentFailed?: boolean;
-    /** What the completion was missing. `done` has no preconditions, so every
-     *  condition that used to refuse one now travels with it instead. The task
-     *  reached the requested terminal state regardless. */
-    advisories?: CompletionAdvisory[];
-}
-export interface CompleteTransport {
-    /** POST /api/tasks/:taskId/complete — atomic move + TASK_OUTPUT comment. */
-    postComplete: (taskId: string, payload: {
-        status: CompletionStatus;
-        summary: string;
-        deliverables: string;
-        verdict: string;
-        report?: TaskReport;
-        deliveryReceipt?: TaskDeliveryReceipt;
-    }) => Promise<CompleteEndpointResponse>;
-    /** Legacy step 1 — POST /api/tasks/:taskId/comments. */
-    addComment: (taskId: string, body: string) => Promise<TaskCommentResult>;
-    /** Legacy step 2 — PATCH /api/tasks/:taskId/move. */
-    moveTask: (taskId: string, status: 'done' | 'failed') => Promise<MovedTaskResult>;
-    /** Session credentials must never fall back to broad user-grain endpoints. */
-    allowLegacyFallback?: boolean;
-}
-export interface DurableCompletionResult {
-    comment: TaskCommentResult | null;
-    movedTask: MovedTaskResult | null;
-    moved: boolean;
-    commentFailed: boolean;
-    advisories?: CompletionAdvisory[];
-    /** true when the atomic /complete endpoint handled the finalization. */
-    usedAtomicEndpoint: boolean;
-}
-/**
- * Mutation endpoints require the task's canonical UUID even though the CLI
- * accepts human-readable refs and titles. Resolve the display reference before
- * writing anything so a 404 from `POST /complete` cannot be mistaken for an
- * old server and fall through to the legacy comment-first path.
- */
-export declare function resolveCanonicalCompletionTaskId(taskRef: string, lookup: (taskRef: string) => Promise<unknown>): Promise<string>;
-/**
- * The canonical id AND the status it is in.
- *
- * The status comes free: this lookup already happens on the user path, so the
- * re-completion guard below costs no extra round trip on the common case. It is
- * also the only way to make the success line honest — `tasks complete` printed
- * "Moved task X to done" unconditionally, so a task that was ALREADY done got a
- * report of a state transition that did not happen.
- */
-export declare function resolveCompletionTarget(taskRef: string, lookup: (taskRef: string) => Promise<unknown>): Promise<{
-    id: string;
-    status: string | null;
-}>;
-/**
- * What a second `tasks complete` on an already-closed task should do.
- *
- * The repo contract calls this "the only terminal closeout", and a task has one
- * canonical outcome — but a second run used to succeed silently, post another
- * TASK_OUTPUT block, and leave `tasks output` rendering the throwaway one as
- * authoritative. The real summary and its deliverables link were still on the task
- * but no longer canonical, which is the worst of the available failures: no
- * error, no warning, exit 0, and a record that now says something different.
- *
- * Refusing outright would break the case that makes retries safe. A completion
- * whose response is lost — a flaky connection, a re-run script — SHOULD be able
- * to run again, and today it collapses server-side on the shared
- * `X-Idempotency-Key`. Turning that into a hard failure would force every
- * caller to write its own is-it-already-done check.
- *
- * So the axis is not "again or not", it is "the SAME again, or something
- * different":
- *
- *   - identical report  → `noop`, exit 0. Nothing to do, and saying so is more
- *     honest than re-posting an identical block.
- *   - differing report  → `refuse`. Superseding a canonical outcome is a real
- *     decision, so it takes `--amend` and says so.
- *   - not yet terminal  → `proceed`, unchanged.
- *
- * Pure, so the rule is testable without a server.
- */
-export declare function decideRecompletion(opts: {
-    priorStatus: string | null;
-    canonicalBody: string | null;
-    incomingBody: string;
-    amend: boolean;
-}): 'proceed' | 'noop' | 'refuse';
-/** A 404 from the /complete endpoint means an old server without the route. */
-export declare function isCompleteEndpointMissing(err: unknown): boolean;
-/**
- * The server finalized a different task than the one we were asked to finalize.
- *
- * The session-scoped completion endpoint derives its task from the run claim
- * and deliberately ignores any client-supplied id — that is what stops a
- * compromised agent process finishing a sibling task. The corollary is that the
- * id resolved on this side is not necessarily the one that moved, so reporting
- * ours would print a success line (and a `--json` `taskId`) for a task that is
- * still open.
- */
-export declare class TaskCompletionMismatchError extends Error {
-    readonly requestedTaskId: string;
-    readonly completedTaskId: string;
-    constructor(requestedTaskId: string, completedTaskId: string);
-}
-/**
- * Finalize a task durably (OSK-1969).
- *
- * Prefers the server's atomic POST /:taskId/complete (CAS move first, then
- * the TASK_OUTPUT comment — no half-finished state if this process dies).
- * Falls back to the legacy client-side two-step (comment, then move) when the
- * server predates the endpoint (HTTP 404). `noMove` keeps the historical
- * comment-only behavior and never touches the endpoint. All transport errors
- * other than the endpoint-missing 404 are rethrown for the caller's uniform
- * error handling.
- */
-export declare function completeTaskDurably(opts: {
+export declare function describePendingReview({ statusType, taskId, resultId, bin, }: {
+    statusType: unknown;
     taskId: string;
-    status: CompletionStatus;
-    body: string;
-    summary: string;
-    deliverables: string;
-    verdict: string;
-    report?: TaskReport;
-    deliveryReceipt?: TaskDeliveryReceipt;
-    noMove: boolean;
-    transport: CompleteTransport;
-}): Promise<DurableCompletionResult>;
+    resultId?: string;
+    bin: string;
+}): string | null;
 export default class TasksComplete extends BaseCommand {
     static description: string;
     static examples: string[];
@@ -146,21 +12,23 @@ export default class TasksComplete extends BaseCommand {
         id: import("@oclif/core/lib/interfaces").Arg<string | undefined, Record<string, unknown>>;
     };
     static flags: {
-        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         task: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        amend: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
-        status: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        status: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         summary: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        deliverables: import("@oclif/core/lib/interfaces").OptionFlag<string[], import("@oclif/core/lib/interfaces").CustomOptions>;
+        body: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'conclusion-type': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'conclusion-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'structured-schema': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'structured-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         deliverable: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        verdict: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
-        outcome: import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        finding: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
         evidence: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        next: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
-        'no-move': import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        remaining: import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'source-run': import("@oclif/core/lib/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'run-id': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'spec-version': import("@oclif/core/lib/interfaces").OptionFlag<number | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'idempotency-key': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        'from-json': import("@oclif/core/lib/interfaces").OptionFlag<string | undefined, import("@oclif/core/lib/interfaces").CustomOptions>;
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
 }
-export {};