ponder-lab / Hybridize-Functions-Refactoring / #2184

79%

push

github

Remove the entire `@tf.function` when de-hybridizing a parameterized decorator (#682)

`convertToEager()` computed the decorator's delete span from the decorator name
alone (`getFullRepresentationString(decorator.func).length() + 1`), so de-hybridizing
a function whose `@tf.function` carries arguments deleted only `@tf.function` and
orphaned the argument list as an invalid bare `(...)`. Extend the span through the
argument list with a matching-bracket scan; a bare `@tf.function` (no following `(`)
is unchanged.

Adds `testConvertToEagerParameterizedDecorator`, an in-memory output comparison that
de-hybridizes `@tf.function(reduce_retracing=True)` and asserts the whole decorator
is removed.

Closes #681


Claude-Session: https://claude.ai/code/session_01FuhM8SsTRB6BraLt3Ph7ZC

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

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

1665 of 2100 relevant lines covered (79.29%)

0.79 hits per line