Conversation
Greptile SummaryThis PR bumps all major and minor dependencies across the monorepo, clearing 16 Key changes:
Confidence Score: 5/5Safe to merge — all config migrations are correct, tests pass, and no vulnerabilities remain Comprehensive test plan was executed (258 unit tests, 3 workers tests, lint, type-check, build budget, publint, attw), all 16 audit vulnerabilities cleared, config migrations correctly follow the new APIs for every bumped tool, and the prior review concern about noConsole was already addressed No files require special attention Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
subgraph Before["Before (versions bumped)"]
B1["biome 1.9\norganizeImports top-level\nfiles.ignore\nnursery.noRestrictedImports"]
B2["tsdown 0.10\nexternal: [/^node:/]"]
B3["@cloudflare/vitest-pool-workers 0.5\ndefineWorkersConfig('/config')"]
B4["vitest 2.1 + vite 5\n(lockfile resolved)"]
B5["typescript 5.9"]
end
subgraph After["After (this PR)"]
A1["biome 2.4\nassist.organizeImports\nfiles.includes negation\nstyle.noRestrictedImports\nnoConsole: error"]
A2["tsdown 0.21\ndeps.neverBundle: [/^node:/]\nfixedExtension: false"]
A3["@cloudflare/vitest-pool-workers 0.14\ncloudflareTest() plugin\n+ defineConfig"]
A4["vitest 4.1 + vite 7.3\n(pnpm.overrides.vite ^7.1)"]
A5["typescript 6.0"]
end
B1 -->|biome migrate| A1
B2 -->|API renamed| A2
B3 -->|/config subpath removed| A3
B4 -->|override added| A4
B5 -->|major bump| A5
A1 & A2 & A3 & A4 & A5 --> Result["0 pnpm audit vulnerabilities\n258/258 unit tests pass\n3/3 workers tests pass\nbuild budget ✓"]
Reviews (2): Last reviewed commit: "invert log statement" | Re-trigger Greptile |
|
@greptile review and summary |
Summary
packages/passmintdevDepspnpm auditfindings (0 vulnerabilities after); all transitive via@cloudflare/vitest-pool-workersandtsdownMajor version bumps
@biomejs/biome1.9 → 2.4 (config migrated viabiome migrate;organizeImportsis now an assist)typescript5.7 → 6.0vitest2.1 → 4.1tsdown0.10 → 0.21@cloudflare/vitest-pool-workers0.5 → 0.14Minor bumps
@changesets/cli2.27 → 2.30turbo2.3 → 2.9@peculiar/asn1-*2.3 → 2.6@arethetypeswrong/cli0.17 → 0.18publint0.3.2 → 0.3.18Config / code follow-ups required by the bumps
tsdown.config.ts:external→deps.neverBundle; addedfixedExtension: falseto keepdist/index.js(tsdown 0.21 defaults to.mjs)vitest.workers.config.ts: migrated fromdefineWorkersConfig('@cloudflare/vitest-pool-workers/config')tocloudflareTest()plugin +defineConfig(the/configsubpath was removed in 0.14)biome.json: migrated to 2.x schemapnpm.overrides.vite: ^7.1.0— vitest 4 peers on Vite ≥6 but the lockfile was resolving to 5Test plan
pnpm audit— 0 vulnerabilities (was 16)pnpm lintcleanpnpm type-checkcleanpnpm test— 258/258 passedpnpm test:workers— 3/3 passedpnpm build— bundle 11.26 KB gzipped (under 15 KB budget), 44.33 KB with deps (under 50 KB budget)pnpm --filter passmint run bundle-guard— zeronode:*refspnpm --filter passmint run publint— cleanpnpm --filter passmint run attw— clean on esm-only profile