Skip to content

Update CLI agent commands for AHP 0.6#327298

Open
kmhalvin wants to merge 1 commit into
microsoft:mainfrom
kmhalvin:kmhalvin/cli-ahp-0.6-update
Open

Update CLI agent commands for AHP 0.6#327298
kmhalvin wants to merge 1 commit into
microsoft:mainfrom
kmhalvin:kmhalvin/cli-ahp-0.6-update

Conversation

@kmhalvin

@kmhalvin kmhalvin commented Jul 24, 2026

Copy link
Copy Markdown

Summary

Upgrade AHP SDK to 0.6 and update CLI for AHP 0.6 compatibility.

  • Files changed: Cargo.toml, Cargo.lock, agent.rs, agent_logs.rs, agent_ps.rs, agent_stop.rs.
  • Key code changes: updated param shapes (AuthenticateParams, SubscribeParams, ListSessionsParams), added scopes: None, adapted session snapshot fields, fixed formatting issues.
  • Dependencies: ahp and ahp-types bumped 0.4 → 0.6.
  • Verification: ran cargo check in cli — no compile errors.

Fixes: #326440

Copilot AI review requested due to automatic review settings July 24, 2026 13:57
@kmhalvin

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

Copilot AI 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.

Pull request overview

Updates the CLI’s Agent Host Protocol integration from AHP 0.4 to 0.6.

Changes:

  • Upgrades ahp and ahp-types dependencies.
  • Adapts authentication, subscription, cancellation, and session-list parameters.
  • Updates session snapshot field access.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
cli/Cargo.toml Upgrades AHP dependencies.
cli/Cargo.lock Locks AHP 0.6 packages.
cli/src/commands/agent.rs Adapts authentication parameters.
cli/src/commands/agent_logs.rs Updates subscriptions and snapshot access.
cli/src/commands/agent_ps.rs Adapts session listing, but misses pagination.
cli/src/commands/agent_stop.rs Updates subscriptions and cancellation actions.

Comment thread cli/src/commands/agent_ps.rs Outdated

Copilot AI 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.

Pull request overview

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Comment thread cli/src/commands/agent_ps.rs Outdated

Copilot AI 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.

Pull request overview

Copilot reviewed 5 out of 6 changed files in this pull request and generated no new comments.

Copilot AI 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.

Pull request overview

Copilot reviewed 5 out of 6 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants