Skip to content

feat(fork): create fork demo with multiple browser agents#35

Open
mattteufel-e2b wants to merge 3 commits into
mainfrom
fork
Open

feat(fork): create fork demo with multiple browser agents#35
mattteufel-e2b wants to merge 3 commits into
mainfrom
fork

Conversation

@mattteufel-e2b

@mattteufel-e2b mattteufel-e2b commented Jul 6, 2026

Copy link
Copy Markdown
  1. add new agent fork demo
  2. update logos across surf app
  3. update logos inside the template. create minimal template in project to make changes here easier in the future.

@cla-bot cla-bot Bot added the cla-signed label Jul 6, 2026
@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
surf Ready Ready Preview, Comment Jul 6, 2026 6:38pm

Request Review

@cursor cursor 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.

Stale comment

Identified 2 net-new high-severity security issues on the current head after deduplication and validation:

  1. Server-side demo credentials are exposed to the browser.
  2. Privileged sandbox snapshot/kill operations trust caller-supplied sandbox IDs without authorization checks.

Requesting changes to address both before merge.

Open in Web View Automation 

Sent by Cursor Security Agent: Security Reviewer

Comment thread app/actions.ts Outdated
Comment thread app/actions.ts

@cursor cursor 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.

Stale comment

Comment thread app/api/chat/route.ts
Comment thread lib/fork/config.ts
@mattteufel-e2b mattteufel-e2b enabled auto-merge (squash) July 6, 2026 18:35
@ben-fornefeld

Copy link
Copy Markdown
Member

Demo credentials missing. Set DEMO_SITE_USERNAME and DEMO_SITE_PASSWORD in .env.local.

can we detect this before building such that it errors out with wrong env setup

@ben-fornefeld

Copy link
Copy Markdown
Member
Screenshot 2026-07-06 at 12 03 42 PM

"E2B SNAPSHOTS" does not tell me much as a user

Can we have the title something like: "Demo - Stateful machine forking"?

@ben-fornefeld

Copy link
Copy Markdown
Member
Screenshot 2026-07-06 at 12 05 15 PM

I feel like the title is a bit much to parse, any better ui here?

Also i am missing a "Back" button to go back to surf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants