Conversation
…panel Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
🤖 Augment PR SummarySummary: Removes the pre-open 🤖 Was this summary useful? React with 👍 or 👎 |
| }); | ||
| } | ||
|
|
||
| void sidePanelApi.open({ windowId: tab.windowId }).catch((error: unknown) => { |
There was a problem hiding this comment.
setOptions({ tabId, path, enabled }) was the only place in this file that explicitly configured/enabled the side panel; after removing it, can you confirm the extension’s manifest/WXT build always provides the correct side_panel.default_path and that the panel is enabled by default for the target tab?
Otherwise sidePanelApi.open() may fail or open an unexpected/blank panel on Chrome versions that require setOptions to establish an active panel for the tab/window first.
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
Summary
sidePanelApi.setOptions()call that was made before opening the side panel inopenSidePanelForContextMenuClicksetOptionscall (configuring tabId, path, and enabled) was redundant sincesidePanelApi.open()handles opening directlyTest plan
🤖 Generated with Claude Code