[JamBalaya56562]

mise oci build writes the project's resolved tools into the image as /etc/mise/config.toml. A tool pinned to several versions got one name = "version" line per version. That is a duplicate key, so mise inside the container could not read its own config:

[tools]
java = ["liberica-25", "liberica-17"]
maven = "3.9.16"

became

# Auto-generated by `mise oci build`. Do not edit.
[tools]
java = "liberica-25.0.4+9"
java = "liberica-17.0.20+10"
maven = "3.9.16"

and mise tools in the container failed with Invalid TOML in config file … duplicate key.

Now each tool is written once. A tool with several versions becomes an array, in the order they were resolved, because the first one takes precedence on PATH. A tool with one version is still a string, and the keys keep the short tool name, as before.

This takes over #12732 by @nitschSB, which pr-closer closed after 7 days of failing checks. Those were GitHub API rate limits on the fork run. They are credited as co-author, and the change is theirs. They later re-opened it as #13761, keyed by the full backend name instead (core:java), and closed that one themselves. Greptile and CodeRabbit both flagged that variant: the image keeps installs under the short name's directory, so a full-name key would make mise look in the wrong place. This keeps #12732's short-name keys.

Greptile's remaining finding on #12732 was that its tests built the grouped map by hand, so the grouping step itself went untested. The new repeated_tool_versions_group_into_one_key test feeds resolved tool versions through the grouping and checks two things: a repeated version collapses, and first-seen order holds.

Validation

• Unit tests in src/oci/builder.rs: a single version renders as a string, several versions render as an ordered array, and repeated resolved versions group into one key.
• With mise 2026.10.4 in a Debian container, jq = ["1.7.1", "1.6"] and mise oci build --from scratch --no-mise embedded jq = "1.7.1" followed by jq = "1.6". The fixed build is covered by the unit tests above on CI; the end-to-end build was not run with it.

AI-assisted — Tool: Claude Code; model: anthropic/claude-opus-5-5; version: unavailable.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Improvements
  • Embedded tool configuration now groups versions by tool, preserves their order, and avoids duplicate versions. Single versions appear as a string; multiple versions appear as an ordered list.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)
• Review profile: CHILL
• Plan: Advanced
• Run ID: b7971235-4c1b-4304-a482-bdac3cf00469

📥 Commits

Reviewing files that changed from the base of the PR and between c270ec0 and a9d7f61.

📒 Files selected for processing (1)

• src/oci/builder.rs

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

The embedded mise.toml generator groups versions by backend short name, removes duplicate versions, and preserves first-seen order. It renders one distinct version as a string and multiple versions as an ordered array. Tests cover these output shapes and repeated versions.

Changes

Embedded configuration

Layer / File(s)	Summary
Group and render tool versions src/oci/builder.rs	The generator groups each tool’s distinct versions in first-seen order. It renders a single version as a string and multiple versions as an ordered array. Tests check both shapes, ordering, and duplicate removal.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: jdx

Merge Risk: ⚪ Minimal · up to a9d7f

The change is mergeable after normal checks; no actionable risk remains identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 8 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the main change: grouping a tool's multiple versions under one key in the embedded OCI configuration.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

• Fix all pre-merge checks with AI

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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

[nitschSB]

Thank you very much