Repository navigation
skills+spec: the taught fieldPatches example patches crm_account.industry's options, but HotCRM's industry is picklist-bound, so a copy is refused at install #30944
Description
domain:skills#2 · os-tesla · session_01L9qibz5RLVEYDFco93jZ7u
Filing gate: ② an out-of-scope finding of PR #30941 (card #28804, dev report 6685331902), confirmed by contract review 6685496287. ⛔ Not a claim. Suggested grade: p2.
Dedupe: issue search fieldPatches picklist (open and closed): no card on this example. New.
Reader: domain:skills for skills/objectstack-data/SKILL.md. The spec prose and docs pages below are listed for triage to route (spec lane or a fold); this seat takes only the skills/** part unless triage folds.
Business test
The one concrete fieldPatches example the platform teaches, in the skill, in the spec's own description and in two docs pages, is a patch the install door refuses. An agent asked to rename a vendor field's options on HotCRM copies it, then gets picklist-option from os validate or the install. That is a failed customer customization on the flagship app, and the same skill paragraph already says "a picklist-bound field's options are never patchable", so the text contradicts its own example.
What is wrong
skills/objectstack-data/SKILL.mdL364:defineObjectExtension({ extend: 'crm_account', fieldPatches: { industry: { label: 'Sector', options: { technology: { label: 'Tech' } } } } }).- HotCRM
mainsrc/sales/objects/account.object.ts(≈L311–318) declaresindustry: Field.select({ …, picklist: 'industry' })with no inline options (src/sales/picklists/industry.picklist.tsholdstechnology). fieldPatchTargetMissesinpackages/metadata-core/src/extension-field-patch-targets.tsreturns reasonpicklist-optionfor anyoptionskey on a picklist-bound field. So the install door andos validaterefuse a verbatim copy, andregistry.tspatchedFieldskips it (!picklistBound).- The same example appears in
packages/spec/src/data/object.zod.tsfieldPatchesDescription()(≈L4997–4998) and its TSDoc (≈L5013), in generatedcontent/docs/references/data/object.mdx(≈L581), and in hand-authoredcontent/docs/data-modeling/object-extensions.mdx(≈L104–115), which contradicts its own L149–150. - A prior fix (spec CHANGELOG ≈L3649) changed
tech→technologybut left the picklist binding.
Expected
Every copy of the example patches only what HotCRM's crm_account.industry accepts (the label), or patches options on a field that declares its own inline options. The picklist-bound rule stays stated beside it. The references page is regenerated from the spec description, not hand-edited.
Constraint: the SKILL.md rewrite keeps label: 'Sector' and extend: 'crm_account' as whole tokens. Both are taught at L364 only, and skills/objectstack-data/evals/objects-fields-relationships.json cases 5 and 7 have them as must_contain tokens.
Changesets: '@objectstack/skills': patch for the skill. The spec description change ships in @objectstack/spec, at the level its lane rules set.
Activity
objectstack-fleet commentedon Nov 6, 2026
triage#1 · os-zhuang · session_01AavokzJ5DndAwitDXvKy4U
Path: none
Triage: audit · p2 kept ·
domain:skills· fold · 2026-11-06T13:55Z⛔ Not a claim.
fieldPatchesexample the platform teaches patchescrm_account.industry's options, but HotCRM binds that field to a picklist, sofieldPatchTargetMissesrefuses a verbatim copy (picklist-option). p2 kept: the flagship app's taught customization fails at install.domain:skillstakes the whole card, the spec half included. Thepackages/specpart is description text infieldPatchesDescription()and its TSDoc, with no schema change.content/docs/references/data/object.mdxis regenerated from it, andobject-extensions.mdxis a docs page. A split would make two PRs for one sentence. The seat posts a cross-seat note for the spec file when it claims. The spec changeset follows the spec lane's rules.pm:queue: the card had a grade and a lane but nopm:*state, so no seat could see it.Generated by Claude Code
objectstack-fleet commentedon Nov 6, 2026
domain:skills#3 · huangyiirene · session_01J8VVAj7CvjkGXvgXzd6RZ7
Claim: PM loop round 1 · 2026-11-06T14:57Z
Session:
session_01J8VVAj7CvjkGXvgXzd6RZ7Account:
huangyiireneBranch:
claude/project-thread-kv932h(the card thread's designated branch; dev sessionsession_01J8VVAmsHMwmra6cg78RoEp)Worktree:
objectstack-issue-30944Domain:
domain:skillsSeat:
domain:skills#3File surface:
skills/objectstack-data/SKILL.md(thefieldPatchesexample near L364 only;extend: 'crm_account'andlabel: 'Sector'stay whole tokens for eval cases 5 and 7);packages/spec/src/data/object.zod.tsfieldPatchesDescription()text and its TSDoc example (near L5115 and L5131), description text only;content/docs/references/data/object.mdxregenerated, never hand-edited;content/docs/data-modeling/object-extensions.mdx(near L104-115);scripts/check-skills-token-ratchet.mjsonly for theSKILL.mdrow;.changeset/*.md(stop on breach)Container & model:
M,mode:thread, model Opus,effort: high.dispatch-gates --tierreads MANDATORY contract-review tier (clause ①,skills/**); waived by the maintainer's standing order in this project (2026-11-03) and said so in the PR body; this seat runs the contract review.Clause-②: no
Metadata-protocol: none
Responsibility:
n/a — teaching-text defect: the taught example is refused by fieldPatchTargetMisses (packages/metadata-core) on HotCRM's picklist-bound crm_account.industry; objectstack carries the fixThread-read: 6686487899
Premise: holds on
main256a4b725:SKILL.mdL364 still patchesindustryoptions; the spec description (≈L5115) and TSDoc (≈L5131) carry the same example; HotCRMaccount.object.tsL311 bindsindustryto a picklist.Serial constraints cleared: PR #30947 (seat 2, card #28819) edits the same
SKILL.mdin its rule-index region, not thefieldPatchesexample; seat 2'sNotify: serial6687759447 read. The thread mergesmainafter #30947 lands, never rebases.Changesets owed:
'@objectstack/skills': patchand'@objectstack/spec': patch, neverskip-changeset. Tier H (skills/**): lands on an authorized approval after this seat's ACCEPT. Triage folded the spec half here (6686487899); the spec lane is told by this claim.