Skip to content

Combine agentspan cli to conductor#89

Open
kowser-orkes wants to merge 1 commit into
mainfrom
feature/combine-agentspan-cli-to-conductor-cli
Open

Combine agentspan cli to conductor#89
kowser-orkes wants to merge 1 commit into
mainfrom
feature/combine-agentspan-cli-to-conductor-cli

Conversation

@kowser-orkes

@kowser-orkes kowser-orkes commented Jul 1, 2026

Copy link
Copy Markdown

What

Folds the standalone AgentSpan CLI into the conductor binary as first‑class subcommands, talking to the single merged Conductor server. The old AgentSpan runtime on :6767 is retired and never referenced — the server comes only from --server / CONDUCTOR_SERVER_URL.

  • conductor agentcompile, delete, execution, get, init, list, prune, respond, run (SSE), status, stream.
  • conductor skilldelete, get, list, pull, register, load, run (streams), serve.

Not migrated

  • Dropped (superseded by Conductor): login / logout, configure → Conductor auth (config save, key/secret→JWT) and the existing secret store; version → root --version.
  • Name collisions (Conductor's command wins): server (start / stop / logs / ps) and update — these managed the retired :6767 runtime / JAR. AgentSpan's data-op server prune survives, rehomed as agent prune.
  • Deferred (in scope): tui, credentials (+ set / bind / bindings)

Followups

  • tui
  • credentials (+ set / bind / bindings)

- Dropped tui, credentials, login, logout
@kowser-orkes kowser-orkes requested review from bradyyie and v1r3n July 1, 2026 01:36
@mp-orkes mp-orkes self-requested a review July 3, 2026 18:12

@mp-orkes mp-orkes 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.

Do not merge to main

These commands hit /api/agent/* endpoints that neither Conductor OSS nor Orkes Conductor implement yet, and they're registered unconditionally with no capability check. In any current server every the command produces 404s with a cryptic error, exactly like conductor-oss/conductor#1197.

If Agentspan needs this, release it from its own branch. To land on main it needs capability detection or a config/feature flag plus a clear error when the server doesn't support it.

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.

2 participants