Repository navigation
Conversation
enyst
left a comment
There was a problem hiding this comment.
I'm an AI agent (Claude Code, based on Opus 5.5) helping Engel Nyst (@enyst) with project work.
This is a draft, so I'm leaving a comment only. The catalog mechanics are fine, but the issue needs triage and a maintainer decision before this can go further.
- Triage. #598 has no labels, so the description gate will fail once this PR is ready for review. A maintainer needs to decide on the request first. The PR template has also changed since you opened this PR (#698), so the description now needs
HUMAN:andAGENT:sections. - Fit (maintainer decision). This is a paid, credit-based hosted search/scrape service. Its docs repo (
chaitin/baizhi-agent-toolkit, created 2026-09-16) is only in Chinese, and it overlaps existing search/scrape entries (Brave, Exa, Tavily, Firecrawl). I'd like a maintainer to weigh in before more work goes into this. - Endpoint not verified. The upstream README documents
https://agent-toolkit.app.baizhi.cloud/mcpwithAuthorization: Bearer, which matches the entry. Unauthenticated HEAD and GET requests to that path return 404, with no 401/405 and no auth challenge. I didn't send MCP requests, so please confirm this path is the Streamable HTTP endpoint as it is served today. - Test (tests/test_catalogs.py#L205). The new test restates the JSON field by field, including exact strings from
notesandcredentialHelp. That means any later copy edit to the entry also needs a test change, which works against the AGENTS.md rule that an integration is one JSON file. The schema tests already cover the entry, so consider dropping this test or reducing it to a single behavioural assertion.
Checked:
- The JSON validates against main's schema, and
catalog-index.jsmatches the build script's output. The PR merges cleanly, and main's catalog tests pass with the entry (117). - The app and docs URLs return 200.
- The required
test,sync-extensionsandvalidate-claude-codechecks haven't run on this fork head.
|
Removed the provider-specific mirror test and normally merged current main in The earlier actual browser/backend tests used synthetic MCP responses; I cannot claim a current authenticated production endpoint check. I am keeping this Draft pending the maintainer fit/readiness decision and the human author's HUMAN section. No live service/model calls or human declarations were made. AI-assisted follow-up. |
Why
Make Baizhi Agent Toolkit discoverable through the existing OpenHands MCP integration catalog, with a fixed remote endpoint and the host's existing API-key input. This is a catalog contribution, not a new MCP client, local proxy, model provider, or application-specific implementation.
I maintain the Baizhi Cloud Agent Toolkit integrations and coordinate product operations, and am contributing through
ct-jarynon behalf of Baizhi Cloud. We would welcome your feedback on catalog suitability and ongoing compatibility.Summary
baizhi-agent-toolkitJSON entry using Streamable HTTP (shttp) and requiredbearerauthentication, with console/docs links and account, credit, and tool-scope guidance.main, preserving its Granola entry; Python consumes the same JSON source.Issue Number
Refs #598. Please triage the request under the repository's enhancement/readiness process. This PR remains a draft; submission is not a claim that the issue has received maintainer approval or the
ready-for-devlabel.The readiness bot reports that the issue needs its
enhancementclassification. My account cannot add that label (AddLabelsToLabelablepermission denied), so maintainer triage is needed; the local body check passes when evaluated as an enhancement.How to Test
From this branch, using the repository's documented test environment:
npm run build:integrations uv sync --group test uv run pytest -q tests/test_catalogs.py tests/test_catalog_schema.py tests/test_integration_catalog_in_sync.pyRevalidated after merging current upstream
main@76bbae25b6bb4ab6dd1623617e4d373c199df58finto PR head6fe09cb0c5142dbf94c7d9584bd2f67a64803afd(2026-09-23):uvwas not available locally; theuvsetup above is the repository's reproduction route, not a command claimed to have been run.react-test-rendereris deprecated and Node's VM module support is experimental; these harness limitations remain disclosed.For the installation check, use a supported Agent Canvas development build consuming this catalog revision:
Bearer, and explicitly install. Confirm successful discovery on the actual backend, then inspect the tool list before use. A Cloud-side synthetic success result is not connection evidence.The install-modal source was reviewed at
OpenHands/OpenHands@9737f713616a1e452f822c2967f0e2c8bf2dc308: it provides a password field, required-key validation, fixed-URL handling, and explicit submission for this catalog shape. The complete Agent Canvas UI flow, credential persistence, production MCP connection, and tool execution have not been run for this contribution. No real API key, model request, or paid tool was used. The full repository test suite was not run.Video/Screenshots
Not supplied: the UI acceptance checks above are still pending. The human-test checkbox is intentionally unchecked.
Notes
Baizhi is a hosted commercial service requiring each user's own account and API key. Tool inputs are sent to the hosted service; users should review their data-handling requirements and service costs before connecting. The linked integration repository publishes configuration, documentation, and tests, not the hosted backend source.
The entry does not set a popularity rank, enable tools for existing users, add automatic approval, or duplicate the credential into an automation secret. The existing host owns credential persistence; this PR does not claim that a masked field proves a particular encryption or storage guarantee.
Landing in this repository is distinct from publishing a package and an Agent Canvas release consuming it. We are happy to maintain the entry and complete compatibility checks with reviewers. This proposal does not imply OpenHands endorsement or an established partnership.
AGENT:
Review follow-up (2026-10-08): merged current main
d008b81c44ae4d56d319e3b8f28ca35fc40aacbcnormally and removed the field-by-field Baizhi test, as requested. Final headbea2aed8d518466b20f611fb380d03c9d702e690differs from that main only in the hand-authored integration JSON and its generated JS index. The JSON is unchanged from the entry used in the earlier runtime evidence.npm run build:integrations, the three catalog test modules above (117 passed),python scripts/sync_extensions.py --check, and diff checks pass on this final tree (Python 3.12.14, pytest 9.1.1, Pydantic 2.13.5, jsonschema 4.26.0; installed SDK 1.53.0). These are local results, not fork CI authorization.Earlier, on 2026-10-01, I exercised the exact entry through released Canvas 1.24.0, TypeScript SDK 1.49.6, and Agent Server 1.50.1 in an isolated actual browser/backend round trip: empty/401, install/reload, masked credential retention, credential replacement and deletion. MCP network responses were synthetic, with zero model or tools/call requests; stored values were synthetic and services were stopped. This does not verify today's production endpoint. The upstream documentation names
/mcpwith Bearer authentication; I have not made an authenticated production MCP request. Unauthenticated HEAD/GET alone do not establish Streamable HTTP behavior.This remains Draft, awaiting a maintainer decision on fit and #598 triage/readiness, authorized fork checks, and the human author's current-template HUMAN note. I have not filled or checked any human declaration.