Skip to content

refactor(cooldown): 精简模型冷却展示与管理接口 - #606

Merged
tbphp merged 3 commits into
mainfrom
tbphp/model-cooldown-ui-polish
Sep 9, 2026
Merged

refactor(cooldown): 精简模型冷却展示与管理接口#606
tbphp merged 3 commits into
mainfrom
tbphp/model-cooldown-ui-polish

Conversation

@tbphp

@tbphp tbphp commented Sep 9, 2026

Copy link
Copy Markdown
Owner

关联 Issue / Related Issue

无关联 Issue;精简 #599 引入的模型冷却展示。

变更内容 / Change Content

  • Bug 修复 / Bug fix
  • 新功能 / New feature
  • 其他改动 / Other changes

凭据详情原本为每个冷却模型单独占一行,列表标记容易挤压,健康页还重复展示模型冷却提示和独立明细。本次将模型名与恢复时间合为可换行的紧凑标签,并移除多余展示及对应管理接口资源。

  • 调整 API Key 状态列宽和标记间距,订阅卡按完整标记换行,适配长名称、窄屏和明暗主题。
  • 删除分组模型冷却筛选、健康页顶部提示和“需要关注的凭据”下方的模型冷却列表;保留分组冷却计数。
  • 清理筛选参数、路由和缓存分支、无用汇总字段、健康模型明细及身份读取逻辑、对应文案和样式。
  • 补齐监控页面翻译依赖,精简中英日文的路由冷却状态文案。

验证:重新运行 make check 通过;相关移除行为完成 RED/GREEN 验证,9 个实际管理接口通过前端资源投影校验。已实际检查桌面、手机、长模型名、明暗主题、订阅详情、健康页和旧筛选链接规范化。

管理 API 同步移除 model_cooldown 筛选参数、凭据汇总与健康总计的 model_cooldown 字段,以及健康响应的 model_cooldown_credentials;分组计数仍保留。网关调度、模型冷却、恢复及保存机制不变,无数据迁移或依赖更新。Mock 数据与验收截图仅保留在本地,未提交;无需更新公开操作文档。

自查清单 / Checklist

  • 我已运行 make check,或在说明中写明无法运行的原因和未验证范围。 / I ran make check, or documented why it could not run and what remains unverified.
  • 本 PR 范围聚焦,未包含无关改动。 / This PR is focused and contains no unrelated changes.
  • 我已更新必要的公开文档或发布说明。 / I updated any required public documentation or release notes.
  • 我已确认提交、日志和测试数据不包含敏感信息。 / I confirmed that commits, logs, and fixtures contain no sensitive data.
  • 如适用,我已说明兼容性或数据迁移影响。 / Where applicable, I documented compatibility or data-migration impact.

Copilot AI lite review requested due to automatic review settings September 9, 2026 06:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-09T07:22:19.915729Z a61de4e New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: QUIET

Plan: Advanced

Run ID: 300375a2-b6cb-4eef-8fcb-fe76eeb6e5d7

📥 Commits

Reviewing files that changed from the base of the PR and between 7d9a2ed and a61de4e.

📒 Files selected for processing (26)
  • internal/control/credential_collection_helpers.go
  • internal/control/credential_mutations.go
  • internal/control/credentials.go
  • internal/control/health.go
  • internal/control/health_test.go
  • internal/control/model_cooldown.go
  • internal/control/model_cooldown_test.go
  • internal/control/runtime_observation.go
  • internal/control/wire_v3_contract_test.go
  • web/src/api/control/types.ts
  • web/src/app/query-keys.ts
  • web/src/app/resources/credentials.ts
  • web/src/app/resources/health.ts
  • web/src/components/ui/CredentialHealthBar.vue
  • web/src/components/ui/ModelCooldownDetails.vue
  • web/src/features/groups/credentials/GroupCredentialsTab.vue
  • web/src/features/groups/group-route.ts
  • web/src/features/monitor/HealthProblemCollection.vue
  • web/src/features/monitor/HealthSummaryStrip.vue
  • web/src/features/monitor/HealthTab.vue
  • web/src/i18n/locales/en-US/group.ts
  • web/src/i18n/locales/en-US/monitor.ts
  • web/src/i18n/locales/ja-JP/group.ts
  • web/src/i18n/locales/ja-JP/monitor.ts
  • web/src/i18n/locales/zh-CN/group.ts
  • web/src/i18n/locales/zh-CN/monitor.ts
💤 Files with no reviewable changes (10)
  • internal/control/model_cooldown.go
  • web/src/features/monitor/HealthTab.vue
  • web/src/i18n/locales/zh-CN/group.ts
  • internal/control/credential_mutations.go
  • web/src/app/query-keys.ts
  • web/src/i18n/locales/en-US/group.ts
  • web/src/features/monitor/HealthSummaryStrip.vue
  • web/src/features/groups/group-route.ts
  • internal/control/runtime_observation.go
  • web/src/i18n/locales/ja-JP/group.ts

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.


📝 Walkthrough

Walkthrough

本次变更移除了模型冷却筛选和凭据级模型冷却详情响应。模型冷却计数改为保留在健康组计数中。凭据页面调整状态徽章、冷却详情和响应式布局。监控页面移除模型冷却详情及提示。监控路由加载 monitorgroupsettings 消息命名空间。英文、日文和中文文案同步更新。

Priority: ➖ Normal

Merge Risk: ⚪ Minimal · up to a61de

This change removes credential-level model-cooldown filtering and detail responses while retaining group-level cooldown counts and updating the related management UI. The supplied validation indicates the revised contracts and displays are aligned, with no remaining merge-blocking risk.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 20 functions across 13 files. (4 skipped: … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed 标题准确概括了本次 PR 对模型冷却展示和管理接口的精简,内容清晰且与主要变更一致。
Description check ✅ Passed PR 描述包含关联 Issue、变更内容、详细说明、验证结果、兼容性影响和完整自查清单。内容与代码变更及 PR 目标一致。
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 20 functions across 13 files. (4 skipped: 4 unsupported.)


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tbphp tbphp changed the title fix(web): 精调模型冷却展示与筛选样式 refactor(cooldown): 精简模型冷却展示与管理接口 Sep 9, 2026
@tbphp tbphp self-assigned this Sep 9, 2026
@tbphp tbphp added the bug Something isn't working label Sep 9, 2026
@tbphp tbphp added this to the v2.0.0 milestone Sep 9, 2026
@tbphp
tbphp merged commit fb18dc9 into main Sep 9, 2026
11 checks passed
@tbphp
tbphp deleted the tbphp/model-cooldown-ui-polish branch September 9, 2026 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants