fix: Operation.isOperationList just check value[0] (#4061)

* Fix list queries.

packages/slate/Readme.md

@@ -1,3 +1,3 @@
 This package contains the core logic of Slate. Feel free to poke around to learn more!
 
-Note: A number of source files contain extracted types for `Interfaces` or `Transforms`. This is done currently to enable custom type extensions as found in `packages/src/interfaces/custom-types.ts`. 
+Note: A number of source files contain extracted types for `Interfaces` or `Transforms`. This is done currently to enable custom type extensions as found in `packages/src/interfaces/custom-types.ts`.

packages/slate/src/interfaces/element.ts

@@ -47,10 +47,7 @@ export const Element: ElementInterface = {
    */
 
   isElementList(value: any): value is Element[] {
-    return (
+    return Array.isArray(value) && value.every(val => Element.isElement(val))
-      Array.isArray(value) &&
-      (value.length === 0 || Element.isElement(value[0]))
-    )
   },
 
   /**

packages/slate/src/interfaces/node.ts

@@ -383,7 +383,7 @@ export const Node: NodeInterface = {
    */
 
   isNodeList(value: any): value is Node[] {
-    return Array.isArray(value) && (value.length === 0 || Node.isNode(value[0]))
+    return Array.isArray(value) && value.every(val => Node.isNode(val))
   },
 
   /**

packages/slate/src/interfaces/operation.ts

@@ -219,8 +219,7 @@ export const Operation: OperationInterface = {
 
   isOperationList(value: any): value is Operation[] {
     return (
-      Array.isArray(value) &&
+      Array.isArray(value) && value.every(val => Operation.isOperation(val))
-      (value.length === 0 || Operation.isOperation(value[0]))
     )
   },
 

packages/slate/src/interfaces/text.ts

@@ -71,7 +71,7 @@ export const Text: TextInterface = {
    */
 
   isTextList(value: any): value is Text[] {
-    return Array.isArray(value) && (value.length === 0 || Text.isText(value[0]))
+    return Array.isArray(value) && value.every(val => Text.isText(val))
   },
 
   /**

packages/slate/test/interfaces/Element/isElementList/not-full-element.tsx

@@ -0,0 +1,17 @@
+import { Element } from 'slate'
+
+export const input = [
+  {
+    children: [],
+  },
+  {
+    type: 'set_node',
+    path: [0],
+    properties: {},
+    newProperties: {},
+  },
+]
+export const test = value => {
+  return Element.isElementList(value)
+}
+export const output = false

packages/slate/test/interfaces/Node/isNodeList/not-full-node.tsx

@@ -0,0 +1,13 @@
+import { Node } from 'slate'
+
+export const input = [
+  {
+    children: [],
+    selection: null,
+  },
+  'a string',
+]
+export const test = value => {
+  return Node.isNodeList(value)
+}
+export const output = false

packages/slate/test/interfaces/Operation/isOperationList/not-full-operaion.tsx

@@ -0,0 +1,17 @@
+import { Operation } from 'slate'
+
+export const input = [
+  {
+    type: 'set_node',
+    path: [0],
+    properties: {},
+    newProperties: {},
+  },
+  {
+    text: '',
+  },
+]
+export const test = value => {
+  return Operation.isOperationList(value)
+}
+export const output = false

packages/slate/test/interfaces/Text/isTextList/not-full-text.tsx

@@ -0,0 +1,17 @@
+import { Text } from 'slate'
+
+export const input = [
+  {
+    text: '',
+  },
+  {
+    type: 'set_node',
+    path: [0],
+    properties: {},
+    newProperties: {},
+  },
+]
+export const test = value => {
+  return Text.isTextList(value)
+}
+export const output = false

site/tsconfig.json

@@ -1,30 +1,20 @@
-{
+{
-  "compilerOptions": {
+  "compilerOptions": {
-    "target": "es5",
+    "target": "es5",
-    "lib": [
+    "lib": ["dom", "dom.iterable", "esnext"],
-      "dom",
+    "allowJs": true,
-      "dom.iterable",
+    "skipLibCheck": true,
-      "esnext"
+    "strict": false,
-    ],
+    "downlevelIteration": true,
-    "allowJs": true,
+    "forceConsistentCasingInFileNames": true,
-    "skipLibCheck": true,
+    "noEmit": true,
-    "strict": false,
+    "esModuleInterop": true,
-    "downlevelIteration": true,
+    "module": "esnext",
-    "forceConsistentCasingInFileNames": true,
+    "moduleResolution": "node",
-    "noEmit": true,
+    "resolveJsonModule": true,
-    "esModuleInterop": true,
+    "isolatedModules": true,
-    "module": "esnext",
+    "jsx": "preserve"
-    "moduleResolution": "node",
+  },
-    "resolveJsonModule": true,
+  "exclude": ["node_modules"],
-    "isolatedModules": true,
+  "include": ["next-env.d.ts", "**/*.ts", "**/*.tsx"]
-    "jsx": "preserve"
+}
-  },
-  "exclude": [
-    "node_modules"
-  ],
-  "include": [
-    "next-env.d.ts",
-    "**/*.ts",
-    "**/*.tsx"
-  ]
-}