Skip to content

v0.9.40: shared file streaming, paid plan for chat, workbench stop fix - #9552

Merged
waleedlatif1 merged 4 commits into
mainfrom
staging
Nov 1, 2026
Merged

waleedlatif1 merged 4 commits into
mainfrom
staging

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

…ing creation (#9549)

* fix(sandbox): initialize workbench history before honoring a stop during creation

A chat workbench created while the caller stopped kept running for the next
call to reuse, but the abort check ran before its history was initialized.
The reused machine then had no history, so it could never be certified and
every later code result in that chat was withheld.

* fix(sandbox): type the certification run helper to the fields both boundaries accept
…uffering them under 100 MB (#9550)

* fix(public-shares): stream shared files with byte ranges instead of buffering them under 100 MB

* fix(public-shares): size a streamed share from storage, audit every delivered response, cover both read paths
@vercel

vercel Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Nov 1, 2026 8:36pm UTC

Request Review

@greptile-apps

greptile-apps Bot commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 4/5

[High impact] The PR changes Chat admission across several entry points and changes public file delivery.

The PR appears safe to merge, with a non-blocking test gap in the new Chat access policy.

Findings

  1. P2 Chat access lacks coverage ▶

Summary

This PR adds three changes:

  • Chat access follows the payer’s plan or prepaid credits.
  • Public shares stream stored files and serve byte ranges.
  • Stopped workbench creation leaves history for the next call.

The new Chat policy needs boundary coverage for free payers and the prepaid-credit exception. No blocking behavior defect was established.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  Share[Public file request] --> Grant[Check current share and file]
  Grant --> Stored{Serve stored bytes?}
  Stored -->|Yes| Range[Resolve requested byte range]
  Range --> Recheck[Open storage and recheck access]
  Recheck --> Stream[Stream selected bytes]
  Stored -->|No or preview requested| Render[Build bounded representation]
  Render --> Buffer[Return buffered bytes]
  Chat[Chat turn] --> Plan{Paid plan or prepaid credits?}
  Plan -->|No| Card[Return upgrade card]
  Plan -->|Yes| Usage[Check usage limits]
  Usage --> Worker[Dispatch permitted turn]
Loading

Reviews (1) · Last reviewed commit: "feat(billing): require a paid plan or pr..." · Reviewed by Greptile

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 26 files

Turn on auto-fix | Re-trigger cubic

…at plan gate (#9553)

* fix(billing): fail closed on legacy Chat payer reads and cover the Chat plan gate

* test(billing): cover the legacy-account subscription read failing closed

* test(billing): pass the route context in the legacy-account failure case
@waleedlatif1
waleedlatif1 merged commit df7709b into main Nov 1, 2026
136 checks passed

This branch was previously deployed

1 inactive deployment
Preview — f8a561d2 Deployed Nov 1, 2026 by vercel[bot]
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.

1 participant