@valbuild/next 0.126.0 → 0.128.0

package/CHANGELOG.md

@@ -1,5 +1,54 @@
 # @valbuild/next
 
+## 0.128.0
+
+### Patch Changes
+
+- Updated dependencies [[`8b52b33`](https://github.com/valbuild/val/commit/8b52b33e1f629f14f66cc71dcc7cf415d210c7d4), [`362fb49`](https://github.com/valbuild/val/commit/362fb49f30d2b04c4ff78d54dca2bf5ad978a05c), [`c595799`](https://github.com/valbuild/val/commit/c59579977a436ec530c6c69f2b340ab9829d97ab)]:
+  - @valbuild/core@0.128.0
+  - @valbuild/language-server@0.128.0
+  - @valbuild/mcp@0.128.0
+  - @valbuild/react@0.128.0
+  - @valbuild/server@0.128.0
+  - @valbuild/shared@0.128.0
+  - @valbuild/ui@0.127.0
+
+## 0.127.0
+
+### Patch Changes
+
+- [#665](https://github.com/valbuild/val/pull/665) [`7072e07`](https://github.com/valbuild/val/commit/7072e07623c953a09ac14388ae22dada0b431ce3) Thanks [@freekh](https://github.com/freekh)! - `.nullable()` no longer drops the field's `.validate(...)` functions.
+
+  `.nullable()` returns a copy of the schema, and most of the schema classes built
+  that copy with an empty list of custom validators — so a validator declared
+  before the `.nullable()` was silently thrown away:
+
+  ```ts
+  s.number()
+    .validate((n) => (n > 100 ? "Too big" : false))
+    .nullable(); // the validator never ran
+  ```
+
+  `array`, `object`, `discriminatedUnion`, `enum`, `number`, `boolean`, `literal`,
+  `keyOf`, `date`, `dateTime`, `code`, `color` and `richtext` were affected;
+  `string`, `record`, `route`, `file` and `image` already carried them over, which
+  is why the bug was easy to miss. Validators now survive `.nullable()` on every
+  schema, in either order.
+
+  A validator on a nullable schema is called with `null` when the value is unset,
+  rather than being skipped — the same thing the schemas that never dropped them
+  have always done. If yours was written before the `.nullable()`, its argument is
+  typed as non-null even though `null` can reach it, so guard for it.
+
+- Updated dependencies [[`7fa8699`](https://github.com/valbuild/val/commit/7fa869974bc895af992a7d5c18b76253636d7d65), [`600308d`](https://github.com/valbuild/val/commit/600308d0174990ad9f5c417147d160273489c65a), [`5b7fe05`](https://github.com/valbuild/val/commit/5b7fe05cec6365f9cd1de9ba31e65ed4a87edb63), [`88262ac`](https://github.com/valbuild/val/commit/88262ac8db068650a664981ef73556457d87741a), [`7072e07`](https://github.com/valbuild/val/commit/7072e07623c953a09ac14388ae22dada0b431ce3), [`29811c3`](https://github.com/valbuild/val/commit/29811c3f8c7e001a950e6f4833af6888e6a4efea), [`9983116`](https://github.com/valbuild/val/commit/99831164c5151aad7ca69de79e1d0d59878be251)]:
+  - @valbuild/core@0.127.0
+  - @valbuild/shared@0.127.0
+  - @valbuild/ui@0.127.0
+  - @valbuild/server@0.127.0
+  - @valbuild/react@0.127.0
+  - @valbuild/language-server@0.127.0
+  - @valbuild/mcp@0.127.0
+
 ## 0.126.0
 
 ### Patch Changes

package/README.md

@@ -385,6 +385,16 @@ import { s } from "./val.config";
 s.string().nullable(); // <- Schema<string | null>
 ```
 
+`.nullable()` can go before or after `.validate(...)` — a validator declared on
+either side of it is kept, and runs on `null` too, so the validator decides for
+itself what an unset value means:
+
+```ts
+s.string()
+  .nullable()
+  .validate((val) => (val === null ? "Please fill this in" : false));
+```
+
 ## Read-only and hidden fields
 
 `.readonly()` renders a field disabled in the Val editor, and `.hidden()` leaves
@@ -1328,6 +1338,27 @@ s.string().validate((val) => {
 });
 ```
 
+`.validate(...)` and `.nullable()` can be written in either order: the validator
+is carried through the copy `.nullable()` makes. On a nullable schema the value
+reaching the validator can be `null`, and it is handed over rather than skipped —
+so a validator declared **before** the `.nullable()` has to guard for it, since
+its argument is still typed as non-null there:
+
+```ts
+s.string()
+  .validate((val) => (val !== null && val.length > 80 ? "Too long" : false))
+  .nullable();
+
+// Declared after, and the argument is typed `string | null`:
+s.string()
+  .nullable()
+  .validate((val) => (val !== null && val.length > 80 ? "Too long" : false));
+```
+
+Not every modifier is order-free, though: a record's `.jsonValues()` changes the
+source shape, so it must come **before** `.validate(...)` and `.preview(...)`,
+and throws with that message if it does not.
+
 ## Get in touch
 
 Join us on [discord](https://discord.gg/cZzqPvaX8k) to get help or give us feedback.

package/client/dist/valbuild-next-client.cjs.dev.js

@@ -7,7 +7,7 @@ var routeFromVal = require('../../dist/routeFromVal-d53d5ffb.cjs.dev.js');
 var core = require('@valbuild/core');
 var stega = require('@valbuild/react/stega');
 var React = require('react');
-var ValOverlayContext = require('../../dist/ValOverlayContext-5bd7628b.cjs.dev.js');
+var ValOverlayContext = require('../../dist/ValOverlayContext-98d33d86.cjs.dev.js');
 require('../../dist/createForOfIteratorHelper-0445603c.cjs.dev.js');
 require('../../dist/unsupportedIterableToArray-c8ab77c9.cjs.dev.js');
 require('../../dist/objectSpread2-58024783.cjs.dev.js');

package/client/dist/valbuild-next-client.cjs.prod.js

@@ -7,7 +7,7 @@ var routeFromVal = require('../../dist/routeFromVal-e0995830.cjs.prod.js');
 var core = require('@valbuild/core');
 var stega = require('@valbuild/react/stega');
 var React = require('react');
-var ValOverlayContext = require('../../dist/ValOverlayContext-d690744f.cjs.prod.js');
+var ValOverlayContext = require('../../dist/ValOverlayContext-9c4ae06d.cjs.prod.js');
 require('../../dist/createForOfIteratorHelper-d4afcad8.cjs.prod.js');
 require('../../dist/unsupportedIterableToArray-0d2087a2.cjs.prod.js');
 require('../../dist/objectSpread2-13f847a9.cjs.prod.js');

package/client/dist/valbuild-next-client.esm.js

@@ -3,7 +3,7 @@ import { g as getJsonEntryStegaRoot, _ as _typeof, a as isJsonValuesRecordSchema
 import { Internal } from '@valbuild/core';
 import { getModuleIds, stegaEncode } from '@valbuild/react/stega';
 import React from 'react';
-import { useValOverlayContext } from '../../dist/ValOverlayContext-e42a2e14.esm.js';
+import { useValOverlayContext } from '../../dist/ValOverlayContext-398baef3.esm.js';
 import '../../dist/createForOfIteratorHelper-5758a730.esm.js';
 import '../../dist/unsupportedIterableToArray-5baabfdc.esm.js';
 import '../../dist/objectSpread2-60d1bd93.esm.js';

package/dist/ValNextProvider-062a8dc9.cjs.dev.js

@@ -11,7 +11,7 @@ var ui = require('@valbuild/ui');
 var navigation = require('next/navigation');
 var Script = require('next/script');
 var React = require('react');
-var ValOverlayContext = require('./ValOverlayContext-5bd7628b.cjs.dev.js');
+var ValOverlayContext = require('./ValOverlayContext-98d33d86.cjs.dev.js');
 var stega = require('@valbuild/react/stega');
 var fallbackColors = require('./fallbackColors-080c1bbc.cjs.dev.js');
 var client = require('@valbuild/shared/client');

package/dist/ValNextProvider-3d2cd7e4.esm.js

@@ -7,7 +7,7 @@ import { VERSION, VAL_APP_PATH, VAL_OVERLAY_ID } from '@valbuild/ui';
 import { useRouter, usePathname } from 'next/navigation';
 import Script from 'next/script';
 import React, { useEffect } from 'react';
-import { ValExternalStore, ValOverlayProvider } from './ValOverlayContext-e42a2e14.esm.js';
+import { ValExternalStore, ValOverlayProvider } from './ValOverlayContext-398baef3.esm.js';
 import { SET_AUTO_TAG_JSX_ENABLED } from '@valbuild/react/stega';
 import { p as prefixStyles, u as useConfigStorageSave, f as floatDarkBg, d as floatLightBg, v as valPrefixedClass, c as cn } from './fallbackColors-149dfbc9.esm.js';
 import { VAL_THEME_SESSION_STORAGE_KEY, isValCanvasFrame } from '@valbuild/shared/client';

package/dist/ValNextProvider-da189afe.cjs.prod.js

@@ -11,7 +11,7 @@ var ui = require('@valbuild/ui');
 var navigation = require('next/navigation');
 var Script = require('next/script');
 var React = require('react');
-var ValOverlayContext = require('./ValOverlayContext-d690744f.cjs.prod.js');
+var ValOverlayContext = require('./ValOverlayContext-9c4ae06d.cjs.prod.js');
 var stega = require('@valbuild/react/stega');
 var fallbackColors = require('./fallbackColors-e1d73b36.cjs.prod.js');
 var client = require('@valbuild/shared/client');

package/dist/{ValOverlayContext-e42a2e14.esm.js → ValOverlayContext-398baef3.esm.js}

@@ -24,20 +24,22 @@ function _createClass(e, r, t) {
 
 var LOAD_TIMEOUT_MS = 10000;
 var ValExternalStore = /*#__PURE__*/function () {
-  // Path-keyed cache of every source seen via update(). The single source of
-  // truth: load state and snapshots are both derived from this, so data can be
-  // queried for any combination of paths even before a subscriber registers.
-
-  // Reference-stable per-subscriberId snapshot cache for useSyncExternalStore.
-  // Built lazily from loadedSources in get() and invalidated in update().
-
-  // One-shot listeners used solely by waitForLoad. Kept separate from the
-  // useSyncExternalStore listeners so waitForLoad never has to register (and
-  // leak) a subscriberId.
-
   function ValExternalStore() {
     var _this = this;
     _classCallCheck(this, ValExternalStore);
+    _defineProperty(this, "listeners", void 0);
+    _defineProperty(this, "loadPromises", void 0);
+    // Path-keyed cache of every source seen via update(). The single source of
+    // truth: load state and snapshots are both derived from this, so data can be
+    // queried for any combination of paths even before a subscriber registers.
+    _defineProperty(this, "loadedSources", void 0);
+    // Reference-stable per-subscriberId snapshot cache for useSyncExternalStore.
+    // Built lazily from loadedSources in get() and invalidated in update().
+    _defineProperty(this, "snapshots", void 0);
+    // One-shot listeners used solely by waitForLoad. Kept separate from the
+    // useSyncExternalStore listeners so waitForLoad never has to register (and
+    // leak) a subscriberId.
+    _defineProperty(this, "loadListeners", void 0);
     _defineProperty(this, "subscribe", function (paths) {
       return function (listener) {
         var subscriberId = createSubscriberId(paths);

package/dist/{ValOverlayContext-5bd7628b.cjs.dev.js → ValOverlayContext-98d33d86.cjs.dev.js}

@@ -32,20 +32,22 @@ function _createClass(e, r, t) {
 
 var LOAD_TIMEOUT_MS = 10000;
 var ValExternalStore = /*#__PURE__*/function () {
-  // Path-keyed cache of every source seen via update(). The single source of
-  // truth: load state and snapshots are both derived from this, so data can be
-  // queried for any combination of paths even before a subscriber registers.
-
-  // Reference-stable per-subscriberId snapshot cache for useSyncExternalStore.
-  // Built lazily from loadedSources in get() and invalidated in update().
-
-  // One-shot listeners used solely by waitForLoad. Kept separate from the
-  // useSyncExternalStore listeners so waitForLoad never has to register (and
-  // leak) a subscriberId.
-
   function ValExternalStore() {
     var _this = this;
     _classCallCheck(this, ValExternalStore);
+    defineProperty._defineProperty(this, "listeners", void 0);
+    defineProperty._defineProperty(this, "loadPromises", void 0);
+    // Path-keyed cache of every source seen via update(). The single source of
+    // truth: load state and snapshots are both derived from this, so data can be
+    // queried for any combination of paths even before a subscriber registers.
+    defineProperty._defineProperty(this, "loadedSources", void 0);
+    // Reference-stable per-subscriberId snapshot cache for useSyncExternalStore.
+    // Built lazily from loadedSources in get() and invalidated in update().
+    defineProperty._defineProperty(this, "snapshots", void 0);
+    // One-shot listeners used solely by waitForLoad. Kept separate from the
+    // useSyncExternalStore listeners so waitForLoad never has to register (and
+    // leak) a subscriberId.
+    defineProperty._defineProperty(this, "loadListeners", void 0);
     defineProperty._defineProperty(this, "subscribe", function (paths) {
       return function (listener) {
         var subscriberId = createSubscriberId(paths);

package/dist/ValOverlayContext-9c4ae06d.cjs.js

@@ -0,0 +1,7 @@
+'use strict';
+
+if (process.env.NODE_ENV === "production") {
+  module.exports = require("./ValOverlayContext-9c4ae06d.cjs.prod.js");
+} else {
+  module.exports = require("./ValOverlayContext-9c4ae06d.cjs.dev.js");
+}

package/dist/{ValOverlayContext-d690744f.cjs.prod.js → ValOverlayContext-9c4ae06d.cjs.prod.js}

@@ -32,20 +32,22 @@ function _createClass(e, r, t) {
 
 var LOAD_TIMEOUT_MS = 10000;
 var ValExternalStore = /*#__PURE__*/function () {
-  // Path-keyed cache of every source seen via update(). The single source of
-  // truth: load state and snapshots are both derived from this, so data can be
-  // queried for any combination of paths even before a subscriber registers.
-
-  // Reference-stable per-subscriberId snapshot cache for useSyncExternalStore.
-  // Built lazily from loadedSources in get() and invalidated in update().
-
-  // One-shot listeners used solely by waitForLoad. Kept separate from the
-  // useSyncExternalStore listeners so waitForLoad never has to register (and
-  // leak) a subscriberId.
-
   function ValExternalStore() {
     var _this = this;
     _classCallCheck(this, ValExternalStore);
+    defineProperty._defineProperty(this, "listeners", void 0);
+    defineProperty._defineProperty(this, "loadPromises", void 0);
+    // Path-keyed cache of every source seen via update(). The single source of
+    // truth: load state and snapshots are both derived from this, so data can be
+    // queried for any combination of paths even before a subscriber registers.
+    defineProperty._defineProperty(this, "loadedSources", void 0);
+    // Reference-stable per-subscriberId snapshot cache for useSyncExternalStore.
+    // Built lazily from loadedSources in get() and invalidated in update().
+    defineProperty._defineProperty(this, "snapshots", void 0);
+    // One-shot listeners used solely by waitForLoad. Kept separate from the
+    // useSyncExternalStore listeners so waitForLoad never has to register (and
+    // leak) a subscriberId.
+    defineProperty._defineProperty(this, "loadListeners", void 0);
     defineProperty._defineProperty(this, "subscribe", function (paths) {
       return function (listener) {
         var subscriberId = createSubscriberId(paths);

package/dist/valbuild-next.cjs.dev.js

@@ -12,7 +12,7 @@ var NextImage = require('next/image');
 var jsxRuntime = require('react/jsx-runtime');
 var ValApp = require('./ValApp-294e5b53.cjs.dev.js');
 var ValModulesClient = require('./ValModulesClient-4ecabf99.cjs.dev.js');
-var version = require('./version-7b12f65c.cjs.dev.js');
+var version = require('./version-bf879c9b.cjs.dev.js');
 var createForOfIteratorHelper = require('./createForOfIteratorHelper-0445603c.cjs.dev.js');
 require('./defineProperty-f90345d7.cjs.dev.js');
 require('./unsupportedIterableToArray-c8ab77c9.cjs.dev.js');

package/dist/valbuild-next.cjs.prod.js

@@ -12,7 +12,7 @@ var NextImage = require('next/image');
 var jsxRuntime = require('react/jsx-runtime');
 var ValApp = require('./ValApp-ab8a96fd.cjs.prod.js');
 var ValModulesClient = require('./ValModulesClient-10c0721c.cjs.prod.js');
-var version = require('./version-ed4739ee.cjs.prod.js');
+var version = require('./version-4edf83c0.cjs.prod.js');
 var createForOfIteratorHelper = require('./createForOfIteratorHelper-d4afcad8.cjs.prod.js');
 require('./defineProperty-8951f469.cjs.prod.js');
 require('./unsupportedIterableToArray-0d2087a2.cjs.prod.js');

package/dist/valbuild-next.esm.js

@@ -11,7 +11,7 @@ import NextImage from 'next/image';
 import { jsx } from 'react/jsx-runtime';
 export { ValApp } from './ValApp-3a70afe1.esm.js';
 export { ValModulesClient, useRegisterValModules } from './ValModulesClient-0dd601a1.esm.js';
-import { V as VERSION } from './version-f8b10637.esm.js';
+import { V as VERSION } from './version-c1db43d1.esm.js';
 import { _ as _createForOfIteratorHelper } from './createForOfIteratorHelper-5758a730.esm.js';
 import './defineProperty-cca5affa.esm.js';
 import './unsupportedIterableToArray-5baabfdc.esm.js';

package/dist/{version-7b12f65c.cjs.dev.js → version-4edf83c0.cjs.prod.js}

@@ -14,7 +14,7 @@ var packageJson = {
 		"next",
 		"react"
 	],
-	version: "0.126.0",
+	version: "0.128.0",
 	scripts: {
 		typecheck: "tsc --noEmit",
 		test: "jest"

package/dist/{version-ed4739ee.cjs.prod.js → version-bf879c9b.cjs.dev.js}

@@ -14,7 +14,7 @@ var packageJson = {
 		"next",
 		"react"
 	],
-	version: "0.126.0",
+	version: "0.128.0",
 	scripts: {
 		typecheck: "tsc --noEmit",
 		test: "jest"

package/dist/{version-f8b10637.esm.js → version-c1db43d1.esm.js}

@@ -12,7 +12,7 @@ var packageJson = {
 		"next",
 		"react"
 	],
-	version: "0.126.0",
+	version: "0.128.0",
 	scripts: {
 		typecheck: "tsc --noEmit",
 		test: "jest"

package/package.json

@@ -12,7 +12,7 @@
     "next",
     "react"
   ],
-  "version": "0.126.0",
+  "version": "0.128.0",
   "main": "dist/valbuild-next.cjs.js",
   "module": "dist/valbuild-next.esm.js",
   "exports": {
@@ -47,13 +47,13 @@
   "dependencies": {
     "client-only": "^0.0.1",
     "server-only": "^0.0.1",
-    "@valbuild/core": "0.126.0",
-    "@valbuild/language-server": "0.126.0",
-    "@valbuild/react": "0.126.0",
-    "@valbuild/mcp": "0.126.0",
-    "@valbuild/server": "0.126.0",
-    "@valbuild/shared": "0.126.0",
-    "@valbuild/ui": "0.126.0"
+    "@valbuild/core": "0.128.0",
+    "@valbuild/language-server": "0.128.0",
+    "@valbuild/mcp": "0.128.0",
+    "@valbuild/react": "0.128.0",
+    "@valbuild/server": "0.128.0",
+    "@valbuild/shared": "0.128.0",
+    "@valbuild/ui": "0.127.0"
   },
   "devDependencies": {
     "@testing-library/react": "^16.3.3",

package/rsc/dist/valbuild-next-rsc.cjs.dev.js

@@ -11,7 +11,7 @@ var stega = require('@valbuild/react/stega');
 var core = require('@valbuild/core');
 var internal = require('@valbuild/shared/internal');
 var server = require('@valbuild/server');
-var version = require('../../dist/version-7b12f65c.cjs.dev.js');
+var version = require('../../dist/version-bf879c9b.cjs.dev.js');
 require('../../dist/createForOfIteratorHelper-0445603c.cjs.dev.js');
 require('../../dist/unsupportedIterableToArray-c8ab77c9.cjs.dev.js');
 require('../../dist/slicedToArray-44036a76.cjs.dev.js');

package/rsc/dist/valbuild-next-rsc.cjs.prod.js

@@ -11,7 +11,7 @@ var stega = require('@valbuild/react/stega');
 var core = require('@valbuild/core');
 var internal = require('@valbuild/shared/internal');
 var server = require('@valbuild/server');
-var version = require('../../dist/version-ed4739ee.cjs.prod.js');
+var version = require('../../dist/version-4edf83c0.cjs.prod.js');
 require('../../dist/createForOfIteratorHelper-d4afcad8.cjs.prod.js');
 require('../../dist/unsupportedIterableToArray-0d2087a2.cjs.prod.js');
 require('../../dist/slicedToArray-ce613de6.cjs.prod.js');

package/rsc/dist/valbuild-next-rsc.esm.js

@@ -7,7 +7,7 @@ import { SET_RSC, stegaEncode, SET_AUTO_TAG_JSX_ENABLED } from '@valbuild/react/
 import { Internal } from '@valbuild/core';
 import { VAL_SESSION_COOKIE } from '@valbuild/shared/internal';
 import { createValServer } from '@valbuild/server';
-import { V as VERSION } from '../../dist/version-f8b10637.esm.js';
+import { V as VERSION } from '../../dist/version-c1db43d1.esm.js';
 import '../../dist/createForOfIteratorHelper-5758a730.esm.js';
 import '../../dist/unsupportedIterableToArray-5baabfdc.esm.js';
 import '../../dist/slicedToArray-aa291011.esm.js';

package/server/dist/valbuild-next-server.cjs.dev.js

@@ -9,7 +9,7 @@ var objectSpread2 = require('../../dist/objectSpread2-58024783.cjs.dev.js');
 var core = require('@valbuild/core');
 var server = require('@valbuild/server');
 var server$1 = require('next/server');
-var version = require('../../dist/version-7b12f65c.cjs.dev.js');
+var version = require('../../dist/version-bf879c9b.cjs.dev.js');
 var mcp = require('@valbuild/mcp');
 require('../../dist/unsupportedIterableToArray-c8ab77c9.cjs.dev.js');
 require('../../dist/defineProperty-f90345d7.cjs.dev.js');

package/server/dist/valbuild-next-server.cjs.prod.js

@@ -9,7 +9,7 @@ var objectSpread2 = require('../../dist/objectSpread2-13f847a9.cjs.prod.js');
 var core = require('@valbuild/core');
 var server = require('@valbuild/server');
 var server$1 = require('next/server');
-var version = require('../../dist/version-ed4739ee.cjs.prod.js');
+var version = require('../../dist/version-4edf83c0.cjs.prod.js');
 var mcp = require('@valbuild/mcp');
 require('../../dist/unsupportedIterableToArray-0d2087a2.cjs.prod.js');
 require('../../dist/defineProperty-8951f469.cjs.prod.js');

package/server/dist/valbuild-next-server.esm.js

@@ -5,7 +5,7 @@ import { _ as _objectSpread2 } from '../../dist/objectSpread2-60d1bd93.esm.js';
 import { Internal } from '@valbuild/core';
 import { createValApiRouter, createValServer } from '@valbuild/server';
 import { NextResponse } from 'next/server';
-import { V as VERSION } from '../../dist/version-f8b10637.esm.js';
+import { V as VERSION } from '../../dist/version-c1db43d1.esm.js';
 import { initValMcp as initValMcp$1 } from '@valbuild/mcp';
 import '../../dist/unsupportedIterableToArray-5baabfdc.esm.js';
 import '../../dist/defineProperty-cca5affa.esm.js';

package/dist/ValOverlayContext-d690744f.cjs.js

@@ -1,7 +0,0 @@
-'use strict';
-
-if (process.env.NODE_ENV === "production") {
-  module.exports = require("./ValOverlayContext-d690744f.cjs.prod.js");
-} else {
-  module.exports = require("./ValOverlayContext-d690744f.cjs.dev.js");
-}