Repository navigation

fix: return tool errors with vault-relative paths and the server's own messages - #666

Merged

aliasunder merged 33 commits into

mainfrom

fix/tool-errors-vault-relative

Oct 11, 2026

Merged

fix: return tool errors with vault-relative paths and the server's own messages#666
aliasunder merged 33 commits into
mainfrom
fix/tool-errors-vault-relative

Commits on Oct 11, 2026
- fix: return tool errors without the server's paths or library text
  
  aliasunder
  and
  claude
  committed
  fc1f498
- test: regenerate the tool-surface snapshots for the description changes
  
  aliasunder
  and
  claude
  committed
  db685a3
- fix(review): make backtick-quoted paths vault-relative in tool errors
  
  aliasunder
  and
  claude
  committed
  f58f9b5
- fix(triage): correct the fenced-block advice and list the read tool's filesystem errors
  
  aliasunder
  and
  claude
  committed
  4a78fc9
- style: clarify the vault-relative error comments and align the docs' terms
  
  aliasunder
  and
  claude
  committed
  d07fb1f
- fix(triage): one decode-error wording and a plainer fenced-block sentence
  
  aliasunder
  and
  claude
  committed
  896affb
- style: readability fixes from the code-quality review
  
  aliasunder
  and
  claude
  committed
  0b39e6f
- test: pin the path rewrite's boundary checks and the fit error's wording
  
  aliasunder
  and
  claude
  committed
  8fa1053
- fix: keep sharp's reason in the PDF page failure log and mend a garbled comment
  
  aliasunder
  and
  claude
  committed
  df483f5
- fix(triage): widen the fenced-block warning and reject impossible review dates
  
  aliasunder
  and
  claude
  committed
  3238292
- test: exact assertions in the vault-orientation prompt tests
  
  aliasunder
  and
  claude
  committed
  bb94476
- test: exact assertions in the daily-review prompt tests
  
  aliasunder
  and
  claude
  committed
  cb828f8
- test: merge two daily-review test pairs that assert the same value
  
  aliasunder
  and
  claude
  committed
  e5e7481
- docs: say client errors name paths vault-relative, never the server's own
  
  aliasunder
  and
  claude
  committed
  a180f7f
- docs: tell wrappers to pass a Node error as cause, not copy its text
  
  aliasunder
  and
  claude
  committed
  c7dfe7f
- fix: label a link to a daily note not created yet instead of calling it broken
  
  aliasunder
  and
  claude
  committed
  40e6e2b
- fix: return the server's own message when the PDF reader fails to start
  
  aliasunder
  and
  claude
  committed
  116d55e
- docs: say which PDF reader start failures a retry recovers
  
  aliasunder
  and
  claude
  committed
  ce20dcf
- docs: scope the vault-relative rule to Node errors
  
  aliasunder
  and
  claude
  committed
  6cdcc1c
- refactor: log prompt failures in the catch instead of inside a describe helper
  
  aliasunder
  and
  claude
  committed
  f909813
- fix: one Errors bullet per remedy in vault_read_file and a remedy for failed page renders
  
  aliasunder
  and
  claude
  committed
  a495e4e
- fix: give too-long paths and impossible daily-note dates their own remedies
  
  aliasunder
  and
  claude
  committed
  d9e474a
- fix: name the full path in vault_read_file's ENAMETOOLONG entry and how much shorter a rename must be
  
  aliasunder
  and
  claude
  committed
  b09803e
- fix: pin the prompt_error log in daily-review and memory-review
  
  aliasunder
  and
  claude
  committed
  96bed4e
- test: pin PDF page rendering failed over HTTP with a PDF that has no pages
  
  aliasunder
  and
  claude
  committed
  1e69329
- style: give the modified-notes section its own helper and state the why in two comments
  
  aliasunder
  and
  claude
  committed
  2861893
- test: pin daily-note errors exactly and name the too-long-to-save case plainly
  
  aliasunder
  and
  claude
  committed
  7f81ba4
- refactor: share the not-a-calendar-date refusal and reword the render-failure remedy
  
  aliasunder
  and
  claude
  committed
  d383e60
- test: raise the per-tool size allowance to 4,217
  
  aliasunder
  and
  claude
  committed
  324ac08
- refactor: name the PDF reads for what they return, not for their error handling
  
  aliasunder
  and
  claude
  committed
  6ef5910
- docs: name the one date that reaches the daily-review catch as client input
  
  aliasunder
  and
  claude
  committed
  e0e9db4
- test: pin the cause in the unreadable-PDF logs of the watcher and the rebuild
  
  aliasunder
  and
  claude
  committed
  c850f26
- test: raise the tool-surface allowance to 4,226 after rebasing onto the date-message change
  claude
  committed
  958ae93

Commits

Commits on Oct 11, 2026