chore: disable tooltip window for now

[zerob13]

Summary by CodeRabbit

Bug Fixes

• Tooltip overlay functionality has been temporarily disabled.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

A private boolean flag isTooltipOverlayEnabled is introduced and set to false, with an early-return check added to getOrCreateTooltipOverlay that prevents tooltip overlay creation or reuse until the flag is enabled.

Changes

Cohort / File(s)	Summary
Tooltip overlay feature flag src/main/presenter/windowPresenter/index.ts	Added private flag isTooltipOverlayEnabled (set to false) with explanatory comment noting temporary instability. Modified getOrCreateTooltipOverlay to return null early if flag is disabled, preventing overlay window operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• fix: windows blink when drag and move #1224: Modifies tooltip overlay event handling and debouncing in the same windowPresenter file, suggesting coordination on tooltip overlay stability improvements.

Poem

🐰 A tooltip rests, its overlay sleeps,
Behind a flag where the feature flag keeps,
"Soon it shall wake," the code gently implies,
While stability grows beneath careful eyes! ✨

🚥 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 'chore: disable tooltip window for now' accurately describes the main change: disabling tooltip overlay functionality by adding a flag and early-return logic.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7a3b378 and a0d27ef.

📒 Files selected for processing (1)

• src/main/presenter/windowPresenter/index.ts

🧰 Additional context used

📓 Path-based instructions (8)

**/*.{ts,tsx,js,jsx}

📄 CodeRabbit inference engine (CLAUDE.md)

Use English for logs and comments in TypeScript/JavaScript code

Files:

• src/main/presenter/windowPresenter/index.ts

**/*.{ts,tsx}

📄 CodeRabbit inference engine (CLAUDE.md)

Use TypeScript with strict type checking enabled

Use OxLint for linting JavaScript and TypeScript files; ensure lint-staged hooks and typecheck pass before commits

Files:

• src/main/presenter/windowPresenter/index.ts

src/main/presenter/**/*.ts

📄 CodeRabbit inference engine (CLAUDE.md)

src/main/presenter/**/*.ts: Use EventBus to broadcast events from main to renderer via mainWindow.webContents.send()
Implement one presenter per functional domain in the main process

Files:

• src/main/presenter/windowPresenter/index.ts

src/main/**/*.ts

📄 CodeRabbit inference engine (CLAUDE.md)

src/main/**/*.ts: Use EventBus from src/main/eventbus.ts for decoupled inter-process communication
Context isolation must be enabled with preload scripts for secure IPC communication

Electron main process code should reside in src/main/, with presenters organized in presenter/ subdirectory (Window, Tab, Thread, Mcp, Config, LLMProvider), and app events managed via eventbus.ts

Files:

• src/main/presenter/windowPresenter/index.ts

**/*.{js,ts,tsx,jsx,vue,mjs,cjs}

📄 CodeRabbit inference engine (.cursor/rules/development-setup.mdc)

All logs and comments must be in English

Files:

• src/main/presenter/windowPresenter/index.ts

**/*.{js,ts,tsx,jsx,mjs,cjs}

📄 CodeRabbit inference engine (.cursor/rules/development-setup.mdc)

Use OxLint as the linter

Files:

• src/main/presenter/windowPresenter/index.ts

**/*.{js,ts,tsx,jsx,vue,json,mjs,cjs}

📄 CodeRabbit inference engine (.cursor/rules/development-setup.mdc)

Use Prettier as the code formatter

Files:

• src/main/presenter/windowPresenter/index.ts

**/*.{ts,tsx,vue}

📄 CodeRabbit inference engine (AGENTS.md)

**/*.{ts,tsx,vue}: Use camelCase for variable and function names; use PascalCase for types and classes; use SCREAMING_SNAKE_CASE for constants
Configure Prettier with single quotes, no semicolons, and line width of 100 characters. Run pnpm run format after completing features

Files:

• src/main/presenter/windowPresenter/index.ts

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build-check (x64)

🔇 Additional comments (2)

src/main/presenter/windowPresenter/index.ts (2)

45-46: LGTM! Clear feature flag with explanatory comment.

The temporary flag is well-documented and follows naming conventions. The "TEMP:" prefix makes it easy to find when re-enabling the feature.

---

1083-1083: LGTM! Safe early return that prevents overlay creation.

The early return is positioned correctly and all callers of getOrCreateTooltipOverlay already handle null returns gracefully. This cleanly disables the tooltip overlay feature without introducing any null pointer risks.

---

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 and usage tips.}