## REVIEWER AUDIT - RETURNED FOR SCOPE ISSUES ### Critical Issue: Scope Creep This PR includes 12 production code/config files modified that are **NOT** in the ticket scope "Add test coverage for recent PRs". ### Production Code Changes (Should Be Separate Tickets) The following changes should have their own tickets: 1. **WebSocket Project Isolation** - agent-bridge.go, agent_channel.rb, mcp_controller.rb 2. **SolidCable Configuration** - config/cable.yml, database.yml, development.rb 3. **Agent Model Changes** - API key generation conditional 4. **Orchestrator Ping Job Refactor** - State gathering simplification 5. **Infrastructure Files** - CLAUDE.md, .dockerignore.backup, run-claude-agent.rb ### Test Changes (These Are Good ✓) The spec file changes are correct: - Updated to use `agent_memory` instead of removed `artifact`/`code_diff` models - Fixed status transitions (review → pending_approval, in_progress → todo) - Updated API endpoints from `/artifacts` to `/memories` ### Missing Requirement The ticket requested MCP controller tests for `handle_list_members` and `handle_assign_ticket` - none were added. ### Action Required 1. Split this PR: Create a new PR with ONLY spec/ changes 2. Create separate tickets for the production code changes 3. Add the missing MCP controller tests The test work is solid - it just needs to be separated from unrelated infrastructure changes.