Chatpane / Tickets / #140 Harden Carrier reliability and desktop behavior

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

[!NOTE]

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:
- @coderabbitai resume to resume automatic reviews.
- @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:
- [ ] ▶️ Resume reviews
- [ ] 🔍 Trigger review

📝 Walkthrough
## Walkthrough The PR separates update discovery from consent-based installation, adds autoplay and custom CSS controls, refines Messenger and tray behavior, introduces bounded MCP page inspection, narrows capabilities, adds Linux theme observation, and updates documentation and validation. ### Changes **Update and settings controls** |Layer / File(s)|Summary| |---|---| |**Update commands and settings UI**
`src-tauri/src/commands.rs`, `dist/settings.html`, `src-tauri/src/lib.rs`, `src-tauri/src/settings.rs`|Update discovery is separated from consented installation with single-flight installation and remembered availability. Settings expose autoplay, Windows tray, custom CSS, and update-install controls.| |**Shortcuts and Help menu**
`inject/src/panel/*`, `src-tauri/inject/panel.js`, `src-tauri/src/menu.rs`|Shortcut metadata, F1 and `/` toggles, Keyboard Shortcuts, and Report an Issue actions are added.| **Messenger and native behavior** |Layer / File(s)|Summary| |---|---| |**Media and Messenger refinements**
`inject/src/messenger/*`, `src-tauri/inject/messenger.js`|Autoplay suppression, live-track counting, cookie-policy detection, emoji reaction tagging, settings-button detection, unread readiness, feature diagnostics, and Windows zoom relayout are updated.| |**Notifications and tray lifecycle**
`src-tauri/src/{notifications,settings,tray,window}.rs`|Notifications are rate-limited, tray notices are persisted and guarded, Windows reveal behavior uses generation tokens, and auto-hide behavior depends on tray availability.| |**Reachability and custom CSS**
`src-tauri/src/{commands,custom_css,url_rules,window}.rs`, `dist/index.html`, `dist/settings.html`|DNS failures are surfaced with an Open anyway bypass, while custom CSS is provisioned, validated, scoped to Messenger URLs, and injected after page load.| **Capabilities and platform support** |Layer / File(s)|Summary| |---|---| |**MCP bridge and capability scopes**
`src-tauri/inject/mcp-bridge.js`, `src-tauri/capabilities/*`, `src-tauri/dev-capabilities/*`|Release permissions are narrowed, development event listening is isolated, and bounded `get-page-state` and `get-page-map` responders are added.| |**Linux theme and documentation**
`src-tauri/src/linux.rs`, `src-tauri/src/lib.rs`, `src-tauri/Cargo.toml`, `README.md`, `docs/index.html`, `inject/src/messenger/lib/injected-assets.test.ts`|Linux portal theme observation is added, update and trust documentation is revised, build metadata is refreshed, and injected-asset tests verify generated artifacts and locale content.| **Estimated code review effort:** 4 (Complex) | ~60 minutes ### Sequence Diagram(s) :::mermaid sequenceDiagram participant SettingsWindow participant UpdateCommands participant Installer SettingsWindow->>UpdateCommands: check_for_updates() UpdateCommands-->>SettingsWindow: available version or up-to-date SettingsWindow->>UpdateCommands: install_update() after confirmation UpdateCommands->>Installer: download and install update Installer-->>SettingsWindow: installation result **Possibly related issues** - [kristofferR/Carrier#135](https://github.com/kristofferR/Carrier/issues/135) — Covers the bundled update, capability, shortcut, tray, autoplay, and heuristic changes. - [kristofferR/Carrier#71](https://github.com/kristofferR/Carrier/issues/71) — Covers the Windows zoom resize dispatch. **Possibly related PRs** - [[kristofferR/Carrier#10](https://github.com/kristofferR/Carrier/issues/10)](https://github.com/kristofferR/Carrier/pull/10) — Related to the Settings-window update flow and tabbed settings UI. - [[kristofferR/Carrier#41](https://github.com/kristofferR/Carrier/issues/41)](https://github.com/kristofferR/Carrier/pull/41) — Related to the Messenger reachability and Open anyway flow. - [[kristofferR/Carrier#11](https://github.com/kristofferR/Carrier/issues/11)](https://github.com/kristofferR/Carrier/pull/11) — Related to the shared unread-badge and MCP bridge paths. **Suggested labels:** `javascript`

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)
| Check name | Status | Explanation | | :------------------------: | :------- | :----------------------------------------------------------------------------------------------------------- | | Docstring Coverage | ✅ Passed | Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. | | 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. | | Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. | | Title check | ✅ Passed | The title is broad, but it accurately reflects the PR’s focus on reliability and desktop behavior hardening. |

✨ Finishing Touches

📝 Generate docstrings
- [ ] Create stacked PR - [ ] Commit on current branch

🧪 Generate unit tests (beta)
- [ ] Create PR with unit tests - [ ] Commit unit tests in branch `codex/windows-issue-sweep`

_{Comment @coderabbitai help to get the list of available commands.}

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

✅ Action performed
Review finished. > Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: coderabbitai[bot]

Rate Limit Exceeded

@kristofferR have exceeded the limit for the number of chat messages per hour. Please wait 11 minutes and 43 seconds before sending another message.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2026-07-18

Originally posted by: kristofferR

@coderabbitai review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Harden Carrier reliability and desktop behavior

Facebook Messenger on your desktop, without the Facebook clutter

Milestone

Searches

Help

#140 Harden Carrier reliability and desktop behavior

Summary

Testing

Summary by CodeRabbit

Related

Discussion

Reviews paused

Rate Limit Exceeded