Skip to content

Add tests for skipProxy with Inertia-style asset scenarios (JS, CSS, SCSS, images)#602

Open
ElMassimo wants to merge 2 commits intomainfrom
claude/test-inertia-no-proxy-ADCXS
Open

Add tests for skipProxy with Inertia-style asset scenarios (JS, CSS, SCSS, images)#602
ElMassimo wants to merge 2 commits intomainfrom
claude/test-inertia-no-proxy-ADCXS

Conversation

@ElMassimo
Copy link
Copy Markdown
Owner

@ElMassimo ElMassimo commented Apr 7, 2026

  • Identify root cause: "packageManager": "pnpm@10.32.1" in package.json causes yarn 1.x CI to fail
  • Remove packageManager field from package.json

…SCSS, images)

Validates that skip_proxy: true correctly generates absolute URLs pointing
directly to the Vite dev server for all asset types. Covers manifest lookups,
tag helpers (client, JS, CSS, SCSS, images, React refresh), and documents
known caveats (CORS, Rails 6 .scss.css, cookies, SSL, Docker networking).

https://claude.ai/code/session_01Dfaxd2gi78PhPtXVqcfkcq
@ElMassimo
Copy link
Copy Markdown
Owner Author

@copilot Remove the packageManager field if set, verify if that fixes tests, otherwise come up with a fix.

Copy link
Copy Markdown

Copilot AI commented Apr 7, 2026

@copilot Remove the packageManager field if set, verify if that fixes tests, otherwise come up with a fix.

Removed the packageManager field from package.json in commit 32ada2a. The field ("pnpm@10.32.1") was causing yarn v1 (used in CI) to fail with "packageManager": "yarn@pnpm@10.32.1" — a yarn v1 bug where it prepends "yarn@" to the value — blocking yarn install in the engine rake tasks tests.

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.

3 participants