Skip to content

fix(workflow-vite): sub-path deploy 404s, source maps, staging build,… - #898

Merged
jordan-a-young merged 2 commits into
masterfrom
fix/workflow-vite-build-defaults
Sep 17, 2026
Merged

jordan-a-young merged 2 commits into
masterfrom
fix/workflow-vite-build-defaults

Conversation

@jordan-a-young

Copy link
Copy Markdown
Member

… schema defaults

  • Set base: './' so asset URLs are relative — fixes 404s when apps are served from Tyk sub-paths (e.g. /tst/appl/my-service/static/nav/)
  • Staging builds: source maps on, minification off, reportCompressedSize on
  • Production builds: source maps off (Rolldown bug workaround via writeBundle plugin that deletes .map files before process exits)
  • Dry-run preview now passes the full resolved config so base, plugins, and modifyViteConfig changes are honoured by the preview server
  • Add development.optimizeDeps to extend Vite pre-bundling without modifyViteConfig; swap react-router-dom for react-router in built-in defaults
  • Fix ekko per-field schema defaults so partial overrides (e.g. ekko.port only) no longer silently drop other ekko fields
  • Align suppressDeprecationWarnings default to false in both packages
  • Align development.open default to '/' in both packages
  • Remove dead code fallbacks from host(), statsLogLevel(), infrastructureLogLevel()
  • Move host default (0.0.0.0) into workflow schema where it actually takes effect
  • Full JSDoc coverage on all index.d.ts interfaces and Settings class methods

Before submitting a pull request, please make sure the following is done:

  1. If you are part of the Availity organization, clone this repository and create your branch off of master. If you are not part of the organization, you will need to fork this repository and the create your branch off of master.
  2. Run yarn in the repository root.
  3. If you've fixed a bug or added code that should be tested, add tests!
  4. Ensure the test suite passes (yarn test). Tip: yarn test --watch TestName is helpful in development.
  5. Make sure your code passed the conventional commits check. Read more about conventional commits

… schema defaults

- Set base: './' so asset URLs are relative — fixes 404s when apps are served
  from Tyk sub-paths (e.g. /tst/appl/my-service/static/nav/)
- Staging builds: source maps on, minification off, reportCompressedSize on
- Production builds: source maps off (Rolldown bug workaround via writeBundle
  plugin that deletes .map files before process exits)
- Dry-run preview now passes the full resolved config so base, plugins, and
  modifyViteConfig changes are honoured by the preview server
- Add development.optimizeDeps to extend Vite pre-bundling without modifyViteConfig;
  swap react-router-dom for react-router in built-in defaults
- Fix ekko per-field schema defaults so partial overrides (e.g. ekko.port only)
  no longer silently drop other ekko fields
- Align suppressDeprecationWarnings default to false in both packages
- Align development.open default to '/' in both packages
- Remove dead code fallbacks from host(), statsLogLevel(), infrastructureLogLevel()
- Move host default (0.0.0.0) into workflow schema where it actually takes effect
- Full JSDoc coverage on all index.d.ts interfaces and Settings class methods
@jordan-a-young
jordan-a-young merged commit a8bc066 into master Sep 17, 2026
5 checks passed
@jordan-a-young
jordan-a-young deleted the fix/workflow-vite-build-defaults branch September 17, 2026 13:45
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