Skip to content

Conversation

@eryajf
Copy link
Member

@eryajf eryajf commented Jan 4, 2026

Summary by CodeRabbit

配置更新

  • Aliyun、Tencent 云提供商和 Jenkins CI/CD 现已在默认配置中禁用。

改进

  • 提供商账户配置现已支持数据库存储方式,提供了更灵活的配置管理能力。

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 4, 2026

Caution

Review failed

The pull request is closed.

功能概览

此 PR 将阿里云和腾讯云账户配置的获取逻辑从内存配置重构为数据库驱动,添加数据库优先的配置加载路径,并在数据库不可用时回退到 YAML 配置。同时更新了 Docker 镜像的包依赖和部分功能的默认开关设置。

变更清单

内聚体 / 文件 变更摘要
基础设施及配置
Dockerfile
新增 --no-cache 参数,扩展包安装范围:增加 bash、curl、wget、alpine-conf、busybox-extras、tzdata,及 nodejs、npm、python3、py3-pip、uv;新增三个环境变量(YARN_REGISTRY、NPM_CONFIG_REGISTRY、UV_INDEX_URL)。
默认配置禁用
config.example.yaml
将三个功能标志从 true 改为 false:aliyun.default.enabled、tencent.default.enabled、cicd.jenkins.enabled,禁用默认的阿里云和腾讯云账户及 Jenkins 集成。
数据库驱动的配置检索
internal/imcp/helpers.go
引入新的数据库驱动配置加载方法 getAliyunConfigByNameFromDBgetTencentConfigByNameFromDB,支持从 ConfigService 加载账户,在 DB 不可用时回退到 YAML 配置;新增 logx 和 service 依赖导入。
处理器函数重构
internal/imcp/handlers_aliyun_oss.go, internal/imcp/handlers_tencent_cos.go
移除内联的账户验证和选择逻辑,委托给新的数据库辅助方法;错误处理直接传播数据库查询结果,函数签名保持不变。
HTTP 服务配置更新
internal/server/http.go
集成新的 ConfigService 以从数据库加载云账户,添加调试日志以指示配置来源(数据库或 YAML),保持现有 YAML 回退路径;无公开函数签名变更。
子模块更新
zenops-web
子项目提交版本更新:从 3cc55e2 更新至 cde4266。

代码审查难度

🎯 3 (中等) | ⏱️ ~20 分钟

庆祝诗

🐰 兔兔跳跃舞蹁跹,
配置改由数据库选,
云账户从 DB 来,
YAML 退避做后援,
迁移完成齐欢颜!✨

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 80f61f6 and 0b64097.

📒 Files selected for processing (7)
  • Dockerfile
  • config.example.yaml
  • internal/imcp/handlers_aliyun_oss.go
  • internal/imcp/handlers_tencent_cos.go
  • internal/imcp/helpers.go
  • internal/server/http.go
  • zenops-web

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 and usage tips.

@eryajf eryajf added the bug Something isn't working label Jan 4, 2026
@eryajf eryajf merged commit a599255 into main Jan 4, 2026
1 of 2 checks passed
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