chore(synapse): unify import error messages for missing AI dependencies#84
Conversation
|
This PR standardizes ImportError messages for missing AI dependencies to improve clarity and debugging consistency. Happy to adjust wording if needed. |
|
Hi @IshanikaAurelia1106 ! Thanks for your PR and welcome to "Logseq Matryca Parser" I'm studying your contribution and I'll keep you updated. Marco |
|
Hi @IshanikaAurelia1106 — thank you for opening this PR and welcome to the project! Unifying the SYNAPSE CI is currently failing (3 tests in Before this is merge-ready, could you please:
Branch hygiene: one PR per concern is perfect here — please keep this scoped to SYNAPSE import errors only. Happy to re-review once CI is green. Thanks again for contributing! |
IshanikaAurelia1106
left a comment
There was a problem hiding this comment.
Updated the tests to use the unified AI dependency message and verified all tests pass locally.
This PR unifies the AI dependency ImportError messages used by the SYNAPSE export helpers.
Previously, different export functions raised inconsistent ImportError messages and referenced different optional packages. This change replaces them with a single shared module-level constant that provides a consistent English message directing users to install the required AI dependencies using:
uv sync --extra ai
The related tests have also been updated to use the shared constant, and a CHANGELOG entry has been added.
Type of change
Checklist