refactor(view): align ViewInterface bindings with Marko's simple-binding preference

[markshust]

Summary

The view-latte and view-twig module.php files wrapped ViewInterface in a closure that manually built LatteView / TwigView. The closure existed for a real reason — the engine (Latte\Engine / Twig\Environment) needs LatteEngineFactory::create() (a method call, not direct autowiring). But the closure was at the wrong level: LatteView and TwigView both have fully-autowirable constructors (only type-hinted interfaces/classes).

Per Marko's binding guidance in architecture.md:

Prefer simple bindings. Only use closures when autowiring can't express what you need. If a class takes only type-hinted interfaces/classes as constructor params, it's autowirable — no closure needed.

The closure should sit on the dependency that genuinely needs the factory method, not bubble up to the parent.

Before

return [
    'bindings' => [
        ViewInterface::class => function (ContainerInterface \$container): ViewInterface {
            \$engine = \$container->get(LatteEngineFactory::class)->create();
            \$resolver = \$container->get(TemplateResolverInterface::class);
            return new LatteView(\$engine, \$resolver);
        },
    ],
];

After

return [
    'bindings' => [
        ViewInterface::class => LatteView::class,
        Engine::class => function (ContainerInterface \$container): Engine {
            return \$container->get(LatteEngineFactory::class)->create();
        },
    ],
];

• ViewInterface binding becomes a simple class mapping (autowired)
• The closure is scoped to Latte\Engine (or Twig\Environment for view-twig) — the actual service that needs custom build logic
• LatteView's autowired constructor receives the engine (from closure) and resolver (autowired); the setLoader() side effect still runs in the constructor

Same restructure applied to both view-latte/module.php and view-twig/module.php to keep them symmetric.

Why this matters

Future copy-paste from view's module.php propagates the correct pattern. View is one of the most-likely-to-be-cargo-culted module configurations in the framework, so getting its shape right matters beyond just these two packages.

Test plan

• packages/view-latte/tests/ — 29 tests pass (ModuleTest updated to verify new shape)
• packages/view-twig/tests/ — 48 tests pass (ModuleTest updated)
• packages/view/tests/ — 43 tests pass (no changes here, regression check)

🤖 Generated with Claude Code