Conversation
Contributor
There was a problem hiding this comment.
Sorry @Zhalslar, your pull request is larger than the review limit of 150000 diff characters
Owner
Author
|
@copilot review |
Contributor
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds a comprehensive link parser plugin for AstrBot that supports parsing and downloading content from multiple platforms including Bilibili, Douyin, Weibo, Xiaohongshu, Kuaishou, Acfun, YouTube, TikTok, and Twitter. The plugin features media card rendering, automated cache cleaning, and platform-specific authentication.
Key changes:
- Complete plugin implementation with parser framework and platform-specific parsers
- PIL-based media card rendering system with emoji support
- Download management with size/duration limits and progress tracking
Reviewed changes
Copilot reviewed 34 out of 47 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| metadata.yaml | Plugin metadata configuration |
| main.py | Plugin entry point and core orchestration |
| core/utils.py | Utility functions for file operations and text processing |
| core/render.py | Media card rendering engine using PIL |
| core/parsers/*.py | Platform-specific parsers for each supported service |
| core/download.py | Download manager with streaming and yt-dlp support |
| core/constants.py | Shared constants and platform enumerations |
| core/clean.py | Automated cache cleanup scheduler |
| core/exception.py | Custom exception hierarchy |
| _conf_schema.json | Configuration schema definition |
| README.md | Documentation and usage guide |
| LICENSE | MIT license with copyright year mismatch |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
[WIP] Fix recent commit issues in code
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Zhalslar
pushed a commit
that referenced
this pull request
Mar 8, 2026
…convert-path-to-uri fix: use file:// URIs for local media in MessageSender
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.