Omnibus CR follow-up: 50+ fixes across recorder, providers, CI, competitive matrix

[jpr5]

Summary

Addresses all bucket (c) and (d) items from PR #122's 7-agent CR, plus correctness issues discovered during this PR's own iterative CR process (6 rounds, 4 fix cycles).

Server core

• readBody 10MB size limit (DoS surface closed)
• Control API: error detail in responses, one-shot race fix (synchronous splice), fixtures_loaded gauge updates on mutations
• normalizeCompatPath magic number replaced with readable expression
• source field ("fixture" | "proxy") added to JournalEntry type

Recorder correctness

• Crash hardening: headersSent guards, clientDisconnected tracking, encoding_format logging
• Content+toolCalls: preserve both (previously dropped content)
• Cohere v2 native detection branch in buildFixtureResponse
• Tool-call IDs preserved from 5 provider formats
• Reasoning/thinking extracted from 4 provider formats (including tool-call-only)
• Multi-block text: filter+join instead of find (Anthropic/Gemini/Bedrock)
• Thinking-only Anthropic responses handled (not error fixture)
• Empty-content responses recorded correctly (not error fixture)
• Ollama /api/generate format detection
• Streaming collapse reasoning propagated

Provider handlers

• Bedrock/Converse: ContentWithToolCallsResponse guards, ResponseOverrides wired into all builders + streaming, Converse-wrapped stream format (matches collapse parser), text_delta type field, Converse error envelope, webSearches warnings
• Cohere v2: reasoning in all builders + streaming, webSearches warnings, response_format forwarding, assistant tool_calls preservation, full ResponseOverrides (finish_reason, usage, id)
• Ollama: ContentWithToolCallsResponse, default stream:true, field validation
• Gemini: tool_call_id collision fix (shared callCounter), thought-part filtering
• All: journal source field on proxy paths, Azure embedding routing

Competitive matrix pipeline

• computeChanges matches actual HTML structure (span class="no")
• applyChanges regex matches actual td>span structure
• updateProviderCounts scoped to competitor column (not global)
• extractFeatures tightened keyword patterns
• Added mokksy/ai-mocks competitor

CI + framework

• --auto instead of --admin on fix-drift merge
• Slack secrets via env vars across 5 workflows
• Script injection prevention in notify-pr.yml (env vars + jq)
• Portable grep -oE, SLACK_WEBHOOK guards
• Router: RegExp g-flag lastIndex reset
• Jest/Vitest: save/restore pre-existing env vars, loadFixtures warnings

Test plan

• 2573/2573 tests passing (69 test files)
• TSC clean
• Prettier + ESLint clean
• 0 merge conflicts with main
• 6 CR rounds converged (R1-R4 on original, R1-R6 on additions)

🤖 Generated with Claude Code

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@copilotkit/aimock@133

commit: 903acf9