Replace JulGetLoggerToLoggerFactory Refaster recipe with imperative visitor

[steve-aom-elliott]

Summary

• Replaces the Refaster-based JulGetLoggerToLoggerFactory recipe with a hand-written JavaIsoVisitor implementation
• Fixes malformed YAML in inline-log4j-api-methods.yml where preconditions: was concatenated onto the description

Problem

The Refaster-generated JulGetLoggerToLoggerFactoryRecipes class was failing to correctly match and transform Logger.getLogger(SomeClass.class.getName()) and Logger.getLogger(SomeClass.class.getCanonicalName()) calls. Instead of simplifying to LoggerFactory.getLogger(SomeClass.class), it was leaving the .getName() / .getCanonicalName() calls intact, causing JulToSlf4jTest.loggerToLoggerFactory() and staticFinalLoggerIsStaticFinal() to fail.

Solution

Replaced the Refaster template with an imperative JavaIsoVisitor that:

1. Matches java.util.logging.Logger.getLogger(String) calls
2. Checks if the argument is a Class.getName() or Class.getCanonicalName() invocation
3. Extracts the class expression and rewrites to LoggerFactory.getLogger(clazz) via JavaTemplate

Also fixed inline-log4j-api-methods.yml where a missing newline caused the preconditions: YAML key to be parsed as part of the description text.

Notes

This was necessary due to

• Add return-type-widening guard to Refaster template engine rewrite-templating#179

Test plan

• All 9 tests in JulToSlf4jTest pass
• Full test suite passes
• recipeCsvGenerate and recipeCsvValidateContent pass

[steve-aom-elliott]

I suppose I'd note that it did convert it with Class rather than Class<?>, though I'm not sure that is really going to make a difference here.

[timtebeek]

Thanks for the fix!