Conversation
1. 移除流程引擎:删除 springboot-starter-flow 模块及全部关联内容 (example-infra-flow、Flow/请假示例、entrust 委托链路、文档、前端页面), 流程引擎已重构为独立仓库:https://github.com/codingapi/flow-engine, README 增加迁移说明 2. 前端 monorepo:admin-ui/mobile-ui 合并至 frontend/(pnpm workspace), apps/pc、apps/mobile 仅保留 pc/mobile 两组指令,公共代码抽至 packages/shared(@springboot-framework/shared),结构参考 flow-frontend 3. 版本管理:采用 ${revision} + flatten-maven-plugin(resolveCiFriendliesOnly), 开发版本锁定 17.3.0-SNAPSHOT(17=JDK 版本,3=Spring Boot 大版本), 发版通过 -Drevision=x.y.z 指定正式版本 Co-Authored-By: Claude <noreply@anthropic.com>
- 新增 .github/workflows/ci.yml:JDK 17 编译 + travis profile 单元测试 + jacoco 覆盖率上传 Codecov(CODECOV_TOKEN 使用仓库密钥), 写法参考 https://github.com/codingapi/flow-engine/blob/main/.github/workflows/ci.yml - 移除 .travis.yml(含明文 CODECOV_TOKEN,已轮换至 GitHub 仓库密钥) - README 徽章:Travis CI 徽章替换为 GitHub Actions Build 徽章, Codecov 徽章指向 17.3.x 分支 Co-Authored-By: Claude <noreply@anthropic.com>
- okxTest 依赖本地代理(127.0.0.1:7890)访问 OKX 外部接口, CI 环境无代理导致 NPE 使构建失败;改为代理缺失/网络受限/接口 限流时通过 JUnit Assumption 跳过,不再作为 CI 强制断言 - actions/checkout、actions/setup-java 升级至 v5(Node 24, 消除 GitHub Actions 的 Node 20 弃用警告) Co-Authored-By: Claude <noreply@anthropic.com>
UserRepositoryTest 保存 user→profile→demo 外键链数据后从不清理, 当测试类执行顺序使 UserRepositoryTest 先于 DemoRepositoryTest/ JdbcQueryTest 运行时(执行顺序随 JDK 构建不同而变化,本地与 CI 不一致),后者的 deleteAll 会因 PROFILE 表残留行触发外键约束冲突, 导致 CI 间歇性失败。为 UserRepositoryTest 增加 @BeforeEach/@AfterEach 按外键顺序(user→profile→demo)清理,使其完全自包含。 Co-Authored-By: Claude <noreply@anthropic.com>
- README 仅保留 17.3.x(JDK 17/Spring Boot 3)与 8.2.x(JDK 8/Spring Boot 2)两条版本线简介, 移除前端框架版本章节,example/frontend 标注为示例工程 - 新增 docs/example-guide.md 示例工程运行手册(启动、登录、API、可选配置、前端) - 修正文档与代码不符的内容:Relation 枚举名系统性错误、FieldChangeEvent→DomainChangeEvent、 SecurityLoginHandler 签名、Token.parseExtra、JoinConditionSQL 构造器、JdbcQuery 方法名、 findAll Example/HQL 行为描述等(含 docs/agents 镜像同步) - 新增 docs/wiki/springboot-starter-script.md,home.md 补齐 script 依赖与链接 - CONTRIBUTING 更新 JDK 要求(17)、分支模型与构建指引 Co-Authored-By: Claude <noreply@anthropic.com>
- FastRepository.findAll 非等值条件被 Example 静默降级为等值匹配: 新增 RequestFilter.isAllEqualFilter(),纯等值走 Example,复杂条件自动切换 HQL - ExampleBuilder 空 catch 吞异常导致过滤条件静默丢失,改为显式抛出 IllegalStateException - DynamicSQLBuilder 空 OR/AND 组合拼出非法 HQL(WHERE AND),改为跳过空片段 - SearchRequest.decode 对非法 Base64 抛 IllegalArgumentException 导致 500,改为按未消费参数处理 - JdbcQuery.countQuery 参数数量相等时不拷贝导致 SQL 参数未绑定 - JdbcQuery.CamelCaseRowMapper 未指定下划线分隔符导致驼峰转换失效 Co-Authored-By: Claude <noreply@anthropic.com>
- 新增/补全五个模块单元测试共约 300 个用例,均以 Java 8 兼容语法编写便于双分支同步: starter(DTO/事件/事务/加密/本地 HttpServer REST 测试)、script(运行时/缓存/控制器)、 security(JWT/Redis 网关/过滤器/加解密)、data-fast(JDBC/动态 SQL/Map 视图)、 data-authorization(JDBC 代理链/列脱敏/SQL 分析) - 覆盖率:starter 51.8%→78.5%、script 64.5%→98.3%、security 68.8%→94.7%、 data-fast 32.9%→89.3%、data-authorization 28.5%→97.1% Co-Authored-By: Claude <noreply@anthropic.com>
GroovyStrategyTest 向全局 GroovyMetadataGenerateStrategyContext 注册的自定义策略 未在结束后清理,测试类执行顺序变化时污染 GroovyScriptControllerTest 的元数据断言。 为两个测试类补充清理逻辑,并将 test.db* 加入 .gitignore 避免 H2 文件库残留。 Co-Authored-By: Claude <noreply@anthropic.com>
- 新增 DDD_CN.md(中文)/ DDD_EN.md(英文),互相提供语言切换链接 - 核心观点:DDD 让业务决定模型,让模型组织软件,让数据与技术服务于业务; 探讨 DDD 在可测试、可复用、可持续三个维度的价值及 Vibe Coding 时代的意义 - README 新增「DDD 观点分享」章节引用全文 Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #210 +/- ##
=============================================
+ Coverage 64.74% 87.99% +23.24%
- Complexity 327 2117 +1790
=============================================
Files 79 210 +131
Lines 1163 5756 +4593
Branches 62 370 +308
=============================================
+ Hits 753 5065 +4312
- Misses 357 567 +210
- Partials 53 124 +71 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe what this PR does / why we need it
Does this pull request fix one issue?
Describe how you did it
Describe how to verify it
Special notes for reviews