Refactor JCA to extract OpenGL/EGL effects into dedicated :core📷effects module

[davidjiagoogle]

This PR modularizes the OpenGL/EGL surface-processing and single-stream effects in the Jetpack Camera App (JCA), separating them into a dedicated :core:camera:effects module.

Key Changes:

1. New Interface: Defined SingleStreamEffectProvider in :core:camera as the injection contract.
2. Modularized Effects: Moved the effects/ package hierarchy (including SingleSurfaceForcingEffect, CopyingSurfaceProcessor, shaders, and GL helper classes) from :core:camera to :core:camera:effects.
3. Decoupled Dependencies: Transferred libs.androidx.graphics.core (OpenGL/EGL wrapper library) to :core:camera:effects's build.gradle.kts.
4. Hilt Injection: Registered SingleStreamEffectProviderImpl using Hilt @IntoSet multi-binding. CameraXCameraSystem now injects Set<SingleStreamEffectProvider> and passes the effect provider to CameraSession dynamically.
5. Configurable Exclusion: By separating effects into a new module, we enable build configurations (like "lite" builds) to completely exclude the :core:camera:effects dependency, preventing the androidx.graphics:graphics-core library (~15KB) from being compiled and bundled into the production APK.

Tried out removing effects module in the app layer in the david/splitEffectsTryout branch

[gemini-code-assist]

Code Review

This pull request modularizes the camera effects by introducing a new :core:camera:effects module and decoupling SingleSurfaceForcingEffect from the core camera session. It defines a SingleStreamEffectProvider interface and injects its implementation using Hilt multibindings. The review feedback highlights a potential compilation error if the effects module is excluded, which can be resolved using @Multibinds. Additionally, suggestions were made to adhere to the repository style guide by restricting visibility to internal, adding KDoc documentation, and adjusting the Hilt component scope.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[temcguir]

The overall direction of moving effects into a dedicated module and injecting them to allow for "lite" builds is solid. However, there are a few architectural issues with the current implementation that prevent it from being truly extensible for arbitrary custom effects.

Most notably, the PR description states that this uses Hilt @IntoSet multi-binding, but the code actually relies on @BindsOptionalOf to inject a single Optional<SingleStreamEffectProvider>. This single-instance injection prevents multiple effects from coexisting and forces developers to completely overwrite the module if they want to inject their own custom effect.

To resolve this, I've left a few inline comments outlining an approach that aligns with our existing LowLightBoost and ImagePostProcessor implementations:

1. Generalize the interface: Rename SingleStreamEffectProvider to a generic CameraEffectProvider.
2. Use a Map Registry: Replace CameraSessionOptionalModule with a @Multibinds module inside :data:camera to provide a Map<CameraEffectFeatureKey, Provider<CameraEffectProvider>>. This guarantees the Map is always present (even if empty in lite builds) and completely eliminates the need for Optional injection.
3. Drive via Settings: Treat the active effect as a configurable UI state by adding it to CameraAppSettings (and SessionSettings), rather than hardcoding it in startCamera. CameraSession can then simply use the setting state to look up the correct provider from the injected Map registry.