已开启
[mirror] openJiuwen-ai/agent-runtime#124: docs: list all supported RUNTIME_DB_TYPE values in zh quick start #475
[mirror] openJiuwen-ai/agent-runtime#124: docs: list all supported RUNTIME_DB_TYPE values in zh quick start #475
已开启
openjiuwen-sync创建于 6 天前
openjiuwen-sync成员
6 天前

Paired: GitHub #124GitCode !475

github_pr: 124
head_repo: itcastWsy/agent-runtime
head_ref: docs/fix-runtime-db-type-list
head_sha: 8a2f129945fd4aabbfabfb3a5378fde8e76f9af8
base_repo: openJiuwen-ai/agent-runtime
base_ref: main
github_repo: openJiuwen-ai/agent-runtime
base_branch: main
base_sha: 34ed6e86b99fab9b3f07d9f063efc1692ab365d2
github_user: itcastWsy

GitHub PR mirror for CodeArts CI.

问题 / Problem

docs/zh/1. 快速开始.md 第 18 行写着 RUNTIME_DB_TYPE 只支持 mysqlsqlite,与代码和仓库内其余所有文档不一致。

Line 18 of docs/zh/1. 快速开始.md states that RUNTIME_DB_TYPE supports only mysql and sqlite, which contradicts both the source code and every other document in this repository.

证据 / Evidence

来源 Source 内容 Content
applications/lowcode_agent/openjiuwen_runtime/examples/lowcode_agent/lowcode_agent_runner.py:44 _VALID_DB_TYPES = {"mysql", "sqlite", "gaussdb", "opengauss"}
docs/zh/1. 快速开始.md:43(同一文件) 支持 mysqlsqlitegaussdbopengauss
docs/zh/2. 配置说明.md:11 支持 mysql、sqlite、gaussdb、opengauss 四个可选值
docs/en/1. Quick Start.md:18 mysql, sqlite, gaussdb, or opengauss
README.md:95 / README_zh.md:95 四个值均列出 / all four values listed
docs/zh/1. 快速开始.md:18(修改前) ❌ 仅 mysqlsqlite / only mysql and sqlite

同一份中文快速开始文档内部就自相矛盾(第 18 行 vs 第 43 行),而且第 46–50 行紧接着还给出了 gaussdb / opengauss 的可选依赖安装说明。读者按第 18 行理解会误以为 Runtime 不支持 GaussDB / openGauss。

The file contradicts itself (line 18 vs. line 43), and lines 46–50 immediately below even explain how to install the async-gaussdb optional dependency for gaussdb / opengauss. A reader stopping at line 18 would wrongly conclude that GaussDB / openGauss is unsupported.

修改 / Change

仅一处文字修改,与同文件第 43 行的写法保持一致。No functional change — one wording fix, aligned with line 43 of the same file.

-- `RUNTIME_DB_TYPE`:支持 `mysql` 和 `sqlite`
+- `RUNTIME_DB_TYPE`:支持 `mysql`、`sqlite`、`gaussdb` 和 `opengauss`

验证 / How to verify

sed -n '44p' applications/lowcode_agent/openjiuwen_runtime/examples/lowcode_agent/lowcode_agent_runner.py
sed -n '18p;43p' "docs/zh/1. 快速开始.md"
likedislike
合并受阻
Oopenjiuwen-sync成员
6 天前 添加了label:github-mirror
atomgit-bot
atomgit-bot
6 天前 评论:

变更摘要

本 PR 为文档修正类改动(源自 GitHub 镜像 PR),仅修改 docs/zh/1. 快速开始.mdRUNTIME_DB_TYPE 配置项的支持值说明:将第 18 行由「仅支持 mysqlsqlite」更正为完整列出四个可选值,消除同文件内部(第 18 行与第 43 行)及与仓库其他文档之间的不一致,不涉及任何代码功能变更。

主要改动

  • 修正 RUNTIME_DB_TYPE 支持值清单:将 docs/zh/1. 快速开始.md 第 18 行的 RUNTIME_DB_TYPE 说明从「支持 mysqlsqlite」改为「支持 mysqlsqlitegaussdbopengauss」,补齐了原先遗漏的两个可选值。
  • 与同文件其他章节保持一致:修改后第 18 行与同文件第 43 行的表述一致,避免读者按第 18 行误判 GaussDB / openGauss 不受 Runtime 支持。
  • 无功能性影响:本次仅为一处文字修改,未改动任何代码逻辑或配置行为。
likedislike
atomgit-bot
atomgit-bot
6 天前 评论:

🤖 本次改动均为数据 / 生成 / 二进制文件(如测试用例、锁文件、媒体资源),没有可审查的代码逻辑,已跳过 AI 代码审查。

likedislike
openJiuwen-bot成员
6 天前 评论:

欢迎来到 openJiuwen 社区

Hey @openjiuwen-sync , 感谢你对社区的贡献.

机器人使用手册

有关指令的使用,可以点击 此处 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。

likedislike
atomgit-bot
atomgit-bot
6 天前 评论:

AtomGit AI 助手使用指南

在 PR 评论中使用命令即可触发。支持在普通评论和代码行评论中使用。

功能 命令 说明
代码审查 /ai review 检查代码质量、潜在问题、安全风险
PR 摘要 /ai summary 生成 PR 变更内容的结构化摘要
代码解释 /ai explain 解释代码变更的逻辑和意图
自由提问 @atomgit-bot <问题> 询问关于本 PR 的任何问题
帮助 /ai help 显示此帮助信息

免责声明

AI 助手可能存在误判,请结合自身判断。可以对评论点 👍 或 👎 帮助我们改进。

likedislike
Oopenjiuwen-sync成员
6 天前 修改了pull request 的描述
OopenJiuwen-bot成员
6 天前 添加了label:openJiuwen-cla/yes
openJiuwen-bot成员
6 天前 评论:

CLA 签署成功

openjiuwen-sync, 感谢您的代码贡献。 所有的代码贡献者都已完成了 CLA 签署。 👍

likedislike
openJiuwen-bot成员
6 天前 评论:
任务名称 结果 日志操作
静态检查 ✅SUCCESS 点此跳转
禁用词扫描 ✅SUCCESS 点此跳转
防投毒检查 ✅SUCCESS 点此跳转
开源合规检查 ✅SUCCESS 点此跳转
UT测试 N/A N/A
ST测试 N/A N/A
build 编译包 N/A N/A
ruff codecheck {{ruff_codecheck_result}} {{ruff_codecheck_link_text}}
issue关联检查 {{issue_linked_result}} {{issue_linked_link_text}}
likedislike
OopenJiuwen-bot成员
6 天前 添加了label:ci-successful
openJiuwen-bot成员
4 天前 评论:

@itcastWsy on GitHub:

补充说明:本 PR 的目标分支已从 main 调整为 develop

依据:deepsearch#157 中维护方反馈「代码应该优先合入 dev 分支」,同一策略应也适用于本仓库——develop 是活跃开发线(当前领先 main 121 个提交)。

切换前已核实:本 PR 唯一改动的 docs/zh/1. 快速开始.mdmaindevelop 上内容完全一致(blob e28545d5),因此 +1/−1 的改动在 develop 上同样成立且无冲突。

likedislike
openJiuwen-bot成员
4 天前 评论:

@itcastWsy on GitHub:

更正上一条评论:经切换后实测,本 PR 不适合将 base 切到 develop,已切回 main,抱歉造成混乱。

实测原因:maindevelop 在本仓库是分叉关系(develop 领先 121 个提交、main 领先 10 个),并非祖先关系。本 PR 的 head 基于 main,若改以 develop 为 base,diff 会额外包含 main 独有的 10 个提交的全部改动(README 重构、Dockerfile 等),GitHub 判定 mergeable=dirty,强行合并会把发布线改动带入 develop

本 PR 保持以 main 为 base(diff 恒为 docs/zh/1. 快速开始.md +1/−1)。「优先合入 dev 分支」的策略由维护者通过 main→develop 的常规同步机制落地即可,无需本 PR 参与分支调整。

likedislike