fix(session-ingest): exclude session_diff items from ingest getAll

[iscekic]

these are often very large and cause oom/serialization issues

[kiloconnect]

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	1
SUGGESTION	0

Issue Details (click to expand)

WARNING

File	Line	Issue
cloudflare-session-ingest/src/dos/SessionIngestDO.ts	265	SQL query selects item_id but the TypeScript type generic doesn't declare it — mismatch between query and type annotation

Files Reviewed (1 file)

• cloudflare-session-ingest/src/dos/SessionIngestDO.ts - 1 issue

Fix these issues in Kilo Cloud

[Copilot]

Pull request overview

This PR fixes OOM and serialization issues in the session ingestion system by excluding large session_diff items from the getAll() query results.

Changes:

• Modified the SQL query in SessionIngestDO.getAll() to filter out items with item_type = 'session_diff'

Comments suppressed due to low confidence (1)

cloudflare-session-ingest/src/dos/SessionIngestDO.ts:216

• The SQL string uses double quotes, which is inconsistent with the rest of the file. Other SQL exec calls in this file and in SessionAccessCacheDO.ts use either single quotes or backticks. For consistency with the existing codebase, consider using single quotes or backticks for the SQL string.

        "SELECT item_id, item_type, item_data FROM ingest_items WHERE item_type != 'session_diff' ORDER BY id"
      )

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.