Skip to content

fix(cli): validate project directory before starting preview#1394

Merged
miguel-heygen merged 1 commit into
heygen-com:mainfrom
manupareekk:fix/preview-validate-project-dir
Jun 13, 2026
Merged

fix(cli): validate project directory before starting preview#1394
miguel-heygen merged 1 commit into
heygen-com:mainfrom
manupareekk:fix/preview-validate-project-dir

Conversation

@manupareekk

Copy link
Copy Markdown
Contributor

Summary

  • hyperframes preview now validates the project directory (must exist + contain index.html) before starting Studio
  • Fixes empty Studio when running hyperframes preview # or other invalid paths
  • Adds a clearer error when # is passed as the directory argument

Test plan

  • bun run test src/utils/project.test.ts in packages/cli
  • hyperframes preview # exits with error (does not show "Studio running")
  • hyperframes preview . from a valid project still works

@manupareekk manupareekk force-pushed the fix/preview-validate-project-dir branch 3 times, most recently from 3d0b9ce to 86fde79 Compare June 12, 2026 23:27
Preview previously started Studio even when the path was invalid (e.g.
`hyperframes preview #`), yielding an empty project view. Align preview
with lint/render by resolving the project up front, and add a clearer
error when `#` is passed as a directory argument.

Co-authored-by: Cursor <cursoragent@cursor.com>
@manupareekk manupareekk force-pushed the fix/preview-validate-project-dir branch from 86fde79 to 8110077 Compare June 12, 2026 23:29

Copy link
Copy Markdown
Collaborator

Merge activity

  • Jun 12, 11:58 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason (Fast-forward merges are not supported for forked repositories. Please create a branch in the target repository in order to merge).

@miguel-heygen miguel-heygen merged commit 2ec0062 into heygen-com:main Jun 13, 2026
35 checks 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.

2 participants