Skip to content

Fix Mailbox - #198

Merged
cooper-grc merged 1 commit into
mainfrom
cg/mailbox-fix
Apr 17, 2026
Merged

Fix Mailbox#198
cooper-grc merged 1 commit into
mainfrom
cg/mailbox-fix

Conversation

@cooper-grc

@cooper-grc cooper-grc commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Fixes mailbox so it handles missing email integrations gracefully and still shows threads. Row Detail now falls back to synthetic threads when no connection is configured.

  • Bug Fixes
    • EmailThreadFetcher returns nil credentials (and logs) when GetConnection returns nil, avoiding errors when email isn’t configured.
    • Row Detail returns synthetic email threads from task outputs if connection lookup returns nil (e.g., shows gmail:thread-123 with correct thread_id).
    • Thread fetching tries both Gmail and Outlook for unqualified refs, skips missing providers, and correctly marks outbound messages.

Written for commit 5842565. Summary will update on new commits.

@github-actions

Copy link
Copy Markdown

✅ Integration Tests PASSED

Gateway: https://api.airstore.ai
Query path: /sources/gmail/unread-emails
Timestamp: 2026-04-17T19:36:08Z

I/O Smoke Tests

Test Status Latency Bytes
list_sources PASS 235ms -
list_gmail PASS 378ms -
list_query_path PASS 588ms -
read_file PASS 515ms 3,238
stat_file PASS 237ms 30,498

Compression A/B Tests

Metric Value
Files tested 9
Total raw bytes 76,271
Total strip bytes 49,358
Total raw tokens 42,790
Total strip tokens 28,493
Tokens saved 14,297
Byte reduction 35%
Token reduction 33%
Min threshold 10%
Cache consistent Yes
Per-file results
File Raw B Strip B Raw Tok Strip Tok Tok Saved Red% Raw ms Strip ms
2024-10-15_Harris-Walz_Organizing_Tea... 2,337 2,305 976 967 9 1% 348ms 235ms
2024-10-15_Stephen_King_via_KamalaHar... 2,904 2,866 1,116 1,103 13 1% 365ms 243ms
2025-07-17_KAYAK_How_to_have_everythi... 1,724 1,670 1,220 1,204 16 3% 405ms 245ms
2025-07-17_Alamo_Drafthouse_Cinema_Th... 19,047 18,224 12,228 11,704 524 4% 439ms 299ms
2025-07-17_Alamo_Drafthouse_Cinema_Th... 18,978 18,155 12,105 11,578 527 4% 442ms 311ms
2026-04-17_Charlie_Anderson_The_real_... 3,238 3,028 1,061 963 98 6% 233ms 245ms
2024-10-15_Knut_at_Sanity_Join_us_tom... 1,272 726 504 209 295 42% 362ms 237ms
2025-07-17_Lindsay_Stanley_TODAY_Conf... 1,876 1,049 657 268 389 44% 346ms 244ms
2025-07-17_NORDSTROM_RACK_FLASH_Travi... 24,895 1,335 12,923 497 12,426 94% 403ms 259ms

Charts

Overall Compression
Raw vs Compressed Bytes
Token Comparison
Reduction by File
Read Latency
I/O Latency

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

@cooper-grc
cooper-grc merged commit 5a01c7b into main Apr 17, 2026
4 checks passed
@cooper-grc
cooper-grc deleted the cg/mailbox-fix branch April 17, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants