Skip to content

Ian/v7 5#707

Open
ianjennings wants to merge 12 commits into
mainfrom
ian/v7-5
Open

Ian/v7 5#707
ianjennings wants to merge 12 commits into
mainfrom
ian/v7-5

Conversation

@ianjennings
Copy link
Copy Markdown
Contributor

No description provided.

- spawn-runner.sh: remove old pyautogui-ws.json reading
- setup-aws.mjs: pass instanceId alongside ip to test context
- hooks.mjs: thread instanceId from context to SDK options
- sdk.js: store instanceId, pass to agent during connect()
- agent/index.js: include instanceId in direct message to API
- sandbox.js: send instanceId in direct HTTP body, wait for runner.ready
  when API provisions credentials, construct VNC URL with port 8080
- sandbox.js: _sendAbly waits up to 30s for reconnect instead of
  failing immediately on transient disconnects (dashcam upload blip)
When API returns reply.url (E2B sandbox), the API-side command proxy
is already listening. Skip the runner.ready wait and use the URL directly.
@socket-security
Copy link
Copy Markdown

socket-security Bot commented Mar 4, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedably@​2.19.09810010098100

View full report

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 4, 2026

Test Results (Linux)

37 tests  +37   15 ✅ +15   2h 36m 4s ⏱️ + 2h 36m 4s
34 suites +34    8 💤 + 8 
 1 files   + 1   14 ❌ +14 

For more details on these failures, see this check.

Results for commit 60d47c1. ± Comparison against base commit 9c5abd5.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 4, 2026

Test Results (Windows)

37 tests    0 ✅  1h 30m 13s ⏱️
34 suites  10 💤
 1 files    27 ❌

For more details on these failures, see this check.

Results for commit 60d47c1.

♻️ This comment has been updated with latest results.

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.

1 participant