Commit 6664b28
build: cache RocksDB C/C++ compile via sccache + add PRD docs (#388)
* build: cache RocksDB C/C++ compile via sccache + add PRD docs
- Wrap CC/CXX with sccache in CI (clippy + build-and-test) and dev.sh
so librocksdb-sys C++ compile is actually cached (was bypassing
RUSTC_WRAPPER, ~18min cold build)
- Add target/ to CI cargo cache path
- Add docs/prd.md + docs/personas-and-user-stories.md synthesized
from .planning/ governance docs
* fix: address CI and documentation review findings
Preserve MSVC selection on Windows while retaining native sccache defaults on Unix. Remove duplicate target cache ownership and align the PRD, user stories, State, and Kanban with the validated review findings.
Constraint: Limit this commit to the seven PR #388 review-fix paths for compiler caching, regression coverage, product documentation, and task state.
Confidence: High; the Windows regression is reproduced by the new shell probe and all seven platform/compiler cases pass after the fix.
Scope-risk: Build wrapper selection, CI cache ownership, and versioned documentation only; no Rust production code, Cargo dependency, Oracle implementation, or Hot Tier implementation changes.
Tested: TDD red reproduced forced GNU CC/CXX on MINGW; green shell probe passed 7/7 cases; Git blob Bash syntax, workflow YAML/cache assertions, documentation ID/table checks, and git diff --cached --check passed.
Not-tested: shellcheck and actionlint are unavailable; full Rust/RocksDB build and new remote Head CI require a later push.
Co-authored-by: OmX <omx@oh-my-codex.dev>
* fix: make compiler regression probe CI-compatible
Give the shell probe the repository ASF license header and preserve its executable bit so Linux can execute the symlinked cargo and sccache test doubles.
Constraint: Limit the follow-up to scripts/tests/test-dev-sccache-env.sh and the two failures reported by new-Head CI.
Confidence: High; the license content matches .licenserc.yaml and the Git index records mode 100755.
Scope-risk: Test harness metadata and header only; no production code, compiler selection logic, workflow topology, or dependency changes.
Tested: shell regression probe passed 7/7 cases; normalized Bash syntax passed; git diff --cached --check passed; staged mode verified as 100755.
Not-tested: GitHub license-eye and hosted Linux execution require the follow-up remote CI run.
Co-authored-by: OmX <omx@oh-my-codex.dev>
---------
Co-authored-by: kiwi-bot <kiwi@example.com>
Co-authored-by: OmX <omx@oh-my-codex.dev>1 parent 5b9d7fd commit 6664b28
7 files changed
Lines changed: 521 additions & 41 deletions
File tree
- .github/workflows
- .planning
- docs
- scripts
- tests
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
73 | 77 | | |
74 | 78 | | |
75 | 79 | | |
| |||
91 | 95 | | |
92 | 96 | | |
93 | 97 | | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
94 | 105 | | |
95 | 106 | | |
96 | 107 | | |
| |||
160 | 171 | | |
161 | 172 | | |
162 | 173 | | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
163 | 181 | | |
164 | 182 | | |
165 | 183 | | |
| |||
234 | 252 | | |
235 | 253 | | |
236 | 254 | | |
237 | | - | |
238 | 255 | | |
239 | 256 | | |
240 | 257 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
16 | 18 | | |
17 | 19 | | |
18 | 20 | | |
19 | 21 | | |
20 | 22 | | |
21 | 23 | | |
22 | 24 | | |
23 | | - | |
| 25 | + | |
24 | 26 | | |
25 | 27 | | |
26 | 28 | | |
| |||
44 | 46 | | |
45 | 47 | | |
46 | 48 | | |
47 | | - | |
| 49 | + | |
48 | 50 | | |
49 | 51 | | |
50 | 52 | | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
55 | 57 | | |
56 | 58 | | |
57 | 59 | | |
| |||
87 | 89 | | |
88 | 90 | | |
89 | 91 | | |
90 | | - | |
| 92 | + | |
91 | 93 | | |
92 | 94 | | |
93 | 95 | | |
94 | 96 | | |
95 | | - | |
| 97 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
10 | 10 | | |
11 | | - | |
| 11 | + | |
12 | 12 | | |
13 | | - | |
| 13 | + | |
14 | 14 | | |
15 | | - | |
| 15 | + | |
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
19 | | - | |
| 19 | + | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
28 | | - | |
29 | | - | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
33 | | - | |
| 33 | + | |
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
37 | | - | |
| 37 | + | |
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
| 41 | + | |
| 42 | + | |
41 | 43 | | |
42 | 44 | | |
43 | 45 | | |
44 | 46 | | |
45 | 47 | | |
46 | | - | |
| 48 | + | |
47 | 49 | | |
48 | 50 | | |
49 | 51 | | |
| |||
55 | 57 | | |
56 | 58 | | |
57 | 59 | | |
58 | | - | |
| 60 | + | |
59 | 61 | | |
60 | 62 | | |
61 | | - | |
| 63 | + | |
62 | 64 | | |
63 | 65 | | |
64 | 66 | | |
65 | | - | |
| 67 | + | |
66 | 68 | | |
67 | 69 | | |
68 | 70 | | |
| |||
80 | 82 | | |
81 | 83 | | |
82 | 84 | | |
83 | | - | |
| 85 | + | |
84 | 86 | | |
85 | 87 | | |
86 | 88 | | |
| |||
94 | 96 | | |
95 | 97 | | |
96 | 98 | | |
97 | | - | |
| 99 | + | |
98 | 100 | | |
99 | | - | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
100 | 107 | | |
101 | | - | |
| 108 | + | |
102 | 109 | | |
103 | 110 | | |
104 | 111 | | |
105 | | - | |
106 | | - | |
107 | | - | |
108 | | - | |
109 | | - | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
110 | 117 | | |
111 | 118 | | |
112 | 119 | | |
| |||
115 | 122 | | |
116 | 123 | | |
117 | 124 | | |
118 | | - | |
119 | | - | |
| 125 | + | |
| 126 | + | |
120 | 127 | | |
121 | 128 | | |
122 | 129 | | |
| 130 | + | |
123 | 131 | | |
124 | 132 | | |
125 | 133 | | |
0 commit comments