Skip to content

fix(): add correct extension for gog and custom games.#473

Open
phobos665 wants to merge 3 commits intoutkarshdalal:masterfrom
phobos665:fix/add-gog-extension
Open

fix(): add correct extension for gog and custom games.#473
phobos665 wants to merge 3 commits intoutkarshdalal:masterfrom
phobos665:fix/add-gog-extension

Conversation

@phobos665
Copy link
Contributor

@phobos665 phobos665 commented Feb 2, 2026

Summary by cubic

Set export file extensions for GOG (.gog), Epic (.epic), and Custom (.pcgame) so archives are correctly named.

Written for commit 3016fe1. Summary will update on new commits.

Summary by CodeRabbit

  • Bug Fixes
    • Updated export file extensions for gaming platform integrations to ensure proper file format handling across Steam, Epic Games, and GOG platforms.

@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Walkthrough

Three app screen classes updated to return new file export extensions: CustomGameAppScreen exports as ".pcgame" instead of ".steam", EpicAppScreen exports as ".epic" instead of ".epicgame", and GOGAppScreen exports as ".gog" instead of "tzst".

Changes

Cohort / File(s) Summary
App Screen Export Extensions
app/src/main/java/app/gamenative/ui/screen/library/appscreen/CustomGameAppScreen.kt, app/src/main/java/app/gamenative/ui/screen/library/appscreen/EpicAppScreen.kt, app/src/main/java/app/gamenative/ui/screen/library/appscreen/GOGAppScreen.kt
Updated getExportFileExtension() return values across three app screen classes. CustomGameAppScreen: ".steam" → ".pcgame". EpicAppScreen: ".epicgame" → ".epic". GOGAppScreen: "tzst" → ".gog" (with logging removal).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Three screens hop with extensions new,
Steam becomes a PCgame too,
Epic shrinks, and GOG takes flight,
Export formats shining bright!
Format changes, clean and neat,
Made this tiny diff complete!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: it states the PR fixes extensions for GOG and custom games, which aligns with the primary modifications across three files (GOG, custom, and Epic app screens).

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@phobos665 phobos665 changed the title fix(): add correct extension for gog games. fix(): add correct extension for gog and custom games. Feb 2, 2026
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