Remove unused Node.js devcontainer feature

[danieldotnl]

Summary

• Drops ghcr.io/devcontainers/features/node:1 from .devcontainer.json.
• This project has no JS/TS sources, no package.json, and no scripts (setup, develop, lint) that invoke node/npm. The feature existed only to back the claude-code feature, which auto-installs Node 18 from NodeSource when it isn't already present (see its install.sh).
• Makes Update ghcr.io/devcontainers/features/node Docker tag to v2 - autoclosed #271 (Renovate node feature bump) obsolete — closing that in favour of this.

Test plan

• Rebuild the devcontainer from scratch and confirm it comes up cleanly.
• Verify claude --version works inside the rebuilt container (claude-code feature installs its own Node).
• Confirm scripts/setup, scripts/develop, and scripts/lint still run.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Updated development environment configuration to streamline feature management and improve consistency in the development setup process.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7968337d-360b-4da3-8965-ed98393e7f26

📥 Commits

Reviewing files that changed from the base of the PR and between 3bb56bb and 577716d.

📒 Files selected for processing (1)

• .devcontainer.json

💤 Files with no reviewable changes (1)

• .devcontainer.json

---

📝 Walkthrough

Walkthrough

The PR removes the Node.js version 24 devcontainer feature from .devcontainer.json. The github-cli and Claude features remain configured. All other devcontainer settings including the image, mounts, ports, and VS Code customizations are unchanged.

Changes

Devcontainer Configuration Update

Layer / File(s)	Summary
Remove Node.js devcontainer feature .devcontainer.json	The ghcr.io/devcontainers/features/node:1 feature with pinned version "24" is removed from the features block, leaving only github-cli and Claude features in the development environment configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A Node fell from the container's nest,
Twenty-four bid its final rest,
The dev setup streamlined, clean and light,
With CLI and Claude shining bright! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: removing the unused Node.js devcontainer feature from .devcontainer.json.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/remove-node-devcontainer-feature

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}