Fix inputs and values attachments to be data-first by gohabereg · Pull Request #127 · editor-js/document-model

gohabereg · 2026-04-17T17:55:57Z

Changed BlockToolAdapter API to make it data-first
Replaced typedi with injectify
Added abstract BlockToolAdapter class
Adapters now connected as a plugin to the core
Core uses only BlockToolAdapter, all the interdependencies between adapters happens inside the DOMAdapter plugin

Notes:

It seems we would need to move towards block id's

github-actions · 2026-04-17T17:57:11Z

⏭️ No files to mutate for ./packages/model

github-actions · 2026-04-17T17:57:23Z

Coverage report for `./packages/dom-adapters`

St.^❔	Category	Percentage	Covered / Total
🟢	Statements	96.43%	27/28
🟢	Branches	86.96%	20/23
🟢	Functions	100%	5/5
🟢	Lines	96.43%	27/28

Test suite run success

11 tests passing in 2 suites.

Report generated by 🧪jest coverage report action from 0ad4ae1

github-actions · 2026-04-17T17:57:23Z

⏭️ No files to mutate for ./packages/core

github-actions · 2026-04-17T17:57:37Z

Coverage report for `./packages/model`

St.^❔	Category	Percentage	Covered / Total
🟢	Statements	100% (+0.11% 🔼)	905/905
🟢	Branches	99.63% (+0.01% 🔼)	266/267
🟢	Functions	97.75% (-0.42% 🔻)	217/222
🟢	Lines	100% (+0.12% 🔼)	869/869

Show files with reduced coverage 🔻

St.^❔	File	Statements	Branches	Functions	Lines
🟢	`...` / index.ts	100%	100%	93.94% (-6.06% 🔻)	100%

Test suite run success

475 tests passing in 25 suites.

Report generated by 🧪jest coverage report action from 0ad4ae1

github-actions · 2026-04-17T17:57:49Z

Coverage report for `./packages/core`

St.^❔	Category	Percentage	Covered / Total
🟢	Statements	100%	129/129
🟢	Branches	100%	46/46
🟢	Functions	100%	27/27
🟢	Lines	100%	120/120

Show new covered files 🐣

St.^❔	File	Statements	Branches	Functions	Lines
🟢	tokens.ts	100%	100%	100%	100%
🟢	`...` / SelectionManager.ts	100%	100%	100%	100%
🟢	`...` / BlockManager.ts	100%	100%	100%	100%
🟢	api/BlocksAPI.ts	100%	100%	100%	100%
🟢	`...` / BlockRenderer.ts	100%	100%	100%	100%
🟢	api/SelectionAPI.ts	100%	100%	100%	100%
🟢	`...` / DocumentAPI.ts	100%	100%	100%	100%

Test suite run success

74 tests passing in 7 suites.

Report generated by 🧪jest coverage report action from 0ad4ae1

github-actions · 2026-04-17T17:58:10Z

Coverage report for `./packages/ot-server`

St.^❔	Category	Percentage	Covered / Total
🟡	Statements	70.97%	22/31
🔴	Branches	20%	1/5
🟡	Functions	75%	6/8
🟡	Lines	68.97%	20/29

Test suite run success

4 tests passing in 1 suite.

Report generated by 🧪jest coverage report action from 0ad4ae1

github-actions · 2026-04-17T17:58:22Z

Coverage report for `./packages/collaboration-manager`

St.^❔	Category	Percentage	Covered / Total
🟢	Statements	92.51%	358/387
🟢	Branches	85.51%	118/138
🟢	Functions	98.15%	53/54
🟢	Lines	92.41%	353/382

Test suite run success

117 tests passing in 7 suites.

Report generated by 🧪jest coverage report action from 0ad4ae1

github-actions · 2026-04-17T18:00:54Z

⏭️ No files to mutate for ./packages/dom-adapters

neSpecc · 2026-04-17T18:28:07Z

+    this.attachInput(dataKey, input);
+  }
+
+  public registerKey(keyRaw: string, type: 'text' | 'value', initialData?: unknown): void {


Suggested change

public registerKey(keyRaw: string, type: 'text' | 'value', initialData?: unknown): void {

public registerDataKey(keyRaw: string, type: 'text' | 'value', initialData?: unknown): void {

…nputs-registration

Copilot

Pull request overview

This PR refactors the editor’s adapter/tool integration to be “data-first” by introducing an abstract BlockToolAdapter in the SDK, moving DOM-specific interdependencies into a dedicated DOM adapter plugin, and migrating DI from typedi to inversify across core + dom-adapters. It also adjusts model event timing (sync vs microtask) to preserve event ordering guarantees.

Changes:

Introduces an SDK BlockToolAdapter base class + new adapter-related events/types, and updates tool constructor typing to support adapter specialization.
Adds DOMAdapters as a core plugin that owns DOM adapter wiring via an Inversify container, including a shared InputsRegistry.
Migrates core and dom-adapters DI from typedi to inversify, and updates event dispatch ordering in the model (BlockAdded sync; BlockRemoved/data-node events microtasked).

Reviewed changes

Copilot reviewed 43 out of 45 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
yarn.lock	Adds Inversify dependency graph; removes typedi.
packages/sdk/src/tools/facades/InlineToolFacade.ts	Caches inline tool instance to enforce singleton behavior.
packages/sdk/src/tools/facades/BaseToolFacade.ts	Broadens `BlockToolConstructor` typing to allow generic adapters.
packages/sdk/src/entities/index.ts	Exports `BlockToolAdapter` as a runtime value; adds adapter plugin type export.
packages/sdk/src/entities/EventBus/events/index.ts	Re-exports adapter events barrel.
packages/sdk/src/entities/EventBus/events/adapter/index.ts	Adds adapter event barrel exports.
packages/sdk/src/entities/EventBus/events/adapter/ValueNodeChanged.ts	Adds adapter event for value updates.
packages/sdk/src/entities/EventBus/events/adapter/KeyRemoved.ts	Adds adapter event for key removal.
packages/sdk/src/entities/EventBus/events/adapter/KeyAdded.ts	Adds adapter event for key addition.
packages/sdk/src/entities/EventBus/events/adapter/AdapterEventType.ts	Defines adapter event type namespace.
packages/sdk/src/entities/EntityType.ts	Adds `PluginType.Adapter` for adapter plugins.
packages/sdk/src/entities/EditorjsPlugin.ts	Makes plugin constructor generic over params/instance.
packages/sdk/src/entities/EditorjsAdapterPlugin.ts	Introduces adapter plugin interfaces + constructor typing.
packages/sdk/src/entities/BlockToolAdapter.ts	Replaces interface with abstract base class that listens to model events and emits adapter events.
packages/sdk/src/entities/BlockTool.ts	Adds adapter generic to block tool constructor/options typing.
packages/playground/src/components/index.ts	Removes `Input` component export.
packages/playground/src/components/Input.vue	Removes old adapter-attached input component.
packages/model/src/entities/EditorDocument/index.ts	Adjusts block event timing; adds `getDataNode`.
packages/model/src/entities/BlockNode/index.ts	Adds `getDataNode`; microtasks data-node add/remove events; changes initialization path.
packages/model/src/EditorJSModel.ts	Adds `getCaret` and `getDataNode` passthroughs.
packages/dom-adapters/tsconfig.json	Enables `emitDecoratorMetadata` for DI.
packages/dom-adapters/src/tokens.ts	Adds Inversify token(s) for dom-adapters container.
packages/dom-adapters/src/index.ts	Adds `DOMAdapters` adapter plugin with Inversify container + adapter factory.
packages/dom-adapters/src/InputsRegistry/index.ts	Adds shared blockIndex/dataKey→HTMLElement registry.
packages/dom-adapters/src/FormattingAdapter/index.ts	Switches to Inversify DI and tool-loaded wiring via core EventBus.
packages/dom-adapters/src/CaretAdapter/index.ts	Switches to shared `InputsRegistry` and Inversify; removes block attachment tracking.
packages/dom-adapters/src/BlockToolAdapter/index.ts	Replaces prior adapter impl with `DOMBlockToolAdapter` extending SDK base adapter and using registry.
packages/dom-adapters/package.json	Adds `inversify` + `reflect-metadata`.
packages/core/src/tools/internal/block-tools/paragraph/index.ts	Updates Paragraph tool to use DOM adapter methods and adapter key events.
packages/core/src/tools/ToolsManager.ts	Migrates DI from typedi to Inversify token injection.
packages/core/src/tokens.ts	Adds core Inversify tokens (EditorConfig, Adapter).
packages/core/src/index.ts	Replaces typedi container with Inversify; introduces plugin container and adapter initialization.
packages/core/src/components/SelectionManager.ts	Removes reliance on FormattingAdapter for apply; formats directly via model using caret segments.
packages/core/src/components/SelectionManager.spec.ts	Updates mocks/tests to match new SelectionManager behavior.
packages/core/src/components/BlockRenderer.ts	New component: renders blocks from model block events using adapter plugin.
packages/core/src/components/BlockRenderer.spec.ts	Adds unit tests for BlockRenderer.
packages/core/src/components/BlockManager.ts	Removes model event/render responsibilities (delegated to BlockRenderer); updates caret lookup.
packages/core/src/components/BlockManager.spec.ts	Updates tests for BlocksManager refactor and caret lookup changes.
packages/core/src/api/index.ts	Migrates API DI to Inversify property injection.
packages/core/src/api/SelectionAPI.ts	Migrates SelectionAPI DI to Inversify.
packages/core/src/api/DocumentAPI/DocumentAPI.ts	Migrates DocumentAPI DI to Inversify.
packages/core/src/api/BlocksAPI.ts	Migrates BlocksAPI DI to Inversify token injection.
packages/core/src/api/BlocksAPI.integration.spec.ts	Updates for microtasked `BlockRemovedEvent`.
packages/core/package.json	Replaces typedi with inversify; keeps reflect-metadata.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

github-actions · 2026-04-27T23:05:46Z

Coverage report for `./packages/core`

Caution

Multiple errors occurred

 1 | Error: The process '/usr/local/bin/yarn' failed with exit code 1
 2 | Getting code coverage data failed.

St.^❔	Category	Percentage	Covered / Total
🟢	Statements	100%	129/129
🟢	Branches	100%	46/46
🟢	Functions	100%	27/27
🟢	Lines	100%	120/120

Test suite run success

73 tests passing in 7 suites.

Report generated by 🧪jest coverage report action from ba5077d

Fix inputs and values attachments to be data-first

0d5817e

neSpecc reviewed Apr 17, 2026

View reviewed changes

gohabereg added 3 commits April 17, 2026 22:01

Merge branch 'main' of github.com:editor-js/document-model into fix/i…

c5ff274

…nputs-registration

Load adapters as plugin

e4c246d

Merge branch 'main' of github.com:editor-js/document-model into fix/i…

40a6409

…nputs-registration

gohabereg requested a review from Copilot April 26, 2026 23:13

Copilot started reviewing on behalf of gohabereg April 26, 2026 23:13 View session

Copilot AI reviewed Apr 26, 2026

View reviewed changes

gohabereg added 3 commits April 27, 2026 23:20

Fix tests

abe46db

Review comments

40c4284

(try to) fix test run for core package

ba5077d

gohabereg added 3 commits April 28, 2026 00:25

Run base branch coverage manually

e424775

Fix unit-tests action: use ArtiomTr with pre-generated coverage files

5990de3

Try just PR branch tests

208c95d

editor-js deleted a comment from github-actions Bot Apr 28, 2026

gohabereg added 3 commits April 28, 2026 17:29

Try clear before build

0996eac

Try no cache

8958a51

Use latest node

0ea33bc

e11sy reviewed Apr 28, 2026

View reviewed changes

Review comments resolved

d265bd3

neSpecc reviewed May 2, 2026

View reviewed changes

gohabereg added 2 commits May 2, 2026 16:07

Resolve review comments

7fe438f

Add destoryBlockToolAdapter method to Adapter plugin

0ad4ae1

neSpecc approved these changes May 4, 2026

View reviewed changes

e11sy approved these changes May 5, 2026

View reviewed changes

gohabereg added this pull request to the merge queue May 5, 2026

Merged via the queue into main with commit 33f2dc0 May 5, 2026
24 checks passed

gohabereg deleted the fix/inputs-registration branch May 5, 2026 00:21

	public registerKey(keyRaw: string, type: 'text' \| 'value', initialData?: unknown): void {
	public registerDataKey(keyRaw: string, type: 'text' \| 'value', initialData?: unknown): void {

Conversation

gohabereg commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./packages/dom-adapters

Test suite run success

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./packages/model

Test suite run success

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./packages/core

Test suite run success

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./packages/ot-server

Test suite run success

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report for ./packages/collaboration-manager

Test suite run success

Uh oh!

github-actions Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

neSpecc Apr 17, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Apr 27, 2026

Coverage report for ./packages/core

Test suite run success

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gohabereg commented Apr 17, 2026 •

edited

Loading

github-actions Bot commented Apr 17, 2026 •

edited

Loading

github-actions Bot commented Apr 17, 2026 •

edited

Loading

Coverage report for `./packages/dom-adapters`

github-actions Bot commented Apr 17, 2026 •

edited

Loading

github-actions Bot commented Apr 17, 2026 •

edited

Loading

Coverage report for `./packages/model`

github-actions Bot commented Apr 17, 2026 •

edited

Loading

Coverage report for `./packages/core`

github-actions Bot commented Apr 17, 2026 •

edited

Loading

Coverage report for `./packages/ot-server`

github-actions Bot commented Apr 17, 2026 •

edited

Loading

Coverage report for `./packages/collaboration-manager`

github-actions Bot commented Apr 17, 2026 •

edited

Loading

Coverage report for `./packages/core`