[IMax153]

Type

• Refactor
• Feature
• Bug Fix
• Optimization
• Documentation Update

Description

Related

• Related Issue From Discord: Parsing Error with xAI Models in @effect/ai-openrouter Due to Unrecognized reasoning_details Format #5965
• Closes From Discord: Parsing Error with xAI Models in @effect/ai-openrouter Due to Unrecognized reasoning_details Format #5965

[changeset-bot]

🦋 Changeset detected

Latest commit: e2f8ae6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@effect/ai-openrouter	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[timm-stelzer-e-farm]

I have a similar issue to the source bug. This PR isn't sufficient to fix it on my end. ReasoningDetailEncryptedFormat apparently also needs to be updated?

Here's part of the patch that fixed it for me. Note: In my actual patch, I also updated the dist files, obviously, but that's not relevant for this PR.

diff --git a/src/Generated.ts b/src/Generated.ts
index db3f7f37321b4ace55f3d8328f2583e0ebbaeead..260b76bd377b352e5e21976a17c9a9829125fc9f 100644
--- a/src/Generated.ts
+++ b/src/Generated.ts
@@ -16,7 +16,7 @@ export class CacheControlEphemeral extends S.Class<CacheControlEphemeral>("Cache
 
 export class ReasoningDetailSummaryType extends S.Literal("reasoning.summary") {}
 
-export class ReasoningDetailSummaryFormat extends S.Literal("unknown", "openai-responses-v1", "anthropic-claude-v1") {}
+export class ReasoningDetailSummaryFormat extends S.Literal("unknown", "openai-responses-v1", "xai-responses-v1", "anthropic-claude-v1", "google-gemini-v1") {}
 
 /**
  * Reasoning summary detail
@@ -34,9 +34,7 @@ export class ReasoningDetailSummary extends S.Class<ReasoningDetailSummary>("Rea
 
 export class ReasoningDetailEncryptedType extends S.Literal("reasoning.encrypted") {}
 
-export class ReasoningDetailEncryptedFormat
-  extends S.Literal("unknown", "openai-responses-v1", "anthropic-claude-v1")
-{}
+export class ReasoningDetailEncryptedFormat extends S.Literal("unknown", "openai-responses-v1", "xai-responses-v1", "anthropic-claude-v1", "google-gemini-v1") {}
 
 /**
  * Encrypted reasoning detail

[magoz]

I confirm @timm-stelzer-e-farm 's changes work for me:

https://github.com/magoz/effect/tree/fix/openrouter-reasoning-formats

[IMax153]

Superceded by #6026