Skip to content

Fix installed-game detection and post-install install path handling#11

Merged
weter11 merged 3 commits intorust-remakefrom
codex/fix-steamflow-install-path-issues
Feb 13, 2026
Merged

Fix installed-game detection and post-install install path handling#11
weter11 merged 3 commits intorust-remakefrom
codex/fix-steamflow-install-path-issues

Conversation

@weter11
Copy link
Owner

@weter11 weter11 commented Feb 13, 2026

Motivation

  • Steam libraries were not being detected properly because the default steam_library_path was a hard-coded nonstandard directory, causing installed games to always appear as not installed.
  • The scanning fallback in scan_installed_app_paths_pathbuf() never reached the Steam-detection fallback when the configured path was invalid, so the code fell back to an unexpanded literal path.
  • After an install completes, the UI stamped a temporary download path as the game's install_path instead of the real installation location, preventing the Play button from appearing.

Description

  • Prefer an actual detected Steam install when constructing LauncherConfig::default() by calling detect_steam_path() and only fall back to ~/Games/SteamFlow if detection fails (file: SteamFlow/src/config.rs).
  • Restructure scan_installed_app_paths_pathbuf() to use the configured path when valid, otherwise fall back to detect_steam_path() and then default_steam_root() (file: SteamFlow/src/library.rs).
  • On download completion in poll_download_progress(), rescan installed app paths with scan_installed_app_paths() and set the library entry's install_path to the real path (and update is_installed accordingly), instead of assigning a $TMPDIR download path (file: SteamFlow/src/ui.rs).

Testing

  • Ran cargo fmt --all in SteamFlow/ to format changes (succeeded).
  • Ran cargo check in SteamFlow/ which completed successfully; build warnings unrelated to these changes remain but are non-blocking.

Codex Task

@weter11 weter11 merged commit 9fb2eb3 into rust-remake Feb 13, 2026
0 of 2 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