Repository navigation
docs(qa): access-security ledger edits from the 18.2.0 sweep (#25835) - #26101
Merged
Merged
Conversation
crud-permission-matrix, walled-environment-metadata-operator-only, activation-write-operator-gate, anonymous-deny-surfaces and anonymous-floor-v18-doors re-read on main and revised. Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GgKuGD9Ats95jBbJRZrf4a
The remaining 22 items of #25835 re-read on main and revised: automated.ref now names what each pin asserts and at which layer, the no-active-org producers follow #25626, and the share-link landing page drops the retired objectui pin. Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GgKuGD9Ats95jBbJRZrf4a
Contributor
Author
席位核对 — ACCEPT at
|
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
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.
Requested by Elon · project thread
Fixes #25835
Wave anchor: #25807 (18.2.0 release test sweep). Docs only: one file,
docs/qa/platform-checklist/areas/access-security.json.What changed
All 65 lines of the card were re-read against
mainatb4199c96before editing (the records measured6679644a, pinb41ed7f68956; the pin is retired, #25745). 62 lines are applied, 3 are answered on the card instead (below). 24 items are revised, plus one more for the retired-pin text, 25 in all. Each revised item has itsrevisionbumped once and one newhistoryentry that cites the record(s) its lines came from (#25812, #25815, #25817, #25823) and says which lines did not hold. ⛔ No item status changed and no clause's assertion changed: almost every line was the ledger overclaiming what a pin asserts, or not naming the layer a pin reaches (RUNNER rule 6). So most edits are toautomated.ref, which now says per clause what each pin asserts, at which layer, and which legs stay hand-driven.sys_userdirectory (sys_user_org_members), not a self-scoped row. The ref no longer readsobjectstack verify --rlsas a pin. It no longer claims clauses 3 and 6 whole, and it says the zoo's out-of-subtree case asserts only a non-2xx. acceptance[6].verify says which 164 cells the pin judges.manage_metadataholder, not the seeded admin.automation-toggle-deny-message.test.ts,automation-write-capability-gate.test.tsandpackaged-activation-ledger-reach.dogfood.test.tswith their layers and personas, and names the unpinned re-hydration leg..not.toBe(401), there is no forms contrast, and there is no post-DELETE read-back.EVERYTHING_ELSEnever readserror.code, the share-link class accepts 401, the docs pin does not compare against the control path, and the 201 is pinned only over a protocol double.authz-identity-objects-boot-refusal.test.tsleft the ref (it pins something else). acceptance[1] and acceptance[2] are named live-run.current-user-endpoints-localization.test.tsjoins the ref. STILL MANUAL names the admin half of /me/apps and the anonymous trio on a real stack.admin_full_accessis exercised.ADMIN_STANDING_NON_TABLE_INPUTSis asserted by no test. The a0–a3 line confirmed the ledger.explain()in-process underisSystem, and that no deny-side explain is pinned.marketplace-install-local-catalog-name-collision.test.tsjoins the ref. The ref names the registerApp layer, the in-process CLI, and the unasserted 50-name cap.share-links-refusal-order.test.tsjoins the ref. Both runtime pins are named in-process, so they score acceptance[0] only.capabilities.N.scope(.2when appended). Themixedline was answered, not applied..objectui-sha. It namespnpm console:buildintopackages/console/dist(whichpnpm builddoes not refresh) andui/apps/console/src/pages/SharedRecordPage.tsx. History line cites #26035.Answered on the card, not edited (with reasons in the card report):
mixedwithout a browser oracle. In this ledgermixedmeans more than one execution surface: 31 other activemixeditems have no screenshot or dom oracle, and this item is driven over the API and the build / CLI.Two lines held only in part on this tree, and the history says so. activation a0/n2:
FLOW_ENABLEMENT_DENY_MESSAGE's sentence is asserted, by value, inautomation-toggle-deny-message.test.ts. anonymous-floor a1:UNAUTHENTICATEDis asserted on other faces of /notifications, /keys and /share-links, but not on the three GET faces the step lists.No line described a product defect, so nothing was filed.
Verified
Gate families derived by
node scripts/pm/dispatch-gates.mjs --commandson head9738ffad(one path vs merge baseb4199c968):node scripts/check-ci-filter-parity.mjs— exit 0node scripts/check-closing-keyword-parity.mjs— exit 0node scripts/check-closing-keyword-parity.mjs --self-test— exit 0node scripts/check-comment-mask-corpus.mjs— exit 0pnpm --filter @objectstack/lint run check:doc-formula-expressions— exit 0pnpm check:cross-package-test-inputs— exit 0pnpm check:doc-authoring— exit 0pnpm check:driver-memory-census— exit 0pnpm check:gitlink-declared— exit 0pnpm check:nul-bytes— exit 0pnpm check:platform-checklist— exit 0pnpm check:refd-timer-probe— exit 0pnpm check:watch-hint-literal— exit 0check:doc-formula-expressionsfirst exited 3 (PREREQUISITE NOT MET) in the fresh worktree; afterpnpm --filter '@objectstack/lint...' build(lock VERDICT command-exit 0) it exits 0.node scripts/pm/dispatch-gates.mjs --ran: 13 derived, 13 run, 0 NOT-MEASURED.node scripts/pm/check-governed-merges.mjs --test docs/qa/platform-checklist/areas/access-security.json: "NOT governed — ordinary queue landing applies". No contract review is triggered, and this is not Tier H.Diff: 1 file, 207 insertions and 57 deletions, so no split is needed. Edits were made as text replacements inside each item, so the file's own formatting is kept (a JSON re-serialise would have rewritten 22 one-line history rows).
Acceptance notes (found beside the lines, outside this card's surface)
packages/qa/dogfood/test/showcase-crud-persona-matrix.dogfood.test.ts: the header comment still says "6showcase_field_ops_delegate×sys_*rows", while its own assertion says 4. This is test-file text and needs no card. Carrier: none.groupre-run before the line drops, so they are left for that run.identity-auth.workspace-org-switchpersonas[2] and RUNNER.md's "Walled personas come from the admin door"membershipCreated: falsesentence are stale too (QA run · identity-auth.admin-lifecycle-operations (1/1) · 17f8217b · 2026-10-25 · 1 PASS / 0 PARTIAL / 0 FAIL / 0 BLOCKED / 0 NOT-RUN #25823). They belong to the identity-auth card and the RUNNER card (qa(checklist): RUNNER.md — 3 stale environment facts from the 18.2.0 release sweep (#25807) #25850).automated.refpaths under objectui "objectui-repo-only", whileui/is now in this repo. share-link-landing-page knownGaps[0] and its ref stay in step with RUNNER until qa(checklist): RUNNER.md — 3 stale environment facts from the 18.2.0 release sweep (#25807) #25850 changes it.skip-changeset: docs only, nothing published.Generated by Claude Code