Skip to content

fix(cli): scope exchanges to the session agent#47

Merged
michiosw merged 2 commits intomainfrom
feat/cli-agent-client-context
Apr 9, 2026
Merged

fix(cli): scope exchanges to the session agent#47
michiosw merged 2 commits intomainfrom
feat/cli-agent-client-context

Conversation

@michiosw
Copy link
Copy Markdown
Contributor

@michiosw michiosw commented Apr 9, 2026

Summary

  • derive the hidden CLI agent OAuth client ID from the CreateSession agentId
  • use that client ID for provider token exchange instead of the fixed bootstrap client
  • mint an agent-scoped gateway token before creating hosted connect sessions

Testing

  • go test ./internal/run -v
  • go test ./...
  • go vet ./...

Dependency

  • depends on the backend change in kontext-dev/kontext that provisions and resolves Hydra clients for CLI agents

Open with Devin

chatgpt-codex-connector[bot]

This comment was marked as resolved.

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@michiosw michiosw force-pushed the feat/cli-agent-client-context branch from 2e03f7b to 4f16a7f Compare April 9, 2026 13:57
@michiosw michiosw merged commit 3c236bc into main Apr 9, 2026
3 checks passed
@michiosw michiosw deleted the feat/cli-agent-client-context branch April 9, 2026 14:24
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