Skip to content

Commit d387644

Browse files
committed
fix(wasm-worker): restore paramBindings in SerializedExtractorOutput and remove duplicate copy in pool (#1331)
The previous commit removed paramBindings from SerializedExtractorOutput to fix a TS2300 duplicate-identifier error, but left two references to ser.paramBindings in wasm-worker-pool.ts (lines 110 and 125), causing TS2339 errors that broke every CI job. Restore paramBindings as an inline import in the protocol interface (matching the style of the other binding fields), and remove the duplicate line 125 copy in pool.ts.
1 parent 9fb1a8b commit d387644

2 files changed

Lines changed: 1 addition & 1 deletion

File tree

src/domain/wasm-worker-pool.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,6 @@ function deserializeResult(ser: SerializedExtractorOutput | null): ExtractorOutp
122122
out.returnTypeMap = returnTypeMap;
123123
}
124124
if (ser.callAssignments?.length) out.callAssignments = ser.callAssignments;
125-
if (ser.paramBindings?.length) out.paramBindings = ser.paramBindings;
126125
return out;
127126
}
128127

src/domain/wasm-worker-protocol.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ export interface SerializedExtractorOutput {
7070
arrayCallbackBindings?: import('../types.js').ArrayCallbackBinding[];
7171
objectRestParamBindings?: import('../types.js').ObjectRestParamBinding[];
7272
objectPropBindings?: import('../types.js').ObjectPropBinding[];
73+
paramBindings?: import('../types.js').ParamBinding[];
7374
newExpressions?: readonly string[];
7475
returnTypeMap?: Array<[string, TypeMapEntry]>;
7576
callAssignments?: CallAssignment[];

0 commit comments

Comments
 (0)