Skip to content

fix: expose project agentId through the managed-agents proxy - #847

Closed
devin-ai-integration[bot] wants to merge 2 commits into
mainfrom
fix/project-public-agentid
Closed

devin-ai-integration[bot] wants to merge 2 commits into
mainfrom
fix/project-public-agentid

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

publicProject in the managed-agents proxy dropped agentId — the canonical primary-agent id the CLI reads from project responses. Since the binding fix (4259909), opencomputer link --project and link --create-project resolve project.agentId through this endpoint; with the field stripped, binding a checkout to any project fails with Project <name> has no primary agent to bind. Fresh projects hit this unconditionally: the project exists (create succeeded) but can never be bound.

Repro: opencomputer login → opencomputer link --create-project foo → CLI creates the project, then fails at bindingFor because agentId is undefined.

The proxy already reads agentId from the upstream project to synthesize the agents list — it just never returned it. This adds ...(agentId ? { agentId } : {}) to the public shape and updates the project-proxy test to expect the field.

Verified end-to-end: deployed to a dev api-edge, link --create-project → deploy → session create now completes on a fresh example.

Link to Devin session: https://app.devin.ai/sessions/1fba978da01b46b0b064847b00455303
Open in Devin Desktop: https://app.devin.ai/desktop/session/1fba978da01b46b0b064847b00455303?variant=devin
Requested by: @motatoes


Devin Review

Polylane reviews this pull request when you ask:

Review with Polylane

The CLI's project binding (oc link) reads agentId — the canonical
primary-agent id — from project responses. publicProject dropped the
field, so 'link --create-project' could never bind a fresh project.

Co-Authored-By: Mohamed <mo@digger.dev>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

Co-Authored-By: Mohamed <mo@digger.dev>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

Closing — this fix was landed independently on main as 261cb816 ("Preserve project primary agent in public API", same ...(agentId ? { agentId } : {}) in publicProject + the same test assertion). After merging main, this branch's diff is empty; nothing left to ship.

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