Skip to content

Search Composio's app list and show each picked app's sign-in status - #479

Merged
Birfy merged 1 commit into
ThinkFlowLab:mainfrom
Birfy:feat/359-composio-app-search
Oct 20, 2026
Merged

Birfy merged 1 commit into
ThinkFlowLab:mainfrom
Birfy:feat/359-composio-app-search

Conversation

@Birfy

@Birfy Birfy commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

Fixes #359

First slice only, per the Owner decision (2026-10-20, option a). The developer scaffold, catalog and publishing stay parked.

What

  • Settings → Connectors, Composio row: the typed "More apps" field and the fixed pill grid are replaced by the picked apps (pills, click to remove) and a Search Composio apps box. It searches Composio's app list and offers the results to add. Before connecting it uses the key being pasted, which the search never saves; after connecting it uses the saved key. With no key, or when the search fails, it filters the built-in common apps instead.
  • Each picked app shows Ready or Needs sign-in, both under the connected row and on the pills in Choose apps.
  • Host: connector-app-search (GET /toolkits?search=) and connector-app-status (GET /tool_router/session/{id}/toolkits). The key goes only in the x-api-key header, redirects are refused, and only { slug, name } with valid slugs comes back. Both endpoints are SHARED_BLOCKED like the other connector endpoints. An app counts as ready when it needs no auth or has an ACTIVE connected account.

Why: the owner chose option (a): replace the typed app name with a search over Composio's apps, and show each app's status.

Test

  • core/test/connectors.test.js: search and status helpers (header-only key, slug filtering, query cap, errors) and the channel endpoints (pasted key not saved, saved key, status map, GitHub refused).
  • core/test/connector-apps.test.js (new): the picker renders pills with status, search results, the common-app fallback and the error hint, plus the status list.
  • core/test/authz.test.js: the new endpoints are blocked with two members.
  • ARCH_DOCS_BASE=origin/main TZ=UTC npm run verify: all green (core 1064, social 291, tree 80, timeline 80, scripts 88, interop 37, live-e2e 16; 0 failures).
  • Not run: live Composio account or Mac instances (fetches are faked in tests).

Risk tier: T2 (estimate; the controller sets the label)

@Birfy Birfy self-assigned this Oct 20, 2026
@Birfy Birfy added agent-pr Pull request opened by an agent enhancement New feature or request area:ui Web/desktop UI and copy area:security Auth, grants, approvals, secrets labels Oct 20, 2026
@Birfy

Birfy commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator Author

Self-review: PASS
Tip SHA: cd1b7cf
Ran:

  • ARCH_DOCS_BASE=origin/main TZ=UTC npm run verify: pass, 0 failures. It covered core lint plus arch-docs OK, plugin lint, build-client --check, ds-bot-patches check, and the test suites: core 1064, social 291, tree 80, timeline 80, .github scripts 88, interop 37, live-e2e 16.
  • node .github/scripts/changelog.mjs check: 13 fragments well formed.
  • CI on the tip: tests-ok, test, plugins (social/tree/timeline), package, risk tier and reviewed tip all pass.
  • Diff checked against the owner design rules:
    • Only the confirmed slice is in it: app search plus per-app status. No scaffold or catalog.
    • The key is used only in the x-api-key header to Composio, never saved by a search, never returned, and never in chat.
    • The new endpoints are SHARED_BLOCKED.
    • Per-Bot connector access is unchanged.
    • The UI is English, with a zh entry for each new string.
      Not run:
  • A live Composio account (fetches are faked in tests).
  • Live Mac instances, the smoke bench, and production Bot Social.

@Birfy Birfy added the awaiting-approval PR is green and waits for the maintainer's approval (T2/T3) or a review PASS at its tip label Oct 20, 2026
@Birfy
Birfy added this pull request to the merge queue Oct 20, 2026
Merged via the queue into ThinkFlowLab:main with commit 1fce305 Oct 20, 2026
8 checks passed
@Birfy
Birfy deleted the feat/359-composio-app-search branch October 20, 2026 03:05
@Birfy Birfy removed awaiting-approval PR is green and waits for the maintainer's approval (T2/T3) or a review PASS at its tip agent-pr Pull request opened by an agent labels Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:security Auth, grants, approvals, secrets area:ui Web/desktop UI and copy enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Plugin and App connect: catalog, guided auth, and a fast developer scaffold (cf. Grok Bot connectors)

1 participant