fix: provide a logger implementation that only shows debug messages o…

…n debug

fixes Unnecessary logging to console #301

esbuild.config.mjs

@@ -48,6 +48,9 @@ const context = await esbuild.context({
     sourcemap: prod ? false : "inline",
     treeShaking: true,
     outfile: "main.js",
+    define: {
+        "process.env.NODE_ENV": JSON.stringify(prod ? "production" : "development"),
+    },
 });
 
 if (prod) {

src/std/index.ts

@@ -1,6 +1,6 @@
 import * as A from "fp-ts/Array";
 import {
-    Either,
+    type Either,
     ap,
     bimap,
     chainW,
@@ -18,12 +18,12 @@ import {
     tryCatch,
     tryCatchK,
 } from "fp-ts/Either";
-import { NonEmptyArray, concatAll as concatAllNea } from "fp-ts/NonEmptyArray";
+import { type NonEmptyArray, concatAll as concatAllNea } from "fp-ts/NonEmptyArray";
 import * as _O from "fp-ts/Option";
-import { Semigroup, concatAll } from "fp-ts/Semigroup";
+import { type Semigroup, concatAll } from "fp-ts/Semigroup";
 import * as TE from "fp-ts/TaskEither";
 import { absurd as _absurd, flow as f, pipe as p } from "fp-ts/function";
-import { BaseSchema, Output, ValiError, parse as parseV } from "valibot";
+import { type BaseSchema, type Output, ValiError, parse as parseV } from "valibot";
 export type Option<T> = _O.Option<T>;
 export type { Either, Left, Right } from "fp-ts/Either";
 export type { NonEmptyArray } from "fp-ts/NonEmptyArray";

src/store/formStore.ts

@@ -294,7 +294,7 @@ export function makeFormEngine({
                 },
             };
             const isVisible = derived(formStore, ($form): E.Either<string, boolean> => {
-                console.log(
+                l.debug(
                     "condition",
                     field.name,
                     field.condition && $form.fields[field.condition.dependencyName],

src/utils/Logger.ts

@@ -2,7 +2,7 @@ type Level = "DEBUG" | "INFO" | "WARN" | "ERROR";
 function noop() {}
 export class Logger {
     private static instance: Logger;
-    #level: Level = "DEBUG";
+    #level: Level = process.env.NODE_ENV === "development" ? "DEBUG" : "ERROR";
     private constructor() {}
 
     static getInstance(): Logger {

src/views/components/Form/RenderField.svelte

@@ -15,20 +15,22 @@
     import MultiSelectField from "./MultiSelectField.svelte";
     import ObsidianSelect from "./ObsidianSelect.svelte";
     import ObsidianToggle from "./ObsidianToggle.svelte";
+    import { logger as l } from "src/utils/Logger";
     import InputSlider from "./inputSlider.svelte";
 
     export let model: ReturnType<FormEngine["addField"]>;
     export let definition: FieldDefinition;
     export let formEngine: FormEngine;
     export let app: App;
+    export let logger = l;
 
     $: value = model.value;
     $: errors = model.errors;
     $: isVisible = model.isVisible;
     $: visibleError = derived(model.isVisible, ($isVisible) =>
         E.isLeft($isVisible) ? [$isVisible.left] : ([] as string[]),
     );
-    $: console.log($isVisible);
+    $: logger.debug($isVisible);
 </script>
 
 {#if E.isLeft($isVisible)}

tsconfig.json

@@ -5,7 +5,7 @@
         "inlineSources": true,
         "resolveJsonModule": true,
         "module": "ESNext",
-        "target": "ES6",
+        "target": "ES2019",
         "allowJs": true,
         "noImplicitAny": true,
         "moduleResolution": "node",
@@ -101,4 +101,4 @@
         "src/**/*",
         "jest.config.ts"
     ]
-}
+}