Repository navigation
docs(qa): wave-3 checklist accuracy slice S10 — the 18 W3-RT findings at 6e11a929 (#27377) - #28748
Merged
Merged
Conversation
… at 6e11a92 Re-verified each W3-RT (QA run #28656) checklist-accuracy row against source before editing; ten items across five area files, each with revision +1 and one history row. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ihL2gLghEqeDKTbDTC7Zd
Contributor
Author
|
domain:devx#2 · marchtian · session_017ihL2gLghEqeDKTbDTC7Zd 席位核对 — ACCEPT ·
|
This was referenced Nov 1, 2026
Closed
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 田騏 · project thread
Fixes #28701
Part of #27377
Clause-②: no
Metadata-protocol: none
维护者速读
6e11a929)留下的 18 条清单准确性发现,逐条对照当前 main 源码复核后改了平台测试清单:5 个区文件、10 个条目,每个条目 revision +1 并追加一行 history。18 条全部落地修改,没有一条在上游已修好。docs/qa/platform-checklist/areas/studio-authoring.json的expression-editors(删掉一条 knownGap、改写 Studio: an expression the engine refuses (RLS USING on a formula field, a null-unsafe validation predicate) keeps autosaving — the server verdict is shown on the field but never holds the save #27482 那条的事实部分,acceptance[4] 的「no PUT issued」原样保留);同文件dashboard-widget-and-filter-types的 acceptance[4].verify(三种情形分开判);integration-system.json的email-template-renderknownGaps[1](邮件流程配方)。pnpm check:platform-checklist,预期最后一行以check-platform-checklist: OK — 16 areas, 352 items开头。What changes
QA run #28656 (W3-RT, at
6e11a929) parked 18 checklist-accuracy findings on anchor comment 6557904764. Each row was re-verified against source at1c5b1773ecbefore it was edited. All 18 rows are edited, across 10 items in the five area files the card names. None had already landed upstream. Every edited item getsrevision+1 and onehistoryrow citing #28656 and #28701. Main's history stays an exact prefix (checked item by item against a parsed copy). No file outside the five is touched. No clause is weakened.studio-authoring.expression-editorsknownGaps[3]celAuthoring.tslintCelPredicate/rlsPredicateVerdictkeep the rule's error severity;celAuthoring.test.tspins it for USING and CHECK. The 0 PUTs is #28656'sPermissionMatrixEditor.serverVerdictHold-27482.test.tsxcase (b); issue and PR states read from GitHubcd ui && pnpm exec vitest run --maxWorkers=1 packages/app-shell/src/views/metadata-admin/celAuthoring.test.tsui/scripts/vitest-invocation-guard.mjs: the--filter … execform runs in the package directory and gets thepackage-cwdverdictMAX_CEL_SUGGESTIONS = 8. That system fields come first is #28656's readingstudio-authoring.access-owd-and-suggestionsknownGaps[1]SUGGESTION_STATEPackageOwdOverviewPanel.refusalClearsOnRevert-26846.test.tsx;suggested-audience-bindings.tsstudio-authoring.held-edit-leave-guardsteps[7]NO_DRAFTbefore and after Leave is the "unchanged" readingnoPendingDraftErrorinmetadata-protocol/src/protocol.ts;meta-item-absent-404.test.tsstudio-authoring.dashboard-widget-and-filter-typesknownGaps[0]InterfacesPillar.heldListItem-27459.test.tsx. The live pass is #28656'sfilterBindingsfalse for every filterDashboardFilterBar.tsxTextFilter;dashboard-filters.tsresolveBoundField(source only, as W3-RT also says)studio-authoring.summary-field-authoringknownGaps[0] / negative[0]DataPillar.summaryHeld-27461.test.tsx,ObjectFieldInspector.summaryAggregand-27461.test.tsx. The live pass is #28656'sstudio-authoring.app-and-nav-propertiessteps[0]/_console/vite.svg), as a runner-environment noteui/apps/console/public/vite.svg. The proxy block is #28656's readingrecords-forms.field-history-trackingsteps[3]?related=timelineRecordPageColumns.tsxtab row (Overview · Activity · panels). The 1440/1000/700 px reading is #28656'sui/playwright.live.config.ts; the spec's root-relativepage.gotointegration-system.email-template-renderknownGaps[1]type:'api'flow with{triggerType:'api', hookId, secret}, a notify node with no inline title/message, fired as the adminio-node-config.zod.ts(template and inline title/message are mutually exclusive);validateApiTriggerSecret. The 403 rule comes fromflow-start-admission.ts; see Acceptance notesobjectstack compile --outputor--no-restart, with an unchanged listener PIDpackages/cli/src/commands/dev.ts(restartdefaults on),compile.ts--outputapprovals.dynamic-approver-routingsteps[4]outputson the approvals route;outputs.next_reviewersparams on the action doorapprovals-routes.tsreadsbody.outputs;action-execution.tsfolds theoutputs.familyapi-backend.formula-stdlib-matrixacceptance[5] / steps[6]protocol.runtime-authoring-gate.test.ts(publishMetaItemrefused with 422 INVALID_METADATA);rest-route-ledger.tsmeta.publishItemapprovals.dynamic-approver-routing, carried line 6539452860Tests
Docs only: no package source changes, so no build closure and no package test suite is owed (os-dev local-verification scope ① and ②).
skip-changeset: the diff touches onlydocs/qa/platform-checklist/areas/*.json, which no package ships.Acceptance notes
flow-start-admission.tsrefuses a self-triggered type only when it declaresrunAs: 'system', and the spec default isuser. The knownGap therefore says QA run · release-18.3.0-w3-retest (13/13) · 6e11a929 · 2026-11-01 · 11 PASS / 0 PARTIAL / 2 FAIL / 0 BLOCKED / 0 NOT-RUN #28656's autolaunched flow was refused with 403, names the rule, and does not claim that every autolaunched flow is refused. The 400 and 403 statuses are QA run · release-18.3.0-w3-retest (13/13) · 6e11a929 · 2026-11-01 · 11 PASS / 0 PARTIAL / 2 FAIL / 0 BLOCKED / 0 NOT-RUN #28656's readings.FOLLOW-UPS.md(append-only, outside this card's files) still cites it as knownGaps[4].pnpm --filter @object-ui/… exec vitest runinvocations that the ui vitest guard refuses, in items this card does not name. They are left for the seat: studio-authoringobject-designer-roundtrip,record-page-roundtrip,permission-matrix-editor-uxandpackaged-automation-studio-lock, and records-formsimport-wizard-encoding-and-hintsandform-dirty-guard.Verification
Gates (head 4ca7c24, after one merge of origin/main)
node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstackderived 16 families at4ca7c24988: 5 paths, +82 / -39, under the 3000 human-merge line. Each ran with stdin from /dev/null. Exit codes were captured before any output was read:node scripts/check-adr-links.mjs:: exit 0node scripts/check-adr-links.mjs --self-test:: 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 0node scripts/ci/plan-test-core-shards.mjs --self-test:: exit 0pnpm --filter @objectstack/lint run check:doc-formula-expressions:: exit 0, afterturbo run build --filter=@objectstack/formula --filter=@objectstack/lint(lock VERDICT command-exit 0). The first run was a prerequisite refusal (exit 3) that measured nothingpnpm check:ci-filter-parity:: 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 0 (OK — 16 areas, 352 items)pnpm check:refd-timer-probe:: exit 0pnpm check:watch-hint-literal:: exit 0A first sweep before
pnpm installmet prerequisite refusals (exit 3, nothing measured) on six gates. Every family above is from the run after the install.--ranreconciliation: "16 derived famil(ies) accounted for — 16 run, 0 NOT-MEASURED (a DERIVED zero — all 16 recorded an exit code and none of them is 3)" (exit 0).🤖 Generated with Claude Code