Skip to content

feat(ios): refine the chat experience#98953

Merged
steipete merged 2 commits into
mainfrom
codex/ios-chat-refinement
Jul 2, 2026
Merged

feat(ios): refine the chat experience#98953
steipete merged 2 commits into
mainfrom
codex/ios-chat-refinement

Conversation

@steipete

@steipete steipete commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Closes #98929

AI-assisted: Codex implemented, simulator-tested, live-tested, and autoreviewed this change.

What Problem This Solves

The native iOS chat felt visually heavy and mechanically assembled: duplicated introductory cards dominated the conversation, a large text status badge competed with the agent identity, assistant replies looked like generic containers, and the idle composer was taller and more fragmented than necessary.

Why This Change Was Made

This refines the chat around the conversation itself. Assistant content now uses open typography while user messages retain clear bubbles, transient assistant states keep the same geometry, the header reduces connection status to a compact semantic icon, and the attachment, text, voice, and send controls form one adaptive Liquid Glass surface on iOS 26 with the existing material fallback on older systems.

The composer starts at 44 points, grows through four lines as the user types, and preserves true 44-point hit regions for attachment, voice, and send. Empty, loading, error, and unavailable states are mutually coherent so introductory content no longer appears over operational feedback.

User Impact

Chat is quieter, faster to scan, and more consistent with modern iOS. The default composer no longer consumes unnecessary vertical space, long drafts expand naturally, assistant streaming no longer snaps between visual treatments, and every compact control remains comfortably tappable and accessible.

Evidence

Full chat hierarchy

Before After
Before: stacked intro cards, large status badge, and split composer After: concise intro, semantic status icon, and unified compact composer

Adaptive composer

State Before After
Compact Before compact composer After compact composer
Expanded Before expanded composer After expanded composer
Light appearance

Refined chat in light appearance

Live Gateway round trip

The app was paired to an isolated local Gateway through the user setup-code path, sent a per-run unique marker through a deterministic OpenAI Responses endpoint, rendered the provider-echoed marker, then relaunched with the paired credentials and opened Control Overview.

Live Gateway round-trip proof

Validated on iPhone 17 Pro, iOS 26.0 Simulator:

  • swift test --package-path apps/shared/OpenClawKit --filter ChatDynamicTypeSourceGuardTests
  • xcodebuild test-without-building -project apps/ios/OpenClaw.xcodeproj -scheme OpenClaw -destination 'platform=iOS Simulator,id=287E61C4-E15A-4794-A8DF-3E6AC37AAF97' -only-testing:OpenClawUITests/OpenClawSnapshotUITests/testChatComposerStartsCompactAndGrowsWithDraft
  • xcodebuild test-without-building ... -only-testing:OpenClawUITests/OpenClawSnapshotUITests/testChatPresentationInLightAppearance
  • xcodebuild test-without-building ... -only-testing:OpenClawUITests/OpenClawSnapshotUITests/testLiveGatewayChatRoundTripAndControlOverview
  • pnpm native:i18n:check
  • SwiftFormat with config/swiftformat
  • git diff --check
  • fresh Codex autoreview: no actionable findings, patch correct at 0.88 confidence

The compact composer UI test additionally asserts that attachment, Talk, and Send each expose at least a 44 by 44 point hittable region. Local pnpm check:changed could not start because the installed Crabbox is 0.15.0 while Blacksmith currently requires 0.22.0; GitHub CI remains the broad changed-surface gate.

@openclaw-barnacle openclaw-barnacle Bot added app: ios App: ios size: L maintainer Maintainer-authored PR labels Jul 2, 2026
@clawsweeper

clawsweeper Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Codex review: needs maintainer review before merge. Reviewed July 2, 2026, 2:49 AM ET / 06:49 UTC.

Summary
The branch changes iOS ChatProTab and shared OpenClawChatUI to render open assistant content, a compact gateway status icon, a unified clean composer surface, and related iOS UI/live-gateway tests.

PR surface: Other +190. Total +190 across 9 files.

Reproducibility: not applicable. this is a visual/product feature PR, not a bug with failing steps. Source inspection plus the PR's before/after screenshots and live Gateway proof show the changed behavior being reviewed.

Review metrics: none identified.

Root-cause cluster
Relationship: fixed_by_candidate
Canonical: #98929
Summary: This PR is the candidate implementation for the linked native iOS chat hierarchy issue.

Members:

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:

  • [P2] Let the remaining required CI jobs finish green before merge.
  • Have an iOS owner confirm the visual direction for the open assistant content and compact status icon.

Risk before merge

  • [P1] The visual hierarchy, open assistant rendering, and compact gateway status icon are product/design choices that need iOS maintainer approval before landing.
  • [P1] Live PR status still had several broad CI jobs in progress during inspection, so merge should wait for the final required checks even though focused proof is present.

Maintainer options:

  1. Decide the mitigation before merge
    Land this after iOS maintainers approve the visual direction and required CI finishes green; if the direction changes, narrow the visual treatment inside ChatProTab/OpenClawChatUI without touching chat transport or state.
  2. Pause or close
    Do not merge this PR until maintainers decide whether the risk is worth taking.

Next step before merge

  • [P2] The protected maintainer label and visual iOS product direction make this a human review/landing decision, and I found no narrow ClawSweeper repair to queue.

Security
Cleared: The diff touches SwiftUI chat presentation, generated native i18n line metadata, and iOS tests; it does not change dependencies, workflows, secrets, package resolution, or other supply-chain surfaces.

Review details

Best possible solution:

Land this after iOS maintainers approve the visual direction and required CI finishes green; if the direction changes, narrow the visual treatment inside ChatProTab/OpenClawChatUI without touching chat transport or state.

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

Not applicable: this is a visual/product feature PR, not a bug with failing steps. Source inspection plus the PR's before/after screenshots and live Gateway proof show the changed behavior being reviewed.

Is this the best way to solve the issue?

Yes, this is a maintainable implementation shape if maintainers accept the visual direction: it stays in the iOS ChatProTab/OpenClawChatUI presentation layer, preserves the clean/full split, and avoids changing chat transport or state.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against 2913d3aee90c.

Label changes

Label changes:

  • add P3: This is a user-visible iOS chat polish and ergonomics feature with limited blast radius, not a confirmed runtime, data-loss, security, crash, or message-delivery defect.
  • add proof: sufficient: Contributor real behavior proof is sufficient. The PR body includes matched before/after screenshots, compact/expanded composer and light-mode screenshots, a live Gateway round-trip screenshot, and concrete simulator/live validation commands.
  • add proof: 📸 screenshot: Contributor real behavior proof includes screenshot evidence. The PR body includes matched before/after screenshots, compact/expanded composer and light-mode screenshots, a live Gateway round-trip screenshot, and concrete simulator/live validation commands.
  • 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 (screenshot): The PR body includes matched before/after screenshots, compact/expanded composer and light-mode screenshots, a live Gateway round-trip screenshot, and concrete simulator/live validation commands.

Label justifications:

  • P3: This is a user-visible iOS chat polish and ergonomics feature with limited blast radius, not a confirmed runtime, data-loss, security, crash, or message-delivery defect.
  • 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 (screenshot): The PR body includes matched before/after screenshots, compact/expanded composer and light-mode screenshots, a live Gateway round-trip screenshot, and concrete simulator/live validation commands.
  • proof: sufficient: Contributor real behavior proof is sufficient. The PR body includes matched before/after screenshots, compact/expanded composer and light-mode screenshots, a live Gateway round-trip screenshot, and concrete simulator/live validation commands.
  • proof: 📸 screenshot: Contributor real behavior proof includes screenshot evidence. The PR body includes matched before/after screenshots, compact/expanded composer and light-mode screenshots, a live Gateway round-trip screenshot, and concrete simulator/live validation commands.
Evidence reviewed

PR surface:

Other +190. Total +190 across 9 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 9 377 187 +190
Total 9 377 187 +190

What I checked:

Likely related people:

  • steipete: CONTRIBUTING.md lists Peter Steinberger as a maintainer, and he authored both the linked design issue and this PR, so he is the clearest product-direction contact beyond merely opening the branch. (role: maintainer sponsor and design proposer; confidence: medium; files: CONTRIBUTING.md, apps/ios/Sources/Design/ChatProTab.swift)
  • mbelinky: CONTRIBUTING.md lists Mariano Belinky for the iOS app area, which makes this a relevant visual/product review route for native iOS UX. (role: listed iOS app area owner; confidence: medium; files: CONTRIBUTING.md, apps/ios/Sources/Design/ChatProTab.swift)
  • ngutman: CONTRIBUTING.md lists Nimrod Gutman for iOS and macOS app ownership, adjacent to shared OpenClawChatUI and companion-app UX decisions. (role: listed iOS app area owner; confidence: medium; files: CONTRIBUTING.md, apps/shared/OpenClawKit/Sources/OpenClawChatUI/ChatView.swift)
  • obviyus: Current-main blame for the sampled ChatProTab/OpenClawChatUI lines points to Ayaan Zaidi's grafted snapshot commit, but the grafted history makes this a weak routing signal. (role: recent area contributor; confidence: low; commits: 3558391a75fd; files: apps/ios/Sources/Design/ChatProTab.swift, apps/shared/OpenClawKit/Sources/OpenClawChatUI/ChatComposer.swift, apps/shared/OpenClawKit/Sources/OpenClawChatUI/ChatMessageViews.swift)
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.

@clawsweeper clawsweeper Bot added proof: sufficient ClawSweeper judged the real behavior proof convincing. proof: 📸 screenshot Contributor real behavior proof includes screenshot evidence. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. labels Jul 2, 2026
@steipete steipete self-assigned this Jul 2, 2026
@steipete

steipete commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Land-ready at 63f9010b42, tree-verified against the clean local rebase on current main.

Completed proof:

  • swift test --package-path apps/shared/OpenClawKit --filter ChatDynamicTypeSourceGuardTests
  • iPhone 17 Pro / iOS 26 Simulator: compact-to-expanded composer, light appearance, and attachment/Talk/Send 44-point geometry UI proof
  • live setup-code-paired Gateway chat round trip with a per-run provider-echoed marker, followed by paired relaunch into Control Overview
  • pnpm native:i18n:check
  • SwiftFormat with config/swiftformat
  • git diff --check
  • fresh Codex autoreview: no actionable findings; patch correct at 0.88 confidence
  • rebased exact-head hosted CI run 28572992933: terminal green
  • rebased exact-head iOS Periphery run 28572992910: terminal green
  • rebased exact-head Blacksmith artifact/Testbox run 28572993071: terminal green

The PR body contains matched before/after compact and expanded composer screenshots, light-appearance proof, and the live Gateway screenshot. No proof gaps remain.

@steipete
steipete requested a review from a team as a code owner July 2, 2026 07:26
@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation channel: matrix Channel integration: matrix app: web-ui App: web-ui gateway Gateway runtime scripts Repository scripts agents Agent runtime and tooling extensions: anthropic-vertex size: XL and removed size: L labels Jul 2, 2026
@steipete
steipete merged commit c4102f5 into main Jul 2, 2026
157 of 163 checks passed
@steipete
steipete deleted the codex/ios-chat-refinement branch July 2, 2026 07:40
@steipete

steipete commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Merged via squash.

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 3, 2026
* feat(ios): refine the chat experience

* feat(ios): refine the chat experience
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling app: ios App: ios app: web-ui App: web-ui channel: matrix Channel integration: matrix docs Improvements or additions to documentation extensions: anthropic-vertex gateway Gateway runtime maintainer Maintainer-authored PR P3 Low-priority cleanup, docs, polish, ergonomics, or speculative work. proof: 📸 screenshot Contributor real behavior proof includes screenshot evidence. proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. scripts Repository scripts size: XL status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve the native iOS chat hierarchy and composer

1 participant