feat: add mcpd config volumes list command

[peteski22]

Summary

• Add mcpd config volumes list <server-name> command to list all user-supplied volume mappings for a given MCP server
• Follows the same patterns as the existing volumes set command (options pattern, cobra.ExactArgs, OutOrStdout(), sorted output)
• Uses flags.RuntimeFile dynamically in help text so users see their actual configured path

Test plan

• Unit tests for happy path (single volume, multiple sorted, empty, nil)
• Unit tests for error paths (server not found, loader error, validation)
• All existing tests pass (make test)
• Linter clean (golangci-lint run --fix -v ./cmd/config/volumes/...)

Closes #198

Summary by CodeRabbit

• New Features

  • Added a list sub-command to display volume mappings for a specified server. Shows configured volumes in a formatted list, with appropriate messaging for servers without volumes.

• Tests

  • Added comprehensive test coverage for the volumes list command, including volume retrieval, error handling, and argument validation scenarios.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 557fc34 and ebbebf3.

📒 Files selected for processing (3)

• cmd/config/volumes/cmd.go
• cmd/config/volumes/list.go
• cmd/config/volumes/list_test.go

---

Walkthrough

Implements a new "list" sub-command for the volumes CLI to display volume mappings for a specified MCP server. The command reads runtime configuration, validates server existence, and outputs volumes in a sorted, formatted manner. Sub-command registration order has been adjusted.

Changes

Cohort / File(s)	Summary
Command Registration cmd/config/volumes/cmd.go	Reorders sub-command constructor registration slice to place NewListCmd before NewSetCmd, establishing the initialisation sequence for CLI sub-commands.
List Command Implementation cmd/config/volumes/list.go	Introduces listCmd type and NewListCmd constructor that creates a Cobra command to list volume mappings for a specified MCP server. Loads runtime configuration, validates server existence, and outputs sorted volumes with appropriate formatting and error handling.
List Command Tests cmd/config/volumes/list_test.go	Adds comprehensive test suite covering command creation, volume listing scenarios (single, multiple, none), argument validation, server not-found errors, and loader error propagation.

Possibly related PRs

• feat: add mcpd config volumes set command #244: Concurrently modifies the volumes command registration in cmd/config/volumes/cmd.go to register NewSetCmd, directly related to the sub-command ordering changes in this PR.

Suggested reviewers

• agpituk
• dpoulopoulos

🚥 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 and concisely describes the main change: adding a new 'list' sub-command for the volumes CLI, which is the primary objective of this PR.
Linked Issues check	✅ Passed	The PR fully implements the requested feature from issue #198: a new 'mcpd config volumes list ' command that lists volume mappings, with comprehensive tests and error handling.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to implementing the volumes list command and its tests; no unrelated modifications to other features or systems are present.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/config-volumes-list

---

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

[javiermtorres]

Minor suggestion, otherwise lgtm

[javiermtorres]

It would be good to use string consts here and reuse them in the tests to avoid false positives, unless we want to verify the string contents themselves.