Repository navigation
v18: Studio's flow text slots insert and prescribe {{ }} (the objectui half of objectstack-ai/objectstack#22110) #11824
Description
Blocked-by: objectstack-ai/objectstack#22110
Filed by the triage seat (objectstack-wide, seat post objectstack-ai/objectstack#6015, session_01AavokzJ5DndAwitDXvKy4U) as the foreseen objectui half of objectstack-ai/objectstack#22110. ⛔ Not a claim.
The unlock keys on the install face. This card leaves pm:blocked when objectui's installed @objectstack/* carries #22110's change. That is the v18 line, and objectui stays on ^17 until GA.
Governing text: ADR-0032 (Accepted) Decision 3, "One delimiter, {{ }}; single { } deleted."
What Studio does today (read on objectui main 9990f9e122)
text/textareaflow node fields are treated as single-brace{var}templates (packages/app-shell/src/views/metadata-admin/inspectors/flow-node-config.ts:334–340).- The brace check skips templates, because single-brace is legal there (
FlowExprIssue.tsx:7and:65;FlowNodeConfigField.tsx:481). - Placeholders and help text teach
{var}:flow-node-config.ts:600,:763and:1102, and the zh help strings atviews/metadata-admin/i18n.ts:6287and:6300. VariableTextInput.tsxinserts variable tokens into these fields.
Direction (once #22110 lands)
- In the slots #22110 converts, and only those (its step 1 enumerates them), inserted tokens, placeholders and help text use
{{ x }}. - The template check flags single-brace in those slots and offers the
{{ }}fix. - ⛔ CEL roles keep their brace-in-CEL check unchanged.
Clause-②: no. Changeset per the objectui release line at that time.
Dedupe: studio flow text slot double brace v18 · VariableTextInput {{ }} · objectui half of #22110.
Activity
objectstack-fleet commentedon Oct 9, 2026
Blocked audit: objectstack-ai/objectstack#22110 closed, so the card moves
pm:blocked→pm:on-hold. The hold is now only the install faceTriage seat (objectstack-wide, seat post objectstack-ai/objectstack#6015) ·
session_01AavokzJ5DndAwitDXvKy4U· 2026-10-09T10:05Z. ⛔ Not a claim, ⛔ not a dispatch.What closed: [v18] flow text slots: read ADR-0032 §3's
{{ }}delimiter instead of single-brace{token}(notify title/message and the other flow string slots), converting only what renders the same objectstack#22110 closedcompletedat 2026-10-09T09:41Z. PR feat(spec)!: flow text slots read the {{ }} delimiter, refusing a single-brace token with its hole spelling (#22110) objectstack#22315 merged as2b61f2d9d6.Why on hold, not unblocked: this card's own unlock keys on the install face. The change is on the v18 line only:
latestis 17.7.0, and no 18.x is published;app-shellpins@objectstack/spec^17.6.0.The console still talks to 17.x, where single-brace is accepted, so Studio's
{var}help is not wrong yet. The precedent is objectui#11864.Restart-when: objectui's
@objectstack/specresolves a release that carries2b61f2d9d6.The slots PR feat(spec)!: flow text slots read the {{ }} delimiter, refusing a single-brace token with its hole spelling (#22110) objectstack#22315 converted (its body and skills(objectstack-automation): SKILL.md and eval id 2 still teach single-brace
{token}in notify text slots, which PR #22315 now refuses (Tier H, owed at #22110) objectstack#22454's reading): the notify node'stitle/message, the screen node'stitle/description, and the end node'smessage. These are this card's "slots #22110 converts". Value slots are untouched, and ⛔ CEL roles keep their brace check.objectstack-fleet commentedon Oct 14, 2026
Carried here: a pending changeset that spec 18 makes false, for whoever lands this card's recipient-picker cutover ·
domain:uiseat 1 ·session_01CsyMRsFJBKNEMa4oPY1b5z· 2026-10-14T01:59Z⛔ Not a claim, ⛔ no label change.
6159189541on objectui PR chore(deps): move @objectstack/* to the next prerelease ^18.0.0-next.0 and follow its contract (objectui#12030) #12337 (objectui#12030, the move to@objectstack/*18.0.0-next.0), ③..changeset/11788-notify-recipients-picker.md:8says the recipient picker's{record.FIELD}recipients are "the values the messaging service already resolves". Under spec 18 that spelling is refused, and PR chore(deps): move @objectstack/* to the next prerelease ^18.0.0-next.0 and follow its contract (objectui#12030) #12337's own pending pin says so.{…}spellings at today's behaviour, pinned as PENDING for this card. That is the Studio flow-dialect cutover, ADR-0032 §3, which ruling6148150305Q3 routes here.major, Changesets pre mode onnext, and the two gate edits (the maintainer's ruling on #12030), after #12030 lands #12334 lands, so the sentence cannot ship false either way.Generated by Claude Code
objectstack-fleet commentedon Oct 14, 2026
Restart:
pm:on-hold→pm:queue·domain:uiseat 1 ·session_01CsyMRsFJBKNEMa4oPY1b5zThe held condition is met. objectui#12030 landed as PR #12337 (
mainc5406c2), so objectui now resolves@objectstack/*18.0.0-next.0. The measurement is the dev's held-card table in6151240488on objectui#12030; it was run against thenext.0tarball and the installed tree, with17.7.0as the negative control.6148150305on objectui#12030 routes it here, and it unlocks with this move.{…}spellings and a blank Notify title, which objectui#12030 pinned as PENDING (the Notify title is filed separately as objectui#12343);.changeset/11788-notify-recipients-picker.md:8(6159203238above).The card returns to
pm:queuefor its lane with its grade unchanged. ⛔ Not a claim. Under the maintainer's current p1-only instruction for this seat (6149890064on objectui#5560), it waits in the queue unless it is p1.Generated by Claude Code
objectstack-fleet commentedon Oct 14, 2026
Claim: PM loop round 1
Session:
session_01XVp19E1MYUTFM3dnkwz8gBAccount:
baozhoutao(the seat's linked user asGET /useranswers it; the card's assignee)Branch:
claude/issue-11824-flow-text-slots-double-braceWorktree:
objectui-issue-11824Domain:
domain:uiSeat:
domain:ui#2File surface:
packages/app-shell/src/views/metadata-admin/(the flow node inspectors,FlowExprIssue.tsx,VariableTextInput.tsx,i18n.ts) and.changeset/11788-notify-recipients-picker.md(one-clause correction) (stop on breach; explain in the report)Container & model: M,
mode:subagent,model: opus(default tier;dispatch-gates --tier --repo objectstack-ai/objectuireads no path-derived mandate)Clause-②: no
Responsibility:
n/a — not a defect cardThread-read: 6159865855
Serial constraints cleared: none. No other
domain:uicard is onpm:dispatched, and objectui has no open PR except Dependabot and the release PR #11600 (read 2026-10-14T05:27Z). This seat's batch siblings objectui#11824, #11958 and #12339 have disjoint file surfaces.Generated by Claude Code
objectstack-fleet commentedon Oct 14, 2026
os-dev-report
{ "issue": 11824, "status": "done", "branch": "claude/issue-11824-flow-text-slots-double-brace", "pr": "https://github.com/objectstack-ai/objectui/pull/12355", "session": "session_01XVp19E1MYUTFM3dnkwz8gB", "premise_still_valid": true, "summary": "A1 measured true against the installed @objectstack/spec 18.0.0-next.0: notify title/message, screen title/description and a refused end message refuse {x} and accept {{ x }}. These five flow text slots are now refMode 'hole': the picker inserts {{ x }}, the en+zh placeholders and help say {{ }}, a single brace is flagged with the double-brace fix, and the unknown-ref note reads hole paths with formatters ignored. mergeServerFlowFields carries hole mode onto the served screen title/description, which the descriptor publishes as plain strings. The screen preview renders {{ }} holes (interpolateHoles); the dead single-brace interpolate was removed. A3 was DISPROVED: recipients is a value slot, and spec 18 refuses {{ record.owner }} there too, prescribing { dialect: 'cel', source: 'record.owner' }. So the picker's cutover follows the ruling's intent in the contract's spelling: the Record field row writes that envelope, a new Expression row writes any other CEL envelope (bare-ref picker, CEL brace check), and Other is literal text. A stored {record.FIELD} string is kept as typed under Other, because the spec ships no conversion. The PENDING pins are flipped to load-bearing ones. The changeset 11788 clause is corrected (frontmatter unchanged), and the new changeset 11824 (app-shell minor) is added. CEL roles and every other text field are unchanged and pinned.", "tests": "head a847d3a (the tree run = the commit). Targeted vitest from repo root: the 4 touched files gave 73 passed; the 36 app-shell test files importing a touched module gave 555 passed / 1 skipped; FlowNodeInspector*+FlowRunner* (32 files) gave 352 passed; every lock VERDICT command-exit 0. The new pin file flowTextSlots.doubleBrace-11824.test.tsx asserts the installed-spec refuse/accept per slot, hole set == the five and a subset of the spec's FLOW_NODE_TEXT_SLOTS, picker insert {{ customer }} per slot, served hole carry, the single-brace alert with the fix, a clean formatter hole, the CEL alert equal to validateExpressionClient('predicate'), the http body still {x}, and interpolateHoles. Gates, all EXIT=0: check:phantom-deps, self-import, unreferenced-sources, doc-example-readers, handler-key-reads, side-effects-array, element-data-source-declaration, spec-symbols, action-forward-parity, designer-field-key-parity, icon-record-names, i18n-keys, i18n-drift, i18n-designer-parity, i18n-dead-keys, new-line-citations, control-bytes, changeset-claims, pending-changeset-literals, test-path-roots, vi-mock-specifiers/inherit/override-shape, installed-pin-claims, plus scripts/check-changeset-fixed.mjs, check-changeset-presence.mjs and check-changeset-no-major.mjs. check:spec-floors EXIT=1 ('The scan collapsed: 0 package(s) declare @objectstack/spec'); its inputs (package.json, changeset config) are untouched by the diff (git diff --name-only BASE..HEAD has 0 such files), so it is pre-existing. eslint narrowed to the 12 changed ts/tsx files (--no-inline-config --format json): 12 files, 0 errors, 10 warnings (react-refresh/only-export-components on files that already exported helpers); eslint.config.js sets no parserOptions project, so type-aware lint is off and untouched files' verdicts cannot move. NOT MEASURED: full pnpm --filter @object-ui/app-shell type-check. Reason: two 540s waits on the verify lock ended in queue-timeout (exit 99), behind holder pid 10691 (objectui-issue-11958 vitest run, held over 33 min). Declared narrowing: tsc -p over a temp tsconfig extending tsconfig.test.json with files = the 12 changed files reached 57 app-shell sources, EXIT=0 with 0 errors; the full type-check is left to CI. The dependency closure was built first (turbo build --filter='@object-ui/app-shell^...': 28/28 successful).", "mcp_calls": "0", "api_writes": "3 planned: git push of the branch (2 pushes: the empty probe + a847d3a); 1 pr_create via the fleet-write relay (POST /repos/objectstack-ai/objectui/pulls draft, plus POST /repos/objectstack-ai/objectui/issues/12355/assignees), with the read-back reporting the stored body identical; 1 os-dev-report comment via the relay (POST /repos/objectstack-ai/objectui/issues/11824/comments). 0 label writes: objectui has no gate reading a dev-applied label, and the dispatch named none.", "open_questions": [], "out_of_scope_findings": [ "class: b · reach: NotifyConfigSchema refuses config.actionUrl '/x/{record.id}', HttpConfigSchema refuses url 'https://a/{record.id}' and headers {'X-A':'{x}'} (measured on installed 18.0.0-next.0), while Studio's notify Click-through URL / http URL fields and the key/value header cells default to the single-brace picker. Seam: spec:FLOW_NODE_TEXT_SLOTS (notify actionUrl, http url, http headers) → renderer:flow-node-config notify.actionUrl / http_request.url + FlowKeyValueField template cell. The ruling limited this card to the five #22110 slots, so not changed here · dedupe: flow text slot actionUrl double brace · http url headers {{ }} Studio · FLOW_NODE_TEXT_SLOTS studio", "class: c · reach: ScriptConfigSchema refuses inputs {a:'{x}'} and ScreenConfigSchema refuses defaults {account:'{account_id}'} (value slots, measured), while Studio's script Inputs help ('{var} references resolve…') and screen Form defaults help ('account → {account_id}') prescribe the refused spelling, and the key/value template cells insert {x}. Named producer: the flow-node-config script.inputs / screen.defaults descriptors and their zh overlay. Same family as the finding above: the remainder of Studio's ADR-0032 flow-dialect cutover; report for one closure card, not single-point cards · dedupe: studio value slot {var} help · key value cell CEL envelope · flow dialect cutover remainder", "carrier: PM · noted, not filed: verify-lock holder pid 10691 (objectui-issue-11958, a vitest run over packages/plugin-form packages/plugin-detail packages/components and many RecordDetailView files) held /tmp/os-heavy-verify.lock for over 33 min, so this run's full type-check never acquired it", "carrier: PM · noted, not filed: check:spec-floors exits 1 on this tree with 'The scan collapsed: 0 package(s) declare @objectstack/spec in a consumer-facing field', with manifests identical to base b9814b3; likely the 18.0.0-next.0 move, owner unknown (承接者:无)" ] }Generated by Claude Code
objectstack-fleet commentedon Oct 14, 2026
Review: ACCEPT — objectui PR #12355 ·
domain:uiseat 2 ·session_01XVp19E1MYUTFM3dnkwz8gB· 2026-10-14T07:26ZReviewed on GitHub at head
a847d3a65f, against the card and ruling, not against the dev's report.main, and its body opensFixes #11824(the only closing keyword).Clause-②: nomatches the claim. The governed sweep (check-governed-merges --pr objectstack-ai/objectui#12355) reads NOT governed, 627 lines.refMode: 'hole'sits on exactly notifytitle/message, screentitle/description, and endmessage.mergeServerFlowFieldscarries it onto a served field of the same key. The CEL predicate check is unchanged (refMode === 'expression'still gates the condition builder), and other text fields keeptemplate.{{ record.owner }}in therecipientsvalue slot too, so the picker writes the CEL envelope{ dialect: 'cel', source: 'record.FIELD' }the spec prescribes. An edited row drops the envelope it was read from (patchRowsetsstored: undefined). A stored{record.FIELD}string stays under Other unchanged. That follows ruling Q3's intent in the contract's spelling. Correction accepted: the PM's A3 was wrong..changeset/11788-notify-recipients-picker.md:8now names the CEL envelope in place of{record.FIELD}, frontmatter unchanged, as carried by6159203238..changeset/11824-flow-text-slots-double-brace.md(@object-ui/app-shell: minor): every bullet matches the diff.exportsmap is only.and./styles.css, and none of the touched modules is re-exported fromsrc/index.ts.Spec Main Shape Gate(advisory), whose 21 annotations are onresolveDataSource.ts(6),useViewData.ts(4) and.github(11). That is the same signature asmain's run38513810543, so it is the base's red (objectui#12325, in flight), not this PR's.tsc -pover the changed files. CI'sType Checkis green on this head, which covers it.inputsand screendefaultshelp still prescribe{x}, which spec 18 refuses) are one family: the rest of Studio's ADR-0032 flow-dialect cutover. Filed as one closure card; the link follows in this seat's next comment on this card. The verify-lock andcheck:spec-floorsnotes are not filed:check:spec-floorsis carried by objectui#12334, and the lock note is a session observation.Next:
pr_ready+automerge_enable.Generated by Claude Code
objectstack-fleet commentedon Oct 14, 2026
Landed ·
domain:uiseat 2 ·session_01XVp19E1MYUTFM3dnkwz8gB· 2026-10-14T07:41Zmaincfb2947. The card closedcompletedby itsFixes #11824, which is the PR body's only closing keyword.origin/main, these files are blob-identical to the accepted heada847d3a65f:flow-node-config.tsFlowRecipientsField.tsx.changeset/11824-flow-text-slots-double-brace.mdpm:dispatchedis removed in the same stroke as this comment.6163229107).Generated by Claude Code