riffer 0.32.0 → 0.32.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0cddafa66f3f24980037c23be5e8584ec888b93c40b2b780c99e7205558d2593
-  data.tar.gz: 5aea2edfeb9d47a4246b15d3f4e16a7a3c1d7941a473355495267b9eb93bc3cc
+  metadata.gz: 06d5acfa86b1320573aeb4dd2135a8540dbda454871a1e49cca8ee7b30e43336
+  data.tar.gz: b216a00806b2f08516b197cf03c4bc23766c756174ab43fc3a5e45fb5a4f83d2
 SHA512:
-  metadata.gz: fad79dcfd8036c3249ea17be785cb6acb193e3a1677f23ee052c89daf39fe97840369a4718e65ebb515bbddeb866608a532b8076a89e2f6c724ddd74d2131e85
-  data.tar.gz: 0c7b1bdb7383c6f9a2b053d2d811a7c4aa29142f8a44aa3fd07071d88b0dac722268f036c9f45887bbf3b10591cb3067193540c2cff766d6ab91aa4644f0dcb8
+  metadata.gz: 6408ae651fb7944a4618eced7dcc19658262560748a6fedf96cab6989cd7497acde23beb1962a4117e67e595f23e870e241d511c60329f10dd2aafa4f2cca8ff
+  data.tar.gz: ce8962451448533b266b8411172e49311616b6b81a3c54a71c4484a249420f16ad474c5bf06e40d7d4e811ec37edf131db150e25b15e001e7970fa56f0ea711c

data/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "0.32.0"
+  ".": "0.32.1"
 }

data/CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.32.1](https://github.com/janeapp/riffer/compare/riffer/v0.32.0...riffer/v0.32.1) (2026-06-10)
+
+
+### Bug Fixes
+
+* respect disable-model-invocation in skills ([#303](https://github.com/janeapp/riffer/issues/303)) ([2cf8719](https://github.com/janeapp/riffer/commit/2cf8719ecc36d748fabf7f03c8427e2b3043d30c))
+
 ## [0.32.0](https://github.com/janeapp/riffer/compare/riffer/v0.31.0...riffer/v0.32.0) (2026-06-08)
 
 

data/docs/13_SKILLS.md

@@ -36,7 +36,11 @@ Review the code for:
 - `name` — lowercase alphanumeric with hyphens, 1-64 chars (must match directory name)
 - `description` — 1-1024 chars, helps the LLM decide when to activate
 
-Additional frontmatter keys are passed through as metadata.
+**Optional frontmatter fields:**
+
+- `disable-model-invocation` — when `true`, the skill is hidden from the catalog and the LLM cannot activate it via the `skill_activate` tool. It stays reachable through the programmatic `activate` config (see [Activated Skills](#activated-skills)), so you can inject it under your own logic instead of the model's. Only the literal value `true` disables invocation; any other value (or its absence) leaves the skill model-invocable.
+
+Any other frontmatter keys are passed through as metadata.
 
 ## Configuring an Agent
 
@@ -81,7 +85,7 @@ end
 
 ### Activated Skills
 
-Load skill instructions into the system prompt at startup (no tool call needed):
+Load skill instructions into the system prompt at startup (no tool call needed). This is also the only way to surface a skill marked `disable-model-invocation: true`, which the model can never activate on its own:
 
 ```ruby
 skills do

data/lib/riffer/agent.rb

@@ -343,9 +343,9 @@ class Riffer::Agent
   #--
   #: () -> Riffer::Messages::System?
   def build_skills_message
-    skills = @context.skills
-    return nil unless skills&.system_prompt
-    Riffer::Messages::System.new(skills.system_prompt)
+    content = @context.skills&.system_prompt
+    return nil if content.nil? || content.empty?
+    Riffer::Messages::System.new(content)
   end
 
   #--
@@ -415,7 +415,7 @@ class Riffer::Agent
 
     skills_config = @config.skills_config
 
-    if skills_config
+    if skills_config && @context.skills&.activatable?
       skill_activate_tool_class = skills_config.activate_tool || Riffer.config.skills.default_activate_tool
 
       if tools.any? { |t| t.name == skill_activate_tool_class.name }

data/lib/riffer/skills/activate_tool.rb

@@ -19,6 +19,7 @@ class Riffer::Skills::ActivateTool < Riffer::Tool
   def call(context:, name:)
     skills_context = context&.skills
     return error("Skills not configured") unless skills_context
+    return error("Unknown skill: '#{name}'") unless skills_context.model_invocable?(name)
 
     text(skills_context.activate(name))
   rescue Riffer::ArgumentError => e

data/lib/riffer/skills/context.rb

@@ -48,6 +48,21 @@ class Riffer::Skills::Context
     @activated.key?(name)
   end
 
+  # Returns whether a skill exists and may be activated by the model.
+  #--
+  #: (String) -> bool
+  def model_invocable?(name)
+    skill = skills[name]
+    !skill.nil? && !skill.disable_model_invocation
+  end
+
+  # Returns whether any skill is available for the model to activate.
+  #--
+  #: () -> bool
+  def activatable?
+    available_skills.any?
+  end
+
   # Returns the complete skills section for the system prompt — the catalog plus
   # any pre-activated skill bodies.
   #--
@@ -65,6 +80,6 @@ class Riffer::Skills::Context
   #--
   #: () -> Array[Riffer::Skills::Frontmatter]
   def available_skills
-    skills.values.reject { |skill| @activated.key?(skill.name) }
+    skills.values.reject { |skill| @activated.key?(skill.name) || skill.disable_model_invocation }
   end
 end

data/lib/riffer/skills/frontmatter.rb

@@ -4,7 +4,8 @@
 require "yaml"
 
 # Immutable value object holding parsed SKILL.md YAML frontmatter. Required
-# fields: +name+ and +description+; unrecognized top-level keys are merged into
+# fields: +name+ and +description+; the optional +disable-model-invocation+
+# flag is recognized, and any other unrecognized top-level keys are merged into
 # +metadata+.
 class Riffer::Skills::Frontmatter
   NAME_PATTERN = /\A[a-z0-9]+(-[a-z0-9]+)*\z/ #: Regexp
@@ -17,6 +18,11 @@ class Riffer::Skills::Frontmatter
   # The skill description (1-1024 chars).
   attr_reader :description #: String
 
+  # Whether the skill opts out of model-driven activation. Hidden from the
+  # catalog and rejected at model activation; still reachable via programmatic
+  # activation.
+  attr_reader :disable_model_invocation #: bool
+
   # Metadata from the spec's +metadata+ field plus any unrecognized top-level
   # keys.
   attr_reader :metadata #: Hash[Symbol, untyped]
@@ -29,7 +35,7 @@ class Riffer::Skills::Frontmatter
   def self.parse(raw)
     yaml, body = split_frontmatter(raw)
     raise Riffer::ArgumentError, "missing YAML frontmatter (expected --- delimiters)" if yaml.empty?
-    [new(name: yaml.delete(:name), description: yaml.delete(:description), metadata: yaml), body]
+    [new(name: yaml.delete(:name), description: yaml.delete(:description), disable_model_invocation: yaml.delete(:"disable-model-invocation"), metadata: yaml), body]
   end
 
   # Parses only the frontmatter from a raw SKILL.md string, ignoring the body.
@@ -39,7 +45,7 @@ class Riffer::Skills::Frontmatter
   def self.parse_frontmatter(raw)
     yaml, _ = split_frontmatter(raw)
     raise Riffer::ArgumentError, "missing YAML frontmatter (expected --- delimiters)" if yaml.empty?
-    new(name: yaml.delete(:name), description: yaml.delete(:description), metadata: yaml)
+    new(name: yaml.delete(:name), description: yaml.delete(:description), disable_model_invocation: yaml.delete(:"disable-model-invocation"), metadata: yaml)
   end
 
   #--
@@ -58,13 +64,15 @@ class Riffer::Skills::Frontmatter
   private_class_method :split_frontmatter
 
   # Raises Riffer::ArgumentError if +name+ or +description+ is invalid.
+  # +disable_model_invocation+ is treated as set only when literally +true+.
   #--
-  #: (name: String, description: String, ?metadata: Hash[Symbol, untyped]) -> void
-  def initialize(name:, description:, metadata: {})
+  #: (name: String, description: String, ?disable_model_invocation: bool, ?metadata: Hash[Symbol, untyped]) -> void
+  def initialize(name:, description:, disable_model_invocation: false, metadata: {})
     validate_name!(name)
     validate_description!(description)
     @name = name.freeze
     @description = description.freeze
+    @disable_model_invocation = (disable_model_invocation == true)
     @metadata = metadata.freeze
   end
 

data/lib/riffer/version.rb

@@ -2,5 +2,5 @@
 # rbs_inline: enabled
 
 module Riffer
-  VERSION = "0.32.0" #: String
+  VERSION = "0.32.1" #: String
 end

data/sig/generated/riffer/skills/context.rbs

@@ -35,6 +35,16 @@ class Riffer::Skills::Context
   # : (String) -> bool
   def activated?: (String) -> bool
 
+  # Returns whether a skill exists and may be activated by the model.
+  # --
+  # : (String) -> bool
+  def model_invocable?: (String) -> bool
+
+  # Returns whether any skill is available for the model to activate.
+  # --
+  # : () -> bool
+  def activatable?: () -> bool
+
   # Returns the complete skills section for the system prompt — the catalog plus
   # any pre-activated skill bodies.
   # --

data/sig/generated/riffer/skills/frontmatter.rbs

@@ -1,7 +1,8 @@
 # Generated from lib/riffer/skills/frontmatter.rb with RBS::Inline
 
 # Immutable value object holding parsed SKILL.md YAML frontmatter. Required
-# fields: +name+ and +description+; unrecognized top-level keys are merged into
+# fields: +name+ and +description+; the optional +disable-model-invocation+
+# flag is recognized, and any other unrecognized top-level keys are merged into
 # +metadata+.
 class Riffer::Skills::Frontmatter
   NAME_PATTERN: Regexp
@@ -16,6 +17,11 @@ class Riffer::Skills::Frontmatter
   # The skill description (1-1024 chars).
   attr_reader description: String
 
+  # Whether the skill opts out of model-driven activation. Hidden from the
+  # catalog and rejected at model activation; still reachable via programmatic
+  # activation.
+  attr_reader disable_model_invocation: bool
+
   # Metadata from the spec's +metadata+ field plus any unrecognized top-level
   # keys.
   attr_reader metadata: Hash[Symbol, untyped]
@@ -38,9 +44,10 @@ class Riffer::Skills::Frontmatter
   def self.split_frontmatter: (String) -> [ Hash[Symbol, untyped], String ]
 
   # Raises Riffer::ArgumentError if +name+ or +description+ is invalid.
+  # +disable_model_invocation+ is treated as set only when literally +true+.
   # --
-  # : (name: String, description: String, ?metadata: Hash[Symbol, untyped]) -> void
-  def initialize: (name: String, description: String, ?metadata: Hash[Symbol, untyped]) -> void
+  # : (name: String, description: String, ?disable_model_invocation: bool, ?metadata: Hash[Symbol, untyped]) -> void
+  def initialize: (name: String, description: String, ?disable_model_invocation: bool, ?metadata: Hash[Symbol, untyped]) -> void
 
   private
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: riffer
 version: !ruby/object:Gem::Version
-  version: 0.32.0
+  version: 0.32.1
 platform: ruby
 authors:
 - Jake Bottrall