Skip to content

Fix Workshop Mods for YOMI:HUSTLE when using bionic steam#1746

Merged
utkarshdalal merged 2 commits into
utkarshdalal:masterfrom
Nightwalker743:fix/yomi-hustle-bionic-steam
Jul 20, 2026
Merged

Fix Workshop Mods for YOMI:HUSTLE when using bionic steam#1746
utkarshdalal merged 2 commits into
utkarshdalal:masterfrom
Nightwalker743:fix/yomi-hustle-bionic-steam

Conversation

@Nightwalker743

@Nightwalker743 Nightwalker743 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Description

Fix Workshop Mods for YOMI:HUSTLE when using bionic steam

Recording

https://drive.google.com/file/d/1E_O43I_LkIXG4x8QPA-Fu3l6uVTVPzZ6/view?usp=drivesdk

Type of Change

  • Bug fix
  • Performance / stability improvement
  • Compatibility improvements
  • Other (requires prior approval)

Checklist

  • If I have access to #code-changes, I have discussed this change there and it has been green-lighted. If I do not have access, I have still provided clear context in this PR. If I skip both, I accept that this change may face delays in review, may not be reviewed at all, or may be closed.
  • This change aligns with the current project scope (core functionality, stability, or performance). If not, it has been explicitly approved beforehand.
  • I have attached a recording of the change.
  • I have read and agree to the contribution guidelines in CONTRIBUTING.md.

Summary by cubic

Fixes YOMI:HUSTLE Workshop mods on Bionic Steam by symlinking original mod ZIPs (no extraction) and restoring missing archives so multiplayer hashes match. Tightens update checks to auto-restore legacy installs that deleted ZIPs.

  • Bug Fixes
    • Added YOMI_HUSTLE_APP_ID and force .zip as flat-file payloads for this game.
    • Detect legacy YOMI installs during sync and update checks; re-download once to restore the original ZIP.
    • Extended WorkshopSymlinker with flatFileExtensions and applied to YOMI in auto and manual-path syncing, with clear warnings when no ZIP payloads are found.

Written for commit 9ef3d4f. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Bug Fixes
    • Improved Workshop mod syncing specifically for Yomi Hustle.
    • ZIP-based mods are now handled more reliably, reducing unintended extraction during setup.
    • Added recovery to restore missing original ZIP archives when older installations left extracted files behind.
    • Enhanced detection and placement of ZIP-based workshop content for both manual and automatic mod configuration.
    • Added warnings when filtered ZIP payloads can’t be found, making sync issues easier to diagnose.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Yomi Hustle workshop handling now preserves ZIP payloads, detects when extracted content requires ZIP restoration, and applies ZIP-specific flat-file filtering in both manual and automatic mod symlink flows.

Changes

Yomi Hustle workshop synchronization

Layer / File(s) Summary
Extension-filtered flat-file symlinking
app/src/main/java/app/gamenative/workshop/WorkshopSymlinker.kt
WorkshopSymlinker.sync propagates normalized extension filters, forces flat-file handling when configured, filters payloads, and logs when no matching files are found.
Yomi Hustle workshop integration
app/src/main/java/app/gamenative/workshop/WorkshopManager.kt
Yomi Hustle skips ZIP extraction, detects missing ZIP archives after prior extraction, and passes ZIP-specific filtering through manual and automatic symlink configuration.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant WorkshopManager
  participant WorkshopSymlinker
  participant WorkshopItemDirectory
  WorkshopManager->>WorkshopSymlinker: Sync Yomi Hustle with ZIP extension filter
  WorkshopSymlinker->>WorkshopItemDirectory: Select ZIP flat-file payloads
  WorkshopSymlinker->>WorkshopItemDirectory: Create symlinks for selected payloads
Loading

Possibly related PRs

Suggested reviewers: utkarshdalal

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly reflects the main YOMI:HUSTLE workshop mod fix and stays concise.
Description check ✅ Passed The description follows the template and includes the required sections, recording, change type, and checklist items.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread app/src/main/java/app/gamenative/workshop/WorkshopManager.kt Outdated
@utkarshdalal
utkarshdalal merged commit 2fb980d into utkarshdalal:master Jul 20, 2026
3 checks passed
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