fix: logs resolves server names + replica handling + tests #14

Merged

michal merged 1 commits from fix/logs-resolve-and-tests into main 2026-02-23 00:12:50 +00:00

Conversation 0 Commits 1 Files Changed 2 +119 -15

michal commented 2026-02-23 00:12:48 +00:00

Owner

Copy Link

Fixes mcpctl logs to accept server names and handle replicas properly.

7 new tests covering all resolution paths.

Fixes `mcpctl logs` to accept server names and handle replicas properly. 7 new tests covering all resolution paths.

michal added 1 commit 2026-02-23 00:12:49 +00:00

fix: logs command resolves server names, proper replica handling ... dd1dfc629d

- `mcpctl logs <server-name>` resolves to first RUNNING instance
- `mcpctl logs <server-name> -i <N>` selects specific replica
- Shows "instance N/M" hint when server has multiple replicas
- Added 5 proper tests: server name resolution, RUNNING preference,
  replica selection, out-of-range error, no instances error

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

michal merged commit fa58c1b5ed into main 2026-02-23 00:12:50 +00:00

michal referenced this issue from a commit 2026-02-23 00:12:50 +00:00

Merge pull request 'fix: logs resolves server names + replica handling + tests' (#14) from fix/logs-resolve-and-tests into main

michal referenced this issue from a commit 2026-02-27 17:05:11 +00:00

Merge pull request 'fix: logs resolves server names + replica handling + tests' (#14) from fix/logs-resolve-and-tests into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

michal

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: michal/mcpctl#14