improve: native iOS look with stock SwiftUI navigation, forms, chat, and talk visualizer by marvkr · Pull Request #99231 · openclaw/openclaw

marvkr · 2026-07-02T20:53:53Z

🤖 AI-assisted: contributor baseline built with Claude Code; maintainer repair, review, simulator proof, and landing performed with Codex.

What Problem This Solves

The iOS app mixed custom cards, hand-rolled headers, capsule controls, and bespoke settings rows with native SwiftUI navigation. The result looked inconsistent across tabs and created parallel presentation paths that were harder to keep correct on both iPhone and iPad.

Why This Change Was Made

Uses stock SwiftUI navigation bars, List, Form, Section, LabeledContent, Toggle, Picker, ContentUnavailableView, bordered controls, native Chat chrome, and a state-driven Talk waveform. The maintainer pass also:

keeps one Chat header on iPhone and iPad;
preserves OpenClaw typography and root-owned light/dark/system appearance;
restores the onboarding capability and trust disclosure;
centers the onboarding content on iPad without changing the phone layout;
preserves opaque gateway credential input behavior;
bundles the MIT notice for the adapted waveform;
removes obsolete Talk and Chat presentation paths; and
rebases over the landed Location Always, permission-prompt, and control-row improvements.

The 27-file repaired diff removes more code than it adds: 1,857 additions and 2,116 deletions, including tests and generated native-i18n inventory.

User Impact

Navigation, settings, Chat, Talk, and onboarding now use coherent native iOS structure on both phone and tablet. Settings pushes and back navigation behave normally, appearance changes apply app-wide without blanking the iPad detail pane, sensitive capability disclosure remains visible during onboarding, and Talk state remains owned by the existing state model.

Evidence

Focused native suite: 114/114 passed (typography, render smoke, RootTabs source/presentation, Talk state, licenses).
iPhone simulator UI suite: 7/7 passed.
iPad simulator UI suite: 3/3 passed.
Final onboarding-width checks: 1/1 on iPhone and 1/1 on iPad; final typography check: 9/9.
Simulator screenshots inspected for all five tabs on both idioms, onboarding, native Talk controls, Chat/Control return navigation, Settings back navigation, and light/dark/system appearance. Whole-screen pixel assertions guard against blank rendering.
pnpm native:i18n:check: 2,461 entries, unchanged. pnpm apple:i18n:check: 2 catalogs, 46 keys.
SwiftFormat and git diff --check pass.
Fresh autoreview found one gateway credential input-trait regression; it was repaired centrally. Two follow-up reviews reported no accepted/actionable findings.
Exact-head GitHub CI is green, including iOS build, native i18n, macOS Swift, iOS Periphery dead-code scan, QA smoke, and all shared test shards.

The Talk simulator proof validates the rendered state projection and controls; it does not claim a physical microphone/audio round-trip.

Before / after (iPhone 17 sim, iOS 26.2)

Screen	Before	After
Welcome
Chat
Talk
Control
Agents
Settings

clawsweeper · 2026-07-02T20:55:24Z

Codex review: needs maintainer review before merge. Reviewed July 3, 2026, 9:10 PM ET / 01:10 UTC.

Summary
This PR replaces much of the iOS app’s custom Pro chrome with native SwiftUI navigation, lists/forms, chat chrome, a Talk waveform, appearance handling, license attribution, tests, and native i18n updates.

PR surface: Other -259. Total -259 across 27 files.

Reproducibility: not applicable. This PR implements a visual/product-direction feature rather than fixing a reproducible existing bug. The behavior is demonstrated by simulator screenshots, a Talk waveform recording, source inspection, and live PR checks.

Review metrics: none identified.

Root-cause cluster
Relationship: fixed_by_candidate
Canonical: #99195
Summary: This PR is the candidate implementation for the open native iOS reskin feature issue; nearby merged iOS navigation and permission PRs are prerequisites or partial overlaps, not replacements.

Members:

canonical: [Feature]: native iOS reskin — replace custom "Pro" chrome with stock SwiftUI (HIG) #99195 - The PR body closes this open feature issue, and the issue remains the broad native iOS reskin tracker.
partial_overlap: fix(ios): back from settings details returns to the originating screen #98898 - Merged a narrower settings-detail back-navigation repair that this PR says it preserves.
partial_overlap: fix(ios): return chat to originating control detail #99245 - Merged a narrower Control-detail Chat return repair that this PR rebased over and preserves.
adjacent_distinct: feat: clarify iOS Location Always permission flow #99247 - Merged adjacent iOS Settings/location-permission work mentioned as a rebase prerequisite, but it solves a distinct permission-flow problem.

Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything.

Merge readiness
Overall: 🐚 platinum hermit
Proof: 🦞 diamond lobster ✨ media proof bonus
Patch quality: 🐚 platinum hermit
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

none.

Risk before merge

[P1] The branch changes the first-impression native iOS UI across Chat, Talk, Settings, onboarding, and shared chat surfaces; automated proof reduces correctness risk but does not replace iOS/UI owner acceptance for the product direction.

Maintainer options:

Require iOS/UI owner acceptance (recommended)
Get explicit owner acceptance for the app-wide native visual direction before merging the current head.
Accept the visual direction as maintainer-owned
Maintainers may intentionally own the app-wide visual shift if they decide the screenshots, GIF, tests, and current code review are enough.
Pause the branch if the direction is not settled
Leave or close the PR without merge if the native reskin should be narrowed or reworked before becoming the iOS baseline.

Next step before merge

[P2] Manual iOS/UI owner review is the remaining action; the current head fixed the prior Periphery blocker and no narrow automated repair is identified.

Security
Cleared: No concrete security or supply-chain concern found; the diff adds SwiftUI UI code, shared chat UI adjustments, tests, and a bundled MIT notice, with dependency and security-sensitive guards passing.

Review details

Best possible solution:

Have the iOS/UI owners decide whether to land this current head as the implementation for #99195 after the remaining required CI finishes cleanly.

Do we have a high-confidence way to reproduce the issue?

Not applicable: this PR implements a visual/product-direction feature rather than fixing a reproducible existing bug. The behavior is demonstrated by simulator screenshots, a Talk waveform recording, source inspection, and live PR checks.

Is this the best way to solve the issue?

Unclear until owner acceptance: the implementation is coherent and the previous mechanical blocker is fixed, but the broad iOS native-reskin direction still needs product/design approval before merge.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against 87d88878787b.

Label changes

Label changes:

add merge-risk: 🚨 other: Merging changes the app-wide iOS visual direction, a product/design risk that tests and CI cannot settle by themselves.
add rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🦞 diamond lobster and patch quality is 🐚 platinum hermit.
add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (recording): The PR body includes after-change simulator screenshots and a Talk waveform GIF; I inspected the Chat and Settings screenshots plus the waveform GIF and they directly show the changed native iOS behavior.
remove rating: 🦐 gold shrimp: Current PR rating is rating: 🐚 platinum hermit, so this older rating label is no longer current.
remove status: ⏳ waiting on author: Current PR status label is status: 👀 ready for maintainer look.

Label justifications:

P2: This is a normal-priority, broad iOS app improvement with meaningful user-facing polish but limited blast radius outside the native app UI.
merge-risk: 🚨 other: Merging changes the app-wide iOS visual direction, a product/design risk that tests and CI cannot settle by themselves.
rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🦞 diamond lobster and patch quality is 🐚 platinum hermit.
status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (recording): The PR body includes after-change simulator screenshots and a Talk waveform GIF; I inspected the Chat and Settings screenshots plus the waveform GIF and they directly show the changed native iOS behavior.
proof: sufficient: Contributor real behavior proof is sufficient. The PR body includes after-change simulator screenshots and a Talk waveform GIF; I inspected the Chat and Settings screenshots plus the waveform GIF and they directly show the changed native iOS behavior.
proof: 🎥 video: Contributor real behavior proof includes video or recording evidence. The PR body includes after-change simulator screenshots and a Talk waveform GIF; I inspected the Chat and Settings screenshots plus the waveform GIF and they directly show the changed native iOS behavior.

Evidence reviewed

PR surface:

Other -259. Total -259 across 27 files.

View PR surface stats

Area	Files	Added	Removed	Net
Source	0	0	0	0
Tests	0	0	0	0
Docs	0	0	0	0
Config	0	0	0	0
Generated	0	0	0	0
Other	27	1857	2116	-259
Total	27	1857	2116	-259

What I checked:

Repository policy read: Read the full root AGENTS.md and the scoped iOS AGENTS.md; the review applied the iOS typography, license, visual proof, and owner-review guidance for broad app UI work. (AGENTS.md:1, 87d88878787b)
iOS scoped policy read: The iOS guide requires branded text handling and license-screen care when touching SwiftUI text and bundled acknowledgements, which guided the Settings, Chat, Talk, and waveform-license review. (apps/ios/AGENTS.md:1, 87d88878787b)
Current main still has the old custom Chat surface: Current main still renders Chat through OpenClawProBackground, custom header content, and hidden navigation, so the branch is not obsolete or implemented on main. (apps/ios/Sources/Design/ChatProTab.swift:64, 87d88878787b)
PR head moves Chat to native navigation chrome: At current PR head, ChatProTab renders a system-background chat surface with navigationTitle, inline title mode, toolbar items, and the gateway status toolbar button rather than the old custom header stack. (apps/ios/Sources/Design/ChatProTab.swift:62, f084b131f706)
PR head keeps native Settings structure and preserves sensitive credential fields: SettingsProTabSections uses native sections, LabeledContent, toggles, and a custom gatewaySecureField that keeps opaque credential entry with accessibility labels and no autocorrection/autocapitalization. (apps/ios/Sources/Design/SettingsProTabSections.swift:831, f084b131f706)
PR head adds state-driven Talk waveform: TalkSiriWaveView maps Talk runtime modes to still, indeterminate, level, input-speech, and speaking waveform power while respecting Reduce Motion. (apps/ios/Sources/Design/TalkSiriWave.swift:30, f084b131f706)

Likely related people:

steipete: Authored the maintainer repair commits on this branch and recent merged iOS visual, Chat, Talk, Liquid Glass, and navigation work that this PR builds on. (role: recent iOS visual and navigation contributor; confidence: high; commits: 36690214c06b, e10d3a2724f3, 5c2917002793; files: apps/ios/Sources/Design/ChatProTab.swift, apps/ios/Sources/Design/TalkProTab.swift, apps/ios/Sources/RootTabs.swift)
joshavant: Recent GitHub history for the sampled iOS Settings, Chat, and shared chat files shows branded typography and license-related work overlapping this PR’s scoped iOS policy surface. (role: recent iOS typography and Settings contributor; confidence: medium; commits: d3848d39f28e, 25c8fbb5e7fc, b5f754b63d1e; files: apps/ios/Sources/Design/SettingsProTabSections.swift, apps/ios/Sources/Design/ChatProTab.swift, apps/shared/OpenClawKit/Sources/OpenClawChatUI/ChatComposer.swift)
mbelinky: CONTRIBUTING.md lists this handle for the iOS app area, which matches the broad iOS app visual-direction decision still needed here. (role: listed iOS app area owner; confidence: medium; files: CONTRIBUTING.md, apps/ios/Sources/Design/SettingsProTabSections.swift)
ngutman: CONTRIBUTING.md lists this handle for iOS and macOS app ownership, relevant to native companion-app UX and navigation changes. (role: listed iOS and macOS app area owner; confidence: medium; files: CONTRIBUTING.md, apps/ios/Sources/RootTabs.swift)
BunsDev: CONTRIBUTING.md lists this handle for UI/UX, matching the remaining visual-direction acceptance question for the app-wide native reskin. (role: listed UI/UX area owner; confidence: medium; files: CONTRIBUTING.md, apps/ios/DESIGN.md)

What the crustacean ranks mean

🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works

ClawSweeper keeps one durable marker-backed review comment per issue or PR.
Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
Maintainers can also comment @clawsweeper review to request a fresh review only.
Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

marvkr · 2026-07-02T21:36:56Z

@clawsweeper re-review

Addressed all review findings on the latest push (6b11de8):

Restored the Approvals → Notifications path push (Back contract from fix(ios): back from settings details returns to the originating screen #98898) with the guard comment.
Restored the onboarding reset observer (.onChange(of: onboardingRequestID)) so root-owned resets resync mounted Settings state; removed the redundant alert-site call.
Restored all 10 source-guarded accessibility identifiers on their native equivalents (chat composer/input/send/realtime/attachment/status, talk toggles + voice settings row, settings appearance menu).
Native i18n inventory synced; Periphery scan is clean locally ("No unused code detected").
Evidence: before/after table now includes an animated Talk waveform demo (state-cycle build) plus per-screen screenshots, in the PR description.

clawsweeper · 2026-07-02T21:36:59Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

github-actions · 2026-07-04T00:49:40Z

Dependency graph guard cleared

This PR no longer has blocked dependency graph changes. A future dependency graph change requires a fresh /allow-dependencies-change comment after the guard blocks that new head SHA.

Current SHA: f084b131f706296e36b239c9a29beb18aca2bd27

steipete · 2026-07-04T01:16:51Z

Maintainer repair and land-ready proof for exact head f084b131f706296e36b239c9a29beb18aca2bd27:

Rebased the contributor baseline over the landed Location Always, node-permission, and Control-row improvements; visible PR diff is 27 files, +1,857/−2,116.
Repaired duplicate iPad Chat navigation, restored onboarding capability/trust disclosure, bundled the waveform MIT notice, preserved branded typography and gateway credential input traits, made appearance app-wide without blanking the iPad split destination, centered iPad onboarding, and deleted obsolete Talk/Chat/design paths.
CI caught six remaining Periphery findings. Five dead design/typography symbols were removed; the existing gateway accessibility label is now attached to both actionable and read-only status pills.

Local proof:

Focused native suites: 114/114 passed before cleanup, after cleanup, and after the current-main rebase. Final bundle: /tmp/openclaw-ios-99231-post-rebase-focused.xcresult.
iPhone simulator UI: 7/7 passed; iPad simulator UI: 3/3 passed.
Final onboarding layout checks: iPhone 1/1, iPad 1/1; typography 9/9.
Visually inspected light/dark/system appearance, all five tabs on phone and tablet, Chat/Control return navigation, Settings back navigation, Talk controls, onboarding, and whole-screen nonblank assertions from xcresult attachments.
pnpm native:i18n:check: 2,461 entries, unchanged.
pnpm apple:i18n:check: 2 catalogs, 46 keys.
SwiftFormat and git diff --check: pass.
Fresh cleanup autoreview: no findings, overall correctness 0.98.

Exact-head hosted proof:

CI run: all checks passed, including iOS build, native i18n, and macOS Swift.
iOS dead-code scan: pass after cleanup.
Real behavior proof: pass.

The public PR body retains the contributor's six-screen before/after table. Maintainer simulator screenshots were inspected locally; no new screenshot files were committed or uploaded. Talk proof covers state projection and controls, not a physical microphone/audio round-trip.

Ready to merge.

steipete · 2026-07-04T01:17:14Z

Merged via squash.

Prepared head SHA: f084b131f706296e36b239c9a29beb18aca2bd27
Landed commit: 5370f64cf09c2a5ff2ef4ed677788032600eab00

openclaw-barnacle Bot added app: ios App: ios size: XL labels Jul 2, 2026

marvkr force-pushed the ios-native-polish branch from 1eaf180 to c617e2a Compare July 2, 2026 21:17

clawsweeper Bot added status: 🛠️ actively grinding The PR author has acted after the latest ClawSweeper review and work remains. and removed status: 📣 needs proof The PR needs real behavior proof before ClawSweeper can clear the contributor ask. labels Jul 2, 2026

marvkr force-pushed the ios-native-polish branch 2 times, most recently from 11a5bb6 to ac7a523 Compare July 2, 2026 21:25

marvkr force-pushed the ios-native-polish branch from ac7a523 to 6b11de8 Compare July 2, 2026 21:36

marvkr force-pushed the ios-native-polish branch from 6b11de8 to 4e8d2b8 Compare July 2, 2026 21:41

marvkr force-pushed the ios-native-polish branch from 4e8d2b8 to 13fa00d Compare July 2, 2026 21:50

PollyBot13 mentioned this pull request Jul 2, 2026

feat: clarify iOS Location Always permission flow #99247

Merged

steipete self-assigned this Jul 3, 2026

steipete requested a review from a team as a code owner July 4, 2026 00:49

openclaw-barnacle Bot added docs Improvements or additions to documentation channel: discord Channel integration: discord channel: imessage Channel integration: imessage labels Jul 4, 2026

marvkr and others added 8 commits July 4, 2026 02:05

improve(ios): adopt native SwiftUI chrome across the app

e5485c9

fix(ios): keep phone chat on the native navigation bar

f965906

fix(ios): render licenses screen as native settings rows

abd5a17

fix(ios): complete native interface polish

3669021

fix(ios): preserve branded native interface

e10d3a2

test(ios): align gateway badge source guard

5c29170

fix(ios): center onboarding on iPad

8d16616

fix(ios): remove dead reskin symbols

f084b13

clawsweeper Bot mentioned this pull request Jul 4, 2026

feat(ios): refresh onboarding setup flow #98868

Merged

github-actions Bot mentioned this pull request Jul 4, 2026

📡 Upstream Digest — 2026-07-04 02:06 UTC curtismercier/openclaw-mods#1201

Open

steipete mentioned this pull request Jul 4, 2026

Polish iOS onboarding and chat critique fixes #99243

Merged

Haderach-Ram mentioned this pull request Jul 4, 2026

Ecosystem Digest — 2026-07-04 Haderach-Ram/openclaw-radar#69

Open

ly85206559 mentioned this pull request Jul 4, 2026

fix(ios): full-row Settings toggles on iOS 26 #99888

Merged

This was referenced Jul 7, 2026

fix(ios): preserve glass surface identity #91977

Closed

fix: restore return key in iOS chat composer #98309

Closed

clawsweeper Bot mentioned this pull request Jul 9, 2026

feat(ios): redesign Control tab to a flatter, card-based layout #98582

Merged

steipete mentioned this pull request Jul 18, 2026

[Bug]: iOS chat content meets opaque black bands at screen edges #110503

Closed

hoon0093 mentioned this pull request Jul 23, 2026

[Bug]: iPad Magic Keyboard Return does not insert a newline in the native chat composer #113059

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

improve: native iOS look with stock SwiftUI navigation, forms, chat, and talk visualizer#99231

improve: native iOS look with stock SwiftUI navigation, forms, chat, and talk visualizer#99231
steipete merged 8 commits into
openclaw:mainfrom
marvkr:ios-native-polish

marvkr commented Jul 2, 2026 •

edited by steipete

Loading

Uh oh!

clawsweeper Bot commented Jul 2, 2026 •

edited

Loading

Uh oh!

marvkr commented Jul 2, 2026

Uh oh!

clawsweeper Bot commented Jul 2, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Jul 4, 2026 •

edited

Loading

Uh oh!

steipete commented Jul 4, 2026

Uh oh!

steipete commented Jul 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Uh oh!

Conversation

marvkr commented Jul 2, 2026 • edited by steipete Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What Problem This Solves

Why This Change Was Made

User Impact

Evidence

Before / after (iPhone 17 sim, iOS 26.2)

Uh oh!

clawsweeper Bot commented Jul 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

marvkr commented Jul 2, 2026

Uh oh!

clawsweeper Bot commented Jul 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Jul 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Dependency graph guard cleared

Uh oh!

steipete commented Jul 4, 2026

Uh oh!

steipete commented Jul 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

marvkr commented Jul 2, 2026 •

edited by steipete

Loading

clawsweeper Bot commented Jul 2, 2026 •

edited

Loading

clawsweeper Bot commented Jul 2, 2026 •

edited

Loading

github-actions Bot commented Jul 4, 2026 •

edited

Loading