Skip to content

reduce lmdb mapsize for tests and remove unused setup-cache#1017

Merged
vykimnguyen merged 3 commits intomainfrom
fix-hanging-tests
Feb 12, 2026
Merged

reduce lmdb mapsize for tests and remove unused setup-cache#1017
vykimnguyen merged 3 commits intomainfrom
fix-hanging-tests

Conversation

@vykimnguyen
Copy link
Contributor

@vykimnguyen vykimnguyen commented Feb 11, 2026

Motivation

Integration tests could be using enough memory to cause OOM/CI cancellations. The cache suite in particular was a major consumer. Reducing LMDB’s mapSize in test lowers peak RSS without changing test behaviour.

Results from running cache.ts integration test suite 10 times (v2):

Metric Baseline (1 GB mapSize) After (256 MB mapSize)
Average peak RSS 3879 MB 2265 MB
Reduction ~42%
Min / max peak RSS 3489–4349 MB 2158–2429 MB

With ATLASPACK_V3=true

Metric Baseline (1 GB mapSize) Optimized (256 MB mapSize)
Average peak RSS 2348.5 MB 2333.2 MB
Min / max peak RSS 2320–2376 MB 2244–2360 MB

Changes

  • Lowers LMDB mapSize option from 1GB to 256MB per integration test
  • Removes setup-cache.ts as it is unused

Checklist

  • Existing or new tests cover this change
  • There is a changeset for this change, or one is not required
  • Added documentation for any new features to the docs/ folder

@changeset-bot
Copy link

changeset-bot bot commented Feb 11, 2026

🦋 Changeset detected

Latest commit: f0fb9f1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 20 packages
Name Type
@atlaspack/test-utils Patch
@atlaspack/cache Patch
atlaspack Patch
@atlaspack/core Patch
@atlaspack/inspector Patch
@atlaspack/packaging-test-harness Patch
@atlaspack/query Patch
@atlaspack/bundler-experimental Patch
@atlaspack/config-default Patch
@atlaspack/config-webextension Patch
@atlaspack/cli Patch
@atlaspack/register Patch
@atlaspack/bundle-stats Patch
@atlaspack/optimizer-image Patch
@atlaspack/reporter-bundle-stats Patch
@atlaspack/transformer-image Patch
@atlaspack/transformer-js Patch
@atlaspack/inspector-frontend Patch
@atlaspack/transformer-compiled-css-in-js Patch
@atlaspack/transformer-tokens Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Feb 11, 2026

📊 Type Coverage Report

Coverage Comparison

Metric Baseline Current Change
Coverage Percentage 92.26% 92.26% ➡️ 0.00%
Correctly Typed 220,731 220,720 -11
Total Expressions 239,225 239,214 -11
Untyped Expressions 18,494 18,494 +0

Files with Most Type Issues (Top 15)

File Issues Affected Lines
packages/core/integration-tests/test/javascript.ts 1152 745
packages/core/integration-tests/test/cache.ts 885 626
packages/core/integration-tests/test/scope-hoisting.ts 622 489
packages/utils/node-resolver-core/test/resolver.ts 476 177
packages/core/integration-tests/test/html.ts 468 294
packages/core/integration-tests/test/sourcemaps.ts 356 176
packages/core/test-utils/src/utils.ts 330 205
packages/core/integration-tests/test/incremental-bundling.ts 298 206
packages/core/core/src/dumpGraphToGraphViz.ts 251 108
packages/core/integration-tests/test/transpilation.ts 230 139
packages/core/integration-tests/test/output-formats.ts 227 161
packages/transformers/webextension/src/WebExtensionTransformer.ts 210 80
packages/core/core/src/requests/BundleGraphRequestRust.ts 194 67
packages/core/integration-tests/test/css-modules.ts 191 107
packages/core/core/src/requests/TargetRequest.ts 190 133

This report was generated by the Type Coverage GitHub Action

@vykimnguyen vykimnguyen marked this pull request as ready for review February 11, 2026 23:36
@vykimnguyen vykimnguyen requested a review from a team as a code owner February 11, 2026 23:36
@vykimnguyen vykimnguyen merged commit 3ad8e88 into main Feb 12, 2026
24 of 25 checks passed
@vykimnguyen vykimnguyen deleted the fix-hanging-tests branch February 12, 2026 06:02
@atlaspack-ci atlaspack-ci bot mentioned this pull request Feb 12, 2026
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