Skip to content

Add description truncation for slash commands in CLI UI #8172

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
RomneyDa merged 3 commits into from
Nov 3, 2025
Merged

Add description truncation for slash commands in CLI UI #8172

RomneyDa merged 3 commits into from
Nov 3, 2025
+15 −2

Conversation

@sestinj
Copy link
Contributor

@sestinj sestinj commented Oct 9, 2025 •
edited
Loading

CON-4280

Summary by cubic

Truncates long slash command descriptions in the CLI UI to 80 characters with an ellipsis for cleaner, readable lists. Also updates the CLI .gitignore to ignore *.cast files.

@sestinj sestinj requested a review from a team as a code owner October 9, 2025 22:48
@sestinj sestinj requested review from tingwai and removed request for a team October 9, 2025 22:48
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 9, 2025
@github-actions
Copy link

github-actions bot commented Oct 9, 2025

⚠️ PR Title Format

Your PR title doesn't follow the conventional commit format, but this won't block your PR from being merged. We recommend using this format for better project organization.

Expected Format:

<type>[optional scope]: <description>

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Oct 9, 2025
View reviewed changes
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.

1 issue found across 2 files

Prompt for AI agents (all 1 issues) 

Understand the root cause of the following 1 issues and fix them.


<file name="extensions/cli/src/ui/SlashCommandUI.tsx">

<violation number="1" location="extensions/cli/src/ui/SlashCommandUI.tsx:13">
Slicing by code units can split surrogate pairs and render broken characters when truncating descriptions that contain emoji. Please truncate by Unicode code points instead to avoid the replacement character in the UI.</violation>
</file>

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

RomneyDa and others added 2 commits October 13, 2025 14:12
@RomneyDa @cubic-dev-ai
Apply suggestion from @cubic-dev-ai[bot]
fef5ed7 
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
@sestinj
chore: format
80f9b49
cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Oct 17, 2025
View reviewed changes
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 1 file

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Oct 19, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 19, 2025
@RomneyDa RomneyDa merged commit be07af5 into main Nov 3, 2025
56 checks passed
@RomneyDa RomneyDa deleted the nate/truncate-slash branch November 3, 2025 19:04
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Nov 3, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Nov 3, 2025
@sestinj
Copy link
Contributor Author

sestinj commented Nov 7, 2025

🎉 This PR is included in version 1.32.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor Author

sestinj commented Nov 18, 2025

🎉 This PR is included in version 1.29.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor Author

sestinj commented Nov 19, 2025

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor Author

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

+1 more reviewer

@tingwai tingwai tingwai approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

lgtm This PR has been approved by a maintainer released size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Issues and PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sestinj @tingwai @RomneyDa