appydave-tools 0.71.0 → 0.72.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7c5fcf6b2bc5edd50a36d64d6692b81cfdd985ef6392d082fbf75967b6b86eff
-  data.tar.gz: 9adb4f242bd80382cd219c4bd1207219e2400a7464f9847a3279590871512d88
+  metadata.gz: 584067a790d262ecf04ab52f20557fc879db4cf968928199ae279438afddc0e9
+  data.tar.gz: c2d31308bd43b6e93e66bbdcff15bcde169fe6cde278de0d03a793c4dc605e1c
 SHA512:
-  metadata.gz: 864b74830389332b3b174e92a5d67d9149230994692fcafa20cf971dc3c60e58d0bb523ec29111d55edb3ceb59208f473c110db1e4f431e1c608be7f837ea75f
-  data.tar.gz: fd2e80db8db74faedcffa0cec1ef6c687cbea646c96abd93d223ba9b7c99d3fdf2a6fff271cfb3d4e6fca674b4f47b12af9a14e0f834575dfbb20aa98322c2d3
+  metadata.gz: 6f8d108fd12581e5e22d1d4125c85794f0df4e94ea8f626dddf1e26bdfcde423bccee1325e4548a5b55c0f33c9f87e7aaaa1d55ead202b8010caa766afdf90ef
+  data.tar.gz: 95b5b25e5b1d6816f271a0a12cb6cfd57ce53652c23fc1fdb34e6eed31eefc5fc63bd5381ea812355184f410d7537f81329ff34fd312cc0b7345727c9dc561ca

data/CHANGELOG.md

@@ -1,3 +1,23 @@
+## [0.71.1](https://github.com/appydave/appydave-tools/compare/v0.71.0...v0.71.1) (2025-12-15)
+
+
+### Bug Fixes
+
+* resolve jump get/remove returning 'No locations found' by making Search#get return results array consistent with search/list ([8ea39b5](https://github.com/appydave/appydave-tools/commit/8ea39b5fdd9bb9fe55c22f9cb86c8b66782c0f09))
+
+# [0.71.0](https://github.com/appydave/appydave-tools/compare/v0.70.0...v0.71.0) (2025-12-14)
+
+
+### Bug Fixes
+
+* allow common mutator methods in Naming/PredicateMethod cop ([4deb4b3](https://github.com/appydave/appydave-tools/commit/4deb4b35b0375bac6d67324a17444dc5d31b5f41))
+
+
+### Features
+
+* add dam ssd-status command and consolidate format_age to FileHelper ([2b09f99](https://github.com/appydave/appydave-tools/commit/2b09f99391a5312032d7feb2cbc125cc1e5011ad))
+* add jump cli ([0887245](https://github.com/appydave/appydave-tools/commit/08872458eb67cd9a25a622b89299817a7bf22207))
+
 # [0.70.0](https://github.com/appydave/appydave-tools/compare/v0.69.0...v0.70.0) (2025-11-26)
 
 

data/docs/backlog.md

@@ -13,6 +13,7 @@ Requirements tracking for AppyDave Tools development.
 | 3 | NFR-1: Improve test coverage for DAM commands | 2025-12-06 | Pending |
 | 4 | FR-3: Jump Location Tool (see spec) | 2025-12-13 | ✅ Implemented 2025-12-13 |
 | 5 | NFR-2: Claude Code Skill for Jump Tool (see below) | 2025-12-14 | ✅ Implemented 2025-12-14 |
+| 6 | BUG-1: Jump CLI get/remove commands fail to find entries (see below) | 2025-12-15 | Pending |
 
 ---
 
@@ -62,6 +63,62 @@ Requirements tracking for AppyDave Tools development.
 
 ---
 
+### BUG-1: Jump CLI get/remove Commands Fail to Find Entries
+
+**Priority**: High - Core functionality broken
+
+**Summary**: The `get` and `remove` commands fail to find locations by key, while `search` successfully finds the same entries.
+
+**Steps to Reproduce**:
+```bash
+# Search finds the entry ✅
+bin/jump.rb search awb-team
+# Result: Shows awb-team with jump alias jwb-team
+
+# Get fails to find the same entry ❌
+bin/jump.rb get awb-team
+# Result: "Location not found"
+
+# Remove also fails ❌
+bin/jump.rb remove test-minimal --force
+# Result: "No locations found."
+```
+
+**Affected Commands**:
+- `get <key>` - Returns "Location not found" or "No locations found"
+- `remove <key>` - Returns "No locations found"
+
+**Working Commands**:
+- `search <terms>` - Works correctly, finds entries
+- `list` - Works correctly, shows all entries
+- `list --format json` - Works correctly
+
+**Suspected Cause**:
+The `get` and `remove` commands likely use a different key lookup mechanism than `search`. Possibilities:
+1. **Exact match vs fuzzy match** - `get` might expect an exact internal key format that differs from what's displayed
+2. **Case sensitivity** - Key matching might be case-sensitive
+3. **Index/cache issue** - Commands might be reading from different data sources or a stale index
+4. **Key field mismatch** - The lookup might be checking a different field than `key`
+
+**Evidence**:
+When `get agent-workflow-build` failed, it suggested: "Did you mean: test-minimal, test-full, dev?" - This suggests the lookup is working but matching against something unexpected (possibly only returning entries without certain fields, or using a different search algorithm).
+
+**Config Location**: `~/.config/appydave/locations.json`
+
+**Investigation Points**:
+- Compare how `search` resolves entries vs `get`/`remove`
+- Check `lib/appydave/tools/name_manager/` for lookup logic
+- Look at `LocationRegistry#find` or similar method
+- Verify the key field being matched against
+
+**Acceptance Criteria**:
+- [ ] `get <key>` finds entry when `search <key>` finds it
+- [ ] `remove <key>` finds entry when `search <key>` finds it
+- [ ] Add regression tests for key lookup consistency
+- [ ] Document root cause in commit message
+
+---
+
 ## Completed Requirements
 
 ### NFR-2: Claude Code Skill for Jump Tool
@@ -72,7 +129,7 @@ Requirements tracking for AppyDave Tools development.
 - Personal skill at `~/.claude/skills/jump/SKILL.md`
 - Description includes brand names (appydave, voz, supportsignal, joy, aitldr, kiros), products (flivideo, storyline, klueless, dam), and trigger keywords
 - Primary mode: Jump CLI commands (`jump search`, `jump get`, `jump list`)
-- Fallback mode: Direct JSON read from `/ad/brains/brand-david/data-systems/collections/jump/current.json`
+- Fallback mode: Direct JSON read from `/ad/brains/brand-dave/data-systems/collections/jump/current.json`
 - Tool restrictions: `allowed-tools: Read, Bash, Grep, Glob`
 - Includes natural language examples and key location reference table
 

data/lib/appydave/tools/jump/cli.rb

@@ -245,7 +245,7 @@ module Appydave
 
           unless target
             output.puts 'Usage: jump generate <target> [--output <file>] [--output-dir <dir>]'
-            output.puts 'Targets: aliases, help, all'
+            output.puts 'Targets: aliases, help, ah-help, all'
             return EXIT_INVALID_INPUT
           end
 
@@ -542,16 +542,18 @@ module Appydave
             Targets:
               aliases     Generate shell alias file (aliases-jump.zsh)
               help        Generate help content for fzf (jump-help.txt)
-              all         Generate both files
+              ah-help     Generate content for aliases-help.zsh (ah function format)
+              all         Generate both aliases and help files
 
             Options:
-              --output <file>       Write to specific file (for aliases, help)
+              --output <file>       Write to specific file (for aliases, help, ah-help)
               --output-dir <dir>    Write to directory (for all)
 
             Examples:
               jump generate aliases
               jump generate aliases --output ~/.oh-my-zsh/custom/aliases-jump.zsh
               jump generate help --output ~/.oh-my-zsh/custom/data/jump-help.txt
+              jump generate ah-help --output ~/.oh-my-zsh/custom/aliases-help.zsh
               jump generate all --output-dir ~/.oh-my-zsh/custom/
           HELP
         end

data/lib/appydave/tools/jump/commands/generate.rb

@@ -6,7 +6,7 @@ module Appydave
       module Commands
         # Generate command creates shell aliases and help content
         class Generate < Base
-          VALID_TARGETS = %w[aliases help all].freeze
+          VALID_TARGETS = %w[aliases help ah-help all].freeze
 
           attr_reader :target, :output_path, :output_dir
 
@@ -28,7 +28,9 @@ module Appydave
               )
             end
 
-            send("generate_#{target}")
+            # Convert hyphenated targets to underscored method names
+            method_name = "generate_#{target.tr('-', '_')}"
+            send(method_name)
           end
 
           private
@@ -43,6 +45,11 @@ module Appydave
             write_or_return(content, output_path, 'jump-help.txt')
           end
 
+          def generate_ah_help
+            content = build_ah_help_content
+            write_or_return(content, output_path, 'aliases-help.zsh')
+          end
+
           def generate_all
             aliases_content = build_aliases_content
             help_content = build_help_content
@@ -115,6 +122,99 @@ module Appydave
             lines.join("\n")
           end
 
+          def build_ah_help_content
+            lines = []
+
+            # Group locations into logical sections
+            sections = group_locations_for_ah_help
+
+            sections.each do |section_name, locations|
+              lines << "## #{section_name}"
+
+              locations.each do |loc|
+                path = format_display_path(loc.path)
+                tags = build_ah_tags(loc)
+
+                # Format: alias (24 chars) + path + tags
+                alias_col = loc.jump.ljust(24)
+                path_with_tags = "#{path}#{' ' * [1, 48 - path.length].max}#{tags}"
+
+                lines << "#{alias_col}#{path_with_tags}"
+              end
+
+              lines << ''
+            end
+
+            lines.join("\n")
+          end
+
+          def format_display_path(path)
+            # Collapse home directory to ~
+            expanded = path_validator.expand(path)
+            home = Dir.home
+            expanded.start_with?(home) ? expanded.sub(home, '~') : expanded
+          end
+
+          def build_ah_tags(loc)
+            tags = [loc.type, loc.brand, loc.client].compact.reject(&:empty?)
+            tags.concat(loc.tags) if loc.tags
+            tags.uniq.map { |t| "##{t}" }.join(' ')
+          end
+
+          def group_locations_for_ah_help
+            sections = {
+              'Base Directories' => [],
+              'Brand Projects - AppyDave' => [],
+              'Brand Projects - Other' => [],
+              'Client Projects' => [],
+              'Video Projects' => [],
+              'Ruby Gems' => [],
+              'Reference & Archives' => [],
+              'Other' => []
+            }
+
+            config.locations.each do |loc|
+              section = determine_ah_section(loc)
+              sections[section] << loc
+            end
+
+            # Remove empty sections and sort locations within each
+            sections.reject { |_, locs| locs.empty? }.transform_values do |locs|
+              locs.sort_by(&:jump)
+            end
+          end
+
+          # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+          def determine_ah_section(loc)
+            type = loc.type&.downcase
+            brand = loc.brand&.downcase
+
+            case type
+            when 'monorepo', 'config'
+              return 'Base Directories' if loc.client.nil? && loc.brand.nil?
+            when 'video'
+              return 'Video Projects'
+            when 'gem'
+              return 'Ruby Gems'
+            when 'client'
+              return 'Client Projects'
+            when 'archive', 'reference'
+              return 'Reference & Archives'
+            end
+
+            return 'Client Projects' if loc.client
+
+            case brand
+            when 'appydave'
+              'Brand Projects - AppyDave'
+            when 'beauty-and-joy', 'david-cruwys', 'aitldr'
+              'Brand Projects - Other'
+            else
+              'Other'
+            end
+          end
+          # rubocop:enable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+
           def group_locations_for_aliases
             grouped = {}
 

data/lib/appydave/tools/jump/search.rb

@@ -66,14 +66,17 @@ module Appydave
         # Get a location by exact key
         #
         # @param key [String] Exact key to find
-        # @return [Hash] Result with success and result/error
+        # @return [Hash] Result with success and results array (consistent with search/list)
         def get(key)
           location = config.find(key)
 
           if location
+            result = location_to_result(location, SCORE_EXACT_KEY)
+            result[:index] = 1
             {
               success: true,
-              result: location_to_result(location, SCORE_EXACT_KEY)
+              count: 1,
+              results: [result]
             }
           else
             suggestions = find_suggestions(key)

data/lib/appydave/tools/version.rb

@@ -2,6 +2,6 @@
 
 module Appydave
   module Tools
-    VERSION = '0.71.0'
+    VERSION = '0.72.0'
   end
 end

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "appydave-tools",
-  "version": "0.71.0",
+  "version": "0.72.0",
   "description": "AppyDave YouTube Automation Tools",
   "scripts": {
     "release": "semantic-release"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: appydave-tools
 version: !ruby/object:Gem::Version
-  version: 0.71.0
+  version: 0.72.0
 platform: ruby
 authors:
 - David Cruwys
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-12-14 00:00:00.000000000 Z
+date: 2025-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel