[jdx]

https://entire.io/gh/jdx/mise/trails/874

mise fmt mangled multiline inline tables (TOML 1.1). Comments inside them were hoisted out of the table, and entries after the first lost their indentation. Reported in discussion 14667.

Before, for this config:

[bootstrap.macos.launchd.agents]
# Use the 1Password SSH agent with any SSH client
SSH_AUTH_SOCK = {
  program = "/bin/sh",
  args = ["-c", "true"],
  run_at_load = true,
}

mise fmt produced:

# Use the 1Password SSH agent with any SSH client
[bootstrap.macos.launchd.agents]
SSH_AUTH_SOCK = {
  program = "/bin/sh",
args = ["-c", "true"],
run_at_load = true,
}

(the comment also moved to after the closing brace in the reporter's larger example). Now the table is left exactly as written, and the rest of the file is still formatted and sorted as before.

The cause is in taplo, which rebuilds inline tables without their newlines. mise fmt now replaces each multiline inline table with a placeholder string before running taplo and restores the original text afterwards. Single-line inline tables are formatted as before. Multiline tables are preserved verbatim rather than reformatted.

Tested with a new unit test using the reporter's config (output must equal input); cargo test --bin mise cli::fmt and cargo clippy --bin mise --all-targets -- -D warnings pass.

Thanks to @ponko2 for the report.

🤖 Generated with Claude Code

AI-assisted — Tool: Claude Code; model: anthropic/claude-sonnet-5-5; version: claude-code_2-1-299_agent.

---

Note

Low Risk
Changes only affect mise fmt output, not config loading or runtime behavior; placeholder collision is guarded by a unique prefix.

Overview
mise fmt no longer destroys multiline inline tables (TOML 1.1). taplo was collapsing them to a single line, which broke indentation and pulled comments out of the table.

The formatter now stashes each multiline inline table (and arrays that contain one) as a unique placeholder string, runs the existing taplo pass via a new format_with_taplo helper, then restores the original table text. Single-line inline tables are still normalized by taplo as before.

Unit tests assert round-trip equality for reporter-style configs, single-line tables with decor-only newlines, and multiline tables inside arrays.

^{Reviewed by Cursor Bugbot for commit ff256bd. Bugbot is set up for automated code reviews on this repo. Configure here.}

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 7 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: Repository YAML (base), Central YAML (inherited), Organization UI (inherited)
• Review profile: CHILL
• Plan: Advanced
• Run ID: 1466b9bd-f470-4250-9112-c1c84bb3056d

📥 Commits

Reviewing files that changed from the base of the PR and between 4330d64 and ff256bd.

📒 Files selected for processing: 1

• src/cli/fmt.rs

• Autofix · 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.}

[greptile-apps]

Confidence Score: 5/5

[Low impact] The PR appears safe to merge.

Summary

mise fmt preserves multiline inline tables while letting taplo format the rest of the document.

• mise fmt keeps multiline inline tables intact while formatting the rest of a config.

No actionable new issues were found. Tests were inspected but not run.

_{Reviews (2) · Last reviewed commit: "fix(fmt): stash only the table's own tex..." · Reviewed by Greptile}

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 8896e9f. Configure here.}