Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Identify written results using processed StoreObject in StoreWriter#processSelectionSet #8996

Merged
merged 16 commits into from
Nov 3, 2021
Merged

Identify written results using processed StoreObject in StoreWriter#processSelectionSet #8996

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
5d01289
Decompose and export makeReadFieldFunction.
benjamn Oct 19, 2021
d3c5280
Use readField wrapper function in processSelectionSet.
benjamn Oct 25, 2021
4c5cb3d
Move policies.identify after field traversal in processSelectionSet.
benjamn Oct 22, 2021
043df16
Make policies.identify take a KeyFieldsContext.
benjamn Oct 22, 2021
fd33a40
Simplify keyFields extraction by removing AliasMap logic.
benjamn Oct 22, 2021
b5c6fab
Test that nested keyFields objects are stably stringified.
benjamn Sep 28, 2021
b880fcc
Export normalizeReadFieldOptions rather than makeReadFieldFunction.
benjamn Oct 25, 2021
f25b3d2
Assemble complete KeyFieldsContext from partial policies.identify input.
benjamn Oct 25, 2021
634e21b
Tolerate policies.identify failure when dataId already provided.
benjamn Oct 27, 2021
39d4f0f
Rely on existence of context.readField in keyFieldsFnFromSpecifier.
benjamn Nov 1, 2021
40fbe62
Hoist normalize function out of extractKeyPath.
benjamn Nov 3, 2021
1d01a67
Slightly better return type for collectSpecifierPaths.
benjamn Nov 3, 2021
ed125f3
Move array logic into extractKeyPath reducer to simplify extractKey.
benjamn Nov 3, 2021
cd99118
Cache immutable KeySpecifier-derived information to improve performance.
benjamn Nov 3, 2021
280ab72
Bump bundlesize limit to 28.45kB (now 28.44kB).
benjamn Nov 1, 2021
ce170ed
Mention PR #8996 in CHANGELOG.md.
benjamn Nov 3, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Slightly better return type for collectSpecifierPaths.
  • Loading branch information
@benjamn
benjamn committed Nov 3, 2021
commit 1d01a67f9e81d9cff1400f29c9019963261b8fac
4 changes: 2 additions & 2 deletions src/cache/inmemory/key-extractor.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ export function keyArgsFnFromSpecifier(specifier: KeySpecifier): KeyArgsFunction
export function collectSpecifierPaths(
specifier: KeySpecifier,
extractor: (path: string[]) => any,
) {
): Record<string, any> {
// For each path specified by specifier, invoke the extractor, and repeatedly
// merge the results together, with appropriate ancestor context.
const merger = new DeepMerger;
Expand Down Expand Up @@ -205,7 +205,7 @@ function normalize<T>(value: T): T {
return collectSpecifierPaths(
Object.keys(value).sort(),
path => extractKeyPath(value, path),
);
) as T;
}
return value;
}
Expand Down