-
Notifications
You must be signed in to change notification settings - Fork 17
feat: 添加 AstrBot Pages 原生管理台 #82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Justice-ocr
wants to merge
8
commits into
DBJD-CR:main
Choose a base branch
from
Justice-ocr:codex/issue-74
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+6,757
−69
Open
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
51f621b
feat: add lightweight astrbot pages entry
Justice-ocr 5d447b0
feat: replace pages bridge with native console
Justice-ocr d9f9104
chore: document pages bridge save paths
Justice-ocr 5babdf8
chore: remove pages directory shortcuts
Justice-ocr 57c880c
chore: remove pages notifications and docs
Justice-ocr 0544c95
chore: trim pages stylesheet whitespace
Justice-ocr e707217
fix: address pages console review
Justice-ocr f64bd3f
fix: stop scheduling after unanswered limit
Justice-ocr File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果
self.session_data中session_id对应的值为None(例如数据损坏或初始化异常),self.session_data.get(session_id, {})将返回None。随后调用.get("unanswered_count", 0)会抛出AttributeError,而该异常并未被try-except (TypeError, ValueError)捕获,从而导致程序崩溃。建议使用更安全的防御性代码。There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已在 e707217 中修复:
_is_unanswered_limit_reached现在会先确认session_info是 dict,再读取unanswered_count;如果数据损坏为None或其他类型,会安全回退为 0,避免AttributeError。