Refactor Dockerfile and devcontainer.json

[johlju]

• Remove .NET SDK installation and update postCreateCommand for GitVersion

TODO: Make sure the devcontainer is correct. This was a quick fix to bump packages inside the container which was not able to start without this AI change.

---

This change is 

[coderabbitai]

Warning

Rate limit exceeded

@johlju has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 7 minutes and 30 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between ee78324 and 07615f8.

📒 Files selected for processing (2)

• .devcontainer/Dockerfile (1 hunks)
• .devcontainer/devcontainer.json (2 hunks)

Walkthrough

Dockerfile simplified: removed Debian/.NET/Git apt installs and moved Node selection to NODE_VERSION=24 using n/npm; adds a gitversion alias. devcontainer.json adds the dotnet feature (v8.0), removes the codecov extension, and updates postCreateCommand to install GitVersion.Tool after npm install.

Changes

Cohort / File(s)	Summary
Devcontainer config ./.devcontainer/devcontainer.json	Adds dotnet feature ghcr.io/devcontainers/features/dotnet:2 with version: "8.0" and installUsingApt: false; removes codecov.codecov extension; updates postCreateCommand to npm install && dotnet tool install --global GitVersion.Tool.
Dockerfile cleanup ./.devcontainer/Dockerfile	Replaces base image version from Node 20 → 24; consolidates Node install with NODE_VERSION=24 using n/npm; removes DEBIAN_FRONTEND handling, apt installs/removals for dotnet/Git/curl/wget/transport and GitVersion.Tool; appends a gitversion alias to /home/node/.bashrc.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant VS as VS Code Dev Containers
  participant DK as Docker Engine
  participant DF as Dockerfile Build
  participant FT as Devcontainer Feature (dotnet)
  participant CT as Container
  participant PC as postCreate

  Dev->>VS: Open project in devcontainer
  VS->>DK: Build image
  DK->>DF: Execute Dockerfile
  DF->>DF: Install Node via n/npm (NODE_VERSION=24)
  DF->>DF: Append gitversion alias to /home/node/.bashrc
  DF-->>DK: Image built (no .NET/Git/GitVersion installed)
  VS->>FT: Apply dotnet feature (ghcr.io/devcontainers/features/dotnet:2 v8.0)
  FT-->>CT: .NET SDK made available in container
  VS->>PC: Run postCreateCommand
  PC->>CT: npm install
  PC->>CT: dotnet tool install --global GitVersion.Tool
  note over CT: GitVersion.Tool available in PATH after postCreate

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Pre-merge checks

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title “Refactor Dockerfile and devcontainer.json” succinctly and accurately highlights the primary change in this pull request by referring directly to the two files that were modified, making it clear for anyone scanning the history what the main update entails.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.
Description Check	✅ Passed	The pull request description clearly outlines the removal of the .NET SDK installation and the update to the postCreateCommand for GitVersion, which directly corresponds to the modifications in both the Dockerfile and devcontainer.json, and it also references the quick package bump fix inside the container, so it is appropriately related to the changeset.

---

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.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.68%. Comparing base (85987df) to head (07615f8).
⚠️ Report is 16 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #190   +/-   ##
=======================================
  Coverage   78.68%   78.68%           
=======================================
  Files           5        5           
  Lines         122      122           
  Branches       24       24           
=======================================
  Hits           96       96           
  Misses         10       10           
  Partials       16       16           

Flag	Coverage Δ
unittests	78.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6c691b9 and fc0a7f0.

📒 Files selected for processing (2)

• .devcontainer/Dockerfile (1 hunks)
• .devcontainer/devcontainer.json (2 hunks)

[coderabbitai]

⚠️ Potential issue | 🟠 Major

Ensure GitVersion shim is runnable without relying on PATH luck. The global tool installs its shims in $HOME/.dotnet/tools, but the alias currently calls dotnet-gitversion without adding that directory to PATH, so the alias still fails in new shells.(learn.microsoft.com) Update the .bashrc snippet to add the tools directory to PATH (and point the alias at the full path) so the command always resolves.

-RUN printf "\nalias gitversion=\"dotnet-gitversion\"\n" >> /home/node/.bashrc
+RUN printf '\nexport PATH="$PATH:$HOME/.dotnet/tools"\nalias gitversion="$HOME/.dotnet/tools/dotnet-gitversion"\n' >> /home/node/.bashrc

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	# Add alias for gitversion (GitVersion will be installed in post-create command)
	RUN printf "\nalias gitversion=\"dotnet-gitversion\"\n" >> /home/node/.bashrc
	# Add alias for gitversion (GitVersion will be installed in post-create command)
	RUN printf '\nexport PATH="$PATH:$HOME/.dotnet/tools"\nalias gitversion="$HOME/.dotnet/tools/dotnet-gitversion"\n' >> /home/node/.bashrc

🤖 Prompt for AI Agents

In .devcontainer/Dockerfile around lines 21 to 22, the .bashrc alias for
gitversion assumes dotnet global tools are on PATH; update the appended snippet
to export $HOME/.dotnet/tools onto PATH and set the alias to the full shim path
so it always resolves. Concretely, append an export that prepends or appends
$HOME/.dotnet/tools to PATH (using $HOME to be portable) and update the alias to
point to $HOME/.dotnet/tools/dotnet-gitversion in the same .bashrc line sequence
so new shells can run the gitversion shim reliably.