dsh-book-reader 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -32,6 +32,18 @@ adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
   fixed zoom is replayed, so the cover no longer flashes for a few hundred
   milliseconds while the zoom settles. The scroll is re-applied after the zoom
   commit so the page stays aligned.
+- `pdf_read_pages` no longer disappears from the model's tool list: its output
+  schema used a property-level `required: false`, which DSH's `defineTool`
+  rejects at registration time. The exception only reached the Host log, so the
+  plugin looked enabled with two tools instead of three.
+
+## [0.3.1] — 2026-10-22
+
+### Fixed
+
+- Text-only `pdf_read_pages` calls (`include_image:false`) no longer crash while
+  rendering their output, and they no longer require an image-capable model
+  route or the attachment store; image reads still require both.
 
 ## [0.2.0] — 2026-10-20
 
@@ -101,6 +113,7 @@ First public release.
 - `pdfjs-dist` is pinned to `6.3.289` to match the host process.
 - MIT licensed.
 
+[0.3.1]: https://github.com/Strepsiades2/dsh-book-reader/releases/tag/v0.3.1
 [0.3.0]: https://github.com/Strepsiades2/dsh-book-reader/releases/tag/v0.3.0
 [0.2.0]: https://github.com/Strepsiades2/dsh-book-reader/releases/tag/v0.2.0
 [0.1.1]: https://github.com/Strepsiades2/dsh-book-reader/releases/tag/v0.1.1

package/lib/index.js

@@ -1246,7 +1246,11 @@ export function renderPageReadContent(value) {
       type: 'text',
       text: `<page pdf="${page.pdf}" printed="${page.printed}">\n${body}${truncation}\n</page>`,
     })
-    blocks.push({ type: 'image', attachment: imageRefFromValue(page.image) })
+    // Text-only reads (include_image:false) never carry an image; emitting an
+    // image block for them crashed output.render on page.image.attachmentId.
+    if (page.image !== undefined) {
+      blocks.push({ type: 'image', attachment: imageRefFromValue(page.image) })
+    }
   }
   return blocks
 }
@@ -2103,11 +2107,17 @@ export function createReadPagesTool({ defineTool: makeTool, ctx, config, core, i
       const path = resolveToolPath(args.path, exec)
         assertInsideWorkspace(path, exec, config)
       const targetStat = await assertPdfFile(path, { statFile, maxFileBytes: config.maxFileBytes })
-      const attachments = ctx.get('attachments')
-      if (attachments === undefined) {
-        throw codedError('cannot read PDF pages as images: no attachment service is mounted', 'PDF_READER_NO_ATTACHMENTS')
+      // Images need the attachment store and an image-capable route; a
+      // text-only read needs neither and must keep working without them.
+      const includeImage = args.include_image !== false
+      let attachments
+      if (includeImage) {
+        attachments = ctx.get('attachments')
+        if (attachments === undefined) {
+          throw codedError('cannot read PDF pages as images: no attachment service is mounted', 'PDF_READER_NO_ATTACHMENTS')
+        }
+        await assertImageCapableRoute(ctx, exec, path)
       }
-      await assertImageCapableRoute(ctx, exec, path)
       const info = (await infoCache.load(path, targetStat)).info
       const resolved = resolveRequestedPages({
         selection: args.pages,
@@ -2115,7 +2125,6 @@ export function createReadPagesTool({ defineTool: makeTool, ctx, config, core, i
         info,
         maxPages: config.maxPagesPerCall,
       })
-      const includeImage = args.include_image !== false
       if (!includeImage) {
         const book = titleFromPath(path)
         const includeText = args.include_text !== false && config.maxTextCharsPerPage > 0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-book-reader",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Read books and papers in DSH: a reading toolbar on the official PDF preview that remembers your page and zoom, plus PDF tools for the model (page map, full-text search, page text and images).",
   "keywords": [
     "dsh",