Skip to content

refactor: Remove helpers orphaned by KMCP removal (v1.0.x backport) - #3599

Closed
EItanya wants to merge 1 commit into
backport/v1.0.x/03-3395from
backport/v1.0.x/04-3562
Closed

refactor: Remove helpers orphaned by KMCP removal (v1.0.x backport)#3599
EItanya wants to merge 1 commit into
backport/v1.0.x/03-3395from
backport/v1.0.x/04-3562

Conversation

@EItanya

@EItanya EItanya commented Nov 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Backports #3562 (5c490d83b) to release/v1.0.x without modifications.

Stacked on backport/v1.0.x/03-3395. Merge it after the PR below it in the stack.

Stack (merge in order, bottom first):

  1. revert: undo squashed backport #3586 on v1.0.x #3595 revert of fix: backport pre-release fixes to v1.0.x #3586
  2. fix(ci): restore upgrade tests from RC3 (v1.0.x backport) #3596 backports fix(ci): restore upgrade tests from RC3 #3558
  3. fix(ui): show when the Substrate lists were read, not a running age (v1.0.x backport) #3597 backports fix(ui): show when the Substrate lists were read, not a running age #3545
  4. feat(session): revert crashed runtimes through RevertSession (v1.0.x backport) #3598 backports feat(session): revert crashed runtimes through RevertSession #3395
  5. refactor: Remove helpers orphaned by KMCP removal (v1.0.x backport) #3599 backports refactor: Remove helpers orphaned by KMCP removal #3562
  6. refactor(api): move MCP header resolution out of v1alpha3 (v1.0.x backport) #3600 backports refactor(api): move MCP header resolution out of v1alpha3 #3563
  7. fix(controller): move a ModelConfig condition's transition time only on a status change (v1.0.x backport) #3601 backports fix(controller): move a ModelConfig condition's transition time only on a status change #3340
  8. fix(adk): defer MCP tool discovery until invocation (v1.0.x backport) #3602 backports adk/mcp: defer tool discovery until invocation #3560

🤖 Generated with Claude Code

@timflannagan @EItanya
refactor: Remove helpers orphaned by KMCP removal (#3562)
4f8dd93 
# Description

Removing the KMCP integration and the `kagent mcp` scaffolder in #3223
left the shared CLI helpers it used without callers. This removes the
Docker executor, project generator, filesystem, Kubernetes config, and
input prompt helpers, trims the kubectl executor to what `kagent
bug-report` still uses, and drops the `logging.FromLogr` adapter whose
only caller was the MCPServer catalog check.

No behavior change. The remaining `deadcode` findings are public STS
helpers and the `WithGRPCTransportCredentials` client option, which are
intentionally kept as consumer-facing extension points.

Signed-off-by: timflannagan <timflannagan@gmail.com>
(cherry picked from commit 5c490d8)
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
EItanya requested a review from a team as a code owner November 9, 2026 18:21
This was referenced Nov 9, 2026
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@EItanya
EItanya added this pull request to stack #3603 November 9, 2026 18:22
@EItanya

EItanya commented Nov 9, 2026

Copy link
Copy Markdown
Contributor Author

Closing: the team decided to leave release/v1.0.x as it is for stability, so neither the revert of #3586 nor the re-stacked backports will land.

Comment left by Claude on behalf of @EItanya

@EItanya EItanya closed this Nov 9, 2026
@EItanya
EItanya deleted the backport/v1.0.x/04-3562 branch November 10, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@EItanya @timflannagan