Skip to content

Remove proof/master runners, epub export, and storyboard acceptance#105

Merged
nicpottier merged 1 commit intomainfrom
nicpottier/fix-web-export
Feb 21, 2026
Merged

Remove proof/master runners, epub export, and storyboard acceptance#105
nicpottier merged 1 commit intomainfrom
nicpottier/fix-web-export

Conversation

@nicpottier
Copy link
Contributor

Summary

This PR removes outdated proof/master CLI runners, EPUB export functionality, and all storyboard acceptance gates. The codebase now uses only the live DAG-based stage-runner for execution and exports web-only with always-fresh asset compilation.

Changes

  • Deleted dead code: Removed proof.ts, master.ts, and all associated CLI runners (superseded by stage-runner's DAG execution)
  • Simplified export: Removed EPUB support and format parameter; now web-only via exportBook(label)
  • Always-fresh assets: Fixed export to always repackage ADT assets, ensuring Tailwind CSS recompilation on download
  • Removed acceptance gates: Eliminated all storyboard acceptance logic that gated export/proof/master stages
  • Simplified UI: Changed ExportButton from dropdown menu to single "Export" button
  • Updated types: Removed proofCompleted from BookSummary, removed EPUB from BookFormat enum

Files Changed

  • 12 files deleted (proof/master runners, epub service, proof routes)
  • 15 files modified (API, studio, types, pipeline)
  • Net: -3,296 lines, +35 insertions
  • All 597 tests pass; typecheck clean

- Deleted standalone proof.ts and master.ts CLI runners, superseded by stage-runner
- Removed proof-runner, proof-service, and proof routes from API (unused by frontend)
- Deleted epub-service and epub export option; export now web-only
- Removed all storyboard acceptance logic that gated export/proof/master stages
- Simplified ExportButton to single "Export" button (no format dropdown)
- Always repackage ADT assets on export to ensure Tailwind compilation is fresh
- Updated integration test to remove proof/master sections (no runner equivalents)
- All 597 tests pass; typecheck clean
@nicpottier nicpottier merged commit c1bef5a into main Feb 21, 2026
1 check passed
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