claude-bot
  • Joined on 2026-08-23
claude-bot created pull request william/gitops-automation#18 2026-08-23 17:41:57 +00:00
MCP bridge: let Hermes delegate to the real Claude subscription (v2)
claude-bot commented on pull request william/gitops-automation#17 2026-08-23 17:27:53 +00:00
MCP bridge: let Hermes delegate to the real Claude subscription

All findings check out against the actual diff. Here's the review:

Correctness / Security bugs

1. Bash is not disallowed — the "read-only" claim is false (agent/src/mcpBridge.js:18-36

claude-bot deleted branch feat/mcp-claude-bridge from william/gitops-automation 2026-08-23 17:27:22 +00:00
claude-bot closed pull request william/gitops-automation#17 2026-08-23 17:27:14 +00:00
MCP bridge: let Hermes delegate to the real Claude subscription
claude-bot created pull request william/gitops-automation#17 2026-08-23 17:25:02 +00:00
MCP bridge: let Hermes delegate to the real Claude subscription
claude-bot commented on pull request william/gitops-automation#16 2026-08-23 17:10:09 +00:00
LiteLLM: drop unsupported params (fixes Hermes routing)

This is a tiny, well-scoped config change, so a full review is quick.

Correctness

  • Global scope is broader than the stated fix. drop_params: true at the top-level litellm_settings
claude-bot created pull request william/gitops-automation#16 2026-08-23 17:09:51 +00:00
LiteLLM: drop unsupported params (fixes Hermes routing)
claude-bot commented on pull request william/gitops-automation#15 2026-08-23 16:26:00 +00:00
Add Hermes web dashboard at hermes.apps.williamturner.eu (dual auth)

Summary

Correctness bugs (confirmed against diff/commit message):

  1. .env.example:50 — hash-generation command still uses Python's crypt.METHOD_SHA512, but the commit message…
claude-bot created pull request william/gitops-automation#15 2026-08-23 16:21:57 +00:00
Add Hermes web dashboard at hermes.apps.williamturner.eu (dual auth)
claude-bot created pull request william/gitops-automation#14 2026-08-23 16:16:26 +00:00
Remove claude-agent Matrix presence — Hermes only
claude-bot commented on pull request william/gitops-automation#13 2026-08-23 16:12:07 +00:00
Re-add claude-subscription LiteLLM route — confirmed working for the real CLI

Both findings check out against the actual diff and repo state. Here's the review:

Correctness bugs

**1. forward_client_headers_to_llm_api: true is set globally, affecting unrelated…

claude-bot created pull request william/gitops-automation#13 2026-08-23 16:08:48 +00:00
Re-add claude-subscription LiteLLM route — confirmed working for the real CLI
claude-bot commented on pull request william/gitops-automation#12 2026-08-23 16:04:13 +00:00
Fix cross-agent reply cascade, route Hermes through LiteLLM

Code Review: Fix cross-agent reply cascade, route Hermes through LiteLLM

Correctness bugs:

  1. docker-compose.yml:64hermes now points OPENAI_BASE_URL at `http://litellm:4000/v…
claude-bot created pull request william/gitops-automation#12 2026-08-23 16:02:21 +00:00
Fix cross-agent reply cascade, route Hermes through LiteLLM
claude-bot commented on pull request william/gitops-automation#11 2026-08-23 15:53:14 +00:00
Fix Hermes: gateway command + disable network-reachable API server

Reviewed the diff (docker-compose.yml only, +10/-5). The core fix — adding command: ["gateway", "run"] and disabling API_SERVER_ENABLED — is sound and well-justified by the commit message.…

claude-bot created pull request william/gitops-automation#11 2026-08-23 15:52:15 +00:00
Fix Hermes: gateway command + disable network-reachable API server
claude-bot commented on pull request william/gitops-automation#10 2026-08-23 15:49:45 +00:00
Add Hermes Agent as a second native Matrix presence

Review summary

Correctness bug:

  • agent/src/matrixBot.js:49 — claude-bot filters out messages addressed to Hermes (@hermes prefix) but never filters messages from Hermes's own bot…
claude-bot created pull request william/gitops-automation#10 2026-08-23 15:47:05 +00:00
Add Hermes Agent as a second native Matrix presence
claude-bot commented on pull request william/gitops-automation#9 2026-08-23 15:37:05 +00:00
claude: Add a note in CONTRIBUTING.md stating that routing is now done automatically without command prefix.

The claims in the doc check out — agent/src/router.js exists and does the chat/code_task classification, and the prior commit confirms prefixes were dropped in favor of routing through…

claude-bot pushed to claude/9b0ae7ad at william/gitops-automation 2026-08-23 15:36:47 +00:00
01580222f9 claude: Add a note in CONTRIBUTING.md stating that routing is now done automatically without command prefix.