Skip to content

Deployment on vercel - #406

Merged
mudabs merged 9 commits into
oss-slu:mainfrom
Remex9:deployment-vercel
Sep 12, 2026
Merged

Deployment on vercel#406
mudabs merged 9 commits into
oss-slu:mainfrom
Remex9:deployment-vercel

Conversation

@Remex9

@Remex9 Remex9 commented Sep 11, 2026 •
edited by mudabs
Loading

Copy link
Copy Markdown
Collaborator

Summary
Deploys Digital Bone Box on Vercel as a single-origin Express app so the current UI and API work from one preview/production URL.

Changes Made

  • Serve templates/ from Express and move the JSON welcome response to /health so / loads the app
  • Use same-origin relative API/HTMX URLs (remove hardcoded 127.0.0.1:8000) and relative image URLs
  • Configure Vercel to copy the frontend assets into public/ and include the required data/templates files in the serverless function bundle.
  • Add root express-rate-limit dependency required by the serverless bundle
  • Update README for one-origin local run and Vercel deploy notes
  • Update boneset-api start script and server tests for /health + HTML /

Testing

  • Ran npm test (existing suites pass; server tests updated for /health and /)
  • Ran npm run lint and passed
  • Manually tested one-origin locally via Express (npm start --prefix boneset-api)
  • Verified Vercel preview: dropdowns, descriptions, images/overlays, bone-data/search APIs, quiz image load

Additional Notes
Local breaking change: open the app at the Express origin (http://127.0.0.1:8000/). Dual http-server + API on different ports no longer works with relative API URLs.

Closes #408

Follow-up Vercel ownership migration: #407

Remex9 and others added 7 commits September 11, 2026 14:19
@Remex9
Add Vercel deployment support
2a60cc5
@Remex9 @cursoragent
Trigger Vercel preview redeploy for deployment-vercel.
3a8637c 
Co-authored-by: Cursor <cursoragent@cursor.com>
@Remex9 @cursoragent
Fix Vercel entry to use api/index.cjs
3db24a3 
Root server.cjs is not a valid functions path; serve the Express app from api/ and copy static assets into public during install.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Remex9
Simplify Vercel api entry and vercel.json
1445fc8
@Remex9 @cursoragent
Use ESM api/index.js entry and drop Vercel rewrites.
9ec9f32 
Co-authored-by: Cursor <cursoragent@cursor.com>
@Remex9
Fix Vercel 404: restore rewrites and public index
40d5d61
@Remex9
Fix Vercel function crash: root rate-limit dep and data includes
4a63c90
This was referenced Sep 11, 2026
Open
Closed
mudabs
mudabs approved these changes Sep 11, 2026
View reviewed changes

@mudabs mudabs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Closes #406

mudabs
mudabs approved these changes Sep 12, 2026
View reviewed changes

@mudabs mudabs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. Tests are working!

@mudabs
mudabs merged commit d3d7bf0 into oss-slu:main Sep 12, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mudabs mudabs mudabs approved these changes
@jacksayshi jacksayshi Awaiting requested review from jacksayshi jacksayshi is a code owner
@UcheWendy UcheWendy Awaiting requested review from UcheWendy UcheWendy is a code owner
@leandrumartin leandrumartin Awaiting requested review from leandrumartin leandrumartin is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Deploy Digital Bone Box to Vercel

2 participants

@Remex9 @mudabs