fix: use favicon as default plugin icon

[wcqqq1214]

Modifications / 改动点

The default placeholder icon for plugins still used the legacy plugin_icon.png after AstrBot switched to the current blue-star logo. This change points the default plugin icon at the official favicon (/favicon.svg) for plugins that do not ship their own logo.png.

• Core files modified (one-line change each, import defaultPluginIcon from "@/assets/images/plugin_icon.png" → import defaultPluginIcon from "/favicon.svg"):

  • dashboard/src/components/shared/ExtensionCard.vue
  • dashboard/src/views/ExtensionPage.vue
  • dashboard/src/views/extension/MarketPluginsTab.vue
  • dashboard/src/views/extension/PluginDetailPage.vue

• No related issue — this is a small, self-contained fix, so no issue was filed.

• Note: the old asset dashboard/src/assets/images/plugin_icon.png has not been deleted; it is still used as the brand logo in Logo.vue and SetupPage.vue.

• This is NOT a breaking change. / 这不是一个破坏性变更。

Screenshots or Test Results / 运行截图或测试结果

Before / 修复前:

After / 修复后:

---

Checklist / 检查清单

• 😊 If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
  / 如果 PR 中有新加入的功能，已经通过 Issue / 邮件等方式和作者讨论过。

• 👀 My changes have been well-tested, and "Verification Steps" and "Screenshots" have been provided above.
  / 我的更改经过了良好的测试，并已在上方提供了"验证步骤"和"运行截图"。

• 🤓 I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in requirements.txt and pyproject.toml.
  / 我确保没有引入新依赖库，或者引入了新依赖库的同时将其添加到 requirements.txt 和 pyproject.toml 文件相应位置。

• 😮 My changes do not introduce malicious code.
  / 我的更改没有引入恶意代码。

Summary by Sourcery

Bug Fixes:

• Correct the fallback plugin icon to reference the current favicon instead of the legacy plugin_icon asset.

Summary by Sourcery

Bug Fixes:

• Correct fallback plugin icons across extension-related views to reference the current favicon instead of the legacy plugin icon asset.

[dosubot]

📄 Knowledge review

Dosu skipped reviewing this PR because your organization has used its 200 included credits for the month. Your usage will reset on 2026-09-01. To have Dosu review this PR before then, ask your organization admin to upgrade to a pro account.

---

[sourcery-ai]

Hey - I've left some high level feedback:

• You’re now hardcoding the same defaultPluginIcon import in four places; consider centralizing this (e.g., a shared constant or composable) so updates to the default icon only have to be done in one file.
• Importing /favicon.svg via ES module syntax may behave differently across bundlers/environments; it might be more robust to reference the public URL directly in templates or via a shared config rather than treating it as a bundled asset.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- You’re now hardcoding the same `defaultPluginIcon` import in four places; consider centralizing this (e.g., a shared constant or composable) so updates to the default icon only have to be done in one file.
- Importing `/favicon.svg` via ES module syntax may behave differently across bundlers/environments; it might be more robust to reference the public URL directly in templates or via a shared config rather than treating it as a bundled asset.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}