From 2ebafaeafce24091054d4878e89f827a97628daf Mon Sep 17 00:00:00 2001
From: Eric Kwoka <43540491+ekwoka@users.noreply.github.com>
Date: Wed, 28 Feb 2024 11:04:39 +0400
Subject: [PATCH] :label: Fixes Types

---
 packages/scope/package.json |  2 +-
 packages/scope/src/index.ts | 84 ++++++++++++++++++++-----------------
 size.json                   |  4 +-
 3 files changed, 48 insertions(+), 42 deletions(-)

diff --git a/packages/scope/package.json b/packages/scope/package.json
index 1532185..8ffcb7b 100644
--- a/packages/scope/package.json
+++ b/packages/scope/package.json
@@ -47,5 +47,5 @@
     "type": "git",
     "url": "https://github.com/ekwoka/alpine-plugins"
   },
-  "homepage": "https://github.com/ekwoka/alpine-plugins/blob/main/packages/xscope/README.md"
+  "homepage": "https://github.com/ekwoka/alpine-plugins/blob/main/packages/scope/README.md"
 }
diff --git a/packages/scope/src/index.ts b/packages/scope/src/index.ts
index aae30da..403e79f 100644
--- a/packages/scope/src/index.ts
+++ b/packages/scope/src/index.ts
@@ -1,28 +1,27 @@
-import type { Alpine } from 'alpinets';
+import type { Alpine } from 'alpinejs';
+import type { Assertion } from 'vitest';
 
 export const Scope = (Alpine: Alpine) => {
   const $scope = Symbol('$scope');
+  type Scopable = { [$scope]?: Map<string, HTMLElement> };
   Alpine.directive('scope', (el, { expression }) => {
-    const context = Alpine.$data(el);
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    const context = Alpine.$data(el) as Scopable;
 
     const rootContext = Alpine.closestDataStack(el)[0];
     if (!rootContext) return;
-    rootContext[$scope] = new Map(
-      context[$scope] as Map<string, HTMLElement> | undefined,
-    ).set(expression, el);
+    rootContext[$scope] = new Map(context[$scope]).set(expression, el);
   });
   Alpine.magic('scope', (el) => {
     return new Proxy(
       {},
       {
         get(_, name: string) {
-          const scopes = Alpine.$data(el)[$scope] as
-            | Map<string, HTMLElement>
-            | undefined;
+          const scopes = (Alpine.$data(el) as Scopable)[$scope];
           if (scopes?.has(name)) return Alpine.$data(scopes.get(name)!);
           const root = Alpine.findClosest(el, (el) =>
             el.matches(`[x-data="${name}"]`),
-          );
+          ) as HTMLElement | undefined;
           if (root) return Alpine.$data(root);
           return undefined;
         },
@@ -46,13 +45,13 @@ if (import.meta.vitest) {
       const root = await render(
         `
         <div x-data="foo">
-          <div x-data="bar">
-            <span id="naked" x-text="value"></span>
-            <span id="foo" x-text="$scope.foo?.value"></span>
-            <span id="bar" x-text="$scope.bar?.value"></span>
-          </div>
+        <div x-data="bar">
+        <span id="naked" x-text="value"></span>
+        <span id="foo" x-text="$scope.foo?.value"></span>
+        <span id="bar" x-text="$scope.bar?.value"></span>
+        </div>
         </div>
-      `.trim(),
+        `.trim(),
       )
         .withComponent('foo', () => ({ value: 'foo' }))
         .withComponent('bar', () => ({ value: 'bar' }))
@@ -70,14 +69,14 @@ if (import.meta.vitest) {
     it('can access explicitely scoped context', async () => {
       const root = await render(
         `
-        <div x-data="{ value: 'foo' }" x-scope="foo">
-          <div x-data="{ value: 'bar' }" x-scope="bar">
-            <span id="naked" x-text="value"></span>
-            <span id="foo" x-text="$scope.foo?.value"></span>
-            <span id="bar" x-text="$scope.bar?.value"></span>
-          </div>
-        </div>
-      `.trim(),
+                <div x-data="{ value: 'foo' }" x-scope="foo">
+                <div x-data="{ value: 'bar' }" x-scope="bar">
+                <span id="naked" x-text="value"></span>
+                <span id="foo" x-text="$scope.foo?.value"></span>
+                <span id="bar" x-text="$scope.bar?.value"></span>
+                </div>
+                </div>
+                `.trim(),
       ).withPlugin(Scope);
       expect((root as HTMLElement).querySelector('#naked')).toHaveTextContent(
         'bar',
@@ -92,13 +91,13 @@ if (import.meta.vitest) {
     it('favors explicitely scoped contexts', async () => {
       const root = await render(
         `
-        <div x-data="foo" x-scope="bar">
-          <div x-data="bar">
-            <span id="naked" x-text="value"></span>
-            <span id="bar" x-text="$scope.bar?.value"></span>
-          </div>
-        </div>
-      `.trim(),
+                        <div x-data="foo" x-scope="bar">
+                        <div x-data="bar">
+                        <span id="naked" x-text="value"></span>
+                        <span id="bar" x-text="$scope.bar?.value"></span>
+                        </div>
+                        </div>
+                        `.trim(),
       )
         .withComponent('foo', () => ({ value: 'foo' }))
         .withComponent('bar', () => ({ value: 'bar' }))
@@ -113,15 +112,15 @@ if (import.meta.vitest) {
     it('does not leak', async () => {
       const root = await render(
         `
-        <div x-data="{ value: 'root' }">
-        <div x-data="foo" x-scope="foo"></div>
-        <div x-data="bar">
-        </div>
-        <span id="naked" x-text="value"></span>
-        <span id="foo" x-text="$scope.foo?.value ?? 'not found'"></span>
-        <span id="bar" x-text="$scope.bar?.value ?? 'not found'"></span>
-        </div>
-      `.trim(),
+                              <div x-data="{ value: 'root' }">
+                              <div x-data="foo" x-scope="foo"></div>
+                              <div x-data="bar">
+                              </div>
+                              <span id="naked" x-text="value"></span>
+                              <span id="foo" x-text="$scope.foo?.value ?? 'not found'"></span>
+                              <span id="bar" x-text="$scope.bar?.value ?? 'not found'"></span>
+                              </div>
+                              `.trim(),
       )
         .withComponent('foo', () => ({ value: 'foo' }))
         .withComponent('bar', () => ({ value: 'bar' }))
@@ -138,3 +137,10 @@ if (import.meta.vitest) {
     });
   });
 }
+declare module 'vitest' {
+  interface Assertion<T> extends AlpineMatchers<T> {}
+}
+
+interface AlpineMatchers<T> {
+  toHaveTextContent: (expected: string) => Assertion<T>;
+}
diff --git a/size.json b/size.json
index 1c09bed..ae67b10 100644
--- a/size.json
+++ b/size.json
@@ -15,8 +15,8 @@
       "raw": 373
     },
     "brotli": {
-      "pretty": "238 B",
-      "raw": 238
+      "pretty": "236 B",
+      "raw": 236
     }
   },
   "xajax": {