-
Notifications
You must be signed in to change notification settings - Fork 2.3k
docs: description required for "Add Extension" in cli - phase 2 #5635
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR standardizes documentation for MCP server extensions by consolidating CLI installation instructions into a reusable component and normalizing the casing of "Goose" to "goose" throughout the documentation. The key changes include:
- Replacing lengthy, repetitive CLI configuration steps with the
CLIExtensionInstructionscomponent across all MCP extension docs - Updating "Goose" to lowercase "goose" in descriptive text while preserving capitalization in proper names and UI elements
- Adding a
commandNoteprop toCLIExtensionInstructionsfor displaying contextual information about commands/URLs - Updating the OpenMetadata link in servers.json from the marketing page to the GitHub repository
Reviewed Changes
Copilot reviewed 32 out of 32 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
documentation/src/components/CLIExtensionInstructions.tsx |
Added commandNote prop to display contextual information for command/URL configuration step |
documentation/static/servers.json |
Updated OpenMetadata link from marketing page to GitHub repository |
Multiple documentation/docs/mcp/*.md files |
Replaced verbose CLI instructions with CLIExtensionInstructions component; standardized "goose" casing |
Comments suppressed due to low confidence (1)
documentation/src/components/CLIExtensionInstructions.tsx:165
- Inconsistent casing: 'Goose' should be 'goose' to match the standardization applied throughout this PR.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <Tabs groupId="interface"> | ||
| <TabItem value="ui" label="goose Desktop" default> | ||
| [Launch the installer](goose://extension?cmd=npx&arg=-y&arg=mcp-remote&arghttp://localhost:8585/mcp&arg=--auth-server-url&arg=http://localhost:8585/mcp&id=openmetadata&name=OpenMetadata&description=OpenMetadata%20integration) | ||
| [Launch the installer](goose://extension?cmd=npx&arg=-y&arg=mcp-remote&arg=http%3A%2F%2Flocalhost%3A8585%2Fmcp&arg=--auth-server-url%3Dhttp%3A%2F%2Flocalhost%3A8585%2Fmcp&arg=--client-id%3Dopenmetadata&arg=--verbose&arg=--clean&arg=--header&arg=Authorization%3A%24%7BAUTH_HEADER%7D&id=openmetadata&name=OpenMetadata&description=Intelligent%20data%20management%20and%20automated%20data%20operations&env=AUTH_HEADER%3DBearer%20YOUR_OPENMETADATA_PAT) |
Copilot
AI
Nov 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The deep-link URL is extremely long and difficult to read/maintain. Consider breaking this into multiple lines or extracting the URL parameters into a more maintainable format.
|
* main: (33 commits) Fix Claude Code provider to default to Auto mode (#5638) (#5642) Scheduler cleanup (#5571) Better search paths and handling of CLI providers (#5554) docs: description required for "Add Extension" in cli - phase 2 (#5635) Remove some logging (#5631) Use session IDs as task IDs for subagents instead of UUIDs (#5398) Fix the naming (#5628) fix: default tetrate model is broken, replace with haiku-4.5 (#5535) (#5587) Fetch less and use the right SHA (#5621) feat(ui): add custom macOS dock menu with New Window option (#5099) feat: remove hints from recipe prompts (#5622) docs: October 2025 Community All-Stars spotlight, Hacktoberfest edition (#5625) differentiate debug/release in cache key (#5613) Unify subrecipe and subagent execution through shared recipe pipeline (#5082) Standardize CLI argument flags and update documentation (#5516) Release 1.13.0 fix: move goosehints/AGENTS.md handling to goose, and out of developer extension (#5575) fix: add standard context menu items to prevent empty right-click menu (#5616) Bump openapi in prepare-release (#5611) docs: add access control section to Developer tutorial (#5615) ...
* main: (83 commits) silence copilot on minor text issues (block#5665) fix: disallow runaway subagent chains (block#5659) chore: remove usage of non-existent env var for log dir (block#5658) clarify agent instructions (block#5655) feat: add check-everything for unified style checks (block#5650) Show errors on failure (block#5643) custom instructions for copilot reviews (block#5646) fix: prevent repeated 404 errors when accessing deleted sessions (block#5644) Flake.nix corrected main (block#5600) fix: goose recipe list can return duplicated entries (block#5645) fix: bedrock creds refresh (block#5599) Fix Claude Code provider to default to Auto mode (block#5638) (block#5642) Scheduler cleanup (block#5571) Better search paths and handling of CLI providers (block#5554) docs: description required for "Add Extension" in cli - phase 2 (block#5635) Remove some logging (block#5631) Use session IDs as task IDs for subagents instead of UUIDs (block#5398) Fix the naming (block#5628) fix: default tetrate model is broken, replace with haiku-4.5 (block#5535) (block#5587) Fetch less and use the right SHA (block#5621) ...
…k#5635) Signed-off-by: Blair Allan <[email protected]>
Summary
This PR updates extension tutorials to standardize CLI setup instructions and migrate to components
Key Changes:
Type of Change
Testing
Manual review of steps, some install tests