feat(specs): add facets query parameter available at run time

[ClaraMuller]

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/CMP-594
Mitigation for #5477 in the meantime.

Changes included:

Add facets parameter as a potential value for run endpoint as it is allowed in the payload

🧪 Test

• CI
• Run locally yarn cli build specs all && yarn cli build clients javascript

[algolia-bot]

💥 Breaking changes detected !!

Either this PR or a previous PR not released yet introduced breaking changes, be careful when merging.
You can find the details in the 'client java@21' CI job.

✔️ Code generated!

Name	Link
🪓 Triggered by	ca792e5e411441b06ae5e447e1c7bb29b88da2da
🍃 Generated commit	54fdb4997d84df83c412d650c5eb8095cc9601de
🌲 Generated branch	generated/clara/feat/specs/facet-available-in-run-endpoint

📊 Benchmark results

Benchmarks performed on the method using a mock server, the results might not reflect the real-world performance.

Language	Req/s
go	1739
javascript	1655
php	1433
csharp	1265
python	1135
java	1028
swift	832
ruby	749
scala	23

[ClaraMuller]

Hello @millotp as a mitigation of #5477, I created this PR, but even without re-defining anything, I have some issues.

[shortcuts]

this one is expected @ClaraMuller you need to add IndexSettingsFacets to https://github.com/algolia/api-clients-automation/blob/main/generators/src/main/java/com/algolia/codegen/AlgoliaSwiftGenerator.java#L32, swift have one big bundle that prevents duplicate names between its sub packages (search, composition)

[ClaraMuller]

Thanks @shortcuts I am not confortable modifying this, can I get your help on this ?

[millotp]

Hey @ClaraMuller, actually it was the same name clashing issue as the last PR, I did some renaming and now there is no conflicts for facets anymore, is this what you expected to generate ?

[ClaraMuller]

@millotp it seems that @shortcuts was suggesting an other solution

[millotp]

it seems that @shortcuts was suggesting an other solution

I spoke with @shortcuts, it wouldn't have worked, the issue was with redocly, not openapi-generator, so we need to fix the spec before changing the generator

[ClaraMuller]

Oh I see in that case if does generate a proper parameter called facets (which is my goal here) let's do it that way.

[millotp]

I couldn't make it work without breaking algoliasearch lite or recommend, so the name of the model will have to stay as IndexSettingsFacets to avoid other clashes

[ClaraMuller]

Hello @millotp
What can Comp API do ? At the moment change is minimal I am a bit at the limit of what I can contribute autnomously.

[sfaiqh]

@millotp @shortcuts is this issue introduced by this PR or has become apparent due to it? Would it be possible to see the duplicated files where they are?

Any help would be appreciated as this change is required for customer testing which we have to do soon.

[shortcuts]

thanks for taking care of it @millotp