lux-hammer 0.3.10 → 0.3.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81af43fc44619384d2b62f58e7d3178a1b6edb22cbb1ef4e836b73e2ee469913
-  data.tar.gz: 47d012b5e54d1e8df3441de01eeaee3e3267f71c4ac8545b80f413d419bd3fc6
+  metadata.gz: 75362aef2908fdd2a3f4e32ab8c6ce7a4c4ae52fce795f5bca5437325a92da49
+  data.tar.gz: 0d318814ed2cdd0da35ca2b66e7637c62d5d2efece0cb24f60da1c5e02f4ad1f
 SHA512:
-  metadata.gz: d1dd7d495b96a6f7023601427dadcd492e593934c6407eb48dcb7c53fea26e33ea44ec40ef3a69856199fa75dd9d6278a8d32b14cbd4733b692dbbfe662f97ee
-  data.tar.gz: 2a9b223844fd645a79403c653a2bf25be28ba5654eef8562738b2f3768b0a72bddfd8b5ceb3dfe68ac7c583ecaaefc18fcee4d64f3f47964a93ff8f12b0fb586
+  metadata.gz: 64a8ce40bf7fc4986b6b0796c72fb9a8e9a13b2acb70c4aaa76cd68d5f68158bc9fbde7d5a0a5f3e586978b97f849f712b5addef336754b896e901f71c9feec2
+  data.tar.gz: 6f12a27b72098ff513ecc216ee8f2c6deb08e3c3a29cf4f1a7e9c6dfb7750a9b00ad13fe514f66fdda7b7a837b62e2526d1af04ca1af85deed1435e2d34b30a3

data/.version

@@ -1 +1 @@
-0.3.10
+0.3.12

data/AGENTS.md

@@ -98,10 +98,20 @@ At Hammerfile (block-DSL) top-level scope only:
 
 At class or `Hammerfile` scope:
 
-* `task :name do ... end`
+* `task :name do ... end` - redefining a task is last-write-wins. A
+  warning fires (and the entry is tagged `(redefined)` in help) **only**
+  when the task it clobbers was also defined inside the main app (cwd).
+  Overriding a task that came from outside - a framework default, plugin,
+  or gem (an absolute path outside cwd) - is treated as an intentional
+  override and stays silent. Locality is read off the location string:
+  `relativize_path` rewrites in-app absolutes to a `.`-relative form, so
+  anything still starting with `/` is external (see `app_local_location?`).
 * `namespace :name do ... end` - `:self` is reserved for the `hammer`
   binary's built-in namespace (see `lib/hammer/builtins.rb`). Defining
-  it from user code raises.
+  it from user code raises. Reopening a namespace **merges** (Rake-style):
+  the same `namespace :db` can be split across files / `load`ed fragments
+  and the blocks accumulate onto one subclass - no warning. Only a
+  duplicate *task* name warns (handled by `task`, last write wins).
 * `dotenv false` - opt out of auto `.env` / `.env.local` loading.
   Only meaningful from the `hammer` binary (`Hammer.cli`); the load
   happens after Hammerfile evaluation, before dispatch. Shell-set vars

data/lib/lux-hammer.rb

@@ -164,7 +164,11 @@ class Hammer
       end
       cmd.handler = handler
 
-      if (prev = commands[cmd.name])
+      # Only warn when overriding a task that was also defined inside the
+      # main app. Overriding one that came from outside - a framework
+      # default, plugin, or gem - is an intentional override, so stay quiet
+      # and don't tag it `(redefined)` in help.
+      if (prev = commands[cmd.name]) && app_local_location?(prev.location)
         cmd.prev_location = prev.location
         warn_redefinition('task', cmd.name, prev.location, cmd.location)
       end
@@ -190,28 +194,29 @@ class Hammer
     #     namespace :users do ... end
     #   end
     #
+    # Reopening a namespace merges: the same `namespace :db do ... end` can
+    # be split across files (Rake-style) and the blocks accumulate onto one
+    # subclass. Only a duplicate *task* name inside warns - that's handled
+    # by `task`. The namespace subclass is created lazily on first mention.
     def namespace(name, &block)
-      sub = Class.new(Hammer)
-      # Track the top-level CLI class so cross-invocation
-      # (`hammer 'ns:cmd'`) from inside a namespaced command dispatches
-      # against the full tree, not just the current namespace.
-      sub.instance_variable_set(:@root, root)
-      # Parent link, so `before` hooks defined further up the namespace
-      # tree can be collected and run outer -> inner before a command.
-      sub.instance_variable_set(:@parent, self)
-      # Share the parent's resolved program_name so help banners show
-      # "myapp ns:cmd" with the same prefix everywhere - and so the value
-      # captured pre-chdir (see `Hammer.cli`) survives into nested classes.
-      sub.instance_variable_set(:@program_name, program_name)
-      sub.instance_variable_set(:@location, source_location_of(block))
-      Hammer.with_target(sub) { sub.class_eval(&block) } if block
-
-      if (prev = @namespaces[name.to_s])
-        sub.instance_variable_set(:@prev_location, prev.instance_variable_get(:@location))
-        warn_redefinition('namespace', name.to_s, prev.instance_variable_get(:@location), sub.instance_variable_get(:@location))
-      end
+      sub = (@namespaces[name.to_s] ||= begin
+        ns = Class.new(Hammer)
+        # Track the top-level CLI class so cross-invocation
+        # (`hammer 'ns:cmd'`) from inside a namespaced command dispatches
+        # against the full tree, not just the current namespace.
+        ns.instance_variable_set(:@root, root)
+        # Parent link, so `before` hooks defined further up the namespace
+        # tree can be collected and run outer -> inner before a command.
+        ns.instance_variable_set(:@parent, self)
+        # Share the parent's resolved program_name so help banners show
+        # "myapp ns:cmd" with the same prefix everywhere - and so the value
+        # captured pre-chdir (see `Hammer.cli`) survives into nested classes.
+        ns.instance_variable_set(:@program_name, program_name)
+        ns.instance_variable_set(:@location, source_location_of(block))
+        ns
+      end)
 
-      @namespaces[name.to_s] = sub
+      Hammer.with_target(sub) { sub.class_eval(&block) } if block
     end
 
     # Register a hook to run before every command in this class (root or
@@ -252,7 +257,23 @@ class Hammer
     # built-in C-defined procs, eval'd blocks).
     def source_location_of(block)
       loc = block&.source_location
-      loc ? "#{loc[0]}:#{loc[1]}" : '(unknown)'
+      loc ? "#{relativize_path(loc[0])}:#{loc[1]}" : '(unknown)'
+    end
+
+    # Trim the cwd prefix off an absolute path so redefinition warnings
+    # read as `./lib/tasks/foo.rb` instead of a long absolute path. Paths
+    # outside cwd (framework / gem files) are left absolute.
+    def relativize_path(path)
+      prefix = "#{Dir.pwd}/"
+      path.start_with?(prefix) ? ".#{path[Dir.pwd.length..]}" : path
+    end
+
+    # True when a captured location lives inside the main app. relativize_path
+    # rewrites in-app absolute paths to a `.`-relative form, so anything still
+    # starting with "/" is an absolute path outside cwd - a framework, plugin,
+    # or gem file. Relative locations are already cwd-anchored, hence local.
+    def app_local_location?(loc)
+      !loc.to_s.start_with?('/')
     end
 
     # Emit a yellow [hammer] warning on stderr when a task/namespace is
@@ -735,7 +756,7 @@ class Hammer
 
     def print_footer
       Shell.say ''
-      Shell.say "powered by hammer - #{HOMEPAGE}", :gray
+      Shell.say "powered by hammer (v#{VERSION}) - #{HOMEPAGE}", :gray
     end
 
     # Hammerfile cheat-sheet shown under `hammer --help`. Same content

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lux-hammer
 version: !ruby/object:Gem::Version
-  version: 0.3.10
+  version: 0.3.12
 platform: ruby
 authors:
 - Dino Reic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-05-27 00:00:00.000000000 Z
+date: 2026-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest