Skip to content

Conversation

USveterandr
Copy link

This PR improves local development on macOS arm64 and updates Wrangler configuration for production/local dev.\n\nChanges included:\n- Replace rolldown-vite with standard Vite (^5.4.11) to fix missing native binding errors\n- Disable @cloudflare/vite-plugin in vite.config.ts for local dev (avoids plugin TypeError during load)\n- Add Vite proxy for /api to http://localhost:8787 to route requests to local Workers dev server\n- Configure wrangler.jsonc: set tsconfig.worker.json, compatibility flags/date, assets SPA handling, observability, unsafe rate limiter bindings, AI/Images/Dispatch/D1/DO/R2/KV, migrations, routes, vars, dev.enable_containers=false\n\nNotes:\n- There are remaining alias resolution issues in Workers build (shared/, worker/) to be addressed separately.\n

drpower added 3 commits October 4, 2025 11:33
@USveterandr USveterandr marked this pull request as ready for review October 4, 2025 15:42
@AshishKumar4
Copy link
Collaborator

Thanks for raising this PR. A couple of things though:

  1. Any particular reasons for switching to npm instead of bun? The missing native bindings errors don't occur in the first place if using bun, and it also speeds up the packages installation process in CI/CD (workers build).
  2. You commited .history folder as well. I dont think thats needed here
  3. Vite Cloudflare plugin is an official way for doing local dev with vite + workers. Any particular reasons why this was to be removed?

Did you try using it with bun instead of npm? The automated setup script should automatically install and configure bun; That's the recommended package manager for this project.

drpower added 12 commits October 11, 2025 15:39
git-subtree-dir: external/drpower-vibe-production
git-subtree-split: 8194f23083c8ea13714eba043ac96e6bcb148962
Add various JSON and text files containing mock API responses, headers, and cookies for local development and testing purposes. Includes health check, registration, and CSRF token responses.
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