[nortonandreev]

Closes #1855 — AI-4264

What & why

Setup said "Setup adds connections; it doesn't remove existing ones…". On a first install there is nothing to leave in place, and changing an existing connection is a Settings action (#1513), so the note is gone from the wizard. Settings is untouched; the note's style is still used elsewhere.

Verification

--treenode-filter "///HarnessesStepViewModelTests/*": 21/21 passed. Headless renders:

Before	After

[linear-code]

AI-4264

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 59 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: ASSERTIVE
• Plan: Advanced
• Run ID: 57494381-a38b-4731-9e12-a313060aacc5

📥 Commits

Reviewing files that changed from the base of the PR and between d1f1dc3 and f0482e1.

📒 Files selected for processing: 1

• src/Capacitor.App/Views/Onboarding/HarnessesPaneView.axaml

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[qodo-code-review]

PR Summary by Qodo

Remove misleading connections note from first-run setup

🐞 Bug fix 🕐 Less than 5 minutes

AI Description

• Removes a connections note that incorrectly implies existing connections need preserving during
 first-run setup.
• Leaves Settings and the remaining setup guidance unchanged.

High-Level Assessment

Removing the note directly is appropriate: it does not apply to first-run setup, while existing-connection changes belong in Settings. No broader UI or settings change is needed.

Files changed (1) +0 / -1

Bug fix (1) +0 / -1

HarnessesPaneView.axamlRemove misleading connections guidance +0/-1

Remove misleading connections guidance

• Removes the note claiming setup leaves parts of existing connections unchanged. The surrounding setup guidance remains in place.

src/Capacitor.App/Views/Onboarding/HarnessesPaneView.axaml

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Tip of the day

💡 Did you know, you can reply 'qodo' on any finding to push back, ask questions, or dig deeper

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[nortonandreev]

@CodeRabbit approve

[coderabbitai]

✅ Action performed

Comments resolved and changes approved.