fix(ai-grok): remove Record<string,unknown> index sig from GrokTextProviderOptions

[citizen204]

Summary

grokSummarize('grok-4.3') and grokSummarize('grok-build-0.1') are not assignable to summarize()'s adapter parameter, producing a type error at every call site.

Fixes #821

Root cause

GrokTextProviderOptions extends both GrokBaseOptions and Record<string, unknown>. Under strictFunctionTypes, function parameters are checked contravariantly. The constraint on summarize<TAdapter extends SummarizeAdapter<string, object>> means the adapter's summarize method must accept SummarizationOptions<object>. Contravariance requires object to be assignable to GrokTextProviderOptions. But object is not assignable to Record<string, unknown> (index-signature types require a compatible index signature from the source), so the check fails.

OpenAI's provider options have no index signature and all-optional named fields, so they pass: object satisfies any all-optional interface.

Fix

Remove Record<string, unknown> from the extends clause of GrokTextProviderOptions. All fields are explicitly typed optional members — the index signature was not needed. After this change, object is assignable to GrokTextProviderOptions (all-optional, no index signature), fixing the contravariant check.

GrokBuildProviderOptions (Omit<GrokTextProviderOptions, 'reasoning'>) is not affected: the named fields are preserved correctly by Omit since there is no longer an index signature to collapse.

The TProviderOptions extends Record<string, unknown> constraint on GrokTextAdapter remains satisfied: a named-property interface with unknown-compatible values is structurally assignable to Record<string, unknown>.

Changes

• packages/ai-grok/src/text/text-provider-options.ts: Remove Record<string, unknown> from GrokTextProviderOptions extends clause
• .changeset/fix-grok-provider-options-index-sig.md: Patch changeset for @tanstack/ai-grok

Summary by CodeRabbit

• Bug Fixes
  • Improved TypeScript type compatibility for Grok summarization adapters, eliminating strict type errors at summarize() call sites.
  • Updated Grok text provider options to rely only on explicitly defined fields, removing the overly broad option shape that caused mismatches under strict function typing.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c19df0e1-f496-4382-afe4-3973568ae469

📥 Commits

Reviewing files that changed from the base of the PR and between 26d183e and 7ba8441.

📒 Files selected for processing (2)

• .changeset/fix-grok-provider-options-index-sig.md
• packages/ai-grok/src/text/text-provider-options.ts

✅ Files skipped from review due to trivial changes (1)

• .changeset/fix-grok-provider-options-index-sig.md

🚧 Files skipped from review as they are similar to previous changes (1)

• packages/ai-grok/src/text/text-provider-options.ts

---

📝 Walkthrough

Walkthrough

GrokTextProviderOptions no longer extends Record<string, unknown>, and the package changeset records the patch release for that type-only fix.

Changes

Grok options type fix

Layer / File(s)	Summary
Options shape and changeset packages/ai-grok/src/text/text-provider-options.ts, .changeset/fix-grok-provider-options-index-sig.md	GrokTextProviderOptions now extends only GrokBaseOptions, removing the catch-all index signature, and the changeset records the patch release for the fix.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related issues

• ci: type-check is excluded from examples/** and testing/** — call-site type regressions slip through test:pr #820 — The PR removes the Grok options index signature that caused the grokSummarize(...) assignability issue described there.

Poem

🐰 I nibble one type, so neat and small,
The index-sig falls, no more at all.
Grok hops through summarize() with ease,
And patch notes rustle in the breeze.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	The type fix matches #821, but the required included-package call-site type coverage for grokSummarize/createGrokSummarize is missing.	Add a type-level regression test in an included package that calls summarize() with grokSummarize('grok-4.3') and grokSummarize('grok-build-0.1').

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly states the Grok provider-options index-signature fix and matches the main change.
Description check	✅ Passed	The description covers the summary, root cause, fix, and changed files, but omits the checklist and release-impact template sections.
Out of Scope Changes check	✅ Passed	The changes stay focused on the Grok provider-options type shape and the related changeset, with no unrelated code edits.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[nx-cloud]

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

_{To disable these notifications, a workspace admin can disable them in workspace settings.}

---

View your CI Pipeline Execution ↗ for commit 079c5a3

Command	Status	Duration	Result
nx run-many --targets=build --exclude=examples/...	❌ Failed	4s	View ↗

---

☁️ Nx Cloud last updated this comment at 2026-06-25 09:48:31 UTC