Skip to content

Conversation

@Janice-shi
Copy link
Contributor

@Janice-shi Janice-shi commented Oct 24, 2025

依赖检查

组件之间的依赖声明,是微服务组件架构下的重要信息,请确保其正确性。

请勾选以下两组选项其中之一:

  • 本次 MR 没有使用上游组件(例如框架、后台组件等)的较新版本提供的特性。

或者:

  • 本次 MR 使用了上游组件(例如框架、后台组件等)的较新版本提供的特性。
  • 在对应的文件中更新了该上游组件的依赖版本(或确认了当前声明的依赖版本已包含本次 MR 使用的新特性)。

提交信息检查

Git 提交信息将决定包的版本发布及自动生成的 CHANGELOG,请检查工作内容与提交信息是否相符,并在以下每组选项中都依次确认。

破坏性变更是针对于下游使用者而言,可以通过本次改动对下游使用者的影响来识别变更类型:

  • 下游使用者不做任何改动,仍可以正常工作时,那么它属于普通变更。
  • 反之,下游使用者不做改动就无法正常工作时,那么它属于破坏性变更。

例如,构件修改了一个属性名,小产品 Storyboard 中需要使用新属性名才能工作,那么它就是破坏性变更。
又例如,构件还没有任何下游使用者,那么它的任何变更都是普通变更。

破坏性变更:

  • ⚠️ 本次 MR 包含破坏性变更的提交,请继续确认以下所有选项:
  • 没有更好的兼容方案,必须做破坏性变更。
  • 使用了 feat 作为提交类型。
  • 标注了 BREAKING CHANGE: 你的变更说明
  • 同时更新了本仓库中所有下游使用者的调用。
  • 同时更新了本仓库中所有下游使用者对该子包的依赖为即将发布的 major 版本。
  • 同时为其它仓库的 Migrating 做好了准备,例如文档或批量改动的方法。
  • 手动验证过破坏性变更在 Migrate 后可以正常工作。
  • 破坏性变更所在的提交没有意外携带其它子包的改动。

新特性:

  • 本次 MR 包含新特性的提交,且该提交不带有破坏性变更,并使用了 feat 作为提交类型。
  • 给新特性添加了单元测试。
  • 手动验证过新特性可以正常工作。

问题修复:

  • 本次 MR 包含问题修复的提交,且该提交不带有新特性或破坏性变更,并使用了 fix 作为提交类型。
  • 给问题修复添加了单元测试。
  • 手动验证过问题修复得到解决。

杂项工作:

即所有对下游使用者无任何影响、且没有必要显示在 CHANGELOG 中的改动,例如修改注释、测试用例、开发文档等:

  • 本次 MR 包含杂项工作的提交,且该提交不带有问题修复、新特性或破坏性变更,并使用了 chore, docs, test 等作为提交类型。

Summary by CodeRabbit

发布说明

  • 新功能

    • 添加Skywalking分析支持,用于性能追踪
    • 新增路由黑名单功能
    • 改进预览模式的初始化流程
    • 扩展公共API,包含WebSocket和模态框功能
  • 改进

    • 优化缓存管理策略
    • 完善网络错误处理
    • 升级多个依赖包至最新版本
  • 其他

    • 更新CI/CD工作流配置
    • 调整主题系统架构

@coderabbitai
Copy link

coderabbitai bot commented Oct 24, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

该PR对多个包进行版本更新和功能扩展,包括:GitHub工作流调整(缓存策略、E2E测试顺序)、依赖版本升级、类型声明扩展、源代码增强(SkyWalking分析、预览机制、主题系统重构)以及构建工具链优化。

Changes

内聚块 / 文件 变更摘要
GitHub工作流更新
.github/workflows/ci.yml, .github/workflows/cleanup-caches.yml, .github/workflows/lerna-version.yml, .github/workflows/update-docs-etc.yml
更新Yarn/Nx缓存键包含yarn.lock;调整E2E测试执行顺序至Codecov后;增加CODECOV_TOKEN;更新Taobao镜像地址;新增PR关闭时缓存清理工作流;Lerna版本锁定至v5
根级配置变更
.gitignore, .size-limit.js, README.md, conf/config.yaml, package.json
添加.history忽略项;提升多个构件尺寸限制;添加storyboard.json监控;更新CI徽章URL;新增YAML配置文件;更新依赖版本
类型声明扩展
declarations/global.d.ts
新增*.gif模块声明;更新*.png为string类型;扩展Window接口添加7个新属性(APP_ROOT_TPL、BOOTSTRAP_UNION_FILE、USE_SKYWALKING_ANALYSIS等)
DLL包版本更新与CHANGELOG
dll/ace/, dll/d3/, dll/echarts/, dll/editor-bricks-helper/, dll/react-dnd/
包版本批量升级(ace 2.0.598→2.0.811、d3 2.0.78→2.0.84等);依赖版本同步更新;CHANGELOG条目新增和重组
DLL清单扩展
dll/editor-bricks-helper/manifest.snapshot.json, packages/brick-dll/manifest.snapshot.json
增加brick-utils、brick-kit、editor-bricks-helper的公共导出项(函数、类型、工具类);扩展@easyops/loader和rc-util导出表面
API文档更新
etc/brick-kit.api.md, etc/brick-types.api.md
新增组件工厂、WebSocket类型、模态栈、数据值选项等公共导出;扩展Story、ContextResolve、Snippet等类型定义;系统类型重构
brick-container核心源码
packages/brick-container/src/index.tsx, packages/brick-container/src/preview.ts, packages/brick-container/src/polyfill.ts
新增SkyWalking追踪初始化;移除预览相关接口;新增replaceUseChildren工具;新增Array polyfill(to-sorted、to-reversed、to-spliced);重构引导流程返回状态
brick-container预览和初始化
packages/brick-container/src/preview/initialize.ts, packages/brick-container/src/preview/listen.ts
新增初始化检查函数;新增跨域预览消息监听器;支持start-preview、ui-test-preview、iframe初始化路径;origin验证逻辑
brick-container服务层
packages/brick-container/serve/getIndexHtml.js, packages/brick-container/serve/getProxies.js, packages/brick-container/serve/getEnv.js, packages/brick-container/serve/liveReload.js
提取getRawIndexHtml和injectIndexHtml函数;添加Skywalking分析标志;扩展代理路径处理;新增WebSocket实时重载模块;环境配置增强
brick-container样式系统重构
packages/brick-container/src/styles/theme/default.css, packages/brick-container/src/styles/theme/dark.css, packages/brick-container/src/styles/theme/index.css, packages/brick-container/src/styles/variables.css
删除light和dark-v2主题CSS变量定义;移除generated.css、default.css、dark.css导入;清理预定义CSS变量
brick-container国际化
packages/brick-container/src/i18n/constants.ts, packages/brick-container/src/i18n/index.ts, packages/brick-container/src/i18n/locales/*
新增NETWORK_ERROR本地化键;设置HTML根元素lang属性;添加中英文错误提示
brick-container工具和脚本
packages/brick-container/tools/link_static_file.py, packages/brick-container/tools/report_installed_*.py, packages/brick-container/tools/utils/get_headers.py, packages/brick-container/tools/utils/custom_get_token_util.py
新增硬链接静态文件管理、Union应用报告、微应用SA报告、令牌获取工具;支持拓扑应用处理
brick-container包配置和部署
packages/brick-container/package.json, packages/brick-container/deploy/package.conf.yaml, packages/brick-container/deploy/*_postscript.sh
版本升级2.84.14→2.98.8;移除nx构建目标和prebuild脚本;更新多个devDependencies;依赖调整(micro_app_service_db移除、logic.micro_app_standalone_service新增);新增部署初始化脚本
brick-container HTML和Webpack
packages/brick-container/src/browse-happy.ejs, packages/brick-container/src/index.ejs, packages/brick-container/webpack/common.js, packages/brick-container/webpack/dev.js
增强浏览器降级提示;添加Skywalking分析标志注入;新增图标资源复制;重构devServer配置为命名导入
brick-container辅助模块
packages/brick-container/src/replaceChildren.ts, packages/brick-container/src/replaceUseChildren.ts, packages/brick-container/src/utils.ts
新增replaceChildren和replaceUseChildren polyfill/工具;新增getSpanId UUID生成函数
其他包更新
packages/babel-preset-next/, packages/babel-preset-prismjs/, packages/brick-http/, packages/brick-icons/, packages/brick-dll/
babel-preset-next添加逻辑赋值转换插件;brick-icons版本升级并重构为外部维护;包版本和依赖同步升级;CHANGELOG维护
cypress测试和git配置
cypress/e2e/home.spec.js, packages/brick-container/.gitignore
测试导航添加调试控制台参数;移除generated.css追踪
PKGBuild配置
.pkgbuild/@easyops/PKGBUILD
在package()函数中新增conf目录复制

Estimated code review effort

🎯 5 (Critical) | ⏱️ ~120 minutes

复杂度分析

  • 异构性极高:涉及18+个不同领域(工作流、类型系统、核心源码、服务层、样式、工具脚本、Python部署脚本)
  • 逻辑密度高:新增SkyWalking追踪、预览系统重构、主题系统完全重构、多层代理和HTML注入逻辑
  • 文件分布广泛:超过100+个文件修改,跨多个包和层次
  • API表面扩展:公共导出项大幅增加(brick-kit、brick-types)
  • 架构变更:样式系统、预览机制、启动流程等核心组件重构
  • 版本依赖链:多个包的级联版本升级需交叉验证兼容性

需要对每个领域进行独立的逻辑验证、集成点检查和回归测试评估。

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title Check ⚠️ Warning 标题 "Janice/fix/standalone na token" 看起来更像分支名称而非正式的 PR 标题。虽然包含了 "fix" 和 "standalone na token" 等信息,但格式不标准,前缀 "Janice/" 通常不应出现在 PR 标题中。更重要的是,从代码变更摘要来看,此 PR 涉及 CI 工作流修改、多个包版本更新、配置文件新增、Python 脚本添加等多个方面的大量改动,而标题仅提及 "standalone na token",无法准确反映这些广泛的变更内容。标题显得过于简略且与实际工作范围不相符。 建议将标题修改为更清晰、更正式的格式,例如:"fix: Add token support and infrastructure improvements for standalone NA" 或根据最主要的变更内容进行调整。标题应移除分支前缀,采用标准的提交消息格式(fix: 描述),并能够准确概括此 PR 的核心目的。
Docstring Coverage ⚠️ Warning Docstring coverage is 2.50% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed PR 描述遵循了仓库提供的中文检查清单模板,在依赖检查部分已完整填写(选择了"没有使用上游组件新特性"选项),在提交信息检查部分确认了这是问题修复类提交(勾选了第一项)。但是,问题修复类别下的两个后续项目("给问题修复添加了单元测试"和"手动验证过问题修复得到解决")均未勾选。此外,模板中的"简单描述"和"Todo"部分被注释且未填写,缺少对实际工作内容的说明。整体而言,描述完成了基本的分类确认但对具体工作内容的文字说明不足。

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.

@Janice-shi Janice-shi changed the base branch from v3 to master October 24, 2025 09:00
@Janice-shi Janice-shi requested a review from qiaofengxi October 24, 2025 09:01
qiaofengxi
qiaofengxi previously approved these changes Oct 24, 2025
@cypress
Copy link

cypress bot commented Oct 24, 2025

next-core    Run #11780

Run Properties:  status check passed Passed #11780  •  git commit 81780e7780 ℹ️: Merge 007224cfe23be299edba7964e1ef0cacd24ff90a into dbbd720b1839dda831ff4c715e4c...
Project next-core
Branch Review refs/pull/4814/merge
Run status status check passed Passed #11780
Run duration 00m 28s
Commit git commit 81780e7780 ℹ️: Merge 007224cfe23be299edba7964e1ef0cacd24ff90a into dbbd720b1839dda831ff4c715e4c...
Committer janice shi
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 6
View all changes introduced in this branch ↗︎

@qiaofengxi qiaofengxi merged commit cbab594 into master Oct 24, 2025
8 checks passed
@qiaofengxi qiaofengxi deleted the janice/fix/standalone-na-token branch October 24, 2025 09:30
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