[AidenJohnston]

Brings main up to beta again. The only change since #179 is #172:

• The wall map's legend wraps instead of overlapping (cherry-picked from Sam's 1faa79b).
• The walk's first appearance is audited too, report-only (from ios tests: audit the walk strictly once held and in motion to report; triage of the UI test failures #165).
• The autopilot's gate waits four minutes instead of two.

#172's iOS CI passed on beta. Merge with "Create a merge commit".

🤖 Generated with Claude Code

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

[codemetrics-ai]

Note

Review skipped

Codemetrics is installed on this repository, but it hasn't been added to your team yet.

To enable AI code reviews for this repo, go to your dashboard and add house-scanning-master to your team's repositories.

[coderabbitai]

Warning

Review limit reached

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.

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 2e1d1614-d42b-4c57-ad41-2217eca9eebd

📥 Commits

Reviewing files that changed from the base of the PR and between d9ccaf2 and 42661c3.

📒 Files selected for processing (5)

• ios/HouseScan/Runtime/LaunchOptions.swift
• ios/HouseScan/Runtime/ScanEngine.swift
• ios/HouseScan/UI/Camera/WallTape.swift
• ios/HouseScanUITests/FullFlowUITests.swift
• ios/HouseScanUITests/ScreenStatesUITests.swift

---

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

[greptile-apps]

Confidence Score: 4/5

[Medium risk] Fixes wall map legend layout and extends test timeouts.

The PR appears safe to merge, though the new moving-walk audit should retain transient findings to provide its intended diagnostic coverage.

Findings

1.  Transient findings go unreported ▶

Fix with agent prompt

### Issue 1
ios/HouseScanUITests/FullFlowUITests.swift:395
The new in-motion audit records only findings seen twice, with a six-second pause between passes. Because the walk replay lasts about six seconds, a contrast issue visible when the walk first appears may be gone by the second pass. It then produces neither an attachment nor an audit log line, leaving the transition-only issues this audit was added to reveal unreported. Please retain first-pass findings for this report-only audit.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The PR adjusts the wall-map legend layout, adds a report-only audit during the walk, and lengthens the autopilot gate timeout. The new moving audit can discard the short-lived findings it is meant to surface.

Diagram

sequenceDiagram
  participant Walk
  participant Test
  participant Audit
  Test->>Walk: Wait for wallWalk to appear
  Test->>Audit: First in-motion pass
  Audit-->>Test: Transient finding
  Test->>Test: Wait six seconds
  Walk-->>Test: Replay settles
  Test->>Audit: Second pass
  Audit-->>Test: Finding absent
  Note over Test,Audit: No repeated finding is reported

Loading

_{Reviews (1) · Last reviewed commit: "Merge pull request #172 from SamGu-NRX/t..."}