Skip to content

Conversation

@PossibleLlama
Copy link
Owner

No description provided.

@PossibleLlama PossibleLlama self-assigned this Dec 15, 2025
Copilot AI review requested due to automatic review settings December 15, 2025 21:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the project to use bubbletea v2 and related charm.land libraries (bubbles v2, lipgloss v2). The migration involves updating import paths from github.com/charmbracelet/* to charm.land/*/v2, adapting to API changes in the new versions, and updating dependency versions in go.mod.

Key changes:

  • Updated all charm.land library imports to v2 versions with new import paths
  • Adapted to bubbletea v2 API changes including key handling, View return type, and window configuration
  • Removed deprecated API calls (SetCursor, SetWindowTitle, WithAltScreen)

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tui/summary.go Updated imports to v2, changed key handling from string-based to code-based comparison, changed View return type to tea.View, moved window configuration from Init to View
tui/styles.go Updated lipgloss import to v2
tui/listFormat.go Updated all charm.land imports to v2
plugins/jira.go Updated bubbletea import to v2
plugins/interface.go Updated bubbletea import to v2
plugins/clickup.go Updated bubbletea import to v2
go.mod Updated Go version, added v2 charm.land dependencies, removed v1 dependencies, updated transitive dependencies
exec/cli/main.go Updated bubbletea import to v2, removed WithAltScreen option

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

}
fallthrough
case tea.KeyCtrlC.String():
case key.Code == 'c' && key.Mod == tea.ModCtrl:
Copy link

Copilot AI Dec 15, 2025

Choose a reason for hiding this comment

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

The key code comparison uses the character 'c' instead of a proper key constant. This is inconsistent with the KeyEscape comparison on line 136. Consider using a proper key constant like tea.KeyC or tea.KeyCtrlC if available in bubbletea v2.

Suggested change
case key.Code == 'c' && key.Mod == tea.ModCtrl:
case key.Code == tea.KeyC && key.Mod == tea.ModCtrl:

Copilot uses AI. Check for mistakes.
@PossibleLlama
Copy link
Owner Author

feat: Upgrade to bubbletea v2

@PossibleLlama PossibleLlama merged commit 3ab91d9 into main Dec 15, 2025
6 checks passed
@PossibleLlama PossibleLlama deleted the bubbletea-v2 branch December 15, 2025 21:55
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.

1 participant