cozyclay 3.1.0 → 3.1.2

package/src/workflow/agent-plan.js

@@ -0,0 +1,59 @@
+// A turn's step checklist as the agent panel keeps it (AgentPlanCard.jsx draws it): the sidecar's plan tool
+// (bin/agent/plan-tool.mjs) sends the whole list in every plan.update frame, and the chat store (agent-client.js) keeps
+// one plan card per turn, right under the turn's prompt. The card never claims more than happened: when the turn ends,
+// the step still under way counts as done only if the turn succeeded, and is left unfinished when it failed or stopped.
+// Pure data, like agent-turn.js.
+
+const STATUSES = new Set(["pending", "active", "done"]);
+const MAX_STEPS = 12, TEXT_MAX = 80;
+
+/** A frame's steps as the card keeps them, or null when they are not a plan. */
+function planSteps(steps) {
+	if (!Array.isArray(steps) || !steps.length || steps.length > MAX_STEPS) return null;
+	const kept = steps.map(step => ({ text: String(step?.text ?? "").replace(/\s+/g, " ").trim().slice(0, TEXT_MAX), status: step?.status }));
+	return kept.every(step => step.text && STATUSES.has(step.status)) ? kept : null;
+}
+
+/** Whether a frame belongs to the plan: its update, or the plan tool's own call (the card shows it, never a step row). */
+export const isPlanFrame = event => event?.type === "plan.update" || (event?.name === "plan" && ["tool.compose", "tool.start", "tool.progress"].includes(event?.type));
+
+/** items after a plan frame: the turn's one plan card, under its prompt, takes the frame's list whole. */
+export function withPlanFrame(items, event, newId) {
+	const steps = event?.type === "plan.update" ? planSteps(event.steps) : null;
+	if (!steps) return items;
+	const prompt = items.findLastIndex(item => item.kind === "user" && !item.steered);
+	const at = items.findIndex((item, i) => i > prompt && item.kind === "plan");
+	if (at !== -1) return items.map((item, i) => i === at ? { ...item, steps } : item);
+	return [...items.slice(0, prompt + 1), { kind: "plan", id: newId(), steps }, ...items.slice(prompt + 1)];
+}
+
+/** A plan whose turn ended ("done", "failed" or "stopped"): the step under way is done only when the turn succeeded. */
+export function settledPlan(plan, status) {
+	const outcome = status === "done" || status === "stopped" ? status : "failed";
+	return { ...plan, outcome, steps: plan.steps.map(step => step.status === "active" ? { ...step, status: outcome === "done" ? "done" : "unfinished" } : step) };
+}
+
+/** items at the end of a turn: its plan card settles (settledPlan); earlier turns' cards already have. */
+export const withPlanEnd = (items, status) => items.map(item => item.kind === "plan" && !item.outcome ? settledPlan(item, status) : item);
+
+/**
+ * A restored transcript's plans (session-store.mjs lists one per change): each turn keeps its last list, under its
+ * prompt; the plan tool's own calls go; and the card settles by how the turn ended: failed when a failure follows, done
+ * when the turn ends on the reply, else stopped (the history ends mid-turn: the panel reloaded while it ran).
+ */
+export function restoredPlans(items) {
+	const turns = [];
+	for (const item of items) {
+		if (item.kind === "tool" && item.name === "plan") continue;
+		if (item.kind === "user" || !turns.length) turns.push([]);
+		turns.at(-1).push(item);
+	}
+	return turns.flatMap(turn => {
+		const rest = turn.filter(item => item.kind !== "plan"), plan = turn.findLast(item => item.kind === "plan");
+		const steps = plan ? planSteps(plan.steps) : null;
+		if (!steps) return rest;
+		const status = rest.some(item => item.kind === "failure") ? "failed" : rest.at(-1)?.kind === "assistant" ? "done" : "stopped";
+		const prompt = rest[0]?.kind === "user" ? 1 : 0;
+		return [...rest.slice(0, prompt), settledPlan({ ...plan, steps }, status), ...rest.slice(prompt)];
+	});
+}

package/src/workflow/keyframe-pack-request.js

@@ -8,6 +8,7 @@
  * window and the listener pair.
  */
 import { exportFailureCode } from "../analytics.js";
+import { saveExport } from "../export-workspace.js";
 
 export const KEYFRAME_PACK_REQUEST = "cozyclay:export-keyframe-pack";
 export const KEYFRAME_PACK_RESULT = "cozyclay:export-keyframe-pack-result";
@@ -64,15 +65,7 @@ export function requestKeyframePack(iframeWindow, {
 	});
 }
 
-/** Save a pack the browser already holds in memory. */
+/** Save a pack the browser already holds in memory; resolves when handed off. */
 export function downloadPack({ name, bytes }) {
-	const url = URL.createObjectURL(new Blob([bytes], { type: "application/zip" }));
-	const anchor = document.createElement("a");
-	anchor.href = url;
-	anchor.download = name || "cozyclay-shot.zip";
-	anchor.click();
-	// Revoking synchronously can race the download in some browsers; one task
-	// later is enough for the click to have been handed off.
-	setTimeout(() => URL.revokeObjectURL(url), 0);
-	return { name: anchor.download, url };
+	return saveExport(new Blob([bytes], { type: "application/zip" }), name || "cozyclay-shot.zip");
 }

package/src/workflow/task-board.css

@@ -31,7 +31,7 @@
 
 .task-dot { flex: none; width: 7px; height: 7px; border-radius: 50%; background: var(--text-3, var(--agent-text-dim)); }
 [data-status="running"] > .task-dot, [data-status="starting"] > .task-dot { background: var(--select, var(--agent-accent)); animation: task-dot-pulse 1.2s ease-in-out infinite; }
-[data-status="waiting"] > .task-dot, [data-status="queued"] > .task-dot { background: none; box-shadow: inset 0 0 0 1.5px var(--text-3, var(--agent-text-dim)); }
+[data-status="waiting"] > .task-dot, [data-status="queued"] > .task-dot, [data-status="review"] > .task-dot { background: none; box-shadow: inset 0 0 0 1.5px var(--text-3, var(--agent-text-dim)); }
 [data-status="done"] > .task-dot { background: var(--ok, var(--agent-ok)); }
 [data-status="failed"] > .task-dot, [data-status="paused"] > .task-dot { background: var(--danger, var(--agent-alert)); }
 @keyframes task-dot-pulse { 50% { opacity: 0.35; } }

package/src/workflow/task-board.js

@@ -2,7 +2,7 @@
 // toggle) or the server's COZYCLAY_STUDIO_TASKS flag: the chat then talks to the main agent (POST /agent/main), which
 // answers at once and dispatches tasks; this board shows them. Pure parts here (the reducer, labels, the switch's
 // storage); the network side is createTaskBoardClient; the view is TaskBoard.jsx.
-//   server events (GET /agent/main/<sessionId>/events): task.queued|waiting|started|resumed|paused|done|failed|cancelled
+//   server events (GET /agent/main/<sessionId>/events): task.queued|waiting|started|resumed|paused|review|done|failed|cancelled
 //     carry the task's view { taskId, title, status, scope, reason?, turnId?, usage, summary? }; task.sceneChanged
 //     { taskId, keys, midCommand }; main.turn { turnId } (the main agent reporting on its own)
 //   editor events (the command bus): task.userEdit { taskId, keys } (the user changed something in its scope)
@@ -25,6 +25,7 @@ export const TASK_STATUS_COPY = {
 	starting: ko("Starting", "시작 중"),
 	running: ko("Running", "작업 중"),
 	paused: ko("Paused: Studio lost", "일시정지: Studio 연결 끊김"),
+	review: ko("Being checked", "확인 중"),
 	done: ko("Done", "완료"),
 	failed: ko("Failed", "실패"),
 	cancelled: ko("Cancelled", "취소됨"),
@@ -92,15 +93,16 @@ export function boardGroups(board) {
 	const groups = { failed: [], running: [], waiting: [], done: [] };
 	for (const id of board.order) {
 		const task = board.tasks[id];
-		const group = task.status === "failed" || task.status === "paused" ? "failed" : task.status === "running" || task.status === "starting" ? "running" : ACTIVE_STATUSES.has(task.status) ? "waiting" : "done";
+		// review: the run ended but the coordinator still checks it, so it waits (and offers no cancel: not in ACTIVE_STATUSES).
+		const group = task.status === "failed" || task.status === "paused" ? "failed" : task.status === "running" || task.status === "starting" ? "running" : ACTIVE_STATUSES.has(task.status) || task.status === "review" ? "waiting" : "done";
 		groups[group].push(task);
 	}
 	return groups;
 }
 
-/** The task ids a turn's dispatch_task steps started, in order (a duplicate points at the task it repeats). */
-export const dispatchedTaskIds = steps => (steps ?? []).filter(step => step.kind === "tool" && step.name === "dispatch_task")
-	.map(step => step.result?.taskId ?? step.result?.duplicateOf).filter(Boolean);
+/** The task ids a turn's dispatch_task and dispatch_plan steps started, in order (a duplicate points at the task it repeats). */
+export const dispatchedTaskIds = steps => (steps ?? []).filter(step => step.kind === "tool" && ["dispatch_task", "dispatch_plan"].includes(step.name))
+	.flatMap(step => (step.name === "dispatch_plan" ? step.result?.tasks ?? [] : [step.result])).map(result => result?.taskId ?? result?.duplicateOf).filter(Boolean);
 
 /** What the task view shares between the headless board (TaskBoard.jsx) and the rows in the chat (TaskInline.jsx). */
 function createTaskViewStore() {

package/tools/loc-baseline.json

@@ -15,7 +15,7 @@
 	"mcp/verify-live-p0.mjs": 264,
 	"mcp/verify-live.mjs": 416,
 	"src/analytics.js": 1038,
-	"src/App.jsx": 4476,
+	"src/App.jsx": 4448,
 	"src/asset-pane.jsx": 643,
 	"src/camera-follow.js": 505,
 	"src/camera-move.js": 289,
@@ -58,7 +58,6 @@
 	"src/shell/BottomDock.jsx": 416,
 	"src/shell/DetailsSlot.jsx": 334,
 	"src/shell/MenuBar.jsx": 632,
-	"src/shell/PreferencesDialog.jsx": 269,
 	"src/shell/ViewportToolbar.jsx": 567,
 	"src/shot-authoring.js": 345,
 	"src/stage-canvas-rigs.jsx": 350,

package/tools/run-tests.mjs

@@ -14,7 +14,7 @@ if (!process.env.COZYCLAY_AGENT_SESSIONS_DIR) {
 }
 
 const NODE_FILES = [
-	"test/motion3/verify-motion3-core.mjs", "test/motion3/verify-playback-frame-store.mjs", "test/verify-playback-render-isolation.mjs", "test/motion3/verify-motion3-idle.mjs", "test/motion3/verify-motion3-flip.mjs", "test/motion3/verify-motion3-timing.mjs", "test/motion3/verify-motion3-self.mjs", "test/motion3/verify-motion3-commands.mjs", "test/motion3/verify-motion3-exitcar.mjs", "test/motion3/verify-motion3-agentloop.mjs", "test/motion3/verify-motion3-search.mjs", "test/motion3/verify-motion3-skills.mjs", "test/motion3/verify-motion3-carried.mjs", "test/motion3/verify-motion3-people.mjs", "test/motion3/verify-motion3-hit.mjs", "test/motion3/verify-motion3-socket.mjs", "test/motion3/verify-motion3-shapes.mjs", "test/motion3/verify-motion3-grasp-fields.mjs", "test/motion3/verify-motion3-finger-contact.mjs", "test/motion3/verify-motion3-grasp-qa.mjs", "test/motion3/verify-motion3-load.mjs", "test/motion3/verify-motion3-throw.mjs", "test/motion3/verify-motion3-catch.mjs", "test/motion3/verify-motion3-grasp-hints.mjs", "test/motion3/verify-motion3-reach-human.mjs", "test/motion3/verify-motion3-stance-grasp.mjs", "test/motion3/verify-motion3-grasp-friction.mjs", "test/motion3/verify-motion3-grasp-measure.mjs", "test/motion3/verify-motion3-contracts.mjs", "test/motion3/verify-motion3-describe.mjs", "test/motion3/verify-motion3-timeline.mjs", "test/motion3/verify-motion3-probes.mjs", "test/motion3/verify-motion3-grab.mjs", "test/motion3/verify-motion3-grab-blend.mjs", "test/motion3/verify-motion3-patch.mjs", "test/motion3/verify-motion3-repair.mjs", "test/motion3/verify-motion3-specstore.mjs", "test/motion3/verify-motion3-visual.mjs", "test/motion3/verify-motion3-goals.mjs", "test/motion3/verify-motion3-prompt.mjs", "test/motion3/verify-motion3-cup.mjs", "test/verify-proxy-figure.mjs", "test/verify-text-object.mjs",
+	"test/motion3/verify-motion3-core.mjs", "test/motion3/verify-playback-frame-store.mjs", "test/verify-playback-render-isolation.mjs", "test/motion3/verify-motion3-idle.mjs", "test/motion3/verify-motion3-life.mjs", "test/motion3/verify-motion3-overlap.mjs", "test/motion3/verify-motion3-flip.mjs", "test/motion3/verify-motion3-timing.mjs", "test/motion3/verify-motion3-self.mjs", "test/motion3/verify-motion3-commands.mjs", "test/motion3/verify-motion3-exitcar.mjs", "test/motion3/verify-motion3-agentloop.mjs", "test/motion3/verify-motion3-search.mjs", "test/motion3/verify-motion3-skills.mjs", "test/motion3/verify-motion3-carried.mjs", "test/motion3/verify-motion3-people.mjs", "test/motion3/verify-motion3-hit.mjs", "test/motion3/verify-motion3-socket.mjs", "test/motion3/verify-motion3-shapes.mjs", "test/motion3/verify-motion3-grasp-fields.mjs", "test/motion3/verify-motion3-finger-contact.mjs", "test/motion3/verify-motion3-grasp-qa.mjs", "test/motion3/verify-motion3-load.mjs", "test/motion3/verify-motion3-throw.mjs", "test/motion3/verify-motion3-catch.mjs", "test/motion3/verify-motion3-grasp-hints.mjs", "test/motion3/verify-motion3-reach-human.mjs", "test/motion3/verify-motion3-stance-grasp.mjs", "test/motion3/verify-motion3-grasp-friction.mjs", "test/motion3/verify-motion3-grasp-measure.mjs", "test/motion3/verify-motion3-contracts.mjs", "test/motion3/verify-motion3-describe.mjs", "test/motion3/verify-motion3-timeline.mjs", "test/motion3/verify-motion3-probes.mjs", "test/motion3/verify-motion3-grab.mjs", "test/motion3/verify-motion3-grab-blend.mjs", "test/motion3/verify-motion3-patch.mjs", "test/motion3/verify-motion3-patch-keys.mjs", "test/motion3/verify-motion3-stuck.mjs", "test/motion3/verify-motion3-repair.mjs", "test/motion3/verify-motion3-specstore.mjs", "test/motion3/verify-motion3-visual.mjs", "test/motion3/verify-motion3-goals.mjs", "test/motion3/verify-motion3-prompt.mjs", "test/motion3/verify-motion3-cup.mjs", "test/motion3/verify-motion3-arm.mjs", "test/verify-proxy-figure.mjs", "test/verify-text-object.mjs",
 	"test/verify-proxy-figure-bounds.mjs",
 	"test/verify-character-capabilities.mjs",
 	"test/verify-root-path.mjs",
@@ -49,7 +49,7 @@ const NODE_FILES = [
 	"test/process/verify-package-telemetry.mjs",
 	"test/process/verify-agent-deps.mjs",
 	"test/verify-agent-image-references.mjs",
-	"test/verify-agent-panel.mjs", "test/verify-agent-turn-view.mjs", "test/verify-agent-reply-text.mjs",
+	"test/verify-agent-panel.mjs", "test/verify-agent-turn-view.mjs", "test/verify-agent-plan.mjs", "test/verify-agent-reply-text.mjs", "test/verify-agent-sculpt-sketch.mjs",
 	"test/verify-agent-host.mjs",
 	"test/verify-agent-routes.mjs",
 	"test/verify-studio-owner-restart.mjs",
@@ -81,14 +81,14 @@ const NODE_FILES = [
 	"test/verify-auto-color.mjs",
 	"test/verify-coplanar-depth.mjs",
 	"test/verify-empty-object.mjs",
-	"test/verify-sculpt-recipe.mjs",
+	"test/verify-sculpt-recipe.mjs", "test/verify-sculpt-shapes.mjs", "test/verify-sculpt-loft-convex.mjs", "test/verify-sculpt-detail.mjs",
 	"test/verify-sculpt-object.mjs",
 	"test/verify-sculpt-compare.mjs",
 	"test/verify-sculpt-run.mjs",
 	"test/verify-sculpt-compare-command.mjs",
 	"test/verify-sculpt-panel.mjs",
 	"test/verify-sculpt-blueprint.mjs",
-	"test/verify-sculpt-mode.mjs",
+	"test/verify-sculpt-mode.mjs", "test/verify-sculpt-csg.mjs", "test/verify-material-presets.mjs",
 	"test/verify-route-owner.mjs",
 	"test/verify-part-colours.mjs",
 	"test/verify-asset-shelf.mjs",
@@ -124,7 +124,7 @@ const NODE_FILES = [
 	"test/verify-mcp-invariants.mjs",
 	"test/verify-offscreen-export.mjs",
 	"test/verify-export-capture.mjs",
-	"test/verify-export-recovery.mjs",
+	"test/verify-export-recovery.mjs", "test/verify-export-workspace.mjs",
 	"test/verify-record-mp4-source.mjs",
 	"test/verify-timeline-extent.mjs",
 	"test/verify-otio.mjs",
@@ -196,7 +196,7 @@ const NODE_FILES = [
 	"mcp/verify-protocol-version.mjs",
 	"mcp/verify-tool-annotations.mjs",
 	"mcp/verify-import-mesh.mjs",
-	"mcp/verify-sculpt-object.mjs",
+	"mcp/verify-sculpt-object.mjs", "mcp/verify-project-camera.mjs",
 	"test/verify-object-path.mjs",
 	"test/verify-object-travel.mjs",
 	"test/verify-object-travel-lean.mjs",
@@ -216,7 +216,7 @@ const BROWSER_FILES = [
 	"test/qa-storyboard-export-browser.mjs",
 	"test/qa-proxy-figure-browser.mjs",
 	"test/qa-rail-lean-browser.mjs",
-	"test/qa-export-recovery-browser.mjs",
+	"test/qa-export-recovery-browser.mjs", "test/qa-export-workspace-browser.mjs",
 	"test/verify-camera-mode-browser.mjs",
 	"test/verify-camera-rail-browser.mjs",
 	"test/verify-cutout-browser.mjs",
@@ -255,7 +255,7 @@ const BROWSER_FILES = [
 // The inventory sweep only picks up `verify*.mjs`; a browser suite named for
 // the QA runner it needs is listed here so it still shows up in the manifest
 // (as an EXCLUDE with its reason) instead of going unmentioned.
-const EXTRA_INVENTORY = ["test/ik/translation-step.mjs", "test/qa-scenes-v5-migration-browser.mjs", "test/qa-agent-scenarios-browser.mjs", "test/qa-execution-outcomes-browser.mjs", "test/qa-export-recovery-browser.mjs", "test/qa-agent-view-toggle-browser.mjs", "test/qa-camera-pointer-lock-browser.mjs", "test/qa-first-edit-browser.mjs", "test/qa-ia-tail-browser.mjs", "test/qa-keyframe-pack-browser.mjs", "test/qa-preview-browser.mjs", "test/qa-reference-slots-browser.mjs", "test/qa-scene-playback-browser.mjs", "test/qa-scene-switcher-browser.mjs", "test/qa-send-to-ai-browser.mjs", "test/qa-view-menu-browser.mjs", "test/qa-live-reconnect-browser.mjs", "test/qa-clipboard-paste-browser.mjs", "test/qa-rail-lean-browser.mjs", "test/qa-rail-reach-browser.mjs", "test/qa-rail-ring-browser.mjs", "test/qa-speed-graph-browser.mjs"];
+const EXTRA_INVENTORY = ["test/ik/translation-step.mjs", "test/qa-scenes-v5-migration-browser.mjs", "test/qa-agent-scenarios-browser.mjs", "test/qa-execution-outcomes-browser.mjs", "test/qa-export-recovery-browser.mjs", "test/qa-agent-view-toggle-browser.mjs", "test/qa-camera-pointer-lock-browser.mjs", "test/qa-first-edit-browser.mjs", "test/qa-ia-tail-browser.mjs", "test/qa-keyframe-pack-browser.mjs", "test/qa-preview-browser.mjs", "test/qa-reference-slots-browser.mjs", "test/qa-scene-playback-browser.mjs", "test/qa-scene-switcher-browser.mjs", "test/qa-send-to-ai-browser.mjs", "test/qa-view-menu-browser.mjs", "test/qa-live-reconnect-browser.mjs", "test/qa-clipboard-paste-browser.mjs", "test/qa-rail-lean-browser.mjs", "test/qa-rail-reach-browser.mjs", "test/qa-rail-ring-browser.mjs", "test/qa-speed-graph-browser.mjs", "test/qa-export-workspace-browser.mjs"];
 
 EXTRA_INVENTORY.push("test/qa-proxy-figure-browser.mjs");
 EXTRA_INVENTORY.push("test/qa-proxy-entry-browser.mjs");