jekyll-esm 0.1.1 → 0.2.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 874a4a2123da08d7db10936aa7c1ef9c0ad4bd4b40e1bbdba9c112c362834648
-  data.tar.gz: 7eb4809d66b456733505bf68cbad740528c333a116bc8c15f05fe68884ed1b5f
+  metadata.gz: 22bef330cea6f3577397e75668c97c47c4e8ae2f1da943fea23043d7b2be4972
+  data.tar.gz: e0658de21883aaf0e8ea61b3f5c42c3643a83d83167144f3f8100ff7b8a19b4b
 SHA512:
-  metadata.gz: e80869a7324675efcec4ee35c88dc8fba77f82f4068a30aee3aa896e512bc2db17eaa3bce7c736f6ad26ec44bc7ff1ceeae1c3c57a4e452c3352687e8298a676
-  data.tar.gz: 3929653faeb53d75ea0754f48dbdacd39c8e2f59b2f0defbccdfdda323e4328b3021542af40055c4865fa5b0dbe7ea1a8e6759a7fa9b6c051825c60822ffa2c9
+  metadata.gz: d4f7cf93956ea11783b5aab896f188306e0687855f3e7e3bc136386f5870a2f0c16022a675dd6df07ffee0ef0def5de161e65e6650deb9f3e93dfc6a74b977b2
+  data.tar.gz: ec16d8d66a36e895b9d6e5ea1b5731000ab785dbd840297940ebac3f5536c4db0e8a7e5b041e37c22f144cd16e85bc4345bf3840d0a3cf98c537951ea98f24ad

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    jekyll-esm (0.1.1)
+    jekyll-esm (0.2.3.1)
       jekyll
       listen
       nokogiri

data/README.md

@@ -41,9 +41,42 @@ plugins:
   - jekyll/esm
 ```
 
+# Ignore package.json!
+Currently You *MUST* exclude package.json in the `_config.yml` otherwise jekyll will go into a loop. Sucks a bit but will try and improve that.
+
+```yml
+exclude:
+  - package.json
+```
+
+## Optimizing
+### Strict mode
+You can run in strict mode to increase logging verbosity from the js package manager.
+
+``` yml
+# _config.yml
+
+esm:
+  strict: true
+```
+
+### data-esm-id attribute
+You can set this on a declaration to prevent Jekyll from processing it more than once (eg, if you have lots of compiled pages, with the same layout, you don't want to run it more than once per importmap declaration):-
+
+``` html
+    <script data-esm-id='1' type="importmap">
+      {
+        "imports": {
+          "/src/index.js": "/src/index.js"
+        }
+      }
+    </script>
+
+If you have multiple importmap declarations with different values, set a different id for each one, or no id at all, but no id is less speed efficient, as jekyll will process them each time for every page.
+
 ### Example `_config.yml`
-You can have a look at the possible configuration options in the fixtures config file at `spec/fixtures/_config.yml` in this repo.
 
+You can have a look at the possible configuration options in the fixtures config file at `spec/fixtures/_config.yml` in this repo.
 
 ## Development
 

data/lib/jekyll/esm.rb

@@ -11,6 +11,7 @@ module Jekyll
   module Esm
     @@existing_esm_packages = []
     @@new_esm_packages = []
+    @@esm_ids = []
 
     class Error < StandardError; end
 
@@ -20,14 +21,27 @@ module Jekyll
       import_maps = doc.search('script[type=importmap]')
 
       import_maps.each do |value|
+        esm_id = value.attributes["data-esm-id"]&.value
+        # declare a data-esm-id so that jekyll will only process an esm declaration once
+        next if @@esm_ids.include?(esm_id)
+        @@esm_ids << esm_id if esm_id
+
         importmap = JSON.parse(value.children[0].content)
         imports = importmap["imports"]
         imports.keys.each do |import_key|
+          # ignore urls
+          next if import_key =~ /https?:\/\/[\S]+/
+          # ignore relative paths
+          next if import_key =~ /(^\.+\/)+/
+          # ignore absolute paths
+          next if import_key =~ /^\/[\S]+/
+
+          # ignore namespaces
           import = import_key.split('/').first
           pkg_path = File.join(page.site.source, 'node_modules', import)
 
           # don't repeatedly attempt to install a package
-          return if Dir.exists?(pkg_path) && @@new_esm_packages.include?(import)
+          next if Dir.exists?(pkg_path) && @@new_esm_packages.include?(import)
 
           @@new_esm_packages << import
 
@@ -47,26 +61,30 @@ module Jekyll
     end
 
     def self.apply(site)
-      existing_packages = Dir.children(File.join(site.source, 'node_modules')).reject { |dir| dir =~ /^\..*$/ }
-      for_removal = existing_packages - @@new_esm_packages.uniq
+      if @@existing_esm_packages.any?
+        for_removal = @@existing_esm_packages - @@new_esm_packages.uniq
 
-      # Remove any packages that are no longer referenced in an esm declaration
-      if for_removal.any?
-        stdout, stderr, status = Open3.capture3(
-          "yarn remove #{for_removal.join(' ')}",
-          chdir: File.expand_path(site.source)
-        )
+        # Remove any packages that are no longer referenced in an esm declaration
+        if for_removal.any?
+          stdout, stderr, status = Open3.capture3(
+            "yarn remove #{for_removal.join(' ')}",
+            chdir: File.expand_path(site.source)
+          )
 
-        if site.config.dig('esm', 'strict')
-          runtime_error = stdout =~ /ERROR in|SyntaxError/
+          if site.config.dig('esm', 'strict')
+            runtime_error = stdout =~ /ERROR in|SyntaxError/
 
-          raise Error, stderr if stderr.size > 0
-          raise Error, stdout if !runtime_error.nil?
+            raise Error, stderr if stderr.size > 0
+            raise Error, stdout if !runtime_error.nil?
+          end
         end
       end
 
+      FileUtils.rm_rf(File.join(site.dest, 'node_modules'))
       FileUtils.cp_r(File.join(site.source, 'node_modules'), File.join(site.dest, 'node_modules'))
+      @@existing_esm_packages = @@new_esm_packages
       @@new_esm_packages = []
+      @@esm_ids = []
     end
   end
 end

data/lib/jekyll/esm/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
   module Esm
-    VERSION = "0.1.1"
+    VERSION = "0.2.3.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-esm
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.3.1
 platform: ruby
 authors:
 - Steve Martin
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-08-17 00:00:00.000000000 Z
+date: 2021-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll