Skip to content

Verify bugfix 60776b0 (screen timeout prevention) already merged#36

Closed
9thLevelSoftware with Copilot wants to merge 1 commit into
beta0.2.1from
copilot/fix-bug-with-id-60776b0
Closed

Verify bugfix 60776b0 (screen timeout prevention) already merged#36
9thLevelSoftware with Copilot wants to merge 1 commit into
beta0.2.1from
copilot/fix-bug-with-id-60776b0

Conversation

Copilot AI commented Dec 31, 2025

Copy link
Copy Markdown
Contributor

Bugfix commit 60776b017da9f45a5af007a4d3e61ecd3d4ca32f was requested for merge to main, but analysis confirms all changes already exist in the codebase.

Changes Already Present

The screen timeout prevention feature is fully implemented via expect/actual pattern:

  • Common: ScreenUtils.kt defines setKeepScreenOn(enabled: Boolean)
  • Android: ActivityHolder pattern with FLAG_KEEP_SCREEN_ON on window
  • iOS: UIApplication.sharedApplication.setIdleTimerDisabled
  • Integration: ActiveWorkoutScreen uses DisposableEffect to enable on mount, disable on unmount
// ActiveWorkoutScreen.kt
DisposableEffect(Unit) {
    setKeepScreenOn(true)
    onDispose {
        setKeepScreenOn(false)
    }
}

Status

No action required. Base commit (78cbbdd) already contains the bugfix. This PR documents the verification.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dl.google.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -XX:+UseParallelGC -Xmx8g -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

I need this bugfix merged to Main: 60776b0


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix bug with ID 60776b017da9f Verify bugfix 60776b0 (screen timeout prevention) already merged Dec 31, 2025
@9thLevelSoftware 9thLevelSoftware deleted the copilot/fix-bug-with-id-60776b0 branch December 31, 2025 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants