@skrr-ai/cli 0.1.22 → 0.1.24

package/dist/commands/spaces/work-sync/telemetry.js

@@ -0,0 +1,96 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const base_command_1 = require("../../../base-command");
+const work_sync_1 = require("../../../lib/work-sync");
+/**
+ * `skrr spaces work-sync telemetry <space>` — Work Sync's own exit criteria.
+ *
+ * Work Sync v1 is ON at 100% (owner decision, 2026-09-08) with two criteria
+ * knowingly overridden rather than satisfied. The server has computed the verdict
+ * the whole time, and nothing outside the API could read it — so checking the
+ * criteria meant querying the production database by hand, which is why nobody
+ * did. An exit criterion that can only be read by hand is one that does not get
+ * read.
+ *
+ * The verdict is printed as the server decided it. This command does not
+ * re-derive it: the same rule that keeps `tasks expectations` reading the server's
+ * judgement rather than computing its own applies here, and for the same reason —
+ * a CLI that reached a different conclusion from the service would leave the
+ * reader unable to tell which was lying.
+ */
+class SpacesWorkSyncTelemetry extends base_command_1.BaseCommand {
+    static description = "Show Work Sync's convergence telemetry for a Space and the server's rollout verdict";
+    static examples = [
+        '<%= config.bin %> spaces work-sync telemetry <space-id>',
+        '<%= config.bin %> spaces work-sync telemetry <space-id> --json',
+    ];
+    static args = {
+        space: core_1.Args.string({ description: 'Space ID', required: true, ignoreStdin: true }),
+    };
+    static flags = {
+        json: core_1.Flags.boolean({ description: 'Output as JSON' }),
+        limit: core_1.Flags.integer({
+            description: 'Most recent telemetry rows to summarize (1-10000, default 5000)',
+        }),
+    };
+    async run() {
+        this.requireAuth();
+        const { args, flags } = await this.parse(SpacesWorkSyncTelemetry);
+        let summary;
+        try {
+            summary = await (0, work_sync_1.fetchWorkSyncTelemetrySummary)(args.space, { limit: flags.limit });
+        }
+        catch (err) {
+            this.handleApiError(err);
+            return;
+        }
+        if (flags.json) {
+            this.log(JSON.stringify(summary, null, 2));
+            return;
+        }
+        this.log(`Verdict:       ${summary.rollout.verdict}`);
+        this.log(`Sampled rows:  ${summary.sampledEvents}`);
+        this.log(`Repair backlog: ${summary.repairBacklog}`);
+        // Zero sampled rows is a REAL answer and the most misread one: it means
+        // nothing has been measured in this Space, not that everything is fine. The
+        // server already distinguishes it (`insufficient_evidence`); say so here too
+        // rather than printing an empty table under a clean-looking verdict.
+        if (summary.sampledEvents === 0) {
+            this.log('');
+            this.log('No telemetry has been recorded for this Space. That is not evidence of health — ' +
+                'it means no board in this Space has reported, so nothing has been measured.');
+            return;
+        }
+        const rows = Object.entries(summary.events).sort(([a], [b]) => a.localeCompare(b));
+        if (rows.length > 0) {
+            this.log('');
+            this.log('Events:');
+            for (const [event, stats] of rows) {
+                const parts = [`count ${stats.count}`];
+                if (stats.mismatchCount > 0)
+                    parts.push(`mismatches ${stats.mismatchCount}`);
+                if (stats.p50DurationMs !== null) {
+                    parts.push(`p50 ${stats.p50DurationMs}ms`, `p95 ${stats.p95DurationMs}ms`);
+                }
+                if (stats.p50Bytes !== null) {
+                    parts.push(`p50 ${stats.p50Bytes}B`, `p95 ${stats.p95Bytes}B`);
+                }
+                this.log(`  ${event.padEnd(20)} ${parts.join(' · ')}`);
+            }
+        }
+        if (summary.rollout.reasons.length > 0) {
+            this.log('');
+            this.log('Holding because:');
+            for (const reason of summary.rollout.reasons) {
+                // An unexplained code is the thing this command exists to avoid: the
+                // reader is being told a rollout is held and needs to know by what.
+                this.log(`  ${reason}`);
+                const explanation = work_sync_1.WORK_SYNC_HOLD_REASONS[reason];
+                if (explanation)
+                    this.log(`    ${explanation}`);
+            }
+        }
+    }
+}
+exports.default = SpacesWorkSyncTelemetry;

package/dist/commands/store/install.js

@@ -11,7 +11,11 @@ class StoreInstall extends base_command_1.BaseCommand {
         '<%= config.bin %> store install agent_abc123 --json',
     ];
     static args = {
-        agentId: core_1.Args.string({ description: 'The store agent to install', required: true }),
+        agentId: core_1.Args.string({
+            description: 'The store agent to install',
+            required: true,
+            ignoreStdin: true,
+        }),
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),

package/dist/commands/store/releases.js

@@ -18,7 +18,7 @@ class StoreReleases extends base_command_1.BaseCommand {
         '<%= config.bin %> store releases agent_abc123 --json',
     ];
     static args = {
-        agentId: core_1.Args.string({ description: 'The store agent', required: true }),
+        agentId: core_1.Args.string({ description: 'The store agent', required: true, ignoreStdin: true }),
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),

package/dist/commands/store/update.js

@@ -11,7 +11,11 @@ class StoreUpdate extends base_command_1.BaseCommand {
         '<%= config.bin %> store update agent_abc123 --strategy replace',
     ];
     static args = {
-        agentId: core_1.Args.string({ description: 'The store agent you installed', required: true }),
+        agentId: core_1.Args.string({
+            description: 'The store agent you installed',
+            required: true,
+            ignoreStdin: true,
+        }),
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),

package/dist/commands/subscriptions/cancel.js

@@ -11,7 +11,11 @@ class SubscriptionsCancel extends base_command_1.BaseCommand {
         '<%= config.bin %> subscriptions cancel agent_abc123 --expected-version 3',
     ];
     static args = {
-        agentId: core_1.Args.string({ description: 'The service to unsubscribe from', required: true }),
+        agentId: core_1.Args.string({
+            description: 'The service to unsubscribe from',
+            required: true,
+            ignoreStdin: true,
+        }),
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),

package/dist/commands/subscriptions/health.js

@@ -22,7 +22,11 @@ class SubscriptionsHealth extends base_command_1.BaseCommand {
         '<%= config.bin %> subscriptions health agent_abc123 --json',
     ];
     static args = {
-        agentId: core_1.Args.string({ description: 'The service to check', required: true }),
+        agentId: core_1.Args.string({
+            description: 'The service to check',
+            required: true,
+            ignoreStdin: true,
+        }),
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),

package/dist/commands/subscriptions/status.js

@@ -10,7 +10,11 @@ class SubscriptionsStatus extends base_command_1.BaseCommand {
         '<%= config.bin %> subscriptions status agent_abc123 --json',
     ];
     static args = {
-        agentId: core_1.Args.string({ description: 'The service to check', required: true }),
+        agentId: core_1.Args.string({
+            description: 'The service to check',
+            required: true,
+            ignoreStdin: true,
+        }),
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),

package/dist/commands/subscriptions/subscribe.js

@@ -11,7 +11,11 @@ class SubscriptionsSubscribe extends base_command_1.BaseCommand {
         '<%= config.bin %> subscriptions subscribe agent_abc123 --space space_xyz --json',
     ];
     static args = {
-        agentId: core_1.Args.string({ description: 'The published service to subscribe to', required: true }),
+        agentId: core_1.Args.string({
+            description: 'The published service to subscribe to',
+            required: true,
+            ignoreStdin: true,
+        }),
     };
     static flags = {
         json: core_1.Flags.boolean({ description: 'Output as JSON' }),

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

@@ -1,8 +1,7 @@
 import { BaseCommand } from '../../base-command';
 /**
- * One row of `GET /api/tasks/:taskId/activity`, which returns the task's raw
- * persisted `ITaskActivityLog` entries — `{ action, timestamp, userId, details }`
- * (packages/data-schemas/src/types/task.ts:162, api/models/Task.js:2239).
+ * One row of the canonical Activity endpoint. Field fallbacks remain so this
+ * CLI can still read an older deployment's embedded Task activity entries.
  *
  * OSK-2044: this renderer read `event` / `actorId` / `createdAt`, none of which
  * that document has, so EVENT and ACTOR were `-` on every row while `details`

package/dist/commands/tasks/activity.js

@@ -6,12 +6,23 @@ const core_1 = require("@oclif/core");
 const data_provider_1 = require("@skrr-ai/data-provider");
 const base_command_1 = require("../../base-command");
 const format_1 = require("../../lib/format");
+const task_legacy_1 = require("../../lib/task-legacy");
 function activityRow(entry) {
+    const actor = entry.actor;
+    const actorText = actor
+        ? actor.kind === 'agent'
+            ? `agent:${actor.agentId}`
+            : actor.kind === 'human'
+                ? `human:${actor.userId}`
+                : actor.kind === 'integration'
+                    ? `integration:${actor.provider}`
+                    : `system:${actor.service}`
+        : (entry.userId ?? entry.actorId);
     return {
-        time: formatTime(entry.timestamp ?? entry.createdAt),
+        time: formatTime(entry.occurredAt ?? entry.timestamp ?? entry.createdAt),
         event: (0, format_1.formatCell)(entry.action ?? entry.event ?? entry.type),
-        actor: (0, format_1.formatCell)(entry.userId ?? entry.actorId ?? entry.actor),
-        details: (0, format_1.formatCell)(entry.details ?? entry.metadata),
+        actor: (0, format_1.formatCell)(actorText),
+        details: (0, format_1.formatCell)(entry.data ?? entry.details ?? entry.metadata),
     };
 }
 /**
@@ -70,7 +81,7 @@ function formatTime(value) {
     return date.toISOString().slice(0, 19).replace('T', ' ');
 }
 class TasksActivity extends base_command_1.BaseCommand {
-    static description = 'Show activity log for a task';
+    static description = 'Show the immutable, multi-actor Activity ledger for a Task';
     static examples = [
         '<%= config.bin %> tasks activity <task-id>',
         '<%= config.bin %> tasks activity <task-id> --limit 50',
@@ -88,7 +99,7 @@ class TasksActivity extends base_command_1.BaseCommand {
         const { args, flags } = await this.parse(TasksActivity);
         let response;
         try {
-            response = await data_provider_1.dataService.getTaskActivity(args.id, flags.limit);
+            response = await data_provider_1.dataService.listTaskActivityEvents(args.id, { limit: flags.limit });
         }
         catch (err) {
             this.handleApiError(err);
@@ -100,7 +111,10 @@ class TasksActivity extends base_command_1.BaseCommand {
         const r = response;
         const items = r?.data ?? [];
         if (items.length === 0) {
-            this.log('No activity.');
+            // "No activity." was a claim about the TASK. It is only ever a claim about
+            // this VIEW: a Task predating the canonical model has its history in the
+            // legacy activity log, which this endpoint does not read.
+            this.log(await (0, task_legacy_1.emptyCanonicalView)(args.id, 'Activity'));
             return;
         }
         (0, format_1.renderTable)(items.map(activityRow), [

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

@@ -1,144 +1,14 @@
-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, advisories, }: {
+    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;
+    advisories?: Array<{
+        code: string;
+        message: string;
+    }>;
+}): string | null;
 export default class TasksComplete extends BaseCommand {
     static description: string;
     static examples: string[];
@@ -146,21 +16,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 {};