Skip to content

fix(generators): getObject has no generic type#6026

Merged
MarioAlexandruDan merged 1 commit intomainfrom
fix/generation-get-object-generic
Mar 3, 2026
Merged

fix(generators): getObject has no generic type#6026
MarioAlexandruDan merged 1 commit intomainfrom
fix/generation-get-object-generic

Conversation

@MarioAlexandruDan
Copy link
Copy Markdown
Contributor

@MarioAlexandruDan MarioAlexandruDan commented Feb 26, 2026

🎟 JIRA Ticket API-65:

Changes included:

  • List changes

🧪 Test

@algolia-bot
Copy link
Copy Markdown
Collaborator

algolia-bot commented Feb 26, 2026

✔️ Code generated!

Name Link
🪓 Triggered by c9e1d968045a4f3602661ac7d1bf402e28e190aa
🍃 Generated commit 84842cd3a351dd8a357db46f1f975df169dada5c
🌲 Generated branch generated/fix/generation-get-object-generic
📊 Benchmark results

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

Language Req/s
javascript 2415
go 2207
php 1733
csharp 1537
python 1399
java 1098
ruby 1032
swift 740
scala 23

@MarioAlexandruDan MarioAlexandruDan changed the title fix: getObject has no generic type fix(generators): getObject has no generic type Feb 26, 2026
@MarioAlexandruDan MarioAlexandruDan self-assigned this Feb 26, 2026
- GenericPropagator: detect inline anonymous response schemas with
  x-is-generic and set x-return-is-generic on the operation so templates
  know T is the direct return type, not a wrapper type parameter
- JS api-single.mustache: use x-return-is-generic to emit
  getObject<T = Record<string, unknown>>(): Promise<T> with a default
  type for backward compatibility
- CTS generators: expose isReturnGeneric in test context for future use

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@MarioAlexandruDan MarioAlexandruDan force-pushed the fix/generation-get-object-generic branch from 742595b to 8bfcc7b Compare February 27, 2026 13:00
@MarioAlexandruDan MarioAlexandruDan marked this pull request as ready for review February 27, 2026 13:18
@MarioAlexandruDan MarioAlexandruDan requested a review from a team as a code owner February 27, 2026 13:18
@MarioAlexandruDan MarioAlexandruDan enabled auto-merge (squash) February 27, 2026 14:00
Copy link
Copy Markdown
Collaborator

@Fluf22 Fluf22 left a comment

Choose a reason for hiding this comment

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

🆗

@MarioAlexandruDan MarioAlexandruDan merged commit 81b565c into main Mar 3, 2026
36 checks passed
@MarioAlexandruDan MarioAlexandruDan deleted the fix/generation-get-object-generic branch March 3, 2026 09:11
algolia-bot added a commit that referenced this pull request Mar 3, 2026
…kip ci]

Co-authored-by: Mario-Alexandru Dan <marioalexandrudan@gmail.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Mar 3, 2026
algolia/api-clients-automation#6026

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Mario-Alexandru Dan <marioalexandrudan@gmail.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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