Skip to content

fix: allow deprecated service-level YAML keys#169

Merged
vigneshrajsb merged 1 commit intomainfrom
codex/allow-legacy-service-keys
Apr 16, 2026
Merged

fix: allow deprecated service-level YAML keys#169
vigneshrajsb merged 1 commit intomainfrom
codex/allow-legacy-service-keys

Conversation

@vigneshrajsb
Copy link
Copy Markdown
Contributor

Summary

This change relaxes service-level YAML schema validation so deprecated keys such as kedaScaleToZero no longer fail validation.

What changed

  • changed services[].additionalProperties to true in the 1.0.0 schema
  • updated generated schema artifacts to match
  • added a regression test proving a config with kedaScaleToZero validates as a no-op

Why

Some dependent repos still include deprecated service-level keys. After the schema cleanup, those configs started failing validation even though the deprecated fields are no longer used at runtime.

This keeps validation backward-compatible while those repos are cleaned up.

Follow-up

After the dependent config cleanup is complete, we should revert this compatibility change and restore stricter service-level schema validation.

Verification

  • pnpm test -- src/server/models/config/index.test.ts

Tradeoff

Service-level validation is now more permissive, so unknown top-level service keys will no longer be rejected by schema validation.

@vigneshrajsb vigneshrajsb marked this pull request as ready for review April 16, 2026 15:38
@vigneshrajsb vigneshrajsb requested a review from a team as a code owner April 16, 2026 15:38
@vigneshrajsb vigneshrajsb changed the title [codex] allow deprecated service-level YAML keys fix: allow deprecated service-level YAML keys Apr 16, 2026
@vigneshrajsb vigneshrajsb merged commit 8fe7693 into main Apr 16, 2026
1 check passed
vigneshrajsb added a commit that referenced this pull request Apr 16, 2026
vigneshrajsb added a commit that referenced this pull request Apr 16, 2026
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