Skip to content

docs(field-types): correct type count (48) and fold barcode into qrcode#24

Merged
os-zhuang merged 1 commit into
mainfrom
docs/field-types-nits
Jun 16, 2026
Merged

docs(field-types): correct type count (48) and fold barcode into qrcode#24
os-zhuang merged 1 commit into
mainfrom
docs/field-types-nits

Conversation

@os-zhuang

Copy link
Copy Markdown
Contributor

Follow-up cleanup after the 9.7.0 upgrade (#23) — three verified field-type-reference nits that were skipped as low-confidence and then confirmed against the framework spec.

  • Type count — the page claimed "53 built-in field types"; the spec FieldType enum (packages/spec/src/data/field.zod.ts) has 48. Corrected.
  • barcode is not a type — only qrcode is in the enum; it covers QR + barcode formats. Removed the bogus barcode row and folded its use-case (EAN / UPC / Code128) into qrcode.
  • Pruned display knobsqrErrorCorrection / barcodeFormat were pruned as dead surface (no runtime reader; renderers ignore them), so they're dropped from the qrcode description.

English + all six translations (de/es/fr/ja/ko/zh-Hans). apps/docs type-check (fumadocs-mdx + tsc) passes.

Note: the same dead-surface prune also left a few other stale display knobs in this table (code theme/lineNumbers, color colorFormat/presetColors, rating allowHalf, …). Those are out of scope here; a dedicated display-knob sync pass can follow if wanted.

🤖 Generated with Claude Code

The reference claimed "53 built-in field types" but the spec FieldType enum
has 48. `barcode` is not a distinct type — `qrcode` covers QR + barcode
formats — and the `qrErrorCorrection` / `barcodeFormat` display knobs were
pruned as dead surface (no runtime reader). Drop the bogus `barcode` row,
fold its use-case (EAN/UPC/Code128) into `qrcode`, and fix the count.
English + all six translations (de/es/fr/ja/ko/zh-Hans); apps/docs type-check
passes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@os-zhuang os-zhuang merged commit f072cbb into main Jun 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant