Skip to content

chore(web): add Vercel configuration - #191

Merged
enaboapps merged 2 commits into
mainfrom
codex/vercel-config-190
Oct 9, 2026
Merged

enaboapps merged 2 commits into
mainfrom
codex/vercel-config-190

Conversation

@enaboapps

Copy link
Copy Markdown
Contributor

Closes #190

Summary

  • Adds vercel.json: Vercel runs npx expo export -p web and serves the static dist/ output.
  • cleanUrls lets the per-route HTML files that Expo's static export generates serve /remote, /settings and /diagnostics.
  • No secrets or environment variables are needed. Set the Vercel project's Node.js version to 24.x to match the repo.

The web app itself lands in #185. Merge this after it so deployments serve a working build.

Validation

  • npx expo export -p web exits 0 and writes dist/index.html, remote.html, settings.html and diagnostics.html.
  • npm run validate: lint, typecheck and Jest pass (594 tests). Expo Doctor's package-version check fails as it already does on main; the dependency update in feat(web): add Switchify Remote browser app #185 fixes it.

🤖 Generated with Claude Code

OwenMcGirr and others added 2 commits October 9, 2026 09:50
- Build the Expo web export with `npx expo export -p web` and serve `dist/`
- Enable cleanUrls so the static per-route HTML files serve /remote, /settings and /diagnostics

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@enaboapps
enaboapps marked this pull request as ready for review October 9, 2026 09:16
@enaboapps
enaboapps merged commit e351e58 into main Oct 9, 2026
3 checks passed
@greptile-apps

greptile-apps Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[High impact] Safe to merge subject to enaboapps's stated prerequisites: merge after #185 and configure the Vercel project to use Node.js 24.x.

T-Rex evidence

Evidence from the check

  • This executed helper records each command, working directory, exit code, and combined output, making the logs traceable.

Evidence from the check

  • This executed script starts the actual Vercel CLI, requests pages and exported assets, and compares response bytes against generated files without emulating routing.

Evidence from the check

  • The before run used this configuration with the same build and output settings but cleanUrls disabled, isolating the routing change.

Command output from the check

  • Executed version, diff, worktree, configuration, and inventory commands confirm the tested environment and absence of tracked source changes.

Command output from the check

  • The configured Expo export completed with exit 0 and generated the requested static pages, confirming the build produces deployable output.

Command output from the check

  • Serving dist directly with the root build configuration failed on its missing package.json, showing why the configured build must run from the repository root.

Command output from the check

  • Vercel’s actual static builder completed the configured Expo export and announced readiness before the timed probe stopped it, confirming correct root invocation.

Command output from the check

  • Actual Vercel HTTP requests returned 404 for the three extensionless named routes while explicit HTML pages and all 22 assets resolved, establishing the control behavior.

Command output from the check

  • Actual Vercel HTTP requests returned matching pages and assets plus correct 308 HTML redirects, rejecting the scoped failure hypothesis.

View artifacts

Summary

Adds vercel.json to build and serve the Expo web app on Vercel.

  • Vercel builds and serves the Expo web app.

Reviews (1) · Last reviewed commit: "Merge branch 'main' into codex/vercel-co..." · Reviewed by Greptile

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.

Add Vercel configuration for the web app

2 participants