test: use current navigation for GIF send roundtrip

[wesbillman]

Authored by Brain on Wes's behalf.

Summary

• Replace the removed Pages → Messages sidebar selector with the existing openPage(page, "Messages") helper.
• Test-only repair against main; preserves every GIF publication receipt, attachment rendering, and unexpected-event assertion. No production or identity changes; feat(identity): add native macOS import, creation and backup #308 is unchanged.

Evidence

• Before: CI run 36197168502 failed this navigation step in both Chromium and WebKit on merge SHA 4168c4364db45d9f0d25ded1f0141d78754b61b5. The stale selector is also present on base d4fa23b07aa7d35be8b88258474e3b4144c22f9f.
• After: bin/pnpm test:browser tests/browser/gifs.spec.mjs --project chromium --project webkit --no-deps — 8/8 passed, both full GIF files, zero retries, 12.2s Playwright elapsed on macOS. Run used base plus this exact diff; test file SHA-256 b603b274ee27ee7a52af86bf4d7258f3708badfbd461631aea6c405db2f200df.
• Browser cases added/removed: 0/0. Existing browser integration coverage is unchanged; no replacement coverage or infrastructure changes.
• Managed security hooks and tracked repository pre-commit/pre-push hooks ran successfully; signed-off commit and remote head verified.

Remaining gates

• Independent read-only review by Pinky found no blockers at af974dd00b46e7a028ba19d42db1027e6fa3d8f6; necessity, helper behavior, and unchanged assertions were checked. The reviewer did not rerun tests.
• Hosted DCO passed. Hosted CI completion and explicit human validation remain unverified by this agent. Full repository validation is deferred to CI; this does not certify feat(identity): add native macOS import, creation and backup #308 or a release.
• Human check: run the browser command above and confirm all eight cases pass.

Originating Buzz channel: 5e3d091d-77b7-4385-872f-59bd8b088393 (implement-identity-setting).

[wesbillman]

Star Lord’s automated source review (via Wes’s account)

No actionable findings. This is a minimal, test-only navigation repair: one existing setup sequence becomes await openPage(page, "Messages").

• The old Pages sidebar selector is not mounted by the current app composition: App.tsx supplies a sidebar callback that ignores the shell’s page-navigation argument. The existing helper instead uses Search Buzz → Pages → Messages, invokes the same page-selection owner, and waits for the dialog to close.
• The helper is already imported and used by the other GIF cases. The exact diff preserves the production-broker fixture, publication/event-ID and acceptance-receipt checks, rendered attachment assertions, and unexpected-event assertion. Browser cases added/removed: 0/0. No production behavior, timeouts, retries, or assertion budgets change.

Pinned scope: head af974dd00b46e7a028ba19d42db1027e6fa3d8f6; base/merge-base d4fa23b07aa7d35be8b88258474e3b4144c22f9f. The reviewed test file’s SHA-256 matches the PR’s reported validation input: b603b274ee27ee7a52af86bf4d7258f3708badfbd461631aea6c405db2f200df.

Validation limits: immutable source and relevant navigation callers inspected; extracted bytes verified; git diff --check passed. The author reports 8/8 Chromium/WebKit cases passing on macOS; I did not rerun them. One hosted-check snapshot showed DCO, Semgrep, and zizmor passing, with JavaScript, Rust/tool integration, browser measurements, and all six functional browser shards still running; Windows was skipped. Full CI completion and explicit human acceptance remain unverified. No tests, builds, PR code, app launches, or live-account operations were executed. Non-blocking COMMENT only—not approval, merge authorization, or validation of #308.