dsh-book-reader 0.3.0 → 0.3.2

package/CHANGELOG.md

@@ -32,6 +32,28 @@ 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.2] — 2026-10-23
+
+### Fixed
+
+- `pdf_read_pages` is now registered synchronously with the other two tools
+  instead of waiting for the `attachments` injection. The later registration
+  made DSH announce `已添加工具：pdf_read_pages` at the start of every chat;
+  image calls still resolve the attachment store lazily, and text-only calls
+  never need it.
+
+## [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 +123,8 @@ First public release.
 - `pdfjs-dist` is pinned to `6.3.289` to match the host process.
 - MIT licensed.
 
+[0.3.2]: https://github.com/Strepsiades2/dsh-book-reader/releases/tag/v0.3.2
+[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
@@ -2313,18 +2322,21 @@ export function apply(ctx, rawConfig) {
     ctx.tools.register(tool)
   }
 
-  ctx.inject(['attachments'], (attachmentCtx) => {
-    attachmentCtx.tools.register(
-      createReadPagesTool({
-        defineTool,
-        ctx: attachmentCtx,
-        config,
-        core,
-        infoCache,
-        statFile: stat,
-      }),
-    )
-  })
+  // Register synchronously so the session's first tool list already holds all
+  // three tools. Waiting for the attachments injection used to register this
+  // one after the session started, which made every chat announce
+  // "tool added: pdf_read_pages". Image calls resolve `attachments` lazily at
+  // execution time; text-only reads do not need it at all.
+  ctx.tools.register(
+    createReadPagesTool({
+      defineTool,
+      ctx,
+      config,
+      core,
+      infoCache,
+      statFile: stat,
+    }),
+  )
 
   if (config.rememberView) {
     ctx.inject(['storageDomain'], (storageCtx) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-book-reader",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "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",