Skip to content

feat(integration): Conductor as a connectable code agent - #48

Merged
justinhelmer merged 2 commits into
mainfrom
feat/conductor-connect
Aug 20, 2026
Merged

feat(integration): Conductor as a connectable code agent#48
justinhelmer merged 2 commits into
mainfrom
feat/conductor-connect

Conversation

@justinhelmer

Copy link
Copy Markdown
Contributor

The server treats conductor as a full peer code-agent executor — it is in the public catalog (comingSoon: false), the Autofix.executor union, and has a handoffToConductor automation action — but polylane integration connect never offered it, so it was only reachable via a raw api call.

  • Add conductor to the connectable types and the picker (code-agent category), with API-key instructions pointing at app.conductor.build/users/api-keys
  • Update the command description and --api-key flag help
  • Mention handoffToConductor in skill/SKILL.md
  • Test: code-agent category now asserts exactly conductor, cursor, devin, factory

🤖 Generated with Claude Code

The API and catalog have carried conductor as a full peer executor
(comingSoon: false, handoffToConductor action) but the CLI picker never
offered it. Adds the picker entry, API-key instructions, and skill doc
mention.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@claude claude Bot 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.

LGTM: clean, minimal wiring of an already-live server contract — the CLI payload {type:'conductor', workspaceId, apiKey} matches the production OpenAPI spec exactly (the codegen'd client union includes that variant verbatim).

Verified server-side in nominal: the conductor catalog entry exists with comingSoon: false (packages/integrations-catalog/index.ts:792), the connect route accepts and handles it (api-integrations .../connect.ts:721, validating the key via ConductorClient.getMe()), the executor union includes conductor (durable-autofixes/schemas/autofixes.ts:30), and handoffToConductor is dispatched (workflows/automations/actions/index.ts:194). Every claim in the PR body checks out.

Ran locally at head ebf6307: codegen, typecheck, lint all clean; 268/268 tests pass; clean-env build + --version smoke OK (SKILL.md is compiled into the bundle); integration connect --help from the built bundle shows Conductor in the picker, description, and --api-key help. CI is 3/3 green at this head.

Nit (non-blocking): the Conductor instructions don't mention the API-key prefix, while the Devin/Cursor/Factory instructions each state theirs — Polylane's docs say Conductor keys are user-scoped and start with sk_, so consider appending "The key starts with sk_." to CODE_AGENTS.conductor.instructions (fine as a follow-up).


Generated by Claude Code

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Auto-approved: Claude reviewed this PR and posted an LGTM verdict (see its review). A repo admin enabled this via the auto-approve workflow.


Generated by Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@justinhelmer
justinhelmer merged commit 775f826 into main Aug 20, 2026
3 checks passed
@justinhelmer
justinhelmer deleted the feat/conductor-connect branch August 20, 2026 00:55
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.

1 participant