Skip to content

fix: deploy benchmark site as Cloudflare static assets - #87

Closed
zhenghaoz wants to merge 1 commit into
mainfrom
fix/cloudflare-assets-config
Closed

zhenghaoz wants to merge 1 commit into
mainfrom
fix/cloudflare-assets-config

Conversation

@zhenghaoz

Copy link
Copy Markdown
Contributor

Cloudflare Workers Builds completes pnpm build, but npx wrangler deploy finds no Wrangler configuration and starts automatic Astro adapter setup. That setup fails on the missing public/.assetsignore path.

Add docs/wrangler.jsonc for Worker xvec, serving the already-generated ./dist directory as Workers Static Assets. Pin Wrangler 4.140.0 in the pnpm dependencies and lockfile so the existing deploy command uses the installed version. Allow workerd's binary installation for local validation, ignore .wrangler, and document the Workers Builds settings. SVGs and other generated files remain only in ignored build directories.

Validation:

  • pnpm install --frozen-lockfile and pnpm build pass.
  • Noninteractive wrangler deploy --dry-run reads all 8 built assets without invoking automatic project setup.
  • Local Wrangler serves the homepage, CSS, JavaScript, logo, default SVG, and configuration-specific SVG with HTTP 200. SVG content types and bytes match the build output.

No live deployment was performed. Cloudflare can retain root directory docs, build command pnpm build, and deploy command npx wrangler deploy.

@codecov

codecov Bot commented Sep 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.39%. Comparing base (6761a87) to head (e466d04).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
- Coverage   81.68%   81.39%   -0.29%     
==========================================
  Files         148      148              
  Lines       30253    30253              
==========================================
- Hits        24712    24625      -87     
- Misses       5539     5626      +87     
  Partials        2        2              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@zhenghaoz zhenghaoz closed this Sep 25, 2026
@zhangzhenghao
zhangzhenghao deleted the fix/cloudflare-assets-config branch September 27, 2026 00:07
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