isRequired on attrs

ueberdosis · May 20, 2022 · 23483d5 · justame · Aug 9, 2022 · 23483d5
1 parent 6fe3024
commit 23483d5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/packages/core/src/helpers/getAttributesFromExtensions.ts b/packages/core/src/helpers/getAttributesFromExtensions.ts
@@ -24,6 +24,7 @@ export function getAttributesFromExtensions(extensions: Extensions): ExtensionAt
     renderHTML: null,
     parseHTML: null,
     keepOnSplit: true,
+    isRequired: false,
   }
 
   extensions.forEach(extension => {
@@ -87,14 +88,20 @@ export function getAttributesFromExtensions(extensions: Extensions): ExtensionAt
     Object
       .entries(attributes)
       .forEach(([name, attribute]) => {
+        const mergedAttr = {
+          ...defaultAttribute,
+          ...attribute,
+        }
+
+        if (attribute.isRequired && attribute.default === undefined) {
+          delete mergedAttr.default;
+        }
+
         extensionAttributes.push({
           type: extension.name,
           name,
-          attribute: {
-            ...defaultAttribute,
-            ...attribute,
-          },
-        })
+          attribute: mergedAttr,
+        });
       })
   })
 

diff --git a/packages/core/src/types.ts b/packages/core/src/types.ts
@@ -128,6 +128,7 @@ export type Attribute = {
   renderHTML?: ((attributes: Record<string, any>) => Record<string, any> | null) | null,
   parseHTML?: ((element: HTMLElement) => any | null) | null,
   keepOnSplit: boolean,
+  isRequired?: boolean,
 }
 
 export type Attributes = {