WORK-103
ID:WORK-103Status:done

Remove legacy Model class, decorators, and createSchema

Once all runes are migrated to createContentModelSchema, delete the legacy API surface from @refrakt-md/runes.

Priority:highComplexity:moderateMilestone:v1.0.0Source:SPEC-032
main View source

Criteria completion

Criteria completion: 7 of 7 (100%) checked; history from Apr 2 to Apr 20%25%50%75%100%Apr 2Apr 2

Relationships

Implements 1
Related 5
Branches 2
main current done
claude/file-naming-convention-LJdwR donechangeset-release/main done
History 4
  1. e900a90
    Created (done)by bjornolofandersson
  2. f262d7b
    Content editedby Claude
  3. d3dd023
    • ☑ All files listed above are deleted
    • ☑ `Model`, `@attribute`, `@group`, `@groupList`, `@id`, `createSchema` are no longer exported from `@refrakt-md/runes`
    • ☑ `NodeStream` and `RenderableNodeCursor` are removed if no longer used outside Model
    • ☑ No remaining imports of deleted modules anywhere in the monorepo (`grep` confirms zero hits)
    • ☑ `npm run build` succeeds for the full monorepo
    • ☑ `npm test` passes
    • ☑ CSS coverage tests pass
    by Claude
  4. 5c72bef
    Created (ready)by Claude
    Add work items WORK-099 through WORK-104 for legacy Model removal

Files to delete

  • packages/runes/src/lib/model.ts — Model base class
  • packages/runes/src/lib/annotations/attribute.ts — @attribute decorator
  • packages/runes/src/lib/annotations/group.ts — @group / @groupList decorators
  • packages/runes/src/lib/annotations/id.ts — @id decorator
  • packages/runes/src/lib/annotations/ — entire directory if empty after above

Exports to remove

From packages/runes/src/lib/index.ts and packages/runes/src/index.ts:

  • Model
  • attribute
  • group, groupList
  • id
  • createSchema
  • NodeStream, RenderableNodeCursor (if only used by Model)

Acceptance Criteria

  • All files listed above are deleted
  • Model, @attribute, @group, @groupList, @id, createSchema are no longer exported from @refrakt-md/runes
  • NodeStream and RenderableNodeCursor are removed if no longer used outside Model
  • No remaining imports of deleted modules anywhere in the monorepo (grep confirms zero hits)
  • npm run build succeeds for the full monorepo
  • npm test passes
  • CSS coverage tests pass

Approach

  1. Remove exports from packages/runes/src/lib/index.ts and packages/runes/src/index.ts
  2. Delete the files
  3. Search the entire monorepo for any remaining imports of the deleted symbols
  4. Full build and test pass

Dependencies

  • WORK-099 (simple core runes migrated)
  • WORK-100 (simple community runes migrated)
  • WORK-101 (moderate runes migrated)
  • WORK-102 (complex runes migrated)

References

  • SPEC-032 (parent spec)

Resolution

Completed: 2026-04-02

Branch: claude/implement-spec-032-2KBbw

What was done

  • Deleted model.ts, entire annotations/ directory (6 files), removed NodeStream from node.ts
  • Removed createSchema, Model, attribute, group, groupList, id, NodeStream exports from lib/index.ts and index.ts
  • Converted SplitLayoutModel from Model subclass to plain splitLayoutAttributes record
  • Updated createContentModelSchema base option to accept plain Record<string, SchemaAttribute> instead of Newable<Model>
  • Updated error message in packages.ts
  • Removed Group interface from interfaces.ts
  • 696 lines deleted, 58 added

Notes

  • RenderableNodeCursor is still used extensively by all runes — kept as-is
  • isFilterMatching in node.ts is still used by util.ts — kept
  • TypedNode and TransformFunction in interfaces.ts are unused but harmless public types — left in place
  • All 7 rune files using base: SplitLayoutModel work unchanged since SplitLayoutModel is now aliased to the plain splitLayoutAttributes object