Skip to content

🐛 修复脚本设置缺少 context-menu 运行时机 - #1652

Merged
CodFrm merged 2 commits into
mainfrom
fix/1649-run-at-context-menu
Aug 4, 2026
Merged

🐛 修复脚本设置缺少 context-menu 运行时机#1652
CodFrm merged 2 commits into
mainfrom
fix/1649-run-at-context-menu

Conversation

@CodFrm

@CodFrm CodFrm commented Aug 4, 2026

Copy link
Copy Markdown
Member

Checklist / 检查清单

  • Fixes mentioned issues / 修复已提及的问题
  • Code reviewed by human / 代码通过人工检查
  • Changes tested / 已完成测试

背景

@run-at context-menu 的运行能力已在 #1442 中实现,但脚本设置页的“运行时机”下拉框没有提供该选项。用户因此无法通过自定义元数据将订阅脚本切换为手动执行,只能直接改写脚本源码。

本次改动

  • 在脚本设置页的运行时机选项中补充 context-menu
  • 选择后清除 early-start 覆盖,并写入 run-at: ["context-menu"]
  • 保留最小组件回归测试:选项缺失或持久化值错误时测试会失败。
  • 清理 ScriptEditor 内 5 条已被相邻交互测试覆盖或仅做属性直出的低价值测试,不改变生产行为。

建议审查重点

  • context-menu 是否作为普通 run-at 值保存,而不会进入 early-start 的特殊分支。
  • 用户覆盖是否通过 selfMetadata 保存,不需要改写订阅脚本源码。
  • 删除的测试是否均由现有交互/分支测试覆盖,且未丢失独立回归信号。

关联

Closes #1649

Related: #1038, #1442

验证

  • pnpm exec vitest run --no-coverage --reporter=verbose <5 个受影响测试文件> — 5 个测试文件、82 个测试通过。
  • pnpm exec vitest run --no-coverage --reporter=verbose src/pages/options/routes/ScriptEditor — 11 个测试文件、128 个测试通过。
  • pnpm run lint — 通过,包括 Prettier、TypeScript、i18n、Issue 模板及 ESLint 检查。
  • pnpm test — 311 个测试文件、3496 个测试全部通过。首次全量运行中 scripts/check-issue-templates.test.mjs 在并行负载下发生一次 340ms 超时;该文件单独重跑 18/18 通过(目标用例 155ms),随后原样全量重跑 3496/3496 通过,未修改无关测试或超时配置。

Screenshots / 截图

N/A — 仅补齐现有下拉框中的受支持选项,未修改布局、样式或主题;交互行为由组件测试覆盖。

@CodFrm CodFrm changed the title 修复脚本设置缺少 context-menu 运行时机 🐛 修复脚本设置缺少 context-menu 运行时机 Aug 4, 2026
@CodFrm
CodFrm merged commit 65121fa into main Aug 4, 2026
9 of 10 checks passed
@CodFrm
CodFrm deleted the fix/1649-run-at-context-menu branch August 4, 2026 09:37
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.

[Feature] 增加手动执行脚本的功能

1 participant