parcel-bundler · devongovett · Sep 23, 2018 · Aug 25, 2018 · Aug 25, 2018 · Aug 25, 2018
diff --git a/src/Parser.js b/src/Parser.js
@@ -20,6 +20,7 @@ class Parser {
     this.registerExtension('vue', './assets/VueAsset');
     this.registerExtension('json', './assets/JSONAsset');
     this.registerExtension('json5', './assets/JSONAsset');
+    this.registerExtension('jsonld', './assets/JSONLDAsset');
     this.registerExtension('yaml', './assets/YAMLAsset');
     this.registerExtension('yml', './assets/YAMLAsset');
     this.registerExtension('toml', './assets/TOMLAsset');

diff --git a/src/assets/HTMLAsset.js b/src/assets/HTMLAsset.js
@@ -49,7 +49,8 @@ const META = {
     'msapplication-square310x310logo',
     'msapplication-square70x70logo',
     'msapplication-wide310x150logo',
-    'msapplication-TileImage'
+    'msapplication-TileImage',
+    'msapplication-config'
   ],
   itemprop: [
     'image',
@@ -64,7 +65,8 @@ const META = {
 const SCRIPT_TYPES = {
   'application/javascript': 'js',
   'text/javascript': 'js',
-  'application/json': false
+  'application/json': false,
+  'application/ld+json': 'jsonld'
 };
 
 // Options to be passed to `addURLDependency` for certain tags + attributes
@@ -247,13 +249,15 @@ class HTMLAsset extends Asset {
       } else if (type === 'tag') {
         if (
           (rendition.type === 'js' && node.tag === 'script') ||
-          (rendition.type === 'css' && node.tag === 'style')
+          (rendition.type === 'css' && node.tag === 'style') ||
+          (rendition.type === 'jsonld' && node.tag === 'script')
         ) {
           node.content = rendition.value;
         }
 
         // Delete "type" attribute, since CSS and JS are the defaults.
-        if (node.attrs) {
+        // Unless it's application/ld+json
+        if (node.attrs && node.attrs.type !== 'application/ld+json') {
           delete node.attrs.type;
         }
       }

diff --git a/src/assets/JSONLDAsset.js b/src/assets/JSONLDAsset.js
@@ -0,0 +1,72 @@
+const urlJoin = require('../utils/urlJoin');
+const isURL = require('../utils/is-url');
+const Asset = require('../Asset');
+const logger = require('../Logger');
+
+// A list of all attributes in a schema that may produce a dependency
+// Based on https://schema.org/ImageObject
+// Section "Instances of ImageObject may appear as values for the following properties"
+const SCHEMA_ATTRS = [
+  'logo',
+  'photo',
+  'image',
+  'thumbnail',
+  'screenshot',
+  'primaryImageOfPage'
+];
+
+class JSONLDAsset extends Asset {
+  constructor(name, options) {
+    super(name, options);
+    this.type = 'jsonld';
+  }
+
+  parse(content) {
+    return JSON.parse(content.trim());
+  }
+
+  collectDependencies() {
+    if (!this.options.publicURL.startsWith('http')) {
+      logger.warn(
+        "Please specify a publicURL using --public-url, otherwise schema asset links won't work"
+      );
+      return;
+    }
+
+    for (let schemaKey in this.ast) {
+      if (SCHEMA_ATTRS.includes(schemaKey)) {
+        this.collectFromKey(this.ast, schemaKey);
+        this.isAstDirty = true;
+      }
+    }
+  }
+
+  // Auxiliary method for collectDependencies() to use for recursion
+  collectFromKey(schema, schemaKey) {
+    if (!schema.hasOwnProperty(schemaKey)) {
+      return;
+    }
+    // values can be strings or objects
+    // if it's not a string, it should have a url
+    if (typeof schema[schemaKey] === 'string') {
+      let assetPath = this.addURLDependency(schema[schemaKey]);
+      if (!isURL(assetPath)) {
+        // paths aren't allowed, values must be urls
+        assetPath = urlJoin(this.options.publicURL, assetPath);
+      }
+      schema[schemaKey] = assetPath;
+    } else {
+      this.collectFromKey(schema[schemaKey], 'url');
+    }
+  }
+
+  generate() {
+    if (this.options.production) {
+      return JSON.stringify(this.ast);
+    } else {
+      return JSON.stringify(this.ast, null, 2);
+    }
+  }
+}
+
+module.exports = JSONLDAsset;
diff --git a/test/integration/schema-jsonld/images/image.jpeg b/test/integration/schema-jsonld/images/image.jpeg
diff --git a/test/integration/schema-jsonld/images/logo.png b/test/integration/schema-jsonld/images/logo.png
diff --git a/test/integration/schema-jsonld/index.html b/test/integration/schema-jsonld/index.html
@@ -0,0 +1,29 @@
+<html>
+<head>
+  <title>hi</title>
+  <link rel="stylesheet" href="other.css">
+</head>
+<body>
+
+<p>text 123</p>
+
+<script type="application/ld+json">
+{
+  "@context": "http://schema.org",
+  "@type": "LocalBusiness",
+  "description": "This is your business description.",
+  "name": "Parcel's parcel",
+  "telephone": "555-111-2345",
+  "openingHours": "Mo,Tu,We,Th,Fr 09:00-17:00",
+  "logo": {
+      "@type": "ImageObject",
+      "url": "images/logo.png",
+      "width": 180,
+      "height": 120
+    },
+  "image": "images/image.jpeg"
+}
+</script>
+
+</body>
+</html>
diff --git a/test/integration/schema-jsonld/other.css b/test/integration/schema-jsonld/other.css
@@ -0,0 +1,3 @@
+.other {
+  color: green;
+}
diff --git a/test/schema-jsonld.js b/test/schema-jsonld.js
@@ -0,0 +1,26 @@
+const {bundle, assertBundleTree} = require('./utils');
+
+describe('schema ld+json', function() {
+  it('Should parse a LD+JSON schema and collect dependencies', async function() {
+    let b = await bundle(__dirname + '/integration/schema-jsonld/index.html', {
+      production: true,
+      publicURL: 'https://place.holder/'
+    });
+
+    await assertBundleTree(b, {
+      name: 'index.html',
+      assets: ['index.html'],
+      childBundles: [
+        {
+          type: 'jpeg'
+        },
+        {
+          type: 'png'
+        },
+        {
+          type: 'css'
+        }
+      ]
+    });
+  });
+});