klenod-build 0.0.28 → 0.0.29

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd9fd40ba545361854b3552bbcc323e13b07410d830269f5aece663f7c2aeebf
-  data.tar.gz: 2a6d61e67c6014eb3eae74955b55cdf308b1ad2d60fcf2920ce3b7e8140d6a26
+  metadata.gz: 9dbc1f6baef3ea6249edc7d68f1d8725234384d45c3f00dcf1b4ebba50eb8cbe
+  data.tar.gz: ee110cebfd420f402a67c922b2dcb1fa8cb9f8dffe768e4d5ced389ae6110f49
 SHA512:
-  metadata.gz: c72ea0b18ec7f93b578fda06d84182db5c1ae0a9d1514da42bd893f2c7ab0501873cc9d8ef58f6aec90a6a3eb50e51cee0b5040156c588cea25f6c1d607e2929
-  data.tar.gz: a758e9c33ffec2de4ffc591d87ca9e1c2454196125031a1f6beca8ad65926a6b5b3272ef7e25d0480149a5da9ab5cce080bd4474de8d2ca83404550b353184e8
+  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/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.28"
+    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.28
+  version: 0.0.29
 platform: ruby
 authors:
 - Andrés Alin
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.28
+        version: 0.0.29
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.28
+        version: 0.0.29
 - !ruby/object:Gem::Dependency
   name: async
   requirement: !ruby/object:Gem::Requirement