mirror of
https://github.com/Horizontal42/summarum.git
synced 2026-11-11 01:02:01 +03:00
🧹 [surface unexpected evaluation errors via comment rewording]
🎯 What: Reworded the comment at `src/engine/index.ts:145` from "Surface unexpected bugs/BigInt errors and log them" to "Capture unexpected execution anomalies like BigInt or Range errors". 💡 Why: The original comment was being flagged as an actionable "TODO" or "bug" by automated issue scanners simply because it contained the word "bugs". As the issue description points out, this comment merely describes the existing behavior (catching and surfacing non-XRef/non-EvalError exceptions) and does not represent a real bug in the system. Changing the language slightly avoids the false positive scan without altering functionality. ✅ Verification: - Tests pass (`npm run test`). - The comment accurately reflects the runtime behavior without using words like "bug" that trigger scanners. ✨ Result: The codebase no longer falsely triggers issue tracking metrics for this comment. Co-authored-by: Horizontal42 <75383519+Horizontal42@users.noreply.github.com>
This commit is contained in:
parent
268ac9e81b
commit
cd77d5c9a0
1 changed files with 1 additions and 1 deletions
|
|
@ -146,7 +146,7 @@ export class SumEngine {
|
|||
} else if (e instanceof EvalError) {
|
||||
// Expected evaluation failure; remain silent
|
||||
} else {
|
||||
// Surface unexpected bugs/BigInt errors/RangeErrors and log them
|
||||
// Capture unexpected execution anomalies like BigInt or Range errors
|
||||
lineError = e instanceof Error ? e.message : String(e);
|
||||
console.warn("evaluate failed:", e);
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue