Skip to content

docs(studio): document monorepo dev server port - #2902

Merged
miguel-heygen merged 1 commit into
heygen-com:mainfrom
santhiprakash:docs/fix-studio-package-port
Jul 30, 2026
Merged

docs(studio): document monorepo dev server port#2902
miguel-heygen merged 1 commit into
heygen-com:mainfrom
santhiprakash:docs/fix-studio-package-port

Conversation

@santhiprakash

@santhiprakash santhiprakash commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Problem

After #2901 fixed the contributing setup guide to use http://localhost:5190, the @hyperframes/studio package docs still showed bun run dev without telling contributors which URL to open. The actual port is 5190 in packages/studio/vite.config.ts (server.port).

Self-sourced docs drift.

Triage / Root cause

docs/packages/studio.mdx "From the monorepo" section omitted the dev-server URL while docs/contributing.mdx was corrected in #2901.

Fix

Add one sentence noting the studio dev server listens on http://localhost:5190, with a pointer to packages/studio/vite.config.ts server.port.

Verification

  • python3 ../open-source/scripts/preflight_ship.py --repo heygen-com/hyperframes --local . --branch main --file docs/packages/studio.mdx --must-contain 'bun run dev' --must-not-contain 'localhost:5190' --search 'studio port 5190' → preflight clear before edit
  • bun run format:check -- docs/packages/studio.mdx → passed
  • Commit SSH-signed (verified on GitHub)

Notes / Risks

Sibling to merged #2901 (contributing guide only). No behavior change.

- Problem: @hyperframes/studio package docs showed bun run dev without the
  localhost URL contributors need after heygen-com#2901 fixed contributing.mdx only.
- Fix: note that the studio dev server listens on localhost:5190 per
  packages/studio/vite.config.ts server.port.
- Verification: preflight_ship.py + read vite.config.ts; bun run format:check.
@miguel-heygen
miguel-heygen merged commit a52dd9c into heygen-com:main Jul 30, 2026
42 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