Skip to content

feat(deps): add markdown library as explicit dependency#86

Merged
xiami762 merged 5 commits intomainfrom
feat/add-markdown-dep
Apr 10, 2026
Merged

feat(deps): add markdown library as explicit dependency#86
xiami762 merged 5 commits intomainfrom
feat/add-markdown-dep

Conversation

@duguwanglong
Copy link
Copy Markdown
Contributor

Add markdown>=3.10.2 to project dependencies to make the implicit dependency explicit and ensure consistent availability across environments.

Add `markdown>=3.10.2` to project dependencies to make the implicit
dependency explicit and ensure consistent availability across environments.

Made-with: Cursor
@duguwanglong duguwanglong requested a review from xiami762 April 10, 2026 09:01
Revert the auto-incremented revision field back to 2 to keep the lock
file consistent with the base branch state.

Made-with: Cursor
Add `reportlab>=4.4.10` (with pillow as transitive dependency) to support
PDF generation capabilities.

Made-with: Cursor
sys.exit() in plugin top-level code raises SystemExit (BaseException),
which was not caught by the previous `except Exception` guard, causing
the entire uvicorn server to crash. Broaden the catch to BaseException
so a misbehaving plugin is skipped with an error log instead of killing
the process. Also log the exception type for easier diagnosis.

Made-with: Cursor
reportlab was added as an explicit dependency but is only needed by the
markdown_to_pdf user plugin, not by the core package. Removing it keeps
the core dependency set minimal; plugin authors should install their own
requirements.

Made-with: Cursor
@xiami762 xiami762 merged commit 10ed1bb into main Apr 10, 2026
2 checks passed
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