delete var

MaxwellBo · Jan 9, 2025 · 7a63736 · 7a63736
1 parent 93e41cd
commit 7a63736
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/celine/deno.json b/celine/deno.json
@@ -1,6 +1,6 @@
 {
   "name": "@celine/celine",
-  "version": "4.2.0",
+  "version": "4.3.0",
   "exports": "./mod.ts",
   "tasks": {
     "dev": "deno run --watch mod.ts"

diff --git a/celine/mod.ts b/celine/mod.ts
@@ -175,13 +175,11 @@ export class CelineModule {
     maybeDefinition?: Definition
   ): void {
     const observer = observerVisibility === "visible" ? this.observer(name) : undefined
-    const variable = this.module._scope.get(name) ||
-      this.module.variable(observer);
-    // if you declare B before A such that B depends on A
-    // I _think_ the Observable runtime declares a sort of "pseudeo-variable" for A
-    // which does not have an Observer attached to it
-    // Thus, we need to force the Observer to be attached to the variable
-    variable._observer = observer;
+    const variable = this.module._scope.get(name) || this.module.variable(observer);
+
+    if (Symbol.keyFor(variable._observer) == 'no-observer' && observerVisibility) {
+      variable.delete();
+    }
 
     const inputs: Inputs = Array.isArray(inputsOrDefinition)
       ? inputsOrDefinition