Skip to content

Conversation

@Fluf22
Copy link
Collaborator

@Fluf22 Fluf22 commented Apr 30, 2025

🧭 What and Why

🎟 JIRA Ticket: DI-3716

Changes included:

  • enable the composition client in the swift generator config
  • add the composition client in the reserved name check so all models are prefixed
  • add a prefix lambda for response and generic types in tests

🧪 Test

  • composition client tests are enabled

@Fluf22 Fluf22 requested a review from a team as a code owner April 30, 2025 16:46
@Fluf22 Fluf22 self-assigned this Apr 30, 2025
@Fluf22 Fluf22 requested review from morganleroi and shortcuts April 30, 2025 16:46
@algolia-bot
Copy link
Collaborator

algolia-bot commented Apr 30, 2025

✔️ Code generated!

Name Link
🪓 Triggered by 077fd80037f186571f8ee2545177ae5e3a3cd1df
🍃 Generated commit 6229ffed72f9577be2a5da248dacd5200d2cca38
🌲 Generated branch generated/feat/swift-generate-composition
📊 Benchmark results

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

Language Req/s
javascript 1639
go 1573
php 1466
csharp 1415
python 1094
java 1089
ruby 806
swift 499
scala 23

@Fluf22 Fluf22 force-pushed the feat/swift-generate-composition branch from 4d238a5 to 7865af7 Compare April 30, 2025 19:57
Copy link
Collaborator

@millotp millotp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice !
There are 6 models that starts with CompositionCompisition, like CompositionCompositionRunSearchResponse, is it possible to not add the prefix in this case ?

@Fluf22
Copy link
Collaborator Author

Fluf22 commented May 2, 2025

Sadly, the prefixing is required on all models to avoid conflicts with other models from the client. eg:

  • BaseSearchResponse -> CompositionBaseSearchResponse
  • CompositionBaseSearchResponse -> CompositionCompositionBaseSearchResponse

@Fluf22 Fluf22 merged commit db6e6f2 into main May 2, 2025
28 checks passed
@Fluf22 Fluf22 deleted the feat/swift-generate-composition branch May 2, 2025 10:38
algolia-bot added a commit that referenced this pull request May 2, 2025
algolia-bot added a commit to algolia/algoliasearch-client-swift that referenced this pull request May 2, 2025
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.

3 participants