Skip to content

fix: preserve Laravel Vite configuration in Docker presets - #544

Merged
dbpolito merged 1 commit into
mainfrom
fix/laravel-vite-config
Nov 12, 2026
Merged

dbpolito merged 1 commit into
mainfrom
fix/laravel-vite-config

Conversation

@dbpolito

Copy link
Copy Markdown
Member

Fixes #486.

Type
Bug Fix Yes
Documentation Yes
Break Change No

Laravel's current 13.x and 12.x configs already configure their Vite plugins. Kool's replacement drops Tailwind, font loading, watcher settings, and user customizations.

  • Leave Vite configuration untouched in both Laravel and Octane presets; remove the obsolete replacement configs.
  • Supply only Docker networking arguments through npm/Yarn/Bun's dev command: --host 0.0.0.0 --port 3001 --strictPort. Keep the existing port mapping.
  • Cover config preservation (including legacy/custom and TypeScript configs) and generated service commands with regression tests.

Browser-checked Laravel 12/Octane and fresh Laravel 13: Tailwind styling is restored; Laravel 13's upstream fonts and HMR work with npm, Yarn, and Bun. Smoke tests used alternate free ports; switching the shared test project to Bun required clearing/fixing existing root-owned Vite caches. Go tests, preset race tests, and lint pass.

Existing projects: restore any Vite config overwritten by an older preset from version control or vite.config.js.bak.* before reapplying. This change does not reconstruct a previously overwritten config.

@github-actions github-actions Bot added release-drafter:fixed Release drafter: Fixed release-drafter:documentation Release drafter: Documentation release-drafter:patch Release drafter: Patch labels Nov 12, 2026
@dbpolito
dbpolito merged commit 602a036 into main Nov 12, 2026
9 of 11 checks passed
@dbpolito
dbpolito deleted the fix/laravel-vite-config branch November 12, 2026 19:12
@dbpolito

Copy link
Copy Markdown
Member Author

Published in 3.8.1. Six standalone binaries and their SHA-256 files are attached; release-guide lint/race tests and release-binary config-preservation smoke checks passed. Added release and migration follow-up to #486, which is closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-drafter:documentation Release drafter: Documentation release-drafter:fixed Release drafter: Fixed release-drafter:patch Release drafter: Patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Laravel+Octane: Erro nos assets do Vite

1 participant