Skip to content

chore(deps): Fix dependabot errors #10

Merged
LinnJS merged 1 commit intomainfrom
chore/dependabot-updates
Nov 28, 2025
Merged

chore(deps): Fix dependabot errors #10
LinnJS merged 1 commit intomainfrom
chore/dependabot-updates

Conversation

@LinnJS
Copy link
Copy Markdown
Owner

@LinnJS LinnJS commented Nov 28, 2025

Description

Consolidates all Dependabot PRs (#1-8) into a single update with necessary compatibility fixes.

GitHub Actions Updates:

  • actions/checkout v4 -> v6
  • actions/upload-artifact v4 -> v5
  • codecov/codecov-action v4 -> v5
  • github/codeql-action v3 -> v4

npm Dependency Updates:

  • happy-dom 15.x -> 20.x
  • vitest 2.x -> 4.x
  • React 18.x -> 19.x
  • eslint-plugin-react-hooks 5.x -> 7.x
  • @types/node 22.x -> 24.x
  • zod 3.x -> 4.x
  • And various other dependency updates

Compatibility Fixes:

  • Fixed React 19 JSX types for custom web components (module augmentation pattern)
  • Updated tests for React 19 boolean attribute behavior (presence-based instead of string values)
  • Fixed vitest 4.x jest-dom matchers import using expect.extend(matchers) pattern
  • Disabled react-hooks/refs rule for web component wrappers

Related Issues

Closes #1, closes #2, closes #3, closes #4, closes #5, closes #6, closes #7, closes #8

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test update

Packages Affected

  • @formkit-gov/core
  • @formkit-gov/react
  • @formkit-gov/store
  • @formkit-gov/wizard
  • @formkit-gov/openapi
  • @formkit-gov/validators
  • @formkit-gov/test-utils
  • Documentation
  • CI/CD

Checklist

Code Quality

  • My code follows the project's coding standards
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

Testing

  • I have added tests that prove my fix is effective or my feature works
  • New and existing unit tests pass locally with my changes
  • I have tested my changes in multiple browsers (if applicable)

Documentation

  • I have updated the documentation accordingly
  • I have added JSDoc comments for new public APIs
  • I have updated Storybook stories (if applicable)

Changesets

  • I have created a changeset for this PR (run pnpm changeset)
  • The changeset accurately describes the changes and their impact

Screenshots (if applicable)

N/A

Testing Instructions

  1. Run pnpm install to install updated dependencies
  2. Run pnpm test to verify all 737 tests pass
  3. Run pnpm typecheck to verify TypeScript compiles
  4. Run pnpm lint to verify no linting errors

Additional Notes

  • All 737 tests pass (589 core + 148 react)
  • No changeset needed as this is a dependency update with no user-facing API changes
  • The react-hooks/refs rule was disabled because we pass refs to VA Design System web components, which is a valid pattern that the new rule incorrectly flags

- Merge all Dependabot PRs (#1-8)
- Update GitHub Actions: checkout v6, upload-artifact v5, codecov-action v5, codeql-action v4
- Update npm dependencies: happy-dom 20.x, vitest 4.x, React 19, eslint-plugin-react-hooks 7.x
- Fix React 19 JSX types for custom web components
- Update tests for React 19 boolean attribute behavior
- Fix vitest 4.x jest-dom matchers import
- Disable react-hooks/refs rule for web component wrappers

Closes #1, closes #2, closes #3, closes #4, closes #5, closes #6, closes #7, closes #8
@LinnJS LinnJS force-pushed the chore/dependabot-updates branch from f7d1c50 to 3292046 Compare November 28, 2025 20:59
@LinnJS LinnJS changed the title Chore/dependabot updates chore(deps): Fix dependabot errors Nov 28, 2025
@LinnJS LinnJS merged commit e6228b2 into main Nov 28, 2025
7 checks passed
@LinnJS LinnJS deleted the chore/dependabot-updates branch December 1, 2025 16:06
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.

1 participant