rivet-design 0.25.0 → 0.26.0

package/README.md

@@ -80,7 +80,7 @@ rivet status --project /path/to/app
 rivet variants start --project /path/to/app --instruction "try three hero layouts" --count 3
 rivet wait <workId> --project /path/to/app --timeout 120
 rivet variants list --project /path/to/app
-rivet variants commit <variantId> --project /path/to/app
+rivet variants get <projectId>/<sessionId>:<variantId> --detail
 rivet changes request --project /path/to/app --instruction "make the header coral" --element "#header"
 rivet changes pull --project /path/to/app
 rivet update
@@ -105,7 +105,7 @@ rivet open --project /path/to/app --caller-agent cursor --caller-model <model> -
 1. `rivet open` starts or attaches to the local Rivet editor server for a project. It can attach to an existing dev server or start the app's standard `dev`, `start`, or `serve` script when possible.
 2. The editor shows your app and variant previews in the browser. Agents use `rivet variants ...` and `rivet changes ...` to enqueue work.
 3. Use `rivet wait`, `rivet status`, or `rivet status --watch` to observe progress.
-4. Commit the chosen variant with `rivet variants commit <variantId>` or use the editor to keep refining.
+4. Copy the chosen direction’s context and ask your coding agent to implement it in your project. Rivet preserves the prototype; your agent owns integration and verification.
 
 If Rivet reports pull mode, staged work can be drained with `rivet changes pull --project <path>`, applied by the agent, then acknowledged with `rivet changes pull --project <path> --ack <leaseId>`.
 

package/README.npm.md

@@ -80,7 +80,7 @@ rivet status --project /path/to/app
 rivet variants start --project /path/to/app --instruction "try three hero layouts" --count 3
 rivet wait <workId> --project /path/to/app --timeout 120
 rivet variants list --project /path/to/app
-rivet variants commit <variantId> --project /path/to/app
+rivet variants get <projectId>/<sessionId>:<variantId> --detail
 rivet changes request --project /path/to/app --instruction "make the header coral" --element "#header"
 rivet changes pull --project /path/to/app
 rivet update
@@ -105,7 +105,7 @@ rivet open --project /path/to/app --caller-agent cursor --caller-model <model> -
 1. `rivet open` starts or attaches to the local Rivet editor server for a project. It can attach to an existing dev server or start the app's standard `dev`, `start`, or `serve` script when possible.
 2. The editor shows your app and variant previews in the browser. Agents use `rivet variants ...` and `rivet changes ...` to enqueue work.
 3. Use `rivet wait`, `rivet status`, or `rivet status --watch` to observe progress.
-4. Commit the chosen variant with `rivet variants commit <variantId>` or use the editor to keep refining.
+4. Copy the chosen direction’s context and ask your coding agent to implement it in your project. Rivet preserves the prototype; your agent owns integration and verification.
 
 If Rivet reports pull mode, staged work can be drained with `rivet changes pull --project <path>`, applied by the agent, then acknowledged with `rivet changes pull --project <path> --ack <leaseId>`.
 

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

@@ -1,4 +1,3 @@
-import type { ChildProcess } from 'child_process';
 import { type PreviewPackageManager } from '../services/ProjectPackageManager';
 import { type DevServerFailureReason } from '../utils/devServerError';
 import type { ProjectContext } from './contracts';
@@ -32,12 +31,6 @@ type PreviewRuntimeDeps = {
     hasUiClients: () => boolean;
     changed: () => void;
     switchPreviewPort?: (port: number | null) => void;
-    setCommittedDevServerHealth?: (port: number | null) => void;
-    startStaticPreviewServer?: (args: {
-        rootPath: string;
-        port: number;
-        host?: string;
-    }) => Promise<ChildProcess>;
     activeVariantServerIdleTtlMs: number;
     devServerRestartDelayMs: number;
 };
@@ -45,9 +38,6 @@ export declare class PreviewRuntime {
     private readonly deps;
     private readonly sessions;
     private readonly environment;
-    private readonly lingeringCommitted;
-    private readonly committedGenerations;
-    private generation;
     constructor(deps: PreviewRuntimeDeps);
     openSession(sessionId: string): void;
     registerVariant(sessionId: string, workItemId: string, worktreePath: string): void;
@@ -75,14 +65,7 @@ export declare class PreviewRuntime {
         retryFailed?: boolean;
     }): Promise<number | undefined>;
     stopVariantServers(sessionId: string): Promise<number>;
-    startCommitted(args: {
-        sessionId: string;
-        destinationPath: string;
-        mode?: 'vite_app' | 'static_preview';
-    }): Promise<number | undefined>;
-    getCommittedPort(sessionId: string): number | undefined;
-    stopLingeringCommitted(): Promise<void>;
-    closeSession(sessionId: string, reason: 'committed' | 'cancel' | 'shutdown'): Promise<void>;
+    closeSession(sessionId: string): Promise<void>;
     resolveCommand(sessionId: string, worktreePath: string, port: number, isFresh: boolean): Promise<ResolvedDevServer>;
     private startVariantProcess;
     private watchVariantProcess;
@@ -91,8 +74,6 @@ export declare class PreviewRuntime {
     private resolveRunnableCommand;
     private resolveStartPort;
     private resolveExistingProjectCwd;
-    private notifyCommittedPort;
-    private notifyCommittedHealth;
     private notifyPreviewPort;
     private clearIdleTimer;
     private requireSession;

package/dist/agent-variants/PreviewRuntime.js

@@ -8,7 +8,6 @@ const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
 const PreviewEnvironment_1 = require("../services/PreviewEnvironment");
 const ProjectPackageManager_1 = require("../services/ProjectPackageManager");
-const StaticPreviewServer_1 = require("../services/StaticPreviewServer");
 const devServerCommand_1 = require("../utils/devServerCommand");
 const devServerError_1 = require("../utils/devServerError");
 const logger_1 = require("../utils/logger");
@@ -25,18 +24,13 @@ class PreviewRuntime {
     deps;
     sessions = new Map();
     environment = new PreviewEnvironment_1.PreviewEnvironment();
-    lingeringCommitted = new Map();
-    committedGenerations = new Map();
-    generation = 0;
     constructor(deps) {
         this.deps = deps;
     }
     openSession(sessionId) {
         if (this.sessions.has(sessionId))
             return;
-        this.committedGenerations.delete(sessionId);
         this.sessions.set(sessionId, {
-            generation: this.generation,
             variants: new Map(),
             failures: new Map(),
         });
@@ -143,94 +137,11 @@ class PreviewRuntime {
         await Promise.all(stops);
         return stops.length;
     }
-    async startCommitted(args) {
-        const state = this.sessions.get(args.sessionId);
-        const committedGeneration = this.committedGenerations.get(args.sessionId);
-        if (state?.generation !== this.generation &&
-            committedGeneration !== this.generation) {
-            return undefined;
-        }
-        const mode = args.mode ?? 'vite_app';
-        try {
-            const port = await this.deps.worktreeManager.getFreePort();
-            const process = mode === 'static_preview'
-                ? await (this.deps.startStaticPreviewServer ?? StaticPreviewServer_1.startStaticPreviewServer)({
-                    rootPath: args.destinationPath,
-                    port,
-                    host: FRESH_DEV_SERVER_HOST,
-                })
-                : await this.deps.worktreeManager.startDevServer(args.destinationPath, port, 'npm', [
-                    'run',
-                    'dev',
-                    '--',
-                    '--port',
-                    String(port),
-                    '--host',
-                    FRESH_DEV_SERVER_HOST,
-                ], { PORT: String(port) });
-            const committed = { process, port, path: args.destinationPath };
-            if (this.sessions.get(args.sessionId) === state &&
-                state?.generation === this.generation) {
-                state.committed = committed;
-            }
-            else if (this.committedGenerations.get(args.sessionId) === this.generation) {
-                this.lingeringCommitted.set(args.sessionId, committed);
-            }
-            else {
-                await this.deps.worktreeManager.stopDevServer(process);
-                return undefined;
-            }
-            this.notifyCommittedPort(port);
-            log.info(`Committed ${mode} dev server up at ${args.destinationPath} on port ${port}`);
-            return port;
-        }
-        catch (error) {
-            log.warn(`Failed to start committed ${mode} dev server at ${args.destinationPath}`, error);
-            return undefined;
-        }
-    }
-    getCommittedPort(sessionId) {
-        return (this.lingeringCommitted.get(sessionId)?.port ??
-            this.sessions.get(sessionId)?.committed?.port);
-    }
-    async stopLingeringCommitted() {
-        this.generation += 1;
-        this.committedGenerations.clear();
-        const entries = [...this.lingeringCommitted.entries()];
-        this.lingeringCommitted.clear();
-        if (entries.length > 0)
-            this.notifyCommittedHealth(null);
-        await Promise.all(entries.map(async ([sessionId, entry]) => {
-            try {
-                await this.deps.worktreeManager.stopDevServer(entry.process);
-            }
-            catch (error) {
-                log.warn(`stopDevServer for lingering committed server ${sessionId}/${entry.port} failed`, error);
-            }
-        }));
-    }
-    async closeSession(sessionId, reason) {
+    async closeSession(sessionId) {
         const state = this.sessions.get(sessionId);
         if (!state)
             return;
-        if (reason === 'committed') {
-            this.committedGenerations.set(sessionId, state.generation);
-        }
-        else {
-            this.committedGenerations.delete(sessionId);
-        }
         await this.stopVariantServers(sessionId);
-        if (state.committed && reason === 'committed') {
-            this.lingeringCommitted.set(sessionId, state.committed);
-        }
-        else if (state.committed) {
-            try {
-                await this.deps.worktreeManager.stopDevServer(state.committed.process);
-            }
-            catch (error) {
-                log.warn(`stopDevServer for committed server ${sessionId} failed`, error);
-            }
-        }
         for (const preview of state.variants.values()) {
             this.environment.forget(preview.worktreePath);
         }
@@ -555,18 +466,6 @@ class PreviewRuntime {
         }
         return path_1.default.join(worktreePath, path_1.default.relative(environment.projectPath, projectContext.projectPath));
     }
-    notifyCommittedPort(port) {
-        this.notifyCommittedHealth(port);
-        this.notifyPreviewPort(port);
-    }
-    notifyCommittedHealth(port) {
-        try {
-            this.deps.setCommittedDevServerHealth?.(port);
-        }
-        catch (error) {
-            log.warn(`setCommittedDevServerHealth(${port}) failed`, error);
-        }
-    }
     notifyPreviewPort(port) {
         try {
             this.deps.switchPreviewPort?.(port);

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

@@ -1,7 +1,7 @@
 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 VariantPickEnvelope, type WorkItem, type WorkItemKind, type WorkItemStatus, type WorkerFidelity, type WorkerRuntimeInfo } from './contracts';
+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 { type VariantWorkItemInput, type VariantWorkItemInputByKind, type VariantWorkItemKind } from './workItemInput';
 export declare const DEFAULT_LEASE_TTL_MS: number;
 export declare const SCAFFOLD_LEASE_TTL_MS: number;
@@ -266,12 +266,6 @@ export declare class SessionStore {
         workItemIds: string[];
         sessionStage: Stage;
     };
-    recordVariantPick(args: {
-        sessionId: string;
-        envelope: VariantPickEnvelope;
-    }): {
-        duplicate: boolean;
-    };
     getStage(sessionId: string): Stage;
     getPrompt(sessionId: string): string;
     getRunLabel(sessionId: string): string;
@@ -297,7 +291,6 @@ export declare class SessionStore {
         input: unknown;
         outputSchema: unknown;
     }>;
-    getVariantPick(sessionId: string): VariantPickEnvelope | undefined;
     validateReportLease(args: {
         sessionId: string;
         workItemId: string;

package/dist/agent-variants/SessionStore.js

@@ -560,9 +560,6 @@ class SessionStore {
         if (target.kind !== 'static_preview' || target.status !== 'succeeded') {
             throw new errors_1.AgentVariantsError('VARIANT_NOT_EDITABLE', 'Only completed static_preview variants can be refined');
         }
-        if (session.variantPick) {
-            throw new errors_1.AgentVariantsError('PENDING_CHANGE_CONFLICT', 'Cannot refine a variant after one has already been sent');
-        }
         const activeRefinement = [...session.workItems.values()].find((item) => {
             if (item.kind !== 'refine_variant')
                 return false;
@@ -614,9 +611,6 @@ class SessionStore {
         if (source.kind !== 'static_preview' || source.status !== 'succeeded') {
             throw new errors_1.AgentVariantsError('VARIANT_NOT_EDITABLE', 'Only completed static_preview variants can be varied');
         }
-        if (session.variantPick) {
-            throw new errors_1.AgentVariantsError('PENDING_CHANGE_CONFLICT', 'Cannot vary a variant after one has already been sent');
-        }
         const activeRefinement = [...session.workItems.values()].find((item) => {
             if (item.kind !== 'refine_variant')
                 return false;
@@ -724,9 +718,6 @@ class SessionStore {
         if (session.projectContext.kind !== 'existing') {
             throw new errors_1.AgentVariantsError('INVALID_STAGE_ACTION', 'Only existing-project sessions can append code_gen variants');
         }
-        if (session.variantPick) {
-            throw new errors_1.AgentVariantsError('PENDING_CHANGE_CONFLICT', 'Cannot append variants after one has already been committed');
-        }
         if (session.stage === 'cancelled' || session.stage === 'failed') {
             throw new errors_1.AgentVariantsError('INVALID_STAGE_ACTION', `Cannot append variants to a ${session.stage} session`);
         }
@@ -749,10 +740,10 @@ class SessionStore {
         const createdWorkItemIds = [];
         for (const [batchSlotIndex, brief] of args.briefs.entries()) {
             const context = resolveVariantWorkItemContext({
-                bundle: args.variantContext ?? session.projectContext.variantContext,
+                bundle: args.variantContext,
                 projectContext: session.projectContext,
                 brief,
-                slotIndex: args.variantContext ? batchSlotIndex : session.briefs.length,
+                slotIndex: batchSlotIndex,
             });
             const visualRef = brief.visualReferenceUrl
                 ? { visualReferenceUrl: brief.visualReferenceUrl }
@@ -814,9 +805,6 @@ class SessionStore {
                 created: false,
             };
         }
-        if (session.variantPick) {
-            throw new errors_1.AgentVariantsError('PENDING_CHANGE_CONFLICT', 'Cannot adopt a variant after one has already been committed');
-        }
         const briefId = `adopted-${(0, crypto_1.randomUUID)().slice(0, 8)}`;
         const item = {
             id: (0, crypto_1.randomUUID)(),
@@ -1094,21 +1082,6 @@ class SessionStore {
         }
         return { workItemIds, sessionStage: session.stage };
     }
-    recordVariantPick(args) {
-        const session = this.requireSession(args.sessionId);
-        if (!['ready', 'degraded'].includes(session.stage)) {
-            throw new errors_1.AgentVariantsError('INVALID_STAGE_ACTION', `Cannot pick variant in stage ${session.stage}`);
-        }
-        if (session.variantPick &&
-            session.variantPick.variantId === args.envelope.variantId) {
-            return { duplicate: true };
-        }
-        if (session.variantPick) {
-            throw new errors_1.AgentVariantsError('PENDING_CHANGE_CONFLICT', `Session already has a variant pick for ${session.variantPick.variantId}`);
-        }
-        session.variantPick = args.envelope;
-        return { duplicate: false };
-    }
     getStage(sessionId) {
         return this.requireSession(sessionId).stage;
     }
@@ -1165,9 +1138,6 @@ class SessionStore {
             };
         });
     }
-    getVariantPick(sessionId) {
-        return this.requireSession(sessionId).variantPick;
-    }
     validateReportLease(args) {
         const session = this.requireSession(args.sessionId);
         const item = this.requireWorkItem(session, args.workItemId);

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

@@ -69,11 +69,8 @@ export declare class WorkspaceInventory {
     getDependencySignal(sessionId: string): AbortSignal | undefined;
     beginCleanup(sessionId: string): boolean;
     isCleanupStarted(sessionId: string): boolean;
-    preserveViteSibling(sessionId: string): void;
     disposeAssignments(args: {
         sessionId: string;
-        isFresh: boolean;
-        isCommitted: boolean;
     }): Promise<void>;
     private requireSession;
 }

package/dist/agent-variants/WorkspaceInventory.js

@@ -1,10 +1,6 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.WorkspaceInventory = void 0;
-const fs_1 = __importDefault(require("fs"));
 const VariantWorkspaceManager_1 = require("./VariantWorkspaceManager");
 const errors_1 = require("./errors");
 const logger_1 = require("../utils/logger");
@@ -25,7 +21,6 @@ class WorkspaceInventory {
             contextAssets: new Map(),
             dependencyAbortController: new AbortController(),
             isCleanupStarted: false,
-            isViteSiblingPreserved: false,
         });
     }
     hasSession(sessionId) {
@@ -131,33 +126,15 @@ class WorkspaceInventory {
     isCleanupStarted(sessionId) {
         return this.sessions.get(sessionId)?.isCleanupStarted ?? false;
     }
-    preserveViteSibling(sessionId) {
-        this.requireSession(sessionId).isViteSiblingPreserved = true;
-    }
     async disposeAssignments(args) {
         const state = this.requireSession(args.sessionId);
-        if (!state.isViteSiblingPreserved) {
-            try {
-                await this.worktreeManager.cleanupSession(args.sessionId, state.freshDestinationParent);
-            }
-            catch (error) {
-                log.warn(`cleanupSession failed for ${args.sessionId}`, error);
-            }
+        try {
+            await this.worktreeManager.cleanupSession(args.sessionId, state.freshDestinationParent);
         }
-        const ownedRoots = new Set(state.workspaceManager.getOwnedArtifactRoots());
-        state.workspaceManager.removeOwnedArtifactWorkspaces();
-        if (!args.isCommitted && args.isFresh) {
-            for (const preview of state.staticPreviews.values()) {
-                if (ownedRoots.has(preview.assetBase))
-                    continue;
-                try {
-                    fs_1.default.rmSync(preview.assetBase, { recursive: true, force: true });
-                }
-                catch (error) {
-                    log.warn(`failed to remove static preview dir ${preview.assetBase} for ${args.sessionId}`, error);
-                }
-            }
+        catch (error) {
+            log.warn(`cleanupSession failed for ${args.sessionId}`, error);
         }
+        state.workspaceManager.removeOwnedArtifactWorkspaces();
         this.sessions.delete(args.sessionId);
     }
     requireSession(sessionId) {

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

@@ -1,7 +1,6 @@
 import { type StaticPreviewRefinementRequest, type StaticPreviewRefinementResult } from './staticPreviewRefinements';
 import type { GuidanceDocument } from '../types/guidance';
 import type { VariantStartRequest, VariantStartResult } from './variant-start-types';
-import type { ChildProcess } from 'child_process';
 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';
@@ -30,16 +29,9 @@ export interface OrchestratorDeps {
         force?: boolean;
         signal?: AbortSignal;
     }) => Promise<void>;
-    materializeProject?: (sourceWorktreePath: string, destinationPath: string) => Promise<void>;
     previewQaRunner?: PreviewQaRunner;
     switchPreviewPort?: (port: number | null) => void;
-    setCommittedDevServerHealth?: (port: number | null) => void;
     variantHistory?: VariantHistoryService;
-    startStaticPreviewServer?: (args: {
-        rootPath: string;
-        port: number;
-        host?: string;
-    }) => Promise<ChildProcess>;
     provisionTimeoutMs?: number;
     activeVariantServerIdleTtlMs?: number;
     devServerRestartDelayMs?: number;
@@ -70,7 +62,6 @@ export declare class AgentVariantsOrchestrator {
     private readonly resolveEnv;
     private readonly telemetry;
     private readonly installDependencies;
-    private readonly materializeProject;
     private readonly previewQaRunner;
     private readonly variantHistory;
     private readonly provisionTimeoutMs;
@@ -80,7 +71,6 @@ export declare class AgentVariantsOrchestrator {
     private readonly workspaceInventory;
     private readonly workspaceProvisioning;
     private readonly previewRuntime;
-    private readonly inflightCommits;
     private activeSessionId;
     private readonly events;
     constructor(deps: OrchestratorDeps);
@@ -137,7 +127,6 @@ export declare class AgentVariantsOrchestrator {
     private getUsableActiveSessionId;
     isStaticPreviewVariant(sessionId: string, workItemId: string): boolean;
     private resolveHistoryParentVariant;
-    getVariantPick(sessionId: string): ReturnType<SessionStore['getVariantPick']>;
     refineVariant(args: {
         guidance?: GuidanceDocument[];
         sessionId: string;
@@ -247,21 +236,7 @@ export declare class AgentVariantsOrchestrator {
         renamedCount: number;
         sessionStage: Stage;
     }>;
-    commitVariant(args: {
-        sessionId: string;
-        variantId: string;
-        skipTeardown?: boolean;
-    }): Promise<{
-        enqueued: boolean;
-        duplicate: boolean;
-        changedFilesCount: number;
-        payloadKind?: 'diff' | 'diff-applied' | 'project-created';
-        destinationPath?: string;
-        previewUrl?: string;
-    }>;
-    private commitVariantImpl;
-    cleanupCommittedSession(sessionId: string): Promise<void>;
-    private assertDestinationAvailable;
+    cleanupSession(sessionId: string): Promise<void>;
     getVariantDiff(sessionId: string, variantId: string): string | undefined;
     private evaluateQaForReport;
     private evaluateExistingCodeGenQa;
@@ -281,16 +256,12 @@ export declare class AgentVariantsOrchestrator {
     private prepareArtifactBase;
     private createArtifactDiff;
     private persistCompletedArtifactVariant;
-    private persistCompletedFreshVariantsForCommit;
-    private persistVariantHistoryAtCommit;
     private markPersistedVariantsCancelled;
     private markPersistedVariantCancelled;
     private markPersistedVariantRemoved;
     private markPersistedVariantRenamed;
     private resolveHistoryProjectPath;
     private readManifest;
-    private preserveUnchosenVariants;
-    private renameWorktreeIntoDestination;
     private teardownSession;
     private ensureResources;
 }