Skip to content

feat: add PDF and DOCX support to read_file using get-md#651

Open
akramcodez wants to merge 1 commit into
Nano-Collective:mainfrom
akramcodez:feature/read-file-pdf-docx-support
Open

feat: add PDF and DOCX support to read_file using get-md#651
akramcodez wants to merge 1 commit into
Nano-Collective:mainfrom
akramcodez:feature/read-file-pdf-docx-support

Conversation

@akramcodez

Copy link
Copy Markdown
Collaborator

Description

This PR upgrades @nanocollective/get-md to the latest version and extends the read_file tool to support PDF and DOCX files by automatically converting them to Markdown. It also adds comprehensive test coverage to verify document conversion, backward compatibility for text files, and graceful handling of conversion failures.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Testing

Automated Tests

  • New features include passing tests in .spec.ts/tsx files
  • All existing tests pass (pnpm test:all completes successfully)
  • Tests cover both success and error scenarios

Manual Testing

  • Tested with Ollama
  • Tested with OpenRouter
  • Tested with OpenAI-compatible API
  • Tested MCP integration (if applicable)

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated (if needed)
  • No breaking changes (or clearly documented)
  • Appropriate logging added using structured logging (see [CONTRIBUTING.md](../CONTRIBUTING.md#logging))

Copilot AI review requested due to automatic review settings July 10, 2026 10:51
@akramcodez akramcodez requested a review from Avtrkrb as a code owner July 10, 2026 10:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

2 participants