More fixes for outbound variables (#1332)

.changeset/thick-chicken-impress.md

@@ -0,0 +1,5 @@
+---
+'houdini': patch
+---
+
+More fixes for outbound unused variables

packages/houdini/src/runtime/client/documentStore.test.ts

@@ -1175,6 +1175,7 @@ export function createStore(
 		pipeline: createPluginHooks(plugins),
 		artifact: {
 			kind: ArtifactKind.Query,
+			stripVariables: [],
 			hash: '1234',
 			raw: 'RAW_TEXT',
 			name: 'TestArtifact',
@@ -1209,6 +1210,7 @@ export function createFragmentStore(
 		client,
 		pipeline: createPluginHooks(plugins),
 		artifact: {
+			stripVariables: [],
 			kind: ArtifactKind.Fragment,
 			hash: '1234',
 			raw: 'RAW_TEXT',
@@ -1244,6 +1246,7 @@ function createStoreMutation(
 		pipeline: createPluginHooks(plugins),
 		artifact: {
 			kind: ArtifactKind.Mutation,
+			stripVariables: [],
 			hash: '1234',
 			raw: 'RAW_TEXT',
 			name: 'TestArtifact_Mutation',

packages/houdini/src/runtime/client/plugins/fetch.ts

@@ -19,7 +19,12 @@ export const fetch = (target?: RequestHandler | string): ClientPlugin => {
 					name: ctx.name,
 					text: ctx.text,
 					hash: ctx.hash,
-					variables: marshalVariables(ctx),
+					variables: { ...marshalVariables(ctx) },
+				}
+
+				// before we move onto the next plugin, we need to strip the variables as they go through
+				for (const variable of ctx.artifact.stripVariables) {
+					delete fetchParams.variables[variable]
 				}
 
 				let fetchFn = defaultFetch(client.url, ctx.fetchParams)

packages/houdini/src/runtime/client/plugins/fetchParams.ts

@@ -7,13 +7,6 @@ export const fetchParams: (fn?: FetchParamFn) => ClientPlugin =
 	(fn = () => ({})) =>
 	() => ({
 		start(ctx, { next, marshalVariables }) {
-			// before we move onto the next plugin, we need to strip the variables as they go through
-			if (ctx.variables) {
-				for (const variable of ctx.artifact.stripVariables) {
-					delete ctx.variables[variable]
-				}
-			}
-
 			next({
 				...ctx,
 				fetchParams: fn({