Skip to content

Simplify Submission: drop type, namespace format, flatten content - #37

Merged
maximiliansoelch merged 2 commits into
mainfrom
30-simplify-submission-type-format-content
Sep 23, 2026
Merged

maximiliansoelch merged 2 commits into
mainfrom
30-simplify-submission-type-format-content

Conversation

@m-messer

Copy link
Copy Markdown
Contributor

Summary

  • Removes redundant type (TEXT/CODE/MATH/MODEL/OTHER) from Submission, FeedbackTarget, and ArtefactProfile; format alone now determines category via a namespaced, pattern-constrained convention (<category>:<subtype>, e.g. text:plain, code:sympy, math:sympy) — this also resolves the prior ambiguity where sympy/wolfram existed under both CODE and MATH.
  • Flattens content's type-mirroring wrapper key ({ text: ... }, { expression: ... }, { code: ... }) to a single generic value key for every artefact type, with format-specific auxiliary fields (entryPoint, imports, notation, diagramType) as siblings.
  • Deletes the now-unused ArtefactType.yml enum.
  • Updates all evaluate.yml/chat.yml/evaluate-health.yml examples to match.

Closes #30. As a side effect, also resolves #28 (no documented fallback key for non-conventional content — value is now the universal key).

Test plan

  • npm run lint passes (matches pre-existing warning baseline on main)
  • npm run bundle succeeds

🤖 Generated with Claude Code

…eld across all submission-related structures
@m-messer
m-messer marked this pull request as ready for review August 11, 2026 10:32
@neagualexa neagualexa assigned m-messer and unassigned neagualexa Aug 21, 2026

@maximiliansoelch maximiliansoelch left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for simplifying this. Code changes look good 👍

@maximiliansoelch maximiliansoelch changed the title Simplify Submission: drop type, namespace format, flatten content (#30) Simplify Submission: drop type, namespace format, flatten content Sep 23, 2026
@maximiliansoelch
maximiliansoelch merged commit 801ce65 into main Sep 23, 2026
2 checks passed
@maximiliansoelch
maximiliansoelch deleted the 30-simplify-submission-type-format-content branch September 23, 2026 17:24
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.

Type, format and content No documented fallback for non-conventional submission.content key names

3 participants