Skip to content

Conversation

tomasz-stefaniak
Copy link
Collaborator

@tomasz-stefaniak tomasz-stefaniak commented Oct 6, 2025

Description

Added banner that appears after N chat messages, and another one that appears in the settings. They can be permanently dismissed.

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Adds a CLI install banner that detects when the Continue CLI (cn) isn’t installed and offers one-click install, copy, and docs. Appears in Config and in Chat after 3 user messages, supports permanent dismissal, and implements Linear CON-4200.

  • New Features
    • Platform-aware detection via “which cn” (mac/linux) or “where cn” (windows) with error handling.
    • Sticky banner in Config; shows in Chat when userMessageCount >= 3.
    • Install command with Copy and Run buttons, plus a “Learn more” link to docs.
    • Permanent dismissal stored under localStorage key hasDismissedCliInstallBanner.
    • Unit tests cover detection, content, interactions, thresholds, and dismissal.

@tomasz-stefaniak tomasz-stefaniak changed the title Tomasz/con 4200 Make it easy to discover Continue CLI (cn) Oct 7, 2025
@tomasz-stefaniak tomasz-stefaniak marked this pull request as ready for review October 7, 2025 15:43
@tomasz-stefaniak tomasz-stefaniak requested a review from a team as a code owner October 7, 2025 15:43
@tomasz-stefaniak tomasz-stefaniak requested review from sestinj and removed request for a team October 7, 2025 15:43
@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Oct 7, 2025
Copy link

github-actions bot commented Oct 7, 2025

⚠️ PR Title Format

Your PR title doesn't follow the conventional commit format, but this won't block your PR from being merged. We recommend using this format for better project organization.

Expected Format:

<type>[optional scope]: <description>

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

Copy link

github-actions bot commented Oct 7, 2025

✅ Review Complete

Code Review Summary

⚠️ Continue configuration error. Please verify that the assistant exists in Continue Hub.


Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 5 files

@Patrick-Erichsen Patrick-Erichsen requested review from Patrick-Erichsen and removed request for sestinj October 7, 2025 17:55
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Oct 8, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 8, 2025
@tomasz-stefaniak tomasz-stefaniak merged commit 376ca3a into main Oct 8, 2025
68 of 70 checks passed
@tomasz-stefaniak tomasz-stefaniak deleted the tomasz/con-4200 branch October 8, 2025 16:59
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Oct 8, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants