Batch of fixes during epic-stack implementation (#203)

* make ENV readonly without making process.env readonly

* use compact json in what gets passed in with varlock/load

* adjust loading logic in browser-like test env

this fixes an issue for vitest runnign jsdom tests and needing to 
execute backend logic first

* allow if(condition) resoler to coerce to boolean

* allow @required/@sensitive to accept undefined

* add isEmpty() and not() resolvers

Author: theoephraim

.changeset/all-shoes-fall.md

@@ -0,0 +1,5 @@
+---
+"varlock": patch
+---
+
+allow if() to take 1 arg to coerce to boolean

.changeset/famous-experts-beg.md

@@ -0,0 +1,5 @@
+---
+"varlock": patch
+---
+
+allow @required/@sensitive to accept undefined

.changeset/red-clocks-strive.md

@@ -0,0 +1,5 @@
+---
+"varlock": patch
+---
+
+make ENV readonly without making process.env readonly

.changeset/slimy-cows-obey.md

@@ -0,0 +1,5 @@
+---
+"varlock": patch
+---
+
+adjust loading behavior for browser testing (vitest jsdom)

.changeset/smooth-bears-join.md

@@ -0,0 +1,5 @@
+---
+"varlock": patch
+---
+
+add not() and isEmpty() resolvers

packages/varlock-website/src/content/docs/reference/functions.mdx

@@ -165,4 +165,29 @@ API_URL=if(eq($GIT_BRANCH, "main"), api.example.com, staging-api.example.com)
 ```
 </div>
 
+<div>
+### `not()`
+
+Negates a value and returns a boolean. Falsy values are - `false`, `""`, `0`, `undefined`, and will be negated to `true`. Otherwise will return `false`.
+
+```env-spec "not"
+# Negate the result of another function
+SHOULD_DISABLE_FEATURE=not(forEnv(production))
+```
+</div>
+
+<div>
+### `isEmpty()`
+
+Returns `true` if the value is `undefined` or an empty string, `false` otherwise.
+
+```env-spec "isEmpty"
+# Check if a value is empty
+HAS_API_KEY=not(isEmpty($API_KEY))
+
+# Use with conditional logic
+API_URL=if(isEmpty($CUSTOM_API_URL), "https://api.default.com", $CUSTOM_API_URL)
+```
+</div>
+
 </div>

packages/varlock/src/auto-load.ts

@@ -11,7 +11,7 @@ import { patchGlobalResponse } from './runtime/patch-response';
 // this also isolates the varlock loading process from the end user process
 
 
-const execResult = execSyncVarlock('load --format json-full', {
+const execResult = execSyncVarlock('load --format json-full-compact', {
   exitOnError: true,
   showLogsOnError: true,
 });

packages/varlock/src/cli/commands/load.command.ts

@@ -15,7 +15,7 @@ export const commandSpec = define({
     format: {
       type: 'enum',
       short: 'f',
-      choices: ['pretty', 'json', 'env', 'json-full'],
+      choices: ['pretty', 'json', 'env', 'json-full', 'json-full-compact'],
       description: 'Format of output',
       default: 'pretty',
     },
@@ -70,8 +70,8 @@ export const commandFn: TypedGunshiCommandFn<typeof commandSpec> = async (ctx) =
     }
   } else if (format === 'json') {
     console.log(JSON.stringify(envGraph.getResolvedEnvObject(), null, 2));
-  } else if (format === 'json-full') {
-    console.log(JSON.stringify(envGraph.getSerializedGraph(), null, 2));
+  } else if (format === 'json-full' || format === 'json-full-compact') {
+    console.log(JSON.stringify(envGraph.getSerializedGraph(), null, format === 'json-full-compact' ? 0 : 2));
   } else if (format === 'env') {
     const resolvedEnv = envGraph.getResolvedEnvObject();
     for (const key in resolvedEnv) {

packages/varlock/src/env-graph/lib/config-item.ts

@@ -273,15 +273,25 @@ export class ConfigItem {
           const usingOptional = requiredDec.name === 'optional';
 
           // need to track if required-ness is dynamic
-          this._isRequiredDynamic = requiredDec.decValueResolver
-            ? requiredDec.decValueResolver.staticValue === undefined : false;
+          // NOTE - if any other resolver ever has a static value, we'll need to change this
+          if (requiredDec.decValueResolver?.fnName !== '\0static') {
+            this._isRequiredDynamic = true;
+          }
 
           const requiredDecoratorVal = await requiredDec.resolve();
-          if (!_.isBoolean(requiredDecoratorVal)) {
-            throw new SchemaError('@required/@optional must resolve to a boolean');
+          // if we got an error, we'll bail and the error will be checked later
+          if (requiredDec.schemaErrors.length) {
+            // but we mark as not required so we don't _also_ get required error
+            this._isRequired = false;
+            return;
+          }
+          if (![true, false, undefined].includes(requiredDecoratorVal)) {
+            throw new SchemaError('@required/@optional must resolve to a boolean or undefined');
+          }
+          if (requiredDecoratorVal !== undefined) {
+            this._isRequired = usingOptional ? !requiredDecoratorVal : requiredDecoratorVal;
+            return;
           }
-          this._isRequired = usingOptional ? !requiredDecoratorVal : requiredDecoratorVal;
-          return;
         }
 
         // Root-level @defaultRequired
@@ -329,6 +339,13 @@ export class ConfigItem {
       const sensitiveDec = def.itemDef.decorators?.find((d) => d.name === 'sensitive');
       if (sensitiveDec) {
         const sensitiveDecValue = await sensitiveDec.resolve();
+        // can bail if the decorator value resolution failed
+        if (sensitiveDec.schemaErrors.length) {
+          return;
+        }
+        if (![true, false, undefined].includes(sensitiveDecValue)) {
+          throw new SchemaError('@sensitive must resolve to a boolean or undefined');
+        }
         if (sensitiveDecValue !== undefined) {
           this._isSensitive = sensitiveDecValue;
           return;

packages/varlock/src/env-graph/lib/decorators.ts

@@ -103,6 +103,7 @@ export abstract class DecoratorInstance {
       this.resolvedValue = await this.decValueResolver.resolve();
     } catch (err) {
       this._schemaErrors.push(err as any);
+      return;
     }
 
     this.isResolved = true;