Skip to content

Conversation

@Soushi888
Copy link
Contributor

@Soushi888 Soushi888 commented Aug 5, 2025

Summary

Resolves GitHub Actions build failures by upgrading Gatsby v4 to v5 for Node.js v20 compatibility.

Root Cause

The build was failing due to lmdb package compilation errors with Node.js v20. The issue was V8 API incompatibility - AccessorSignature was removed in newer V8 versions, causing native module compilation to fail.

Changes Made

  • Gatsby: v4.12.1 → v5.13.7 (Node.js v20 compatible)
  • React: v17 → v18 (modern compatibility)
  • All Gatsby plugins: Updated to v5 compatible versions
  • TypeScript: v4.6.3 → v5.3.3
  • Type definitions: Updated to match new React/Node versions
  • Removed: gatsby-image (deprecated, replaced by gatsby-plugin-image)

Test Plan

  • Local build passes (npm run build)
  • No compilation errors
  • GitHub Actions build passes (will be tested by this PR)
  • Site functionality verified

Breaking Changes

  • None expected for end users
  • Development environment requires Node.js v18+ (already in use)

Notes

  • Sass deprecation warnings are cosmetic and don't affect functionality
  • 23 npm audit vulnerabilities remain but are in dev dependencies only

- Upgrade Gatsby from v4.12.1 to v5.13.7 for Node.js v20 support
- Update all Gatsby plugins to v5 compatible versions
- Upgrade React from v17 to v18
- Update TypeScript and type definitions
- Remove deprecated gatsby-image (replaced by gatsby-plugin-image)
- Fixes lmdb V8 AccessorSignature compilation errors

Resolves GitHub Actions build failures with Node.js v20
- Enable workflow to run on pull requests to main branch
- Add conditional deployment (only deploy on main branch pushes)
- Separate build step for PR validation
- Update workflow name to reflect dual purpose
@Soushi888 Soushi888 merged commit 0d1d42b into main Aug 5, 2025
1 check passed
@Soushi888 Soushi888 deleted the fix/upgrade-gatsby-v5-node20 branch August 5, 2025 02:03
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.

2 participants