Skip to content

Codex/provider mvp support surface - #1

Merged
creatang merged 20 commits into
mainfrom
codex/provider-mvp-support-surface
Mar 28, 2026
Merged

creatang merged 20 commits into
mainfrom
codex/provider-mvp-support-surface

Conversation

@creatang

Copy link
Copy Markdown
Owner

背景

当前项目容易让人误以为 NeoCode MVP 已正式支持 OpenAI、Anthropic、Gemini 多 Provider。

但从实际实现看,当前真正可用的只有 OpenAI-compatible,Anthropic 和 Gemini 仍然只是 scaffold。

这会带来几个问题:

  • 用户误以为多 Provider 已可用
  • 后续 UI 容易暴露未实现 Provider
  • 模型选择和配置容易出错
  • MVP 能力边界不够清晰

本次改动

本 PR 只修改 internal/provider/,先把 Provider 层的支持状态和能力边界定义清楚,作为后续 UI / 配置 / 文档的单一事实来源。

主要改动

  • 为 Provider 增加统一描述信息

    • 名称
    • 展示名
    • 支持级别
    • 是否属于 MVP 正式支持
    • 是否当前可用
    • 可展示模型列表
  • 扩展 Provider 能力

    • 增加 Descriptor()
    • 增加统一 Describe() 辅助方法
  • OpenAI-compatible 明确标记为

    • 当前唯一 MVP 正式支持 Provider
    • 当前唯一可用 Provider
  • Anthropic / Gemini 明确标记为

    • scaffolded
    • 当前 MVP 不可用
  • Anthropic / Gemini 的错误信息改为明确说明

    • 当前仅为 scaffold
    • 当前 MVP 仅正式支持 OpenAI-compatible
  • 为 Registry 增加筛选能力

    • Descriptors()
    • MVPDescriptors()
    • AvailableDescriptors()

预期收益

  • 降低误导性
  • 降低首次使用失败率
  • 为 UI / 配置层提供统一的 Provider 状态来源
  • 让 MVP 能力边界更可信

验证

已执行:

go test ./internal/provider/...

Yumiue and others added 20 commits March 25, 2026 17:12
fix: 修复记忆模块误写入与错误分类问题
…-stopgap

fix(provider): 避免流式错误静默结束
- docs: 文件名统一使用 kebab-case,正文全面中文化,并严格按官方规范重写 agents.md。
- refactor(tui): 拆分臃肿的 app.go (分为 app, update, view, commands),将所有交互统一收敛至 slash command (/)。
- feat(tools): 文件系统新增 grep, glob 和 edit 工具,其中 edit 包含严格的防歧义替换拦截机制。
- test: 为 config, registry 和 openai 补充表驱动测试(引入 httptest 模拟 SSE 流),核心模块覆盖率达 80%+。
- chore: 清除全局硬编码配置,并移除 Markdown 文档中无意义的空行。
feat: rebuild neocode as a local coding agent MVP
@creatang
creatang merged commit b4da678 into main Mar 28, 2026
1 check passed
@creatang
creatang deleted the codex/provider-mvp-support-surface branch April 10, 2026 05:40
creatang pushed a commit that referenced this pull request Apr 14, 2026
test(config): improve selection state branch coverage
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