Skip to content

Command mode suggestions and alias resolution#131

Merged
yimsk merged 8 commits intodevelopfrom
feature/command-mode-suggestions
Jan 11, 2026
Merged

Command mode suggestions and alias resolution#131
yimsk merged 8 commits intodevelopfrom
feature/command-mode-suggestions

Conversation

@yimsk
Copy link
Copy Markdown
Contributor

@yimsk yimsk commented Jan 11, 2026

Summary

  • Show suggestions while typing in command mode
  • Display alias resolution (→ service/resource) for recognized aliases
  • Fix diff command: use ID instead of Name for completion
  • Code cleanup: remove dead code, use constants and style helpers

Test plan

  • Type :ec2 and verify suggestions appear
  • Type alias (e.g., :sg) and verify → ec2/security-groups shown
  • Test :diff tab completion with resources
  • Verify m → d workflow still works

yimsk added 6 commits January 11, 2026 02:32
@yimsk
Show command mode suggestions and alias resolution
214e3ed 
- Display completion suggestions in command mode (fixes #72)
- Show alias resolution (e.g., :sg → ec2/security-groups) (fixes #73)
- Add Ctrl+C to exit command mode
- Remove textInput width constraint to allow suggestions display
- Update suggestions on activation
@yimsk
Remove command input debug leftover and SetWidth dead code
94b8522
@yimsk
Replace magic number with commandInputWidth constant
eb2ba60
@yimsk
Use ui.NoStyle() instead of inline lipgloss.NewStyle()
3ecaf17
@yimsk
Fix diff command: use ID instead of Name for completion
fb2c5eb 
- DiffCompletionProvider: GetResourceNames→GetResourceIDs (avoid space issues)
- CharLimit: 50→150 (support :diff id1 id2 with long IDs)
- Tab completion: Reset() before SetValue (fix display offset bug)
- handleDiffMsg: match by GetID instead of GetName
@yimsk
Rename DiffMsg fields: LeftName/RightName → LeftID/RightID
2090c25
@claude

This comment was marked as resolved.

Sign in to view
yimsk added 2 commits January 11, 2026 05:49
@yimsk
Improve command mode: show Enter destination, add tests
6482d70 
- Highlight where Enter will navigate (alias resolution or prefix match)
- Other suggestions shown in white (not highlighted)
- Dynamic width: expand to 50 when input > 30 chars
- Exclude exact match from suggestions (e.g., "sq" not shown when typing "sq")
- Add Reset() comment explaining cursor offset fix
- Add tests: Ctrl+C exit, alias resolution, dynamic width
@yimsk
Improve command mode: bash-style tab, alias prefix match, error display
36dcb46 
- Tab completion: expand to common prefix first, then cycle
- Prefix match now includes aliases (cd→cdn→cloudfront)
- Show error for unknown commands (3s auto-clear)
- Extract resolvePrefixMatch helper for consistency
@claude

This comment was marked as resolved.

Sign in to view
@yimsk yimsk merged commit f1ab5d7 into develop Jan 11, 2026
5 checks passed
@yimsk yimsk deleted the feature/command-mode-suggestions branch January 11, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yimsk