Add browser WebMCP list and invoke commands - #247
Merged
Conversation
sjmiller609
approved these changes
Sep 8, 2026
kernel-internal Bot
added a commit
that referenced
this pull request
Sep 9, 2026
Bumps kernel-go-sdk to v0.100.1-0.20260909170029-19b510c645d0 (19b510c). Fix a broken merge on this branch. Merging main brought in cmd/browsers_webmcp.go (PR #247, `browsers webmcp list` / `invoke`), which collided with the duplicate webmcp surface this branch had generated earlier (`webmcp list-tools` / `invoke-tool`). The BrowsersCmd struct ended up with two `webmcp` fields, so the branch did not compile. Keep main's reviewed implementation and drop this branch's duplicate: BrowserWebmcpService, WebmcpListTools/WebmcpInvokeTool, their input structs, the second `webmcp` command group, and its run funcs. README already documents only the `list`/`invoke` spelling. Restore the vault entitlement and limit output that a024421 removed. The SDK re-added those response fields in this range, so the CLI surfaces them again: - `org limits get`: "Max Vaults" and "Vaults Used" rows (OrgLimits.MaxVaults / .VaultsUsed). Null max_vaults renders as "unlimited". - `org entitlements`: the "Vaults" feature row and the "Max vaults" limit row (OrgEntitlementsFeatures.Vaults / OrgEntitlementsLimits.MaxVaults). - `vaults create` help: the free-plan 3-vault cap wording, matching the SDK's restored Upsert doc comment, pointing at `kernel org limits get`. The SDK also narrowed direct-to-VM browser routing back to curl/telemetry-stream/computer/playwright/process, dropping `fs` and `logs/stream`. The CLI does not override the routing subresource list, so it picks this up with no code change. Coverage analysis: full enumeration of all api.md methods against the CLI command tree found no missing commands. The 5 config-registry methods are x-cli-skip in openapi.yaml, which covers this range's only schema change (Analysis.expires_at and the new "expired" status). A field-by-field sweep of every *Params struct against every registered CLI flag found no missing flags; BrowserCurlParams.ResponseEncoding is intentionally unused because `browsers curl` streams raw bytes through the SDK's raw curl client. Tested against the live API: org limits get (shows Max Vaults / Vaults Used), org entitlements (shows Vaults / Max vaults), vaults create/list/delete (usage count returns to its prior value), browsers create/delete, browsers webmcp list, browsers fs list-files. Full `go build ./...`, `go vet ./...` and `go test ./...` pass. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
browsers webmcp list <id-or-name>with a readable table and raw response output via--jsonor-o json.browsers webmcp invokewith a required opaque tool reference, JSON-object input from a flag/file/stdin, and an optional positive timeout. Preserve JSON numbers in input and output.outcome_unknownerrors, and return non-zero for API errors, tool errors, and cancellations.Validation
go test ./...go vet ./...