chore(deps): consolidate dependabot updates - #1550
Conversation
Updates from Dependabot PRs: - @amplitude/unified: 1.1.28 -> 1.1.32 - @codemirror/state: 6.7.1 -> 6.7.4 - @codemirror/view: 6.43.9 -> 6.43.11 - @mui/material: 9.3.1 -> 9.4.0 - @mui/system: 9.3.0 -> 9.4.0 - @next/third-parties: 16.3.0 -> 16.3.4 - @sentry/nextjs: 10.70.0 -> 10.73.0 - @sentry/react: 10.70.0 -> 10.73.0 - @tanstack/react-query: 5.101.4 -> 5.102.8 - @xyflow/react: 12.11.2 -> 12.11.6 - @zumer/snapdom: 2.24.1 -> 2.24.15 - happy-dom: 20.11.2 -> 20.14.0 - import-in-the-middle: 3.3.3 -> 3.4.0 - next: 16.3.0 -> 16.3.4 - posthog-js: 1.415.1 -> 1.428.3 - @amplitude/analytics-core: 2.54.1 -> 2.55.0 - @biomejs/biome: 2.5.7 -> 2.5.12 - @testing-library/react: 16.3.2 -> 16.3.3 - @testing-library/user-event: 14.6.3 -> 14.6.7 - @types/node: 26.2.0 -> 26.5.0 - @types/react-dom: 19.2.4 -> 19.2.7 - @vitejs/plugin-react: 6.0.5 -> 6.1.1 - baseline-browser-mapping: 2.11.15 -> 2.11.21 - globals: 17.9.0 -> 17.12.0 - postcss: 8.5.26 -> 8.5.28 - vite: 8.2.1 -> 8.2.2 - @playwright/test: 1.62.1 -> 1.63.0 - @storybook/addon-docs: 10.5.8 -> 10.6.0 - @storybook/addon-vitest: 10.5.8 -> 10.6.0 - @storybook/react: 10.5.8 -> 10.6.0 - @storybook/react-vite: 10.5.8 -> 10.6.0 - playwright: 1.62.1 -> 1.63.0 - storybook: 10.5.8 -> 10.6.0 - @tsdown/css: 0.22.14 -> 0.23.0 - tsdown: 0.22.14 -> 0.23.0 - typedoc-plugin-markdown: 4.12.0 -> 4.13.0 - vitest: 4.1.10 -> 4.1.11 - @vitest/coverage-v8: 4.1.10 -> 4.1.11 - @vitest/browser-playwright: 4.1.10 -> 4.1.11 - fast-uri: 4.1.2 -> 4.1.4 - sqlparse: 0.5.5 -> 0.6.0 - dbt-core: 1.12.0 -> 1.12.4 - dbt-core-experimental-parser: 2.0.0a5 -> 2.0.0rc2 - opentelemetry-api: — -> 1.44.0 Align workspace overrides and Vitest companion packages. Resolve sqlparse with stable dbt-core 1.12.4 instead of the bot lockfile that removed sqlparse and selected dbt-core 2.0.0b1. Verify Biome 2.5.12 on macOS arm64 and update its version guard. Assert inherited cell color using its computed value in happy-dom. Refs: #1548, #1547, #1546, #1535, #1509 Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Jared Scott <jared.scott@datarecce.io>
Code Review: PR #1550SHA BlockersNone. IssuesNone. Notes
Checks run
|
wcchang1115
left a comment
There was a problem hiding this comment.
GO at ab7d07cb. Full review: #1550 (comment) — no blockers, no issues, 5 notes.
PR checklist
What type of PR is this?
chore(deps)
What this PR does / why we need it:
Consolidates all five open Dependabot PRs, including fixes for the reported Next.js, Vitest, fast-uri, and sqlparse advisories. Updates all three frontend workspaces and their overrides so pinned overrides do not silently retain older versions.
The grouped PR supplies Next.js 16.3.4 and fast-uri 4.1.4, superseding the individual 16.3.3 and 4.1.3 PRs. Aligns Vitest's coverage and browser companion packages at 4.1.11.
Which issue(s) this PR fixes:
Closes #1548
Closes #1547
Closes #1546
Closes #1535
Closes #1509
Special notes for your reviewer:
inheritto a color value. No application behavior change.externaloption still works but emits a deprecation warning recommendingdeps.neverBundle.Package changes
Security review
No high-confidence vulnerabilities identified in the combined working-tree diff.
pnpm audit --jsonreports zero advisories. Fresh PyPI checks report no advisories or yanked releases for the four changed Python packages. Existing GitHub alerts were checked for the security context of each update. The subsequent full Python dependency audit found an unchanged development dependency advisory in jaraco.context; see recommendations below. This review covers the changeset, not a full application penetration test.Validation
CI=true pnpm install --frozen-lockfilepnpm lint:fix,pnpm lint, andpnpm exec biome check --stagedpnpm type:check(all three workspaces)pnpm test: 211 files passed; 4,444 tests passed, 5 skippedpnpm --filter @datarecce/ui buildandpnpm run builduv lock --checkanduv sync --locked --extra dev --extra mcpuv run --locked --extra dev --extra mcp make checkuv run --locked --extra dev --extra mcp python -m pytest tests/ -q: 2,026 passed, 5 skipped (Python 3.12)recce serversmoke test: HTTP 200 with built Next.js asset referencescore.hooksPath=js/.husky/_Backend tests run directly through pytest to avoid
make testchanging the hook configuration. Existing deprecation warnings and an openpyxl cleanup warning remain non-fatal. The multi-version tox matrix was not run; no adapter code changed.Audit-driven recommendations
Fresh registry audit at 2026-09-14 01:02 UTC (09:02 Taipei): queried 95 distinct npm dependencies across all three manifests, 44 Python runtime/dev/build requirements across both pyprojects, and 15 GitHub Actions. Compared the manifests and committed lockfiles with live registry metadata; no local-cache “latest” values were used. The audit found 24 npm and 27 Python packages with newer releases, plus the transitive security finding below. No tracked Dockerfiles were found. All recommendations below are advisory and have not been applied.
Prioritize: existing development dependency advisory
pip-audit --path .venv/lib/python3.12/site-packages --format jsonfound one advisory: jaraco.context 6.0.1, CVE-2026-23949 / GHSA-58pv-8j8x-9vj2, high severity path traversal when extracting malicious tarballs. Recommend 6.1.2 (the first fixed release is 6.1.0). Dependency chain:recce[dev] → twine 7.0.0 → keyring 25.7.0 → jaraco-context 6.0.1. The same version is inmain; no application call tojaraco.contextor its tarball helper was found. This is a known vulnerable dependency, not a confirmed new exploitable application path. pip-audit skipped the local development version of Recce itself because it is not published on PyPI.Recommended next maintenance bundle (15 packages; not included here)
These releases are at least seven days old and their metadata supports Python 3.10. Review resolver compatibility and release notes, particularly the pre-1.0 packages, before applying them. The security update should take priority.
Recommended separate migrations
GitHub Actions migration PR
Eleven actions have newer major versions. Update them in a dedicated CI migration, verifying runner requirements and artifact producer/consumer compatibility; keep upload/download-artifact changes together.
The pinned
actions/staleSHA matches its annotatedv10.2.0tag; its drift is to a newer major, not an incorrectly pinned existing tag. Floating major refs for setup-node, pnpm/action-setup, claude-code-action, and workflow-dispatch are on their current major.Deferred: releases younger than seven days
Recheck next cycle. Upgrade React with its DOM and type packages, Sentry as a family, and Next.js with third-parties. Biome still requires the macOS arm64 verification and guard acknowledgment.
Deferred: compatibility and requirement floors
The audit also checked
>=and~=floors. Requests, rich, openpyxl, flake8, and pyyaml already resolve to latest but have older declared floors. Review floors separately against supported dbt/Python versions; raising them is not required to update this lockfile. Recce Cloud still declares Python >=3.9, so do not copy Python-3.10-only minimums into its manifest.hatchlingis an unpinned isolated build dependency (latest 1.32.0), so no lockfile “current version” can be asserted for it.Configured ignores for
@dagrejs/dagreandhtml2canvas-prowere honored. Consider adding agithub-actionsecosystem and explicit Recce Cloud Python coverage to Dependabot; the current config covers root Python and/jsonly. Retain the existing intentional ignores.Dependabot may file new PRs after merge: packages can release between this audit and merge, and changing dependabot.yml triggers a full rescan (this PR does not change that file).
Does this PR introduce a user-facing change?:
Dependency security and maintenance updates; no intended UI or API behavior changes.