Imperative Cache API

.changeset/selfish-islands-clean.md

@@ -0,0 +1,5 @@
+---
+'houdini': patch
+---
+
+Add experimental imperative api for cache

package.json

@@ -36,6 +36,7 @@
 		"graphql": "^15.8.0",
 		"prettier": "^2.7.0",
 		"turbo": "^1.5.5",
+		"typescript": "^4.9",
 		"vite": "^4.0.1",
 		"vitest": "^0.23.4"
 	},

packages/houdini/src/codegen/generators/artifacts/artifacts.test.ts

@@ -3051,7 +3051,7 @@ describe('mutation artifacts', function () {
 			        fields: {
 			            users: {
 			                type: "User",
-			                keyRaw: "users(stringValue: $value, boolValue: true, floatValue: 1.2, intValue: 1)",
+			                keyRaw: "users(boolValue: true, floatValue: 1.2, intValue: 1, stringValue: $value)",
 
 			                list: {
 			                    name: "All_Users",
@@ -3522,7 +3522,7 @@ describe('mutation artifacts', function () {
 			        fields: {
 			            users: {
 			                type: "User",
-			                keyRaw: "users(stringValue: $value, boolValue: true, floatValue: 1.2, intValue: 1)",
+			                keyRaw: "users(boolValue: true, floatValue: 1.2, intValue: 1, stringValue: $value)",
 
 			                list: {
 			                    name: "All_Users",
@@ -3627,7 +3627,7 @@ describe('mutation artifacts', function () {
 			        fields: {
 			            users: {
 			                type: "User",
-			                keyRaw: "users(stringValue: $value, boolValue: true, floatValue: 1.2, intValue: 1)",
+			                keyRaw: "users(boolValue: true, floatValue: 1.2, intValue: 1, stringValue: $value)",
 
 			                list: {
 			                    name: "All_Users",
@@ -3851,7 +3851,7 @@ test('operation inputs', async function () {
 		        fields: {
 		            user: {
 		                type: "User",
-		                keyRaw: "user(id: $id, filter: $filter, filterList: $filterList, enumArg: $enumArg)",
+		                keyRaw: "user(enumArg: $enumArg, filter: $filter, filterList: $filterList, id: $id)",
 		                nullable: true,
 
 		                selection: {

packages/houdini/src/codegen/generators/artifacts/fieldKey.ts

@@ -44,11 +44,12 @@ export default function fieldKey(config: Config, field: graphql.FieldNode): stri
 		}
 	}, {})
 
+	const args = Object.keys(argObj)
+	args.sort()
+
 	let key =
 		Object.values(argObj).length > 0
-			? `${attributeName}(${Object.entries(argObj)
-					.map((entries) => entries.join(': '))
-					.join(', ')})`
+			? `${attributeName}(${args.map((key) => `${key}: ${argObj[key]}`).join(', ')})`
 			: attributeName
 
 	// if the field is paginated, key it differently so other documents can ask for the non paginated value without conflict

packages/houdini/src/codegen/generators/artifacts/operations.ts

@@ -179,7 +179,7 @@ function operationObject({
 			operationWhen[i ? 'must_not' : 'must'] = arg.value.fields.reduce(
 				(obj, arg) => ({
 					...obj,
-					[arg.name.value]: convertValue(arg.value).value,
+					[arg.name.value]: convertValue(config, arg.value).value,
 				}),
 				{}
 			)
@@ -203,7 +203,7 @@ function operationObject({
 			operationWhen[which] = directive.arguments?.reduce(
 				(filters, argument) => ({
 					...filters,
-					[argument.name.value]: convertValue(argument.value).value,
+					[argument.name.value]: convertValue(config, argument.value).value,
 				}),
 				{}
 			)

packages/houdini/src/codegen/generators/artifacts/pagination.test.ts

@@ -649,7 +649,7 @@ test("sibling aliases don't get marked", async function () {
 
 		            friends: {
 		                type: "UserConnection",
-		                keyRaw: "friends(first: 10, filter: \\"hello\\")",
+		                keyRaw: "friends(filter: \\"hello\\", first: 10)",
 
 		                selection: {
 		                    fields: {

packages/houdini/src/codegen/generators/artifacts/selection.ts

@@ -273,7 +273,7 @@ export default function selection({
 				fieldObj.filters = (field.arguments || []).reduce(
 					(filters, arg) => ({
 						...filters,
-						[arg.name.value]: convertValue(arg.value),
+						[arg.name.value]: convertValue(config, arg.value),
 					}),
 					{}
 				)

packages/houdini/src/codegen/generators/artifacts/utils.ts

@@ -2,7 +2,7 @@ import type { ExpressionKind } from 'ast-types/lib/gen/kinds'
 import * as graphql from 'graphql'
 import * as recast from 'recast'
 
-import { HoudiniError } from '../../../lib'
+import { Config, HoudiniError } from '../../../lib'
 
 const AST = recast.types.builders
 
@@ -81,12 +81,12 @@ export function deepMerge(filepath: string, ...targets: {}[]): {} {
 	)
 }
 
-export function convertValue(val: graphql.ValueNode) {
+export function convertValue(config: Config, val: graphql.ValueNode) {
 	// figure out the value to use
 	let value
 	let kind
 
-	// the value of the arg is always going to be a
+	// the value of the arg is always going to be a scalar
 	if (val.kind === graphql.Kind.INT) {
 		value = parseInt(val.value, 10)
 		kind = 'Int'

packages/houdini/src/codegen/generators/typescript/addReferencedInputTypes.ts

@@ -15,7 +15,7 @@ export function addReferencedInputTypes(
 	body: StatementKind[],
 	visitedTypes: Set<string>,
 	missingScalars: Set<string>,
-	rootType: graphql.TypeNode
+	rootType: graphql.TypeNode | graphql.GraphQLInputType
 ) {
 	// try to find the name of the type
 	const { type } = unwrapType(config, rootType)