@grunnverk/kodrdriv 1.5.0 → 1.5.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.
@@ -1,79 +0,0 @@
1
- # Bug: tree_publish passes incorrect --config-dir path
2
-
3
- ## Problem
4
-
5
- When `tree_publish` executes `kodrdriv publish` for each package, it passes `--config-dir` with the parent directory path, but kodrdriv's config discovery expects either:
6
- 1. The full path to the config directory (`.kodrdriv`), or
7
- 2. No `--config-dir` flag at all (to allow hierarchical discovery)
8
-
9
- ## Current Behavior
10
-
11
- When `tree_publish` runs from `/Users/tobrien/gitw/grunnverk/kodrdriv`, it passes:
12
- ```
13
- --config-dir "/Users/tobrien/gitw/grunnverk/kodrdriv"
14
- ```
15
-
16
- This causes kodrdriv to look for a config directory named `kodrdriv` instead of `.kodrdriv`.
17
-
18
- When `tree_publish` runs from `/Users/tobrien/gitw/grunnverk`, it passes:
19
- ```
20
- --config-dir "/Users/tobrien/gitw/grunnverk"
21
- ```
22
-
23
- This causes kodrdriv to look for a config directory named `grunnverk` instead of `.kodrdriv`.
24
-
25
- ## Expected Behavior
26
-
27
- The config file exists at: `/Users/tobrien/gitw/grunnverk/.kodrdriv/config.yaml`
28
-
29
- `tree_publish` should either:
30
- 1. Pass `--config-dir "/Users/tobrien/gitw/grunnverk/.kodrdriv"` (full path to config directory), or
31
- 2. Not pass `--config-dir` at all and let hierarchical discovery find `.kodrdriv` automatically
32
-
33
- ## Error Details
34
-
35
- ```
36
- Command failed: kodrdriv publish --config-dir "/Users/tobrien/gitw/grunnverk"
37
- [winston] ... "Using hierarchical discovery: configDirName=grunnverk, startingDir=/Users/tobrien/gitw"
38
- [winston] ... "Checking for config directory: /Users/tobrien/gitw/grunnverk"
39
- [winston] ... "Found config directory at level 0: /Users/tobrien/gitw/grunnverk"
40
- [winston] ... "Attempting to load config file: /Users/tobrien/gitw/grunnverk/config.yaml"
41
- [winston] ... "Config file not found at /Users/tobrien/gitw/grunnverk/config.yaml"
42
- ```
43
-
44
- The system is looking for `/Users/tobrien/gitw/grunnverk/config.yaml` but the actual file is at `/Users/tobrien/gitw/grunnverk/.kodrdriv/config.yaml`.
45
-
46
- ## Location of Issue
47
-
48
- The issue is in `/Users/tobrien/gitw/grunnverk/commands-tree/src/commands/tree.ts` around line 2476:
49
-
50
- ```typescript
51
- if (runConfig.configDirectory) globalOptions.push(`--config-dir "${runConfig.configDirectory}"`);
52
- ```
53
-
54
- ## Investigation Needed
55
-
56
- 1. Find where `runConfig.configDirectory` is set in the tree_publish flow
57
- 2. Determine if it should:
58
- - Be set to the full path of the `.kodrdriv` directory (e.g., `/Users/tobrien/gitw/grunnverk/.kodrdriv`)
59
- - Not be set at all (let hierarchical discovery work)
60
- - Be set differently based on how kodrdriv's config discovery works
61
-
62
- ## Suggested Fix
63
-
64
- Option 1: If `configDirectory` should point to the config directory itself:
65
- - When setting `runConfig.configDirectory`, append `/.kodrdriv` to the directory path
66
- - Or detect the config directory location and use that full path
67
-
68
- Option 2: If hierarchical discovery should handle it:
69
- - Don't set `runConfig.configDirectory` when it would point to a parent directory
70
- - Only set it when explicitly provided by the user or when pointing to the actual `.kodrdriv` directory
71
-
72
- ## Testing
73
-
74
- After fixing, verify that:
75
- 1. `tree_publish` can find the config at `/Users/tobrien/gitw/grunnverk/.kodrdriv/config.yaml`
76
- 2. The publish workflow completes successfully
77
- 3. Both scenarios work:
78
- - Running from `/Users/tobrien/gitw/grunnverk/kodrdriv`
79
- - Running from `/Users/tobrien/gitw/grunnverk`