Skip to content

[None][chore] Rename .claude skills with trtllm- prefix and drop ci-failure-retrieval#14234

Merged
venkywonka merged 2 commits into
NVIDIA:mainfrom
kaiyux:user/kaiyu/update_skills
May 18, 2026
Merged

[None][chore] Rename .claude skills with trtllm- prefix and drop ci-failure-retrieval#14234
venkywonka merged 2 commits into
NVIDIA:mainfrom
kaiyux:user/kaiyu/update_skills

Conversation

@kaiyux
Copy link
Copy Markdown
Member

@kaiyux kaiyux commented May 18, 2026

Summary

  • Rename .claude/skills/flashinfer-upgradetrtllm-flashinfer-upgrade.
  • Rename .claude/skills/serve-config-guidetrtllm-serve-config-guide.
  • Delete .claude/skills/ci-failure-retrieval (superseded by the trtllm-agent-toolkit plugin).
  • Update .claude/README.md and AGENTS.md to reflect the new skill names and remove the deleted reference.

Test plan

  • git diff shows only file renames + reference updates; no behavior change.
  • .claude/README.md example uses /trtllm-serve-config-guide.
  • AGENTS.md no longer references the removed ci-failure-retrieval skill.

🤖 Generated with Claude Code

Summary by CodeRabbit

Documentation

  • Skill naming standardized – Updated skill references to include consistent prefix naming conventions across documentation.
  • CI failure retrieval skill removed – Removed documentation and workflow for CI test failure analysis functionality.
  • Updated documentation references – Refreshed example commands and skill documentation to reflect current naming structure.

Review Change Stack

…ailure-retrieval

- Rename flashinfer-upgrade -> trtllm-flashinfer-upgrade
- Rename serve-config-guide -> trtllm-serve-config-guide
- Delete ci-failure-retrieval (superseded by trtllm-agent-toolkit plugin)
- Update README.md and AGENTS.md references

Signed-off-by: Kaiyu Xie <26294424+kaiyux@users.noreply.github.com>
@kaiyux kaiyux requested a review from a team as a code owner May 18, 2026 02:55
@kaiyux kaiyux requested review from laikhtewari and nv-guomingz May 18, 2026 02:55
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 18, 2026

📝 Walkthrough

Walkthrough

This PR standardizes AI skill naming by adding the trtllm- prefix to skill metadata and updating references, while removing deprecated CI failure retrieval skill documentation from the AGENTS.md guide.

Changes

AI Skills Naming Standardization

Layer / File(s) Summary
Skill metadata naming standardization
.claude/skills/trtllm-flashinfer-upgrade/SKILL.md, .claude/skills/trtllm-serve-config-guide/SKILL.md, .claude/README.md
Frontmatter name fields updated to include the trtllm- prefix for consistency. The README manual invocation example is updated to reference /trtllm-serve-config-guide.
CI failure retrieval skill removal
AGENTS.md
The "Retrieving CI Test Failures from a PR" documentation section is removed, with the "Trouble Shooting" section following directly after.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main changes: renaming skills with trtllm- prefix and removing ci-failure-retrieval skill.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The PR description clearly explains the changes (skill renames and deletion) and includes a test plan with verifications performed.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Collaborator

@yihwang-nv yihwang-nv left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

Copy link
Copy Markdown
Collaborator

@QiJune QiJune left a comment

Choose a reason for hiding this comment

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

LGTM

Following the rename of flashinfer-upgrade/serve-config-guide to the
trtllm- prefix and the deletion of ci-failure-retrieval, no skills or
agents use the ci- or serve- prefixes anymore. Remove those rows from
the naming convention table and broaden the trtllm- description to
cover dependency upgrades and serving configuration.

Signed-off-by: Kaiyu Xie <26294424+kaiyux@users.noreply.github.com>
@kaiyux
Copy link
Copy Markdown
Member Author

kaiyux commented May 18, 2026

/bot skip --comment "skill changes"

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48867 [ skip ] triggered by Bot. Commit: 4ac9908 Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48867 [ skip ] completed with state SUCCESS. Commit: 4ac9908
Skipping testing for commit 4ac9908

Link to invocation

@venkywonka venkywonka merged commit ca0aad2 into NVIDIA:main May 18, 2026
14 of 16 checks passed
@kaiyux kaiyux deleted the user/kaiyu/update_skills branch May 19, 2026 00:44
KleinBlueC pushed a commit to KleinBlueC/TensorRT-LLM that referenced this pull request May 19, 2026
…ailure-retrieval (NVIDIA#14234)

Signed-off-by: Kaiyu Xie <26294424+kaiyux@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants