feat(linux): auto-hide menu bar for cleaner window appearance

[xsistens]

Summary

• Sets autoHideMenuBar: true in the BrowserWindow options
• Hides the default Electron menu bar on Linux for a cleaner look
• Users can still toggle the menu bar by pressing Alt

Context

On Linux, Electron shows a native menu bar by default that takes up vertical space and doesn't match the app's custom UI. This single-line change hides it while keeping it accessible via Alt key.

Test plan

• Launch Maestro on Linux — verify menu bar is hidden by default
• Press Alt — verify menu bar toggles visible
• Verify no impact on macOS/Windows (property is Linux-specific in practice)

Summary by CodeRabbit

• New Features
  • Menu bar now automatically hides, providing more screen space for content.

[pedramamini]

Thank you for this contribution. We're deep in the throws of getting 0.15.0 out the door and then implementing a short code freeze to complete some refactoring, but then will swing over to playing with this.

[pedramamini]

@greptile review
@coderabbitai review

[coderabbitai]

@pedramamini: I'll review the changes in this PR right away.

✅ Actions performed

Review triggered.

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.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

Added autoHideMenuBar: true to BrowserWindow options and reformatted the did-fail-load event handler to use a multi-line callback format while preserving existing error handling logic that ignores errorCode -3 and reports to logger and Sentry.

Changes

Cohort / File(s)	Summary
Window Manager Configuration src/main/app-lifecycle/window-manager.ts	Added autoHideMenuBar: true to BrowserWindow constructor options and reformatted did-fail-load event handler callback to multi-line format without functional changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding autoHideMenuBar property to hide the menu bar on Linux for a cleaner appearance.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[greptile-apps]

Greptile Summary

Added autoHideMenuBar: true to BrowserWindow options to hide the default Electron menu bar on Linux. Users can still access it by pressing Alt. The change also includes unrelated formatting improvements to the did-fail-load event handler.

• Single-line addition improves UI consistency across platforms
• Aligns with existing custom window chrome (titleBarStyle: 'hiddenInset')
• No functional impact on macOS (which doesn't use in-window menu bars)
• Windows behavior already auto-hides menu bars by default in most cases

Confidence Score: 5/5

• This PR is safe to merge with no risk
• Single-line configuration change using a well-documented Electron option. No logic changes, no security implications, and the feature is easily reversible if issues arise
• No files require special attention

Important Files Changed

Filename	Overview
src/main/app-lifecycle/window-manager.ts	Added autoHideMenuBar: true to hide default Electron menu bar on Linux, plus unrelated formatting changes

_{Last reviewed commit: 92ed0f5}