[PR #41] [MERGED] Improve ACP tool call visibility #107

New issue

Closed

opened 2026-02-27 10:22:15 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 10:22:15 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/mikeyobrien/ralph-orchestrator/pull/41
Author: @mikeyobrien
Created: 1/9/2026
Status: ✅ Merged
Merged: 1/9/2026
Merged by: @mikeyobrien

Base: main ← Head: acp-tool-visibility

📝 Commits (2)

8d619b0 Improve ACP tool call visibility
31170a5 Move issue 39 error tests into formatter suite

📊 Changes

9 files changed (+624 additions, -31 deletions)

View changed files

➕ AGENTS.md (+7 -0)
📝 src/ralph_orchestrator/__main__.py (+164 -0)
📝 src/ralph_orchestrator/adapters/acp.py (+306 -27)
📝 src/ralph_orchestrator/adapters/acp_models.py (+18 -4)
📝 src/ralph_orchestrator/adapters/claude.py (+21 -0)
📝 src/ralph_orchestrator/error_formatter.py (+19 -0)
📝 src/ralph_orchestrator/orchestrator.py (+16 -0)
📝 tests/test_acp_adapter.py (+42 -0)
📝 tests/test_error_formatter.py (+31 -0)

📄 Description

Summary

improve ACP tool call parsing/visibility and show tool calls in non-verbose mode
surface ACP tool metadata in console output and refine unknown tool status labels
consolidate issue #39 error handling tests into the formatter suite

Testing

source .venv/bin/activate && pytest tests/test_acp_adapter.py
source .venv/bin/activate && pytest tests/test_error_formatter.py

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/mikeyobrien/ralph-orchestrator/pull/41 **Author:** [@mikeyobrien](https://github.com/mikeyobrien) **Created:** 1/9/2026 **Status:** ✅ Merged **Merged:** 1/9/2026 **Merged by:** [@mikeyobrien](https://github.com/mikeyobrien) **Base:** `main` ← **Head:** `acp-tool-visibility` --- ### 📝 Commits (2) - [`8d619b0`](https://github.com/mikeyobrien/ralph-orchestrator/commit/8d619b01142a471aee28432954c04a75932750d3) Improve ACP tool call visibility - [`31170a5`](https://github.com/mikeyobrien/ralph-orchestrator/commit/31170a5787eb5ba1c947af8deb84272ae19a4930) Move issue 39 error tests into formatter suite ### 📊 Changes **9 files changed** (+624 additions, -31 deletions) <details> <summary>View changed files</summary> ➕ `AGENTS.md` (+7 -0) 📝 `src/ralph_orchestrator/__main__.py` (+164 -0) 📝 `src/ralph_orchestrator/adapters/acp.py` (+306 -27) 📝 `src/ralph_orchestrator/adapters/acp_models.py` (+18 -4) 📝 `src/ralph_orchestrator/adapters/claude.py` (+21 -0) 📝 `src/ralph_orchestrator/error_formatter.py` (+19 -0) 📝 `src/ralph_orchestrator/orchestrator.py` (+16 -0) 📝 `tests/test_acp_adapter.py` (+42 -0) 📝 `tests/test_error_formatter.py` (+31 -0) </details> ### 📄 Description ## Summary - improve ACP tool call parsing/visibility and show tool calls in non-verbose mode - surface ACP tool metadata in console output and refine unknown tool status labels - consolidate issue #39 error handling tests into the formatter suite ## Testing - source .venv/bin/activate && pytest tests/test_acp_adapter.py - source .venv/bin/activate && pytest tests/test_error_formatter.py --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>