[saddlepaddle]

Summary

Mobile drew Lucide's cloud for the Cloud scope; the desktop sidebar draws Tabler's. Mobile now draws the same Tabler glyph in the home scope chip, the filter sheet, the scope picker, and the cloud provisioning screen.

react-icons is DOM-only and @tabler/icons-react-native would ship its whole barrel through Metro for one glyph, so the path is built with createLucideIcon and slots into the existing Icon wrapper.

Test plan

• biome clean on apps/mobile/screens
• tsc --noEmit reports no errors in the touched files (the remaining errors are pre-existing and unrelated)
• not simulator-verified; the path is copied verbatim from the react-icons Tabler build desktop imports

🤖 Generated with Claude Code

---

Summary by cubic

Aligns mobile's cloud scope icon with the Tabler cloud glyph the desktop sidebar draws, so the same workspace reads the same on both surfaces. The new glyph appears in the home scope chip, the filter sheet, the scope picker, and the cloud provisioning screen, replacing Lucide's cloud.

The shared CloudIcon is built with createLucideIcon so it keeps working with the existing Icon wrapper, avoiding @tabler/icons-react-native, which would ship its whole barrel through Metro for a single glyph.

^{Written for commit fc60777. Summary will update on new commits.}

Summary by CodeRabbit

• Style
  • Standardized the cloud icon across filtering, scope navigation, and workspace provisioning screens.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 0bb54314-189c-4318-854f-059818bcccec

📥 Commits

Reviewing files that changed from the base of the PR and between 2340152 and fc60777.

📒 Files selected for processing (6)

• apps/mobile/screens/(authenticated)/(home)/filter/FilterScreen.tsx
• apps/mobile/screens/(authenticated)/(home)/filter/scope/ScopeFilterScreen.tsx
• apps/mobile/screens/(authenticated)/(home)/home/components/ScopeBar/ScopeBar.tsx
• apps/mobile/screens/(authenticated)/components/CloudIcon/CloudIcon.tsx
• apps/mobile/screens/(authenticated)/components/CloudIcon/index.ts
• apps/mobile/screens/(authenticated)/workspace/[id]/components/CloudWorkspaceProvisioningState/CloudWorkspaceProvisioningState.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.

---

📝 Walkthrough

Walkthrough

The mobile app adds a shared CloudIcon and replaces Lucide Cloud icons in cloud scope filters, the scope bar, and workspace provisioning.

Changes

Shared Cloud Icon

Layer / File(s)	Summary
Define and export the shared icon apps/mobile/screens/(authenticated)/components/CloudIcon/*	Defines CloudIcon with a cloud-shaped path and re-exports it from the components module.
Use the shared icon in authenticated screens apps/mobile/screens/(authenticated)/(home)/filter/FilterScreen.tsx, apps/mobile/screens/(authenticated)/(home)/filter/scope/ScopeFilterScreen.tsx, apps/mobile/screens/(authenticated)/(home)/home/components/ScopeBar/ScopeBar.tsx, apps/mobile/screens/(authenticated)/workspace/[id]/components/CloudWorkspaceProvisioningState/CloudWorkspaceProvisioningState.tsx	Replaces the Lucide Cloud icon with CloudIcon in the scope accessory, Cloud row, cloud chip, and provisioning frame.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: kitenite

Merge Risk: ⚪ Minimal · up to fc607

The cloud icon change appears ready to merge; no actionable issue remains in the reviewed changes.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 6 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the mobile cloud glyph change and matches the primary purpose of the pull request.
Description check	✅ Passed	The description explains the change, its rationale, affected screens, implementation choice, and testing status. It does not include the template's Checklist section or simulator verification, but the…
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.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

[expo]

No installed development build can load this pull request as an update: its native fingerprint is 63273df0c28c and no development build has it. Add the mobile-build label to build one.

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ✅ Neon database branch

Thank you for your contribution! 🎉