Skip to content

fix: worker handlers should read analysisModel from user config fallback#5

Closed
story2u wants to merge 4 commits intosaturndec:mainfrom
story2u:fix/worker-analysismodel-user-config-fallback
Closed

fix: worker handlers should read analysisModel from user config fallback#5
story2u wants to merge 4 commits intosaturndec:mainfrom
story2u:fix/worker-analysismodel-user-config-fallback

Conversation

@story2u
Copy link

@story2u story2u commented Feb 28, 2026

  • Changed 6 worker handlers to use getProjectModelConfig instead of direct DB read
  • Added regression tests for user config fallback scenario
  • Fixes issue where user-configured analysisModel was not recognized by workers

Summary

  • What changed:
  • Why:

Architecture Plan Sync (Required)

  • Master plan file: /Users/earth/Desktop/waoowaoo/docs/architecture-unification-master-plan.md
  • I updated the phase status board (✅/🔄/⏸/⚠️) in the same PR.
  • I recorded concrete file-level changes under the corresponding phase.
  • I kept the plan date/current-context consistent with the code changes in this PR.

Phase Status Delta (Required)

Phase Before After Evidence (file path)
e.g. Phase 1.2 🔄 src/lib/query/task-target-overlay.ts

Guardrails Checklist (Required)

  • No compatibility layer introduced.
  • No silent fallback introduced.
  • No hidden default or fake data introduced.
  • Errors remain explicit and traceable.

Validation

  • Commands run:
  • Manual verification:

Risks

  • Known impact/risk:
  • Follow-up tasks:

saturndec and others added 4 commits February 27, 2026 19:25
- Changed 6 worker handlers to use getProjectModelConfig instead of direct DB read
- Added regression tests for user config fallback scenario
- Fixes issue where user-configured analysisModel was not recognized by workers
saturndec added a commit that referenced this pull request Feb 28, 2026
### ✨ 新功能
- 增加 OpenAI 兼容图片、视频格式支持
### 🐛 修复
- 修复默认模型配置后项目模型需要二次选择的问题
- 修复部分情况 resolution 无法读取的问题
- 修复模型链路为 LangGraph
- 修复默认参数无选择问题
- 修复关闭计费依然触发计费问题
- 修复 openai-compatible 被误判为原生 OpenAI 推理问题
- 修复 JSON 解析失败问题
### ⚙️ 优化
- 修改为默认计费 off
- 增强提示词 JSON 格式限制
p1927 pushed a commit to p1927/waoowaoo-2 that referenced this pull request Mar 4, 2026
### ✨ 新功能
- 增加 OpenAI 兼容图片、视频格式支持
### 🐛 修复
- 修复默认模型配置后项目模型需要二次选择的问题
- 修复部分情况 resolution 无法读取的问题
- 修复模型链路为 LangGraph
- 修复默认参数无选择问题
- 修复关闭计费依然触发计费问题
- 修复 openai-compatible 被误判为原生 OpenAI 推理问题
- 修复 JSON 解析失败问题
### ⚙️ 优化
- 修改为默认计费 off
- 增强提示词 JSON 格式限制
androidly pushed a commit to androidly/waoowaoo that referenced this pull request Mar 7, 2026
### ✨ 新功能
- 增加 OpenAI 兼容图片、视频格式支持
### 🐛 修复
- 修复默认模型配置后项目模型需要二次选择的问题
- 修复部分情况 resolution 无法读取的问题
- 修复模型链路为 LangGraph
- 修复默认参数无选择问题
- 修复关闭计费依然触发计费问题
- 修复 openai-compatible 被误判为原生 OpenAI 推理问题
- 修复 JSON 解析失败问题
### ⚙️ 优化
- 修改为默认计费 off
- 增强提示词 JSON 格式限制
@saturndec saturndec closed this Mar 7, 2026
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.

3 participants