CedarCopilot wants to merge 2 commits into staging from worktree-send-it-drive-picker-excel
Libra found a regression: Picking an already linked Drive file does not create another entry.
Libra ran this behavior against the change and confirmed the check detects when it breaks.
Changed code: google-slides-manager.tsx.
Live on prod, watching, 2 days left
Libra has verdicts on 0 of 2 tracked behaviors on prod; 2 are still being checked. Libra checks hourly for 3 days after each deploy.
.xlsx in Drive is spreadsheetml.sheet, not the native Sheets type, so it needs its own entry.KB_DRIVE_MIMES (native Sheets, .xlsx, .xls, .csv). Both the settings manager and the in-chat request-file card use it, so a file picked in one place shows in the other.readDocumentTool only builds a viewer URL).pnpm run autofix:local , all green except chatHistoryList.test.ts (date-bucket assertion in code this PR does not touch)apps/mail/tests/modules/integrations/useGooglePicker.test.ts (new) , list contents, comma-joined setMimeTypes, viewer URL per type; passes alongside requestDriveFileRenderer.test.tsxNo new server behavior or logging: this is a client-side constant change, so there is no Axiom/CloudWatch signal.
.xlsx/.csv/Sheets alongside Docs and Slides, and a picked .xlsx appears in the linked list with a spreadsheet icon.🤖 Generated with Claude Code
The behavior appears safe to merge, but the changed import must satisfy the repository’s explicit absolute-import requirement first.
### Issue 1
apps/mail/modules/integrations/google-slides-manager.tsx:12-18
The expanded picker import adds `SPREADSHEET_MIMES` and `driveFileUrl` through `./use-google-picker`, but the repository requires absolute imports. This requirement must be satisfied before merging.
```suggestion
import {
KB_DRIVE_MIMES,
MIME_PRESENTATION,
SPREADSHEET_MIMES,
driveFileUrl,
useGooglePicker,
} from '@/modules/integrations/use-google-picker';
```
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.
The PR expands the shared knowledge-base Drive picker MIME list to include native Sheets, Excel and CSV files. It also adds spreadsheet icons and viewer URLs, updates the settings copy, and adds picker tests.
Reviews (1) · Last reviewed commit: "Merge origin/staging"
Libra has not measured any production surfaces for this change yet.