feat(registry): workflow registry system — named registries, versioning, caching#97
Merged
feat(registry): workflow registry system — named registries, versioning, caching#97
Conversation
Introduce a configurable named-registry system for distributing and running shared workflows. Registries can be GitHub repos or local directories, configured once in ~/.conductor/registries.toml and referenced by short name. - Add src/conductor/registry/ package (config, resolver, index, cache, github helpers, errors) - Add `conductor registry` CLI subcommand group (list, add, remove, set-default, update, show) - Modify run/resume/validate to accept registry refs (name[@registry][@Version]) in addition to local file paths - Remove init/templates feature (init.py, templates/, test_init.py) - Add httpx dependency for GitHub API fetching - Add 133 new tests (unit + integration + CLI) - Update README, cli-reference, design doc, and conductor skill Supersedes PR #88. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The type checker flags `ref.workflow`, `ref.registry_name`, and `ref.registry_entry` as `str | None` when passed to functions expecting non-optional args. Add assert narrowing in each branch to satisfy the type checker. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
When adding the first registry (no default configured yet), it is automatically set as the default. Subsequent registries still require --default to change the default. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
GitHub fetch helpers now automatically pick up a token from `gh auth token` when available, enabling access to private registry repos. Falls back to unauthenticated requests for public repos. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The index loader was calling httpx.get directly, bypassing the auth token from gh CLI. Refactored _load_github_index to use fetch_file_text from github.py, which adds the Bearer token for private repos. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
When `conductor registry show <name>` matches a configured registry name, it displays the registry details and its workflows instead of trying to parse the name as a workflow reference. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
`conductor registry show <workflow>` now fetches the workflow and displays its input parameters (name, type, required, default, description) in a table, plus an example run command. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Split the show UX cleanly: - `conductor registry show <registry>` — registry info + workflow list - `conductor show <workflow-ref>` — workflow details, inputs table, and example run command Both registry show and list now hint at `conductor show` for details. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Local path registries now return the source file path directly instead of copying to cache. This means edits to workflow files are reflected immediately without needing `conductor registry update`. Versioning is skipped for path registries since there's no version control. GitHub registries still use the cache as before. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The show command now displays three tables: - Inputs: name, type, required, default, description - Agents: name, type, description, routes (incl. parallel/for-each) - Outputs: field name and template expression Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
v0.2.0 is missing copilot.session.PermissionHandler, causing the SDK to be detected as unavailable. Pin to >=0.2.2 which has the required exports. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
1 participant
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.
Summary
Introduces a configurable named-registry system for distributing and running shared workflows. Supersedes #88 and addresses #78.
Full design doc:
docs/design/registry.mdWhat's new
~/.conductor/registries.tomlconductor registry {list,add,remove,set-default,update,show}conductor run qa-bot@official@1.2.3(or justqa-botfor latest from default)~/.conductor/cache/registries/, explicit versions immutable,latestre-resolved on updateindex.yamlat registry root lists workflows, descriptions, versions, and pathsWhat's removed
conductor init/conductor templatescommandssrc/conductor/cli/init.pyandsrc/conductor/templates/directoryChanges
New:
src/conductor/registry/(6 modules)config.pyregistries.toml, load/save/add/remove helpersresolver.pyname[@registry][@version], file-vs-registry detectionindex.pyindex.yamlfrom GitHub or local pathcache.pygithub.pyerrors.pyRegistryErrorextendingConductorErrorModified
src/conductor/cli/app.py—run/resume/validateaccept registry refs;registrysubcommand wired in;init/templatesremovedpyproject.toml— addedhttpx>=0.27.0docs/cli-reference.md, conductor skill docsTests
tests/test_registry/(unit + integration) andtests/test_cli/test_registry_commands.pyReference syntax
Closes #78