Skip to content

fix: preserve mixed asset query semantics#57

Draft
ScriptedAlchemy wants to merge 3 commits into
codex/config-dependency-watchfrom
codex/asset-query-parity
Draft

fix: preserve mixed asset query semantics#57
ScriptedAlchemy wants to merge 3 commits into
codex/config-dependency-watchfrom
codex/asset-query-parity

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Collaborator

Summary

  • tighten the generic non-CSS ?url asset rule so it does not steal mixed ?url&raw or ?url&inline requests
  • preserve CSS ?url handling outside the generic asset rule
  • add coverage for query ordering and mixed-query exclusions

Validation

  • pnpm test:core
  • pnpm build
  • git diff --check

Notes

Stacked on codex/config-dependency-watch.

@ScriptedAlchemy ScriptedAlchemy force-pushed the codex/asset-query-parity branch from 1f2dcfd to 88aa4ec Compare June 27, 2026 06:06
@ScriptedAlchemy ScriptedAlchemy force-pushed the codex/config-dependency-watch branch from 8772c70 to b0a7155 Compare June 27, 2026 06:06
@github-actions

github-actions Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Benchmark Results

Compared PR head 90ec8f4 against base cc3133e.

Total median wall time: 11.99s -> 11.64s (-2.8%, 1.03x speedup)
Compiler ready median: 9.80s -> 9.61s (-1.9%)
Route load median: 1.96s -> 2.00s (+2.0%)

Benchmark Base total Head total Delta Head ready Head routes Speedup Head RSS p95
large-355-ssr-esm 11.99s 11.64s -2.8% 9.61s 2.00s 1.03x -

Profile: large; mode: dev; iterations: 3; warmup: 0.
Workflow run

@ScriptedAlchemy ScriptedAlchemy force-pushed the codex/asset-query-parity branch from 88aa4ec to 8b08bd7 Compare June 28, 2026 19:58
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