ponder-lab / Hybridize-Functions-Refactoring / 26925541690

78%

push

github

Fix `getImportContext` early return missing later TensorSpec-reaching imports (#586)

* Fix `getImportContext` early return that missed later TensorSpec-reaching imports

`getImportContext` returned on the first tensorflow import, so a bare
`from tensorflow import function` short-circuited the scan and missed a later
`import tensorflow as tf` (or a `TensorSpec` named in the same statement) that
does make the signature emittable — the refactor fell back to a bare
`@function` and dropped the `input_signature`.

Rework to a full pass with precedence: a qualified `import tensorflow [as X]`
(which qualifies `function`, `TensorSpec`, and the dtype constants under one
prefix) wins over a wildcard, which wins over a named `from` import (`function`
unqualified; `TensorSpec` reachable only if also named). Existing import-shape
results are unchanged.

Adds shape-A (bundled `from tensorflow import function, TensorSpec, float32`)
and shape-B (mixed `from ... import function` + `import tensorflow as tf`)
fixtures/tests via the shared helper. Full suite green (512, 0 failures).

Closes #578.

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

* Align bundled-import fixture comment/Javadoc with the actual import

Address #586 review: the shape-A comment and Javadoc quoted
`from tensorflow import function, TensorSpec, float32` but the fixture also
imports `constant` for the call site; include it in the quoted import.

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

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

22 of 22 new or added lines in 1 file covered. (100.0%)

1303 of 1705 relevant lines covered (76.42%)

0.76 hits per line