test(e2e): stabilize config-read id-label test

[skevetter]

Summary

• add a minimal runtime fixture for the config-read custom ID-label integration test
• keep the feature-rich fixture for parser coverage
• remove the incidental Node feature installation path from DevsyUp

Validation

• static fixture checks passed
• CodeRabbit CLI review passed with no findings
• GitLab runner validation is expected to run on this draft PR

The local Go/E2E check was blocked by the host toolchain mismatch (Go 1.27.1 versus cached Go 1.25.5 artifacts).

Summary by CodeRabbit

• Tests
  • Updated configuration-read end-to-end testing to use dedicated fixtures for custom ID-label scenarios.
  • Added a container configuration to support consistent execution of these test scenarios.

[netlify]

✅ Deploy Preview for devsydev canceled.

Name	Link
🔨 Latest commit	1a301c1
🔍 Latest deploy log	https://app.netlify.com/projects/devsydev/deploys/6aa3b476db588800087accd8

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: cf638eea-e6d3-454b-83f1-36afb02c6d44

📥 Commits

Reviewing files that changed from the base of the PR and between 3e3d1cf and 1a301c1.

📒 Files selected for processing (2)

• e2e/tests/configread/configread.go
• e2e/tests/configread/testdata-id-label/.devcontainer/devcontainer.json

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The custom ID-label config-read test now uses a dedicated fixture directory. The directory includes a devcontainer configuration named “Config Read Custom ID Label” and based on the specified Ubuntu image.

Changes

Config-read custom ID-label test

Layer / File(s)	Summary
Dedicated fixture configuration e2e/tests/configread/configread.go, e2e/tests/configread/testdata-id-label/.devcontainer/devcontainer.json	The test copies fixtures from testdata-id-label. The fixture directory defines the custom devcontainer configuration and base image.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 1a301

The fixture split is consistent with the custom ID-label test flow and introduces no identified merge-blocking risk.

🚥 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 identifies the main change: stabilizing the end-to-end config-read test for the custom ID-label case.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch humble-porcupine

---

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

[netlify]

✅ Deploy Preview for images-devsy-sh canceled.

Name	Link
🔨 Latest commit	1a301c1
🔍 Latest deploy log	https://app.netlify.com/projects/images-devsy-sh/deploys/6aa3b476750f360008f5555c

[skevetter]

@greptileai review

[greptile-apps]

Greptile Summary

Isolates the custom ID-label integration test from incidental Node feature installation by introducing a minimal runtime fixture.

• Preserves the feature-rich fixture for parser coverage.
• Retains the existing custom-label startup and configuration-read assertions.
• No actionable issues identified. Runtime E2E validation was not performed during this review.

Confidence Score: 5/5

The PR appears safe to merge, with no actionable regressions identified.

The fixture change preserves custom ID-label coverage and existing parser coverage, while container startup does not depend on the removed Node feature.

Important Files Changed

Filename	Overview
e2e/tests/configread/configread.go	Switches only the custom ID-label integration test to the minimal fixture, preserving its assertions and other parser tests.
e2e/tests/configread/testdata-id-label/.devcontainer/devcontainer.json	Adds a name-and-image runtime fixture using the existing base image without incidental feature installation.

_{Reviews (1): Last reviewed commit: "test(e2e): isolate config-read id-label ..." | Re-trigger Greptile}

[mergify]

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

• Queue this pull request