[xHeaven]

Replacing nette/php-generator with our in-house engine.

Benchmark	Nette	Now	Time	Memory
Generate a class	215µs	83µs	−61%	−47%
Manipulate a stub	907µs	417µs	−54%	−83%
Manipulate an 800-line class	14.7ms	7.4ms	−50%	−85%
Detect stub types	347µs	158µs	−55%	−66%

This is breaking, because I removed the manipulate() method. It ran string edits on the printed code, which then required printing, reparsing and printing again. This is also one of the reasons why we pinned the package version in the first place, since we lost comments, had weird imports, etc. Stub replacements now apply to the source before parsing. If you want to change generated classes, use the proper model methods from now on: setNamespace(), addMethod(), removeClassAttribute() and so on.

[github-actions]

Benchmark Results

Comparison of feat/generation against 3.x (86b06a9455206fef95386d07aa7dd2f9a1e1a6a0).

Open to see the benchmark results

Benchmark	Set	Mem. Peak	Time	Variability
ViewRenderBench(benchPlainHtml)	-	22.182mb -0.04%	620.739μs -15.60%	±1.46% -50.69%
ViewRenderBench(benchExpressions)	-	24.772mb -0.04%	670.208μs -5.56%	±2.91% +54.60%
ViewRenderBench(benchControlFlow)	-	44.875mb -0.02%	819.977μs -6.14%	±2.66% +65.52%
ViewRenderBench(benchViewComponent)	-	70.391mb -0.01%	1.607ms -5.88%	±1.75% +126.67%
GenericRouteMatcherBench(benchMatch)	Dynamic	4.683mb -0.20%	3.171μs +5.93%	±2.53% +93.70%
ContainerBench(benchAutowireSimple)	-	4.095mb -0.23%	3.389μs +6.12%	±2.91% +240.28%

_{Generated by phpbench against commit ecc47be}

[xHeaven]

Fixed the CI issue by readding nette/schema with a version pin. Oh boy.

[brendt]

Looks great. As far as I'm considered, I think we can merge this as a minor despite the breaking change. However, is there a way we can catch it with Rector and do something reasonable with it just in case someone used it on the project/package level?

[xHeaven]

Looks great. As far as I'm considered, I think we can merge this as a minor despite the breaking change. However, is there a way we can catch it with Rector and do something reasonable with it just in case someone used it on the project/package level?

Not really, not reliably. In my opinion it's fine to merge as-is.