klenod-build 0.0.27 → 0.0.29

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 583e74384e061c34bf2da93cd71156adc1b4d8c9d43dc86fec044364ff00f049
-  data.tar.gz: afa864b06807f97cf1e68d04128c77d930136982fccabcd91013893491b12f28
+  metadata.gz: 9dbc1f6baef3ea6249edc7d68f1d8725234384d45c3f00dcf1b4ebba50eb8cbe
+  data.tar.gz: ee110cebfd420f402a67c922b2dcb1fa8cb9f8dffe768e4d5ced389ae6110f49
 SHA512:
-  metadata.gz: 2d33327a02fc995e93eb644b558dcee29fe32dd1223f6dc2a50fa47e922d7bddb50f81065bc4df3f95297e43cc9e4aa7b89336f723edc762e7531457956a180e
-  data.tar.gz: 7e29a8f5aaacef578ca6efa7c2e10e0595afee4058d8a58726899f2d97f28c12b7c5376931a6d0ab993b4c11b6ea8c4bb065c0a4abbab18df51ebf14069a27a0
+  metadata.gz: cae3766524cc6f31a74420abed1f8eaec845204f2465dfc1b48b9ff0f6ad7695dbc648a7394e58940338d1f8ce052beee3cb6cef7fc7c97ec18510f6badd85f6
+  data.tar.gz: 6a4230dcc4503ff87c4974f9cbd8065cc8348f66ca54f47a6f44e8419367b0901a0562e1dc0bd8bc633cf7d86fae621b4171b63e07b716f887d93b71e3d44e5a

data/lib/klenod/build/context.rb

@@ -88,6 +88,10 @@ module Klenod
 
       DEFAULT_PLUGINS = DefaultPlugins.new.freeze
 
+      # `:development` collects, evaluates, and reloads on demand. `:build`
+      # only collects, for writing a bundle; `klenod build` always uses it.
+      MODES = %i[development build].freeze
+
       def self.default_plugins
         [
           Plugins::RubyPlugin.new,
@@ -116,6 +120,12 @@ module Klenod
         namespace: Module.new,
         analysis: false
       )
+        # Plugins compare the mode with `==`, so an unknown one would quietly
+        # behave like neither, e.g. CSS that is not minified.
+        unless MODES.include?(mode)
+          raise ArgumentError, "Unknown mode #{mode.inspect}; expected one of #{MODES.map(&:inspect).join(", ")}"
+        end
+
         @source_dir = source_dir
         plugins = plugins.to_a if plugins.equal?(DEFAULT_PLUGINS)
         @plugins = plugins

data/lib/klenod/build/graph/invalidator.rb

@@ -223,7 +223,11 @@ module Klenod
               :failed
             )
           )
-          mods[module_id] = FailedModule.new(error)
+          # Only an evaluated module needs the placeholder, so demand raises the
+          # error instead of serving stale exports. The failed record already
+          # covers a collected-only module, and a placeholder would make the
+          # next update treat it as evaluated and run its code.
+          mods[module_id] = FailedModule.new(error) if mods.key?(module_id)
         end
 
         def diff_assets(previous_assets, current_assets)

data/lib/klenod/build/graph.rb

@@ -934,7 +934,8 @@ module Klenod
               imports,
               record.source_map,
               record.version,
-              Runtime::Mod.constant_name_for(module_id.to_s)
+              Runtime::Mod.constant_name_for(module_id.to_s),
+              record.transformed_hash
             )
           ]
         end

data/lib/klenod/build/plugins/haml_plugin/parser.rb

@@ -8,7 +8,10 @@ module Klenod
     module Plugins
       module HamlPlugin
         def self.parse_haml(source, module_id: nil)
-          parser = ParserWithMetadata.new({})
+          # Haml's default preserve list (`textarea`, `pre`, `code`) sets
+          # `nuke_inner_whitespace` as if the tag used `<`. Klenod reads that
+          # flag as an explicit whitespace marker, so disable the list.
+          parser = ParserWithMetadata.new(preserve: [])
           parser.call(source)
         rescue ::Haml::SyntaxError => error
           raise ParseError.new(error, source: source, module_id: module_id)

data/lib/klenod/build/plugins/haml_plugin/transformer/ruby_builder.rb

@@ -494,6 +494,9 @@ module Klenod
                   kind, prefix, pattern = VARIABLE_TOKEN_KINDS[type]
                   receiver = @variables[kind]
                   next unless receiver
+                  # `:@name` names the variable rather than reading it, as in
+                  # `instance_variable_get(:@name)`.
+                  next if tokens[index - 1]&.fetch(1) == :on_symbeg
 
                   # Parenthesize the receiver so a low-precedence expression
                   # such as `a || b` stays one operand, e.g. in `$*.fetch(:x)`.

data/lib/klenod/build/version.rb

@@ -2,6 +2,6 @@
 
 module Klenod
   module Build
-    VERSION = "0.0.27"
+    VERSION = "0.0.29"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: klenod-build
 version: !ruby/object:Gem::Version
-  version: 0.0.27
+  version: 0.0.29
 platform: ruby
 authors:
 - Andrés Alin
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.27
+        version: 0.0.29
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.27
+        version: 0.0.29
 - !ruby/object:Gem::Dependency
   name: async
   requirement: !ruby/object:Gem::Requirement