Skip to content

feat: add text processing with media - #1

Merged
gabewillen merged 2 commits into
mainfrom
feat/process-text-media
Aug 7, 2026
Merged

feat: add text processing with media#1
gabewillen merged 2 commits into
mainfrom
feat/process-text-media

Conversation

@gabewillen

Copy link
Copy Markdown
Contributor

Adds Python bindings for Cortext text processing with optional binary media and MIME types.

Tests: .venv/bin/pytest -q (9 passed).

Copilot AI lite review requested due to automatic review settings August 7, 2026 00:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Python API surface for processing text with optional binary media attachments (plus MIME type), wiring up the corresponding native entrypoints and documenting the new usage.

Changes:

  • Add Cortext.process_text_with_media_json() / Cortext.process_text_with_media() methods that marshal optional media into _NativeMedia and call the native “with options” function.
  • Extend native library configuration to detect and type the optional text+media symbols.
  • Add a unit test to verify that binary bytes + MIME type are forwarded into the native media struct, and document the feature in the README.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
cortext/__init__.py Adds native symbol detection/ctypes signatures and the new process_text_with_media* methods.
tests/test_cortext.py Adds a unit test that validates media bytes + MIME type marshalling into _NativeMedia.
README.md Documents how to call process_text_with_media with Media(...) or raw bytes + media_mimetype.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cortext/__init__.py Outdated
@gabewillen
gabewillen merged commit 5ecda86 into main Aug 7, 2026
3 checks passed
@gabewillen
gabewillen deleted the feat/process-text-media branch August 7, 2026 04:12
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