Skip to content

Teach enum Field authoring - #4

Closed
raghubetina wants to merge 1 commit into
codex/schema-validation-routingfrom
codex/enum-authoring-guidance
Closed

Teach enum Field authoring#4
raghubetina wants to merge 1 commit into
codex/schema-validation-routingfrom
codex/enum-authoring-guidance

Conversation

@raghubetina

Copy link
Copy Markdown
Contributor

Summary

  • Document the supported enum Field boundary, ordered value identity, and ordinal semantics.
  • Add a complete schema-valid example plus local-authoring and successful-push evals.
  • Refresh capability diagnostics so enum structure is accepted while defaults and Validations remain fail-closed.
  • Document reproducible initialized-state handling for server-backed eval runs.

Stack

Verification

  • sh script/check: 11/11 passing
  • npm audit --audit-level=low: 0 vulnerabilities
  • gh skill publish --dry-run: passed with the existing license and tag-protection advisories
  • The exact enum fixture was accepted by API commit 3282954 and persisted low, medium, high in order with ordinality enabled
  • Local max-effort review: approved with no remaining findings

Document stable value identities and ordinal semantics.

Add local and push evals to keep importer and recovery boundaries honest.
@raghubetina

Copy link
Copy Markdown
Contributor Author

Stacked follow-up #5 is now open on top of this branch: #5

It updates the Skill for the Field-default importer in firstdraft/firstdraft#200 and pins the complete server baseline through firstdraft/firstdraft#202. The enum guidance in this PR remains accurate at its API #196 baseline; with the later API slices included, the statement that enum defaults remain a capability gap is superseded. #5 teaches all four tagged default variants, default ownership without a new subject UUID, omission versus literal null, canonical decimal strings, enum-value rename coupling, and recovery evals. It does not change the bundled schema or generic CLI.

@raghubetina

Copy link
Copy Markdown
Contributor Author

This slice has landed on main through #6, which integrated the complete reviewed Skill stack. This PR remains the focused review history for enum Field authoring; no implementation was discarded. Closing as superseded by #6.

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