CedarCopilot wants to merge 1 commit into main from staging
Live on prod, 0 of 8 surfaces working, 2 days left
unknown
unknown
unknown
unknown
unknown
unknown
unknown
unknown
unknown
unknown
unknown
unknown
unknown
Libra found 8 production surfaces on prod but could not judge any of them yet. 8 surfaces had no requests at all. Libra attributed 13 failures to this change at medium confidence and is still confirming them. Libra checks hourly for 3 days after each deploy.
Not safe to merge until the Sol tool-call configuration and repository type-safety requirement are fixed. The correlation-log and rollout-identity concerns are non-blocking.
### Issue 1
apps/server/src/mastra/utils/prompt-cache-fetch.ts:690
This starts `appendLlmCallLog` and immediately discards its promise, even though that promise owns both the database update and connection cleanup. The caller can finish while the write is still pending, so execution records can be missing LLM correlation IDs when the runtime stops. Make this logging operation awaitable with suppressed failures, or register it as runtime background work.
### Issue 2
apps/server/src/services/crm/stage-classification.ts:100-101
Using the fixed `system` identity makes every stage-classification backfill share one feature-flag evaluation and cache entry. Percentage and targeted OSS rollout rules therefore cannot vary for the actual users being processed, despite the caller already having `userId` and `orgId`. Pass a real rollout identity into `classifyEventStage` and use it for the flag check.
### Issue 3
apps/server/src/mastra/agents/automation-agent.ts:256-260
This change casts the dynamic `Object.fromEntries` result to a complete record without validating that every required key exists. That violates the repository directive against using `as` assertions to silence shape errors. Construct the record with inferred keys or validate and narrow its shape instead. This repository requirement must be satisfied before merging.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
This change adds OSS-model routing and LLM-call correlation logging across agent workflows. It must not merge until Sol-routed tool workflows send the required OpenAI reasoning option and the unchecked record assertion is replaced. The correlation logging and CRM stage-classification rollout identity issues are non-blocking but should be addressed to preserve complete tracing and gradual rollout behavior.
Reviews (1) · Last reviewed commit: "Merge pull request #2835 from CedarCopil..."
| Surface | Requests | Errors | p95 | Users | Verdict |
|---|---|---|---|---|---|
| /api/trpc/admin.documents.writeGuarded | 0 → 0 | 0 → 0 (0%) | not measured | 0 | No traffic No requests recorded since this deploy. |
| cron-task-queue.processAopAutomations | 1 → 0 | 0 → 0 (0%) | 1531 ms → not measured | 0 | No traffic No requests recorded since this deploy. |
| execute_tool write-document | 0 → 0 | 0 → 0 (0%) | not measured | 0 | No traffic No requests recorded since this deploy. |
| execute_tool read-document | 0 → 0 | 0 → 0 (0%) | not measured | 0 | No traffic No requests recorded since this deploy. |
| execute_tool list-documents | 0 → 0 | 0 → 0 (0%) | not measured | 0 | No traffic No requests recorded since this deploy. |
| /api/trpc/documents.lastEditors | 0 → 0 | 0 → 0 (0%) | not measured | 0 | No traffic No requests recorded since this deploy. |
| POST /api/trpc/documents.lastEditors | 0 → 0 | 0 → 0 (0%) | not measured | 0 | No traffic No requests recorded since this deploy. |
| /api/trpc/documents.getDoc | 0 → 0 | 0 → 0 (0%) | not measured | 0 | No traffic No requests recorded since this deploy. |