Skip to content

feat: implement update checking#33

Merged
lusu007 merged 1 commit intomainfrom
feat/update-check
May 4, 2026
Merged

feat: implement update checking#33
lusu007 merged 1 commit intomainfrom
feat/update-check

Conversation

@lusu007
Copy link
Copy Markdown
Contributor

@lusu007 lusu007 commented May 4, 2026

Pull Request

Description

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 💥 Breaking change
  • ♻️ Refactoring
  • 📚 Documentation
  • 🔧 Chore

Related Issues

None

Testing

  • Unit tests pass
  • Manual testing completed
  • New tests added for new functionality

Checklist

  • I have performed a self-review of my own code
  • Tests have been added/updated and pass (if needed)
  • Documentation has been updated (if needed)

@lusu007 lusu007 self-assigned this May 4, 2026
Copilot AI review requested due to automatic review settings May 4, 2026 17:35
@lusu007 lusu007 force-pushed the feat/update-check branch from 85c3731 to 2645817 Compare May 4, 2026 17:39
Copy link
Copy Markdown

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

Adds CLI release update checking so grounds version and grounds doctor can surface newer releases and, when possible, suggest how to upgrade. This fits the CLI’s diagnostics/versioning surface by extending the existing version and doctor commands.

Changes:

  • Added release lookup, version comparison, and install-method detection under internal/version.
  • Extended grounds version and grounds doctor to report update status and richer diagnostic output.
  • Updated README command docs and added unit tests for the new version/doctor behavior.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
README.md Documents version --check and doctor’s update-warning behavior.
internal/version/install_method.go Adds install source detection and upgrade command hints.
internal/version/install_method_test.go Adds unit tests for install-method detection.
internal/version/check.go Adds latest-release lookup and version comparison logic.
internal/version/check_test.go Adds tests for release checking and version comparison.
cmd/grounds/commands/version.go Extends grounds version with optional update checks and hints.
cmd/grounds/commands/version_test.go Adds command-level coverage for version --check.
cmd/grounds/commands/doctor.go Adds version checking and new structured doctor output/spinner flow.
cmd/grounds/commands/doctor_test.go Adds tests for the new doctor rendering and version-check behavior.

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

Comment thread cmd/grounds/commands/doctor.go
Comment thread internal/version/install_method.go
Comment thread internal/version/check.go Outdated
Comment thread internal/version/install_method.go
Comment thread cmd/grounds/commands/doctor.go
@lusu007 lusu007 force-pushed the feat/update-check branch from 2645817 to ea99605 Compare May 4, 2026 17:45
@lusu007 lusu007 force-pushed the feat/update-check branch from ea99605 to d053b4b Compare May 4, 2026 18:25
@lusu007 lusu007 merged commit 0c8367d into main May 4, 2026
4 checks passed
@lusu007 lusu007 deleted the feat/update-check branch May 4, 2026 18:27
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