Skip to content

Align Blaxel wiring with sibling providers - #45

Merged
Nabhag8848 merged 1 commit into
feat/blaxel-providerfrom
cursor/blaxel-wiring-fixes-7ca2
Nov 9, 2026
Merged

Nabhag8848 merged 1 commit into
feat/blaxel-providerfrom
cursor/blaxel-wiring-fixes-7ca2

Conversation

@Nabhag8848

Copy link
Copy Markdown
Member

Summary

Follow-up to the Blaxel review on #44. Compared Modal/Daytona patterns and Blaxel’s own SDKs, then only shipped fixes that reduce Go↔TS drift without inventing unsupported native APIs.

Shipped

  • Ports examples: stop using Blaxel-reserved 443/8080; use 3000/3001
  • Go floors (match TS + Modal’s memory-floor pattern): memory ≥ 256 MiB; max-age TTL ≥ 5m; idle TTL ≥ 1m
  • TS delete: return Deleted + terminal after success (same as Go Blaxel / Daytona intent)
  • Idle disable: TS now rejects like Go (fail-loud; silent omit was wrong)
  • Docs: region wording (Go requires Config.Region; TS accepts Config.region or BL_REGION); scope includes workspace; configuration setup link for Blaxel

Explicitly skipped (would add debt)

  • Go create timeout: Go Blaxel SDK NewInstance has no create-timeout option (TS @blaxel/core does). Fake-mapping via context.WithTimeout would not cancel/delete like Blaxel’s creation timeout.
  • TS standby mapping: @blaxel/core SandboxInstance exposes status but not runtime state (RUNNING/STANDBY). Reading private fields is debt; Go already maps correctly.
  • Full <ForProvider> expansion across shared guides: large permanent surface; dedicated blaxel.mdx + examples already cover Blaxel.

Test plan

  • GOWORK=off go test ./... in sdks/go/providers/blaxel
  • npm test / npm run build for @sandbox-kit/blaxel
  • Build Go + TS create-blaxel-with-ports examples
Open in Web Open in Cursor 

Fix reserved ports in examples, add Go TTL/memory floors to match TS,
mark delete as Deleted/terminal in TS like Go, reject idle-disable
consistently, and correct region/scope docs without inventing Go create
timeout or private-field standby mapping.

Co-authored-by: Nabhag Motivaras <Nabhag8848@users.noreply.github.com>
@cursor cursor Bot mentioned this pull request Nov 9, 2026
6 tasks done
@Nabhag8848
Nabhag8848 merged commit c3ef6ba into feat/blaxel-provider Nov 9, 2026
3 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