|
32 | 32 | - [x] 2.3 Update SlashCommandWriter.__init__ to accept the flag parameter |
33 | 33 | - [x] 2.4 Update error handling in CLI to show different messages for default vs explicit paths |
34 | 34 |
|
35 | | -- [ ] 3.0 Update `_load_prompts()` to handle default vs explicit paths differently |
| 35 | +- [x] 3.0 Update `_load_prompts()` to handle default vs explicit paths differently |
36 | 36 | - Demo Criteria: "Default path falls back to bundled prompts; explicit path fails immediately without fallback" |
37 | 37 | - Proof Artifact(s): "Test: `test_writer_default_path_fallback` and `test_writer_explicit_path_no_fallback` pass" |
38 | | - - [ ] 3.1 Modify `_load_prompts()` to check the flag for explicit vs default |
39 | | - - [ ] 3.2 Only attempt fallback to bundled prompts when using default path |
40 | | - - [ ] 3.3 Raise clear error for explicit non-existent paths without fallback |
41 | | - - [ ] 3.4 Write tests for both scenarios (default with fallback, explicit without fallback) |
| 38 | + - [x] 3.1 Modify `_load_prompts()` to check the flag for explicit vs default |
| 39 | + - [x] 3.2 Only attempt fallback to bundled prompts when using default path |
| 40 | + - [x] 3.3 Raise clear error for explicit non-existent paths without fallback |
| 41 | + - [x] 3.4 Write tests for both scenarios (default with fallback, explicit without fallback) |
42 | 42 |
|
43 | | -- [ ] 4.0 Improve error messages for better user guidance |
| 43 | +- [x] 4.0 Improve error messages for better user guidance |
44 | 44 | - Demo Criteria: "Error messages clearly distinguish scenarios and provide actionable guidance" |
45 | 45 | - Proof Artifact(s): "CLI output showing clear, distinct error messages for each failure scenario" |
46 | | - - [ ] 4.1 Create different error messages for "default path not found" vs "explicit path not found" |
47 | | - - [ ] 4.2 Include information about attempted fallback in error messages |
48 | | - - [ ] 4.3 Show the actual paths that were checked |
49 | | - - [ ] 4.4 Update existing error handling in CLI to use new messages |
| 46 | + - [x] 4.1 Create different error messages for "default path not found" vs "explicit path not found" |
| 47 | + - [x] 4.2 Include information about attempted fallback in error messages |
| 48 | + - [x] 4.3 Show the actual paths that were checked |
| 49 | + - [x] 4.4 Update existing error handling in CLI to use new messages |
50 | 50 |
|
51 | | -- [ ] 5.0 Ensure backward compatibility and verify existing tests pass |
| 51 | +- [~] 5.0 Ensure backward compatibility and verify existing tests pass |
52 | 52 | - Demo Criteria: "All existing tests pass; development workflow still works; custom prompts paths still work" |
53 | 53 | - Proof Artifact(s): "Test suite: All tests pass; Manual: Run from project root works; Manual: Custom --prompts-dir works" |
54 | | - - [ ] 5.1 Run full test suite to ensure no regressions |
55 | | - - [ ] 5.2 Update or remove tests that expected old behavior |
56 | | - - [ ] 5.3 Test development workflow (running from project root with local prompts) |
57 | | - - [ ] 5.4 Test custom prompts directory still works when explicitly specified |
| 54 | + - [x] 5.1 Run full test suite to ensure no regressions |
| 55 | + - [x] 5.2 Update or remove tests that expected old behavior |
| 56 | + - [x] 5.3 Test development workflow (running from project root with local prompts) |
| 57 | + - [x] 5.4 Test custom prompts directory still works when explicitly specified |
58 | 58 | - [ ] 5.5 Manual test with uvx installation from GitHub to verify remote install works |
0 commit comments