Commit 4acf70c
feat(chapel): compile mass-panic under Chapel 2.8.0 and wire diff subcommand
Bug fixes from code review:
- spawn() called with list(string) instead of array — add .toArray()
- proc ref classifyRisk invalid syntax — remove erroneous ref prefix
- image.generatedAt/scanSurface and report.createdAt never populated
- loadTemporalIndex discarded all prior snapshot history on each save —
replaced with loadSnapshotCount + loadRawSnapshots + raw-splice append
- dateString() returned epoch seconds — now shells to date -u ISO 8601
- findingsOnly config flag declared but never wired up — fixed
- buildEdges loop used (i+1)..#nodes.size causing out-of-bounds on last element
Chapel 2.8.0 compatibility:
- label is a reserved keyword — renamed all fields/params to tag/snapTag
- map initializer requires new map(K, V) syntax
- string.find(str, int) removed — replaced with character-scan iteration
- string indexing/slicing and casts are now throwing — wrapped in try/catch
- list.this() throws — wrapped in try!
- comparator requires relativeComparator interface
- isFile/isDir throw — safeIsFile/safeIsDir helpers added
- Math module needed for exp() in Imaging
- Map module missing from Temporal.chpl imports
- Protocol.basename conflicts with Path.basename — removed duplicate
New: diff subcommand wired through MassPanic.main() → runDiff() →
diffSnapshots() using loadSnapshotSummaries() line-by-line parser.
Justfile: chapel-build, chapel-build-toolbox, chapel-clean, chapel-scan, chapel-diff.
Mason.toml: bumped to chplVersion 2.8.0.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>1 parent 5c5916a commit 4acf70c
6 files changed
Lines changed: 504 additions & 157 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
133 | 133 | | |
134 | 134 | | |
135 | 135 | | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
136 | 157 | | |
137 | 158 | | |
138 | 159 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
6 | | - | |
| 6 | + | |
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| 33 | + | |
33 | 34 | | |
34 | 35 | | |
35 | 36 | | |
| |||
203 | 204 | | |
204 | 205 | | |
205 | 206 | | |
206 | | - | |
207 | | - | |
208 | | - | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
209 | 212 | | |
210 | 213 | | |
211 | | - | |
212 | | - | |
| 214 | + | |
| 215 | + | |
213 | 216 | | |
214 | | - | |
215 | | - | |
| 217 | + | |
| 218 | + | |
216 | 219 | | |
217 | 220 | | |
218 | 221 | | |
219 | 222 | | |
220 | 223 | | |
221 | 224 | | |
222 | | - | |
| 225 | + | |
223 | 226 | | |
224 | | - | |
225 | | - | |
| 227 | + | |
| 228 | + | |
226 | 229 | | |
227 | 230 | | |
228 | 231 | | |
| |||
260 | 263 | | |
261 | 264 | | |
262 | 265 | | |
263 | | - | |
| 266 | + | |
264 | 267 | | |
265 | 268 | | |
266 | 269 | | |
| |||
0 commit comments