Skip to content

Feat/file upload input - #15

Merged
gitsad merged 8 commits into
mainfrom
feat/file-upload-input
Apr 29, 2026
Merged

gitsad merged 8 commits into
mainfrom
feat/file-upload-input

Conversation

@gitsad

@gitsad gitsad commented Apr 29, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Adds a file field type for MDMA forms — forms can now declare file upload inputs alongside text/email/number/etc. Includes a default file input UI in the React renderer (overridable via ElementOverridesContext), spec schema + validator defaults, and authoring guidance in the prompt pack so LLMs know to use it.

While in the same area, also hardens the fixer prompt (Duplicate ID must rename instead of delete; LLM must not invent thinking blocks as commentary) and reorganizes fixer evals — multi-step flow tests now live in tests-fixer-flow.yaml with variantKey: flow, and flow-ordering exclusion is conditional on the variant. eval:fixer 10/10, eval:fixer-flow 7/7.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation

Packages Affected

  • @mobile-reality/mdma-spec
  • @mobile-reality/mdma-renderer-react
  • @mobile-reality/mdma-runtime
  • @mobile-reality/mdma-prompt-pack

Also affects @mobile-reality/mdma-validator (schema defaults for the new file field type) — not listed in the template above.

Checklist

  • My code follows the existing code style (pnpm format and pnpm lint pass).
  • I have added or updated tests that cover my changes.
  • All tests pass (pnpm test).
  • Type-checking passes (pnpm typecheck).
  • I have added a changeset (pnpm changeset) if this change affects published packages.
  • New or changed MDMA schemas are backwards-compatible (or marked as breaking).

@gitsad
gitsad merged commit ffd33f8 into main Apr 29, 2026
1 check passed
@gitsad
gitsad deleted the feat/file-upload-input branch April 29, 2026 14:32
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.

3 participants