withastro/astro - CodSpeed

astro

Performance History

Latest Results

fix: use os.availableParallelism() for image optimization queue concurrency Replace os.cpus().length with os.availableParallelism() to respect cgroup CPU quotas in containers. This prevents oversubscription and reduces peak memory usage in CPU-limited Docker/Kubernetes builds. os.availableParallelism() is available since Node 18.14 and returns the same value as os.cpus().length on unconstrained hosts, making this change behavior-neutral outside containers. Fixes #17425

triagebot/fix-17425

8 hours ago

fix(render): render popover as a boolean attribute on custom elements handleBooleanAttribute() unconditionally stringified boolean values (e.g. `popover="true"`) for any tag name containing a hyphen, so that `<el-popover popover>` rendered an invalid attribute value per the Popover API spec (only "auto", "manual", or absent are valid). This broke light-dismiss behavior in libraries that check the attribute value strictly. popover is now always rendered as a bare attribute or omitted, regardless of tag name. Fixes #17398

jiwonyoon-dev:fix/popover-custom-element-boolean-attr

14 hours ago

fix(sessions): eagerly remove storage entry in destroy() to prevent leaks on error paths

iamkaleemsajjad-hue:astro-w2

17 hours ago

Merge branch 'main' into astro-css-fix

thelazylamaGit:astro-css-fix

18 hours ago

Skip URL normalization writes that would not change the path Assigning `url.pathname` re-parses and re-serializes the whole URL, and costs more than parsing a URL from scratch (~212ns vs ~186ns here). For an ordinary request path such as `/about` the pathname is already decoded and free of duplicate slashes, so both assignments in `normalizeUrl` are no-ops that still pay that cost. Guard each assignment on the value actually changing. Skipping a write whose target already equals the current pathname is by definition a no-op, so behavior is unchanged. Measured on the normalization block: 858ns to 421ns for a plain path. `normalizeUrl` is on the hot path of every SSR request: FetchState calls it to build `state.url`, and `createNormalizedUrl` covers the rewrite path. The collapse has to stay after the decode is written back rather than being folded into it: the pathname setter rewrites `\` to `/`, so a decoded backslash only becomes a duplicate slash once assigned (`/a%5C/b` -> `/a\/b` -> `/a//b` -> `/a/b`). Add unit tests for `normalizeUrl`, which had none, pinning that ordering along with the decode, duplicate-slash and re-encoding cases.

iseraph-dev:perf/normalize-url-guard

1 day ago

Parse each request URL once by caching it on the request Adapters need a request's URL before handing the request to `match()` and `render()`, and each step parsed the same string over again: a typical SSR request parsed `request.url` three or four times. Add `getRequestURL()`, which parses `request.url` on first use and keeps the result on the request behind a symbol, the same way render options already travel with a request. Export it from `astro/app` so adapters, including community ones, can share the parse instead of each opening with `new URL(request.url)`. This replaces the pre-parsed `URL` that an earlier revision threaded through `RenderOptions` and `match()`. That contract asked callers to pass a URL equal to `new URL(request.url)` and not to reuse it afterwards, and nothing enforced either half. A URL derived from `request.url` on demand cannot disagree with it, since `request.url` is immutable and a rewritten request is a different object. It also needs no signature changes, so `createRequestFromNodeRequest` keeps returning a `Request` and simply hands over the URL it already parsed. FetchState deliberately keeps parsing its own URL: it rewrites `url.pathname`, and `state.url` is handed to user code as `Astro.url`. Cloning the shared URL instead is not an option, because constructing a URL from another URL serializes and re-parses it, and so costs more than parsing `request.url` again. Node needs no adapter change; it benefits from the seeding above. Netlify's SSR function never parsed a URL of its own, so only its cache provider changes.

iseraph-dev:perf/request-url-cache

1 day ago

Deserialize each route once when loading the manifest deserializeManifest built every route twice on startup: once for the routes array it returns, and once more assigned onto the input object, which nothing reads again. That second pass redid the work, including recompiling the route regex, and threw the result away. Drop it. Both callers pass a fresh manifest they never read back, and the returned routes are their own copies, so nothing observes the removed mutation. No behavior change.

iseraph-dev:perf/dedupe-route-deserialization

1 day ago

fix(prefetch): swallow fetch-fallback rejection to avoid unhandled 'Load failed' on WebKit (#17405) * fix(prefetch): swallow fetch-fallback rejection (WebKit Load failed) * changeset: prefetch fetch-fallback rejection

main

1 day ago

Latest Branches

Use `os.availableParallelism()` for image optimization queue to fix OOM in CPU-limited containers#17427

9 hours ago

5b37822

triagebot/fix-17425

fix(render): render popover as a boolean attribute on custom elements#17422

15 hours ago

9a942d0

jiwonyoon-dev:fix/popover-custom-element-boolean-attr

fix(sessions): route session diagnostics through AstroLogger and reset partial state after regeneration failure#17421

18 hours ago

a5c247a

iamkaleemsajjad-hue:astro-w2

Home Terms Privacy Docs