fix(docs): update web-types code generation to match schema

piotrtomiak · piotrtomiak · commit aff098180bca · 2019-10-18T12:58:22.000+02:00
diff --git a/scripts/create-web-types.js b/scripts/create-web-types.js
@@ -24,13 +24,14 @@ let directiveGroups = {}
 
 // Base web-types object
 const webTypes = {
-  $schema: '',
+  $schema: 'https://raw.githubusercontent.com/JetBrains/web-types/master/schema/web-types.json',
   framework: 'vue',
   name: libraryName,
   version: libraryVersion,
   contributions: {
     html: {
       'types-syntax': 'typescript',
+      'description-markup': 'markdown',
       // Components get placed here
       tags: [],
       // Directives get placed in here
@@ -115,7 +116,7 @@ const computePropDefault = ({ default: def, type }) => {
 }
 
 // Process a single component's meta and definition/class objects
-const processComponentMeta = (meta, groupRef, docUrl) => {
+const processComponentMeta = (meta, groupRef, groupDescription, docUrl) => {
   const componentName = meta.component
 
   // Pull information from the component definition/class
@@ -138,17 +139,15 @@ const processComponentMeta = (meta, groupRef, docUrl) => {
     return obj
   }, {})
 
-  const tagName = kebabCase(componentName)
-
   // Build the tag reference
   const tag = {
     name: componentName,
     source: {
       module: libraryName,
       symbol: componentName
     },
-    'docs-url': docUrl,
-    description: `'${componentName}' - BootstrapVue component '${tagName}'`,
+    'doc-url': docUrl,
+    description: groupDescription,
     attributes: []
   }
 
@@ -170,14 +169,15 @@ const processComponentMeta = (meta, groupRef, docUrl) => {
       const prop = {
         name: propName,
         value: {
+          kind: 'expression',
           type: type,
-          default: computePropDefault($prop)
         },
+        default: computePropDefault($prop),
         'doc-url': docUrl
       }
       // Add required prop is required
       if ($prop.required) {
-        prop.value.required = true
+        prop.required = true
       }
       if (type === 'boolean') {
         // Deprecated. Use 'value' property instead. Specify only if type is
@@ -215,10 +215,13 @@ const processComponentMeta = (meta, groupRef, docUrl) => {
         event.description = eventObj.description
       }
       if (Array.isArray(eventObj.args)) {
-        event.arguments = eventObj.args.map(arg => {
+        event.arguments = eventObj.args.map((arg, index) => {
           arg = typeof arg === 'object' ? arg : { arg: arg }
+          let name = arg.arg
+            || (arg.type ? computePropType(arg) : undefined)
+            || "arg" + index
           const argument = {
-            name: arg.arg,
+            name: name.charAt(0).toLowerCase() + name.slice(1),
             'doc-url': docUrl
           }
           if (arg.description) {
@@ -278,7 +281,7 @@ const processComponentMeta = (meta, groupRef, docUrl) => {
     // Add the aliases
     $aliases.forEach(alias => {
       const aliasTag = { ...tag, name: alias, source: { ...tag.source, symbol: alias } }
-      aliasTag.description = `'${alias}' '${kebabCase(alias)}' (Alias for ${tag.description})`
+      aliasTag.description = `${tag.description}\n\n*Alias for ${tag.name}*`
       webTypes.contributions.html.tags.push(aliasTag)
     })
   }
@@ -310,12 +313,11 @@ const processDirectiveMeta = (directiveMeta, docUrl) => {
 
   // Add in argument details
   if (arg) {
-    // TODO as this is missing from the schema def
-    // https://github.com/JetBrains/web-types/issues/7
     attribute['vue-argument'] = {
       // RegExpr string pattern for argument
       pattern: arg.pattern,
-      description: arg.description
+      description: arg.description,
+      required: arg.required
     }
   }
 
@@ -363,7 +365,7 @@ const processComponentGroup = groupSlug => {
 
   // Process each component
   componentsMeta.forEach(meta => {
-    processComponentMeta(meta, groupRef, docUrl)
+    processComponentMeta(meta, groupRef, groupMeta.description, docUrl)
   })
 
   // Process any directives provided in the meta
diff --git a/src/directives/popover/package.json b/src/directives/popover/package.json
@@ -6,8 +6,9 @@
     "description": "Add BootstrapVue popovers to any element on your site, using Bootstrap v4 CSS for styling and animations. Popovers are tooltips on steroids.",
     "directive": "VBPopover",
     "arg": {
-      "pattern": "[a-zA-Z][a-ZA-Z0-9_\\-]*",
-      "description": "ID of element to append the popover markup when visible. Optional, defaults to the body"
+      "pattern": "[a-zA-Z][a-zA-Z0-9_\\-]*",
+      "description": "ID of element to append the popover markup when visible. Optional, defaults to the body",
+      "required": false
     },
     "expression": [
       "String",
diff --git a/src/directives/scrollspy/package.json b/src/directives/scrollspy/package.json
@@ -6,8 +6,9 @@
     "description": "Automatically activate BootstrapVue navigation or list group components based on scroll position to indicate which link is currently active in the viewport.",
     "directive": "VBScrollspy",
     "arg": {
-      "pattern": "[a-zA-Z][a-ZA-Z0-9_\\-]*",
-      "description": "ID of element to monitor scrolling on (if not provided defaults to the body element)"
+      "pattern": "[a-zA-Z][a-zA-Z0-9_\\-]*",
+      "description": "ID of element to monitor scrolling on (if not provided defaults to the body element)",
+      "required": false
     },
     "expression": [
       "String",
diff --git a/src/directives/tooltip/package.json b/src/directives/tooltip/package.json
@@ -6,8 +6,9 @@
     "description": "Add custom BootstrapVue tooltips to any element. Tooltips can be triggered by hovering, focusing, or clicking an element.",
     "directive": "VBTooltip",
     "arg": {
-      "pattern": "[a-zA-Z][a-ZA-Z0-9_\\-]*",
-      "description": "ID of element to append the tooltip markup when visible. Optional, defaults to the body"
+      "pattern": "[a-zA-Z][a-zA-Z0-9_\\-]*",
+      "description": "ID of element to append the tooltip markup when visible. Optional, defaults to the body",
+      "required": false
     },
     "expression": [
       "String",
