chore: sync master into dev (2026-08-15) - #9697
Closed
LIghtJUNction wants to merge 7 commits into
Closed
Conversation
dosubot
Bot
added
the
size:M
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
astrbot-docs | b9e565f | Aug 15 2026, 09:23 AM |
dosubot
Bot
added
area:webui
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The
updatedAtDisplaycomputed uses an inlineIntl.DateTimeFormatconfiguration; consider extracting this into a shared date/time formatting helper so plugin detail timestamps stay consistent with other parts of the dashboard and can be adjusted in one place. - The new
tm('detail.info.updatedAt')label relies on locale entries; double-check that allextension.jsonlocale files define this key to avoid runtime fallback behavior or missing labels in non-English languages.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The `updatedAtDisplay` computed uses an inline `Intl.DateTimeFormat` configuration; consider extracting this into a shared date/time formatting helper so plugin detail timestamps stay consistent with other parts of the dashboard and can be adjusted in one place.
- The new `tm('detail.info.updatedAt')` label relies on locale entries; double-check that all `extension.json` locale files define this key to avoid runtime fallback behavior or missing labels in non-English languages.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
…red (#9665) * fix(config): don't rotate dashboard password on password_change_required * style: format test_config.py with ruff
Keep model reasoning capability metadata out of provider request configuration, sanitize legacy source-backed records, preserve standalone provider fields, and add a reasoning_effort request-body preset.
dosubot
Bot
added
size:L
Contributor
Author
|
Closed in favor of #9710, which contains the conflict resolutions on a dedicated branch and preserves both |
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.
Summary
Synchronize the 3 commits currently on
masterbut missing fromdev.Included commits:
70b95e0feat: show plugin update time in marketplace details (feat: show plugin update time in marketplace details #9690)0fceeb3docs: add plugin attribution guidelinese262522fix(dashboard): adapt trace colors to theme (fix(dashboard): adapt trace colors to theme #9688)No force-push or history rewrite is involved. This PR will only be merged after GitHub reports it conflict-free and all required checks pass.
Summary by Sourcery
Sync dev branch with recent master changes around plugin details, trace theming, and plugin author guidelines.
New Features:
Bug Fixes:
Enhancements:
Documentation: