appydave-tools 0.71.0 → 0.71.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/CHANGELOG.md +13 -0
- data/docs/backlog.md +57 -0
- data/lib/appydave/tools/jump/search.rb +5 -2
- data/lib/appydave/tools/version.rb +1 -1
- data/package.json +1 -1
- metadata +2 -2
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: ff2949280923d4d700c79c676aff2969cc95a0808fe857bee5a20c14f33e8524
|
|
4
|
+
data.tar.gz: 4d8616d22c94aae3316c310ed07da76cf5739d39c10771b43f09362d77eff70b
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 16b926b801da0246092980d859729ce9c408a92644881877e54ccf1fc8a2ef43141ba0b78cc32fb31c0c3d2eaa41d53d224b4552fd63f49a798c52ad5a70a3d6
|
|
7
|
+
data.tar.gz: 5a21ef6d3d0629ca756527210b9e1f208a01aa752f0c93229d1cd3f7bd1c7116ab72cc83357c069984976061f15b2ffe17fd0cbbf784cc60d4fb5b3f1a073c4a
|
data/CHANGELOG.md
CHANGED
|
@@ -1,3 +1,16 @@
|
|
|
1
|
+
# [0.71.0](https://github.com/appydave/appydave-tools/compare/v0.70.0...v0.71.0) (2025-12-14)
|
|
2
|
+
|
|
3
|
+
|
|
4
|
+
### Bug Fixes
|
|
5
|
+
|
|
6
|
+
* allow common mutator methods in Naming/PredicateMethod cop ([4deb4b3](https://github.com/appydave/appydave-tools/commit/4deb4b35b0375bac6d67324a17444dc5d31b5f41))
|
|
7
|
+
|
|
8
|
+
|
|
9
|
+
### Features
|
|
10
|
+
|
|
11
|
+
* add dam ssd-status command and consolidate format_age to FileHelper ([2b09f99](https://github.com/appydave/appydave-tools/commit/2b09f99391a5312032d7feb2cbc125cc1e5011ad))
|
|
12
|
+
* add jump cli ([0887245](https://github.com/appydave/appydave-tools/commit/08872458eb67cd9a25a622b89299817a7bf22207))
|
|
13
|
+
|
|
1
14
|
# [0.70.0](https://github.com/appydave/appydave-tools/compare/v0.69.0...v0.70.0) (2025-11-26)
|
|
2
15
|
|
|
3
16
|
|
data/docs/backlog.md
CHANGED
|
@@ -13,6 +13,7 @@ Requirements tracking for AppyDave Tools development.
|
|
|
13
13
|
| 3 | NFR-1: Improve test coverage for DAM commands | 2025-12-06 | Pending |
|
|
14
14
|
| 4 | FR-3: Jump Location Tool (see spec) | 2025-12-13 | ✅ Implemented 2025-12-13 |
|
|
15
15
|
| 5 | NFR-2: Claude Code Skill for Jump Tool (see below) | 2025-12-14 | ✅ Implemented 2025-12-14 |
|
|
16
|
+
| 6 | BUG-1: Jump CLI get/remove commands fail to find entries (see below) | 2025-12-15 | Pending |
|
|
16
17
|
|
|
17
18
|
---
|
|
18
19
|
|
|
@@ -62,6 +63,62 @@ Requirements tracking for AppyDave Tools development.
|
|
|
62
63
|
|
|
63
64
|
---
|
|
64
65
|
|
|
66
|
+
### BUG-1: Jump CLI get/remove Commands Fail to Find Entries
|
|
67
|
+
|
|
68
|
+
**Priority**: High - Core functionality broken
|
|
69
|
+
|
|
70
|
+
**Summary**: The `get` and `remove` commands fail to find locations by key, while `search` successfully finds the same entries.
|
|
71
|
+
|
|
72
|
+
**Steps to Reproduce**:
|
|
73
|
+
```bash
|
|
74
|
+
# Search finds the entry ✅
|
|
75
|
+
bin/jump.rb search awb-team
|
|
76
|
+
# Result: Shows awb-team with jump alias jwb-team
|
|
77
|
+
|
|
78
|
+
# Get fails to find the same entry ❌
|
|
79
|
+
bin/jump.rb get awb-team
|
|
80
|
+
# Result: "Location not found"
|
|
81
|
+
|
|
82
|
+
# Remove also fails ❌
|
|
83
|
+
bin/jump.rb remove test-minimal --force
|
|
84
|
+
# Result: "No locations found."
|
|
85
|
+
```
|
|
86
|
+
|
|
87
|
+
**Affected Commands**:
|
|
88
|
+
- `get <key>` - Returns "Location not found" or "No locations found"
|
|
89
|
+
- `remove <key>` - Returns "No locations found"
|
|
90
|
+
|
|
91
|
+
**Working Commands**:
|
|
92
|
+
- `search <terms>` - Works correctly, finds entries
|
|
93
|
+
- `list` - Works correctly, shows all entries
|
|
94
|
+
- `list --format json` - Works correctly
|
|
95
|
+
|
|
96
|
+
**Suspected Cause**:
|
|
97
|
+
The `get` and `remove` commands likely use a different key lookup mechanism than `search`. Possibilities:
|
|
98
|
+
1. **Exact match vs fuzzy match** - `get` might expect an exact internal key format that differs from what's displayed
|
|
99
|
+
2. **Case sensitivity** - Key matching might be case-sensitive
|
|
100
|
+
3. **Index/cache issue** - Commands might be reading from different data sources or a stale index
|
|
101
|
+
4. **Key field mismatch** - The lookup might be checking a different field than `key`
|
|
102
|
+
|
|
103
|
+
**Evidence**:
|
|
104
|
+
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).
|
|
105
|
+
|
|
106
|
+
**Config Location**: `~/.config/appydave/locations.json`
|
|
107
|
+
|
|
108
|
+
**Investigation Points**:
|
|
109
|
+
- Compare how `search` resolves entries vs `get`/`remove`
|
|
110
|
+
- Check `lib/appydave/tools/name_manager/` for lookup logic
|
|
111
|
+
- Look at `LocationRegistry#find` or similar method
|
|
112
|
+
- Verify the key field being matched against
|
|
113
|
+
|
|
114
|
+
**Acceptance Criteria**:
|
|
115
|
+
- [ ] `get <key>` finds entry when `search <key>` finds it
|
|
116
|
+
- [ ] `remove <key>` finds entry when `search <key>` finds it
|
|
117
|
+
- [ ] Add regression tests for key lookup consistency
|
|
118
|
+
- [ ] Document root cause in commit message
|
|
119
|
+
|
|
120
|
+
---
|
|
121
|
+
|
|
65
122
|
## Completed Requirements
|
|
66
123
|
|
|
67
124
|
### NFR-2: Claude Code Skill for Jump Tool
|
|
@@ -66,14 +66,17 @@ module Appydave
|
|
|
66
66
|
# Get a location by exact key
|
|
67
67
|
#
|
|
68
68
|
# @param key [String] Exact key to find
|
|
69
|
-
# @return [Hash] Result with success and
|
|
69
|
+
# @return [Hash] Result with success and results array (consistent with search/list)
|
|
70
70
|
def get(key)
|
|
71
71
|
location = config.find(key)
|
|
72
72
|
|
|
73
73
|
if location
|
|
74
|
+
result = location_to_result(location, SCORE_EXACT_KEY)
|
|
75
|
+
result[:index] = 1
|
|
74
76
|
{
|
|
75
77
|
success: true,
|
|
76
|
-
|
|
78
|
+
count: 1,
|
|
79
|
+
results: [result]
|
|
77
80
|
}
|
|
78
81
|
else
|
|
79
82
|
suggestions = find_suggestions(key)
|
data/package.json
CHANGED
metadata
CHANGED
|
@@ -1,14 +1,14 @@
|
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
|
2
2
|
name: appydave-tools
|
|
3
3
|
version: !ruby/object:Gem::Version
|
|
4
|
-
version: 0.71.
|
|
4
|
+
version: 0.71.1
|
|
5
5
|
platform: ruby
|
|
6
6
|
authors:
|
|
7
7
|
- David Cruwys
|
|
8
8
|
autorequire:
|
|
9
9
|
bindir: exe
|
|
10
10
|
cert_chain: []
|
|
11
|
-
date: 2025-12-
|
|
11
|
+
date: 2025-12-15 00:00:00.000000000 Z
|
|
12
12
|
dependencies:
|
|
13
13
|
- !ruby/object:Gem::Dependency
|
|
14
14
|
name: activemodel
|