Skip to content

Make scene-spec the default Manim path for all clients - #65

Merged
jmjava merged 1 commit into
mainfrom
feat/scene-spec-default-pipeline
Aug 2, 2026
Merged

Make scene-spec the default Manim path for all clients#65
jmjava merged 1 commit into
mainfrom
feat/scene-spec-default-pipeline

Conversation

@jmjava

@jmjava jmjava commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • generate-all auto-runs scene-spec-generate when manim segments exist but animations/specs/*.scene.yaml is missing
  • Retiming matches hyphenated / script labels to Whisper's single glued words (e.g. setup-agent-prompts.sh, version-controlled)
  • CLI/README/AGENTS/dogfood hints document specs as the default; ASCII fix in _box docstring for manim_scene_lint

Test plan

  • pytest tests/test_pipeline.py tests/test_scene_spec.py::test_sync_row_labels_hyphenated_label_matches_glued_whisper_word
  • Fresh client with manim segments and no specs: docgen generate-all --skip-tts creates specs then compiles
  • Existing specs: generate-all retimes without LLM unless --regen-scene-specs

Made with Cursor

Auto-generate missing specs on first generate-all, document the consumer flow, and match Whisper-glued hyphenated compounds so script names and versioned labels retime reliably.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@jmjava
jmjava merged commit 1dfdf6b into main Aug 2, 2026
5 checks passed
@jmjava
jmjava deleted the feat/scene-spec-default-pipeline branch August 2, 2026 03:04
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.

1 participant