[objectstack-fleet]

Requested by Warren · project thread
Fixes #27095

Clause-②: no
Metadata-protocol: none

维护者速读

• 改了什么:给两个新测试(Studio Access pillar: the permission matrix draws some object grants differently from what the runtime enforces (detail withheld pending maintainer) #27010、Studio Access: saving a permission set drops draft values written by another door (description, packageId) — the save starts from the published body and copies back only editor-managed keys #26837)的假客户端补上 withPreviewDrafts,只改测试,不改产品代码。
• 为什么改:Studio Access: the permission matrix lists no objects while the package's objects are still drafts ("0 objects"), so a new package cannot be secured before its first publish #26840 让包内访问矩阵经草稿叠加读取对象,这两个测试与它交错合并,假客户端缺该方法,main 的 Test UI 因此变红。
• 风险与回滚:两行测试替身改动,零产品风险;回滚即 revert 本 PR。
• 先看:PermissionMatrixEditor.objectGrantInherited-27010.test.tsx 与 PermissionMatrixEditor.packageDoorDraftBase-26837.test.tsx 的 makeClient。
• 两分钟验证:pnpm --dir ui exec vitest run packages/app-shell/src/views/metadata-admin/PermissionMatrixEditor.objectGrantInherited-27010 packages/app-shell/src/views/metadata-admin/PermissionMatrixEditor.packageDoorDraftBase-26837,预期全绿。
• 需要你做的:无,席位落地。

What changes

withPreviewDrafts() { return this; } added to the fake client of the two package-door matrix tests that merged beside #26840 (PR #27064): #27010's objectGrantInherited-27010 and #26837's packageDoorDraftBase-26837. Same addition as the other matrix doubles. No product code changes.

Why

Main went red on Test UI (shards 11 and 12) after queue merges crossed: #27064 made the package door read objects through MetadataClient.withPreviewDrafts(true), and these two new tests' fakes did not have that method.

Tests

• Red leg (the two files as on main): Test Files 2 failed, Tests 4 failed.
• Green leg: those two files plus draftObjects-26840: Test Files 3 passed, Tests 28 passed.
• Sweep: every app-shell test that mounts the matrix (74 files). They pass at 76e90df6 with the Studio Access pillar: the permission matrix draws some object grants differently from what the runtime enforces (detail withheld pending maintainer) #27010 fix (714 tests). The other 36 files without withPreviewDrafts never mount the matrix under a packageId, so they never call it.

Verification

Derived gates are in the os-dev report on #27095.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JQwxbYLTHkXTxBf89Sud8r

---

Generated by Claude Code

[objectstack-fleet]

席位核对

ACCEPT · seat domain:ui#1 · session_01JQwxbZeBv2y91towxasSUT · head 039a8750 · 2026-10-28T06:04Z

席位意见：修 main 红。#27064 和 #27080 在队列里与 #27010 交叉合入，两个测试的假 client 缺 withPreviewDrafts；本 PR 只给这两个假 client 各补一行，产品代码不动。

1. Shape: Fixes #27095; 2 test files, +4/−0. Each fake gains withPreviewDrafts() { return this; } with a one-line citation. Labelled skip-changeset.
2. Premise measured: on main 76e90df6 the Studio Access pillar: the permission matrix draws some object grants differently from what the runtime enforces (detail withheld pending maintainer) #27010 file's package-door case failed at L103 (check run 122646737436), and on dc87dfb5 all 3 tests of the Studio Access: saving a permission set drops draft values written by another door (description, packageId) — the save starts from the published body and copies back only editor-managed keys #26837 file failed on shard 12 (run 40637892600). Both fakes lacked the method that the package-door load now calls. Returning this keeps each fake's list / get answering as before, which is what those tests assert against. Dev: red leg 4 failing on main, green leg 28/28 with draftObjects-26840; sweep of 74 matrix-mounting files (714 tests) green. The other 36 fakes without the method never mount the matrix under a packageId.
3. Gates / check-runs: PR check-runs on 039a8750: 43 success, 16 skipped, 0 failure; mergeable_state clean. The dev's derived gates and card report follow; the seat lands now because main is red.
4. Accepted deviations: the branch was recreated from origin/main with a plain push (the old remote branch had been deleted after fix(app-shell): list a package's draft objects in the Access matrix, badged as drafts #27064 merged), so no force was used.
5. Out of scope: the merge queue merged fix(app-shell): list a package's draft objects in the Access matrix, badged as drafts #27064 and fix(app-shell): a package-door permission save keeps the pending draft's values #27080 although their queue builds failed Test UI; reported to the coordinator for the ci lane. Skills-docs: none — test doubles only.
6. Landing: not Tier H; the seat readies and arms auto-merge.

---

Generated by Claude Code