improvement(academy): set 2, pages for the new video wave (workflow embeds, 2K players, full sequencing)#5382
Conversation
…lls pages, sequenced in meta.json — pages for the four approved videos, blob src pattern, chapters offset by each recorded intro
…(mothership/agents/logs-debugging/deployment), not other academy videos
…pts pedagogy), sequenced after tables/intro
…-bases/connectors, tables/operations, files/object) + retimed chapters on the redone tables/files intros (recomposed/working-day cuts) + full meta.json sequencing — Chat · Agents(5) · Tables(3) · Files(2) · KB(2)
…eo-workflows.ts registry — invoice intake, Qualify, memory, table ops, tools/skills agents, support-desk, content-agent) + plain pedagogical headings throughout (no poetry: 'The warm and cold split' → 'The same agent, with and without memory', 'From one pass to a loop' → 'What tools change', etc.); files/intro's embed swapped to the video's machine
…p4) — 20 pages rewritten from academy-preview/*-light-with-intro, files/intro plays the new files-intro cut, workflows/logs gains its src (video now exists); use-case placeholders stay src-less (no videos yet)
PR SummaryLow Risk Overview Adds
Across existing and new pages, video URLs move from Reviewed by Cursor Bugbot for commit 184b19d. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR adds the next Academy content wave and updates the video set.
Confidence Score: 4/5The Academy pages are mostly safe, but the support-desk workflow embed has a broken condition branch configuration.
apps/docs/components/workflow-preview/academy-video-workflows.ts; apps/docs/content/docs/en/academy/workflows/logs.mdx Important Files Changed
Reviews (1): Last reviewed commit: "biome: format academy-video-workflows.ts" | Re-trigger Greptile |
| conditions: [ | ||
| { id: 'condition-if', label: 'If', value: '<triage.urgent>' }, | ||
| { id: 'condition-else', label: 'else' }, | ||
| ], |
There was a problem hiding this comment.
Condition Branches Are Ignored
The workflow preview renderer reads condition rows from branches, but this new support-desk preview writes them under conditions. When the Chat page renders this workflow, the Urgent? block has no If/else branch rows or matching source handles, so the two outgoing condition edges attach to handles the node never creates.
| conditions: [ | |
| { id: 'condition-if', label: 'If', value: '<triage.urgent>' }, | |
| { id: 'condition-else', label: 'else' }, | |
| ], | |
| branches: [ | |
| { id: 'condition-if', label: 'If', value: '<triage.urgent>' }, | |
| { id: 'condition-else', label: 'else' }, | |
| ], |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| @@ -8,7 +8,7 @@ import { WhatYouWillLearn } from '@/components/ui/what-you-will-learn' | |||
| import { VideoChapters } from '@/components/ui/video-chapters' | |||
| import { WorkflowPreview, ROUTER_TRIAGE_WORKFLOW } from '@/components/workflow-preview' | |||
There was a problem hiding this comment.
This page changes from the src-less "Coming soon" state to a real player that points at a newly introduced blob URL. If academy/workflows-logs.mp4 is not present in the production blob set when this lands, VideoPlaceholder treats the page as playable and renders a broken video instead of falling back to the placeholder.
…ows:[] (the renderer's real fields — conditions: was never read, so Urgent? rendered without if/else handles and branch edges dangled); bgColor aligned to the product condition example
|
Review fixes in
|
…idea> — support-desk Triage and content-agent Writer message rows now match the product's Start output (same pattern as the file's other workflows)
…ex (grounded in the audited codebase facts: memory modes, the ten table ops, coerce/refuse, deploy surfaces + draft-vs-deployed, file parser formats; the FAQ component emits FAQPage JSON-LD for SEO); em-dashes 202 → 0 with hand-fixed splices; added copy where sparse (agent block placement, when-to-use-memory); SEO phrasing sprinkled naturally into FAQ answers ('visual platform for building AI workflows and agents', 'no-code', model names), not over-indexed
… pass (unquoted YAML scalars with a second colon broke every page)
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 184b19d. Configure here.
There was a problem hiding this comment.
Content agent preview incomplete
Medium Severity
The new AV_CONTENT_AGENT_WORKFLOW embed is described in the same change as block-for-block video reproduction, but its parallel container only contains a single Writer block. The chat/building page text still describes multiple generation steps (image, deck, voiceover) alongside drafting and scoring, so the diagram understates the machine learners expect from the lesson.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 184b19d. Configure here.



Companion to simstudioai/sim-academy-videos PR #18 (Academy set 2 → main). Written supplements for the new video wave, in the videos' own pedagogical register.
What's here
chat/intro,chat/building,agents/block,agents/tool-calling,agents/skills,agents/memory,knowledge-bases/connectors,tables/operations,tables/workflow-columns,files/object— house template (video → what-you-will-learn → chapters → prose), chapters timed to each recorded intro + VO.academy-video-workflows.ts— a new workflow-preview registry reproducing, block-for-block, the workflows the videos actually build (invoice intake, the Qualify agent ± tools/skills, the memory agent, the Table query, support-desk with condition branches, content-agent with its parallel container). Every page embeds its video's machine.tables/introandfiles/intro(their videos were replaced by the recomposed/working-day cuts — new chapters; files/intro's embed swapped to the new machine),workflows/logs(gains its player src — the video now exists).academy/<slug>.mp4, 2560×1440@60): 20 pages rewritten off the oldacademy-previewURLs.Register
No poetry — plain explainer prose throughout, matching the narration register ("The same agent, with and without memory", not "The warm and cold split"); Related-documentation links point at real docs routes, not other academy pages.
Not included
agents-patterns/files-operations(DRAFT — variant selection open)