Skip to content

Refactor recorder modules into tested JVM components - #4

Merged
ElliotBadinger merged 2 commits into
refactor/phase1-modularization-kts-hiltfrom
codex/complete-comprehensive-app-refactoring
Oct 5, 2025
Merged

Refactor recorder modules into tested JVM components#4
ElliotBadinger merged 2 commits into
refactor/phase1-modularization-kts-hiltfrom
codex/complete-comprehensive-app-refactoring

Conversation

@ElliotBadinger

Copy link
Copy Markdown
Owner

Summary

  • convert the data and recorder feature modules to JVM builds with explicit javax.inject support and remove obsolete Hilt bindings
  • implement a stateful RecordingRepository backed by new domain models and propagate dedicated use cases for listening, recording, and saving
  • rewrite the RecordingViewModel around coroutine scope management and add comprehensive repository, use case, and view model unit tests

Testing

  • ./gradlew :domain:test :data:test :features:recorder:test

https://chatgpt.com/codex/tasks/task_b_68e07d7d28848323a5a357432a7fbf78

@openhands-ai

openhands-ai Bot commented Oct 4, 2025

Copy link
Copy Markdown

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • unit-tests
    • Android CI
    • Cross-Platform CI
    • Cross-Platform CI
    • Android CI

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #4 at branch `codex/complete-comprehensive-app-refactoring`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

@ElliotBadinger
ElliotBadinger merged commit 7c1e2d9 into refactor/phase1-modularization-kts-hilt Oct 5, 2025
22 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant