Skip to content

Add preserve_leaf_arrays, properties, and passthrough to FlattenedProperty#6291

Draft
MattDevy wants to merge 4 commits into
mainfrom
add-flattened-property-fields
Draft

Add preserve_leaf_arrays, properties, and passthrough to FlattenedProperty#6291
MattDevy wants to merge 4 commits into
mainfrom
add-flattened-property-fields

Conversation

@MattDevy

Copy link
Copy Markdown
Contributor

Summary

Adds three missing fields to FlattenedProperty in specification/_types/mapping/complex.ts.

All three are @availability stack since=9.4.0 stability=stable. A new FlattenedPropertyPassthrough class is added to hold the { priority: integer } object shape.

Test plan

  • Verify generated schema includes all three new fields on FlattenedProperty
  • Verify FlattenedPropertyPassthrough appears in the schema with required priority: integer
  • Verify @availability stack since=9.4.0 stability=stable is reflected in output for all three fields
  • Confirm fields are absent from serverless schema (stack-only annotations)

@github-actions

github-actions Bot commented May 20, 2026

Copy link
Copy Markdown
Contributor

Following you can find the validation changes against the target branch for the API.

API Status Request Response
indices.create 🔴 1521/1587 → 1524/1587 1587/1587

You can validate this API yourself by using the make validate target.

@MattDevy
MattDevy marked this pull request as ready for review May 21, 2026 09:48
@MattDevy
MattDevy requested review from flobernd and l-trotta May 21, 2026 09:48
@l-trotta

Copy link
Copy Markdown
Contributor

wait where's properties?

@MattDevy

Copy link
Copy Markdown
Contributor Author

wait where's properties?

I added properties, then removed it as it clashes with the "properties" key name / the compiler freaked out.

Let me put this one back in draft, might need to work out how to re-add it properly

@MattDevy
MattDevy marked this pull request as draft May 21, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants