feat(core): extract ShortcutsPlugin into @editorjs/shortcuts-plugin (#180)

Move the last in-tree EditorjsPlugin out of packages/core into a
standalone @editorjs/shortcuts-plugin package, mirroring the
clipboard-plugin extraction (#129). Closes #171.

- New package packages/plugins/shortcuts-plugin scaffolded from
  clipboard-plugin (package.json, tsconfigs, eslint, jest, stryker).
- ShortcutsPlugin.ts moved verbatim into src/index.ts (no source
  changes) and covered by a new index.spec.ts (12 cases, 100% coverage)
  that closes the pre-existing test gap.
- Wire core: add the workspace dependency, tsconfig project references
  (tsconfig + tsconfig.build), and switch the import to the package;
  delete the now-empty src/plugins directory.
- Dedicated CI workflow shortcuts-plugin.yml (package-check shape).
- Move the "Keyboard shortcuts plugin" requirement out of the core
  OpenSpec spec into a new specs/shortcuts-plugin/spec.md.
- Ignore .claude/worktrees/ in .gitignore.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: ilyamore88

.github/workflows/shortcuts-plugin.yml

@@ -0,0 +1,20 @@
+name: Shortcuts plugin check
+on:
+  pull_request:
+  merge_group:
+
+permissions:
+  contents: read
+  pull-requests: write
+  issues: write
+  actions: read
+
+jobs:
+  package-check:
+    uses: ./.github/workflows/package-check.yml
+    with:
+      package-name: '@editorjs/shortcuts-plugin'
+      working-directory: './packages/plugins/shortcuts-plugin'
+      include-mutations: true
+    secrets:
+      stryker_dashboard_api_key: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}

.gitignore

@@ -31,3 +31,4 @@ jest-report.json
 **/.env
 /--help/
 .claude/settings.local.json
+.claude/worktrees/

openspec/specs/core/spec.md

@@ -95,13 +95,3 @@ The system SHALL provide `UndoRedoManager`, which batches consecutive model even
 - **THEN** the default undo/redo behavior is suppressed
 
 Implemented in `src/components/UndoRedoManager.ts`, validated by its co-located `.spec.ts`.
-
-### Requirement: Keyboard shortcuts plugin
-The system SHALL provide a `ShortcutsPlugin` (an `EditorjsPlugin`) that maps keyboard shortcuts declared in a tool's `options.shortcut` to inline-tool application through the `EditorAPI`.
-
-#### Scenario: Triggering an inline tool via shortcut
-- **GIVEN** an inline tool is registered with `options.shortcut` set to a key combination (e.g. `CMD+B`)
-- **WHEN** that key combination is pressed while the editor has focus
-- **THEN** `ShortcutsPlugin` applies the corresponding inline tool to the current selection via the `EditorAPI`
-
-Implemented in `src/plugins/ShortcutsPlugin.ts`.

openspec/specs/shortcuts-plugin/spec.md

@@ -0,0 +1,49 @@
+# Shortcuts Plugin
+
+## Purpose
+
+`@editorjs/shortcuts-plugin` is a built-in `EditorjsPlugin` that maps keyboard shortcuts declared in a tool's `options.shortcut` to inline-tool application through the `EditorAPI`. It subscribes to tool-loaded events to collect shortcuts and to delegated keydown events to dispatch them.
+
+## Requirements
+
+### Requirement: Keyboard shortcuts plugin
+The system SHALL provide a `ShortcutsPlugin` (an `EditorjsPlugin`) that, on construction, subscribes to the `core:tool:loaded` and `ui:key-down` events, registers each loaded tool's string `options.shortcut`, and applies the matching inline tool to the current selection via the `EditorAPI` when its shortcut is pressed.
+
+#### Scenario: Registering a shortcut from a loaded tool
+- **GIVEN** a tool is loaded whose merged `options.shortcut` is a string (e.g. `CMD+B`)
+- **WHEN** the `core:tool:loaded` event fires
+- **THEN** the plugin registers a mapping from that shortcut string to the tool's name
+
+#### Scenario: Ignoring tools without a string shortcut
+- **GIVEN** a tool is loaded whose `options.shortcut` is absent or is not a string
+- **WHEN** the `core:tool:loaded` event fires
+- **THEN** the plugin registers no shortcut for that tool
+
+#### Scenario: Triggering an inline tool via shortcut
+- **GIVEN** an inline tool is registered with `options.shortcut` set to a key combination (e.g. `CMD+B`)
+- **WHEN** that key combination is pressed while the editor has focus
+- **THEN** the plugin prevents the native event's default action and applies the corresponding inline tool to the current selection via `api.selection.applyInlineTool`
+
+#### Scenario: Matching only the first registered shortcut
+- **GIVEN** more than one registered shortcut would match the keydown
+- **WHEN** the `ui:key-down` event fires
+- **THEN** the plugin applies only the first matching tool and stops
+
+#### Scenario: Ignoring keydown during IME composition
+- **GIVEN** the native keydown event has `isComposing === true`
+- **WHEN** the `ui:key-down` event fires
+- **THEN** the plugin performs no matching and leaves the native event untouched
+
+#### Scenario: Tolerating a missing caret when applying a tool
+- **GIVEN** applying the inline tool throws an `IndexError` (e.g. no caret in a text input)
+- **WHEN** a matching shortcut is dispatched
+- **THEN** the plugin swallows the error and leaves the editor unchanged, while any other error propagates
+
+#### Scenario: Releasing shortcuts on destroy
+- **GIVEN** a `ShortcutsPlugin` instance has registered shortcuts
+- **WHEN** `destroy()` is called
+- **THEN** it clears the registered shortcuts so subsequent keydowns dispatch nothing
+
+Shortcuts for block tools and block tunes (a `shortcuts` map in tool `options`) are reserved for future work and not yet implemented.
+
+Implemented in `src/index.ts`, validated by its co-located `.spec.ts`.

packages/core/package.json

@@ -49,6 +49,7 @@
     "@editorjs/model": "workspace:^",
     "@editorjs/paragraph": "workspace:^",
     "@editorjs/sdk": "workspace:^",
+    "@editorjs/shortcuts-plugin": "workspace:^",
     "inversify": "^8.1.0",
     "reflect-metadata": "^0.2.2"
   }

packages/core/src/index.ts

@@ -23,7 +23,7 @@ import { Paragraph } from '@editorjs/paragraph';
 import { BoldInlineTool } from '@editorjs/bold';
 import { ItalicInlineTool } from '@editorjs/italic';
 import { LinkInlineTool } from '@editorjs/inline-link';
-import { ShortcutsPlugin } from './plugins/ShortcutsPlugin.js';
+import { ShortcutsPlugin } from '@editorjs/shortcuts-plugin';
 import { DOMAdapters } from '@editorjs/dom-adapters';
 import { BlocksManager } from './components/BlockManager.js';
 import { BlockRenderer } from './components/BlockRenderer.js';

packages/core/tsconfig.build.json

@@ -30,6 +30,9 @@
     },
     {
       "path": "../plugins/clipboard-plugin/tsconfig.build.json"
+    },
+    {
+      "path": "../plugins/shortcuts-plugin/tsconfig.build.json"
     }
   ]
 }

packages/core/tsconfig.json

@@ -49,6 +49,9 @@
     },
     {
       "path": "../plugins/clipboard-plugin/tsconfig.build.json"
+    },
+    {
+      "path": "../plugins/shortcuts-plugin/tsconfig.build.json"
     }
   ]
 }

packages/plugins/shortcuts-plugin/.gitignore

@@ -0,0 +1,24 @@
+.yarn/*
+!.yarn/patches
+!.yarn/plugins
+!.yarn/releases
+!.yarn/sdks
+!.yarn/versions
+
+# Swap the comments on the following lines if you don't wish to use zero-installs
+# Documentation here: https://yarnpkg.com/features/zero-installs
+#!.yarn/cache
+#.pnp.*
+
+# IDE
+.idea/*
+
+node_modules/*
+dist/*
+
+# tests
+coverage/
+reports/
+
+# stryker temp files
+.stryker-tmp

packages/plugins/shortcuts-plugin/README.md

@@ -0,0 +1,3 @@
+# Shortcuts Plugin
+
+Built-in Editor.js Plugin that maps keyboard shortcuts declared in a tool's `options.shortcut` to inline-tool application.