Skip to content

docs: document integration catalog subcommands#3206

Open
Quratulain-bilal wants to merge 2 commits into
github:mainfrom
Quratulain-bilal:docs/integration-catalog-subcommands
Open

docs: document integration catalog subcommands#3206
Quratulain-bilal wants to merge 2 commits into
github:mainfrom
Quratulain-bilal:docs/integration-catalog-subcommands

Conversation

@Quratulain-bilal

Copy link
Copy Markdown
Contributor

closes #3205

docs/reference/integrations.md did not mention the catalog subcommand group at all, even though it is registered in code (src/specify_cli/__init__.py:1277) and the extension/preset/workflow references all document their catalog equivalents.

this adds a Catalog Management section covering:

  • specify integration catalog list (__init__.py:2556)
  • specify integration catalog add <url> [--name] (__init__.py:2611)
  • specify integration catalog remove <index> (__init__.py:2643)
  • resolution order (env SPECKIT_INTEGRATION_CATALOG_URL -> project -> user -> built-in), matching the code in src/specify_cli/integrations/catalog.py

structure mirrors the existing Catalog Management section in docs/reference/workflows.md. docs-only change, no code touched. separate from #3174 (which covers search/info/scaffold).


note: i used an ai assistant to help investigate and write this up.

the integration reference omits the 'specify integration catalog'
subcommand group (list/add/remove) that exists in code, while the
extension, preset, and workflow references all document their catalog
equivalents. add a catalog management section matching that structure.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds missing documentation for the specify integration catalog subcommand group so the integrations reference matches the CLI’s existing catalog-management capabilities.

Changes:

  • Documented specify integration catalog list/add/remove.
  • Documented catalog resolution order (env → project → user → built-in), consistent with the integration catalog stack behavior.
Show a summary per file
File Description
docs/reference/integrations.md Adds a new “Catalog Management” section for integration catalog list/add/remove and resolution order.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 2
  • Review effort level: Low

Comment thread docs/reference/integrations.md Outdated

## Catalog Management

Integration catalogs control where `search` and `install` look for integrations. Catalogs are checked in priority order.
Comment thread docs/reference/integrations.md Outdated

@mnriem mnriem left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address Copilot feedback

- catalogs are consulted by the discovery commands (search/info), not
  install; install resolves from the built-in registry
- 'catalog list' shows project sources as removable only when configured,
  otherwise active sources are non-removable
@Quratulain-bilal

Quratulain-bilal commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

both addressed in fb697be:

changed the intro to say the discovery commands (search/info) consult catalogs, since install resolves from the built-in registry, not the catalog stack
reworded the list description so it doesn't claim built-in sources are always shown; project sources are removable only when configured, otherwise the active sources are non-removable

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Low

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.

Docs: integration reference missing catalog subcommands

3 participants