Skip to content

[Reader] Expose Article Language in CLI - RW-45999#27

Closed
rasulkireev wants to merge 4 commits into
masterfrom
rasul/rw-45999-cli-language
Closed

[Reader] Expose Article Language in CLI - RW-45999#27
rasulkireev wants to merge 4 commits into
masterfrom
rasul/rw-45999-cli-language

Conversation

@rasulkireev

@rasulkireev rasulkireev commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

The CLI can now pass Reader article language through document creation and bulk metadata edits, matching the Rekindled support shipped in readwiseio/rekindled#10398.

This refreshes cached tool schemas, adds a small Reader-specific schema shim while production listTools catches up, and surfaces the server's maxLength metadata in generated command help. The README examples now show language on create and bulk-edit paths.

Validation

  • npm test
  • npm run build
  • node dist/index.js reader-create-document --help shows --language with max length: 30
  • raw production listTools smoke: create-document and bulk-edit do not yet advertise language, so the local schema shim is still needed
  • git diff --check

Post-Deploy Monitoring & Validation

  • Validation window: first publish after merge, owner: release owner.
  • Healthy signal: readwise reader-create-document --help includes --language, and readwise reader-bulk-edit-document-metadata accepts documents containing language.
  • Watch for npm publish/build failures in the Publish npm package workflow and user reports of unknown --language options after upgrading.
  • Rollback trigger: if the Reader language schema shim causes command registration or tool-call argument regressions, revert this PR and publish a patch release.

Related

Related to RW-45999.


Compound Engineering
GPT-5

@linear-code

linear-code Bot commented Jul 9, 2026

Copy link
Copy Markdown

RW-45999

@rasulkireev rasulkireev closed this Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant