Skip to content

Commit 7854e40

Browse files
committed
docs(tools): record why the Babysit GitHub tools are registry-only
The same branch added four user-facing GitHub tools through the full block-exposure recipe (v2 variant, tools.access, dropdown, subBlocks) and five internal ones through none of it. The distinction is deliberate — the five are called by the Pi Babysit handler via executeTool, which resolves against the registry rather than any block's access list — but nothing in CI encodes it, and `check-block-registry.ts` silently skips ids it cannot find. Worth stating because the trap is non-obvious: `GitHubV2Block` builds its access list by appending `_v2` to every entry, so adding one of these to `tools.access` without first adding a v2 variant would point the block at an id that does not exist.
1 parent 234693e commit 7854e40

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

apps/sim/tools/registry.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6286,6 +6286,14 @@ export const tools: Record<string, ToolConfig> = {
62866286
github_list_tags_v2: githubListTagsV2Tool,
62876287
github_create_pr_review: githubCreatePRReviewTool,
62886288
github_create_pr_review_v2: githubCreatePRReviewV2Tool,
6289+
/**
6290+
* Internal to the Pi Babysit handler, which calls them through `executeTool`.
6291+
* Deliberately registry-only: no `_v2` variant and no entry in the GitHub
6292+
* block's `tools.access`, unlike the user-facing tools added alongside them.
6293+
* `GitHubV2Block` derives its access list by appending `_v2` to every entry,
6294+
* so adding one of these there without first adding a v2 would point the block
6295+
* at an id that does not exist. Nothing in CI encodes that, hence this note.
6296+
*/
62896297
github_list_review_threads: githubListReviewThreadsTool,
62906298
github_reply_review_thread: githubReplyReviewThreadTool,
62916299
github_resolve_review_thread: githubResolveReviewThreadTool,
@@ -6299,6 +6307,7 @@ export const tools: Record<string, ToolConfig> = {
62996307
github_list_workflow_runs_v2: githubListWorkflowRunsV2Tool,
63006308
github_get_workflow_run: githubGetWorkflowRunTool,
63016309
github_get_workflow_run_v2: githubGetWorkflowRunV2Tool,
6310+
/** Internal to Pi Babysit — see the review-thread tools above. */
63026311
github_job_logs: githubJobLogsTool,
63036312
github_status_check_rollup: githubStatusCheckRollupTool,
63046313
github_cancel_workflow_run: githubCancelWorkflowRunTool,

0 commit comments

Comments
 (0)