Skip to content

Fix schema config being dropped in ActionSchemaFileCache.getSchemaSource#2016

Merged
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:compfixes
Mar 18, 2026
Merged

Fix schema config being dropped in ActionSchemaFileCache.getSchemaSource#2016
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:compfixes

Conversation

@curtisman
Copy link
Member

Summary

getSchemaSource was returning config: undefined instead of config: schemaContent.config, causing schema config to be dropped for TypeScript schemas. This meant:

  1. Config changes would not invalidate the cache hash
  2. Downstream schema parsing would not receive the config

Changes

  • actionSchemaFileCache.ts: Pass through schemaContent.config instead of hardcoded undefined
  • actionSchemaFileCache.spec.ts: Added tests verifying config is preserved and affects cache hash

getSchemaSource was returning config: undefined instead of
config: schemaContent.config, causing schema config to be
dropped for TypeScript schemas. This meant config changes
would not invalidate the cache hash, and downstream schema
parsing would not receive the config.

Added tests to verify config is preserved and affects hash.
@curtisman curtisman added this pull request to the merge queue Mar 18, 2026
Merged via the queue into microsoft:main with commit 187132f Mar 18, 2026
13 of 15 checks passed
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