refactor(editor): remove in-editor heading drag handle

Section reorder remains available from the TOC via moveSection.
AGENTS.md notes TOC-only reorder and expands migration safety and CLI
guidance.

Made-with: Cursor

Author: HMarzban

AGENTS.md

@@ -25,8 +25,8 @@
 
 ## Learned Workspace Facts
 
-- docs.plus / docsy — Bun monorepo (packages/\*); main app @docs.plus/webapp (Next.js Pages Router), backend @docs.plus/hocuspocus; editor code under packages/webapp/src/components/TipTap/ (extensions, nodes, plugins); `src/lib/` removed — all shared utilities in `src/utils/`, feature-local helpers stay colocated; server-side `TiptapTransformer.toYdoc` / nested→flat migration must use an extension set that covers **every** node/mark in stored docs (e.g. **TaskList** / **TaskItem** from `@tiptap/extension-list` aligned with the webapp), not StarterKit alone — missing types fail encode, not flatten logic
-- Editor uses flat heading schema (`heading block*`) with decoration-based sections; `attrs['toc-id']` renders as `data-toc-id`; shared heading utilities (computeSection, moveSection, canMapDecorations, transactionAffectsNodeType, matchSections) in TipTap/extensions/shared/
+- docs.plus / docsy — Bun monorepo (packages/\*); main app @docs.plus/webapp (Next.js Pages Router), backend @docs.plus/hocuspocus; editor code under packages/webapp/src/components/TipTap/ (extensions, nodes, plugins); `src/lib/` removed — all shared utilities in `src/utils/`, feature-local helpers stay colocated; server-side `TiptapTransformer.toYdoc` / nested→flat migration must use an extension set that covers **every** node/mark in stored docs (e.g. **TaskList** / **TaskItem** from `@tiptap/extension-list` aligned with the webapp), not StarterKit alone — missing types fail encode, not flatten logic; **batch migrations must fail closed** (do not overwrite stored Yjs for a doc when transform/encode fails — keep prior bytes and surface the doc id); **run migration CLI** via `bun run migrate:nested-to-flat` from `packages/hocuspocus.server` after root `bun install` — invoking the script path alone from an arbitrary cwd can break Bun resolution of `yjs` for `@hocuspocus/transformer`
+- Editor uses flat heading schema (`heading block*`) with decoration-based sections; `attrs['toc-id']` renders as `data-toc-id`; shared heading utilities (computeSection, moveSection, canMapDecorations, transactionAffectsNodeType, matchSections) in TipTap/extensions/shared/; **section reorder is TOC-only** (`useTocDrag` / `moveHeading` + `moveSection`) — there is no in-editor heading drag handle extension
 - **HeadingScale (mandatory spec):** `extensions/heading-scale/heading-scale.ts` — **dynamic** heading **font size by rank within a section**, **not** fixed per HTML level or a Google-style ladder. **Each H1 starts a new section**; within a section, **distinct** heading levels are sorted and sizes are **interpolated evenly between 20pt (max) and 12pt (min)**; same level twice in one section → **same visual size**; **one distinct level in a section → 20pt**. Title (first top-level H1) is included as part of section 1. **Decorations only** (`--hd-size`, `--hd-rank`, `--hd-total`); never write sizes into the document. Plugin state `{ fingerprint, decorations }` with fingerprint = top-level heading **levels in order** (e.g. `1,2,4,1,3`); **full rebuild when fingerprint changes** or `y-sync$` meta; else **map** the decoration set. **Do not** replace this with fixed per-level pt maps — that breaks the agreed behavior.
 - Editor perf: jank is React/Zustand re-renders, not ProseMirror; never put UI flags in `useEditor` deps; `shouldRerenderOnTransaction: false` on collab; decoration plugins should avoid full rebuilds on every keystroke — use `transactionAffectsNodeType(tr, 'heading')` **or** a cheaper structural check (HeadingScale uses **heading-level fingerprint**, not only `transactionAffectsNodeType`); placeholder uses `@docs.plus/extension-placeholder` (O(1) via state.init/apply) — do NOT replace with TipTap's built-in (O(N) doc.descendants)
 - Zustand: monolithic 7-slice store; all `useStore` calls must use leaf selectors — never `(state) => state` or `(state) => state.settings`

packages/webapp/src/components/TipTap/TipTap.tsx

@@ -44,7 +44,6 @@ import ShortUniqueId from 'short-unique-id'
 import { IndexeddbPersistence } from 'y-indexeddb'
 import * as Y from 'yjs'
 
-import { HeadingDrag } from './extensions/heading-drag'
 import { HeadingFilter } from './extensions/heading-filter'
 import { HeadingFold, headingFoldPluginKey } from './extensions/heading-fold'
 import { HeadingScale } from './extensions/heading-scale'
@@ -157,7 +156,6 @@ const Editor = ({
     HeadingFold.configure({
       documentId: docName
     }),
-    HeadingDrag,
     HeadingFilter.configure({
       foldAdapter: {
         getFoldedIds: (state) => {