silas 0.6.2 → 0.6.3

data/lib/silas/tool.rb

@@ -23,16 +23,16 @@ module Silas
     class << self
       def description(text = nil)
         @description = text if text
-        @description || ""
+        inherited_setting(:@description) || ""
       end
 
       def param(name, type = :string, desc: nil)
-        param_refinements[name.to_sym] = { type: type.to_s, desc: desc }
+        own_param_refinements[name.to_sym] = { type: type.to_s, desc: desc }
       end
 
       def approval(policy = nil, &block)
         @approval_policy = block || policy unless policy.nil? && block.nil?
-        @approval_policy || :never
+        inherited_setting(:@approval_policy) || :never
       end
       alias approval_policy approval
 
@@ -40,7 +40,7 @@ module Silas
       def idempotent!    = @effect_mode = :idempotent
       def at_most_once!  = @effect_mode = :at_most_once
 
-      def effect_mode = @effect_mode || :at_most_once
+      def effect_mode = inherited_setting(:@effect_mode) || :at_most_once
 
       def tool_name
         name.demodulize.underscore
@@ -79,9 +79,42 @@ module Silas
 
       private
 
-      def param_refinements
+      # Ruby does NOT inherit class-level instance variables. Without this walk,
+      # factoring shared declarations into a base class —
+      #
+      #   class MoneyTool < Silas::Tool
+      #     transactional!
+      #     approval :always
+      #   end
+      #   class IssueRefund < MoneyTool; end
+      #
+      # — silently drops both in the subclass, and drops them into the LEAST
+      # safe defaults: no database transaction (:at_most_once) and no human
+      # gate (:never). Silently disarming the two declarations the ledger acts
+      # on is the worst failure this DSL could have, so settings resolve up the
+      # ancestry: nearest explicit declaration wins.
+      def inherited_setting(ivar)
+        klass = self
+        while klass.respond_to?(:inherited_setting, true)
+          value = klass.instance_variable_get(ivar)
+          return value unless value.nil?
+
+          klass = klass.superclass
+        end
+        nil
+      end
+
+      def own_param_refinements
         @param_refinements ||= {}
       end
+
+      # Merged down the ancestry so a base class's `param` refinements survive;
+      # a subclass redeclaring the same param wins.
+      def param_refinements
+        return own_param_refinements unless superclass.respond_to?(:param_refinements, true)
+
+        superclass.send(:param_refinements).merge(own_param_refinements)
+      end
     end
 
     # Instance-side delegation so a resolved instance answers the Ledger.

data/lib/silas/version.rb

@@ -1,3 +1,3 @@
 module Silas
-  VERSION = "0.6.2"
+  VERSION = "0.6.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: silas
 version: !ruby/object:Gem::Version
-  version: 0.6.2
+  version: 0.6.3
 platform: ruby
 authors:
 - Daniel St Paul
@@ -146,7 +146,9 @@ files:
 - app/views/silas/channels/approvals/error.html.erb
 - app/views/silas/channels/approvals/show.html.erb
 - app/views/silas/inbox/invocations/_approval_card.html.erb
+- app/views/silas/inbox/invocations/_detail.html.erb
 - app/views/silas/inbox/invocations/_invocation.html.erb
+- app/views/silas/inbox/sessions/_child.html.erb
 - app/views/silas/inbox/sessions/_cost.html.erb
 - app/views/silas/inbox/sessions/_row.html.erb
 - app/views/silas/inbox/sessions/index.html.erb
@@ -175,11 +177,13 @@ files:
 - docs/conventions.md
 - docs/evals.md
 - docs/guarantees.md
+- docs/headless.md
 - docs/inbox-and-api.md
 - docs/memory.md
 - docs/providers.md
 - docs/sandbox.md
 - docs/tools.md
+- docs/traces.md
 - docs/tutorial.md
 - docs/vs-eve.md
 - docs/why-silas.md