Add gated Claw discovery and API projection#3091
Conversation
|
@giodl73-repo is attempting to deploy a commit to the OpenClaw Foundation Team on Vercel. A member of the Team first needs to authorize it. |
|
Codex review: needs real behavior proof before merge. Reviewed July 19, 2026, 10:10 PM ET / July 20, 2026, 02:10 UTC. Summary Reproducibility: not applicable. this is a new experimental discovery capability, not a report of broken established behavior. The supplied test harness covers intended paths but does not constitute a failure reproduction. Review metrics: 2 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Proof guidance:
Risk before merge
Maintainer options:
Next step before merge
Maintainer decision needed
Security Review detailsBest possible solution: Sponsor the experimental Claw registry contract as a staged feature, retain the exact Do we have a high-confidence way to reproduce the issue? Not applicable: this is a new experimental discovery capability, not a report of broken established behavior. The supplied test harness covers intended paths but does not constitute a failure reproduction. Is this the best way to solve the issue? Unclear: feature-gating and bounded summaries are a sensible safety shape, but maintainers must first confirm that this is the intended public Claw discovery contract and then require live runtime proof of that contract. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against aaa73625ed41. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
|
Restacked onto the refreshed publication slice (#3090) at |
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
241db05 to
8774028
Compare
|
@clawsweeper re-review |
8774028 to
3424d29
Compare
|
Final discovery slice is restacked at 3424d29. Disabled list/search/named reads fail closed; enabled public responses expose only the bounded safe summary and never a full parsed manifest. Public/API tests (2 files, 680 tests), schema build, ClawHub typecheck, diff checks, and the final cumulative Codex review pass. |
|
@clawsweeper re-review |
|
🦞👀 Command router queued. I will update this comment with the next step. |
e8101a8 to
c2393ea
Compare
|
Independent three-agent review fixes are applied at |
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
Exact-head CI update: the earlier Claws lint failure is fixed. |
c2393ea to
a5e8408
Compare
|
Final exact-head correction: |
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: Add gated Claw discovery and API projection This is item 1/1 in the current shard. Shard 6/8. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
Summary - Adds feature-gated Claw list, search, package-detail, and version-detail APIs. - Fails closed while
CLAWHUB_EXPERIMENTAL_CLAWSis disabled: explicit filters are rejected, unscoped results omit Claws, and named reads return not found. - Projects only boundedclawManifestSummarydata; the version endpoint returns the summary for the requested release rather than the latest release. - Reuses existing visibility, channel, moderation, scan, and viewer-access rules. This is ClawHub PR 3 of 4 for OpenClaw RFC #27 and its registry package contract. ## Stack 1. #3089: schema and storage 2. #3090: gated publication 3. #3091: gated discovery APIs 4. #3092: hosted feed and OpenClaw bridge proof ## Real behavior proof Behavior or issue addressed: Published Claws can be discovered without exposing workspace instructions, MCP environment references, cron messages, package coordinates, or a full parsed manifest. Real environment tested: Windows 11 source worktree ata5e840893dbc, Bun 1.3.14, real public package/query and HTTP API handler test harnesses. Exact steps or command run after this patch: 1.bun run test -- convex/httpApiV1.handlers.test.ts convex/packages.public.test.ts -t Claw2.bun x tsc --noEmit3.bun run lint:oxlintEvidence after fix:text Test Files 2 passed (2) Tests 25 passed | 657 skipped (682) root typecheck: passed type-aware lint: passedObserved result after fix: Disabled reads fail closed across filtered, unfiltered, and named paths. Enabled detail and version responses contain only the bounded summary associated with the selected release. What was not tested: No live deployed HTTP endpoint; tests execute the real handler/query implementations with in-memory Convex contexts. ## Validation - 25 focused Claw discovery/API tests pass. - Schema build, root and CLI typechecks, type-aware lint, format, andgit diff --checkpass. This remains draft and depends on #3090.