Format baseline sync files

[aaditagrawal]

What Changed

Formats the baseline sync files so the lower stack branch passes the project checks before the upstream sync slices land.

Why

This keeps the stack rooted on a clean formatting baseline and avoids mixing mechanical formatting with later behavioral sync work.

UI Changes

None.

Checklist

• Part 1 of the upstream sync stack
• Validation on stack tip: vp check
• Validation on stack tip: vp run typecheck
• Validation on stack tip: vp run lint:mobile
• Full vp test had one ACP timeout flake; rerunning that file passed

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR makes two minor object-construction adjustments: GeminiCliServerManager instantiation in GeminiCliAdapter.ts uses a conditional object literal instead of conditional spread, and toUnifiedPatch in appSettings.ts spreads providersPatch[provider] directly without a nullish fallback.

Changes

Object construction cleanup

Layer / File(s)	Summary
Conditional object literal simplification apps/server/src/provider/Layers/GeminiCliAdapter.ts	Replaces conditional spread with a conditional object literal when constructing GeminiCliServerManager options.
Patch merge simplification apps/web/src/appSettings.ts	Removes the nullish fallback when spreading providersPatch[provider] in toUnifiedPatch.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Related PRs: None identified.

Suggested labels: None

Suggested reviewers: None

🐰 Two tiny tweaks, spread just right,
No more fallback hiding out of sight,
Objects built with cleaner form,
Code review effort? Barely warm.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title matches the PR’s broad goal of formatting baseline sync files and is related to the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description check	✅ Passed	The PR description follows the required template and includes clear What Changed, Why, UI Changes, and Checklist sections.

✨ 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 sync/upstream-20260701-format-baseline

---

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.}

[aaditagrawal]

• Record upstream ancestry for July sync #135
• Add Claude Sonnet 5 default #134
• Sync chat timeline and mobile UX #133
• Sync preview browser automation #132
• Sync desktop WSL support #131
• Format baseline sync files #130 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[aaditagrawal]

Merge activity

• Jul 3, 8:15 AM UTC: @aaditagrawal merged this pull request with Graphite.