Merge pull request #317 from appwrite/chore-fix-audit-and-dependencies

chore: update dependencies

package.json

@@ -23,8 +23,8 @@
         "@appwrite.io/pink": "^0.0.4",
         "@aw-labs/appwrite-console": "^13.1.0",
         "@popperjs/core": "^2.11.6",
-        "@sentry/svelte": "^7.36.0",
-        "@sentry/tracing": "^7.36.0",
+        "@sentry/svelte": "^7.44.2",
+        "@sentry/tracing": "^7.44.2",
         "analytics": "^0.8.1",
         "echarts": "^5.4.1",
         "logrocket": "^3.0.1",
@@ -34,34 +34,34 @@
         "web-vitals": "^3.1.1"
     },
     "devDependencies": {
-        "@playwright/test": "^1.30.0",
-        "@sveltejs/adapter-static": "1.0.6",
-        "@sveltejs/kit": "^1.3.10",
-        "@sveltejs/vite-plugin-svelte": "^2.0.2",
-        "@testing-library/dom": "^8.20.0",
+        "@playwright/test": "^1.31.2",
+        "@sveltejs/adapter-static": "2.0.1",
+        "@sveltejs/kit": "1.13.0",
+        "@sveltejs/vite-plugin-svelte": "^2.0.3",
+        "@testing-library/dom": "^9.0.1",
         "@testing-library/jest-dom": "^5.16.5",
         "@testing-library/svelte": "^3.2.2",
         "@testing-library/user-event": "^14.4.3",
         "@types/prismjs": "^1.26.0",
-        "@typescript-eslint/eslint-plugin": "^5.50.0",
-        "@typescript-eslint/parser": "^5.50.0",
-        "@vitest/ui": "^0.28.5",
-        "eslint": "^8.33.0",
-        "eslint-config-prettier": "^8.6.0",
+        "@typescript-eslint/eslint-plugin": "^5.56.0",
+        "@typescript-eslint/parser": "^5.56.0",
+        "@vitest/ui": "^0.29.7",
+        "eslint": "^8.36.0",
+        "eslint-config-prettier": "^8.8.0",
         "eslint-plugin-svelte3": "^4.0.0",
-        "jsdom": "^21.1.0",
+        "jsdom": "^21.1.1",
         "pre-commit": "^1.2.2",
-        "prettier": "^2.8.3",
+        "prettier": "^2.8.6",
         "prettier-plugin-svelte": "^2.9.0",
-        "sass": "^1.58.0",
-        "svelte": "^3.55.1",
-        "svelte-check": "^3.0.3",
+        "sass": "^1.59.3",
+        "svelte": "^3.57.0",
+        "svelte-check": "^3.1.4",
         "svelte-jester": "^2.3.2",
-        "svelte-preprocess": "^5.0.1",
+        "svelte-preprocess": "^5.0.3",
         "tslib": "^2.5.0",
-        "typescript": "^4.9.5",
-        "vite": "^4.1.1",
-        "vitest": "^0.28.5"
+        "typescript": "^5.0.2",
+        "vite": "^4.2.1",
+        "vitest": "^0.29.7"
     },
     "type": "module",
     "pre-commit": [

src/lib/components/card.svelte

@@ -1,16 +1,31 @@
 <script lang="ts">
     import { clickOnEnter } from '$lib/helpers/a11y';
 
+    type BaseProps = {
+        isTile?: boolean;
+        isDashed?: boolean;
+        danger?: boolean;
+    };
+
+    type ButtonProps = {
+        isButton: true;
+        href?: never;
+    };
+
+    type AnchorProps = {
+        href: string;
+        isButton?: never;
+    };
+
+    type $$Props = BaseProps & (ButtonProps | AnchorProps | BaseProps);
+
     export let isTile = false;
     export let isDashed = false;
     export let isButton = false;
     export let danger = false;
     export let href: string = null;
 
-    //allows to add the href attribute without throwing an error
-    let attributes = { href } as Record<string, string>;
-
-    function getElement(): string {
+    function getElement() {
         switch (true) {
             case !!href:
                 return 'a';
@@ -30,6 +45,6 @@
     class:is-danger={danger}
     on:click
     on:keyup={clickOnEnter}
-    {...attributes}>
+    {href}>
     <slot />
 </svelte:element>

src/lib/components/dropListLink.svelte

@@ -4,13 +4,10 @@
     export let href: string;
     export let icon: string = null;
     export let disabled = false;
-
-    //allows to add the disabled attribute to <a> tag without throwing an error
-    let attributes = { disabled } as Record<string, boolean>;
 </script>
 
 <li class="drop-list-item" on:click on:keyup={clickOnEnter}>
-    <a {href} class="drop-button" {...attributes}>
+    <a {href} class="drop-button" class:is-disabled={disabled}>
         <span class="text"><slot /></span>
         {#if icon}
             <span class={`icon-${icon}`} aria-hidden="true" />

src/lib/elements/forms/button.svelte

@@ -23,9 +23,6 @@
 
     $: internalDisabled = (submit && $isSubmitting) || disabled;
 
-    //allows to add the disabled attribute to <a> tag without throwing an error
-    $: attributes = { internalDisabled } as Record<string, boolean>;
-
     function track() {
         if (!event) {
             return;
@@ -40,11 +37,11 @@
 {#if href}
     <a
         on:click={track}
-        {...attributes}
         {href}
         target={external ? '_blank' : ''}
         rel={external ? 'noopener noreferrer' : ''}
         class="button"
+        class:is-disabled={disabled}
         class:is-only-icon={round}
         class:is-secondary={secondary}
         class:is-text={text}

src/lib/elements/pill.svelte

@@ -11,9 +11,6 @@
     export let href: string = null;
     export let trim = false;
 
-    //allows to add the disabled attribute to <a> tag without throwing an error
-    let attributes = { disabled } as Record<string, boolean>;
-
     let element: HTMLButtonElement;
 
     const isOverflowing = (elem: HTMLButtonElement, iterator = 1) => {
@@ -44,7 +41,6 @@
 
 {#if href}
     <a
-        {...attributes}
         {href}
         target={external ? '_blank' : '_self'}
         rel={external ? 'noopener noreferrer' : ''}