Skip to content

feat: show audit report source fields and hit summaries - #897

Merged
LordofAvernus merged 7 commits into
zjrc_3.2408from
dev-zjrc-sql-audit-report-summary
Sep 28, 2026
Merged

LordofAvernus merged 7 commits into
zjrc_3.2408from
dev-zjrc-sql-audit-report-summary

Conversation

@LordofAvernus

@LordofAvernus LordofAvernus commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

关联的 issue

https://github.com/actiontech/sqle-ee/issues/3155

描述你的变更

  • 接入 audit_summary 类型与请求,支撑任务级汇总
  • 报告头 BasicInfoWrapper 展示数据源/地址/实例类型/SQL 来源/来源文件
  • 新增等级命中数与规则细则组件,配套 i18n 与单测快照
  • 错误等级命中展示为「错误 N(P0 a,P1 b)」,次数为 0 的一侧不显示

确认项(pr提交后操作)

Tip

请在指定复审人之前,确认并完成以下事项,完成后✅


  • 我已完成自测
  • 我已记录完整日志方便进行诊断
  • 我已在关联的issue里补充了实现方案
  • 我已在关联的issue里补充了测试影响面
  • 我已确认了变更的兼容性,如果不兼容则在issue里标记 not_compatible
  • 我已确认了是否要更新文档,如果要更新则在issue里标记 need_update_doc

Made with Cursor

LordofAvernus and others added 6 commits September 23, 2026 16:02
Wire shared SQLE task client to the new task-level audit summary API.

Co-authored-by: Cursor <cursoragent@cursor.com>
Cover quick-audit detail header fields, level hits, and rule details.

Co-authored-by: Cursor <cursoragent@cursor.com>
Render datasource, address, instance type, schema, SQL source, and files.

Co-authored-by: Cursor <cursoragent@cursor.com>
Fetch audit_summary once and render dedicated blocks on report detail.

Co-authored-by: Cursor <cursoragent@cursor.com>
Keep Detail unit coverage aligned with header and summary blocks.

Co-authored-by: Cursor <cursoragent@cursor.com>
Align UI_VERSION and CE/EE VersionModal snaps with the feature branch tip.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Coverage report

St.❔
Category Percentage Covered / Total
🟢 Statements 90.01% 15379/17085
🟡 Branches 73.03% 6468/8857
🟢 Functions 90.39% 5359/5929
🟢 Lines 90.53% 15115/16696

Test suite run success

2321 tests passing in 629 suites.

Report generated by 🧪jest coverage report action from ff4f675

Render P0 and P1 inside the error hit count so zero sides stay hidden and the summary reads as one phrase.
@LordofAvernus
LordofAvernus merged commit e350831 into zjrc_3.2408 Sep 28, 2026
9 checks passed
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.

1 participant