chore(registry)!: Remove deprecated llm actions#2208
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 361d9b9f26
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| namespace: llm.openai | ||
| name: chat_completion |
There was a problem hiding this comment.
Keep deprecated llm actions until workflows are migrated
Deleting this template removes llm.openai.chat_completion from the registry immediately, but draft execution/commit paths still validate action names against the current registry index (validate_registry_action_args in tracecat/validation/service.py, called from draft run and commit routers), so any existing draft or imported workflow that still references llm.* now fails validation with “action not found” instead of running. Unless all persisted workflows are migrated in the same rollout, this is a user-facing regression that blocks editing/running legacy workflows.
Useful? React with 👍 / 👎.
Motivation
llm.*namespaces must be removed to align with the newai.actionusage.Description
packages/tracecat-registry/tracecat_registry/templates/tools/for Anthropic, Bedrock, Gemini (including Vertex variant), Ollama, and OpenAI.llm.*icon namespace mappings fromfrontend/src/components/icons.tsxand updated theUDFIconsheader comment to reflect the supportedainamespace instead ofllm.chore(registry): remove deprecated llm actions.Testing
rg -n "namespace: llm|\"llm\." frontend packages/tracecat-registry/tracecat_registry/templatesto validate no remainingllmnamespaces and it returned no hits (success).pnpm -C frontend checkwhich completed successfully with no fixes required (success).Codex Task
Summary by cubic
Removed deprecated llm.* registry action templates and frontend icon mappings to complete the migration to the ai.action namespace. This reduces confusion and aligns the codebase with current AI action usage.
Written for commit 361d9b9. Summary will update on new commits.