root/fusion
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: resolve Grok ACP adapter TypeScript errors for CI typecheck/build

Browse Source
This commit is contained in:
gsxdsm
2026-07-11 23:07:58 -07:00
parent de8d00a475
commit 82f007b273
2 changed files with 17 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
plugins/fusion-plugin-grok-runtime/src/__tests__/runtime-adapter.test.ts
View File

@@ -11,7 +11,7 @@ plugin suite and manual smoke against `grok agent stdio`.
*/
function makeFakeAcpAdapter(overrides?: {
createSession?: AcpAdapterFactory extends (s: never) => infer R ? R["createSession"] : never;
createSession?: ReturnType<AcpAdapterFactory>["createSession"];
promptWithFallback?: (
session: AgentSession,
prompt: string,

21
plugins/fusion-plugin-grok-runtime/src/runtime-adapter.ts
View File

@@ -124,10 +124,14 @@ function resetTurnAccum(session: GrokSession): void {
function collectCustomTools(options: AgentRuntimeOptions): ToolLike[] {
const fromCustom = Array.isArray(options.customTools) ? (options.customTools as ToolLike[]) : [];
// Some call sites pass tools as an array of ToolDefinitions instead of "coding"/"readonly".
const maybeToolsArray = Array.isArray((options as { tools?: unknown }).tools)
? ((options as { tools: ToolLike[] }).tools)
: [];
/*
FNXC:GrokAcp 2026-07-11-19:00:
AgentRuntimeOptions.tools is typed as "coding"|"readonly"|undefined, but some call sites pass
an array of ToolDefinitions. Narrow via Array.isArray on the tools field, then cast the array
value only — never cast the whole options object to { tools: ToolLike[] } (TS2352).
*/
const toolsField = (options as { tools?: unknown }).tools;
const maybeToolsArray = Array.isArray(toolsField) ? (toolsField as ToolLike[]) : [];
return [...fromCustom, ...maybeToolsArray];
}
@@ -211,9 +215,16 @@ export class GrokRuntimeAdapter implements AgentRuntime {
constructor(options?: GrokRuntimeAdapterOptions) {
this.binary = options?.binary ?? "grok";
/*
FNXC:GrokAcp 2026-07-11-19:00:
AcpRuntimeAdapter returns ACP AgentSession shapes (acp/types); AcpAdapterFactory is typed
against Grok AgentSessionResult (messages/state). createSession always runs
ensureGrokSessionShape after the ACP create, so the production factory is a deliberate
structural bridge via unknown rather than unifying the two session interfaces here.
*/
this.createAcpAdapter =
options?.createAcpAdapter ??
((settings) => new AcpRuntimeAdapter(settings));
((settings) => new AcpRuntimeAdapter(settings) as unknown as ReturnType<AcpAdapterFactory>);
}
async createSession(