Feature: accept any buffer instance when processing file data

[ArEnSc]

refactor: optimize file handling to avoid redundant buffer conversions

• Change _get_file_details() to return Buffer instead of bytes
• Add special handling for memory view inputs to avoid unnecessary conversion
• Support all buffer protocol objects (bytes, bytearray, memoryview, array.array, etc.)
• Update type annotations throughout to use Buffer type from typing_extensions
• Improve error messages to mention "buffer" instead of just "bytes"

This change reduces memory overhead when working with buffer protocol objects
by preserving memoryview objects instead of converting them to bytes unnecessarily.

Closes #46

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[ArEnSc]

@ncoghlan I cannot find the location of which I confirm the CLA. I think the site maybe broken ?

[baonudesifeizhai]

@ncoghlan I cannot find the location of which I confirm the CLA. I think the site maybe broken ?

[ArEnSc]

I have read the CLA Document and I hereby sign the CLA

[ArEnSc]

@ncoghlan I cannot find the location of which I confirm the CLA. I think the site maybe broken ?

thanks bro

[ncoghlan]

This is a nice improvement, thanks!

I'd like to arrange the updated checks in _get_file_details a bit differently, but the details are beyond what GitHub will tolerate as a multi-line suggestion, so I'll change those locally and push them up to the branch.

[ArEnSc]

This is a nice improvement, thanks!

I'd like to arrange the updated checks in _get_file_details a bit differently, but the details are beyond what GitHub will tolerate as a multi-line suggestion, so I'll change those locally and push them up to the branch.

Sounds good to me!

[ArEnSc]

@ncoghlan I don't think I can merge it requires me to have write permissions I believe, it just shows close with comment on my side.

[ncoghlan]

@ncoghlan I don't think I can merge it requires me to have write permissions I believe, it just shows close with comment on my side.

Aye, there was a gap between approval and merging as I was also running the full test suite against my local LM Studio instance (the automated CI's coverage is unfortunately still limited for now).

Local tests were all green, though, so thanks for the enhancement!

[ncoghlan]

Follow up PR to add test cases for the newly accepted file data input types: #152

[ncoghlan]

This will be released as part of lmstudio-python 1.5.0.