fix: add assets.directory to generated wrangler.jsonc

[james-elicx]

Summary

• Wrangler 4.69+ rejects an assets object that lacks a directory field, causing vinext deploy to fail with: The 'assets' property in your configuration is missing the required 'directory' property
• Added directory: "dist/client" to the generated assets block in generateWranglerConfig() — this is always the correct value since @cloudflare/vite-plugin unconditionally writes static assets to dist/client/
• Updated the existing test asserting the assets object shape, and added a dedicated test for the directory field

Notes

• not_found_handling: "none" is intentional and unchanged — it ensures unmatched requests fall through to the user Worker (enabling SSR) rather than being handled by the asset worker directly

Fixes #219

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/vinext@569

commit: 7c5368d

[github-actions]

Example	Preview	Production	Original
app-router-cloudflare	preview	production
pages-router-cloudflare	preview	production
app-router-playground	preview	production	original
realworld-api-rest	preview	production
nextra-docs-template	preview	production
benchmarks	preview	production
hackernews	preview	production	original

[james-elicx]

/bigbonk review

[ask-bonk]

Clean, minimal fix. The directory: "dist/client" value is correct — @cloudflare/vite-plugin unconditionally writes static assets there, and wrangler 4.69+ requires the field when assets is an object.

Tests look good: the existing assertion was updated to include the new field, and the dedicated test for assets.directory provides clear documentation of the requirement and why the value is correct.

No concerns.

[ask-bonk]

Approved PR #569. The change is straightforward and correct:

• Source change (deploy.ts:401-403): Adds directory: "dist/client" to the generated assets block in generateWranglerConfig(), with a clear comment explaining the wrangler 4.69+ requirement.
• Test changes (deploy.test.ts): Updates the existing toEqual assertion to include the new field, and adds a dedicated test that specifically validates assets.directory is "dist/client" with comments documenting why.

The value "dist/client" is always correct because @cloudflare/vite-plugin unconditionally writes static assets to that path. The existing not_found_handling: "none" is preserved and correct for SSR fallthrough.

github run