Skip to content

feat(connectors): catalog-driven MCP OAuth for Ardot and Dida365 - #414

Open
huangcheng wants to merge 3 commits into
TencentCloud:developfrom
huangcheng:feature/ardot-remote-connector
Open

feat(connectors): catalog-driven MCP OAuth for Ardot and Dida365#414
huangcheng wants to merge 3 commits into
TencentCloud:developfrom
huangcheng:feature/ardot-remote-connector

Conversation

@huangcheng

Copy link
Copy Markdown
Contributor

Summary

  • Add catalog-driven remote MCP OAuth (oauth_issuer / mcp_url / oauth_resource / oauth_scopes) so Notion-style connectors share one DCR+PKCE + callback path.
  • Ship 腾讯设计 Ardot and 滴答清单 (dida365) with one-click OAuth, manual Bearer token paste, streamable HTTP probe, and logos.
  • Align UI/OAuth UX (auto-save after authorize, hide redundant「打开授权页」when one-click is available) and document in CHANGELOG.
  • Fix macOS pre-commit flake: Chrome runtime-dir unit test now matches non-Linux tempfile paths.

Test plan

  • uv run pytest tests/unit/test_connectors.py tests/unit/connectors/test_mcp_oauth_ssrf.py tests/integration/test_connectors_api.py -q
  • Dashboard: open Connectors → see Ardot + 滴答清单 with correct logos
  • 滴答清单 / Notion / Ardot: one-click OAuth completes and auto-saves; manual token paste still works
  • Public HTTP callback for MCP OAuth kinds returns HTTPS-required error; loopback/local OK
  • Hard-refresh Vite (:5173) or rebuilt :9000 SPA after pull

@huangcheng
huangcheng force-pushed the feature/ardot-remote-connector branch from f9b220b to fa5c1d9 Compare August 25, 2026 08:36
Unify Notion-style remote MCP OAuth behind catalog fields so new issuers
are mostly config, and add 腾讯设计 Ardot plus 滴答清单 with one-click
auth, manual token paste, and shared HTTPS callback handling.

Also align the Chrome runtime-dir unit test with non-Linux temp paths so
macOS pre-commit stays green.
@huangcheng
huangcheng force-pushed the feature/ardot-remote-connector branch from 0cae86d to ac23226 Compare August 25, 2026 09:39
huangcheng and others added 2 commits August 25, 2026 18:06
setattr(os, "getuid") fails on Windows where the attribute does not
exist; only replace it on platforms that already have getuid.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants