Skip to content

Commit 868fa63

Browse files
andyclaude
andcommitted
CLAUDE.md: document the eccejobmonitor case-sensitive "all" pitfall
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01ScTj9M9ctPk9ySXcDdt9j9
1 parent fd520a7 commit 868fa63

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

‎CLAUDE.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,20 @@ Both per-user, both non-root, both started automatically by the
166166
job/comm layer, general utilities. No wx dependency.
167167

168168
### Known pitfall classes (found more than once — check for siblings)
169+
- **`eccejobmonitor`'s "enable all parse types" check is case-sensitive**
170+
(`scripts/eccejobmonitor`, `PDTypesEnable()`) — it only recognizes
171+
lowercase `all`, but `Launch.C` hardcodes `"parseTypes ALL"`
172+
(uppercase) when invoking it. The mismatch makes it silently delete
173+
every not-yet-enabled parse descriptor from its live-monitoring match
174+
table, so any `Frequency=all`-style trace property (`GEOMTRACE`, and
175+
presumably any other code's equivalent) never gets extracted during
176+
a run — no error, the job completes normally, only the trace data is
177+
missing. Scalar single-value properties (`TE`, ...) are unaffected,
178+
so this looks exactly like a per-property parsing bug and not the
179+
systemic one it is. Fixed script-side (case-insensitive) rather than
180+
touching `Launch.C`, so it doesn't need a C++ rebuild — if a fresh
181+
build still drops a trace property with an otherwise-correct
182+
`Begin`/`End` match, this fix predates it and something new is wrong.
169183
- **wx3.2/GTK3 layout reentrancy**: `wxWindow::DoSetSize` → `wxEVT_SIZE`
170184
→ `Layout()` → reposition children → another `DoSetSize`, sometimes
171185
non-convergent (stack-overflow crash) or asynchronous (crashes well

0 commit comments

Comments
 (0)