Skip to content

feat: add Pool column to browsers list#117

Merged
masnwilliams merged 5 commits intomainfrom
mason/pool-column-browsers-list
Feb 26, 2026
Merged

feat: add Pool column to browsers list#117
masnwilliams merged 5 commits intomainfrom
mason/pool-column-browsers-list

Conversation

@masnwilliams
Copy link
Contributor

@masnwilliams masnwilliams commented Feb 19, 2026

Summary

  • Add a "Pool" column to kernel browsers list table output showing the pool ID for browsers acquired from a pool (or "-" for regular browsers)
  • Uses SDK ExtraFields to extract pool_id until the Go SDK is regenerated with the new field

Depends on kernel/kernel#1286 for the API to return pool_id.

Test plan

  • kernel browsers list with pooled browsers shows pool ID in Pool column
  • kernel browsers list with regular browsers shows "-" in Pool column
  • kernel browsers list --output json includes pool_id for pooled browsers

Made with Cursor


Note

Low Risk
Low risk: CLI-only output change plus a minor SDK version bump; the main impact is altered table columns/order and reliance on the updated SDK response shape for Pool.

Overview
Adds a new "Pool" column to kernel browsers list table output, populating it from browser.Pool (name preferred, then ID, else -).

Bumps github.com/kernel/kernel-go-sdk from v0.39.0 to v0.40.0 (with corresponding go.sum updates) to support the new field in list responses.

Written by Cursor Bugbot for commit 835d46a. This will update automatically on new commits. Configure here.

@masnwilliams masnwilliams requested a review from Sayan- February 19, 2026 23:04
Show pool_id in the browser list table for browsers acquired from a
pool. Extracts pool_id from SDK ExtraFields until the SDK is regenerated
with the new field.

Related: kernel/kernel#1286
Co-authored-by: Cursor <cursoragent@cursor.com>
@masnwilliams masnwilliams force-pushed the mason/pool-column-browsers-list branch from f866b7c to 34d1e41 Compare February 19, 2026 23:09
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@kernel-internal
Copy link
Contributor

kernel-internal bot commented Feb 19, 2026

🔧 CI Fix Available

I've pushed a fix for the CI failure. The issue was that the replace directive in go.mod pointed to a private repository (github.com/stainless-sdks/kernel-go) that CI cannot access.

The fix removes the replace directive and uses the SDK's ExtraFields mechanism to extract pool data from the API response, which works with the public SDK.

👉 Click here to create a PR with the fix

Copy link
Contributor

@Sayan- Sayan- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changes lgtm - one suggestion. once we publish the new sdk please re-request review and we'll 🚢

masnwilliams and others added 2 commits February 24, 2026 16:43
Update browser list to display pool name (falling back to pool ID)
using the new structured pool object from the API response.

Co-authored-by: Cursor <cursoragent@cursor.com>
@masnwilliams masnwilliams requested a review from Sayan- February 24, 2026 22:14
@Sayan-
Copy link
Contributor

Sayan- commented Feb 25, 2026

changes lgtm! once we release and update the sdk dep I'll approve

Remove the replace directive pointing to github.com/stainless-sdks/kernel-go
and upgrade github.com/kernel/kernel-go-sdk from v0.37.0 to v0.40.0.

Co-authored-by: Mason Williams <masnwilliams@users.noreply.github.com>
@masnwilliams masnwilliams merged commit f22a08b into main Feb 26, 2026
2 checks passed
@masnwilliams masnwilliams deleted the mason/pool-column-browsers-list branch February 26, 2026 22:31
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.

3 participants