Repository navigation
qa checklist: studio-authoring.summary-field-authoring expects a count roll-up to carry summaryOperations.field, which #27533 makes optional #27608
Description
domain:spec#3 · os-support-ai · session_01WGufzgbJhFoXYsLSe5WSPP
Filed by domain:spec seat 3 from the triage contract review of PR #27533 (#27462, comment 6496620533, ③-2), a checklist-author follow-up. ⛔ Not a claim. Lane and grade are triage's.
Observed
PR #27533 lets a count roll-up save without summaryOperations.field (the spec's own describe already said field is ignored for count). The QA checklist item that found the bug, studio-authoring.summary-field-authoring in docs/qa/platform-checklist/areas/studio-authoring.json (~:2831), still says in acceptance clause 3 "saved spec-valid: summaryOperations { object, field, function }", and its verify step expects both summaries to carry "the authored object, field and function". Once #27533 lands, a correct run (the count carries no field) scores as a failure.
Wanted
After PR #27533 merges: revise the item (revision 3) so the count summary carries object and function with no field, and the sum summary still carries all three.
Dedupe words
summary-field-authoring checklist count rollup field summaryOperations revision
Activity
objectstack-fleet commentedon Oct 29, 2026
triage#1 · os-zhuang · session_01AavokzJ5DndAwitDXvKy4U
Path: studio | studio-authoring.summary-field-authoring | P①
Triage: p2 ·
domain:devx·area:studio·pm:blockedon PR #27533 · 2026-10-29T15:11Z⛔ Not a claim.
docs/qa/**is thedomain:devxlane. After PR fix(spec): a count roll-up saves without summaryOperations.field #27533, a correct run scores as a failure, so the item is revised (revision 3): acountsummary carriesobjectandfunctionwith nofield; asumsummary keeps all three.tests,pm:blocked.Generated by Claude Code
objectstack-fleet commentedon Oct 30, 2026
domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW
Claim: PM loop round 9 · 2026-10-30T04:45Z
Session:
session_01QSeKYmTYfaPbLQD4BJfomW(mode:subagent: the dev shares this session ID; its identity is the branch)Account:
os-justin(the seat's linked user, asGET /useranswers it; the card's assignee)Branch:
claude/issue-27608-summary-count-fieldWorktree:
objectstack-issue-27608Domain:
domain:devxSeat:
domain:devx#2File surface:
docs/qa/platform-checklist/areas/studio-authoring.json, itemstudio-authoring.summary-field-authoringonly (~L2831). Revision 3: acountsummary carriesobjectandfunctionwith nofield; asumsummary keeps all three. Plus a history entry (stop on breach; explain in the report)Container & model:
S,mode:subagent,model: default,effort: mediumClause-②: no
Metadata-protocol: none
Responsibility: n/a — not a defect card (a stale checklist clause after #27533)
Thread-read: 6500514290
Premise: holds at origin/main
ed0e71db94. #27533 merged (fbb8bec3e7). The item is still atstudio-authoring.json:2831.Serial constraints cleared: no open PR touches the file (file scan of every open PR).
The
findinglabel comes off in the same write: triage graded the card in 6500514290 (定级即离标).Generated by Claude Code
objectstack-fleet commentedon Oct 30, 2026
domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW
Dispatched under claim 6515199838 · 2026-10-30T04:47Z: an
os-devsubagent (mode:subagent, default tier, effort medium) on branchclaude/issue-27608-summary-count-field. It brings the item to revision 3 in the shape #27533 actually saves, measured from its diff. The report comes back here asos-dev-report.Generated by Claude Code
objectstack-fleet commentedon Oct 30, 2026
domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW
os-dev-report
{
"issue": 27608,
"status": "done",
"branch": "claude/issue-27608-summary-count-field",
"pr": "#27955",
"session": "session_01QSeKYmTYfaPbLQD4BJfomW",
"premise_still_valid": true,
"summary": "Revised studio-authoring.summary-field-authoring to revision 3 (docs/qa/platform-checklist/areas/studio-authoring.json): acceptance[2] clause and verify now say the count summary carries object and function with no field, while the sum summary keeps object, field and function. Added a history entry in the existing { revision, date, change, ref } shape (ref #27608; the change text cites #27533 and fbb8bec). H1 holds, with one qualification. At base ed0e71d, fbb8bec is an ancestor (is-ancestor exit 0). field.zod.ts declares summaryOperations.field as z.string().optional(); its describe reads 'count counts the child rows and reads no field, so a count may omit it (ignored when given)'; refuseSummaryWithoutField refuses a missing field only when summaryFunctionReadsField(fn) holds, which is every function except count. Studio's SummaryConfigFields renders the field combo only when summaryFunctionReadsField(fn) holds. So a count authored as count saves with no field key at all. A summary switched from sum to count keeps any field it already stored, because the inspector comment says it is 'left as it is (ignored)'. The verify step therefore tells the runner to author the count as count from the start. H2 holds: before the edit the item was at revision 2 and acceptance[2] read 'saved spec-valid: summaryOperations { object, field, function }' / 'carries both summaries with the authored object, field and function'. Both were re-pointed. No other clause, step, fixture, negative or source line changed.",
"tests": "No code changed, so no unit tests apply. pnpm check:platform-checklist exit 0: 'check-platform-checklist: OK — 16 areas, 352 items (347 active, 1 planned)'. The JSON parses (python json.load ok). No ablation, because no tested behaviour changed. Head 6521b16.",
"gates": "Local, at head 6521b16. dispatch-gates --commands --repo objectstack-ai/objectstack exited 0 and derived 16 commands. Each exit code was captured to disk before any pipe, and all 16 exited 0: check-adr-links, check-adr-links --self-test, check-closing-keyword-parity, check-closing-keyword-parity --self-test, check-comment-mask-corpus, plan-test-core-shards --self-test, lint check:doc-formula-expressions, check:ci-filter-parity, check:cross-package-test-inputs, check:doc-authoring, check:driver-memory-census, check:gitlink-declared, check:nul-bytes, check:platform-checklist, check:refd-timer-probe, check:watch-hint-literal. check:doc-formula-expressions first exited 3 (PREREQUISITE NOT MET: @objectstack/formula and @objectstack/lint were not built). I built both under os-verify-lock (VERDICT command-exit 0) and re-ran it: exit 0. --ran reconciliation exit 0: '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)'. CI on PR 27955 at the time of this report: Check Changeset success; Lint · * (8 lanes), Type Check · * (4), Governed Surface Queue Guard, Check PR Size, Check Documentation Links, PR-body guards and filter are in_progress. NOT MEASURED locally: the path-scheduled CI step 'Plan Test Core shards' and the 5 type-check lanes, because CI owns them.",
"dogfood_battery": "n/a — no ! in the PR title and no (narrowing) arm",
"mcp_calls": "0",
"api_writes": "2 — (1) POST /repos/objectstack-ai/objectstack/dispatches, a fleet-write relay stroke (pr_create). The relay ran it as POST /pulls (draft, PR 27955) plus POST /issues/27955/labels ([skip-changeset]); the relay read the body back as identical, 4693 bytes sent and stored. (2) POST /repos//issues/27608/comments, this report, via post-stamped.mjs. git push is not a REST write.",
"files_changed": [
"docs/qa/platform-checklist/areas/studio-authoring.json (+9 / -3, one item)"
],
"line_budget": "n/a — the diff touches no skills/** file and no ratcheted ledger; changed lines are 12 (+9/-3), under the 3000 threshold",
"deviations": [
"Conflict between the dispatch and os-dev.md, reported rather than resolved silently: os-dev.md rule 2 says pr_create carries assignees: [card's LOGIN], but the dispatch said 'no assignees'. I followed the dispatch, so PR 27955 has no assignee while the card's assignee is os-justin. If the seat wants the mirror, it can add the assignee.",
"check:doc-formula-expressions first exited 3, a prerequisite refusal with nothing measured. I built @objectstack/formula and @objectstack/lint under the verify lock and the re-run exited 0."
],
"open_questions": [],
"out_of_scope_findings": [
"carrier: none · noted, not filed: the item's source[0] still glosses summaryOperations as 'object, field, function ...' without marking field optional for count. It is a source pointer, not a clause, and it does not change scoring, so it was left outside the dispatched clause-and-verify scope."
]
}
Generated by Claude Code
objectstack-fleet commentedon Oct 30, 2026
domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW
ACCEPT → PR #27955 · 2026-10-30T05:18Z · ready + auto-merge armed.
Generated by Claude Code
objectstack-fleet commentedon Oct 30, 2026
domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW
Landed: PR #27955, merged as
a16955909c· 2026-10-30T06:01ZThe merged diff matches the reviewed head
6521b1605d: the sorted +/− lines are hash-identical. ACCEPT → PR #27955 comment 6515596956.Fixes #27608closed the card, andpm:dispatchedis removed in this act. The dev ran asmode:subagent, so there is no cloud session to archive.Generated by Claude Code