ponder-lab / Hybridize-Functions-Refactoring / 25198528697

71%

main: 72%

Build Type

Pull #435

github

Committed by khatchad

Commit Message

Surface tensor-analysis errors via the existing logger (#401)

`PythonTensorAnalysisEngine.performAnalysis` accumulates per-pointer
errors during the run (NPEs, IAEs, etc.); without this fix the
returned `Map<PointerKey, AnalysisError>` from `getErrors()` was
silently discarded. Hybridize would then make refactoring decisions
on a partially-degraded analysis with no diagnostic surface.

Now logs at WARN with the pointer key, full error toString, and a
count summary, mirroring the pattern already used in `TestTensorflow2Model`
on the Ariadne side and in `PythonDriver`'s LSP error reporting.

`com.ibm.wala.cast.lsp.AnalysisError` is not exported as API from
Ariadne's OSGi bundle (Tycho ECJ flags it as access-restricted), so
the error map is erased to `Map<?, ?>` and the value's `toString` is
relied on for the diagnostic message. If Ariadne later exports the
type, the erasure can be tightened to `Map<PointerKey, AnalysisError>`.

Closes #401.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #435: Surface tensor-analysis errors via the existing logger (#401)

Coverage Stats

2 of 4 new or added lines in 1 file covered. (50.0%)

23 existing lines in 1 file now uncovered.

1093 of 1545 relevant lines covered (70.74%)

0.71 hits per line