fix(config): correct portsAttributes JSON struct tag typo

[skevetter]

Summary

The PortsAttributes field in DevContainerConfigBase had an incorrect JSON struct tag — portAttributes instead of portsAttributes (missing the 's'). This caused the devcontainer spec's portsAttributes property to be silently ignored during JSON unmarshaling, meaning all port attribute configuration from devcontainer.json was dropped.

Summary by CodeRabbit

• Bug Fixes
  • Fixed port attributes configuration serialization to use the correct key format for configuration file handling.

[netlify]

✅ Deploy Preview for devsydev canceled.

Name	Link
🔨 Latest commit	378291c
🔍 Latest deploy log	https://app.netlify.com/projects/devsydev/deploys/69ea3b9248099a00080fd75f

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cd75d16a-683e-4a81-a025-5ce1df1ff672

📥 Commits

Reviewing files that changed from the base of the PR and between e477cf8 and 378291c.

📒 Files selected for processing (1)

• pkg/devcontainer/config/config.go

---

📝 Walkthrough

Walkthrough

The JSON serialization tag for the PortsAttributes field in DevContainerConfigBase has been updated from portAttributes to portsAttributes in a single struct field declaration. No logic, control flow, or functional behavior was modified.

Changes

Cohort / File(s)	Summary
JSON Serialization Tag Update pkg/devcontainer/config/config.go	Updated the JSON tag for DevContainerConfigBase.PortsAttributes field from portAttributes to portsAttributes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 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 accurately describes the main change: correcting a typo in the portsAttributes JSON struct tag from 'portAttributes' to 'portsAttributes'.
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.}

---

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