fix(global-commands): use isContentEditable for the editable guard

[waleedlatif1]

Summary

• Follow-up to feat(sidebar): add Cmd+B shortcut to toggle sidebar collapse #5618, addressing Greptile's ancestor-editability finding on the release PR
• Replace the attribute-string check in the global-command editable guard with element.isContentEditable — the browser's computed editability, which handles focusable descendants of a contenteditable root, contenteditable="false" islands, invalid attribute values, and plaintext-only natively (same approach as Mousetrap/hotkeys-js)
• Tests stub isContentEditable (jsdom doesn't implement it) and add a focused-descendant case

Type of Change

• Bug fix

Testing

Provider tests pass (7/7)

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jul 12, 2026 5:06am

[cursor]

PR Summary

Low Risk
Small, localized change to keyboard shortcut gating in the workspace provider, with updated unit tests and no auth or data-path impact.

Overview
Fixes the editable guard for global shortcuts (allowInEditable: false) so it matches browser behavior instead of reading the contenteditable attribute on the focused node only.

isEditableElement now uses HTMLElement.isContentEditable, so focus inside a rich-text editor (including tabbable descendants under a contenteditable root) still blocks shortcuts like Mod+K, while contenteditable="false" (e.g. read-only editor) no longer blocks them.

Tests stub isContentEditable in jsdom and add coverage for a focused descendant inside a contenteditable root.

^{Reviewed by Cursor Bugbot for commit b722f2d. Configure here.}

[greptile-apps]

Greptile Summary

This PR updates the global shortcut editable guard. The main changes are:

• Uses browser-computed isContentEditable for editable-region detection.
• Keeps inputs and textareas blocked for non-editable shortcuts.
• Adds tests for contenteditable roots, descendants, and contenteditable="false" cases.

Confidence Score: 5/5

This looks safe to merge.

• No blocking issues found in the changed code.

Important Files Changed

Filename	Overview
apps/sim/app/workspace/[workspaceId]/providers/global-commands-provider.tsx	Updates the editable guard to use computed contenteditability while preserving input and textarea handling.
apps/sim/app/workspace/[workspaceId]/providers/global-commands-provider.test.tsx	Adds focused coverage for contenteditable descendants and stubs computed editability for jsdom.

_{Reviews (2): Last reviewed commit: "chore(lint): keep focusable span in edit..." | Re-trigger Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit b722f2d. Configure here.}