Skip to content

fix(nix): align devShell to Node 22 LTS#68

Merged
Jesssullivan merged 1 commit intomainfrom
fix/devshell-node22
Apr 18, 2026
Merged

fix(nix): align devShell to Node 22 LTS#68
Jesssullivan merged 1 commit intomainfrom
fix/devshell-node22

Conversation

@Jesssullivan
Copy link
Copy Markdown
Owner

Summary

  • Dev shell pinned to Node 20, now Node 22 LTS
  • Aligns with acuity-middleware (Node 22) and MassageIthaca (requires >=22)
  • CI already tests both Node 20 and 22 — this only changes the local dev shell default

Tracked

  • TIN-101 (toolchain convergence — align Nix devShells with CI toolchains)

Dev shell was pinned to Node 20 while acuity-middleware uses 22 and
MassageIthaca requires >=22. CI already tests both 20 and 22.
Aligning to Node 22 LTS reduces environment-dependent drift.

Tracked: TIN-101 (toolchain convergence).
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 18, 2026

Greptile Summary

Upgrades the Nix dev shell from Node 20 to Node 22 LTS by swapping nodejs_20 for nodejs_22 in flake.nix. This aligns the local development environment with CI (which already tests Node 22) and with sibling projects that require >=22.

Confidence Score: 5/5

Safe to merge — single-attribute swap with no logic, security, or compatibility concerns.

The change is a one-line attribute rename from nodejs_20 to nodejs_22 in the Nix dev shell. It introduces no new logic, no security surface, and aligns with an already-tested CI matrix. No issues found.

No files require special attention.

Important Files Changed

Filename Overview
flake.nix Bumps dev shell Node.js from nodejs_20 to nodejs_22; single-line attribute swap, no other changes.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Developer runs nix develop] --> B[flake.nix evaluated]
    B --> C{Node.js version}
    C -->|Before PR| D[nodejs_20 — Node 20 LTS]
    C -->|After PR| E[nodejs_22 — Node 22 LTS]
    E --> F[Matches CI matrix & sibling project requirements]
    D --> G[Misaligned with CI and sibling projects]
Loading

Reviews (1): Last reviewed commit: "fix(nix): align devShell to Node 22 LTS" | Re-trigger Greptile

@Jesssullivan Jesssullivan merged commit fa56f9a into main Apr 18, 2026
5 checks passed
@Jesssullivan Jesssullivan deleted the fix/devshell-node22 branch April 18, 2026 15: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.

1 participant