mirror of
https://github.com/THU-MAIC/OpenMAIC.git
synced 2026-10-17 05:29:46 +08:00
* feat(storage): add an agent-session store with PG backend and layered contracts (#1163) * feat(storage): add agent-session store with PG backend and layered contracts * test(storage): avoid BigInt literals for pre-ES2020 root typecheck * fix(storage): close agent-session store review findings * docs(storage): align hook ordering and contention-probe claims with the code * ci: run on the agent-workbench integration branch * chore(storage): bump to 0.5.0 for the agent-session store * fix(storage): carry replay compaction across page boundaries * feat(agent): add the driver model contract and stage route dialect (#1165) * feat(agent): add the driver model contract and stage route dialect * fix(agent): validate route context windows and clarify dialect precedence * feat(agent): adapt the agent-session store and runtime foundations (#1167) * feat(agent): adapt the agent-session store and runtime foundations * feat(agent): resolve request owner identity via an anonymous cookie * docs(agent): document the opt-in compaction default and harden edge cases * feat(agent): add the background session runner (#1169) * feat(agent): add the background session runner * feat(agent): wire the runner into startup behind feature flags * fix(agent): stop clean interruptions from consuming the attempt budget * fix(storage): charge the attempt budget for abandoned leases but not clean parks * docs(storage): document the attempt-charging contract and decouple its tests * feat(agent): add agent session and owner event streams (#1170) * feat(agent): add agent session and owner event streams * fix(agent): close the session-existence oracle and document the owner seam * feat(agent): add agent session lifecycle routes (#1171) * feat(agent): add agent session lifecycle routes * fix(agent): validate session-create input and preserve the owner cookie on errors * refactor(storage): drop the unused active-stage API from the agent-session contract (#1174) * refactor(storage): drop the unused active-stage API from the agent-session contract Tools address stages explicitly on every call, so the store keeps no mutable session-level stage pointer. Removes resolveActiveStage and setActiveStage from the store interface, their PG implementations, the active_stage_changed lifecycle event, the session_active_stage owner event variant, and the contract tests pinning them. The active_stage_id column and the DDL check constraint stay untouched for schema compatibility. * chore(storage): bump @openmaic/storage to 0.7.0 for the contract removal * docs: document the agent runtime configuration surface (#1176) * fix(agent): repair orphaned and late tool results across interruption boundaries (#1180) * fix(agent): repair orphaned and late tool results across interruption boundaries A crash, shutdown, or provider failure can leave the durable transcript with tool calls that have no result, or with results ordered illegally for the provider. Three failure modes were fixed: - Orphaned tool calls: a run that died between an assistant tool-call frame and its result left a dangling call in the entry tree. Resume no longer synthesizes and persists receipts for it: interrupted results are a read-time provider view owned by a shared read-boundary repair, which returns the original array for a healthy transcript and never mutates the tree. - Late parallel results: a parallel tool can finish while pi unwinds an aborted assistant frame, leaving result(A), assistant(aborted), result(B) in durable order. Strict providers reject non-contiguous results, so the read-boundary repair moves existing results next to their owning assistant frame (in call order), omits incomplete unwind frames, and synthesizes receipts only for genuinely missing calls. - Interrupted calls at the write boundary: a call still in flight when the run winds down (shutdown, lease loss, cancellation, provider failure) had no receipt at all. The runner now tracks in-flight calls from their assistant frames and, before the terminal flush, appends an interrupted-result receipt for each still-orphaned call through the same attempt-fenced write chain, so a lease-stealing zombie never writes and the next claim sees a provider-safe transcript. * test(agent): pin the runner wiring for interruption-boundary tool repair * feat(agent): add neutral tool foundation libraries (#1184) * feat(agent): register a web_search tool on the session runner (#1185) * feat(storage): add a per-session URL trust gate (#1186) * feat(agent): add the skills system (#1189) * feat(agent): add the skills system (builtin directories and durable user skills) * fix(storage): serialize the user-skill quota check-and-insert per owner Two concurrent creates at the 50-skill boundary both counted 49 rows and both inserted (READ COMMITTED, no lock), overshooting the quota contract. The create transaction now takes a per-owner pg_advisory_xact_lock first, and the same-name idempotency check runs before the count check so an at-least-once retry of the create that committed as the owner's 50th row still returns its durable receipt instead of a quota error. The 23505 backstop is retained for writes that do not take the lock. * fix(agent): share unstorable-character validation and align skill lookup * feat(agent): add session materials and a fetch_url tool behind the URL trust gate (#1190) * feat(agent): add session materials and a fetch_url tool behind the URL trust gate * fix(agent): harden session material fetching * feat(storage): add an ownership scope to stage documents (#1191) * feat(agent): add material read and search tools (#1192) * feat(agent): add stage read and patch tools (#1194) * feat(agent): add page generation and deck editing tools (#1198) * test(storage): keep the PG contract suite order-independent (#1200) * fix(agent): revoke deleted-session URL authority and reject private ISATAP endpoints (#1199) * fix(storage): revoke deleted session URL authority * fix(ssrf): reject private ISATAP endpoints in strict fetches * chore(storage): bump to 0.11.1 for the session-URL authority fix * feat(agent): add roster and voice registration tools (#1201) * feat(agent): add folder organisation tools (#1202) * feat(api): add stage and material HTTP routes (#1203) * feat(workbench): add the client data layer (#1204) * feat(workbench): add the client data layer * docs(workbench): write the ported comments in English * chore(edit): remove the in-editor agent panel (#1210) * chore(edit): remove the in-editor agent panel * style: apply prettier formatting * fix(agent): report the runtime as unusable without a database (#1207) * fix(agent): report the runtime as unusable without a database * style: apply prettier formatting * feat(agent): add image, video and pptx import tools (#1211) * feat(workbench): add the agent chat surface (#1205) * feat(workbench): add the agent chat surface * docs(workbench): write the ported comments in English * fix(workbench): label the folder and rename tools on the timeline * fix(workbench): label the roster and voice tools on the timeline The reconciliation test iterates every tool the runner registers and requires a display label of its own. The roster and voice-clone tools (list_voices, set_roster, clip_audio, register_voice) reached the integration base with the roster/voice-registration tools but never gained presentation rows, so they fell through to the default branch and rendered their wire names. Port their rows from the reference implementation (labels and i18n keys verbatim) and extend the reconciliation allowlist with ROSTER_TOOL_NAMES and VOICE_CLONE_TOOL_NAMES, so a future tool cannot enter the product without a label. * feat(agent): add the material extraction lifecycle (#1212) * feat(storage): add material extraction lifecycle * feat(agent): execute queued material extraction * style: apply prettier formatting * style: satisfy prefer-const in the extraction runner * test: give material fixtures the extraction lifecycle fields The media-tools slice and the extraction lifecycle slice were each green in isolation but never compiled together: the lifecycle made derivedFrom and extraction required on AgentSessionMaterial while the media-tool fixtures predate them. * chore: remove stray task notes * fix(workbench): label the extraction lifecycle tools on the timeline * feat(workbench): add the workspace shell (#1206) * feat(workbench): add the workspace shell * docs(workbench): write the ported comments in English * i18n(workbench): align workspace keys across locales * fix(workbench): adopt the landed data layer and label the extraction tools - replace the sibling-slice seam stubs with the real data-layer modules - drop ambient declarations now shadowed by landed files - port timeline labels for the extraction lifecycle tools from the reference - align the new i18n keys across all locales * ci: retrigger * feat(api): folder routes, stage-meta viewer surfaces, and the material upload contract (#1215) * fix(storage): restore capability-based stage access * fix(api): bind document access to request owner * fix(agent): restore three-state stage access on the tool layer Port probeStageAccess and the three-state StageAccess (owned / foreign / missing / tombstoned) and gate every stageId-bearing stage tool on an owned probe, mirroring the reference per tool: - move_to_folder, rename_stage, read_stage_outline refuse a non-owned stage with the single not-yours message before touching the store. - The course/DSL toolset and the roster toolset are wrapped by withOwnerStageAuthorization: read_stage, patch_stage, grep_stage and every writer refuse a foreign stage with the same message and refusal shape. - Scene preview keeps its own probe and its own refusal text, and is registered beside the course toolset (never double-gated). - The runner injects one probe factory at the three call sites. Tests: the dsl cross-owner test premise (a foreign stage is readable by id) encoded an invented capability-read policy that the reference does not have at the tool layer; it now asserts foreign read/patch/grep are all refused while the owner still reads. Curriculum cross-owner assertions were already the reference's and now pass with the probes in place. * docs: correct per-file test counts in the fidelity report * test: fix type errors in stage-access fidelity test * test: adapt media-tool and gate suites to the owner-scoped store seam * feat(api): add owner-scoped course-folder HTTP routes Port the reference implementation's /api/folders family (list, create, rename, delete with ungroup/remove modes, and folder membership) onto the owner-bound document store, replacing its provider-based auth with the existing withRequestOwnerId / owner-scoped store seams. The storage package's folder store grows the pieces the routes need: DocumentFolder.order (schema column + max+1 assignment + ordering), renameFolder, deleteFolder(mode) with captured member ids, and setStageFolder(stageId, folderId | null) with idempotent un-filing. FolderNameError moves into folder-name-validation.ts (stage-storage re-exports it, keeping import sites intact). Every route gates on the configured agent runtime (plain 404 when off or unconfigured), keeps the reference's machine codes and envelopes, and is covered by gate tests plus a behavior suite. * feat(api): add stage-meta viewer surfaces for the classroom Port the reference implementation's viewer-facing stage state — can-edit / collected / published / generation-complete — on top of the stage-access base (stage_meta + tombstones). stage_meta gains published_at and generation_complete columns plus a stage_bookmarks table; the reference's deployment-specific origin/claimed_at columns are stripped. New gated routes: GET /api/stage-meta/[stageId] (per-viewer facts, 404 for absent/tombstoned, never returns the owner id), GET /api/stages/[id]/status, POST generation-complete / publish / unpublish (owner-only), POST /api/bookmarks. The resolver lives in lib/server/stage-access.ts. Wiring: a fetchStageMeta client with the reference's three-outcome contract, stage-store isOwner/isBookmarked/readOnly fields (upstream single-user defaults, no-op until the sidecar answers) plus setViewerAccess, the classroom apply path computing readOnly = !(isOwner || isBookmarked), the Stage editability gate, and a sidecar probe after each classroom load. A sidecar 'absent' answer keeps the editable default here because the classroom also serves local-only courses; server writes stay owner-enforced. * feat(api): port the reference material upload contract Rewrite POST /api/materials to the reference implementation's upload shape so the workbench uploader (uploadWorkbenchMaterial, which posts no session id and expects a flat 201 view) works unchanged: owner-scoped upload with mime normalization/validation (415), per-class size caps checked on the declared content-length and the streamed body (413), empty body (400), quota (429), sha256 reserve->store->finalize lifecycle with abandon on failure, flat { materialId, originalName, bytes, mime, extraction } 201, and an x-request-id echo. Adds the owner-scoped material library (owner_material table + quota + 24h lazy sweep, bytes in the host's asset registry as the neutral replacement for the reference's object-storage byte path) and the material cap configuration. The session-scoped GET list is left as-is; the reference's owner-material extraction worker is not ported (the branch's session-material extraction lifecycle already covers extraction). Gate tests now cover all 23 persistence routes across the three runtime env states; the materials behavior suite pins the new contract. * feat(media): add an optional local ffmpeg media extractor (#1213) Adds a local ffmpeg/ffprobe pipeline as a second media extraction provider behind the extractor registry, ported faithfully from the reference implementation: duration probing, keyframe-safe chunking, per-chunk ASR with timeout and deadline budgets, and timestamped transcript assembly. - Availability probing feeds the registry's candidate selection: the provider simply is not a candidate when ffmpeg/ffprobe are absent. - With neither ffmpeg nor a cloud provider configured, extraction fails with an actionable message naming both enablement paths. - Media materials route through the same extraction lifecycle and lease fence as documents; no parallel queue. - Tests inject the executable resolver so the missing-ffmpeg path is the default-tested one; the real pipeline test is skip-if-unavailable. - @openmaic/storage 0.13.0 -> 0.14.0 (media routing in the material lifecycle surface). * feat(storage): per-scene monotonic revisions via database triggers (#1214) * feat(storage): per-scene monotonic revisions via database triggers Restore the reference implementation's freshness granularity: a per-scene monotonic revision maintained by database triggers, so every writer (HTTP routes, agent tools, jobs, manual SQL) bumps it without application cooperation. - Companion revision tables + trigger functions in the storage package's idempotent schema bootstrap, with the lock-order invariant, pg_notify wakeup and the suppression switch for batch writers. - ensureDocumentSchema gained a dollar-quote-aware statement splitter. - The freshness and manifest routes serve per-scene revisions. - Mutation-verified: dropping the triggers turns the revision tests red. - @openmaic/storage 0.13.0 -> 0.14.0. * fix: forward the freshness manifest through the owner-bound store * feat(workbench): add the Pro entry points and preserve the mode-transition semantics (#1208) * feat(workbench): add the Pro entry points * feat(workbench): preserve Pro mode transition semantics * fix(workbench): drop ambient declarations shadowed by landed slices * fix(workbench): drop ambient declarations shadowed by the landed shell * feat: port workspace shell sibling modules Port the 16 leaf modules the Pro workspace shell imports but that were only ambient-declared, replacing the compile-time bridge with real implementations adapted from the sibling-slice reference: pure workbench helpers (session title, rail tab, course-chat bootstrap, created-course tabs, course-tabs memory, workspace navigation, pane navigation, pro-edit sizing, existing-course minting, first-message session), the neutral brand context and course-rename server API, the server-action session delete, the home discovery hook, the classroom pane host with its load-policy leaf, the theme toggle and floating-layer owner, plus the floating-layer-owner wiring the dialog/dropdown/tooltip portals stamp. Also add the workbench-shell locale copy for all 12 locales, port the reference tests for the ported modules, and drop types/workbench-sibling-slices.d.ts now that every declaration has a real implementation. * docs: keep ported comments in English and deployment-neutral * docs: announce 1.0.0 and refresh the feature overview (#1216) * docs: announce 1.0.0 and refresh the feature overview * docs: finalize 1.0.0 README after feature merge * fix(agent): control-plane routes answer 404, not 500, without a database The agent control-plane routes gated only on the runtime flag, so an enabled-but-unconfigured deployment (flag on, DATABASE_URL empty) answered 500 from a store that cannot connect. Gate them on the configured check instead, matching the stage/material routes: the whole surface is cleanly absent until both the flag and the database are present. The status probe keeps reporting both bits. * test: mock both runtime gate exports in the control-plane route suites * fix(agent): abort in-flight TTS on cancel and bound each provider request with a timeout (#1217) The generate_tts / scene-tts path checked the runner's AbortSignal between actions but never created the provider HTTP requests with it, so a session cancel left a hung synthesis fetch in flight until a restart repaired the tool result. Thread the signal end-to-end: TTSModelConfig carries an optional signal, generateTTS combines it with a per-request timeout (TTS_REQUEST_TIMEOUT_MS, default 30s, ported from the reference runtime's TTS bounds) via AbortSignal.any, and every provider fetch (openai, azure, glm, qwen incl. voice-clone + audio download, voxcpm, minimax, doubao, elevenlabs, lemonade) is created with that signal. A timeout now fails the tool call with TTSRequestTimeoutError (a clear retryable error) instead of wedging the session; a caller cancel propagates as the interruption so the runner settles the session as cancelled without a restart. Tests: hung-provider simulation rejects at the timeout with the retryable error; abort mid-flight aborts the captured request signal and surfaces the interrupted shape; removing the signal wiring makes the abort tests fail (red), restoring them turns green. * fix(workbench): PG-mode home listing via owner stages; keep the interrupted terminal course card (#1218) Finding 1: with server persistence on, listStages resolved to the generic GET /api/persistence/documents listing, which the capability model deliberately answers 403 FORBIDDEN_DOCUMENTS for (reads by id, listings owner-only). The home/workspace library now lists through the owner-scoped GET /api/stages surface (same anonymous-owner cookie the workbench uses) when server persistence is enabled; the server-side 403 is untouched. Finding 2: a run interrupted (session_interrupted) and repaired (session_resumed) that ends cancelled before agent_end stranded its pending classroom sightings, so the timeline's terminal card lost the course the answer produced. session_end (cancelled) now flushes the pending sightings into the same course card set agent_end paints, before the stopped caption. * chore(workbench): remove the bookmark concept and the saved-courses drawer (#1219) * chore(classroom): remove the bookmark ('collected') concept entirely The stage-meta viewer port introduced a bookmark surface (stage_bookmarks table, POST /api/bookmarks, the isBookmarked sidecar field, and a readOnly rule that let a saved course stay editable). The product has no such concept, so remove it as a closure: - delete the /api/bookmarks route and the stage_bookmarks table plus its query helpers from the persistence bootstrap - drop isBookmarked from GET /api/stage-meta/[stageId] - simplify the classroom read-only rule to readOnly = !isOwner across the sidecar client, ownership signal, classroom load, stage store and the classroom page - keep publish/unpublish, generation-complete, isOwner and isPublic exactly as they were - update the gate and stage-meta route suites and the README mentions The workspace rail's Bookmark glyphs and comments describe the upstream saved-courses (favorites) section, which is driven by isOwner and renders no collect affordance; they are kept as unrelated homonyms. * chore(workbench): remove the saved-courses drawer UI The first pass removed the bookmark data model but kept the rail's "Saved courses" drawer, judging it a separate surface driven by `isOwner === false`. The home/workspace listing is owner-scoped, so that flag can never occur: `allSaved` is permanently empty and the drawer (plus the collapsed-rail Bookmark mini-button) is a dead affordance. Remove it: the SavedDrawer component and its mount, the savedOpen / savedSection state, the allSaved / matchedSaved derivations, the 'saved' variant of the course-list renderers, the mini Bookmark glyph, the drawer-only CSS, and the drawer's i18n keys from all 12 locales. The courses tab is now exactly one folders tree. The authored/favorites split in workspace-tree.ts goes with it; the tree module no longer reads `isOwner`. The discovery course type keeps the field — the shell still reads it for read-only gating. Upstream has no collect concept; the drawer could only ever render empty here. The reference implementation HAS this drawer (its favorites come from its account system), so this removal is a deliberate upstream product decision, not a fidelity bug. * fix(workbench): restore the attach entry, add the rail settings entry, pin all three entry points (#1221) * fix(workbench): restore the composer attach entry by gating it on the live runtime The AttachButton's rollout probe read a `materialsEnabled` field that this branch's /api/agent/runtime never answers (the materials routes gate on the runtime itself, like the stages), so the gate could never pass and the attach button never rendered — the Pro launch and chat composers showed only the @-mention and enhance glyphs. Substitute the field with the runtime's `enabled` value, which IS the upload action's precondition: POST /api/materials answers 404 whenever it is false, so the render condition now equals the action precondition (no dead button). The button's label (`proMode.attach`) is a user-visible string that becomes visible again; port the reference implementation's own translations verbatim into the 11 locales that still carried the Chinese copy. * feat(workbench): add the settings entry to the rail's bottom-left cluster The reference's rail foot carries a cluster of utilities (its saved-courses drawer, the language switcher, the display toggle). This branch removed the drawer — it could only ever render empty here — and the product decision is to fill that freed spot with the settings entry. Add a settings trigger to the foot cluster (expanded rail, beside the language and display toggles, and on the collapsed strip) and mount the model/provider SettingsDialog in the rail, wired to the trigger. It is the same dialog the classic home opens from its header pill; the workspace had no settings entry of its own, so nothing is duplicated within a surface. * test(workbench): pin the restored upload, attach, and settings entry points Covers the three restored entry points: - the courses-tab upload control: rendered beside the course name filter, wired to the discovery hook's ZIP import trigger, disabled while an import runs, and gated by the same condition as its action (the courses tab); - the composer attach control: an actual render of AttachButton under both probe answers (visible when the runtime says the upload path is live, hidden otherwise), its mounts in the launch and chat composers, the branch's runtime-field substitution in the probe, and the reference's own `proMode.attach` copy in all 12 locales; - the settings entry: the trigger in the rail's foot cluster (expanded and collapsed), beside the language and display toggles, opening the SettingsDialog the rail mounts. * chore(config): the Pro workbench flag implies the MAIC Editor gate (#1223) A workbench build without the editor toggle has no way to edit a course: enabling NEXT_PUBLIC_PRO_WORKBENCH_ENABLED while forgetting NEXT_PUBLIC_MAIC_EDITOR_ENABLED produced exactly that split-brain bundle. The workbench IS Pro mode, so its flag now implies the editor gate; the standalone flag remains for deployments that want the classroom editor without the workbench. Documents both flags in .env.example. * fix(agent): wake SSE tails and the runner on durable deltas (streaming fidelity) (#1222) The Pro workbench chat did not stream: the session/owner SSE routes polled the durable event log on a 5s/30s clock with no wakeup, so message_update deltas (written at 150ms cadence) reached the browser in poll-sized blocks and the thinking strip only mounted after the whole reasoning text had accumulated. Port the reference's LISTEN/NOTIFY delta path: - storage: add in-transaction wake hooks (onSessionEventAppended, onOwnerEventAppended, onCancelRequested) so a host queues pg_notify in the same transaction as the durable append; align readEventsAfterForReplay to rank the bounded page so the first delta after the cursor is always kept (the live tail can never starve). Bump @openmaic/storage to 0.18.0. - app: port the process-wide event-notify bus (dedicated LISTEN client, self-check probe, reconnect backoff; notify through the storage transaction surface), wire the store hooks, subscribe both SSE routes before the initial read with the reference's initializing gate, and give the runner one {kind:'session'} subscription whose wake runs the cancel check and the message drain. Polls stay as the lossy-NOTIFY backstop. - lifecycle: start/stop the bus from instrumentation. Tests: storage hook + compaction contract; route wakeup latency; runner wakeup wiring with a fake agent; bus unit tests; PG contracts proving a real append wakes the routes and a live SSE route forwards a message_update on the wakeup, and that a rolled-back append never wakes. Also fix the pre-existing park-attempt-budget PG test TRUNCATE (missing CASCADE against newer FK tables). * fix(storage): asset writes self-deadlocked against pooled PostgreSQL (#1225) * fix(storage): refuse the non-transactional byte-write deadlock configuration A byte store whose plain write() runs on its own pooled connection cannot be invoked from inside a registry write transaction: after the transaction has claimed the blob-row lock, that write blocks on the lock the transaction just took while the transaction waits on the write - a self-deadlock PostgreSQL cannot detect (one side is idle in transaction). There is no lock-safe ordering for such a writer: bytes must be written after the row claim (writing before it lets the collector delete the bytes while the upsert waits), and any second-connection write after the claim is the deadlock. The configuration is therefore detected and refused: - AssetByteStore gains writesOutsideRegistryDatabase?: true, declaring that the layer's plain byte operations cannot contend for the registry's row locks. - PgAssetStore refuses put()/replace() up front (and defends coordinatedWrite) when the byte store has no writeWith and does not declare the flag, throwing a clear configuration error before any row is claimed. - The collector mirrors the guard on its delete path (deleteWith or a declared out-of-registry layer, else a configuration error). - The object store declares the flag (its out-of-transaction write remains legitimate); the in-registry PostgreSQL byte column provides writeWith / deleteWith instead. - Write transactions (put/replace/remove) set SET LOCAL lock_timeout = 30s so any future lock-contention variant fails loudly instead of hanging. Bumps @openmaic/storage to 0.18.0. * fix(persistence): forward the transactional byte methods through the lazy asset byte-store wrapper The no-bucket case of lazyAssetByteStore returned a bare { write, read, delete } and dropped writeWith/readWith even though the underlying PgAssetByteStore has them. The registry's hasTransactionalWriter duck check then failed and put() fell back to the byte store's own pooled connection, which blocks forever on the blob-row lock the registry transaction just took when the bytes live in the same PostgreSQL - the production self-deadlock. The no-bucket layer is statically PgAssetByteStore, so its transaction-pinned methods are forwarded eagerly (typed against the real signatures via PgForwardedByteStore). The bucket case keeps its lazy-probing semantics: no transactional writer exists there, the signed-URL method stays absent or lazy exactly as documented, and the wrapper now declares writesOutsideRegistryDatabase so the registry may run the plain write inside its transaction. New tests pin the wrapper's transactional capability red-to-green and assert put()/resolve() route byte traffic through the transaction-pinned queryable. * fix(home): cap the generate-prep ingest drain at 3s so Generate never waits the full server budget The classic home flow's Generate click drained in-flight ingests for the full 15s server budget. Cap the wait at GENERATE_DRAIN_CAP_MS (3000ms, documented as a UX bound) and reuse the existing timeout fallback: sources that miss the cap proceed on the legacy byte path and each late-resolving id is released. * chore(storage): bump to 0.19.0 over the concurrently landed 0.18.0 * fix(agent): bound every tool call with a timeout; never resurrect a cancelled session (#1226) * fix(agent): bound every tool call with a global timeout and settle it on cancel A tool await that neither resolves nor rejects wedges the session forever: the lease keeps heartbeating and the driver never reaches its next cancel checkpoint. Race every tool execution (in buildAgent) against a hard budget (OPENMAIC_AGENT_TOOL_TIMEOUT_MS, default 10 min, per-tool overrides for known long runners) and against the caller's AbortSignal, so even a signal-ignoring await cannot keep a cancelled session running. On timeout the call rejects with AgentToolTimeoutError; the agent loop turns the rejection into a structured error tool-result the agent can retry or proceed from, and the abort signal is delivered to the tool's in-flight work through a derived controller. Zombie-tool updates after settlement are dropped. * fix(storage): never re-lease a cancel-requested session; settle it as cancelled on claim The claim scan treated a session with cancel_requested_at set as a normal claim candidate: after a restart it re-leased the same session for attempt N+1 and resumed generating despite the pending cancel. claimNextSession now settles such candidates as cancelled under the claim lock (status cancelled, attempt reset, lease and cancel request cleared, terminal session_end event and owner projection) instead of leasing them, then keeps scanning. Bump @openmaic/storage to 0.18.0. * docs: takeaway-style 1.0.0 announcement with bilingual guide links The 1.0.0 head is now a short takeaway block — badge links to the official user guides (English and Chinese), five one-line highlights, and pointers into Features and the workbench setup section — instead of six dense paragraphs. The detailed provider-neutrality and freshness notes move into the Features workbench section, phrased database- neutrally (the announcement no longer names a specific database). Release date corrected to August 27. * fix(workbench): restore editor chrome, mode transition, streaming, materials, mentions, folders (#1229) * fix(workbench): wire workspace folder routes * fix(editor): restore reference workbench chrome * fix(workbench): persist composer materials and course refs * fix(workbench): preserve live reasoning frames * fix(persistence): back off failed streaming saves * chore(workbench): retire stale slice seams * test(editor): cover element pin layer * chore(storage): bump to 0.21.0 for the user-message ref/material fields * chore(editor): translate ported code comments to English * fix(agent): fence durable tool writes and consume cancel requests atomically (#1230) * fix(agent): enforce provider force-off in agent tools and scrub vendor identity from tool results (#1231) * fix(materials): serialize per-owner quota reservations and make crashed uploads reclaimable (#1232) * fix(editor): resolve dock-bar i18n keys, remove dock height drag, wire element referencing (#1233) * fix(workbench): send the opening session message exactly once with refs intact (#1234) * feat(editor): port timeline TTS preview single-flight and voice-all state latching (#1235) * fix(media): restore the reference classic media chain (#1236) * fix(import): adapt imported PPTX canvas size so decks render without overflow (#1237) * fix(editor): complete element referencing — renderer DOM contract and GenUI picking aligned with the reference (#1238) * test(providers): reconcile the provider-config vendor-token debt count after the main merge The integration line's AK/SK fallback for the managed document provider adds occurrences that main's allowlist snapshot predates. Same mixed-composition debt category the group already documents; no new vendor behavior. * test(providers): reconcile vendor-token debt counts with the integration line The main-merge brought main's neutrality-guard snapshot next to integration features it predates (media-extractor fallback chain, local voice-profile deletion semantics, the enabled-TTS helper). Same debt categories the guard already documents; counts updated to the guard's own tally and two grouped entries added. No new vendor behavior. * fix(agent): carry reasoning through the completions dialect so the thinking strip renders (#1239) * feat(skills): add Feynman and spiral curriculum methods (#1240) * feat(agent): port missing reference tools and skills (parity audit) (#1241) * feat(media): retire asset-registry wiring; media and materials follow the reference byte model (#1242) * fix(classroom): center adapted canvases in the stage and send back navigation home during generation (#1243) * feat(settings): skill management with real list, download, delete, and upload (#1244) * feat(settings): skill management section with real list, detail, and zip download * feat(skills): owner skill delete and upload across storage, API, and settings * fixup! feat(settings): skill management section with real list, detail, and zip download chore: neutralize a reference note in the settings header comment * fix(media): persist origin-independent classroom-media references from the agent runtime (#1245) * feat(editor): float the insert toolbar in the outer frame with collapse (#1246) The insert strip was bounded to the slide card, so it could only ever sit on top of slide content: the card's overflow clipped it and it could not be parked in the padding beside the slide. Move it into the studio frame the element picker's panel already roams (CanvasOverlayPortal + the frame selector), so both canvas overlays share one bounding container and their handles behave the same. While picking, the strip rises over the picker and goes inert, which is the z-order CANVAS_OVERLAY_Z already documents. Add a fold beside the grip: the chevron collapses the strip to that grip row and back, with the buttons unmounted rather than hidden. The fold is session-local state owned by EditShell, next to the drag offset, so a surface swap keeps it; nothing is persisted. Expanding a strip parked at the bottom edge re-clamps through the same bounds rule the keyboard move uses. Co-authored-by: Claude Fable 5 <noreply@anthropic.com> * fix(workbench): align the chat timeline's left edge with the composer (#1247) * fix(agent): fence session claims while an ask_user question is outstanding (#1248) * fix(agent): settle-time rescue tracks real delivery instead of a count offset (#1249) * fix(persistence): migrate owner_material to oss_key and drop legacy asset_id (#1250) * docs(readme): surface the 1.0.0 user guide badges at the top (#1253) * fix(workbench): show newly created folders in the sidebar without reload (#1254) * docs(readme): add the release version prefix and drop the opt-in framing * fix(workbench): single-source the chat gutter so timeline and composer share a left edge (#1255) The transcript and the composer each established their own column: their own `px-*` gutter and their own `mx-auto w-full max-w-*` centering wrapper. Equal padding values were never enough, because the two columns are centered inside different containing blocks — the transcript's is a scroll container, whose content box is narrower than the composer footer's by the scrollbar's width: transcript text left = pad + (pane - 2*pad - scrollbar - measure) / 2 composer box left = pad + (pane - 2*pad - measure) / 2 The padding cancels out of the difference and what remains is `-scrollbar/2` at every padding value, so the transcript sat half a scrollbar to the left of the composer and tuning the two paddings against each other could not move it. The column is now established once, by the nearest common ancestor of both (`chatColumn`), and the scroll viewport and the composer footer are siblings inside it that add no horizontal inset of their own. The cap carries the gutter on top of the 760px reading measure, so the text column keeps its width. The handed-over question row drops the padding that indented it past the agent's prose; framed rows keep their own inner padding, which is what a card's border sitting on the column edge means. Co-authored-by: Claude Fable 5 <noreply@anthropic.com> * fix(workbench): lock pane-embedded classroom to edit mode (#1256) The workspace right pane painted the full learning chrome — speed control, play button, learner avatars, mic bar — for a course the agent had just created, then flipped to edit once the first scene landed. resolveStageChromeMode treated playback as the DEFAULT branch for a hosted classroom, so every shortfall fell into it: a course whose tab opens at stage_link time has no scenes yet, so currentSceneId is null and isHostedSceneEditable is false. A folded pane parked the playback root behind the fold and cross-faded it out over the pane on unfold, and a failed editor chunk dropped into playback permanently. Lock it at the pane instead of defaulting per entry path: - WorkbenchPanelProvider — the single element that mounts a classroom into the workspace — publishes editPinned (visible && !playback). Every entry path passes through it, so none of them decides. - The hosted resolution can no longer degrade to playback. Start Learning (workbenchLearning, new input, split out from pane visibility) is the one door; everything else resolves between the neutral loading shell and edit. - Stage's chrome dispatch is exhaustive on chromeMode, so the playback root is no longer the else-branch of a condition about the current scene. No flicker: chromeMode is resolved during render, and preloadEditor now answers synchronously (isEditorPreloaded) so a remount with the chunk already registered paints edit on the first frame. A failed import is no longer cached forever, so the lock cannot strand the pane. Standalone classrooms keep their stored mode unchanged. --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
389 lines
18 KiB
TypeScript
389 lines
18 KiB
TypeScript
'use client';
|
|
|
|
import { useCallback, useEffect, useRef, useState } from 'react';
|
|
import { useRouter } from 'next/navigation';
|
|
import { AnimatePresence, motion } from 'motion/react';
|
|
import { useStageStore } from '@/lib/store';
|
|
import {
|
|
isCurrentSceneEditable,
|
|
isHostedSceneEditable,
|
|
resolveStageChromeMode,
|
|
} from '@/lib/edit/stage-mode';
|
|
import { isMaicEditorEnabled, isProWorkbenchEnabled } from '@/lib/config/feature-flags';
|
|
import { EditChromeRoot } from '@/components/edit/EditChromeRoot';
|
|
import {
|
|
PlaybackChromeRoot,
|
|
type PlaybackChromeRootHandle,
|
|
} from '@/components/edit/PlaybackChromeRoot';
|
|
import { InteractiveIframeHost } from '@/components/scene-renderers/InteractiveIframeHost';
|
|
import { CHROME_EASE } from '@/lib/edit/transitions';
|
|
import { enterEditMode } from '@/lib/edit/enter-edit-mode';
|
|
import { isEditorPreloaded, preloadEditor } from '@/lib/edit/preload-editor';
|
|
import { WorkbenchReturnControl } from '@/components/workbench/WorkbenchReturnControl';
|
|
import { resolveClassroomBackControl } from '@/lib/workbench/classroom-back-control';
|
|
import { resolveClassroomHeaderControls } from '@/lib/workbench/classroom-header-controls';
|
|
import { useWorkbenchStore } from '@/lib/workbench/session-store';
|
|
import { useWorkbenchPanelState } from '@/lib/workbench/panel-context';
|
|
import { workspaceHref } from '@/lib/workbench/workspace-panes';
|
|
import { exitProPlaybackToStandalone } from '@/lib/workbench/pro-playback-exit';
|
|
|
|
/**
|
|
* Stage — top-level classroom container. Standalone classrooms dispatch
|
|
* between the two chrome roots based on `useStageStore.mode`:
|
|
*
|
|
* mode === 'edit' → EditChromeRoot
|
|
* mode === 'playback' / 'autonomous' → PlaybackChromeRoot
|
|
*
|
|
* When this classroom is HOSTED — mounted inside the Pro workspace's classroom
|
|
* pane rather than filling a route — the dispatch below is not replaced, it is
|
|
* merely stripped of the chrome that the host already provides: no header, no
|
|
* global controls or Pro switch (the pane IS Pro-locked). Hosted chrome is
|
|
* EDIT-LOCKED: the pane declares the lock once (`WorkbenchPanelState.editPinned`)
|
|
* and this component reads it back, so no entry path — a course the agent just
|
|
* created, a restored tab, a tab switch, a reload — gets to decide otherwise.
|
|
* The learning chrome is reachable only through Start Learning, which clears
|
|
* the lock at the pane. Everything else resolves synchronously between the
|
|
* neutral loading shell and edit, so a hosted first paint can never be
|
|
* playback. Deliberately not a third
|
|
* `StageMode` — `StageMode` lives in the
|
|
* published `@openmaic/dsl` and is persisted with the stage, whereas "this is
|
|
* rendered inside the workspace right now" is view state that must not outlive
|
|
* the tab.
|
|
*
|
|
* The host itself owns the three-pane layout, the conversation, the session
|
|
* stream and the course sync. Stage's only responsibilities are: mode
|
|
* dispatch and Pro Switch toggle wiring (calls into
|
|
* PlaybackChromeRoot.teardown via ref before flipping mode).
|
|
*/
|
|
export function Stage({
|
|
classroomId,
|
|
onRetryOutline,
|
|
}: {
|
|
classroomId?: string;
|
|
onRetryOutline?: (outlineId: string) => Promise<void>;
|
|
}) {
|
|
const { mode, setMode, scenes, currentSceneId, generatingOutlines, stage } = useStageStore();
|
|
const router = useRouter();
|
|
const enteringWorkbench = useRef(false);
|
|
const proWorkbenchFlag = isProWorkbenchEnabled();
|
|
const editorEnabled = isMaicEditorEnabled();
|
|
const [proRuntime, setProRuntime] = useState<'pending' | 'on' | 'off'>(
|
|
proWorkbenchFlag ? 'pending' : 'off',
|
|
);
|
|
useEffect(() => {
|
|
if (!proWorkbenchFlag) return;
|
|
let cancelled = false;
|
|
fetch('/api/agent/runtime')
|
|
.then((res) => (res.ok ? res.json() : null))
|
|
.then((body) => {
|
|
if (!cancelled) setProRuntime(body?.enabled === true ? 'on' : 'off');
|
|
})
|
|
.catch(() => {
|
|
if (!cancelled) setProRuntime('off');
|
|
});
|
|
return () => {
|
|
cancelled = true;
|
|
};
|
|
}, [proWorkbenchFlag]);
|
|
const proWorkbenchEntry = proWorkbenchFlag && proRuntime === 'on';
|
|
const currentScene = useStageStore((s) => s.getCurrentScene());
|
|
// The reference implementation makes editing owner-only. `isOwner` is true for the stage creator and
|
|
// defaults to true with browser storage (single-user IndexedDB), so this gate is
|
|
// a no-op upstream but hides Pro mode from visitors / bookmarked viewers in
|
|
// server-backed mode — their saves would not pass the owner check anyway.
|
|
const isOwner = useStageStore((s) => s.isOwner);
|
|
const readOnly = useStageStore((s) => s.readOnly);
|
|
const canEditOwnedStage = isOwner && !readOnly;
|
|
|
|
// Hosted by the Pro workspace's classroom pane. Ambient rather than a prop
|
|
// because `Stage` is built by `ClassroomSurface`, which is mounted by both
|
|
// the route and the pane and has no business knowing which.
|
|
//
|
|
// The provider is the host boundary. Session state intentionally does not
|
|
// participate: it outlives route transitions and previously made an ordinary
|
|
// classroom inherit the workspace's edit chrome after SPA navigation.
|
|
const workbenchPanel = useWorkbenchPanelState();
|
|
const inWorkspacePane = workbenchPanel.hosted;
|
|
const hosted = proWorkbenchFlag && inWorkspacePane;
|
|
// Pane view state comes from the pane boundary, not from the attached-chat
|
|
// fold. `attach()` intentionally rebuilds that fold when Chat changes; using
|
|
// its panelOpen bit here made the keyed edit/playback roots swap for one
|
|
// frame and produced a full classroom flash.
|
|
const workbenchPlayback = workbenchPanel.playback;
|
|
// The classroom is showing exactly when it is hosted and the pane says it is
|
|
// edit-locked. The lock is computed once, at the provider the workspace
|
|
// mounts the classroom through (`WorkbenchPanelProvider`), so this is the
|
|
// pane's answer being read back rather than a second derivation that could
|
|
// disagree with it.
|
|
const workbenchShowingClassroom = hosted && workbenchPanel.editPinned;
|
|
|
|
// Single decision for the classroom chrome's top-left back affordance:
|
|
// plain classroom → home arrow; full-screen playback → "Back to workspace";
|
|
// every other hosted form (the workspace pane) → hidden (the conversation
|
|
// and the navigation tree are already beside the classroom, and a home
|
|
// arrow would exit the workspace). See lib/workbench/classroom-back-control.ts.
|
|
const classroomBackControl = resolveClassroomBackControl(hosted, workbenchPlayback);
|
|
const classroomHeaderControls = resolveClassroomHeaderControls(hosted, workbenchPlayback);
|
|
|
|
// Predicate for "can the user enter Pro mode for the current scene?".
|
|
// Single source of truth feeds the Header's Pro Switch state and the
|
|
// auto-exit effect below; keeping them in lock-step prevents an
|
|
// edit-mode entry that would immediately auto-exit.
|
|
const isEditable =
|
|
canEditOwnedStage &&
|
|
isCurrentSceneEditable({
|
|
currentSceneId,
|
|
sceneCount: scenes.length,
|
|
generatingOutlineCount: generatingOutlines.length,
|
|
hasCurrentScene: !!currentScene,
|
|
});
|
|
|
|
// Hosted generation is page-granular: once the current page materialises,
|
|
// the human may edit it while the agent writes later scene IDs. Keep the
|
|
// stricter whole-deck generation gate above for standalone Pro mode.
|
|
const currentStageMatchesHost = !classroomId || stage?.id === classroomId;
|
|
const hostedSceneEditable = isHostedSceneEditable({
|
|
editorEnabled,
|
|
isOwner: canEditOwnedStage,
|
|
stageMatchesHost: currentStageMatchesHost,
|
|
currentSceneId,
|
|
sceneCount: scenes.length,
|
|
generatingOutlineCount: generatingOutlines.length,
|
|
hasCurrentScene: !!currentScene,
|
|
});
|
|
const chromeEditable = hosted ? hostedSceneEditable : isEditable;
|
|
// Seeded from the module-level registry rather than always starting at
|
|
// `idle`: once the editor chunk has been imported in this tab, a remount
|
|
// (course switch, reopened tab) must resolve to the edit chrome during the
|
|
// FIRST render. Starting at `idle` would spend a paint on the neutral
|
|
// loading shell waiting for an import that already finished.
|
|
const [editorPreloadState, setEditorPreloadState] = useState<
|
|
'idle' | 'loading' | 'ready' | 'failed'
|
|
>(() => (isEditorPreloaded() ? 'ready' : 'idle'));
|
|
|
|
useEffect(() => {
|
|
if (!hosted || !workbenchShowingClassroom || !hostedSceneEditable) return;
|
|
// Already registered — do not knock the state back to `loading`, which
|
|
// would blank an edit chrome that is on screen and correct.
|
|
if (isEditorPreloaded()) {
|
|
setEditorPreloadState('ready');
|
|
return;
|
|
}
|
|
let cancelled = false;
|
|
setEditorPreloadState('loading');
|
|
preloadEditor()
|
|
.then(() => {
|
|
if (!cancelled) setEditorPreloadState('ready');
|
|
})
|
|
.catch((error) => {
|
|
console.error('[Stage] hosted editor preload failed', error);
|
|
if (!cancelled) setEditorPreloadState('failed');
|
|
});
|
|
return () => {
|
|
cancelled = true;
|
|
};
|
|
}, [hosted, workbenchShowingClassroom, hostedSceneEditable]);
|
|
|
|
// The workspace owns the edit/playback intent: the pane's edit lock is the
|
|
// edit intent, while Start Learning (`workbenchPlayback`) is the one signal
|
|
// that releases it. Resolve that intent in render rather than mutating the
|
|
// transient stage-store mode in an effect — otherwise every hosted course
|
|
// paints PlaybackChromeRoot once before the effect can run, and a course
|
|
// switch can inherit stale mode.
|
|
const chromeMode = resolveStageChromeMode({
|
|
storedMode: mode,
|
|
hosted,
|
|
workbenchShowingClassroom,
|
|
workbenchLearning: hosted && workbenchPlayback,
|
|
isEditable: chromeEditable,
|
|
hasCurrentScene: !!currentScene,
|
|
stageMatchesHost: currentStageMatchesHost,
|
|
editorReady: editorPreloadState === 'ready',
|
|
editorLoadFailed: editorPreloadState === 'failed',
|
|
});
|
|
|
|
const playbackRef = useRef<PlaybackChromeRootHandle>(null);
|
|
|
|
// Pro Switch handler. Edit→playback is a plain flip (PlaybackChromeRoot
|
|
// will mount fresh; its engine effect re-inits). Playback→edit must
|
|
// await SSE / engine / TTS teardown so PlaybackChromeRoot is quiescent
|
|
// before it unmounts.
|
|
const handleToggleEditMode = useCallback(async () => {
|
|
if (mode === 'edit') {
|
|
setMode('playback');
|
|
return;
|
|
}
|
|
// Load the editor chunk (fonts + slide surface) BEFORE flipping mode,
|
|
// so the edit chrome animates in with its content already present and
|
|
// the slide surface registered — no mid-animation pop-in / NOOP flash.
|
|
// Runs concurrently with teardown; the import is promise-cached so it's
|
|
// a no-op on subsequent toggles.
|
|
await enterEditMode({
|
|
teardown: () => playbackRef.current?.teardown(),
|
|
preload: preloadEditor,
|
|
activate: () => setMode('edit'),
|
|
// Stay in playback so the failure surfaces rather than half-entering
|
|
// edit mode.
|
|
onError: (error) => console.error('[Stage] Pro mode entry failed during teardown', error),
|
|
});
|
|
}, [mode, setMode]);
|
|
|
|
// Auto-exit edit mode when the current scene becomes uneditable
|
|
// (pending generation, no scenes, currently generating).
|
|
useEffect(() => {
|
|
if (mode === 'edit' && !isEditable) {
|
|
setMode('playback');
|
|
}
|
|
}, [mode, isEditable, setMode]);
|
|
|
|
// Non-owners and transport-fenced owners get no Pro toggle at all: without a
|
|
// handler the Header/CommandBar omit the whole switch. Editable owners keep
|
|
// it while a scene is still generating (rendered disabled), matching upstream.
|
|
const toggleHandler = editorEnabled && canEditOwnedStage ? handleToggleEditMode : undefined;
|
|
const setPanelOpen = useWorkbenchStore((s) => s.setPanelOpen);
|
|
|
|
/**
|
|
* In the reference implementation, Pro Mode on a regular classroom is the workspace, not the old
|
|
* right-rail editor. The conversation on the left is where further instructions
|
|
* go; the classroom stays on the right.
|
|
*
|
|
* It opens the course and NOTHING else. Minting a conversation here created one
|
|
* empty session per Pro-mode entry — leave and come back three times, three rows
|
|
* in the rail — and named each one after the classroom, which said the two were
|
|
* one object. The workspace decides what the middle column holds (the user's most
|
|
* recent conversation, or an empty composer whose first message creates one), so
|
|
* this is now a plain navigation with no request behind it.
|
|
*/
|
|
const handleEnterWorkbench = useCallback(() => {
|
|
if (!stage?.id || enteringWorkbench.current) return;
|
|
enteringWorkbench.current = true;
|
|
try {
|
|
setPanelOpen(true, true);
|
|
router.replace(workspaceHref({ sessionId: null, courseId: stage.id }));
|
|
} finally {
|
|
enteringWorkbench.current = false;
|
|
}
|
|
}, [router, setPanelOpen, stage?.id]);
|
|
|
|
const handleExitWorkbench = useCallback(async () => {
|
|
if (!stage?.id) return;
|
|
await exitProPlaybackToStandalone({
|
|
stageId: stage.id,
|
|
teardown: () => playbackRef.current?.teardown(),
|
|
// Hosted playback is view state and may be masking a stale standalone
|
|
// `edit` mode. Commit playback before leaving the workspace so the
|
|
// ordinary classroom stays on the exact learning surface the user saw.
|
|
setMode,
|
|
replace: (href) => router.replace(href),
|
|
onTeardownError: (error) => console.error('[Stage] workbench exit teardown failed', error),
|
|
});
|
|
}, [router, setMode, stage?.id]);
|
|
|
|
// The embedded pane is already Pro-locked, so it has no switch. Full-screen
|
|
// learning exposes an active switch whose off transition exits the workspace
|
|
// and returns to the ordinary classroom route.
|
|
const chromeToggleHandler = hosted
|
|
? workbenchPlayback
|
|
? handleExitWorkbench
|
|
: undefined
|
|
: !isOwner || proRuntime === 'pending'
|
|
? undefined
|
|
: proWorkbenchEntry
|
|
? handleEnterWorkbench
|
|
: toggleHandler;
|
|
|
|
// Mode swap choreography — a clean opacity cross-fade. Both roots layer
|
|
// via `absolute inset-0` so they coexist for the ~280ms window without
|
|
// one popping out before the other arrives. The outgoing root keeps
|
|
// rendering its canvas during exit so `canvasStore` (the shared scale
|
|
// writer) doesn't briefly read zero.
|
|
//
|
|
// Deliberately NO transform (translateY) on these layers: the edit
|
|
// chrome hosts the Pro Switch / settings pill, which morph across the
|
|
// swap via `layoutId`. A transform on this ancestor distorts motion's
|
|
// layout measurement (the pill visibly drifts) and the blurred chrome
|
|
// would repaint its backdrop-filter every frame while translating. A
|
|
// pure fade keeps layout static so the shared elements land precisely.
|
|
//
|
|
// The classroom itself is built once and mounted either full-bleed (the
|
|
// route) or inside the workspace's classroom pane: there is exactly one of
|
|
// these expressions in the app, so "the pane IS the classroom" is
|
|
// structural rather than aspirational.
|
|
const classroomChrome = (
|
|
<AnimatePresence initial={false}>
|
|
{/* The dispatch is exhaustive on `chromeMode` on purpose. The learning
|
|
chrome is reached ONLY by an explicit playback resolution, so it can
|
|
no longer be inherited as the else-branch of a condition about
|
|
something else — a resolved `edit` with no scene to hand shows the
|
|
neutral shell and waits, exactly as `loading` does. */}
|
|
{chromeMode === 'edit' && currentScene ? (
|
|
<motion.div
|
|
key="edit"
|
|
initial={{ opacity: 0 }}
|
|
animate={{ opacity: 1 }}
|
|
exit={{ opacity: 0 }}
|
|
transition={{ duration: 0.28, ease: CHROME_EASE }}
|
|
className="absolute inset-0 flex"
|
|
>
|
|
<EditChromeRoot
|
|
scene={currentScene}
|
|
isEditable={chromeEditable}
|
|
onToggleEditMode={chromeToggleHandler}
|
|
/>
|
|
</motion.div>
|
|
) : chromeMode === 'playback' || chromeMode === 'autonomous' ? (
|
|
<motion.div
|
|
key="playback"
|
|
initial={{ opacity: 0 }}
|
|
animate={{ opacity: 1 }}
|
|
exit={{ opacity: 0 }}
|
|
transition={{ duration: 0.28, ease: CHROME_EASE }}
|
|
className="absolute inset-0 flex"
|
|
>
|
|
<PlaybackChromeRoot
|
|
ref={playbackRef}
|
|
onRetryOutline={onRetryOutline}
|
|
canEnterProMode={workbenchPlayback || isEditable}
|
|
onEnterProMode={chromeToggleHandler}
|
|
proModeActive={hosted && workbenchPlayback}
|
|
headerBackControl={
|
|
classroomBackControl === 'workbench-return' ? <WorkbenchReturnControl /> : undefined
|
|
}
|
|
hideHeaderBackControl={classroomBackControl === 'hidden'}
|
|
hideHeader={!classroomHeaderControls.showHeader}
|
|
hideHeaderGlobalControls={!classroomHeaderControls.showGlobalControls}
|
|
hideHeaderCourseActions={!classroomHeaderControls.showCourseActions}
|
|
/>
|
|
</motion.div>
|
|
) : (
|
|
<div
|
|
key="loading"
|
|
data-testid="stage-editor-loading"
|
|
className="absolute inset-0 flex bg-background"
|
|
aria-busy="true"
|
|
/>
|
|
)}
|
|
</AnimatePresence>
|
|
);
|
|
|
|
return (
|
|
<div className="relative flex flex-1 overflow-hidden">
|
|
{/* The edit-mode guard has been removed (#1961 decision change 2026-08-23): a
|
|
new agent version directly replaces the canvas; the user's typed data is
|
|
protected by the write-path veto/retry channel (see lib/store/stage.ts),
|
|
and there is no longer an "agent modified this page" banner or a manual
|
|
reload entry. */}
|
|
{classroomChrome}
|
|
{/* Full-screen playback steps the workspace aside; the return to the
|
|
conversation lives in the classroom header's left slot (passed as
|
|
`headerBackControl` above), so it never floats over the courseware.
|
|
The fold and pane state live in the store, so the return restores
|
|
them exactly. */}
|
|
{/* Keep-alive host for interactive scene iframes (#619). Lives here, above
|
|
the mode-swap subtree, so its iframes survive Pro mode toggles and
|
|
scene switches instead of reloading on every remount. */}
|
|
<InteractiveIframeHost />
|
|
</div>
|
|
);
|
|
}
|