Skip to content

Fix resume asset paths for front-door mount#102

Merged
dytsou merged 1 commit into
mainfrom
fix/resume-base-path
Jul 12, 2026
Merged

Fix resume asset paths for front-door mount#102
dytsou merged 1 commit into
mainfrom
fix/resume-base-path

Conversation

@dytsou

@dytsou dytsou commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Problem

At dy.tsou.me/resume/, the browser threw MIME errors: assets were referenced from the site root (/assets/index-*.css) instead of /resume/assets/.... The front door routes /assets/... to the root site, which returns index.html (text/html), so styles/scripts refused to load.

Cause

The Vite base only becomes /resume/ when GITHUB_PAGES/CF_PAGES is set. The GitHub Actions CI build sets neither, so it defaulted to / and emitted absolute root asset paths.

Fix

  • Set BASE_PATH=/resume/ on the CI build step so assets emit as /resume/assets/....
  • Add a build-output check that fails if index.html isn't referencing /resume/assets/.
  • Correct deploy-contract.json basePath to /resume/.

Verified locally: built index.html now references /resume/assets/index-*.{js,css}.

@cloudflare-workers-and-pages

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
resume fbec580 Jul 12 2026, 08:46 AM

@sonarqubecloud

Copy link
Copy Markdown

@dytsou
dytsou merged commit d23504e into main Jul 12, 2026
11 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.

1 participant