Skip to content

feat(desktop): add Cline as a tier-2 preset harness - #3519

Open
JPeetz wants to merge 1 commit into
block:mainfrom
JPeetz:worktree-cline-preset-harness
Open

feat(desktop): add Cline as a tier-2 preset harness#3519
JPeetz wants to merge 1 commit into
block:mainfrom
JPeetz:worktree-cline-preset-harness

Conversation

@JPeetz

@JPeetz JPeetz commented Jul 29, 2026

Copy link
Copy Markdown

Summary

  • Adds Cline (cline --acp) as a tier-2 PresetHarness entry. Cline ships native ACP support upstream (npm i -g cline), so this is a plain flag-preset like opencode/kimi — no adapter, no PATH resolution gap.
  • No open or closed Buzz PR previously touched Cline; confirmed via gh pr list --repo block/buzz search across gemini/openclaw/claude/cline/aider/continue/windsurf/copilot/codex.

Test plan

  • cargo fmt --check clean
  • cargo clippy --all-targets -- -D warnings clean
  • node desktop/scripts/check-file-sizes.mjs exit 0 (desktop file-size ratchet)
  • Live handshake spike against installed cline binary

@JPeetz
JPeetz requested a review from a team as a code owner July 29, 2026 11:21
@JPeetz
JPeetz force-pushed the worktree-cline-preset-harness branch from 44ad60d to c66861b Compare August 4, 2026 11:35
@JPeetz

JPeetz commented Aug 4, 2026

Copy link
Copy Markdown
Author

Force-pushed a rewrite. Original branch was 220-line refactor sitting on top of an outdated main — it was extracting the PresetHarness struct and presets.rs module out of discovery.rs. That extraction has already landed upstream in #3225 (Devin preset).

Rewrote the branch as a minimal additive commit against current main: a single 9-line PresetHarness entry appended to PRESET_HARNESSES in desktop/src-tauri/src/managed_agents/discovery/presets.rs. Zero refactor surface. mergeable_state is now blocked (waiting on review, not on conflicts). CI green (Semgrep OSS, zizmor, DCO all pass).

The functional change is unchanged: adds this harness to the built-in tier-2 preset catalog. It uses the same ACP-mode invocation pattern the original PR proposed.

Adds Cline to the built-in preset harness catalog. Cline exposes a
native ACP mode via 'cline --acp' — no adapter needed, no vendor CLI
underlying_cli linkage required.

Ships against the existing preset infrastructure (PresetHarness struct
+ discovery/presets.rs module extracted in block#3225). Zero refactor
surface — a single 9-line additive entry into PRESET_HARNESSES.

Signed-off-by: JPeetz <[email protected]>
@JPeetz
JPeetz force-pushed the worktree-cline-preset-harness branch from c66861b to 96d5a45 Compare September 3, 2026 14:06
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

🔐 Codex Security Review

Status: review required for the current range.

The current range is 7a9a5233d9d755e715be0c585cf7850e935d28cf...96d5a45c133f108fe32413fded271f31c923ff9a.
A new review must complete for this exact range. When manual authorization
is required, a Block organization member must comment exactly
@buzz-security-review 96d5a45c133f108fe32413fded271f31c923ff9a to authorize a new review.
Any previous review applies only to its recorded range.

@JPeetz

JPeetz commented Sep 3, 2026

Copy link
Copy Markdown
Author

Rebased on latest main (was 279 commits behind). The old CI failure (Aug 4) had pruned logs and was likely a workflow-side issue. Fresh CI run would be appreciated — could you approve the workflow to run? 🙏

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