feat(cli): add download progress indicator to upgrade

[larryro]

Summary

• Show real-time download progress (bytes/total/percentage) during tale upgrade so users can tell whether the download is progressing or stuck
• Remove dev mode guard so local builds can run the full upgrade flow
• Dev builds (version containing "dev") always download the latest release

Test plan

• Build CLI locally: cd tools/cli && bun run build:linux
• Run dist/tale upgrade and verify progress indicator displays during download
• Run tale upgrade | cat and verify no garbled output in non-TTY mode
• Run tale upgrade on an up-to-date installation and verify it skips download

Summary by CodeRabbit

• Bug Fixes

  • Resolved upgrade blocking issue affecting compiled binary installations, ensuring updates work seamlessly across all installation types.
  • Enhanced upgrade detection for development builds with improved version comparison logic.

• New Features

  • Added interactive progress indicator displaying real-time download status during upgrades for better visibility.
  • Implemented efficient streaming support enabling faster and more stable package downloads.

[greptile-apps]

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 59b74e35-0409-4fd4-9d60-2be0cb48d837

📥 Commits

Reviewing files that changed from the base of the PR and between 0b79ba7 and 00d84af.

📒 Files selected for processing (1)

• tools/cli/src/lib/actions/upgrade.ts

---

📝 Walkthrough

Walkthrough

This change modifies the CLI self-upgrade implementation in a single file. It removes a guard that blocked upgrades based on the executable path, adds streaming-based binary download with progress indication (using TTY and color environment detection), introduces a helper function to format byte sizes, and adjusts the upgrade decision logic to treat development builds as requiring updates.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat(cli): replace tale update with tale upgrade for self-upgrading CLI #1223: Introduced the upgrade flow implementation that this PR modifies, making it directly relevant for understanding the overall upgrade mechanism and ensuring consistency with the base implementation.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the primary change: adding a download progress indicator to the upgrade feature, which aligns with the main implementation of streaming responses with progress tracking.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/cli-download-progress

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}