Skip to content

[auto-change] BUG-070: workflow-live wiki and community_change_context tools return JSON-RPC deserialize error while get_status works#562

Closed
Jonnyton wants to merge 1 commit intomainfrom
auto-change/issue-553-codex-25466144879
Closed

[auto-change] BUG-070: workflow-live wiki and community_change_context tools return JSON-RPC deserialize error while get_status works#562
Jonnyton wants to merge 1 commit intomainfrom
auto-change/issue-553-codex-25466144879

Conversation

@Jonnyton
Copy link
Copy Markdown
Owner

@Jonnyton Jonnyton commented May 6, 2026

Fixes #553

Writer family: Codex
Required checker family: Claude

Automated community-loop change produced by the subscription-backed Codex lane.

…t tools return JSON-RPC deserialize error while get_status works

Fixes #553

Writer family: Codex

Required checker family: Claude
@Jonnyton Jonnyton added checker:claude PR requires checker/reviewer from the Claude-family. writer:codex Automated PR was written by a Codex/OpenAI-family model. labels May 6, 2026
@Jonnyton
Copy link
Copy Markdown
Owner Author

Jonnyton commented May 6, 2026

Loop review policy

  • Writer family: Codex
  • Required checker family: Claude

Do not accept this PR with a same-family checker.

@Jonnyton Jonnyton added the checker:claude PR requires checker/reviewer from the Claude-family. label May 6, 2026
@Jonnyton
Copy link
Copy Markdown
Owner Author

Jonnyton commented May 6, 2026

Same-family Codex review note, not the required Claude/Cowork checker key.

I reviewed #562 from an isolated worktree. Focused verification: python -m pytest tests/test_universe_server_mcp_structured_results.py -q -> 4 passed; python -m ruff check tests/test_universe_server_mcp_structured_results.py -> clean. GitHub policy checks are green.

Design read: this is test-only. It adds direct MCP call_tool coverage for wiki and community_change_context returning both structured content and text content, which is the right regression shape for the structured-result class that BUG-070 exposed. It does not itself change runtime behavior, so merge should depend on Claude/Cowork confirming that the runtime fix already landed in the current server code and that this test meaningfully guards the failure class. If the required checker believes BUG-070 was a lower transport-level JSON-RPC serialization issue rather than the FastMCP structured-content wrapper path, send it back for a transport-level regression or live-session proof.

No merge without Claude/Cowork checker and explicit host approval naming PR #562.

@Jonnyton Jonnyton added auto-fix-reviewed Automated writer reached a final no-PR outcome; retry manually if needed. auto-fix-superseded Older auto-change PR was closed because a newer branch superseded it. labels May 8, 2026
@Jonnyton
Copy link
Copy Markdown
Owner Author

Jonnyton commented May 8, 2026

Auto-change PR closed as superseded

This stale-base PR from auto-change/issue-553-codex-25466144879 was superseded by #702 from auto-change/issue-553-codex-25572187023.
The community-loop queue should review the newer PR instead of carrying both branches.

@Jonnyton Jonnyton closed this May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-fix-reviewed Automated writer reached a final no-PR outcome; retry manually if needed. auto-fix-superseded Older auto-change PR was closed because a newer branch superseded it. checker:claude PR requires checker/reviewer from the Claude-family. writer:codex Automated PR was written by a Codex/OpenAI-family model.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG-070] workflow-live wiki and community_change_context tools return JSON-RPC deserialize error while get_status works

1 participant