Skip to content

fix 🐛: 兼容历史稳定资源 ID - #1

Merged
snowykami merged 2 commits into
mainfrom
codex/fix-legacy-resource-ids
Sep 14, 2026
Merged

snowykami merged 2 commits into
mainfrom
codex/fix-legacy-resource-ids

Conversation

@snowykami

@snowykami snowykami commented Sep 14, 2026 •

Copy link
Copy Markdown
Member

根因

CLI 只接受当前 24 位十六进制资源 ID,历史版本已持久化并仍被平台使用的稳定 ID 会在本地校验阶段被拒绝,用户无法操作这些资源。

此外,仓库内置 OpenAPI 快照已落后当前 luna-devops/main,导致任意 CLI PR 都会在快照一致性门禁失败。

修复

  • 精确接受当前 project/app/target/release ID。
  • 仅兼容平台真实生成过的历史 project、app、target 与 system target 格式。
  • release 仍只接受当前真实格式,不引入无法解析的猜测规则。
  • 更新错误提示与覆盖正常、历史、system、畸形输入及 agent 模式测试。
  • 机械同步当前已发布的 luna-devops/main OpenAPI 快照及生成的 schema/operation 目录;不包含尚未合并的平台 PR 契约。

合并顺序

先合并平台 PR https://github.com/LiteyukiStudio/luna-devops/pull/18。平台合并后,本 PR 再同步一次最新平台 main 契约并等待 CI 通过,最后合并本 PR,避免 CLI main 再次产生快照漂移。

验证

  • pnpm check
  • pnpm check:release-scripts
  • pnpm check:contract
  • pnpm check:skills
  • LUNA_PLATFORM_ROOT=/Users/sfkm/Developer/liteyuki-studio/devops pnpm check:platform-coverage
  • git diff --check

以上均通过。

@coderabbitai

coderabbitai Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: d520cdab-faff-49af-beaa-5026f71e4c78


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.

@snowykami
snowykami merged commit 617c257 into main Sep 14, 2026
2 checks passed
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.

1 participant