Skip to content

Complete oneOf property values#326

Open
WilliamK112 wants to merge 1 commit into
microsoft:mainfrom
WilliamK112:codex/complete-oneof-property-values
Open

Complete oneOf property values#326
WilliamK112 wants to merge 1 commit into
microsoft:mainfrom
WilliamK112:codex/complete-oneof-property-values

Conversation

@WilliamK112

Copy link
Copy Markdown

Summary

  • Look through allOf, anyOf, and oneOf value schemas when building property completion snippets.
  • Preserve existing fallback behavior for unknown property schemas while still inserting : $1 when multiple value proposals exist.
  • Add regression coverage for a oneOf property schema that previously inserted only the property name.

Fixes #84

Validation

  • PATH="/Users/William/.cache/codex-runtimes/codex-primary-runtime/dependencies/node/bin:$PATH" npm test
  • git diff --check

Signed-off-by: WilliamK112 <164879897+WilliamK112@users.noreply.github.com>
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.

JSON Schema “oneOf”: autocompletion of property name doesn’t insert a colon

1 participant