Skip to content

feat(alias): create for mk, del for rm - #10

Merged
stilliard merged 1 commit into
mainfrom
feat/create-del-aliases
Aug 24, 2026
Merged

feat(alias): create for mk, del for rm#10
stilliard merged 1 commit into
mainfrom
feat/create-del-aliases

Conversation

@stilliard

Copy link
Copy Markdown
Owner

Two more command aliases alongside add/remove/list — wired into the dispatcher, the zsh/bash completions and the help text. Tests added for both.

🤖 Generated with Claude Code

Adds two more command aliases alongside the existing add/remove/list,
wired into the dispatcher, the zsh/bash completions and the help text.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 24, 2026 14:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The only remaining finding is a non-blocking completion nit.

Pull request overview

Adds create and del aliases for mk and rm, with documentation, completion updates, and tests.

Changes:

  • Registers both command aliases.
  • Updates help text, README, and shell completions.
  • Adds alias behavior tests.
File summaries
File Description
wt.sh Adds alias dispatch and completion support; nit remains for first-word zsh candidates.
test/rm.bats Tests the del alias.
test/mk.bats Tests the create alias.
README.md Documents the new aliases.
Review details

Suppressed comments (1)

wt.sh:559

  • The bash completion branch below recognizes del only after it has already been typed, while the first-word candidates at line 553 still omit create and del; consequently neither new alias is suggested by wt <TAB>. Add the aliases (and the existing add/remove/list aliases for consistency) to the first-word candidates.
        rm|remove|del|merged)
  • Files reviewed: 4/4 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread wt.sh

@stilliard stilliard left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

👍

@stilliard
stilliard merged commit f178600 into main Aug 24, 2026
1 check passed
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.

2 participants