Skip to content

fix(sandbox): correct SDK examples and retry guidance - #188

Merged
drewstone merged 2 commits into
mainfrom
fix/sandbox-sdk-contract
Oct 1, 2026
Merged

drewstone merged 2 commits into
mainfrom
fix/sandbox-sdk-contract

Conversation

@drewstone

@drewstone drewstone commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

The Sandbox quickstart and SDK reference now make every TypeScript example a separate program. Each block imports the SDK, creates its own client, validates its configuration, and initializes the resources it uses. Temporary machines and GPU leases are released in finally; reconnect examples explicitly require saved sandbox, session, and turn IDs.

Correct obsolete batch/task calls and qualify completed-turn deduplication by retained cache records. Handle the published SDK's nullable client.get() result. Task sessions clone the supplied public repository; example shell commands no longer assume missing project files. Include tsx installation and the command to run a saved example.

Validation on drew-gtr-pro:

  • All 12 TypeScript fences compile independently against the registry package @tangle-network/sandbox@0.58.8, with strict NodeNext/ES2022 checks and skipLibCheck: false. The extracted files contain no added ambient declarations or hidden setup.
  • Docs build, Next lint, formatting, harness wording checks, and whitespace checks pass. Current-head checks finished: nine successful checks and two neutral Netlify informational checks.
  • Chrome verified the hosted overview, quickstart, and SDK reference from Netlify deploy 6abd9fdae34172000859a076. Every rendered TypeScript block includes its own client and key guard; quickstart blocks include cleanup. At 400 pixels, both code pages have body and root widths of 400 pixels. The temporary viewport was reset.
  • An independent source review checked resource cleanup, cloned-Git task setup, nullable sandbox lookup, and saved-ID/retention boundaries.

The quickstart preview and SDK reference preview remain the review surfaces. The requested Sandbox documentation changes have completed source and browser review.

No authenticated funding, Sandbox provisioning, paid agent/GPU execution, or provider integration was performed for this documentation change. Compilation and rendered-page checks establish example syntax/API compatibility and presentation; they do not establish live paid execution. Prior fixture runs are not delivery proof for this revision.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@netlify

netlify Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for tangle-docs ready!

Name Link
🔨 Latest commit a07f073
🔍 Latest deploy log https://app.netlify.com/projects/tangle-docs/deploys/6abd9fdae34172000859a076
😎 Deploy Preview https://deploy-preview-188--tangle-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

tangletools
tangletools previously approved these changes Sep 30, 2026

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

✅ Auto-approved PR — 3a768aad

Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-09-30T03:18:06Z

@drewstone
drewstone marked this pull request as draft September 30, 2026 05:52
@drewstone
drewstone marked this pull request as ready for review October 1, 2026 08:27

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

✅ Auto-approved PR — a07f0732

Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-10-01T08:27:42Z

@drewstone
drewstone merged commit 8154f81 into main Oct 1, 2026
11 checks passed
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.

2 participants