Skip to content
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.

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

chore: minor bugfixes #8

Merged
merged 1 commit into from
Oct 23, 2024
Merged

chore: minor bugfixes #8

merged 1 commit into from
Oct 23, 2024

Conversation

Kvadratni
Copy link
Owner

This pull request includes several changes to improve the Goose IntelliJ plugin by refactoring utility methods and updating the startup activity and profile selection logic. The most important changes include adding new utility methods for fetching available providers and toolkits, refactoring the startup activity to use these new methods, and simplifying the profile selection logic.

Refactoring and Utility Methods:

  • Added methods GooseUtils.getAvailableProviders and GooseUtils.getToolkitsWithDescriptions to fetch available providers and toolkits with descriptions. (src/main/kotlin/com/block/gooseintellij/utils/GooseUtils.kt) [1] [2] [3]

Startup Activity Improvements:

  • Updated GoosePluginStartupActivity to use the new utility methods GooseUtils.getAvailableProviders and GooseUtils.getToolkitsWithDescriptions. (src/main/kotlin/com/block/gooseintellij/actions/GoosePluginStartupActivity.kt) [1] [2]

Profile Selection Simplification:

  • Simplified SelectGooseProfileAction by removing the redundant getAvailableProviders method and using GooseUtils.getAvailableProviders instead. (src/main/kotlin/com/block/gooseintellij/actions/SelectGooseProfileAction.kt) [1] [2]
  • Replaced inline logic for fetching toolkits with descriptions with GooseUtils.getToolkitsWithDescriptions in SelectGooseProfileAction. (src/main/kotlin/com/block/gooseintellij/actions/SelectGooseProfileAction.kt) [1] [2] [3]

Code Cleanup:

  • Refactored GooseUtils.startGooseSession to remove the usingSqGoose parameter and determine the Goose instance internally. (src/main/kotlin/com/block/gooseintellij/utils/GooseUtils.kt) [1] [2]
  • Improved readability by reformatting code blocks in SelectGooseProfileAction. (src/main/kotlin/com/block/gooseintellij/actions/SelectGooseProfileAction.kt) [1] [2]

@Kvadratni Kvadratni merged commit 6dba48e into main Oct 23, 2024
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