Skip to content

ci: add Node.js and Ubuntu version matrix to test workflows#259

Merged
Mossaka merged 4 commits intomainfrom
copilot/enhance-test-matrix-node-versions
Jan 17, 2026
Merged

ci: add Node.js and Ubuntu version matrix to test workflows#259
Mossaka merged 4 commits intomainfrom
copilot/enhance-test-matrix-node-versions

Conversation

Copy link
Contributor

Copilot AI commented Jan 17, 2026

Expands CI test matrix to cover Node.js 18/22 and Ubuntu 22.04/latest, with optimization to run full matrix only on main branch pushes.

CI Changes

  • test-integration.yml: Added matrix strategy to unit-tests, test-basic-firewall, and test-robustness jobs
    • Full matrix (4 combinations) on push to main
    • Minimal matrix (ubuntu-latest + Node 22) on PRs for faster feedback
    • Artifact names include matrix values to prevent conflicts
strategy:
  fail-fast: false
  matrix:
    os: ${{ github.event_name == 'push' && fromJSON('["ubuntu-22.04", "ubuntu-latest"]') || fromJSON('["ubuntu-latest"]') }}
    node-version: ${{ github.event_name == 'push' && fromJSON('["18", "22"]') || fromJSON('["22"]') }}

Documentation

  • docs/compatibility.md: New file documenting supported Node.js versions (18+), Ubuntu versions, Docker requirements, and CI test matrix strategy
  • README.md: Added Requirements section with version info and link to compatibility docs
  • docs/quickstart.md: Expanded prerequisites with specific version requirements
Original prompt

This section details on the original issue you should resolve

<issue_title>[Long-term] Enhance test matrix with multiple Node and OS versions</issue_title>
<issue_description>## Context
From CI/CD Pipeline Gap Assessment (Discussion #227)

Description

Expand the test matrix to include multiple Node versions (18, 22) and Ubuntu versions to ensure broad compatibility.

Acceptance Criteria

  • Add Node.js 18 and 22 to test matrix
  • Add multiple Ubuntu versions (20.04, 22.04, latest)
  • Ensure tests pass on all matrix combinations
  • Document supported versions
  • Consider matrix optimization to balance coverage vs CI time

Estimated Effort

To be determined (quarterly goal)

Related

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 17, 2026 04:41
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Co-authored-by: Mossaka <5447827+Mossaka@users.noreply.github.com>
Copilot AI changed the title [WIP] Enhance test matrix with multiple Node and OS versions ci: add Node.js and Ubuntu version matrix to test workflows Jan 17, 2026
Copilot AI requested a review from Mossaka January 17, 2026 04:45
@Mossaka Mossaka marked this pull request as ready for review January 17, 2026 09:10
@github-actions
Copy link
Contributor

Test Coverage Report

Metric Coverage Covered/Total
Lines 77.19% 1300/1684
Statements 77.27% 1333/1725
Functions 77.17% 142/184
Branches 69.76% 450/645
Coverage Thresholds

The project has the following coverage thresholds configured:

  • Lines: 38%
  • Statements: 38%
  • Functions: 35%
  • Branches: 30%

Coverage report generated by `npm run test:coverage`

…matrix-node-versions

# Conflicts:
#	.github/workflows/test-integration.yml
@github-actions
Copy link
Contributor

github-actions bot commented Jan 17, 2026

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

@github-actions
Copy link
Contributor

github-actions bot commented Jan 17, 2026

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions
Copy link
Contributor

github-actions bot commented Jan 17, 2026

🎬 THE ENDSmoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨

@github-actions
Copy link
Contributor

Smoke Test Results

Last 2 Merged PRs:

  • ci: pin GitHub Actions to commit SHAs for supply chain security
  • chore: delete existing firewall tests and migrate smoke tests

Test Results:

  • ✅ GitHub MCP (fetched recent PRs)
  • ✅ Playwright (navigated to github.com, title contains "GitHub")
  • ✅ File Writing (created test file successfully)
  • ✅ Bash Tool (verified file content)

Overall Status: PASS

AI generated by Smoke Claude

@github-actions
Copy link
Contributor

Smoke Test Results

Last 2 Merged PRs:

Test Results:

  • ✅ GitHub MCP: Retrieved PR data
  • ❌ Playwright: Browser dependencies unavailable in container
  • ✅ File Writing: Created /tmp/gh-aw/agent/smoke-test-copilot-21092107551.txt
  • ✅ Bash Tool: File verified successfully
  • ✅ HTTP Test: GitHub page title contains "GitHub"

Overall Status: FAIL (Playwright test failed)

cc: @Mossaka @copilot

AI generated by Smoke Copilot

@Mossaka Mossaka merged commit b5e62d9 into main Jan 17, 2026
33 of 34 checks passed
@Mossaka Mossaka deleted the copilot/enhance-test-matrix-node-versions branch January 17, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Long-term] Enhance test matrix with multiple Node and OS versions

2 participants