Conversation
- CI/CD workflows for testing, linting, and building - An updated release process using GoReleaser - New basic commands list, paths, reinstall, and repair
There was a problem hiding this comment.
Pull request overview
This PR is a complete rewrite of the BetterDiscord CLI, modernizing the codebase with improved architecture, better error handling, and cross-platform support. The changes reorganize code into logical packages, add platform-specific implementations, improve the command interface, and set up comprehensive build/release automation.
Changes:
- Restructured code with internal packages (discord, betterdiscord, models, utils) for better separation of concerns
- Rewrote install/uninstall commands to use flags instead of positional arguments with improved error handling
- Added platform-specific path resolution (Windows, Linux, macOS) with support for Flatpak, Snap, and WSL
- Set up automated CI/CD with GoReleaser, GitHub Actions, and package manager integrations
Reviewed changes
Copilot reviewed 31 out of 34 changed files in this pull request and generated 15 comments.
Show a summary per file
| File | Description |
|---|---|
| utils/*.go | Deleted old utility files, functionality moved to internal packages |
| internal/utils/paths.go | New utility functions for file operations and filtering |
| internal/models/*.go | Data models for Discord channels and GitHub API responses |
| internal/discord/*.go | Core Discord detection, injection, and process management logic |
| internal/betterdiscord/*.go | BetterDiscord installation, download, and setup logic |
| cmd/install.go | Rewritten install command using flags with improved error handling |
| cmd/uninstall.go | Rewritten uninstall command using flags with improved error handling |
| cmd/completion.go | New shell completion command for bash/zsh/fish/powershell |
| main.go | Updated entry point with log configuration |
| package.json | Updated go-npm dependency version |
| go.mod | Updated Go version and dependencies |
| Taskfile.yml | New task automation configuration for development and CI |
| .goreleaser.yaml | Updated release configuration with package manager integrations |
| .github/workflows/*.yml | New CI and release workflows |
| README.md | Comprehensive documentation with usage examples and development guide |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 37 out of 40 changed files in this pull request and generated 5 comments.
Comments suppressed due to low confidence (1)
internal/utils/download.go:53
- Spelling error in comment: "Writer" should be "Write".
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 37 out of 40 changed files in this pull request and generated 9 comments.
Comments suppressed due to low confidence (1)
internal/utils/download.go:53
- There's a typo in the comment: "Writer the body to file" should be "Write the body to file".
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1 participant
No description provided.