Skip to content

fix: remove broken /graph, /orchestrator, and /chat endpoints#137

Merged
fyxme merged 1 commit intomainfrom
fix/remove-broken-endpoints
Mar 23, 2026
Merged

fix: remove broken /graph, /orchestrator, and /chat endpoints#137
fyxme merged 1 commit intomainfrom
fix/remove-broken-endpoints

Conversation

@fyxme
Copy link
Member

@fyxme fyxme commented Mar 23, 2026

Summary

  • Remove deprecated /graph endpoints (infrastructure API client, CLI commands, MCP tools: list_cloud_accounts, search_infrastructure, get_infrastructure_graph)
  • Remove non-productionised /orchestrator endpoints (orchestrator API client, CLI commands, MCP tools: list_code_reviews, get_code_review)
  • Remove /chat/comments endpoints where backend service is not deployed (chat API client, CLI commands, chat_bridge, MCP tools: list_finding_comments, create_finding_comment)
  • Clean up generator mappings and MCP server tool registration

Test plan

  • go build ./... passes
  • go vet ./... passes
  • No remaining references to /graph/, /orchestrator/, or /chat/ in internal/ (except WebSocket dialer for interactive chat)
  • CLI builds and runs — removed commands no longer appear in nullify api --help
  • Verify no regression in existing MCP tools and CLI commands

These endpoints return 404s because their backend services are not
deployed. Remove all references to avoid exposing non-functional
MCP tools and CLI commands to users.

- /graph (deprecated): remove infrastructure API client, CLI commands,
  and MCP tools (list_cloud_accounts, search_infrastructure,
  get_infrastructure_graph)
- /orchestrator (not productionised): remove orchestrator API client,
  CLI commands, and MCP tools (list_code_reviews, get_code_review)
- /chat/comments (service not deployed): remove chat API client, CLI
  commands, chat_bridge, and MCP tools (list_finding_comments,
  create_finding_comment)
@fyxme fyxme added the type:refactor [PATCH] Refactoring or improving of existing code label Mar 23, 2026
@fyxme fyxme enabled auto-merge March 23, 2026 04:23
@fyxme fyxme added the patch Patch version updates (fixes) label Mar 23, 2026
@fyxme fyxme added this pull request to the merge queue Mar 23, 2026
Merged via the queue into main with commit 76741fe Mar 23, 2026
2 of 4 checks passed
@fyxme fyxme deleted the fix/remove-broken-endpoints branch March 23, 2026 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch Patch version updates (fixes) type:refactor [PATCH] Refactoring or improving of existing code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants