[waleedlatif1]

Summary

Attaching a fourth short page previously failed even when the combined content was small. Pages now share a 150,000-character and 450,000-byte serialized snapshot budget, allowing more sources within the existing worst-case snapshot budget. The general 100-context ceiling and per-page limits remain in place.

The composer, API contract, and headless processing use the same validator. Duplicate snapshot IDs remain rejected. User documentation no longer advertises a three-page restriction. Extension version 0.1.4 is already present in staging.

Type of Change

• Improvement

Testing

The test-only baseline at 7766f43 failed in CI 41948231723: the real packaged-extension flow expected four attachments and received three. The server cases also failed before implementation. Final CI is running for 7f19875; it verifies ten-page capture, preview/removal, exact source preservation, and separate combined character/serialized-byte rejection.

Full tests and app builds run in CI. Scoped formatting and whitespace checks passed locally. Independent cleanup reviews found no additional source changes needed.

Checklist

• Tests passed the test-audit authoring gate
• Implementation complete
• Current-head CI complete
• Both reviews clean
• Screenshot gallery updated from final CI

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
docs	Ready	Preview	Nov 5, 2026 1:42am UTC

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable issues were found.

Summary

Replaces the three-page restriction with a shared budget of 150,000 text characters and 450,000 serialized bytes.

• The composer accepts more short page snapshots per message.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Captured page snapshots] --> B[Composer budget check]
  B --> C[API contract check]
  C --> D[Server snapshot and budget checks]
  D --> E[Quoted page sources for chat]
  V[Shared budget helper] -.-> B
  V -.-> C
  V -.-> D

Loading

_{Reviews (3) · Last reviewed commit: "fix(chat): validate page snapshots befor..." · Reviewed by Greptile}

[cubic-dev-ai]

All reported issues were addressed across 5 files

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 6 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}