[shovon]

Cleans up the inside of @pbnotes/pages along the lines of the bunnyhug skill. The public interface is unchanged: pages.ts exports every name it did before.

Structure

• pages.ts is wiring only (514 → 155 lines): Pages, HANDLES, pagesFold, PagesView, getPages (the one read that needs both blocks and properties), and the re-exports.
• The block slice is split in three:
  • blocks/blocks.ts: schema, tree operations, the fold.
  • blocks/queries.ts: the reads. linking moves here from reading, which breaks the blocks ↔ reading import cycle.
  • blocks/commands.ts: the event builders and the commands.
• Shared pieces get their own modules, so properties no longer imports blocks:
  • events/: NAMESPACE, Named, payload helpers.
  • titles/: spell, titleOf.
  • journal/: DATE_PATTERN, today.
• The block fold works on any state with blocks and titles, so blocks.ts no longer knows properties exist. PagesView is the commands' BlocksView plus properties.
• Less repetition: the builders share build(), and commands on existing blocks share onExisting().

Behaviour change

The block fold now checks each event before applying it, like the property fold already did. A malformed block event, which only another program writing to the folder could produce, is skipped whole.

Event	Before	Now
block.created / block.edited with payload: null	Fold throws, so the folder can't open (#62)	Skipped
block.created with non-string text	Shown, then every query throws	Skipped
block.edited with non-string text	Text replaced with a non-string	Skipped
block.created with no text, no id, or a numeric id	Shown, but can't be edited or deleted	Skipped (could hide a block that showed before)
block.created with no page	Filed under a page called "undefined"	Same
block.created with non-string after	Added at the end of the page	Same

Open question: the "no text / numeric id" blocks could be coerced (String(id), text ?? "") instead of skipped, so they stay visible.

Testing

• New test in blocks.test.ts pins the malformed-event behaviour. It fails on main and passes here.
• pages tests: 58 pass. CLI tests pass. The 65 app tests that import @pbnotes/pages pass.
• tsc --noEmit passes for the app, and for pages on its own under --strict.

🤖 Generated with Claude Code