Skip to content

feat(sdk): advance SDK dev sandbox lane to dgarson/fork#184

Merged
dgarson merged 4 commits intodgarson/forkfrom
feat/sdk-dev-sandbox
Mar 3, 2026
Merged

feat(sdk): advance SDK dev sandbox lane to dgarson/fork#184
dgarson merged 4 commits intodgarson/forkfrom
feat/sdk-dev-sandbox

Conversation

@dgarson
Copy link
Owner

@dgarson dgarson commented Feb 24, 2026

Summary

Included commit stack on current PR head

Readiness Checklist

  • Branch is pushed and up to date with intended SDK sandbox commits.
  • Prior merged prerequisite PR feat(sdk): scaffold SDK, sandbox, and CLI package boundaries #77 is included in branch history.
  • Merge conflicts with dgarson/fork resolved (mergeable=MERGEABLE).
  • CI checks on refreshed head complete and green.
  • Reviewer sign-off.

Notes

  • Refreshed with latest dgarson/fork and conflict resolution in pnpm-lock.yaml via lockfile regeneration.

@dgarson
Copy link
Owner Author

dgarson commented Feb 24, 2026

Status update (SDK Dev Sandbox lane):\n\n- ✅ Forward-progress PR is now open: #184\n- ✅ Verified expected commit stack is present (1853293, 7f20c97, 29f9447).\n- ⚠️ Current mergeability: CONFLICTING () against .\n- ⚠️ CI checks: none started yet on this PR at time of posting.\n\nRemaining blockers:\n1. Large branch drift from (branch is behind base by ~1241 commits), causing merge conflicts.\n2. CI signal unavailable until conflict/rebase strategy is applied (or workflows trigger post-update).\n\nProposed next actions:\n1. Rebase/cherry-pick SDK sandbox commits onto latest in a fresh integration branch.\n2. Push updated branch to this PR to clear conflicts.\n3. Re-run CI and triage any isolated test/workflow failures.

@dgarson
Copy link
Owner Author

dgarson commented Feb 24, 2026

Status update (SDK Dev Sandbox lane):

Remaining blockers:

  1. Large branch drift from base (feat/sdk-dev-sandbox is behind dgarson/fork by ~1241 commits), causing merge conflicts.
  2. No CI signal yet until branch is updated/rebased and workflows run.

Proposed next actions:

  1. Rebase/cherry-pick SDK sandbox commits onto latest dgarson/fork in a fresh integration branch.
  2. Push updated branch to this PR to clear conflicts.
  3. Re-run CI and triage any isolated test/workflow failures.

This commit adds the initial package scaffolding for bs-tim-6:

- @openclaw/sdk: typed client library with ToolClient, SessionClient,
  ResourceClient, and error types
- @openclaw/sandbox: local sandbox runtime with lifecycle state machine
  (idle -> starting -> ready -> busy -> terminating/failed)

Packages include:
- package.json, tsconfig, vitest config
- Core type definitions
- Runtime implementation with event streaming
- Unit tests for state machine and basic lifecycle

Coordinated with Sandy on package boundaries per spec.

Refs: bs-tim-6
* feat(cli): add sdk/sandbox quickstart verification tests

* feat(cli): harden sandbox cleanup and deterministic quickstart checks

* feat(sdk-sandbox): add fixture utilities and local dev scaffolding support

* docs(cli): publish SDK + Sandbox quickstart guide and test flow

* feat(cli): add sandbox keep-alive execution option
@dgarson dgarson force-pushed the feat/sdk-dev-sandbox branch from 29f9447 to 8cc08fd Compare February 25, 2026 23:12
@dgarson
Copy link
Owner Author

dgarson commented Feb 25, 2026

@dgarson urgent unblock pass complete. Rebased this PR by rebuilding head from latest dgarson/fork and cherry-picking only the SDK sandbox lane commits.

Included commits (new SHAs on rebased head):

Actions taken:

Remaining gates:

  • label workflow check: queued/running
  • label-issues workflow check: queued/running
  • backfill-pr-labels: skipped (expected)

No code changes beyond the three listed commits.

@dgarson
Copy link
Owner Author

dgarson commented Feb 26, 2026

Fresh readiness pass on PR #184:

Current state:

  • Mergeability: MERGEABLE
  • Merge state: UNSTABLE (due to non-complete checks)
  • Head commit: 8cc08fd7ffdae75b7c87dbf1e7c42f3a6af25a1f
  • Base commit: 9c09e52a623dff61a0ac4ec4f5b0ac7eb7bd5c9a

Exact remaining gates:

  1. label check — QUEUED
  2. label-issues check — QUEUED
  3. Reviewer approval/sign-off — not yet present

Non-blocking/expected:

No additional code change required from branch side at this time; waiting on queued checks + review.

@dgarson
Copy link
Owner Author

dgarson commented Mar 3, 2026

Readiness update (2026-03-02):

  • Merge state: MERGEABLE / CLEAN
  • Conflicts: resolved by merging latest dgarson/fork into PR head in commit 26314034d87409627d3cf3332d9c8d6ba353d31c
  • Current checks on PR head: none reported / none required

Exact remaining gate(s):

  1. Reviewer approval/sign-off

Once approved, this PR is ready to merge.

@dgarson
Copy link
Owner Author

dgarson commented Mar 3, 2026

Readiness update (2026-03-02, post-conflict-fix):

  • Merge state: MERGEABLE (conflicts cleared via 26314034d87409627d3cf3332d9c8d6ba353d31c)
  • Current checks on refreshed head are still in progress/pending:
    • docs-scope
    • secrets
    • label
    • label-issues

Exact remaining gate(s):

  1. Pending CI checks above must complete successfully
  2. Reviewer approval/sign-off

After those two gates clear, PR #184 is merge-ready.

@dgarson dgarson merged commit fe88555 into dgarson/fork Mar 3, 2026
2 of 6 checks passed
@dgarson dgarson deleted the feat/sdk-dev-sandbox branch March 3, 2026 03:10
@dgarson
Copy link
Owner Author

dgarson commented Mar 3, 2026

Final merge-readiness check (subagent):\n\n- Head SHA: 26314034d87409627d3cf3332d9c8d6ba353d31c\n- Merge conflicts: none (head includes merge from dgarson/fork)\n- CI/check status: blocked in queue, no active test failures yet\n\nCurrent checks on head:\n- queued: docs-scope, secrets, label, label-issues\n- skipped: ios, backfill-pr-labels\n\nBlocker detail:\n- Queued jobs are pinned to runner label blacksmith-16vcpu-ubuntu-2404 and currently have no assigned runner (runner_id: 0), so they are not starting.\n\nAction taken:\n- Cancelled stale queued Labeler run on older head SHA: https://github.com/dgarson/clawdbot/actions/runs/22420117293\n\nConclusion:\n- PR is code-mergeable from a conflict perspective, but not merge-ready until required queued checks actually start and pass.

@dgarson
Copy link
Owner Author

dgarson commented Mar 3, 2026

Status update (automated):

PR #184 is already merged into dgarson/fork (merged at 2026-03-03T03:10:19Z).

Exact gates observed:

  • PR state: MERGED
  • Merge commit: fe88555b72483e392ca603da730b3c51f4a52afa
  • Base branch protection on dgarson/fork: none configured (no required status checks / review gates enforced by branch protection)
  • Conflict state: none (merge completed successfully)
  • Current head-commit check runs (26314034d87409627d3cf3332d9c8d6ba353d31c):
    • queued: docs-scope, secrets, label, label-issues
    • skipped: ios, backfill-pr-labels
    • failing: none

No additional conflict/failing-check remediation was needed because the PR is already merged.

Clear signal: merge objective is satisfied for PR #184.

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