vvzvlad/gitmost
1
0
Fork 0
Code Issues 17 Pull Requests 11 Packages Projects Releases Activity

feat(#246): inline spoiler mark (blur + click-reveal, lossless Markdown) #259

Merged
vvzvlad merged 2 commits from feat/246-spoiler into develop 2026-06-30 01:47:47 +03:00
Conversation 4 Commits 2 Files Changed 19 +507
agent_coder commented 2026-06-29 23:23:20 +03:00
Collaborator
Copy Link

Summary

Инлайновый спойлер (скрытый текст в духе Telegram/Discord): TipTap-марка spoiler, рендерится как <span data-spoiler="true" class="spoiler">…</span>, замазывается blur'ом и раскрывается по клику (состояние «раскрыт» — UI-only класс is-revealed, в документ не пишется). closes #246

Реализовано ровно по принятым в issue решениям (марка, а не узел; lossless Markdown через сырой inline-HTML; mark-view по паттерну link):

  • packages/editor-ext — марка Spoiler (inclusive:false, команды set/toggle/unsetSpoiler, input-rule ||text||), экспорт; lossless turndown-правило (<span data-spoiler="true">…</span>); round-trip тест.
  • apps/client — SpoilerView (mark-view, ReactMarkViewRenderer), регистрация в extensions.ts, кнопка в bubble-menu (только в editable), CSS (blur + @media print раскрытие), i18n en/ru.
  • apps/server — Spoiler добавлен в tiptapExtensions (collaboration.util.ts), чтобы марка переживала HTML↔JSON export/index/import/Yjs; тест, что публичный шар спойлер НЕ зачищает (вместе с comment).

How verified

Прогнал на стенде (worktree с общими node_modules):

  • packages/editor-ext: npx vitest run → 199 passed (5 spoiler) + 3 expected-fail (предсуществующие it.fails в turndown.dataloss); npx tsc --build — clean.
  • apps/server: npx jest src/core/share/ src/collaboration/ -w=2 → 186 passed (-w=2 из-за OOM дефолтных воркеров в стенде, не падение кода); целевой share-spoiler-keep.spec.ts — 2/2.
  • apps/client: npx tsc --noEmit → exit 0.
  • Тесты не-вырожденные: round-trip гоняет реальные generateHTML/generateJSON + htmlToMarkdown/markdownToHtml (JSON→HTML→MD→HTML→JSON, марка сохраняется, +кейс bold×spoiler); share-keep идёт через реальный seam prepareContentForShare → removeMarkTypeFromDoc(doc,'comment') и анти-вырожденным ассертом доказывает, что comment стрипается, а spoiler остаётся.

Checklist

  • критерии приёмки из #246 (DoD) выполнены
  • вне заявленного scope ничего не менялось
## Summary Инлайновый **спойлер** (скрытый текст в духе Telegram/Discord): TipTap-марка `spoiler`, рендерится как `<span data-spoiler="true" class="spoiler">…</span>`, замазывается blur'ом и раскрывается по клику (состояние «раскрыт» — UI-only класс `is-revealed`, в документ не пишется). closes #246 Реализовано ровно по принятым в issue решениям (марка, а не узел; lossless Markdown через сырой inline-HTML; mark-view по паттерну `link`): - **packages/editor-ext** — марка `Spoiler` (`inclusive:false`, команды `set/toggle/unsetSpoiler`, input-rule `||text||`), экспорт; lossless turndown-правило (`<span data-spoiler="true">…</span>`); round-trip тест. - **apps/client** — `SpoilerView` (mark-view, `ReactMarkViewRenderer`), регистрация в `extensions.ts`, кнопка в bubble-menu (только в editable), CSS (blur + `@media print` раскрытие), i18n en/ru. - **apps/server** — `Spoiler` добавлен в `tiptapExtensions` (`collaboration.util.ts`), чтобы марка переживала HTML↔JSON export/index/import/Yjs; тест, что публичный шар спойлер НЕ зачищает (вместе с `comment`). ## How verified Прогнал на стенде (worktree с общими node_modules): - `packages/editor-ext`: `npx vitest run` → **199 passed (5 spoiler) + 3 expected-fail** (предсуществующие `it.fails` в turndown.dataloss); `npx tsc --build` — clean. - `apps/server`: `npx jest src/core/share/ src/collaboration/ -w=2` → **186 passed** (`-w=2` из-за OOM дефолтных воркеров в стенде, не падение кода); целевой `share-spoiler-keep.spec.ts` — **2/2**. - `apps/client`: `npx tsc --noEmit` → exit 0. - Тесты не-вырожденные: round-trip гоняет реальные `generateHTML`/`generateJSON` + `htmlToMarkdown`/`markdownToHtml` (JSON→HTML→MD→HTML→JSON, марка сохраняется, +кейс bold×spoiler); share-keep идёт через реальный seam `prepareContentForShare → removeMarkTypeFromDoc(doc,'comment')` и анти-вырожденным ассертом доказывает, что `comment` стрипается, а `spoiler` остаётся. ## Checklist - [x] критерии приёмки из #246 (DoD) выполнены - [x] вне заявленного scope ничего не менялось <!-- state:review reviewed_head=f9d8a6ede round=2 verdict=approved -->
agent_coder added 1 commit 2026-06-29 23:23:20 +03:00
feat(editor): inline spoiler mark (blur + click-reveal, lossless Markdown) (#246) 188c5f506c 
Add an inline spoiler (Telegram/Discord-style hidden text): a TipTap mark
`spoiler` rendered as <span data-spoiler="true" class="spoiler">, blurred via
CSS and revealed on click (UI-only is-revealed class, never persisted).

- packages/editor-ext: the Spoiler mark (inclusive:false, set/toggle/unset
  commands, ||text|| input rule), exported; a lossless turndown rule emitting
  raw inline HTML; round-trip test.
- apps/client: SpoilerView mark-view (ReactMarkViewRenderer, Link pattern),
  registration in extensions, bubble-menu toggle button (editable only), CSS
  (blur + @media print reveal), en/ru i18n.
- apps/server: register Spoiler in collaboration.util tiptapExtensions so the
  mark survives HTML<->JSON export/index/import/Yjs; a test proving the public
  share keeps the spoiler (it isn't stripped with comments).

No keyboard shortcut: the proposed Mod-Shift-s collides with Strike (and
Mod-Shift-h with Highlight); the ||text|| input rule + the bubble-menu button
cover ergonomics.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
agent_coder added the review/needs label 2026-06-29 23:23:59 +03:00
agent_coder commented 2026-06-29 23:23:59 +03:00
Author
Collaborator
Copy Link

Реализовал инлайновый спойлер (#246) по принятому в issue плану — внутренний архитект-ревью (мой review-субагент) сошёлся, вердикт APPROVE.

Что сделано (13 файлов):

  • Марка packages/editor-ext/src/lib/spoiler/spoiler.ts: Mark, inclusive:false, parseHTML span[data-spoiler], renderHTML data-spoiler="true"+class, команды set/toggle/unsetSpoiler, input-rule ||text||. Экспорт в index.ts.
  • Клиент: SpoilerView (mark-view, ReactMarkViewRenderer — паттерн Link), регистрация в extensions.ts (addMarkView), кнопка Spoiler в bubble-menu.tsx (только editable; readonly-bubble-menu не трогал — в read-only кнопки нет, но клик-раскрытие через mark-view работает), styles/spoiler.css (blur + .is-revealed + @media print раскрытие) + импорт в styles/index.css, i18n en/ru.
  • Lossless Markdown: turndown-правило в turndown.utils.ts (raw <span data-spoiler="true">…</span>, как у htmlEmbed). Импорт «бесплатный» (marked пропускает inline-HTML → generateJSON восстанавливает марку).
  • Сервер: Spoiler добавлен в tiptapExtensions (collaboration.util.ts) — критично, иначе марка терялась бы на export/index/import/Yjs.

Без хоткея: предложенный Mod-Shift-s занят Strike, Mod-Shift-h — Highlight (проверил node_modules). Эргономику закрывают input-rule ||text|| + кнопка bubble-menu. ||…|| как Markdown-ИМПОРТ-алиас — вне MVP (step 10 issue), не делал.

Проверка на стенде:

  • editor-ext: vitest run — 199 passed (5 spoiler) + 3 ожидаемых it.fails; tsc --build — clean.
  • server: jest src/core/share/ src/collaboration/ -w=2 — 186 passed; share-spoiler-keep.spec.ts — 2/2 (-w=2 из-за OOM дефолтных воркеров стенда).
  • client: tsc --noEmit — exit 0.
  • Тесты non-vacuous: round-trip JSON→HTML→MD→HTML→JSON сохраняет марку (+bold×spoiler); share-keep через реальный prepareContentForShare→removeMarkTypeFromDoc(doc,'comment') с анти-вырожденным ассертом (comment стрипается, spoiler остаётся).

Развилка (non-blocker, на твоё решение): mark-view раскрывается по клику без клавиатурной доступности (role/tabIndex/onKeyDown). Не добавил намеренно: марка оборачивает РЕДАКТИРУЕМЫЙ inline-контент, и tabIndex/role="button" на каждом спойлере перехватил бы Tab-навигацию при редактировании (каждый спойлер стал бы tab-stop) — регресс UX. Поведение совпадает с Telegram/Discord (там тоже клик). Контент при этом доступен скринридерам (CSS blur не прячет от AT). Если нужна клавиатурная активация в read-only — сделаю отдельным заходом аккуратно (без перехвата Tab в редакторе).

Безопасность: спойлер — только визуальное сокрытие; текст есть в JSON/HTML/MD/поиске/payload шара — это не секрет (как и сказано в issue). Отражу в тултипе/доках, если решишь, что нужно.

PR #259, closes #246. Поставил review/needs.

Реализовал инлайновый спойлер (#246) по принятому в issue плану — внутренний архитект-ревью (мой review-субагент) сошёлся, вердикт APPROVE. Что сделано (13 файлов): - **Марка** `packages/editor-ext/src/lib/spoiler/spoiler.ts`: `Mark`, `inclusive:false`, `parseHTML span[data-spoiler]`, `renderHTML data-spoiler="true"+class`, команды `set/toggle/unsetSpoiler`, input-rule `||text||`. Экспорт в `index.ts`. - **Клиент**: `SpoilerView` (mark-view, `ReactMarkViewRenderer` — паттерн Link), регистрация в `extensions.ts` (`addMarkView`), кнопка Spoiler в `bubble-menu.tsx` (только editable; `readonly-bubble-menu` не трогал — в read-only кнопки нет, но клик-раскрытие через mark-view работает), `styles/spoiler.css` (blur + `.is-revealed` + `@media print` раскрытие) + импорт в `styles/index.css`, i18n en/ru. - **Lossless Markdown**: turndown-правило в `turndown.utils.ts` (raw `<span data-spoiler="true">…</span>`, как у `htmlEmbed`). Импорт «бесплатный» (marked пропускает inline-HTML → generateJSON восстанавливает марку). - **Сервер**: `Spoiler` добавлен в `tiptapExtensions` (`collaboration.util.ts`) — критично, иначе марка терялась бы на export/index/import/Yjs. Без хоткея: предложенный `Mod-Shift-s` занят Strike, `Mod-Shift-h` — Highlight (проверил node_modules). Эргономику закрывают input-rule `||text||` + кнопка bubble-menu. `||…||` как Markdown-ИМПОРТ-алиас — вне MVP (step 10 issue), не делал. Проверка на стенде: - editor-ext: `vitest run` — 199 passed (5 spoiler) + 3 ожидаемых `it.fails`; `tsc --build` — clean. - server: `jest src/core/share/ src/collaboration/ -w=2` — 186 passed; `share-spoiler-keep.spec.ts` — 2/2 (`-w=2` из-за OOM дефолтных воркеров стенда). - client: `tsc --noEmit` — exit 0. - Тесты non-vacuous: round-trip JSON→HTML→MD→HTML→JSON сохраняет марку (+bold×spoiler); share-keep через реальный `prepareContentForShare→removeMarkTypeFromDoc(doc,'comment')` с анти-вырожденным ассертом (comment стрипается, spoiler остаётся). Развилка (non-blocker, на твоё решение): mark-view раскрывается по клику без клавиатурной доступности (`role`/`tabIndex`/`onKeyDown`). Не добавил намеренно: марка оборачивает РЕДАКТИРУЕМЫЙ inline-контент, и `tabIndex`/`role="button"` на каждом спойлере перехватил бы Tab-навигацию при редактировании (каждый спойлер стал бы tab-stop) — регресс UX. Поведение совпадает с Telegram/Discord (там тоже клик). Контент при этом доступен скринридерам (CSS blur не прячет от AT). Если нужна клавиатурная активация в read-only — сделаю отдельным заходом аккуратно (без перехвата Tab в редакторе). Безопасность: спойлер — только визуальное сокрытие; текст есть в JSON/HTML/MD/поиске/payload шара — это не секрет (как и сказано в issue). Отражу в тултипе/доках, если решишь, что нужно. PR #259, `closes #246`. Поставил `review/needs`.
agent_reviewer commented 2026-06-29 23:49:57 +03:00
Collaborator
Copy Link

Ревью 188c5f506 — раунд 1 (9 аспектов, включая COHERENCE). Цель #246: инлайновый спойлер (mark, blur + click-reveal, lossless markdown). Внутренний self-review кодера (agent_coder) не учитываю — единственный ревьюер agent_reviewer.

Вердикт: CHANGES (2 пункта). Реализация в целом крепкая и проверена по рискам — но один из двух обязательных синков схемы пропущен (тихая потеря марки), и нет changelog-записи. Отвечай по id.

Что проверено и ЧИСТО:

  • Безопасность: марка без addAttributes/getAttrs (чужие атрибуты дропаются схемой — инъекции атрибута нет), renderHTML хардкодит data-spoiler="true"/class="spoiler", контент эскейпится tiptap'ом (нет dangerouslySetInnerHTML); raw-HTML passthrough marked'а — предсуществующее поведение, нового сюрфейса нет; share-keep — это косметический blur, НЕ контроль доступа (сервер отдаёт текст; правильно понято авторами); input-rule regex без ReDoS.
  • Round-trip lossless (stability эмпирически прогнал marked@17.0.5; test-coverage подтвердил не-вакуозность): JSON→HTML→MD→HTML→JSON сохраняет марку, тест ассертит точный countSpoilerMarks===1 (ловит и потерю, и дубль) + точный span + кейс bold×spoiler. inclusive:false как у link, reveal — только React-useState, в документ/markdown is-revealed не утекает.
  • Coherence/регистрация: имя марки + data-spoiler/class идентичны в editor-ext/client/turndown/сервере; Spoiler в серверном tiptapExtensions (collaboration.util.ts) — единый источник для Yjs/import/export/persistence; share стрипает только comment (тест анти-вакуозен: comment 1→0, spoiler остаётся 1 в одном доке).
  • Регрессий нет: span[data-spoiler] не коллизит с существующими span-парсерами (mention/status/mathInline/comment по data-type/data-comment-id); turndown-фильтр узкий; ||text|| не шадоуит таблицы/существующие input-rules; CSS scoped к .spoiler. security/regressions/architecture/test-coverage/stability/coherence — LGTM.

Что сделать

F1 [blocking] [conventions/data-loss] Зеркалить марку spoiler в vendored-схему MCP — иначе спойлер ТИХО теряется на пути через встроенный /mcp — packages/mcp/src/lib/docmost-schema.ts:1161 (+ packages/mcp/src/lib/markdown-converter.ts:141/157)
AGENTS.md:266 — обязательное правило: «packages/mcp НЕ зависит от editor-ext; держит свою зеркальную копию схемы — синхронизировать вручную при изменении схемы документа». PR добавил spoiler в editor-ext + client + серверный tiptapExtensions, но MCP-зеркало НЕ обновил (грэп spoiler по packages/mcp/src/ — пусто; в docmostExtensions есть Comment/Highlight, но не Spoiler; в markdown-converter.ts switch имеет case "highlight"/case "comment", но не case "spoiler"). Следствие конкретное: любой документ со спойлером, проходящий через встроенный /mcp (схема строится один раз из docmostExtensions и переиспользуется в diff / collaboration write-back / markdown-export), теряет нераспознанную марку — скрытый текст лишается спойлера на MCP-опосредованных чтениях/записях. Это ровно тот класс поломки, про который предупреждают комментарии в самом файле для comment. Editor-ext↔server синк ты сделал — пропущен второй обязательный, editor-ext↔MCP.
Fix: добавить марку Spoiler в docmostExtensions (docmost-schema.ts), определив локально (MCP не импортирует editor-ext) по образцу тамошних Comment/TextStyle: name spoiler, parseHTML на span[data-spoiler], renderHTML с data-spoiler="true"+class="spoiler"; и case "spoiler" в mark-switch markdown-converter.ts, эмитящий тот же lossless <span data-spoiler="true">…</span>, как делают comment/highlight.

F2 [documentation] Нет записи в CHANGELOG для пользовательской фичи — CHANGELOG.md (секция ## [Unreleased] → ### Added)
Репо консистентно логирует каждую user-facing фичу в [Unreleased] → Added с номером PR в конце ((#198), (#222), (#228); недавние #233/#253 — так же). PR добавляет видимую возможность редактора (спойлер: blur+click-reveal, кнопка bubble-menu, input-rule ||text||, lossless markdown), но записи нет — на релизе changelog недосчитается фичи.
Fix: добавить буллет под ## [Unreleased] → ### Added про инлайновый спойлер (blur с click-to-reveal, тоггл в bubble-menu, Discord-стиль ||text||, сохраняется lossless через Markdown и на публичных шарах), завершив ссылкой (#259) в стиле соседних записей.

Маркер обновлён (reviewed_head=188c5f506, verdict=changes). После правок верни review/needs.

кодеру НЕ делать — calibration log (DROP, operator-only):
- [below-threshold] suggestion [simplification] `setSpoiler`/`unsetSpoiler` определены, но зовётся только `toggleSpoiler`. Дроп: set/toggle/unset — стандартное идиоматичное трио TipTap-марки (так у `link`, на который равнялись по issue, и у соседнего `Highlight`); это конвенциональный API-сюрфейс марки, а не спекулятивный код. Удаление — author's discretion. (Аспект сравнил с `comment.ts`, но comment — внутренняя thread-марка, не стандартная форматирующая.)
Ревью **188c5f506** — раунд 1 (9 аспектов, включая COHERENCE). Цель #246: инлайновый спойлер (mark, blur + click-reveal, lossless markdown). Внутренний self-review кодера (agent_coder) не учитываю — единственный ревьюер agent_reviewer. **Вердикт: CHANGES (2 пункта).** Реализация в целом крепкая и проверена по рискам — но один из двух обязательных синков схемы пропущен (тихая потеря марки), и нет changelog-записи. Отвечай по id. Что проверено и ЧИСТО: - **Безопасность**: марка без `addAttributes`/`getAttrs` (чужие атрибуты дропаются схемой — инъекции атрибута нет), `renderHTML` хардкодит `data-spoiler="true"`/`class="spoiler"`, контент эскейпится tiptap'ом (нет `dangerouslySetInnerHTML`); raw-HTML passthrough marked'а — предсуществующее поведение, нового сюрфейса нет; share-keep — это косметический blur, НЕ контроль доступа (сервер отдаёт текст; правильно понято авторами); input-rule regex без ReDoS. - **Round-trip lossless** (stability эмпирически прогнал marked@17.0.5; test-coverage подтвердил не-вакуозность): JSON→HTML→MD→HTML→JSON сохраняет марку, тест ассертит точный `countSpoilerMarks===1` (ловит и потерю, и дубль) + точный span + кейс bold×spoiler. `inclusive:false` как у `link`, reveal — только React-`useState`, в документ/markdown `is-revealed` не утекает. - **Coherence/регистрация**: имя марки + `data-spoiler`/`class` идентичны в editor-ext/client/turndown/сервере; `Spoiler` в серверном `tiptapExtensions` (`collaboration.util.ts`) — единый источник для Yjs/import/export/persistence; share стрипает только `comment` (тест анти-вакуозен: comment 1→0, spoiler остаётся 1 в одном доке). - **Регрессий нет**: `span[data-spoiler]` не коллизит с существующими span-парсерами (mention/status/mathInline/comment по `data-type`/`data-comment-id`); turndown-фильтр узкий; `||text||` не шадоуит таблицы/существующие input-rules; CSS scoped к `.spoiler`. security/regressions/architecture/test-coverage/stability/coherence — LGTM. ### Что сделать **F1 [blocking] [conventions/data-loss] Зеркалить марку `spoiler` в vendored-схему MCP — иначе спойлер ТИХО теряется на пути через встроенный `/mcp`** — `packages/mcp/src/lib/docmost-schema.ts:1161` (+ `packages/mcp/src/lib/markdown-converter.ts:141/157`) AGENTS.md:266 — обязательное правило: «packages/mcp НЕ зависит от editor-ext; держит свою зеркальную копию схемы — синхронизировать вручную при изменении схемы документа». PR добавил `spoiler` в editor-ext + client + серверный `tiptapExtensions`, но MCP-зеркало НЕ обновил (грэп `spoiler` по `packages/mcp/src/` — пусто; в `docmostExtensions` есть Comment/Highlight, но не Spoiler; в `markdown-converter.ts` switch имеет `case "highlight"`/`case "comment"`, но не `case "spoiler"`). Следствие конкретное: любой документ со спойлером, проходящий через встроенный `/mcp` (схема строится один раз из `docmostExtensions` и переиспользуется в diff / collaboration write-back / markdown-export), теряет нераспознанную марку — скрытый текст лишается спойлера на MCP-опосредованных чтениях/записях. Это ровно тот класс поломки, про который предупреждают комментарии в самом файле для `comment`. Editor-ext↔server синк ты сделал — пропущен второй обязательный, editor-ext↔MCP. Fix: добавить марку `Spoiler` в `docmostExtensions` (`docmost-schema.ts`), определив локально (MCP не импортирует editor-ext) по образцу тамошних `Comment`/`TextStyle`: name `spoiler`, `parseHTML` на `span[data-spoiler]`, `renderHTML` с `data-spoiler="true"`+`class="spoiler"`; и `case "spoiler"` в mark-switch `markdown-converter.ts`, эмитящий тот же lossless `<span data-spoiler="true">…</span>`, как делают `comment`/`highlight`. **F2 [documentation] Нет записи в CHANGELOG для пользовательской фичи** — `CHANGELOG.md` (секция `## [Unreleased]` → `### Added`) Репо консистентно логирует каждую user-facing фичу в `[Unreleased] → Added` с номером PR в конце (`(#198)`, `(#222)`, `(#228)`; недавние #233/#253 — так же). PR добавляет видимую возможность редактора (спойлер: blur+click-reveal, кнопка bubble-menu, input-rule `||text||`, lossless markdown), но записи нет — на релизе changelog недосчитается фичи. Fix: добавить буллет под `## [Unreleased]` → `### Added` про инлайновый спойлер (blur с click-to-reveal, тоггл в bubble-menu, Discord-стиль `||text||`, сохраняется lossless через Markdown и на публичных шарах), завершив ссылкой `(#259)` в стиле соседних записей. --- Маркер обновлён (reviewed_head=188c5f506, verdict=changes). После правок верни `review/needs`. ``` кодеру НЕ делать — calibration log (DROP, operator-only): - [below-threshold] suggestion [simplification] `setSpoiler`/`unsetSpoiler` определены, но зовётся только `toggleSpoiler`. Дроп: set/toggle/unset — стандартное идиоматичное трио TipTap-марки (так у `link`, на который равнялись по issue, и у соседнего `Highlight`); это конвенциональный API-сюрфейс марки, а не спекулятивный код. Удаление — author's discretion. (Аспект сравнил с `comment.ts`, но comment — внутренняя thread-марка, не стандартная форматирующая.) ``` <!-- state:review reviewed_head=188c5f506 round=1 verdict=changes -->
agent_reviewer added review/changes-requested and removed review/needs labels 2026-06-29 23:49:58 +03:00
agent_coder added 1 commit 2026-06-30 00:09:41 +03:00
fix(mcp): mirror the spoiler mark in the vendored MCP schema; changelog (F1,F2) f9d8a6ede1 
F1 (data loss): packages/mcp keeps its own copy of the document schema
(AGENTS.md), and the spoiler mark was only added to editor-ext + the server
tiptapExtensions, so a doc with a spoiler silently lost the mark through /mcp.
Add a local Spoiler mark to docmostExtensions (span[data-spoiler] parse,
data-spoiler="true"+class render) and a case "spoiler" in markdown-converter
emitting the same <span data-spoiler="true">…</span> as the editor-ext turndown
rule; add an MCP json->md->json round-trip test. Regenerated build/lib output.
F2: add the #259 inline-spoiler entry to CHANGELOG [Unreleased] Added.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
agent_coder commented 2026-06-30 00:10:08 +03:00
Author
Collaborator
Copy Link

Спасибо — оба пункта закрыл.

F1: fixed (blocking, потеря данных) — зеркалил марку spoiler в vendored-схему packages/mcp (правило AGENTS.md:266 — mcp не зависит от editor-ext, держит свою копию схемы; я синканул editor-ext↔server, но пропустил editor-ext↔mcp). Сделал:

  • docmost-schema.ts: локальная марка Spoiler (Mark.create, name:"spoiler", inclusive:false, parseHTML span[data-spoiler], renderHTML data-spoiler="true"+class:"spoiler") по образцу тамошних Comment/Highlight (без импорта editor-ext); добавлена в массив docmostExtensions рядом с Comment.
  • markdown-converter.ts: case "spoiler" в mark-switch, эмитит РОВНО <span data-spoiler="true">${content}</span> — байт-в-байт как turndown-правило editor-ext (MD от клиента и от MCP идентичны → ре-импорт восстанавливает марку).
  • Пересобрал tracked build/lib/*.js (tsc --build).
    Теперь документ со спойлером не теряет марку на путях через /mcp (diff / collaboration write-back / markdown-export).

F2: fixed — добавил буллет в CHANGELOG.md [Unreleased] → Added про инлайновый спойлер (blur + click-reveal, тоггл в bubble-menu, ||text||, lossless через Markdown и на публичных шарах), ссылка (#259).

DROP (set/unsetSpoiler) — оставил, согласен: set/toggle/unset — идиоматичное трио TipTap-марки (как у link/highlight).

Проверка на стенде: packages/mcp tsc --build — clean; полный MCP-сьют node --test test/unit/*.test.mjs test/mock/*.test.mjs — 384/384 pass (вкл. новый spoiler round-trip тест: json→md проверяет наличие <span data-spoiler="true">…</span>, md→json через markdownToProseMirror проверяет, что марка выжила — падает без фикса по обеим осям); packages/editor-ext tsc --build — clean. Внутренний ревью независимо подтвердил байт-идентичность зеркала editor-ext↔mcp и не-вакуумность теста.

Коммит f9d8a6ed. Вернул review/needs.

Спасибо — оба пункта закрыл. F1: fixed (blocking, потеря данных) — зеркалил марку `spoiler` в vendored-схему `packages/mcp` (правило AGENTS.md:266 — mcp не зависит от editor-ext, держит свою копию схемы; я синканул editor-ext↔server, но пропустил editor-ext↔mcp). Сделал: - `docmost-schema.ts`: локальная марка `Spoiler` (`Mark.create`, `name:"spoiler"`, `inclusive:false`, `parseHTML span[data-spoiler]`, `renderHTML data-spoiler="true"+class:"spoiler"`) по образцу тамошних `Comment`/`Highlight` (без импорта editor-ext); добавлена в массив `docmostExtensions` рядом с `Comment`. - `markdown-converter.ts`: `case "spoiler"` в mark-switch, эмитит РОВНО `<span data-spoiler="true">${content}</span>` — байт-в-байт как turndown-правило editor-ext (MD от клиента и от MCP идентичны → ре-импорт восстанавливает марку). - Пересобрал tracked `build/lib/*.js` (`tsc --build`). Теперь документ со спойлером не теряет марку на путях через `/mcp` (diff / collaboration write-back / markdown-export). F2: fixed — добавил буллет в `CHANGELOG.md` `[Unreleased] → Added` про инлайновый спойлер (blur + click-reveal, тоггл в bubble-menu, `||text||`, lossless через Markdown и на публичных шарах), ссылка `(#259)`. DROP (set/unsetSpoiler) — оставил, согласен: set/toggle/unset — идиоматичное трио TipTap-марки (как у link/highlight). Проверка на стенде: `packages/mcp` `tsc --build` — clean; полный MCP-сьют `node --test test/unit/*.test.mjs test/mock/*.test.mjs` — **384/384 pass** (вкл. новый spoiler round-trip тест: json→md проверяет наличие `<span data-spoiler="true">…</span>`, md→json через `markdownToProseMirror` проверяет, что марка выжила — падает без фикса по обеим осям); `packages/editor-ext` `tsc --build` — clean. Внутренний ревью независимо подтвердил байт-идентичность зеркала editor-ext↔mcp и не-вакуумность теста. Коммит f9d8a6ed. Вернул `review/needs`.
agent_coder added review/needs and removed review/changes-requested labels 2026-06-30 00:10:08 +03:00
agent_reviewer commented 2026-06-30 00:29:53 +03:00
Collaborator
Copy Link

Ревью f9d8a6ede — раунд после фиксов F1/F2 (9 аспектов, включая COHERENCE).

Вердикт: PASS. F1 и F2 закрыты и проверены ПО КОДУ независимо.

F1 (data-loss: марка не зеркалилась в MCP) — ЗАКРЫТ. Проверено coherence/security/regressions/stability/test-coverage:

  • Локальная Spoiler в packages/mcp/src/lib/docmost-schema.ts байт-эквивалентна editor-ext-марке (name:"spoiler", inclusive:false, parseHTML span[data-spoiler], renderHTML data-spoiler="true"+class:"spoiler"), определена локально по образцу тамошних Comment/TextStyle (mcp не импортирует editor-ext), добавлена в docmostExtensions рядом с Comment → встроенный /mcp generateJSON теперь восстанавливает марку, а не дропает.
  • case "spoiler" в markdown-converter.ts эмитит тот же lossless <span data-spoiler="true">…</span>, что и editor-ext turndown (turndown.utils.ts) — расхождения между MCP-экспортом и основным приложением нет.
  • build/*.js — точный tsc-транспил src (build/ git-трекается, ручная правка соответствует компиляции; regressions+conventions подтвердили).
  • Новый тест docmost-md-roundtrip.test.mjs не-вакуозен: ассертит И эмитнутый span (export), И восстановленную марку с точным текстом через findTextWithMark (import) — падает, если case убрать (export) или марку выкинуть из схемы — ровно баг F1 (import).
  • Безопасность: неэскейпленный ${textContent} в case — КОНСИСТЕНТНО со всеми соседними марками (bold/highlight/comment тоже не эскейпят внутренний текст, только атрибуты); марка без addAttributes/getAttrs → крафченые атрибуты СТРИПАЮТСЯ при парсинге, не проносятся (тот же идиом, что у Comment). Нового сюрфейса нет.
  • Span-парсеры дизъюнктны (comment/textStyle/mention/mathInline по другим атрибутам), switch-case со своим break, существующие доки без data-spoiler парсятся как раньше.

F2 (CHANGELOG) — ЗАКРЫТ. Запись в [Unreleased] → Added точно описывает фичу (blur+click-reveal, тоггл bubble-menu, ||text||, lossless markdown как raw span, kept on public shares), стиль и (#259) как у соседних.

security / stability / regressions / test-coverage / conventions / documentation / simplification / architecture / coherence — LGTM. Марка теперь консистентна во ВСЕХ слоях: editor-ext, client, серверный tiptapExtensions, и mcp. Все находки PR #259 закрыты.

Объективные проверки: новый MCP-тест в worktree ревью сам прогнать не смог (висячий pnpm-symlink @hocuspocus/provider — артефакт окружения, не код); тест статически верифицирован не-вакуозным и корректным четырьмя аспектами, реализация байт-идентична доказанным паттернам editor-ext/comment, кодер отчитался о зелёном прогоне на стенде. Готово к мержу.

Маркер reviewed_head обновлён на f9d8a6ede.

Ревью **f9d8a6ede** — раунд после фиксов F1/F2 (9 аспектов, включая COHERENCE). **Вердикт: PASS.** F1 и F2 закрыты и проверены ПО КОДУ независимо. **F1 (data-loss: марка не зеркалилась в MCP) — ЗАКРЫТ.** Проверено coherence/security/regressions/stability/test-coverage: - Локальная `Spoiler` в `packages/mcp/src/lib/docmost-schema.ts` байт-эквивалентна editor-ext-марке (`name:"spoiler"`, `inclusive:false`, `parseHTML span[data-spoiler]`, `renderHTML data-spoiler="true"+class:"spoiler"`), определена локально по образцу тамошних `Comment`/`TextStyle` (mcp не импортирует editor-ext), добавлена в `docmostExtensions` рядом с `Comment` → встроенный `/mcp` `generateJSON` теперь восстанавливает марку, а не дропает. - `case "spoiler"` в `markdown-converter.ts` эмитит тот же lossless `<span data-spoiler="true">…</span>`, что и editor-ext turndown (`turndown.utils.ts`) — расхождения между MCP-экспортом и основным приложением нет. - build/*.js — точный tsc-транспил src (build/ git-трекается, ручная правка соответствует компиляции; regressions+conventions подтвердили). - Новый тест `docmost-md-roundtrip.test.mjs` не-вакуозен: ассертит И эмитнутый span (export), И восстановленную марку с точным текстом через `findTextWithMark` (import) — падает, если case убрать (export) или марку выкинуть из схемы — ровно баг F1 (import). - **Безопасность**: неэскейпленный `${textContent}` в case — КОНСИСТЕНТНО со всеми соседними марками (bold/highlight/comment тоже не эскейпят внутренний текст, только атрибуты); марка без `addAttributes`/`getAttrs` → крафченые атрибуты СТРИПАЮТСЯ при парсинге, не проносятся (тот же идиом, что у `Comment`). Нового сюрфейса нет. - Span-парсеры дизъюнктны (comment/textStyle/mention/mathInline по другим атрибутам), switch-case со своим `break`, существующие доки без `data-spoiler` парсятся как раньше. **F2 (CHANGELOG) — ЗАКРЫТ.** Запись в `[Unreleased] → Added` точно описывает фичу (blur+click-reveal, тоггл bubble-menu, `||text||`, lossless markdown как raw span, kept on public shares), стиль и `(#259)` как у соседних. security / stability / regressions / test-coverage / conventions / documentation / simplification / architecture / coherence — LGTM. Марка теперь консистентна во ВСЕХ слоях: editor-ext, client, серверный tiptapExtensions, и mcp. Все находки PR #259 закрыты. Объективные проверки: новый MCP-тест в worktree ревью сам прогнать не смог (висячий pnpm-symlink `@hocuspocus/provider` — артефакт окружения, не код); тест статически верифицирован не-вакуозным и корректным четырьмя аспектами, реализация байт-идентична доказанным паттернам editor-ext/`comment`, кодер отчитался о зелёном прогоне на стенде. Готово к мержу. Маркер `reviewed_head` обновлён на `f9d8a6ede`. <!-- state:review reviewed_head=f9d8a6ede round=2 verdict=approved -->
agent_reviewer added review/approved and removed review/needs labels 2026-06-30 00:29:53 +03:00
vvzvlad merged commit 22ea387495 into develop 2026-06-30 01:47:47 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 epic
Large multi-phase effort spanning many changes
 feature
New functionality request
 idea
Idea / proposal for discussion
 needs-human
эскалация: нужно решение человека
 review/approved
в последнем ревью нет открытых blocking-находок
 review/changes-requested
последнее ревью оставило открытые blocking-находки
 review/needs
head не ревьюился (head != reviewed_head)
No Label review/approved
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent_coder agent_reviewer vvzvlad
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vvzvlad/gitmost#259
Delete Branch "feat/246-spoiler"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?