Skip to content

chore(registry)!: Remove deprecated llm actions#2208

Merged
topher-lo merged 1 commit intomainfrom
codex/remove-all-llm-udfs-from-codebase
Feb 27, 2026
Merged

chore(registry)!: Remove deprecated llm actions#2208
topher-lo merged 1 commit intomainfrom
codex/remove-all-llm-udfs-from-codebase

Conversation

@topher-lo
Copy link
Contributor

@topher-lo topher-lo commented Feb 27, 2026

Motivation

  • The codebase no longer supports LLM UDFs and needs to remove deprecated LLM action templates and UI references to avoid confusion and future regressions.
  • Registry templates and frontend icon mappings referencing llm.* namespaces must be removed to align with the new ai.action usage.

Description

  • Deleted deprecated LLM registry action templates under packages/tracecat-registry/tracecat_registry/templates/tools/ for Anthropic, Bedrock, Gemini (including Vertex variant), Ollama, and OpenAI.
  • Removed llm.* icon namespace mappings from frontend/src/components/icons.tsx and updated the UDFIcons header comment to reflect the supported ai namespace instead of llm.
  • Committed changes with the conventional commit message chore(registry): remove deprecated llm actions.

Testing

  • Ran rg -n "namespace: llm|\"llm\." frontend packages/tracecat-registry/tracecat_registry/templates to validate no remaining llm namespaces and it returned no hits (success).
  • Ran frontend checks with pnpm -C frontend check which 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.

  • Refactors
    • Deleted LLM templates for Anthropic, Bedrock, Gemini (and Vertex), Ollama, and OpenAI.
    • Removed llm.* icon mappings in frontend and updated header comment to reference ai.

Written for commit 361d9b9. Summary will update on new commits.

@topher-lo topher-lo changed the title chore(registry): remove deprecated llm actions chore(registry)!: Remove deprecated llm actions Feb 27, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 8 files

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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".

Comment on lines -7 to -8
namespace: llm.openai
name: chat_completion

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge 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 👍 / 👎.

@topher-lo topher-lo merged commit 24d39fd into main Feb 27, 2026
19 checks passed
@topher-lo topher-lo deleted the codex/remove-all-llm-udfs-from-codebase branch February 27, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant