Repository navigation
chore(deps): consolidate open Renovate and Dependabot updates - #192
Conversation
Dependencies: yargs 18, p-retry 9, @napi-rs/keyring 2, dotenv 18, @types/node 26, plus patch and minor bumps for faker, execa, playwright, tsdown, turbo, oxlint, oxfmt, axios and npm. Node 26 in .node-version. Actions: setup-node 7.1.0, upload-artifact 7.0.2, download-artifact 8.0.2, codeql-action 4.39.3, sonarqube-scan 8.4.1, ketryx-github-action 2.0.0. Transitive security bumps from Dependabot land through the regenerated lockfile. Test and config fixes carried along: - class mocks use function implementations (required by vitest 4+) - vitest config imports defineConfig from vitest/config - tsdown external for msw now covers msw/node, which was being bundled - yargs 18 resolves main() after --help instead of throwing Left out on purpose: openid-client 6 (full API rewrite of auth.ts), TypeScript 7 (typedoc and tsdown dts need the 5.x API), and vitest 4/5 plus msw 3 (both pull vite 7+, whose lightningcss dependency is MPL-2.0, which license-checking-config.json prohibits). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
OX Security reviewed this pull request — nothing to fix.
Branch |
|
Answering a review question from @tobi-ax on
vitest 5 was in this PR until the license hook blocked it (vite 7+ pulls in lightningcss, MPL-2.0), so the mocks got fixed first and the bump dropped later. I kept the mock change because it is harmless on vitest 3 and saves whoever does the upgrade from hitting it again. Written with Claude. |
Summary
Replaces the open bot PRs (11 Renovate, 7 Dependabot) plus the rate-limited items from the Dependency Dashboard with one branch built fresh from main.
Code changes are confined to tests and tool config:
cli.test.ts,cli-functions.spec.ts,auth.spec.ts,token-storage.spec.tsusefunctionimplementations (vitest 4+ rejects arrow functions for anything called withnew, so this is ready for that upgrade)vite.config.tsimportsdefineConfigfromvitest/configtsdown.config.tsexternal pattern now matchesmsw/node, which was being inlined into the published@aignostics/sdk/testbundle (565 KB → 24 KB)main()after--helpinstead of throwing; the help test asserts on the printed usage onlyDeliberately left out, each needs its own decision
license-checking-config.jsonlists MPL-2.0 as prohibited and the pre-commit hook blocks the lockfile. It is dev-only and transitive, but allowlisting it is a legal call. Until then the vitest security advisory stays open.Issuer/ClientAPI is gone,auth.tsneeds a rewrite.Evidence
Local on Node 24: build, typecheck, lint, format check, unit tests and coverage thresholds pass for both packages (12 test files). Built
bin.cjsandbin.jsrun with ESM-only yargs 18 from a CJS entry.The e2e job already fails on main (403 from staging on
applications versions get), unrelated to this change.Merge Danger
Door: two-way
Revert restores the previous lockfile and pins. Nothing is published by merge alone; release-please cuts the release.
Blast Radius: consumers
@aignostics/clinow depends on ESM-only yargs 18 (Node 20.19+, 22.12+ or newer) whileenginesstill says>=18. Node 18 is EOL. I leftenginesalone; say the word and I bump it.token-storage.tsalready wraps every call in try/catch with a file fallback, so behavior is unchanged.🤖 Generated with Claude Code