rivet-design 0.26.2 → 0.27.1

package/dist/agent-variants/DirectionRefReservations.d.ts

@@ -0,0 +1,11 @@
+import type { VariantHistoryService } from '../services/VariantHistoryService';
+import type { DirectionRef } from '../types/direction-ref';
+export declare class DirectionRefReservations {
+    private readonly taken;
+    reserve(label: string): DirectionRef;
+    load(args: {
+        projectPath: string;
+        history: VariantHistoryService;
+    }): Promise<void>;
+}
+//# sourceMappingURL=DirectionRefReservations.d.ts.map

package/dist/agent-variants/DirectionRefReservations.js

@@ -0,0 +1,47 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.DirectionRefReservations = void 0;
+const directionRefs_1 = require("../services/directionRefs");
+const projectDirections_1 = require("../services/projectDirections");
+const logger_1 = require("../utils/logger");
+const log = (0, logger_1.createLogger)('DirectionRefReservations');
+class DirectionRefReservations {
+    taken = new Set();
+    reserve(label) {
+        const directionRef = (0, directionRefs_1.nextDirectionRef)({ label, taken: this.taken });
+        this.taken.add(directionRef);
+        return directionRef;
+    }
+    async load(args) {
+        let directions;
+        try {
+            directions = await (0, projectDirections_1.listProjectDirections)({
+                kind: 'history',
+                history: args.history,
+                projectPath: args.projectPath,
+            });
+        }
+        catch (error) {
+            log.warn('Direction ref history could not be listed', error);
+            return;
+        }
+        for (const direction of directions) {
+            this.taken.add(direction.directionRef);
+            if (direction.manifest.directionRef)
+                continue;
+            try {
+                await args.history.assignDirectionRef({
+                    projectPath: direction.projectPath,
+                    sessionId: direction.manifest.sessionId,
+                    variantId: direction.manifest.variantId,
+                    directionRef: direction.directionRef,
+                });
+            }
+            catch (error) {
+                log.warn(`Direction ref backfill failed for ${direction.manifest.sessionId}:${direction.manifest.variantId}`, error);
+            }
+        }
+    }
+}
+exports.DirectionRefReservations = DirectionRefReservations;
+//# sourceMappingURL=DirectionRefReservations.js.map

package/dist/agent-variants/PreviewRuntime.js

@@ -14,12 +14,14 @@ const logger_1 = require("../utils/logger");
 const variantWorkspaceFiles_1 = require("./variantWorkspaceFiles");
 const variantsTelemetry_1 = require("./variantsTelemetry");
 const errorMessage_1 = require("../utils/errorMessage");
+const failureReason_1 = require("../utils/failureReason");
 const pathContainment_1 = require("../utils/pathContainment");
 const log = (0, logger_1.createLogger)('PreviewRuntime');
 const FRESH_DEV_SERVER_HOST = '127.0.0.1';
 const FRESH_DEV_SERVER_DEFAULT_PORT = 5173;
 const DEV_SERVER_START_MAX_ATTEMPTS = 3;
 const DEV_SERVER_MAX_AUTO_RESTARTS = 2;
+const REPORTED_FAILURE_REASONS = new Set(['dependency_install_failed', 'spawn_error']);
 class PreviewRuntime {
     deps;
     sessions = new Map();
@@ -180,9 +182,11 @@ class PreviewRuntime {
             }).catch((error) => {
                 throw new devServerError_1.DevServerStartError({
                     message: (0, errorMessage_1.errorMessage)(error),
+                    summary: (0, errorMessage_1.errorMessage)(error),
                     detail: '',
                     reason: 'dependency_install_failed',
                     portInUse: false,
+                    reasonCode: (0, failureReason_1.failureReasonCode)(error),
                 });
             })
             : null;
@@ -305,13 +309,13 @@ class PreviewRuntime {
                                 reason: 'dependency_install_failed',
                                 portInUse: false,
                                 detail: (0, errorMessage_1.errorMessage)(installError),
-                            }, startedAt, (0, devServerError_1.isDevServerStartError)(installError) ? installError : undefined, missingDepsOutput);
+                            }, startedAt, installError, missingDepsOutput);
                             return undefined;
                         }
                     }
                     continue;
                 }
-                this.failVariant(args, { reason, portInUse }, startedAt, devError, missingDepsOutput);
+                this.failVariant(args, { reason, portInUse }, startedAt, error, missingDepsOutput);
                 return undefined;
             }
             finally {
@@ -356,9 +360,13 @@ class PreviewRuntime {
         });
     }
     failVariant(args, failure, startedAt, error, missingDeps) {
+        const devError = (0, devServerError_1.isDevServerStartError)(error) ? error : undefined;
+        const detail = devError?.detail ||
+            failure.detail ||
+            (error === undefined ? '' : (0, errorMessage_1.errorMessage)(error));
         this.requireSession(args.sessionId).failures.set(args.workItemId, {
             ...failure,
-            ...(error?.detail ? { detail: error.detail } : {}),
+            ...(detail ? { detail } : {}),
         });
         let errorCode = 'DEV_SERVER_START_FAILED';
         if (failure.reason === 'dependency_install_failed') {
@@ -366,15 +374,19 @@ class PreviewRuntime {
         }
         if (failure.reason === 'missing_dev_script')
             errorCode = 'DEV_SCRIPT_MISSING';
+        const reportedError = REPORTED_FAILURE_REASONS.has(failure.reason)
+            ? error
+            : undefined;
         (0, variantsTelemetry_1.trackFreshDevServerFailed)(this.deps.telemetry, {
             sessionId: args.sessionId,
             variantId: args.workItemId,
             errorCode,
             reason: failure.reason,
             portInUse: failure.portInUse,
-            exitCode: error?.exitCode,
-            signal: error?.signal,
+            exitCode: devError?.exitCode,
+            signal: devError?.signal,
             missingDeps,
+            error: reportedError,
         });
         (0, variantsTelemetry_1.trackPreviewFailed)(this.deps.telemetry, {
             sessionId: args.sessionId,
@@ -385,9 +397,10 @@ class PreviewRuntime {
             reason: failure.reason,
             portInUse: failure.portInUse,
             durationMs: Date.now() - startedAt,
-            exitCode: error?.exitCode,
-            signal: error?.signal,
+            exitCode: devError?.exitCode,
+            signal: devError?.signal,
             missingDeps,
+            error: reportedError,
         });
         this.deps.changed();
     }

package/dist/agent-variants/SessionStore.d.ts

@@ -2,6 +2,8 @@ import type { GuidanceDocument } from '../types/guidance';
 import type { WorkerSelection } from '../types/change-request-types';
 import { type VariantReference } from '../types/variant-reference';
 import { type Brief, type ActiveVariantSnapshot, type LeasedWorkItemDescriptor, type ProjectContext, type Stage, type StartVariantsBriefContent, type StaticPreviewTargetElement, type Target, type TerminalSummary, type VariantContextBinding, type VariantContextBundle, type VariantQaResult, type VariantWorkItemContext, type WorkItem, type WorkItemKind, type WorkItemStatus, type WorkerFidelity, type WorkerRuntimeInfo } from './contracts';
+import { DirectionRefReservations } from './DirectionRefReservations';
+import type { DirectionRef } from '../types/direction-ref';
 import { type VariantWorkItemInput, type VariantWorkItemInputByKind, type VariantWorkItemKind } from './workItemInput';
 export declare const DEFAULT_LEASE_TTL_MS: number;
 export declare const SCAFFOLD_LEASE_TTL_MS: number;
@@ -60,7 +62,8 @@ export declare const buildImageRefs: (artifacts: ReadonlyArray<{
 export declare class SessionStore {
     private readonly sessions;
     private readonly clock;
-    constructor(clock?: SessionStoreClock);
+    readonly directionRefs: DirectionRefReservations;
+    constructor(clock?: SessionStoreClock, directionRefs?: DirectionRefReservations);
     propose(args: ProposeArgs): ProposeResult;
     reportBriefs(args: {
         sessionId: string;
@@ -163,6 +166,7 @@ export declare class SessionStore {
         references?: VariantReference[];
         previewRoutes?: string[];
         runLabel?: string;
+        sourceDirectionRef?: DirectionRef;
     }): {
         workItemId: string;
         briefId: string;
@@ -180,12 +184,17 @@ export declare class SessionStore {
         references?: VariantReference[];
         previewRoutes?: string[];
         runLabel?: string;
+        sourceDirectionRef?: DirectionRef;
         projectContext: ProjectContext;
     }): {
         sessionId: string;
         workItemId: string;
         briefId: string;
     };
+    claimDirectionRef(args: {
+        sessionId: string;
+        workItemId: string;
+    }): DirectionRef;
     private findAdoptedVariant;
     assignVariantPreviewRoutes(args: {
         sessionId: string;

package/dist/agent-variants/SessionStore.js

@@ -7,6 +7,7 @@ const crypto_1 = require("crypto");
 const errors_1 = require("./errors");
 const contracts_1 = require("./contracts");
 const briefContent_1 = require("./briefContent");
+const DirectionRefReservations_1 = require("./DirectionRefReservations");
 const runLabel_1 = require("./runLabel");
 const runPlan_1 = require("./runPlan");
 const workItemInput_1 = require("./workItemInput");
@@ -185,8 +186,10 @@ const resolveVariantWorkItemContext = (args) => {
 class SessionStore {
     sessions = new Map();
     clock;
-    constructor(clock = SYSTEM_CLOCK) {
+    directionRefs;
+    constructor(clock = SYSTEM_CLOCK, directionRefs = new DirectionRefReservations_1.DirectionRefReservations()) {
         this.clock = clock;
+        this.directionRefs = directionRefs;
     }
     propose(args) {
         const sessionId = args.sessionId ?? (0, crypto_1.randomUUID)();
@@ -345,6 +348,7 @@ class SessionStore {
                     input: {
                         briefId: brief.briefId,
                         briefLabel: brief.label,
+                        directionRef: this.directionRefs.reserve(brief.label),
                         briefBody: sel.bodyOverride ?? brief.body,
                         instruction: session.prompt,
                         guidance: session.guidance,
@@ -376,6 +380,7 @@ class SessionStore {
                     input: {
                         briefId: brief.briefId,
                         briefLabel: brief.label,
+                        directionRef: this.directionRefs.reserve(brief.label),
                         briefBody: sel.bodyOverride ?? brief.body,
                         instruction: session.prompt,
                         guidance: session.guidance,
@@ -582,6 +587,7 @@ class SessionStore {
                 guidance: args.guidance ?? [],
                 briefId: targetInput.briefId,
                 briefLabel: targetInput.briefLabel,
+                ...(0, workItemInput_1.shownDirectionRef)(targetInput),
                 briefBody: targetInput.briefBody,
                 references: targetInput.references,
                 ...(targetInput.briefNotes
@@ -660,6 +666,7 @@ class SessionStore {
                     },
                 }
                 : sourceInput.context;
+            const briefLabel = brief?.label ?? `${labelBase}${suffix}`;
             const item = {
                 id: (0, crypto_1.randomUUID)(),
                 kind: 'static_preview',
@@ -668,7 +675,8 @@ class SessionStore {
                 dependsOn: [],
                 input: {
                     briefId: `${sourceInput.briefId ?? 'v'}-vary-${(0, crypto_1.randomUUID)().slice(0, 8)}`,
-                    briefLabel: brief?.label ?? `${labelBase}${suffix}`,
+                    briefLabel,
+                    directionRef: this.directionRefs.reserve(briefLabel),
                     briefBody: brief?.body ?? prettyInstruction,
                     ...(brief?.notes ? { briefNotes: brief.notes } : {}),
                     ...(previewRoutes.length ? { previewRoutes } : {}),
@@ -757,6 +765,7 @@ class SessionStore {
                 input: {
                     briefId: brief.briefId,
                     briefLabel: brief.label,
+                    directionRef: this.directionRefs.reserve(brief.label),
                     briefBody: brief.body,
                     references: (0, variant_reference_1.mergeVariantReferences)(sourceInput?.references, context?.references),
                     instruction: args.instruction,
@@ -815,6 +824,9 @@ class SessionStore {
             input: {
                 briefId,
                 briefLabel: args.label,
+                ...(args.sourceDirectionRef
+                    ? { sourceDirectionRef: args.sourceDirectionRef }
+                    : {}),
                 ...(args.description ? { briefBody: args.description } : {}),
                 ...(args.runLabel ? { runLabel: args.runLabel } : {}),
                 adoptedFrom: { ...args.source },
@@ -854,6 +866,7 @@ class SessionStore {
             references: args.references,
             previewRoutes: args.previewRoutes,
             runLabel: args.runLabel,
+            sourceDirectionRef: args.sourceDirectionRef,
         });
         return {
             sessionId,
@@ -861,6 +874,15 @@ class SessionStore {
             briefId: adopted.briefId,
         };
     }
+    claimDirectionRef(args) {
+        const item = this.requireWorkItem(this.requireSession(args.sessionId), args.workItemId);
+        if (!(0, workItemInput_1.isVariantWorkItem)(item))
+            throw new errors_1.AgentVariantsError('WORK_ITEM_NOT_FOUND', `Work item ${item.id} is a ${item.kind} work item, not a direction`);
+        const directionRef = item.input.directionRef ??
+            this.directionRefs.reserve(item.input.briefLabel ?? '');
+        item.input.directionRef = directionRef;
+        return directionRef;
+    }
     findAdoptedVariant(session, source) {
         for (const item of session.workItems.values()) {
             if (item.removed || !(0, workItemInput_1.isVariantWorkItem)(item))
@@ -1427,6 +1449,7 @@ class SessionStore {
                     ? { previewRoutes: [...input.previewRoutes] }
                     : {}),
                 briefId: input.briefId ?? '',
+                ...(0, workItemInput_1.shownDirectionRef)(input),
                 label: input.briefLabel ?? 'Variant',
                 references: input.references,
                 status: item.status,

package/dist/agent-variants/WorkItemRequestTimes.d.ts

@@ -0,0 +1,6 @@
+export declare class WorkItemRequestTimes {
+    private readonly requestedAt;
+    record(workItemIds: readonly string[], requestedAt: number): void;
+    elapsedSinceRequestMs(workItemId: string, at: number | undefined): number | null;
+}
+//# sourceMappingURL=WorkItemRequestTimes.d.ts.map

package/dist/agent-variants/WorkItemRequestTimes.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.WorkItemRequestTimes = void 0;
+class WorkItemRequestTimes {
+    requestedAt = new Map();
+    record(workItemIds, requestedAt) {
+        for (const workItemId of workItemIds) {
+            if (!this.requestedAt.has(workItemId)) {
+                this.requestedAt.set(workItemId, requestedAt);
+            }
+        }
+    }
+    elapsedSinceRequestMs(workItemId, at) {
+        const requestedAt = this.requestedAt.get(workItemId);
+        if (requestedAt === undefined || at === undefined)
+            return null;
+        return at - requestedAt;
+    }
+}
+exports.WorkItemRequestTimes = WorkItemRequestTimes;
+//# sourceMappingURL=WorkItemRequestTimes.js.map

package/dist/agent-variants/WorktreeOrchestrator.d.ts

@@ -4,7 +4,7 @@ import type { VariantStartRequest, VariantStartResult } from './variant-start-ty
 import { AgentVariantsError } from './errors';
 import { SessionStore, type ApprovedBriefSelection, type ApproveResult, type ProposeArgs, type ProposeResult, type ReportCompleteResult, type RequestWorkResult } from './SessionStore';
 import type { ActiveProjectContext, AdoptPastVariantArgs, BootstrapPastVariantSessionArgs, BootstrapPastVariantSessionResult, ActiveVariantSnapshot, Brief, ProjectContext, Stage, TerminalSummary, VariantContextBinding, HostVariantWorkItemSnapshot, VariantQaResult, WorkItemStatus } from './contracts';
-import type { VariantWorkItemInput } from './workItemInput';
+import { type VariantWorkItemInput } from './workItemInput';
 import { VariantHistoryService } from '../services/VariantHistoryService';
 import type { MCPVariantsTelemetryMetadata } from '../services/MCPVariantsTelemetry';
 import type { ProjectEnvironment, WorktreeManagerLike } from './variant-project-runtime';
@@ -249,6 +249,7 @@ export declare class AgentVariantsOrchestrator {
     private readonly pendingHistoryPersists;
     private trackHistoryPersist;
     settleHistoryPersists(): Promise<void>;
+    loadDirectionRefs(): Promise<void>;
     private getSavedVariant;
     isHistoryDurableForRestart(isForced?: boolean): Promise<boolean>;
     private persistCompletedExistingVariant;