Skip to content

feat(schema): manage Oracle-family triggers through existing object views - #68

Closed
efeire wants to merge 17 commits into
mainfrom
codex/schema-trigger-management
Closed

efeire wants to merge 17 commits into
mainfrom
codex/schema-trigger-management

Conversation

@efeire

@efeire efeire commented Oct 21, 2026 •

Copy link
Copy Markdown
Owner

当前实现:feat(schema): manage Oracle-family triggers in existing object workflows

变更说明 / Change Description

补齐 Oracle 系触发器的完整管理闭环,复用树、对象浏览器、表结构、源码标签/弹窗、依赖和无效对象诊断入口。

Schema / 所属对象 → 精确身份与完整定义 → SQL 预览/确认 → 单语句执行 → 真实状态与错误读回
                                                        └─ 结果未知:保留草稿,只读核实,不自动重放
  • Oracle、OceanBase Oracle、DM8 及已确认方言的 JDBC 使用可选 exact 元数据;保留旧调用、DM6 和其他数据库的既有合同。
  • 原始 owner/name、物理对象 ID、parent/scope 与执行 Schema 分开绑定;复杂定义及禁用状态完整保留,结构化修改须证明可无损重建。
  • 接齐发现/筛选/分页、新建、替换、删除、启停、支持的 COMPILE、parent 导航和现有依赖/诊断。写入后权威读回,执行成功不冒充 VALID。
  • main 新全局远程搜索沿用 exact/catalog/缓存合同;搜索投影中的菜单及批量动作按物理 ID 核实。确认删除后失效旧请求、精确移除原行并只读补查;unknown 保留原行和恢复入口,迟到响应不能复活已删对象。
  • 精确触发器元数据与包源码来源校验共用读取入口,保留 Schema、签名、可选来源及旧调用契约;结构编辑器同时保留触发器恢复与草稿元数据加载保护。

变更类型 / Change Type

  • 新功能 / New feature
  • Bug 修复 / Bug fix
  • 文档更新 / Documentation

涉及前端 / Frontend Changes

  • 已附可公开截图。

Schema 触发器列表、完整复杂定义及禁用/VALID 读回

OB Web GUI 已覆盖复杂源码保存、预览取消、撤销重做、启停/编译、INVALID 诊断与原稿修正、所属对象/依赖导航、分页、精确名称及删除取消/确认。全局远程搜索中的动作按物理身份执行;确认删除后移除搜索行,保留父表。

验证 / Validation

  • 已记录的生产入口回归覆盖精确身份、复杂定义、分页/搜索、批量删除、取消、迟到响应和状态/错误读回;相关格式、lint、类型检查及构建有通过证据,独立规范/需求审查及必改增量复核已完成。

  • OceanBase Oracle 4.2.5.7_107 原生 Agent/JDBC API 与 Web GUI 工作流已验收,包括 disabled/VALID、成功回执仍 INVALID、编译错误及精确清理。该版本的嵌入双引号触发器名返回 INVALID/ORA-00900,边界空格名称被拒绝;保留原名和错误,不宣称此类名称可编译。

  • exact 混合读取保留 INDEX、TYPE/TYPE_BODY 与 TRIGGER 的身份、统一排序及分页,已有 Rust/JDBC 回归覆盖。本次主线适配仅做静态差异核对与独立规范/需求审查,未运行本地测试、类型检查、构建、GUI 或实库;历史运行证据仅用于未变范围。

  • 当前提交的 CI 与自动审查以当前实现的 Checks 为准。

  • 未测与暂缓: 原生 Oracle、DM8 实库及对应真实对象 GUI;原生 Desktop GUI。Windows H2 两项真实进程测试曾在外层 60 秒超时,错峰仍失败,未计通过;Linux CI 不消除该本机缺口。

  • make check、make cargo-check-fast 未完整执行。

  • 包含主线 MCP Redis 错误文案的纯格式修正,以满足 Cargo fmt;不改变命令分派或权限行为。

Merge Danger

Door: one-way — 确认后的 DDL 可能已提交,代码回退不能撤销数据库修改;unknown/部分成功只读核实,不自动重放。

Blast Radius: Schema — Oracle 系触发器元数据、两套运输与现有交互入口;可选参数缺省保留旧合同。CI 依赖安装沿用主线共享脚本。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 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-10-21T15:43:00.380550Z 67392f7 PR opened
🔒 Security Review ✅ Completed 2026-10-24T09:26:55.382158Z 45ef19d 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.

@efeire
efeire force-pushed the codex/schema-trigger-management branch from 0a9ebcf to 45ef19d Compare October 24, 2026 09:20
@efeire efeire closed this Oct 24, 2026
@efeire
efeire deleted the codex/schema-trigger-management branch October 25, 2026 23:54
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.

8 participants