ask-skills 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 95968a26e22f7fe0aa8c2523f42a1768d551469f7f1fd431f049a9ed7cfdd4fe
-  data.tar.gz: b7e833ef0093db5c81bc0a0ee8e800534db02cfdad3a33bdb9ba681fec296d15
+  metadata.gz: 944cddd3dbc0de89913d1deb45668542f189e154bf3d40cc00468184689aeb38
+  data.tar.gz: a200f4181fa77aabc25dc6e1d0e2454d450159e5c64fa0cf7acf698910b8269b
 SHA512:
-  metadata.gz: 6faf0bf8bf7b78d5e39d5ef50bbf7391b894d70955bdb07f3941c394721d4a67d7bcbab44f81f72bbb87ac9537df80fa6302b4c021669b12f369e1d886ef65d2
-  data.tar.gz: 6903279580ab3cc1d2bd0e35a1005da8cbbc3a0005557439ecd2af20e0ca398b2d6fb15b8cea72bf0f38b4cfd657a2c5c95d17af8125a0f1ede1439ec07bc0ae
+  metadata.gz: bf049da94d308df2c5bb89164adc488abcc625ef65db0e2e14d73b981c03c9a0fe5d8dcf53b7f4037c437c1d3464a6515edf170ad439e28bf2f2afcf0fc7abb7
+  data.tar.gz: '03947665d57b76371ce167641b2b98a7d6e8f0587cc2af11c134a282906322c1f72cc4b7d45972c19738189d50eb45fc23cccffbbf1341a6c14dd663f26e0eff'

data/CHANGELOG.md

@@ -1,3 +1,45 @@
+## [0.3.0] - 2026-07-21
+
+### Added
+
+- **New discovery paths** — skills are now discovered from `agents/shared/skills/` and `app/agents/shared/skills/` alongside the legacy `.agents/skills/`. These paths follow the existing `agents/` convention already established for agent definitions.
+
+  ```
+  agents/
+  ├── health_check/
+  ├── daily_report/
+  └── shared/
+      ├── tools/          ← shared tools
+      └── skills/         ← new: shared skills
+          └── rails_debug/SKILL.md
+  ```
+
+- **Per-agent skills** — `Ask::Skills.discover(agent_dir:)` discovers skills scoped to a specific agent directory. These have highest priority over shared, legacy, user, gem, and built-in skills.
+
+  ```
+  agents/health_check/
+  ├── agent.rb
+  ├── instructions.md
+  └── skills/             ← new: only available to health_check agent
+      └── nginx_debug/SKILL.md
+  ```
+
+- **ask-agent integration** — `Ask::Agent::Session` accepts `agent_dir:` parameter. When creating a session from an agent definition via `Ask::Agent.new("name")`, per-agent skills are auto-discovered and included. No configuration needed.
+
+### Changed
+
+- `Ask::Skills.discover` now accepts `agent_dir:` keyword. When provided, the source list starts with the per-agent skills directory (highest priority). Shared project skills from `agents/shared/skills/` and `app/agents/shared/skills/` are included in the default source list.
+- Backward compatible — legacy `.agents/skills/` continues to work.
+
+### Tested
+
+- 10 new integration tests: discovery from `agents/shared/skills/`, `app/agents/shared/skills/`, per-agent skills, priority ordering, backward compatibility with `.agents/skills/`, and source list verification.
+- Full suite: 79 tests, 217 assertions — 0 failures.
+
+## [0.2.2] - 2026-06-25
+
+### Changed
+- Infrastructure: rubocop, overcommit, bin/setup, CI matrix, gemspec test, .minitest config.
 # Changelog
 
 ## [0.2.0] - 2026-06-10

data/lib/ask/skills/version.rb

@@ -1,5 +1,5 @@
 module Ask
   module Skills
-    VERSION = "0.2.1"
+    VERSION = "0.3.0"
   end
 end

data/lib/ask/skills.rb

@@ -16,33 +16,29 @@ module Ask
     end
 
     class << self
-      def discover(sources: nil)
-        Registry.new(sources || default_sources)
+      # Discover skills from all configured sources.
+      #
+      # @param agent_dir [String, nil] optional agent directory to discover
+      #   per-agent skills from (e.g. "agents/health_check")
+      # @param sources [Array<Source::Base>, nil] custom source list
+      # @return [Registry]
+      def discover(sources: nil, agent_dir: nil)
+        all_sources = sources || build_source_list(agent_dir: agent_dir)
+        Registry.new(all_sources)
       end
 
+      # Default sources when no custom sources or agent directory given.
+      # Legacy `.agents/skills/` is kept for backward compatibility.
       def default_sources
-        [
-          # Highest priority first — first source wins in Registry
-          Source::Filesystem.new(project_dir: ".agents/skills"),
-          Source::Filesystem.new(user_dir: "~/.config/ask/skills"),
-          Source::Gems.new,
-          Source::Filesystem.new(dir: builtin_skills_dir),
-        ]
+        build_source_list
       end
 
+      # Built-in skills that ship with the gem.
       def builtin_skills_dir
-        # Skills live in lib/ask/skills/<skill_name>/SKILL.md
-        # __dir__ in this file (lib/ask/skills.rb) is lib/ask/
-        # The skill directories are in lib/ask/skills/
         File.join(__dir__, "skills")
       end
 
       # Load a skill from an arbitrary markdown file path.
-      # Parses frontmatter if present, otherwise uses the filename as the skill name.
-      #
-      # @param path [String] absolute or relative path to a markdown (.md) file
-      # @return [Skill] a skill with the file's content as instructions
-      # @raise [Errno::ENOENT] if the file does not exist
       def load_file(path)
         path = File.expand_path(path)
         content = File.read(path)
@@ -61,7 +57,6 @@ module Ask
       end
 
       # Simple frontmatter parsing for skill files.
-      # Returns a hash of key-value pairs from between --- markers.
       def parse_frontmatter(content)
         return {} unless content.start_with?("---\n")
         end_idx = content.index("\n---\n", 4)
@@ -86,6 +81,38 @@ module Ask
         body = content[(end_idx + 5)..] || ""
         body.sub(/\A\n/, "").strip
       end
+
+      private
+
+      # Build the prioritized source list.
+      # Order: per-agent → shared project → legacy → user → gems → built-in
+      def build_source_list(agent_dir: nil)
+        sources = []
+
+        # Per-agent skills (highest priority — agent-specific first)
+        if agent_dir
+          per_agent = File.join(agent_dir, "skills")
+          sources << Source::Filesystem.new(dir: per_agent)
+        end
+
+        # Shared project skills
+        sources << Source::Filesystem.new(project_dir: "agents/shared/skills")
+        sources << Source::Filesystem.new(project_dir: "app/agents/shared/skills")
+
+        # Legacy project skills (backward compat)
+        sources << Source::Filesystem.new(project_dir: ".agents/skills")
+
+        # User-global skills
+        sources << Source::Filesystem.new(user_dir: "~/.config/ask/skills")
+
+        # Gems
+        sources << Source::Gems.new
+
+        # Built-in
+        sources << Source::Filesystem.new(dir: builtin_skills_dir)
+
+        sources
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ask-skills
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Kaka Ruto