Skip to content

fix: upgrade security workflow to Node.js v22 - #67

Merged
arpayidcloud-byte merged 1 commit into
mainfrom
fix/security-workflow-nodejs
Jul 26, 2026
Merged

fix: upgrade security workflow to Node.js v22#67
arpayidcloud-byte merged 1 commit into
mainfrom
fix/security-workflow-nodejs

Conversation

@arpayidcloud-byte

Copy link
Copy Markdown
Owner

Summary

Fix security scan workflow by upgrading Node.js version to v22.

Problem

Security scan workflow (PR #66) failed with:

Error [ERR_UNKNOWN_BUILTIN_MODULE]: No such built-in module: node:sqlite

Root Cause

  • Security workflow used Node.js v20
  • node:sqlite module only available in Node.js 22+
  • Main CI workflow already uses Node.js v22 ✅

Solution

Upgrade security-scan.yml to use Node.js v22:

node-version: '22'  # Was: '20'

Compatibility

  • ✅ package.json: "engines": {"node": ">=20.0.0"}
  • ✅ Main CI: Already using Node.js v22
  • ✅ GitHub Actions: Fully supports Node.js v22
  • ✅ Codebase: No breaking changes

Impact

  • Security scans will now pass
  • OWASP Top 10 tests will run successfully
  • Daily scheduled scans will work
  • Aligned with main CI workflow

Related

- Update .github/workflows/security-scan.yml to use Node.js 22
- Required for node:sqlite module support
- Aligned with main CI workflow (already using Node 22)
- Fixes security scan failures in PR #66

Related: Phase 5.3 Security Tests (PR #66)
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@arpayidcloud-byte
arpayidcloud-byte merged commit c6ce131 into main Jul 26, 2026
1 of 2 checks passed
@arpayidcloud-byte
arpayidcloud-byte deleted the fix/security-workflow-nodejs branch July 26, 2026 16:44
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