houdini-svelte 0.17.6 → 0.17.7

package/.turbo/turbo-compile.log

@@ -1,5 +1,5 @@
-houdini-svelte:compile: cache hit, replaying output 0a2f71d41e809878
+houdini-svelte:compile: cache hit, replaying output c48e722ecc9917d1
 houdini-svelte:compile: 
-houdini-svelte:compile: > houdini-svelte@0.17.6 compile /home/runner/work/houdini/houdini/packages/houdini-svelte
+houdini-svelte:compile: > houdini-svelte@0.17.7 compile /home/runner/work/houdini/houdini/packages/houdini-svelte
 houdini-svelte:compile: > scripts build --plugin
 houdini-svelte:compile: 

package/.turbo/turbo-typedefs.log

@@ -1,5 +1,5 @@
-houdini-svelte:typedefs: cache hit, replaying output a06350fabda49747
+houdini-svelte:typedefs: cache hit, replaying output 228bbb4f90c4c367
 houdini-svelte:typedefs: 
-houdini-svelte:typedefs: > houdini-svelte@0.17.6 typedefs /home/runner/work/houdini/houdini/packages/houdini-svelte
+houdini-svelte:typedefs: > houdini-svelte@0.17.7 typedefs /home/runner/work/houdini/houdini/packages/houdini-svelte
 houdini-svelte:typedefs: > scripts typedefs --plugin
 houdini-svelte:typedefs: 

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # houdini-svelte
 
+## 0.17.7
+
+### 🐛 Fixes
+
+-   [#689](https://github.com/HoudiniGraphql/houdini/pull/689) [`db67e82`](https://github.com/HoudiniGraphql/houdini/commit/db67e822fcec2362538bda548d058fa7e3397ffa) Thanks [@AlecAivazis](https://github.com/AlecAivazis)! - Fix duplicate import for generated subscription stores
+
 ## 0.17.6
 
 ### 🐛 Fixes

package/build/plugin-cjs/index.js

@@ -179965,7 +179965,7 @@ ${paginationMethod ? `import _PaginationArtifact from '${config2.artifactImportP
     config2.paginationQueryName(doc.name)
   )}'` : ""}
 
-// create the query store
+// create the fragment store
 
 export class ${storeName} extends ${store_class} {
     constructor() {
@@ -180132,7 +180132,6 @@ async function subscriptionStore({ config: config2, plugin_root }, doc) {
   const artifactName = `${doc.name}`;
   const { store_class, statement } = store_import2(config2, "subscription");
   const storeContent = `${statement}
-import { SubscriptionStore } from '../runtime/stores'
 
 export class ${storeName} extends ${store_class} {
 	constructor() {

package/build/plugin-esm/index.js

@@ -179961,7 +179961,7 @@ ${paginationMethod ? `import _PaginationArtifact from '${config2.artifactImportP
     config2.paginationQueryName(doc.name)
   )}'` : ""}
 
-// create the query store
+// create the fragment store
 
 export class ${storeName} extends ${store_class} {
     constructor() {
@@ -180128,7 +180128,6 @@ async function subscriptionStore({ config: config2, plugin_root }, doc) {
   const artifactName = `${doc.name}`;
   const { store_class, statement } = store_import2(config2, "subscription");
   const storeContent = `${statement}
-import { SubscriptionStore } from '../runtime/stores'
 
 export class ${storeName} extends ${store_class} {
 	constructor() {

package/build/test-cjs/index.js

@@ -294525,7 +294525,7 @@ ${paginationMethod ? `import _PaginationArtifact from '${config3.artifactImportP
     config3.paginationQueryName(doc.name)
   )}'` : ""}
 
-// create the query store
+// create the fragment store
 
 export class ${storeName} extends ${store_class} {
     constructor() {
@@ -294692,7 +294692,6 @@ async function subscriptionStore({ config: config3, plugin_root }, doc) {
   const artifactName = `${doc.name}`;
   const { store_class, statement } = store_import2(config3, "subscription");
   const storeContent = `${statement}
-import { SubscriptionStore } from '../runtime/stores'
 
 export class ${storeName} extends ${store_class} {
 	constructor() {

package/build/test-esm/index.js

@@ -294514,7 +294514,7 @@ ${paginationMethod ? `import _PaginationArtifact from '${config3.artifactImportP
     config3.paginationQueryName(doc.name)
   )}'` : ""}
 
-// create the query store
+// create the fragment store
 
 export class ${storeName} extends ${store_class} {
     constructor() {
@@ -294681,7 +294681,6 @@ async function subscriptionStore({ config: config3, plugin_root }, doc) {
   const artifactName = `${doc.name}`;
   const { store_class, statement } = store_import2(config3, "subscription");
   const storeContent = `${statement}
-import { SubscriptionStore } from '../runtime/stores'
 
 export class ${storeName} extends ${store_class} {
 	constructor() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "houdini-svelte",
-  "version": "0.17.6",
+  "version": "0.17.7",
   "description": "The svelte plugin for houdini",
   "type": "module",
   "dependencies": {
@@ -9,7 +9,7 @@
     "ast-types": "^0.15.1",
     "estree-walker": "^3.0.1",
     "graphql": "^15.8.0",
-    "houdini": "^0.17.6",
+    "houdini": "^0.17.7",
     "minimatch": "^5.1.0",
     "recast": "^0.21.5",
     "svelte": "^3.52.0"

package/src/plugin/codegen/stores/fragment.test.ts

@@ -0,0 +1,62 @@
+import { fs, CollectedGraphQLDocument, path } from 'houdini'
+import { mockCollectedDoc } from 'houdini/test'
+import * as recast from 'recast'
+import * as typeScriptParser from 'recast/parsers/typescript'
+import { test, expect } from 'vitest'
+
+import runPipeline from '..'
+import '../..'
+import { test_config } from '../../../test'
+import { stores_directory } from '../../kit'
+
+test('generates a store for every query', async function () {
+	const config = await test_config()
+	const plugin_root = config.pluginDirectory('test-plugin')
+
+	// the documents to test
+	const docs: CollectedGraphQLDocument[] = [
+		mockCollectedDoc(`fragment TestFragment1 on User { id }`),
+		mockCollectedDoc(`fragment TestFragment2 on User { id }`),
+	]
+
+	// execute the generator
+	await runPipeline({ config, documents: docs, plugin_root, framework: 'kit' })
+
+	// look up the files in the artifact directory
+	const files = await fs.readdir(stores_directory(plugin_root))
+
+	// and they have the right names
+	expect(files).toEqual(expect.arrayContaining(['TestFragment1.js', 'TestFragment2.js']))
+	// and type definitions exist
+	expect(files).toEqual(expect.arrayContaining(['TestFragment1.d.ts', 'TestFragment2.d.ts']))
+
+	const contents = await fs.readFile(path.join(stores_directory(plugin_root), 'TestFragment1.js'))
+	const parsed = recast.parse(contents!, {
+		parser: typeScriptParser,
+	}).program
+
+	await expect(parsed).toMatchInlineSnapshot(
+		`
+		import { FragmentStore } from '$houdini/plugins/houdini-svelte/runtime/stores'
+		import artifact from '$houdini/artifacts/TestFragment1'
+
+
+		// create the fragment store
+
+		export class TestFragment1Store extends FragmentStore {
+		    constructor() {
+		        super({
+					artifact,
+					storeName: "TestFragment1Store",
+					variables: true,
+					
+				})
+			}
+		}
+
+		export const GQL_TestFragment1 = new TestFragment1Store()
+
+		export default GQL_TestFragment1
+	`
+	)
+})

package/src/plugin/codegen/stores/fragment.ts

@@ -38,7 +38,7 @@ ${
 		: ''
 }
 
-// create the query store
+// create the fragment store
 
 export class ${storeName} extends ${store_class} {
     constructor() {

package/src/plugin/codegen/stores/mutation.test.ts

@@ -0,0 +1,56 @@
+import { fs, CollectedGraphQLDocument, path } from 'houdini'
+import { mockCollectedDoc } from 'houdini/test'
+import * as recast from 'recast'
+import * as typeScriptParser from 'recast/parsers/typescript'
+import { test, expect } from 'vitest'
+
+import runPipeline from '..'
+import '../..'
+import { test_config } from '../../../test'
+import { stores_directory } from '../../kit'
+
+test('generates a store for every query', async function () {
+	const config = await test_config()
+	const plugin_root = config.pluginDirectory('test-plugin')
+
+	// the documents to test
+	const docs: CollectedGraphQLDocument[] = [
+		mockCollectedDoc(`mutation TestMutation1 { updateUser { id }  }`),
+		mockCollectedDoc(`mutation TestMutation2 { updateUser { id }  }`),
+	]
+
+	// execute the generator
+	await runPipeline({ config, documents: docs, plugin_root, framework: 'kit' })
+
+	// look up the files in the artifact directory
+	const files = await fs.readdir(stores_directory(plugin_root))
+
+	// and they have the right names
+	expect(files).toEqual(expect.arrayContaining(['TestMutation1.js', 'TestMutation2.js']))
+	// and type definitions exist
+	expect(files).toEqual(expect.arrayContaining(['TestMutation1.d.ts', 'TestMutation2.d.ts']))
+
+	const contents = await fs.readFile(path.join(stores_directory(plugin_root), 'TestMutation1.js'))
+	const parsed = recast.parse(contents!, {
+		parser: typeScriptParser,
+	}).program
+
+	await expect(parsed).toMatchInlineSnapshot(
+		`
+		import artifact from '$houdini/artifacts/TestMutation1'
+		import { MutationStore } from '$houdini/plugins/houdini-svelte/runtime/stores'
+
+		export class TestMutation1Store extends MutationStore {
+			constructor() {
+				super({
+					artifact,
+				})
+			}
+		}
+
+		export const GQL_TestMutation1 = new TestMutation1Store()
+
+		export default GQL_TestMutation1
+	`
+	)
+})

package/src/plugin/codegen/stores/query.test.ts

@@ -9,7 +9,7 @@ import '../..'
 import { pipeline_test, test_config } from '../../../test'
 import { stores_directory } from '../../kit'
 
-test('generates a store for every query', async function () {
+test('generates a query store for every query', async function () {
 	const config = await test_config()
 	const plugin_root = config.pluginDirectory('test-plugin')
 

package/src/plugin/codegen/stores/subscription.test.ts

@@ -0,0 +1,59 @@
+import { fs, CollectedGraphQLDocument, path } from 'houdini'
+import { mockCollectedDoc } from 'houdini/test'
+import * as recast from 'recast'
+import * as typeScriptParser from 'recast/parsers/typescript'
+import { test, expect } from 'vitest'
+
+import runPipeline from '..'
+import '../..'
+import { test_config } from '../../../test'
+import { stores_directory } from '../../kit'
+
+test('generates a store for every query', async function () {
+	const config = await test_config()
+	const plugin_root = config.pluginDirectory('test-plugin')
+
+	// the documents to test
+	const docs: CollectedGraphQLDocument[] = [
+		mockCollectedDoc(`subscription TestSubscription1 { newUser { id }  }`),
+		mockCollectedDoc(`subscription TestSubscription2 { newUser { id }  }`),
+	]
+
+	// execute the generator
+	await runPipeline({ config, documents: docs, plugin_root, framework: 'kit' })
+
+	// look up the files in the artifact directory
+	const files = await fs.readdir(stores_directory(plugin_root))
+
+	// and they have the right names
+	expect(files).toEqual(expect.arrayContaining(['TestSubscription1.js', 'TestSubscription2.js']))
+	// and type definitions exist
+	expect(files).toEqual(
+		expect.arrayContaining(['TestSubscription1.d.ts', 'TestSubscription2.d.ts'])
+	)
+
+	const contents = await fs.readFile(
+		path.join(stores_directory(plugin_root), 'TestSubscription1.js')
+	)
+	const parsed = recast.parse(contents!, {
+		parser: typeScriptParser,
+	}).program
+
+	await expect(parsed).toMatchInlineSnapshot(
+		`
+		import { SubscriptionStore } from '$houdini/plugins/houdini-svelte/runtime/stores'
+
+		export class TestSubscription1Store extends SubscriptionStore {
+			constructor() {
+				super({
+					artifact,
+				})
+			}
+		}
+
+		export const GQL_TestSubscription1 = new TestSubscription1Store()
+
+		export default GQL_TestSubscription1
+	`
+	)
+})

package/src/plugin/codegen/stores/subscription.ts

@@ -17,7 +17,6 @@ export async function subscriptionStore(
 
 	// the content of the store
 	const storeContent = `${statement}
-import { SubscriptionStore } from '../runtime/stores'
 
 export class ${storeName} extends ${store_class} {
 	constructor() {