From 8e03b6588ebefbe5279d68cd69871f3448740146 Mon Sep 17 00:00:00 2001
From: Matthias <tuttiacc@hotmail.com>
Date: Thu, 5 Dec 2024 18:41:41 +0100
Subject: [PATCH 1/2] backport svelte race condition

---
 ...{{pascal_case collection_name}}.svelte.hbs |  6 ++++--
 ...{{pascal_case entry_type.name}}.svelte.hbs |  8 +++++---
 ....svelte{{\302\241if}}{{\302\241each}}.hbs" |  7 ++++---
 ...ntry_type.name}}.svelte{{\302\241if}}.hbs" |  8 +++++---
 ...al_case entry_type.name}}Detail.svelte.hbs |  6 ++++--
 ...renceable.name}}.svelte{{\302\241if}}.hbs" |  6 ++++--
 ...renceable.name}}.svelte{{\302\241if}}.hbs" |  6 ++++--
 .../svelte/web-app/ui/src/App.svelte.hbs      | 19 +++++++++++++------
 .../svelte/web-app/ui/src/contexts.ts.hbs     |  8 +++++++-
 9 files changed, 50 insertions(+), 24 deletions(-)

diff --git a/templates/ui-frameworks/svelte/collection/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case collection_name}}.svelte.hbs b/templates/ui-frameworks/svelte/collection/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case collection_name}}.svelte.hbs
index 56a099e95..75a44471e 100644
--- a/templates/ui-frameworks/svelte/collection/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case collection_name}}.svelte.hbs	
+++ b/templates/ui-frameworks/svelte/collection/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case collection_name}}.svelte.hbs	
@@ -10,11 +10,12 @@ import type {
   NewEntryAction,
   HolochainError,
 } from '@holochain/client';
-import { clientContext } from '../../contexts';
+import { type ClientContext, clientContext } from '../../contexts';
 import {{pascal_case referenceable.name}}Detail from './{{pascal_case referenceable.name}}Detail.svelte';
 import type { {{pascal_case coordinator_zome_manifest.name}}Signal } from './types';
 
-const client: AppClient = (getContext(clientContext) as any).getClient();
+let client: AppClient;
+const appClientContext = getContext<ClientContext>(clientContext);
 
 let hashes: Array<{{referenceable.hash_type}}> = [];
 let loading = false;
@@ -31,6 +32,7 @@ onMount(async () => {
       throw new Error(`The author input is required for the {{pascal_case collection_name}} element`);
     }
   {{/if}}
+  client = await appClientContext.getClient();
   await fetch{{pascal_case (plural referenceable.name)}}();
   client.on('signal', signal => {
     if (signal.zome_name !== '{{coordinator_zome_manifest.name}}') return;
diff --git a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/Create{{pascal_case entry_type.name}}.svelte.hbs b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/Create{{pascal_case entry_type.name}}.svelte.hbs
index cd1c9eae6..a0c7a6edf 100644
--- a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/Create{{pascal_case entry_type.name}}.svelte.hbs	
+++ b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/Create{{pascal_case entry_type.name}}.svelte.hbs	
@@ -1,7 +1,7 @@
 <script lang="ts">
 import { createEventDispatcher, getContext, onMount } from 'svelte';
 import type { AppClient, Record, EntryHash, AgentPubKey, ActionHash, DnaHash, HolochainError } from '@holochain/client';
-import { clientContext } from '../../contexts';
+import { type ClientContext, clientContext } from '../../contexts';
 import type { {{pascal_case entry_type.name}}{{#each entry_type.fields}}{{#if (eq field_type.type "Enum")}}, {{field_type.label}}{{/if}}{{/each}} } from './types';
 {{#uniq_lines}}
   {{#each entry_type.fields}}
@@ -13,7 +13,8 @@ import type { {{pascal_case entry_type.name}}{{#each entry_type.fields}}{{#if (e
 {{/uniq_lines}}
 
 const dispatch = createEventDispatcher();
-const client: AppClient = (getContext(clientContext) as any).getClient();
+let client: AppClient;
+const appClientContext = getContext<ClientContext>(clientContext);
 
 {{#each entry_type.fields}}
   {{#if widget }}
@@ -38,7 +39,7 @@ export let {{camel_case field_name}}{{#if (eq cardinality "single")}}!{{/if}}: {
 $: {{#each entry_type.fields}}{{camel_case field_name}}{{#unless @last}}, {{/unless}}{{/each}};
 $: is{{pascal_case entry_type.name}}Valid = true{{#each entry_type.fields}}{{#if widget}}{{#if (eq cardinality "single")}} && {{> (concat field_type.type "/" widget "/is-valid") variable_to_validate=(camel_case field_name) }}{{/if}}{{#if (eq cardinality "vector")}} && {{camel_case field_name}}.every(e => {{> (concat field_type.type "/" widget "/is-valid") variable_to_validate="e" }}){{/if}}{{/if}}{{/each}};
 
-onMount(() => {
+onMount(async () => {
 {{#each entry_type.fields}}
   {{#if (not widget) }}
     {{#if (ne cardinality "option")}}
@@ -48,6 +49,7 @@ onMount(() => {
     {{/if}}
   {{/if}}
 {{/each}}
+  client = await appClientContext.getClient();
 });
 
 async function create{{pascal_case entry_type.name}}() {
diff --git "a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#each entry_type.fields}}{{#if (and linked_from (not (eq linked_from.hash_type 'AgentPubKey') ) )}}{{pascal_case (plural ..\302\241entry_type.name)}}For{{pascal_case linked_from.name}}.svelte{{\302\241if}}{{\302\241each}}.hbs" "b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#each entry_type.fields}}{{#if (and linked_from (not (eq linked_from.hash_type 'AgentPubKey') ) )}}{{pascal_case (plural ..\302\241entry_type.name)}}For{{pascal_case linked_from.name}}.svelte{{\302\241if}}{{\302\241each}}.hbs"
index 77ec8bb56..592afb251 100644
--- "a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#each entry_type.fields}}{{#if (and linked_from (not (eq linked_from.hash_type 'AgentPubKey') ) )}}{{pascal_case (plural ..\302\241entry_type.name)}}For{{pascal_case linked_from.name}}.svelte{{\302\241if}}{{\302\241each}}.hbs"	
+++ "b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#each entry_type.fields}}{{#if (and linked_from (not (eq linked_from.hash_type 'AgentPubKey') ) )}}{{pascal_case (plural ..\302\241entry_type.name)}}For{{pascal_case linked_from.name}}.svelte{{\302\241if}}{{\302\241each}}.hbs"	
@@ -10,11 +10,12 @@ import type {
   NewEntryAction,
   HolochainError,
 } from '@holochain/client';
-import { clientContext } from '../../contexts';
+import { type ClientContext, clientContext } from '../../contexts';
 import type { {{pascal_case ../entry_type.name}}, {{pascal_case ../coordinator_zome_manifest.name}}Signal } from './types';
 import {{pascal_case ../entry_type.name}}Detail from './{{pascal_case ../entry_type.name}}Detail.svelte';
 
-const client: AppClient = (getContext(clientContext) as any).getClient();
+let client: AppClient;
+const appClientContext = getContext<ClientContext>(clientContext);
 
 let hashes: Array<ActionHash> | undefined = [];
 let loading: boolean;
@@ -28,7 +29,7 @@ onMount(async () => {
   if ({{camel_case linked_from.singular_arg}} === undefined) {
     throw new Error(`The {{camel_case linked_from.singular_arg}} input is required for the {{pascal_case (plural ../entry_type.name)}}For{{pascal_case linked_from.name}} element`);
   }
-
+  client = await appClientContext.getClient();
   await fetch{{pascal_case (plural ../entry_type.name)}}();
 
   client.on('signal', async signal => {
diff --git "a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if crud.update}}Edit{{pascal_case entry_type.name}}.svelte{{\302\241if}}.hbs" "b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if crud.update}}Edit{{pascal_case entry_type.name}}.svelte{{\302\241if}}.hbs"
index c45ce5ee3..f7b9f8198 100644
--- "a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if crud.update}}Edit{{pascal_case entry_type.name}}.svelte{{\302\241if}}.hbs"	
+++ "b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if crud.update}}Edit{{pascal_case entry_type.name}}.svelte{{\302\241if}}.hbs"	
@@ -2,7 +2,7 @@
 import { createEventDispatcher, getContext, onMount } from 'svelte';
 import type { AppClient, Record, EntryHash, AgentPubKey, DnaHash, ActionHash, HolochainError } from '@holochain/client';
 import { decode } from '@msgpack/msgpack';
-import { clientContext } from '../../contexts';
+import { type ClientContext, clientContext } from '../../contexts';
 import type { {{pascal_case entry_type.name}}{{#each entry_type.fields}}{{#if (eq field_type.type "Enum")}}, {{field_type.label}}{{/if}}{{/each}} } from './types';
 {{#uniq_lines}}
   {{#each entry_type.fields}}
@@ -13,7 +13,8 @@ import type { {{pascal_case entry_type.name}}{{#each entry_type.fields}}{{#if (e
   {{/each}}
 {{/uniq_lines}}
 
-const client: AppClient = (getContext(clientContext) as any).getClient();
+let client: AppClient;
+const appClientContext = getContext<ClientContext>(clientContext);
 const dispatch = createEventDispatcher();
 
 export let currentRecord!: Record;
@@ -35,7 +36,7 @@ let {{camel_case field_name}}: Array<{{> (concat field_type.type "/type")}} | un
 $: {{#each (filter entry_type.fields "widget")}}{{camel_case field_name}}{{#unless @last}}, {{/unless}}{{/each}};
 $: is{{pascal_case entry_type.name}}Valid = true{{#each entry_type.fields}}{{#if widget}}{{#if (eq cardinality "single")}} && {{> (concat field_type.type "/" widget "/is-valid") variable_to_validate=(camel_case field_name) }}{{/if}}{{#if (eq cardinality "vector")}} && {{camel_case field_name}}.every(e => {{> (concat field_type.type "/" widget "/is-valid") variable_to_validate="e" }}){{/if}}{{/if}}{{/each}};
 
-onMount(() => {
+onMount(async () => {
   if (!currentRecord) {
     throw new Error(`The currentRecord input is required for the Edit{{pascal_case entry_type.name}} element`);
   }
@@ -44,6 +45,7 @@ onMount(() => {
     throw new Error(`The original{{pascal_case entry_type.name}}Hash input is required for the Edit{{pascal_case entry_type.name}} element`);
   }
 {{/if}}
+  client = await appClientContext.getClient();
 });
 
 async function update{{pascal_case entry_type.name}}() {
diff --git a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case entry_type.name}}Detail.svelte.hbs b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case entry_type.name}}Detail.svelte.hbs
index 90ecc97f5..e6d60265b 100644
--- a/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case entry_type.name}}Detail.svelte.hbs	
+++ b/templates/ui-frameworks/svelte/entry-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{pascal_case entry_type.name}}Detail.svelte.hbs	
@@ -2,7 +2,7 @@
 import { createEventDispatcher, onMount, getContext } from 'svelte';
 import { decode } from '@msgpack/msgpack';
 import type { Record, ActionHash, AppClient, EntryHash, AgentPubKey, DnaHash, HolochainError } from '@holochain/client';
-import { clientContext } from '../../contexts';
+import { type ClientContext, clientContext } from '../../contexts';
 import type { {{pascal_case entry_type.name}}{{#each entry_type.fields}}{{#if (eq field_type.type "Enum")}}, {{field_type.label}}{{/if}}{{/each}} } from './types';
 {{#if crud.update}}
 import Edit{{pascal_case entry_type.name}} from './Edit{{pascal_case entry_type.name}}.svelte';
@@ -16,7 +16,8 @@ import Edit{{pascal_case entry_type.name}} from './Edit{{pascal_case entry_type.
   {{/each}}
 {{/uniq_lines}}
 
-const client: AppClient = (getContext(clientContext) as any).getClient();
+let client: AppClient;
+const appClientContext = getContext<ClientContext>(clientContext);
 const dispatch = createEventDispatcher();
 
 let loading: boolean = false;
@@ -35,6 +36,7 @@ onMount(async () => {
   if ({{camel_case entry_type.name}}Hash === undefined) {
     throw new Error(`The {{camel_case entry_type.name}}Hash input is required for the {{pascal_case entry_type.name}}Detail element`);
   }
+  client = await appClientContext.getClient();
   await fetch{{pascal_case entry_type.name}}();
 });
 
diff --git "a/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and bidireccional (and to_referenceable (ne from_referenceable.hash_type 'AgentPubKey')))}}{{pascal_case (plural from_referenceable.name)}}For{{pascal_case to_referenceable.name}}.svelte{{\302\241if}}.hbs" "b/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and bidireccional (and to_referenceable (ne from_referenceable.hash_type 'AgentPubKey')))}}{{pascal_case (plural from_referenceable.name)}}For{{pascal_case to_referenceable.name}}.svelte{{\302\241if}}.hbs"
index 6489b3217..542463e94 100644
--- "a/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and bidireccional (and to_referenceable (ne from_referenceable.hash_type 'AgentPubKey')))}}{{pascal_case (plural from_referenceable.name)}}For{{pascal_case to_referenceable.name}}.svelte{{\302\241if}}.hbs"	
+++ "b/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and bidireccional (and to_referenceable (ne from_referenceable.hash_type 'AgentPubKey')))}}{{pascal_case (plural from_referenceable.name)}}For{{pascal_case to_referenceable.name}}.svelte{{\302\241if}}.hbs"	
@@ -10,11 +10,12 @@ import type {
   NewEntryAction,
   HolochainError,
 } from '@holochain/client';
-import { clientContext } from '../../contexts';
+import { type ClientContext, clientContext } from '../../contexts';
 import {{pascal_case from_referenceable.name}}Detail from './{{pascal_case from_referenceable.name}}Detail.svelte';
 import type { {{pascal_case coordinator_zome_manifest.name}}Signal } from './types';
 
-const client: AppClient = (getContext(clientContext) as any).getClient();
+let client: AppClient;
+const appClientContext = getContext<ClientContext>(clientContext);
 
 let hashes: Array<{{from_referenceable.hash_type}}> | undefined;
 let loading = false;
@@ -31,6 +32,7 @@ onMount(async () => {
 
   try {
     loading = true;
+    client = await appClientContext.getClient();
     const links: Array<Link> = await client.callZome({
       cap_secret: null,
       role_name: '{{dna_role_name}}',
diff --git "a/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and to_referenceable (ne to_referenceable.hash_type 'AgentPubKey'))}}{{pascal_case (plural to_referenceable.name)}}For{{pascal_case from_referenceable.name}}.svelte{{\302\241if}}.hbs" "b/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and to_referenceable (ne to_referenceable.hash_type 'AgentPubKey'))}}{{pascal_case (plural to_referenceable.name)}}For{{pascal_case from_referenceable.name}}.svelte{{\302\241if}}.hbs"
index 7f8792a7c..40d112646 100644
--- "a/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and to_referenceable (ne to_referenceable.hash_type 'AgentPubKey'))}}{{pascal_case (plural to_referenceable.name)}}For{{pascal_case from_referenceable.name}}.svelte{{\302\241if}}.hbs"	
+++ "b/templates/ui-frameworks/svelte/link-type/ui/src/{{dna_role_name}}/{{coordinator_zome_manifest.name}}/{{#if (and to_referenceable (ne to_referenceable.hash_type 'AgentPubKey'))}}{{pascal_case (plural to_referenceable.name)}}For{{pascal_case from_referenceable.name}}.svelte{{\302\241if}}.hbs"	
@@ -10,11 +10,12 @@ import type {
   NewEntryAction,
   HolochainError,
 } from '@holochain/client';
-import { clientContext } from '../../contexts';
+import { type ClientContext, clientContext } from '../../contexts';
 import {{pascal_case to_referenceable.name}}Detail from './{{pascal_case to_referenceable.name}}Detail.svelte';
 import type { {{pascal_case coordinator_zome_manifest.name}}Signal } from './types';
 
-const client: AppClient = (getContext(clientContext) as any).getClient();
+let client: AppClient;
+const appClientContext = getContext<ClientContext>(clientContext);
 
 let hashes: Array<{{to_referenceable.hash_type}}> | undefined;
 let loading = false;
@@ -31,6 +32,7 @@ onMount(async () => {
 
   try {
     loading = true;
+    client = await appClientContext.getClient();
     const links: Array<Link> = await client.callZome({
       cap_secret: null,
       role_name: '{{dna_role_name}}',
diff --git a/templates/ui-frameworks/svelte/web-app/ui/src/App.svelte.hbs b/templates/ui-frameworks/svelte/web-app/ui/src/App.svelte.hbs
index e2748fd01..81fc46309 100644
--- a/templates/ui-frameworks/svelte/web-app/ui/src/App.svelte.hbs
+++ b/templates/ui-frameworks/svelte/web-app/ui/src/App.svelte.hbs
@@ -7,7 +7,7 @@
 {{/if}}
 
   import logo from './assets/holochainLogo.svg';
-  import { clientContext } from './contexts';
+  import { type ClientContext, clientContext } from './contexts';
 
 {{#if holo_enabled}}
   const IS_HOLO = ['true', '1', 't'].includes(import.meta.env.VITE_APP_IS_HOLO?.toLowerCase())
@@ -21,6 +21,15 @@
   let error: HolochainError | undefined;
   let loading = false;
 
+  const appClientContext = {
+    getClient: async () => {
+      if (!client) {
+        client = await AppWebsocket.connect();
+      }
+      return client;
+    },
+  };
+
   onMount(async () => {
   {{#if holo_enabled}}
     try {
@@ -37,7 +46,7 @@
         });
         (client as WebSdk).signUp({ cancellable: false });
       } else {
-        client = await AppWebsocket.connect();
+        client = await appClientContext.getClient();
         loading = false;
       }
     } catch (e) {
@@ -48,7 +57,7 @@
   {{else}}
     try {
       loading = true;
-      client = await AppWebsocket.connect();
+      client = await appClientContext.getClient();
     } catch (e) {
       error = e as HolochainError;
     } finally {
@@ -57,9 +66,7 @@
   {{/if}}
   });
 
-  setContext(clientContext, {
-    getClient: () => client,
-  });
+  setContext<ClientContext>(clientContext, appClientContext);
 </script>
 
 <main>
diff --git a/templates/ui-frameworks/svelte/web-app/ui/src/contexts.ts.hbs b/templates/ui-frameworks/svelte/web-app/ui/src/contexts.ts.hbs
index 9f41a2624..76cade143 100644
--- a/templates/ui-frameworks/svelte/web-app/ui/src/contexts.ts.hbs
+++ b/templates/ui-frameworks/svelte/web-app/ui/src/contexts.ts.hbs
@@ -1 +1,7 @@
-export const clientContext = 'AppClient';
\ No newline at end of file
+import { type AppClient } from "@holochain/client";
+
+export const clientContext = "AppClient";
+
+export type ClientContext = {
+  getClient: () => Promise<AppClient>;
+};
\ No newline at end of file

From 11c3aaaa14a293f8eb81d0dd73703f832696c01c Mon Sep 17 00:00:00 2001
From: Matthias <tuttiacc@hotmail.com>
Date: Thu, 5 Dec 2024 18:44:24 +0100
Subject: [PATCH 2/2] bump scaffolding version

---
 Cargo.lock | 2 +-
 Cargo.toml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index 47bbab13b..751744ea2 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1867,7 +1867,7 @@ dependencies = [
 
 [[package]]
 name = "holochain_scaffolding_cli"
-version = "0.3000.10"
+version = "0.3000.11"
 dependencies = [
  "anyhow",
  "build-fs-tree",
diff --git a/Cargo.toml b/Cargo.toml
index b454d9004..32673d317 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -1,7 +1,7 @@
 [package]
 edition = "2021"
 name = "holochain_scaffolding_cli"
-version = "0.3000.10"
+version = "0.3000.11"
 description = "CLI to easily generate and modify holochain apps"
 license = "CAL-1.0"
 homepage = "https://developer.holochain.org"