Skip to content

Commit 22f8a66

Browse files
committed
address review feedback
use TIncrementalItem instead of TItems to constrain to ReadonlyArray of TIncrementalItem
1 parent de97191 commit 22f8a66

File tree

1 file changed

+18
-16
lines changed

1 file changed

+18
-16
lines changed

Diff for: src/execution/types.ts

+18-16
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,12 @@ export interface FormattedExecutionResult<
3737
export interface ExperimentalIncrementalExecutionResults<
3838
TInitial = ObjMap<unknown>,
3939
TData = ObjMap<unknown>,
40-
TItems = ReadonlyArray<unknown>,
40+
TIncrementalItem = unknown,
4141
TExtensions = ObjMap<unknown>,
4242
> {
4343
initialResult: InitialIncrementalExecutionResult<TInitial, TExtensions>;
4444
subsequentResults: AsyncGenerator<
45-
SubsequentIncrementalExecutionResult<TData, TItems, TExtensions>,
45+
SubsequentIncrementalExecutionResult<TData, TIncrementalItem, TExtensions>,
4646
void,
4747
void
4848
>;
@@ -70,25 +70,27 @@ export interface FormattedInitialIncrementalExecutionResult<
7070

7171
export interface SubsequentIncrementalExecutionResult<
7272
TData = ObjMap<unknown>,
73-
TItems = ReadonlyArray<unknown>,
73+
TIncrementalItem = unknown,
7474
TExtensions = ObjMap<unknown>,
7575
> {
7676
pending?: ReadonlyArray<PendingResult>;
77-
incremental?: ReadonlyArray<IncrementalResult<TData, TItems, TExtensions>>;
77+
incremental?: ReadonlyArray<
78+
IncrementalResult<TData, TIncrementalItem, TExtensions>
79+
>;
7880
completed?: ReadonlyArray<CompletedResult>;
7981
hasNext: boolean;
8082
extensions?: TExtensions;
8183
}
8284

8385
export interface FormattedSubsequentIncrementalExecutionResult<
8486
TData = ObjMap<unknown>,
85-
TItems = ReadonlyArray<unknown>,
87+
TIncrementalItem = unknown,
8688
TExtensions = ObjMap<unknown>,
8789
> {
8890
hasNext: boolean;
8991
pending?: ReadonlyArray<PendingResult>;
9092
incremental?: ReadonlyArray<
91-
FormattedIncrementalResult<TData, TItems, TExtensions>
93+
FormattedIncrementalResult<TData, TIncrementalItem, TExtensions>
9294
>;
9395
completed?: ReadonlyArray<FormattedCompletedResult>;
9496
extensions?: TExtensions;
@@ -119,46 +121,46 @@ export interface FormattedIncrementalDeferResult<
119121
extensions?: TExtensions;
120122
}
121123

122-
interface StreamItemsRecordResult<TItems = ReadonlyArray<unknown>> {
124+
interface StreamItemsRecordResult<TIncrementalItem = unknown> {
123125
errors?: ReadonlyArray<GraphQLError>;
124-
items: TItems;
126+
items: ReadonlyArray<TIncrementalItem>;
125127
}
126128

127129
export interface IncrementalStreamResult<
128-
TItems = ReadonlyArray<unknown>,
130+
TIncrementalItem = unknown,
129131
TExtensions = ObjMap<unknown>,
130-
> extends StreamItemsRecordResult<TItems> {
132+
> extends StreamItemsRecordResult<TIncrementalItem> {
131133
id: string;
132134
subPath?: ReadonlyArray<string | number>;
133135
extensions?: TExtensions;
134136
}
135137

136138
export interface FormattedIncrementalStreamResult<
137-
TItems = Array<unknown>,
139+
TIncrementalItem = unknown,
138140
TExtensions = ObjMap<unknown>,
139141
> {
140142
errors?: ReadonlyArray<GraphQLFormattedError>;
141-
items: TItems;
143+
items: ReadonlyArray<TIncrementalItem>;
142144
id: string;
143145
subPath?: ReadonlyArray<string | number>;
144146
extensions?: TExtensions;
145147
}
146148

147149
export type IncrementalResult<
148150
TData = ObjMap<unknown>,
149-
TItems = ReadonlyArray<unknown>,
151+
TIncrementalItem = unknown,
150152
TExtensions = ObjMap<unknown>,
151153
> =
152154
| IncrementalDeferResult<TData, TExtensions>
153-
| IncrementalStreamResult<TItems, TExtensions>;
155+
| IncrementalStreamResult<TIncrementalItem, TExtensions>;
154156

155157
export type FormattedIncrementalResult<
156158
TData = ObjMap<unknown>,
157-
TItems = ReadonlyArray<unknown>,
159+
TIncrementalItem = unknown,
158160
TExtensions = ObjMap<unknown>,
159161
> =
160162
| FormattedIncrementalDeferResult<TData, TExtensions>
161-
| FormattedIncrementalStreamResult<TItems, TExtensions>;
163+
| FormattedIncrementalStreamResult<TIncrementalItem, TExtensions>;
162164

163165
export interface PendingResult {
164166
id: string;

0 commit comments

Comments
 (0)