Repository navigation
Unify extension discovery, inline references and organization Plugin creation - #171
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Plugins, standalone Skills, Composio integrations and direct MCP servers previously had separate discovery, configuration and chat-context paths. This change provides one catalog and selection contract for chat and Agent configuration, plus organization-managed Plugin and Skill creation.
Resulting behavior
/plugins, then filter by type or connection state. Setup and account choices appear when needed. Package/provider/MCP logos use their original source, with name initials as fallback.@in chat, filter the common picker and select a reference directly into the editable text. A ready resource inserts with Enter, Tab or click; a chooser opens only when a real component/account decision is necessary. Delete and Undo work inline, and sent history keeps the highlighting. Plain typed captions grant no resource authority.Paired native client: Mobile #69.
Validation and limits
The source journals pin each earlier acceptance to the source actually tested. The earlier web/model proof, native server acceptance at
4cf80f98c, Creator E2E and search-reliability browser checks remain distinct; they do not certify a production deployment or physical devices.Current-main integration includes
9b7f8b057. On PR head5a37f1bd2, all four CodeQL analyses and the Workspace file discovery regression succeeded, with no unresolved review threads. The CodeQL security aggregation is NEUTRAL, because two Main analysis configurations are absent from its automatic baseline comparison. An independent complete comparison found the same 29 open alert IDs/rules/files/lines on PR and Main, with no additional open PR alerts. The two corrected test findings are nowfixed; the seven reviewed fingerprint findings aredismissedwith individual explanations. Search reliability, inline reliability, PI registry and the real native workspace-tree regressions also pass after integration.The full integrated production build passed with exit 0, including TypeScript, page generation and CLI version injection (
2026.10.15.2). Product sources were unchanged during that build; the final commit contains only the documented test/harness corrections. The known nonfatal Turbopack cache-compression shutdown warning is separate from the successful build result.@browser check confirms direct inline insertion, no second dialog for a ready Skill, and no picker for/or an ordinary email address. No new model message was sent in that check.The warm search p95 target of 300 ms remains open. On the final measured pre-integration build, local combined search had a 590 ms median / 1142 ms p95, and MRU search 781 ms / 1908 ms (12 samples each, shared host). The remaining fresh catalog/Core/MRU work is documented; this PR does not claim the latency target passed. Live Composio availability, external OAuth/business actions, hardware keyboards, active screen readers and real IME composition remain separate checks. The configured release/focused TypeScript checks pass; the broad legacy all-scripts profile retains its documented baseline failures.
Review paths
@on Home and in a Notebook chat. Select with keyboard or click, edit/Delete/Undo, restore a draft, send and reopen history. Confirm the inline reference and exact hidden context.Implementation/acceptance · Creator evidence · Search reliability and measurements · Native evidence
Screenshots