Skip to content

fix: remove leftover post_training and toolgroups CLI commands#333

Merged
cdoern merged 1 commit intollamastack:mainfrom
leseb:rm-rag
Apr 7, 2026
Merged

fix: remove leftover post_training and toolgroups CLI commands#333
cdoern merged 1 commit intollamastack:mainfrom
leseb:rm-rag

Conversation

@leseb
Copy link
Copy Markdown
Contributor

@leseb leseb 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 merged commit aab810c into llamastack:main Apr 7, 2026
45 checks passed
@leseb leseb deleted the rm-rag branch April 7, 2026 14:21
@stainless-app stainless-app bot mentioned this pull request 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.

2 participants