Skip to content

Usability fixes#11

Merged
utkarshdalal merged 3 commits into
masterfrom
usability-fixes
May 27, 2025
Merged

Usability fixes#11
utkarshdalal merged 3 commits into
masterfrom
usability-fixes

Conversation

@utkarshdalal
Copy link
Copy Markdown
Owner

No description provided.

@utkarshdalal utkarshdalal merged commit 0ff5b7f into master May 27, 2025
1 check failed
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 21, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 23, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 23, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 28, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 28, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 28, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 29, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request Apr 30, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request May 2, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request May 3, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
TideGear added a commit to TideGear/GameNative that referenced this pull request May 4, 2026
The real-Steam launch path is stable across DD / 868-HACK / Shotgun King /
Baba Is You — fold in the remaining structural fixes and strip the
investigation-era diagnostic scaffolding that was left behind.

SteamUtils.kt:
- Replace hardcoded canonical228980Depots table with a runtime PICS resolver
  driven by the child acf's SharedDepots. Fixes permanent "gray Play" on
  games whose shared-redist depot set diverges from DD's (was silently
  writing InstalledDepots{} + BytesToDownload=52MB on those).
- Add SHA-256 verify-and-reheal to STEAM_DLL_REPLACED / STEAM_DLL_RESTORED
  markers in replaceSteamApi/restoreSteamApi so a stale marker can no
  longer lie about on-disk state (2nd-launch black-screen repro).
- Add validateAcfShape() self-check: after writing any child or 228980
  manifest, log Timber.e on shapes known to cause gray Play (Update
  Required bit set, BytesToDownload > 0, InstalledDepots empty when it
  shouldn't be). Catches future regressions loudly instead of silently.

Diagnostic cleanup:
- Drop SteamFixDiagnostics object, STEAM_HELPER_WINDOW_CLASSES,
  REAL_STEAM_STALL_WATCHDOG_MS, and all Timber.tag("SteamFix") log lines
  across SteamUtils, SteamService, SteamAutoCloud, SteamTokenLogin,
  MainViewModel, XServerScreen, ImageFsInstaller.
- Rewrite the comments that referenced the investigation's numbered
  SteamFix list (utkarshdalal#8, utkarshdalal#9, utkarshdalal#11, utkarshdalal#16utkarshdalal#24) so they stand on their own.

UX default:
- Default disableSteamOverlay to true in Container.java, ContainerData.kt,
  and PrefManager.kt. Overlay-on is an opt-in now.

Housekeeping:
- .gitignore: exclude .claude/ session state.
- Drop tracked .claude/scheduled_tasks.lock.
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.

1 participant