@sanity/validation 2.21.11-reference-updates.62 → 2.22.3-reference-updates.54

package/dist/dts/Rule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Rule.d.ts","sourceRoot":"","sources":["../../src/Rule.ts"],"names":[],"mappings":"AAAA,OAAO,EAGL,SAAS,EAUV,MAAM,eAAe,CAAA;AA0DtB,QAAA,MAAM,IAAI,EAAE,SAwVX,CAAA;AAED,eAAe,IAAI,CAAA"}
+{"version":3,"file":"Rule.d.ts","sourceRoot":"","sources":["../../src/Rule.ts"],"names":[],"mappings":"AAAA,OAAO,EAGL,SAAS,EAUV,MAAM,eAAe,CAAA;AA0DtB,QAAA,MAAM,IAAI,EAAE,SA2VX,CAAA;AAED,eAAe,IAAI,CAAA"}

package/dist/dts/util/normalizeValidationRules.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"normalizeValidationRules.d.ts","sourceRoot":"","sources":["../../../src/util/normalizeValidationRules.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,UAAU,EAAE,IAAI,EAAqB,MAAM,eAAe,CAAA;AA+ElE;;GAEG;AACH,MAAM,CAAC,OAAO,UAAU,wBAAwB,CAAC,OAAO,EAAE,UAAU,GAAG,SAAS,GAAG,IAAI,EAAE,CAqCxF"}
+{"version":3,"file":"normalizeValidationRules.d.ts","sourceRoot":"","sources":["../../../src/util/normalizeValidationRules.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,UAAU,EAAE,IAAI,EAAqB,MAAM,eAAe,CAAA;AA6ElE;;GAEG;AACH,MAAM,CAAC,OAAO,UAAU,wBAAwB,CAAC,OAAO,EAAE,UAAU,GAAG,SAAS,GAAG,IAAI,EAAE,CAqCxF"}

package/dist/dts/validateDocument.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"validateDocument.d.ts","sourceRoot":"","sources":["../../src/validateDocument.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,cAAc,EACd,MAAM,EACN,UAAU,EACV,iBAAiB,EACjB,gBAAgB,EAGjB,MAAM,eAAe,CAAA;AAYtB;;GAEG;AACH,wBAAgB,uBAAuB,CACrC,IAAI,EAAE,OAAO,EACb,UAAU,EAAE,UAAU,EAAE,GACvB,UAAU,GAAG,SAAS,CAiBxB;AAED,wBAA8B,gBAAgB,CAC5C,GAAG,EAAE,cAAc,EACnB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,IAAI,CAAC,iBAAiB,EAAE,mBAAmB,CAAC,GACrD,OAAO,CAAC,gBAAgB,EAAE,CAAC,CA2B7B;AAED;;;;;;GAMG;AACH,aAAK,iBAAiB,CAAC,CAAC,IAAI;KACzB,CAAC,IAAI,MAAM,QAAQ,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC,CAAC,EAAE,CAAC,CAAC,SAAS,QAAQ,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,SAAS;CAC5F,CAAA;AAED,aAAK,mBAAmB,GAAG;IACzB,KAAK,EAAE,OAAO,CAAA;CACf,GAAG,iBAAiB,CAAC,iBAAiB,CAAC,CAAA;AAExC,wBAAsB,YAAY,CAAC,EACjC,KAAK,EACL,IAAI,EACJ,IAAS,EACT,MAAM,EACN,GAAG,aAAa,EACjB,EAAE,mBAAmB,GAAG,OAAO,CAAC,gBAAgB,EAAE,CAAC,CA8FnD"}
+{"version":3,"file":"validateDocument.d.ts","sourceRoot":"","sources":["../../src/validateDocument.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,cAAc,EACd,MAAM,EACN,UAAU,EACV,iBAAiB,EACjB,gBAAgB,EAMjB,MAAM,eAAe,CAAA;AAYtB;;GAEG;AACH,wBAAgB,uBAAuB,CACrC,IAAI,EAAE,OAAO,EACb,UAAU,EAAE,UAAU,EAAE,GACvB,UAAU,GAAG,SAAS,CAiBxB;AAED,wBAA8B,gBAAgB,CAC5C,GAAG,EAAE,cAAc,EACnB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,IAAI,CAAC,iBAAiB,EAAE,mBAAmB,CAAC,GACrD,OAAO,CAAC,gBAAgB,EAAE,CAAC,CA2B7B;AAED;;;;;;GAMG;AACH,aAAK,iBAAiB,CAAC,CAAC,IAAI;KACzB,CAAC,IAAI,MAAM,QAAQ,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC,CAAC,EAAE,CAAC,CAAC,SAAS,QAAQ,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,SAAS;CAC5F,CAAA;AAED,aAAK,mBAAmB,GAAG;IACzB,KAAK,EAAE,OAAO,CAAA;CACf,GAAG,iBAAiB,CAAC,iBAAiB,CAAC,CAAA;AAExC,wBAAsB,YAAY,CAAC,EACjC,KAAK,EACL,IAAI,EACJ,IAAS,EACT,MAAM,EACN,GAAG,aAAa,EACjB,EAAE,mBAAmB,GAAG,OAAO,CAAC,gBAAgB,EAAE,CAAC,CA4HnD"}

package/dist/dts/validators/objectValidator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"objectValidator.d.ts","sourceRoot":"","sources":["../../../src/validators/objectValidator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,UAAU,EAAc,MAAM,eAAe,CAAA;AAKrD,QAAA,MAAM,gBAAgB,EAAE,UAiEvB,CAAA;AAED,eAAe,gBAAgB,CAAA"}
+{"version":3,"file":"objectValidator.d.ts","sourceRoot":"","sources":["../../../src/validators/objectValidator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAC,UAAU,EAAc,MAAM,eAAe,CAAA;AAKrD,QAAA,MAAM,gBAAgB,EAAE,UAwDvB,CAAA;AAED,eAAe,gBAAgB,CAAA"}

package/lib/Rule.js

@@ -232,6 +232,17 @@ var Rule = (_temp = _class = class Rule {
       constraint: fn
     }]);
   }
+  /**
+   * @deprecated use `Rule.custom` instead
+   */
+
+
+  block(fn) {
+    return this.cloneWithRules([{
+      flag: 'custom',
+      constraint: fn
+    }]);
+  }
 
   min(len) {
     return this.cloneWithRules([{
@@ -383,13 +394,6 @@ var Rule = (_temp = _class = class Rule {
     }]);
   }
 
-  block(blockValidator) {
-    return this.cloneWithRules([{
-      flag: 'block',
-      constraint: blockValidator
-    }]);
-  }
-
   fields(rules) {
     if (this._type !== 'Object') {
       throw new Error('fields() can only be called on an object type');

package/lib/index.js

@@ -9,12 +9,7 @@ Object.defineProperty(exports, "Rule", {
     return _Rule.default;
   }
 });
-Object.defineProperty(exports, "validateDocument", {
-  enumerable: true,
-  get: function get() {
-    return _validateDocument.default;
-  }
-});
+exports.default = void 0;
 Object.defineProperty(exports, "inferFromSchema", {
   enumerable: true,
   get: function get() {
@@ -27,7 +22,12 @@ Object.defineProperty(exports, "inferFromSchemaType", {
     return _inferFromSchemaType.default;
   }
 });
-exports.default = void 0;
+Object.defineProperty(exports, "validateDocument", {
+  enumerable: true,
+  get: function get() {
+    return _validateDocument.default;
+  }
+});
 
 var _Rule = _interopRequireDefault(require("./Rule"));
 

package/lib/util/convertToValidationMarker.js

@@ -3,8 +3,8 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.isNonNullable = isNonNullable;
 exports.default = convertToValidationMarker;
+exports.isNonNullable = isNonNullable;
 
 var _ValidationError = _interopRequireDefault(require("../ValidationError"));
 

package/lib/util/normalizeValidationRules.js

@@ -7,8 +7,6 @@ exports.default = normalizeValidationRules;
 
 var _Rule = _interopRequireDefault(require("../Rule"));
 
-var _blockValidator = require("../validators/blockValidator");
-
 var _slugValidator = require("../validators/slugValidator");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
@@ -61,7 +59,6 @@ function baseRuleReducer(inputRule, type) {
   if (type.name === 'slug') return baseRule.custom(_slugValidator.slugValidator);
   if (type.name === 'reference') return baseRule.reference();
   if (type.name === 'email') return baseRule.email();
-  if (type.name === 'block') return baseRule.block(_blockValidator.blockValidator);
   return baseRule;
 }
 

package/lib/validateDocument.js

@@ -3,8 +3,8 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.resolveTypeForArrayItem = resolveTypeForArrayItem;
 exports.default = validateDocument;
+exports.resolveTypeForArrayItem = resolveTypeForArrayItem;
 exports.validateItem = validateItem;
 
 var _uniqBy2 = _interopRequireDefault(require("lodash/uniqBy"));
@@ -141,7 +141,10 @@ function _validateItem() {
     var nestedChecks = [];
     var selfIsRequired = rules.some(rule => rule.isRequired());
     var shouldRunNestedObjectValidation = // run nested validation for objects
-    (type === null || type === void 0 ? void 0 : type.jsonType) === 'object' && (!!value || (value === null || value === undefined) && selfIsRequired);
+    (type === null || type === void 0 ? void 0 : type.jsonType) === 'object' && ( // if the value is truthy
+    !!value || // or
+    // (the value is null or undefined) and the top-level value is required
+    (value === null || value === undefined) && selfIsRequired);
 
     if (shouldRunNestedObjectValidation) {
       var fieldTypes = type.fields.reduce((acc, field) => {
@@ -190,9 +193,32 @@ function _validateItem() {
         } : value.indexOf(item)),
         type: resolveTypeForArrayItem(item, type.of)
       }))));
+    } // markDefs also do no run nested validation if the parent object is undefined
+    // for a similar reason to arrays
+
+
+    var shouldRunNestedValidationForMarkDefs = (0, _types.isBlock)(value) && value.markDefs.length && (0, _types.isBlockSchemaType)(type);
+
+    if (shouldRunNestedValidationForMarkDefs) {
+      var _type$fields = _slicedToArray(type.fields, 1),
+          spanChildrenField = _type$fields[0];
+
+      var spanType = spanChildrenField.type.of.find(_types.isSpanSchemaType);
+      var annotations = ((spanType === null || spanType === void 0 ? void 0 : spanType.annotations) || []).reduce((map, annotationType) => {
+        map.set(annotationType.name, annotationType);
+        return map;
+      }, new Map());
+      nestedChecks = nestedChecks.concat(value.markDefs.map(markDef => validateItem(_objectSpread(_objectSpread({}, restOfContext), {}, {
+        parent: value,
+        value: markDef,
+        path: path.concat(['markDefs', {
+          _key: markDef._key
+        }]),
+        type: annotations.get(markDef._type)
+      }))));
     }
 
-    var results = (yield Promise.all([...selfChecks, ...nestedChecks])).flat(); // run `dedupeValidationMarkers` if `_fieldRules` are present because they can
+    var results = (yield Promise.all([...selfChecks, ...nestedChecks])).flat(); // run `uniqBy` if `_fieldRules` are present because they can
     // cause repeat markers
 
     if (rules.some(rule => rule._fieldRules)) {

package/lib/validators/dateValidator.js

@@ -3,8 +3,8 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.isRecord = isRecord;
 exports.default = void 0;
+exports.isRecord = isRecord;
 
 var _format = _interopRequireDefault(require("date-fns/format"));
 

package/lib/validators/objectValidator.js

@@ -79,23 +79,6 @@ var objectValidators = _objectSpread(_objectSpread({}, _genericValidator.default
 
     return reference;
   }(),
-  block: function () {
-    var _block = _asyncToGenerator(function* (validateBlock, value, message, context) {
-      var result = yield validateBlock(value, context);
-
-      if (typeof result === 'string') {
-        return message || result;
-      }
-
-      return result;
-    });
-
-    function block(_x5, _x6, _x7, _x8) {
-      return _block.apply(this, arguments);
-    }
-
-    return block;
-  }(),
   assetRequired: (flag, value, message) => {
     if (!value || !value.asset || !value.asset._ref) {
       var assetType = flag.assetType || 'Asset';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanity/validation",
-  "version": "2.21.11-reference-updates.62+f98497b413",
+  "version": "2.22.3-reference-updates.54+c8d2373956",
   "description": "Validation and warning infrastructure for Sanity projects",
   "main": "./lib/index.js",
   "types": "./dist/dts",
@@ -32,8 +32,8 @@
     "directory": "packages/@sanity/validation"
   },
   "devDependencies": {
-    "@sanity/client": "2.21.11-reference-updates.62+f98497b413",
-    "@sanity/schema": "2.21.11-reference-updates.62+f98497b413"
+    "@sanity/client": "2.22.3-reference-updates.54+c8d2373956",
+    "@sanity/schema": "2.22.3-reference-updates.54+c8d2373956"
   },
   "peerDependencies": {
     "@sanity/client": "^2.0.0"
@@ -44,9 +44,9 @@
     }
   },
   "dependencies": {
-    "@sanity/types": "2.21.11-reference-updates.62+f98497b413",
+    "@sanity/types": "2.22.3-reference-updates.54+c8d2373956",
     "date-fns": "^2.16.1",
     "lodash": "^4.17.15"
   },
-  "gitHead": "f98497b413e20f06fcca0923f540b8ae2c8ae564"
+  "gitHead": "c8d237395618194918d18736ccbce8a937200b6a"
 }

package/src/Rule.ts

@@ -4,13 +4,13 @@ import {
   RuleClass,
   CustomValidator,
   RuleSpecConstraint,
-  BlockValidator,
   FieldRules,
   ValidationContext,
   RuleSpec,
   ValidationMarker,
   RuleTypeConstraint,
   Validator,
+  Block,
 } from '@sanity/types'
 import {cloneDeep, get} from 'lodash'
 import ValidationErrorClass from './ValidationError'
@@ -219,6 +219,13 @@ const Rule: RuleClass = class Rule implements IRule {
     return this.cloneWithRules([{flag: 'custom', constraint: fn as CustomValidator}])
   }
 
+  /**
+   * @deprecated use `Rule.custom` instead
+   */
+  block(fn: CustomValidator<Block>): Rule {
+    return this.cloneWithRules([{flag: 'custom', constraint: fn as CustomValidator}])
+  }
+
   min(len: number): Rule {
     return this.cloneWithRules([{flag: 'min', constraint: len}])
   }
@@ -336,10 +343,6 @@ const Rule: RuleClass = class Rule implements IRule {
     return this.cloneWithRules([{flag: 'reference'}])
   }
 
-  block(blockValidator: BlockValidator): Rule {
-    return this.cloneWithRules([{flag: 'block', constraint: blockValidator}])
-  }
-
   fields(rules: FieldRules): Rule {
     if (this._type !== 'Object') {
       throw new Error('fields() can only be called on an object type')

package/src/util/normalizeValidationRules.ts

@@ -1,6 +1,5 @@
 import {SchemaType, Rule, RuleTypeConstraint} from '@sanity/types'
 import RuleClass from '../Rule'
-import {blockValidator} from '../validators/blockValidator'
 import {slugValidator} from '../validators/slugValidator'
 
 const ruleConstraintTypes: {[P in Lowercase<RuleTypeConstraint>]: true} = {
@@ -54,7 +53,6 @@ function baseRuleReducer(inputRule: Rule, type: SchemaType) {
   if (type.name === 'slug') return baseRule.custom(slugValidator)
   if (type.name === 'reference') return baseRule.reference()
   if (type.name === 'email') return baseRule.email()
-  if (type.name === 'block') return baseRule.block(blockValidator)
   return baseRule
 }
 

package/src/validateDocument.test.ts

@@ -1,6 +1,6 @@
 /// <reference types="@sanity/types/parts" />
 
-import {Rule, SchemaType, SanityDocument, Schema, ArraySchemaType} from '@sanity/types'
+import {Rule, SchemaType, SanityDocument, Schema, ArraySchemaType, Block} from '@sanity/types'
 import createSchema from 'part:@sanity/base/schema-creator'
 import validateDocument, {resolveTypeForArrayItem, validateItem} from './validateDocument'
 import convertToValidationMarker from './util/convertToValidationMarker'
@@ -149,14 +149,17 @@ describe('validateItem', () => {
             },
           ],
         },
-        {name: 'registeredString', type: 'string', validation},
+        {name: 'registeredString', title: 'Registered String', type: 'string', validation},
         {
+          title: 'Registered Object Field',
           name: 'registeredObjectField',
           type: 'object',
           fields: [{name: 'foo', type: 'string', validation}],
         },
       ],
     })
+    // ensures there are no schema formatting issues
+    expect(schema._validation).toHaveLength(0)
 
     await expect(
       validateItem({
@@ -229,6 +232,9 @@ describe('validateItem', () => {
       ],
     })
 
+    // ensures there are no schema formatting issues
+    expect(schema._validation).toHaveLength(0)
+
     await expect(
       validateItem({
         document: undefined,
@@ -254,17 +260,232 @@ describe('validateItem', () => {
     ])
   })
 
+  it('runs nested validation for markDefs', async () => {
+    const linkValidationSpy = jest.fn(() => true as const)
+    const internalLinkSpy = jest.fn(() => 'mock invalid response')
+
+    const schema = createSchema({
+      types: [
+        {
+          name: 'post',
+          title: 'Post',
+          type: 'document',
+          fields: [
+            {name: 'title', type: 'string'},
+            {name: 'body', type: 'string'},
+          ],
+        },
+        {
+          name: 'registeredEditor',
+          type: 'object',
+          title: 'Registered Editor',
+          fields: [
+            {
+              name: 'editor',
+              type: 'array',
+              of: [
+                {
+                  type: 'block',
+                  marks: {
+                    annotations: [
+                      {
+                        name: 'exampleAnnotation',
+                        type: 'object',
+                        fields: [{name: 'value', type: 'string'}],
+                      },
+                    ],
+                  },
+                },
+              ],
+            },
+          ],
+        },
+        {
+          name: 'blockTest',
+          type: 'document',
+          title: 'blockTest',
+          fields: [
+            {
+              name: 'content',
+              title: 'Content',
+              type: 'array',
+              of: [
+                {
+                  type: 'block',
+                  marks: {
+                    annotations: [
+                      {
+                        name: 'link',
+                        type: 'object',
+                        title: 'link',
+                        fields: [{name: 'url', type: 'url'}],
+                        validation: (rule: Rule) => rule.custom(linkValidationSpy),
+                      },
+                      {
+                        name: 'internalLink',
+                        type: 'object',
+                        title: 'Internal link',
+                        fields: [{name: 'reference', type: 'reference', to: [{type: 'post'}]}],
+                        validation: (rule: Rule) => rule.custom(internalLinkSpy),
+                      },
+                      {name: 'nestedEditor', type: 'registeredEditor'},
+                    ],
+                  },
+                },
+              ],
+            },
+          ],
+        },
+      ],
+    })
+
+    expect(schema._validation).toHaveLength(0)
+
+    const nestedBlock: Block = {
+      _type: 'block',
+      _key: 'nested-block-key',
+      children: [
+        {
+          _key: 'some-key',
+          _type: 'span',
+          text: 'hey',
+          marks: ['example-annotation-key'],
+        },
+      ],
+      markDefs: [
+        {
+          _type: 'exampleAnnotation',
+          _key: 'example-annotation-key',
+          value: 5,
+        },
+      ],
+      style: 'normal',
+    }
+
+    const block: Block = {
+      _key: 'block-key',
+      _type: 'block',
+      children: [
+        {
+          _key: 'child-0',
+          _type: 'span',
+          marks: ['0', '1', '2'],
+          text: 'hey',
+        },
+      ],
+      markDefs: [
+        {
+          _key: '0',
+          _type: 'link',
+          url: 'https://example.com',
+        },
+        {
+          _key: '1',
+          _type: 'internalLink',
+          _ref: 'post-id',
+        },
+        {
+          _key: '2',
+          _type: 'nestedEditor',
+          editor: [nestedBlock],
+        },
+      ],
+      style: 'normal',
+    }
+
+    const document: SanityDocument = {
+      _id: 'mock-id',
+      _type: 'blockTest',
+      _createdAt: '2021-11-15T21:06:41.812Z',
+      _rev: 'example-ref',
+      _updatedAt: '2021-11-15T21:06:41.812Z',
+      content: [block],
+    }
+
+    const result = await validateItem({
+      document: document,
+      parent: undefined,
+      path: [],
+      type: schema.get('blockTest'),
+      value: document,
+      getDocumentExists: undefined,
+    })
+
+    expect(result).toMatchObject([
+      {
+        item: {message: 'mock invalid response', paths: []},
+        level: 'error',
+        path: ['content', {_key: 'block-key'}, 'markDefs', {_key: '1'}],
+        type: 'validation',
+      },
+      // this tests for nested markDef validation
+      {
+        item: {
+          message: 'Expected type "String", got "Number"',
+          paths: [],
+        },
+        level: 'error',
+        path: [
+          'content',
+          {_key: 'block-key'},
+          'markDefs',
+          {_key: '2'},
+          'editor',
+          {_key: 'nested-block-key'},
+          'markDefs',
+          {_key: 'example-annotation-key'},
+          'value',
+        ],
+        type: 'validation',
+      },
+    ])
+
+    expect(linkValidationSpy.mock.calls).toMatchObject([
+      [
+        {
+          _key: '0',
+          _type: 'link',
+          url: 'https://example.com',
+        },
+        {
+          document: {_id: 'mock-id'},
+          parent: {_key: 'block-key'},
+          path: ['content', {_key: 'block-key'}, 'markDefs', {_key: '0'}],
+          type: {name: 'link'},
+        },
+      ],
+    ])
+
+    expect(internalLinkSpy.mock.calls).toMatchObject([
+      [
+        {
+          _key: '1',
+          _ref: 'post-id',
+          _type: 'internalLink',
+        },
+        {
+          document: {_id: 'mock-id'},
+          parent: {_key: 'block-key'},
+          path: ['content', {_key: 'block-key'}, 'markDefs', {_key: '1'}],
+          type: {name: 'internalLink'},
+        },
+      ],
+    ])
+  })
+
   it('resolves an array item type if there is just one type', async () => {
     const schema = createSchema({
       types: [
         {
           name: 'values',
+          title: 'Values',
           type: 'array',
           // note that there is only one type available
           of: [{type: 'arrayItem'}],
           validation: (rule: Rule) => rule.required(),
         },
         {
+          title: 'Array Item',
           name: 'arrayItem',
           type: 'object',
           fields: [{name: 'title', type: 'string'}],
@@ -272,6 +493,9 @@ describe('validateItem', () => {
       ],
     })
 
+    // ensures there are no schema formatting issues
+    expect(schema._validation).toHaveLength(0)
+
     const values = [
       {
         // note how this doesn't have a _type
@@ -308,6 +532,7 @@ describe('validateItem', () => {
         {
           name: 'root',
           type: 'object',
+          title: 'Root',
           fields: [
             {
               name: 'level1Object',
@@ -344,6 +569,10 @@ describe('validateItem', () => {
       ],
     })
 
+    // ensures there are no schema formatting issues
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    expect((schema as any)._validation).toHaveLength(0)
+
     const value = {
       level1Object: {level3String: 'a string'},
       level1Array: [{level2Number: 5}],

package/src/validateDocument.ts

@@ -6,6 +6,9 @@ import {
   ValidationMarker,
   isKeyedObject,
   isTypedObject,
+  isBlock,
+  isBlockSchemaType,
+  isSpanSchemaType,
 } from '@sanity/types'
 import {uniqBy} from 'lodash'
 import typeString from './util/typeString'
@@ -182,9 +185,39 @@ export async function validateItem({
     )
   }
 
+  // markDefs also do no run nested validation if the parent object is undefined
+  // for a similar reason to arrays
+  const shouldRunNestedValidationForMarkDefs =
+    isBlock(value) && value.markDefs.length && isBlockSchemaType(type)
+
+  if (shouldRunNestedValidationForMarkDefs) {
+    const [spanChildrenField] = type.fields
+    const spanType = spanChildrenField.type.of.find(isSpanSchemaType)
+
+    const annotations = (spanType?.annotations || []).reduce<Map<string, SchemaType>>(
+      (map, annotationType) => {
+        map.set(annotationType.name, annotationType)
+        return map
+      },
+      new Map()
+    )
+
+    nestedChecks = nestedChecks.concat(
+      value.markDefs.map((markDef) =>
+        validateItem({
+          ...restOfContext,
+          parent: value,
+          value: markDef,
+          path: path.concat(['markDefs', {_key: markDef._key}]),
+          type: annotations.get(markDef._type),
+        })
+      )
+    )
+  }
+
   const results = (await Promise.all([...selfChecks, ...nestedChecks])).flat()
 
-  // run `dedupeValidationMarkers` if `_fieldRules` are present because they can
+  // run `uniqBy` if `_fieldRules` are present because they can
   // cause repeat markers
   if (rules.some((rule) => rule._fieldRules)) {
     return uniqBy(results, (rule) => JSON.stringify(rule))

package/src/validators/objectValidator.ts

@@ -51,15 +51,6 @@ const objectValidators: Validators = {
     return true
   },
 
-  block: async (validateBlock, value, message, context) => {
-    const result = await validateBlock(value, context)
-    if (typeof result === 'string') {
-      return message || result
-    }
-
-    return result
-  },
-
   assetRequired: (flag, value, message) => {
     if (!value || !value.asset || !value.asset._ref) {
       const assetType = flag.assetType || 'Asset'

package/dist/dts/validators/blockValidator.d.ts

@@ -1,3 +0,0 @@
-import { BlockValidator } from '@sanity/types';
-export declare const blockValidator: BlockValidator;
-//# sourceMappingURL=blockValidator.d.ts.map

package/dist/dts/validators/blockValidator.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"blockValidator.d.ts","sourceRoot":"","sources":["../../../src/validators/blockValidator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAmB,cAAc,EAAC,MAAM,eAAe,CAAA;AAG9D,eAAO,MAAM,cAAc,EAAE,cAmD5B,CAAA"}

package/lib/validators/blockValidator.js

@@ -1,66 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.blockValidator = void 0;
-
-var _validateDocument = require("../validateDocument");
-
-function asyncGeneratorStep(gen, resolve, reject, _next, _throw, key, arg) { try { var info = gen[key](arg); var value = info.value; } catch (error) { reject(error); return; } if (info.done) { resolve(value); } else { Promise.resolve(value).then(_next, _throw); } }
-
-function _asyncToGenerator(fn) { return function () { var self = this, args = arguments; return new Promise(function (resolve, reject) { var gen = fn.apply(self, args); function _next(value) { asyncGeneratorStep(gen, resolve, reject, _next, _throw, "next", value); } function _throw(err) { asyncGeneratorStep(gen, resolve, reject, _next, _throw, "throw", err); } _next(undefined); }); }; }
-
-var blockValidator = /*#__PURE__*/function () {
-  var _ref = _asyncToGenerator(function* (value, context) {
-    var _type$fields, _type$fields$find;
-
-    if (value.markDefs.length === 0) return [];
-    var type = context.type;
-
-    if (!type) {
-      throw new Error("Schema `type` was not provided in validation context");
-    }
-
-    if (type.jsonType !== 'object') {
-      throw new Error("Expected schema type with jsonType `object` but found `".concat(type.jsonType, "` instead."));
-    }
-
-    var childrenType = (_type$fields = type.fields) === null || _type$fields === void 0 ? void 0 : (_type$fields$find = _type$fields.find(field => field.name === 'children')) === null || _type$fields$find === void 0 ? void 0 : _type$fields$find.type;
-    var spanType = (childrenType === null || childrenType === void 0 ? void 0 : childrenType.jsonType) === 'array' ? childrenType === null || childrenType === void 0 ? void 0 : childrenType.of.find(ofType => ofType.name === 'span') : null; // Validate every markDef (annotation) value
-    // eslint-disable-next-line no-warning-comments
-    // @ts-expect-error TODO (eventually): fix these types
-
-    var activeAnnotationTypes = spanType === null || spanType === void 0 ? void 0 : spanType.annotations.filter(annotation => // eslint-disable-next-line no-warning-comments
-    // @ts-expect-error TODO (eventually): fix these types
-    value.markDefs.map(def => def._type).includes(annotation.name));
-    var annotationValidations = [];
-    value.markDefs.forEach(markDef => {
-      var annotationType = activeAnnotationTypes.find(aType => aType.name === markDef._type);
-      var validations = (0, _validateDocument.validateItem)({
-        document: context.document,
-        value: markDef,
-        type: annotationType,
-        path: (context.path || []).concat(['markDefs', {
-          _key: markDef._key
-        }]),
-        parent: value,
-        getDocumentExists: context.getDocumentExists
-      });
-      annotationValidations.push(validations);
-    });
-    var results = yield Promise.all(annotationValidations);
-    var flattened = results.flat();
-    if (!flattened.length) return [];
-    return flattened.map(res => {
-      res.item.paths = [res.path];
-      return res.item;
-    });
-  });
-
-  return function blockValidator(_x, _x2) {
-    return _ref.apply(this, arguments);
-  };
-}();
-
-exports.blockValidator = blockValidator;

package/src/validators/blockValidator.ts

@@ -1,55 +0,0 @@
-import {ValidationMarker, BlockValidator} from '@sanity/types'
-import {validateItem} from '../validateDocument'
-
-export const blockValidator: BlockValidator = async (value, context) => {
-  if (value.markDefs.length === 0) return []
-
-  const {type} = context
-
-  if (!type) {
-    throw new Error(`Schema \`type\` was not provided in validation context`)
-  }
-  if (type.jsonType !== 'object') {
-    throw new Error(
-      `Expected schema type with jsonType \`object\` but found \`${type.jsonType}\` instead.`
-    )
-  }
-
-  const childrenType = type.fields?.find((field) => field.name === 'children')?.type
-  const spanType =
-    childrenType?.jsonType === 'array'
-      ? childrenType?.of.find((ofType) => ofType.name === 'span')
-      : null
-
-  // Validate every markDef (annotation) value
-  // eslint-disable-next-line no-warning-comments
-  // @ts-expect-error TODO (eventually): fix these types
-  const activeAnnotationTypes: any[] = spanType?.annotations.filter((annotation) =>
-    // eslint-disable-next-line no-warning-comments
-    // @ts-expect-error TODO (eventually): fix these types
-    value.markDefs.map((def) => def._type).includes(annotation.name)
-  )
-  const annotationValidations: Promise<ValidationMarker[]>[] = []
-  value.markDefs.forEach((markDef: any) => {
-    const annotationType = activeAnnotationTypes.find((aType) => aType.name === markDef._type)
-    const validations = validateItem({
-      document: context.document,
-      value: markDef,
-      type: annotationType,
-      path: (context.path || []).concat(['markDefs', {_key: markDef._key}]),
-      parent: value,
-      getDocumentExists: context.getDocumentExists,
-    })
-    annotationValidations.push(validations)
-  })
-
-  const results = await Promise.all(annotationValidations)
-  const flattened = results.flat()
-
-  if (!flattened.length) return []
-
-  return flattened.map((res) => {
-    res.item.paths = [res.path]
-    return res.item
-  })
-}