Skip to content

fix: remove leftover post_training and toolgroups CLI commands#332

Closed
cdoern wants to merge 1 commit intollamastack:mainfrom
cdoern:fix/remove-post-training-toolgroups-cli
Closed

fix: remove leftover post_training and toolgroups CLI commands#332
cdoern wants to merge 1 commit intollamastack:mainfrom
cdoern:fix/remove-post-training-toolgroups-cli

Conversation

@cdoern
Copy link
Copy Markdown
Collaborator

@cdoern cdoern commented Apr 7, 2026

Summary

  • Remove post_training CLI module which references removed post_training API types (AlgorithmConfig, TrainingConfig) and client.post_training resource
  • Remove toolgroups CLI module which references removed toolgroup_register_params type and client.toolgroups resource
  • These broken imports caused the entire CLI (llama-stack-client) to crash on any invocation

Test plan

  • Verified llama-stack-client --help runs successfully after the fix
  • Verified all remaining CLI submodule imports succeed

🤖 Generated with Claude Code

These CLI modules reference types and client resources that no longer
exist (post_training API, toolgroup_register_params, client.toolgroups),
causing the CLI to crash on any invocation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@meta-cla meta-cla bot added the cla signed label Apr 7, 2026
@cdoern cdoern closed this Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants