Skip to content

[#45] Add status and status details command#56

Open
Justxd22 wants to merge 1 commit intopgmoneta:mainfrom
Justxd22:issue-45-status-command
Open

[#45] Add status and status details command#56
Justxd22 wants to merge 1 commit intopgmoneta:mainfrom
Justxd22:issue-45-status-command

Conversation

@Justxd22
Copy link
Copy Markdown
Contributor

@Justxd22 Justxd22 commented Mar 3, 2026

Summary

  • Add get_status and get_status_details MCP tools using Command::STATUS (7) and Command::STATUS_DETAILS (8)
  • New files: src/client/status.rs, src/handler/status.rs
  • Add unit tests for tool registration, response parsing and field translation

Closes #45

@jesperpedersen
Copy link
Copy Markdown
Member

All these are on hold for rmcp-1.0. They require documentation and test cases

@Justxd22
Copy link
Copy Markdown
Contributor Author

Justxd22 commented Mar 7, 2026

blocked for #66!

@jesperpedersen
Copy link
Copy Markdown
Member

@Justxd22 Can you rebase ?

@jesperpedersen
Copy link
Copy Markdown
Member

@Justxd22 Look at the client interaction as well

@Justxd22 Justxd22 force-pushed the issue-45-status-command branch from 237cc98 to fee49a0 Compare April 21, 2026 22:41
@Justxd22
Copy link
Copy Markdown
Contributor Author

@jesperpedersen sorry i forgot about this PR 😅

Comment thread src/handler/status.rs Outdated
Comment thread src/handler/status.rs Outdated
Comment thread src/handler/status.rs Outdated
Comment thread src/handler/status.rs Outdated
@jesperpedersen
Copy link
Copy Markdown
Member

@Justxd22 Please, see

  • doc/manual/en/79-mcp-api.md
  • src/bin/client/client.rs
  • src/client/info.rs
  • src/llm/mod.rs
  • src/llm/openai.rs
  • tests/

@Justxd22 Justxd22 force-pushed the issue-45-status-command branch from fee49a0 to 7b06ed1 Compare April 23, 2026 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Command: Status (Details)

2 participants