Spaces:
Running
Running
Commit
·
c4a9ea8
1
Parent(s):
2e33030
Update CLAUDE.md with reasoning trace parsing fix documentation
Browse files- Document the reasoning trace parsing fix added 2025-08-07
- Explain the problem, solution, and benefits
- Update existing reasoning trace section to note well-formed requirement
- Add technical details about the fix
CLAUDE.md
CHANGED
|
@@ -52,6 +52,8 @@ The application now supports displaying reasoning traces from models like NuMark
|
|
| 52 |
**Implementation Details:**
|
| 53 |
- New `reasoning-parser.js` module handles detection and parsing of reasoning traces
|
| 54 |
- Supports multiple reasoning formats (`<think>`, `<thinking>`, `<reasoning>` tags)
|
|
|
|
|
|
|
| 55 |
- Formats numbered steps from reasoning content for structured display
|
| 56 |
- Caches parsed reasoning to avoid reprocessing
|
| 57 |
- Exports include optional reasoning trace content
|
|
@@ -329,4 +331,29 @@ The application now displays model processing information when available:
|
|
| 329 |
**Implementation Notes:**
|
| 330 |
- The model info panel only appears when `inference_info` column exists
|
| 331 |
- Supports datasets processed with UV scripts via HF Jobs
|
| 332 |
-
- Gracefully handles datasets without model metadata
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 52 |
**Implementation Details:**
|
| 53 |
- New `reasoning-parser.js` module handles detection and parsing of reasoning traces
|
| 54 |
- Supports multiple reasoning formats (`<think>`, `<thinking>`, `<reasoning>` tags)
|
| 55 |
+
- **Important:** Only well-formed traces with both opening AND closing tags are parsed
|
| 56 |
+
- Malformed traces (missing closing tags) are displayed as plain text
|
| 57 |
- Formats numbered steps from reasoning content for structured display
|
| 58 |
- Caches parsed reasoning to avoid reprocessing
|
| 59 |
- Exports include optional reasoning trace content
|
|
|
|
| 331 |
**Implementation Notes:**
|
| 332 |
- The model info panel only appears when `inference_info` column exists
|
| 333 |
- Supports datasets processed with UV scripts via HF Jobs
|
| 334 |
+
- Gracefully handles datasets without model metadata
|
| 335 |
+
|
| 336 |
+
### Reasoning Trace Parsing Fix (Added 2025-08-07)
|
| 337 |
+
|
| 338 |
+
Fixed an issue where reasoning traces with incomplete or malformed XML tags would cause parsing errors:
|
| 339 |
+
|
| 340 |
+
**Problem:**
|
| 341 |
+
- Some model outputs contained opening `<think>` tags without closing `</think>` tags
|
| 342 |
+
- This appeared to be truncated or malformed model output
|
| 343 |
+
- The parser would attempt to parse these incomplete traces, causing confusion
|
| 344 |
+
|
| 345 |
+
**Solution:**
|
| 346 |
+
- Updated `detectReasoningTrace()` to require BOTH opening and closing tags
|
| 347 |
+
- Added console warnings when incomplete traces are detected
|
| 348 |
+
- Malformed traces are now displayed as plain text instead of being parsed
|
| 349 |
+
|
| 350 |
+
**Benefits:**
|
| 351 |
+
- Cleaner handling of incomplete model outputs
|
| 352 |
+
- No confusing partial reasoning panels for malformed content
|
| 353 |
+
- Maintains full functionality for well-formed reasoning traces
|
| 354 |
+
- Helpful console warnings for debugging
|
| 355 |
+
|
| 356 |
+
**Technical Details:**
|
| 357 |
+
- File: `js/reasoning-parser.js`
|
| 358 |
+
- Only traces with complete XML tags (`<think>...</think>`, `<thinking>...</thinking>`, etc.) are parsed
|
| 359 |
+
- Incomplete traces log: "Incomplete reasoning trace detected - missing closing tags"
|