chore: sync dx-toolkit packages with hosted tool surface (balance, discover) - #91
Conversation
…scover) Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
|
Droid finished @EdwardIrby's task —— View job Droid review complete. Clean docs-only PR that syncs the MCP README tool surface with the hosted server, adding you-eco, you-balance, and you-discover and updating the default/non-default categorization. No findings to flag. |
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
|
Droid finished @EdwardIrby's task —— View job Droid review complete. Clean docs-only PR that syncs the MCP README tool surface with the hosted server. The capability surface (7 tools), default tool set (5 tools), and profile=free note are all internally consistent after the two commits (add eco/balance/discover, then drop eco). No code, dependency, or security changes. No findings to flag. |
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
EdwardIrby
changed the title
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
|
Droid finished @EdwardIrby's task —— View job Droid is reviewing code and running a security check… Review complete. The PR correctly syncs the hosted tool surface across packages, adding you-balance and you-discover to generated contracts, typed schemas, and most documentation. Two README code examples that claim to show "the default tools plus finance" still list the old four-tool array and are missing you-balance and you-discover, which means users following those examples would silently lose two default tools. |
![factory-droid[bot]](https://avatars.githubusercontent.com/in/358017?s=60&v=4){kind=small}
| ``` | ||
|
|
||
| `createYouClient()` returns the underlying MCP client. Call `await client.tools()` to resolve the default hosted tool set (`you-search`, `you-research`, and `you-contents`) unless you scope it with `tools`, and call `await client.close()` when finished. | ||
| `createYouClient()` returns the underlying MCP client. Call `await client.tools()` to resolve the default hosted tool set (`you-search`, `you-contents`, `you-research`, `you-balance`, and `you-discover`) unless you scope it with `tools`, and call `await client.close()` when finished. |
There was a problem hiding this comment.
[P1] "Default tools plus finance" code example is missing you-balance and you-discover
This line now defines the default tool set as you-search, you-contents, you-research, you-balance, and you-discover, but the code example around line 188 (tools: ['you-search', 'you-research', 'you-contents', 'you-finance']) was not updated to include the two new default tools. Users following that "If you want the default tools plus finance" example would inadvertently drop you-balance and you-discover from their tool set.
| ``` | ||
|
|
||
| `createYouClient()` returns the underlying MCP client. Call `await client.getTools()` to resolve the default hosted tool set (`you-search`, `you-research`, and `you-contents`) unless you scope it with `tools`, and call `await client.close()` when finished. | ||
| `createYouClient()` returns the underlying MCP client. Call `await client.getTools()` to resolve the default hosted tool set (`you-search`, `you-contents`, `you-research`, `you-balance`, and `you-discover`) unless you scope it with `tools`, and call `await client.close()` when finished. |
There was a problem hiding this comment.
[P1] "Default tools plus finance" code example is missing you-balance and you-discover
This line now defines the default tool set as you-search, you-contents, you-research, you-balance, and you-discover, but the code example around line 183 (tools: ['you-search', 'you-research', 'you-contents', 'you-finance']) was not updated to include the two new default tools. Users following that "If you want the default tools plus finance" example would inadvertently drop you-balance and you-discover from their tool set.
…t-plus-finance example Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
|
Droid finished @EdwardIrby's task —— View job Droid review complete. Clean sync PR that adds you-balance and you-discover to generated CLI contracts, typed API schemas, and all README documentation. The previously flagged issues (code examples missing the new default tools) were already addressed in this commit. No new correctness or security findings. |
|
Fixed in fef84a7. Both the langchain and ai-sdk-plugin "default tools plus finance" examples now list the full default set plus finance: |
1 participant
Summary
Syncs the dx-toolkit packages with the current hosted You.com MCP tool surface, which has added
you-balanceandyou-discover. Source of truth:youdotcom-mcp-server/src/tools.ts.Changes by package
mcp(docs)you-balanceandyou-discover.you-financedocumented as the non-default tool;you-ecointentionally not advertised.cli(code, required)TOOL_CONTRACT(src/tools.ts) viascripts/update-cli-tools.tsto addyou-balanceandyou-discover. The CLI gatesschema/execute/help on this contract, so both are required for the CLI to invoke them.surfaceVersionbumped to2026.07.23; newcontractHash.cli.spec.ts.api(code, typed DX)you-balanceto the generator allowlist (scripts/update-api-schemas.tsknownTools) and regeneratedpackages/api/src/tool-schemas.tsagainst the live server.YouBalanceInput(Record<string, unknown>) andYouBalanceOutput({ data: { attributes: { balance: number }, id, type } }) now exported frommain.ts.renderApiSchemasfixture/assertions inscripts/tests/update-api-schemas.spec.ts.you-discoverintentionally not typed (its value is dynamic external results; the API client already handles any tool via the genericcall/schema/tools).langchain+ai-sdk-plugin(docs)you-balanceandyou-discover. Both packages are tool-agnostic passthroughs; no code changes.Notes
server.jsonchanges (CI/publish workflow manages versions;server.jsonis tool-agnostic).ecois not advertised anywhere.Validation
bun --cwd packages/{cli,api,langchain,ai-sdk-plugin,mcp} checkall pass.bun --cwd packages/cli test(26 pass),bun --cwd packages/api test(10 pass).bun test scripts/tests/update-cli-tools.spec.ts scripts/tests/update-api-schemas.spec.ts(4 pass).