Repository navigation
refactor: Maven group 与 Kotlin 根包改为 app.tinyui - #12
Conversation
仓库已进 tiny-ui 组织,坐标也脱离个人命名空间:group `wang.harlon` → `app.tinyui`(反转域名 tinyui.app,Central 已按域名验证),Kotlin 包 `wang.harlon.tinyui` → `app.tinyui`,目录 git mv 保留历史。sample 的 applicationId / bundle id 跟着改成 `app.tinyui.sample`。 同步改:gradle.properties GROUP、composite-substitutions 契约、schema 生成脚本的 --package 与 generate.ts 硬编码 import、API dump、文档坐标。quickjs-kmp 仍是 `wang.harlon:quickjs-kmp`。 `wang.harlon:tinyui:0.1.0` 留在 Central 不再更新,下一版从 0.2.0 起走新坐标。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Reviewer's Guide本 PR 将 TinyUI 的 Maven group 和 Kotlin 根包从 File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've reviewed your changes and they look great!
Sourcery assessment
Needs a human reviewer. Changing the Maven coordinate and every public Kotlin package breaks existing consumers, while changing the Android applicationId and iOS bundle identifier makes the released app appear as a different application and can strand existing installed data and upgrade paths. Reverting the source does not retract a published app or Maven artifact, nor automatically repair consumers that adopted the new identities.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe pull request renames the Kotlin namespace and Maven coordinates from ChangesNamespace and artifact migration
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Refactor 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 30 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
改了什么
仓库已进
tiny-ui组织,坐标一并脱离个人命名空间:wang.harlon→app.tinyui(反转域名 tinyui.app;Central 命名空间已按域名验证)wang.harlon.tinyui→app.tinyui,8 组目录git mv保留历史applicationId/ iOS bundle id →app.tinyui.samplegradle.propertiesGROUP、gradle/composite-substitutions契约、pnpm schema的--package与generate.ts硬编码 import、API dump、文档命名表加app.tinyui一行quickjs-kmp仍是wang.harlon:quickjs-kmp(通用库,留个人命名空间)。wang.harlon:tinyui:0.1.0留在 Central 不再更新,下一版从0.2.0起走新坐标,TrendingAI 届时换坐标。验证
./gradlew build通过(compose 双端测试、sample APK / iOS framework)pnpm build && pnpm test、pnpm schema:check零 diffpublishToMavenLocal -PVERSION_NAME=0.2.0:~/.m2/app/tinyui/下 AAR、两个 iOS klib、pom/module 齐全,pom scm 指向tiny-ui/tinyui,依赖wang.harlon:quickjs-kmp:0.1.1🤖 Generated with Claude Code
Summary by Sourcery
Adopt
app.tinyuias the TinyUI Maven group and Kotlin namespace across the library, samples, tooling, and documentation.Enhancements:
app.tinyuiwhile retaining the existing QuickJS dependency coordinates.Build:
app.tinyuigroup.Documentation:
Chores:
wang.harlon:quickjs-kmpdependency coordinate while transitioning TinyUI to the new namespace.Summary by CodeRabbit
Breaking Changes
wang.harlon.tinyuitoapp.tinyui; update imports and references in consuming applications.app.tinyui:tinyuiandapp.tinyui:tinyui-updates.Documentation
Samples
app.tinyuinamespace.