Skip to content

fix(config): reject non-object raw yaml#189

Open
MicroMilo wants to merge 1 commit into
OpenBMB:mainfrom
MicroMilo:codex/align-raw-yaml-validation
Open

fix(config): reject non-object raw yaml#189
MicroMilo wants to merge 1 commit into
OpenBMB:mainfrom
MicroMilo:codex/align-raw-yaml-validation

Conversation

@MicroMilo

Copy link
Copy Markdown

Summary

  • make parseConfigYaml reject null, array, and primitive YAML roots
  • align raw YAML validation with the save path root-type contract
  • add focused coverage for non-object raw YAML inputs

Verification

  • npm --workspace ui exec vitest run server/services/pilotdeckConfig.raw-yaml.test.js
  • npm exec -- tsc --noEmit -p tsconfig.json

Notes

This is a real validate/save mismatch: /api/config/validate could normalize non-object YAML into defaults while the PUT path rejected the same input.

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