Skip to content

improvement(academy): set 2, pages for the new video wave (workflow embeds, 2K players, full sequencing)#5382

Merged
waleedlatif1 merged 12 commits into
stagingfrom
feat/docs-academy-videos-set-2
Jul 3, 2026
Merged

improvement(academy): set 2, pages for the new video wave (workflow embeds, 2K players, full sequencing)#5382
waleedlatif1 merged 12 commits into
stagingfrom
feat/docs-academy-videos-set-2

Conversation

@ouiliame

@ouiliame ouiliame commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

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

  • Seven new pages: 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.
  • Retimed/updated: tables/intro and files/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).
  • All players → the 2K blob set (academy/<slug>.mp4, 2560×1440@60): 20 pages rewritten off the old academy-preview URLs.
  • meta.json fully sequenced: Workflows(6) → Chat(2) → Agents(5) → Tables(3) → Files(2) → Knowledge Bases(2) → Use Cases.

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

  • Pages for agents-patterns / files-operations (DRAFT — variant selection open)
  • Use-case pages keep src-less placeholders (videos not produced)

ouiliame added 7 commits July 2, 2026 11:23
…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
…-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)
@vercel

vercel Bot commented Jul 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jul 3, 2026 12:36am

Request Review

@cursor

cursor Bot commented Jul 2, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation and static MDX/TS content only; no runtime product logic, auth, or data paths.

Overview
Expands the English Academy with written supplements for the second video wave: new MDX lessons for Chat (intro, building with chat), Agents (block, tool calling, skills, memory), Tables (operations, workflow columns), Files (file object), and Knowledge bases (connectors), each using the shared lesson layout (video, learn list, chapters, prose, FAQ).

Adds academy-video-workflows.ts with block-accurate PreviewWorkflow definitions for the machines shown in those videos (invoice intake, Qualify ± tools/skills, memory agent, table query, support desk, content agent). Pages that match a video swap or add WorkflowPreview embeds to that registry instead of generic previews.

meta.json reorders and extends the sidebar: Workflows → Chat → Agents (five lessons) → Tables (three) → Files (two) → Knowledge bases (two) → Use cases.

Across existing and new pages, video URLs move from academy-preview/* to academy/<slug>.mp4 (2K blob set). workflows/logs gets a player src; files/intro and tables/intro get retimed chapters and updated embeds where the cut changed. Widespread copy tweaks (punctuation, FAQs on many workflow/agent pages) align tone with the narration; use-case pages stay placeholder-only for video.

Reviewed by Cursor Bugbot for commit 184b19d. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread apps/docs/components/workflow-preview/academy-video-workflows.ts
@greptile-apps

greptile-apps Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds the next Academy content wave and updates the video set.

  • New Chat, Agents, Tables, Files, and Knowledge Bases Academy pages.
  • A new Academy workflow-preview registry for page embeds.
  • Existing Academy players moved to the academy/ blob paths.
  • Academy sidebar sequencing updated in meta.json.

Confidence Score: 4/5

The Academy pages are mostly safe, but the support-desk workflow embed has a broken condition branch configuration.

  • The new workflow registry can render the Chat workflow incorrectly because the condition rows are stored under the wrong field.
  • The Logs page now depends on a new production blob URL and has no page-level fallback if that asset is missing.
  • The navigation and most MDX content changes are straightforward.

apps/docs/components/workflow-preview/academy-video-workflows.ts; apps/docs/content/docs/en/academy/workflows/logs.mdx

Important Files Changed

Filename Overview
apps/docs/components/workflow-preview/academy-video-workflows.ts Adds Academy-specific workflow preview constants, with one condition block using the wrong field for branch rows.
apps/docs/content/docs/en/academy/meta.json Adds the new Academy pages to the sidebar sequence with matching page entries.
apps/docs/content/docs/en/academy/workflows/logs.mdx Switches the Logs Academy page from a placeholder card to a real video source.
apps/docs/content/docs/en/academy/**/*.mdx Adds and refreshes Academy pages with updated video sources and workflow embeds.

Reviews (1): Last reviewed commit: "biome: format academy-video-workflows.ts" | Re-trigger Greptile

Comment on lines +274 to +277
conditions: [
{ id: 'condition-if', label: 'If', value: '<triage.urgent>' },
{ id: 'condition-else', label: 'else' },
],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 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.

Suggested change
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'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Missing Video Fallback

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
@ouiliame

ouiliame commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Review fixes in c9de5fe17:

  • Condition branches (cursor + greptile P1): fixed — the support-desk preview's Urgent? block now uses rows: [] + branches: (the fields the renderer actually reads), matching the CONDITION_ROUTE example; bgColor aligned to the product condition example while there.
  • workflows-logs blob (greptile P2): verified present — academy/workflows-logs.mp4 returns HTTP 200 on the production blob (uploaded with the full 2K set before this PR), so the player has a live asset at merge time.

Comment thread apps/docs/components/workflow-preview/academy-video-workflows.ts Outdated
ouiliame added 2 commits July 2, 2026 16:41
…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)
ouiliame added 2 commits July 2, 2026 17:30
…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)

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ 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.

},
{ id: 'candidates-evaluator', source: 'candidates', target: 'evaluator' },
{ id: 'evaluator-scores', source: 'evaluator', target: 'scores' },
],

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 184b19d. Configure here.

@waleedlatif1 waleedlatif1 changed the title Academy set 2 — pages for the new video wave (workflow embeds, 2K players, full sequencing) improvement(academy): set 2, pages for the new video wave (workflow embeds, 2K players, full sequencing) Jul 3, 2026
@waleedlatif1 waleedlatif1 merged commit 3e71084 into staging Jul 3, 2026
19 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/docs-academy-videos-set-2 branch July 3, 2026 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants