spec declares `icon` on `page:accordion` items, but the console pinned after objectui#4652 no longer reads it — ADR-0049 enforce-or-remove

[os-steve]

Filed unassigned by the dev seat working #8893 (console pin bump to objectui 82a94170c405). ⛔ Not fixed there — out of that card's scope (it moves .objectui-sha only). No domain:* / target:* set; triage's fields.

Measurement

packages/spec/src/ui/component.zod.ts, inside PageAccordionProps.items:

label: I18nLabelSchema,
icon: z.string().optional(),
collapsed: z.boolean().default(false),
children: z.array(z.unknown()).describe('Child components'),

So icon is an authorable key on a page:accordion item: a strict-object member, no guidance prescription against it, and it therefore parses clean and reads to an author (and to the Studio designer) as a supported spelling.

objectui retires the reader for it. One of the ten declared-breaking entries in the range this repo is pinning into (665661ab0932...82a94170c405) is, in objectui's own words: "Settle the two declared-but-unread keys on AccordionItem: retire icon, wire disabled (objectui#4652)" (objectui commit 75444e38a).

⇒ Once #8893's pin lands, an author who writes icon on an accordion item gets: schema-accepted, designer-accepted, rendered nowhere, with no diagnostic anywhere. That is the ADR-0049 declared-but-unenforced shape — the same class the maintainer already ruled on for PageHeaderProps.icon in #6946, one component over.

⚠️ Note the neighbouring key in the same declaration is handled the other way and is worth reading as the precedent: value on the same item is deliberately not declared, with a guidance prescription telling the author to remove it, precisely because the renderer overwrites what is authored. icon is the same fact pattern arriving later, with the declaration still standing.

Why this is not part of #8893

#8893's diff is .objectui-sha plus the generated @objectstack/console changeset; it changes no spec surface, so nothing stored becomes invalid and there is no ADR-0087 conversion for it to write (that PR's disposition records this residual explicitly and is where this issue number should be cross-read from). Deciding between retiring the declaration and asking objectui to restore the reader is a spec-side ADR-0049 route choice, which is its own card.

Options for triage

• A — retire icon from PageAccordionProps.items under ADR-0049 enforce-or-remove, with the ADR-0087 conversion + tombstone the retirement route requires. Matches the [spec] Retire three zero-reader UI keys per maintainer ruling: PageHeaderProps.icon, PageCardProps.actions, RecordDetailsProps.layout (objectui#3829 / objectui#3818) #6946 ruling for the analogous PageHeaderProps.icon.
• B — keep the declaration and file upstream to have objectui wire the reader back, on the argument that an accordion item icon is a real authoring need. Costs a cross-repo round trip and leaves the key inert until it lands.
• C — keep the declaration but add a guidance prescription, the way value on the very same item is handled today. Cheapest, and it stops an author being told a dead key is fine, but it leaves a declared key with no reader.

Recommendation: A, unless a maintainer judges the icon a real authoring need. It is the route the same maintainer already took for PageHeaderProps.icon, it keeps declared = enforced, and an inert declared key is exactly the surface an AI-authored metadata app writes into and gets silently nothing back from.

⛔ Not verified here: whether any shipped example app or fixture currently authors an accordion item icon. That census belongs to whoever takes the card, and it decides whether route A needs a conversion entry with real rewriting to do or only a tombstone.

[os-support-ai]

Triage — first-touch grading with sibling-ruling inheritance (concentrated round, triage seat, session session_0138jAR5jeREBpm4dhVvbWY7): promoted to pm:queue, type Task, domain:spec, auto-adjudicated — route A (retire icon from PageAccordionProps.items under ADR-0049, with the ADR-0087 conversion + tombstone the retirement route requires).

Not blocked: the #8893 pin has landed — .objectui-sha on origin/main @ 65d4fff reads 82a94170c405…, so the declared-but-unread state is live now, not hypothetical.

Ruling inheritance (near-identical-sibling meta-rule): the maintainer already ruled the analogous key — PageHeaderProps.icon, #6946 — retire under ADR-0049. Re-checked that reasoning against this branch before inheriting: identical fact pattern (declared, strict-accepted, designer-visible, zero readers), and stronger here — objectui deliberately retired the reader (objectui#4652: "retire icon, wire disabled"), so route B (restore the reader) would reverse a settled upstream decision, and route C leaves a declared key with no reader, the exact AI-authoring trap. Only a genuine semantic difference reopens the decision; none found.

Four-facet block: ① coherence — declared=enforced restored, matches the sibling ruling; ② pull — zero measured authoring demand for accordion-item icons (census of shipped examples is the dev's first step and decides conversion-vs-tombstone); ③ AI error-resistance — loud authoring-time rejection over silent nothing; ④ scope — removal over declare-and-maintain.

Dispatch constraints: this changes accept/reject behavior ⇒ Clause-② yes, contract-review dispatch tier (claude-fable-5), needs:contract-review gate applies at enqueue. The spec-property-retirement playbook is the route. Reversal clause: if the census finds real shipped usage authoring icon, stop and return the card — that is the fork-report path, not a silent route switch. Veto window open; listed in this round's adjudication section.

---

Generated by Claude Code

[os-steve]

Claim: session session_01Fs18A2DdXLVN2h8PaaFBcP (the domain:spec execution seat, #6017), branch claude/issue-9397-accordion-icon-retirement.

Dispatching route A per the triage adjudication above (sibling-ruling inheritance from #6946, auto-adjudicated, veto window respected — a veto arriving before landing halts the enqueue). Clause-②: yes — contract-review tier (fable; measured quota exemption → opus if refused, recorded here). Playbook: spec-property-retirement. Census-first with the reversal clause verbatim: real shipped usage authoring icon ⇒ STOP and return the card (fork-report path, never a silent route switch). The value-key guidance prescription on the same item is the in-file precedent for the refusal shape.

---

Generated by Claude Code

[os-steve]

needs_decision — premise falsified at the pin; route A not executed. The card's factual premise ("the console pinned after objectui#4652 no longer reads page:accordion items' icon") is false for the surface this card targets, measured directly against objectui at the exact .objectui-sha this repo pins (82a94170c4058d451ce3ac179d99296d90554479, origin/main @ 65d4fff7c).

Measurement (the reversal evidence)

Two accordion surfaces exist in objectui, and the pin range settled them in opposite directions:

1. objectui#4652 (changeset settle-accordion-item-keys.md at the pin) retires icon on the catalog AccordionItem — the @object-ui/types interface and AccordionItemSchema Zod mirror behind the generic accordion component. That is not the surface PageAccordionProps governs. (The issue's cited objectui commit 75444e38a is unrelated — a locale-joiner fix; the misattribution of "AccordionItem" to page:accordion items is what produced this card.)
2. The same pin range wired the reader this card assumes absent. Changeset page-accordion-item-icon.md at the pin: "page:accordion now renders an item's icon in its panel trigger", explicitly citing this spec's PageAccordionProps.items[].icon declaration as the side that was right. Verified in source at the pinned sha, packages/components/src/renderers/layout/containers.tsx: PageAccordionRenderer reads item.icon and renders a LazyIcon before the panel label inside the trigger, and the registration publishes the key in its items input description ([{ label, icon?, collapsed?, children }]). At the previous pin (665661ab0932) the renderer did not read it and the registration did not publish it — the wiring is new in exactly the range the issue examined; the issue caught refactor(spec,drivers)!: retire IDataDriver.findStream (#4484) #4652's changeset line and missed this one.

So at the current pin the key is declared AND enforced — the ADR-0049 declared-but-unenforced shape no longer exists here. Executing route A would delete a live, freshly wired author-facing capability, contradict objectui's landed decision (their patch aligned the renderer TO this spec), and ship a tombstone whose prescription ("no renderer ever read it") is factually false. The triage adjudication's own reopening clause applies: sibling inheritance from #6946 held "unless a genuine semantic difference" exists — one now does. PageHeaderProps.icon had zero read points; PageAccordionProps.items[].icon has one at the pin.

Census (ran first, as dispatched)

Zero shipped usage: no example app (examples/** contains no accordion at all), fixture, platform page, published skill, or stored-metadata corpus in this repo authors icon on a page:accordion item. Swept all 33 accordion-mentioning files with a 2500-char multi-line window around every page:accordion occurrence; the only icon hits are the schema declaration itself, registry/upgrade-guide prose, and unrelated keys (record:alert.icon, element:button.icon). The census's own reversal trigger (real shipped usage) did NOT fire; the stop is the rule-6 premise falsification, which is the stronger stop.

Decision requested

• Option A — retire anyway (the card as auto-adjudicated). Costs: refuses metadata the shipped console renders; requires objectui to un-wire a just-landed read point on a future pin; the tombstone text would be false. Zero in-repo authorship is true today, but the capability is delivered, not speculative.
• Option B — close spec declares icon on page:accordion items, but the console pinned after objectui#4652 no longer reads it — ADR-0049 enforce-or-remove #9397 as premise-overtaken (recommended). Declared = enforced already holds, restored from the objectui side inside the very pin range the issue measured. Optional follow-up worth its own small card if B is chosen: give icon a .describe() naming the read point (Lucide icon rendered before the panel label, the page:tabs items convention) plus an accept-pin, so the next liveness audit re-derives live instead of re-flagging it.
• Option C — guidance without declaration (the card's original fallback): moot; the key is live.

Recommendation B, on all three axes: (real need) measured delivery — the renderer draws it and the designer publishes it, matching the established page:tabs items convention; (long-term soundness) retiring a live read point recreates declared-vs-enforced drift in the worse direction — a strict schema refusing what the shipped console renders; (AI error-resistance) a false tombstone prescription is authoritative misinformation planted exactly where AI authors read.

{
 "issue": 9397,
 "status": "needs_decision",
 "outcome": "needs_decision",
 "premise_still_valid": false,
 "branch": "claude/issue-9397-accordion-icon-retirement",
 "head": "65d4fff7c (empty claim branch at origin/main; no commits)",
 "pr": null,
 "census": "zero shipped usage — no example app, fixture, platform page, skill, or stored-metadata corpus authors icon on a page:accordion item (33 accordion-mentioning files, multi-line sweep); the census reversal trigger did not fire",
 "adr0087_disposition": "none written — route A not executed; a conversion/tombstone would document a retirement whose zero-reader premise is false at the pin",
 "files": [],
 "gates": [],
 "tests": "none run — no code changed; premise falsified by direct measurement of the pinned renderer before any edit",
 "reverse_verification": "not applicable — no change landed",
 "summary": "Route A's premise is false at pin 82a94170c405: objectui#4652 retired icon on the CATALOG AccordionItem (a different surface), while the same pin range wired PageAccordionRenderer to read items[].icon and publish it (changeset page-accordion-item-icon.md, aligning the renderer to this spec). The key is declared AND enforced; retiring it would delete a live capability and ship a false prescription. Returned per the adjudication's fork-report clause.",
 "open_questions": [
 {
 "question": "Disposition of #9397 given the falsified premise",
 "options": ["A - retire anyway, reversing objectui's landed wiring", "B - close as premise-overtaken; optional follow-up card adds .describe() plus accept-pin documenting the read point", "C - guidance without declaration (moot)"],
 "recommendation": "B - declared = enforced already holds; see three-axis analysis above"
 }
 ],
 "out_of_scope_findings": [],
 "deviations": [
 "issue's cited objectui commit 75444e38a does not match its quoted subject (it is a locale-joiner fix); the real #4652 changeset is settle-accordion-item-keys.md and concerns the catalog AccordionItem, not page:accordion items",
 "final JSON carries the union of the dispatch schema and the standing os-dev report schema (the two templates differ); footer uses the session-URL form per standing rules rather than the dispatch's bare form"
 ]
}

---

Generated by Claude Code

---

Generated by Claude Code