Skip to content

ref(core): 统一应用与 Peer 角色命名 - #181

Merged
xiaoland merged 2 commits into
mainfrom
feat/app-peer-naming
Nov 5, 2026
Merged

xiaoland merged 2 commits into
mainfrom
feat/app-peer-naming

Conversation

@xiaoland

@xiaoland xiaoland commented Nov 4, 2026 •

Copy link
Copy Markdown
Member

将 Python 中代表 InKCre 应用与运行参与者的 Client 命名统一为 App / Peer,明确 Web 应用在异构 Peer 系统中的角色。

范围与迁移

  • 开发种子的 Peer 展示名改为 web-development,生产 profile 的展示名改为 web。既有 Peer UUID 继续承担身份。
  • 部署、安全、邮件和观测文档采用 App / Peer,Memos 的 Web 操作入口采用 Peers → Config。
  • HTTP、OAuth、MCP、IMAP、数据库和第三方 SDK 使用各自协议中的 client 术语。历史 migration、changelog 和旧 CLIENT_* 清理逻辑继续服务于已有版本的兼容。
  • inkcre_core/ 纳入 Core 发布归属,SDK 单独变更通过既有 fragment 检查记录发布意图。

旧开发库可通过正常 db:init --profile development 或 pdm run db:seed-dev 更新规范种子的展示名。生产 profile 的新默认值供后续连接配置使用,已保存的 Peer 名称由部署拥有者管理。

发布项目为 inkcre-core,Towncrier fragment 已记录。Memos 的变动属于操作文档。

合并基线与验证

当前候选 49b2dce 基于 main fd2b92d,父 PR #175 和 #178 已按顺序合并。git range-diff 确认命名源码与原候选一致,共享引用继承 main 中的 Hub 17fc19e7230187a7a989815e7b8303bbb380f478。

与原审阅提交 4314c2f 比较,最终树额外包含主线 #180 的 ChatGPT 接入文件和最终共享引用。对应 launcher、部署配置及新主线文档已经核对。

本轮真实 check:releases --base fd2b92d、共享引用检查及 git diff --check 通过。当前 head 的云端 required checks 是最终合并依据。

历史局部集合为 17 passed、1 skipped,取得于移除私有映射测试之前。后续采用真实发布意图命令验证 SDK 归属,并完成类型检查及 Ruff。用户要求将本地验证限定为静态和局部检查,端到端验证由云端执行。

Kotlin 的只读检查发现 35 处 client 均属于 OkHttp、OSS 等协议或 SDK 语义,应用目录已使用 app。

交付与回滚

用户已授权本组 PR 合并。当前执行者复核了命名残留、身份兼容、发布归属和重排内容,待 required checks 满足后通过普通 squash 合并。整组完成后,指定 Dogfood 聊天按 main 更新正式自部署。

回滚可整体撤销本 PR。开发种子默认展示名随正常 seed 操作收敛。

本 PR 直接基于 main,已合并父 PR 为 #178,祖先为 #175。跨仓跟踪为 InKCre/docs#55,共享命名合同已通过 InKCre/docs#56 合入 Hub。

- Web 默认展示名使用 web,保留 Peer UUID 和历史兼容标识
- 当前部署与扩展文档使用 App 和 Peer 描述产品角色
- 补充独立 SDK 源码的 Core 发布归属及回归验证
- 保留真实 check:releases 作为 SDK 发布归属修复的验证依据
@xiaoland
xiaoland force-pushed the feat/app-peer-naming branch from 4314c2f to 49b2dce Compare November 5, 2026 09:55
@xiaoland
xiaoland changed the base branch from feat/observability-unification to main November 5, 2026 09:55
@xiaoland
xiaoland marked this pull request as ready for review November 5, 2026 09:56
@xiaoland
xiaoland merged commit 7f56f7c into main Nov 5, 2026
7 of 8 checks passed
@github-project-automation github-project-automation Bot moved this from Review / acceptance to Done in InKCre Current Work Nov 5, 2026

This branch was successfully deployed

1 active deployment
preview — 49b2dce9 Deployed Nov 5, 2026 by xiaoland via impl-py preview cleanup #559
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant