CI/CD ???? + ??????? - #3
Merged
Merged
Conversation
介绍四窗口浮动架构（A/B/C/D），其中C/D窗口生命周期解耦，并改进了定位与测量。修复内容：窗口C默认高度减小，窗口D延迟至数据填充后显示，移除药丸式徽章，卡片透明度传播，以及更安全的长按处理。 UI与设置：通过UIConfig/AppConfig/SettingsService和SettingsPage暴露可配置的长按时长（300–3000ms）；CTA按钮现在能检测到缺失模型并显示禁用标签。HomePage的模型检查使用ProviderStorage。 主题与序列化：ThemeManager导入现在返回ImportResult，包含验证、注释去除、名称/ID冲突检查；预览/颜色转换已修复，以避免色彩空间问题。 文档与资源：更新了README/CHANGELOG。
- Job拆分: lint(detekt) → test(单元测试+Kover) → build(APK) → release(GitHub Release) - lint job: detekt + Android lint + 报告上传 - test job: 单元测试 + dorny/test-reporter PR显示结果 + Kover覆盖率 - build job: 仅编译APK + 签名 + 上传artifact - release job: 直接从build下载release-apk, 消除6个重复步骤 - 使用YAML锚点 &setup/&signing 消除重复配置 - 添加 checks:write / pull-requests:write 权限用于test-reporter
Townrain
pushed a commit
that referenced
this pull request
Jul 29, 2026
PR #3 commit a034ecb 把 runner 从 Robolectric 换回 JUnit4，导致 LayoutParams 真实构造函数抛 RuntimeException("Stub!") 崩溃，29 个测试挂；CI 项目前用 continue-on-error: true 掩盖。恢复 RobolectricTestRunner + Config sdk=30。 另外 2 个 assertion 写错了：Window B 期望 0x0 实际是估算面板尺寸 464x96；Window C 期望 height 0 实际是 60dp*density。修正后两个也匹配代码真实返回值。 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Townrain
pushed a commit
that referenced
this pull request
Jul 29, 2026
PR #3 多塞进来一个 FloatingWindowContent mega-Composable（722 行起 line 350-722），grep 全仓库零引用 — 没有任何文件 import 它。真正在用的是拆分好的 WindowAContent/WindowBContent/WindowCContent/WindowDContent 4 个小 Composable。这段老的单窗口时代遗留逻辑跟拆分后的新组件重复一遍。整个删掉 app 不受影响，文件从 722 行缩到 326 行。 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Townrain
pushed a commit
that referenced
this pull request
Jul 29, 2026
PR #3 描述说新增 detekt 静态检查，但代码里 libs.versions.toml 没声明 detekt 版本、build.gradle.kts 也没 apply 插件，./gradlew detekt 永远报 Task not found；靠 continue-on-error:true 把错声盖掉当装饰品。直接删掉这个死步骤。 dorny/test-reporter 同时被两层棉花塞住：step 级 continue-on-error:true + action 级 fail-on-error:false。test-reporter 永远不会亮红，CI 永远绿。删 step 级 continue-on-error，action 级 fail-on-error 改 true，让 PR 测试失败时 reporter 真能在 PR 评论区亮红。 注：test step 自己的 continue-on-error:true 暂留 — 因为本地无 Android SDK 不能验证 55 个测试全绿，等下次 push CI 跑过再清理这一行。 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Townrain
pushed a commit
that referenced
this pull request
Jul 29, 2026
PR #3 commit a034ecb 把 runner 从 Robolectric 换回 JUnit4，导致 LayoutParams 真实构造函数抛 RuntimeException("Stub!") 崩溃，29 个测试挂；CI 项目前用 continue-on-error: true 掩盖。恢复 RobolectricTestRunner + Config sdk=30。 另外 2 个 assertion 写错了：Window B 期望 0x0 实际是估算面板尺寸 464x96；Window C 期望 height 0 实际是 60dp*density。修正后两个也匹配代码真实返回值。 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Townrain
pushed a commit
that referenced
this pull request
Jul 29, 2026
PR #3 多塞进来一个 FloatingWindowContent mega-Composable（722 行起 line 350-722），grep 全仓库零引用 — 没有任何文件 import 它。真正在用的是拆分好的 WindowAContent/WindowBContent/WindowCContent/WindowDContent 4 个小 Composable。这段老的单窗口时代遗留逻辑跟拆分后的新组件重复一遍。整个删掉 app 不受影响，文件从 722 行缩到 326 行。 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Townrain
pushed a commit
that referenced
this pull request
Jul 29, 2026
PR #3 描述说新增 detekt 静态检查，但代码里 libs.versions.toml 没声明 detekt 版本、build.gradle.kts 也没 apply 插件，./gradlew detekt 永远报 Task not found；靠 continue-on-error:true 把错声盖掉当装饰品。直接删掉这个死步骤。 dorny/test-reporter 同时被两层棉花塞住：step 级 continue-on-error:true + action 级 fail-on-error:false。test-reporter 永远不会亮红，CI 永远绿。删 step 级 continue-on-error，action 级 fail-on-error 改 true，让 PR 测试失败时 reporter 真能在 PR 评论区亮红。 注：test step 自己的 continue-on-error:true 暂留 — 因为本地无 Android SDK 不能验证 55 个测试全绿，等下次 push CI 跑过再清理这一行。 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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
2 participants
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.
????
? CI/CD ?????????,??????????,? Run #6 ????? Run #13 ?????
Phase 1 - ????(? CI ??)
Phase 2 - ???
????
????