Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review: Car Mechanic Simulator 2021 #16765

Open
3 of 14 tasks
zsjavorszky opened this issue Dec 7, 2024 · 2 comments
Open
3 of 14 tasks

Review: Car Mechanic Simulator 2021 #16765

zsjavorszky opened this issue Dec 7, 2024 · 2 comments

Comments

@zsjavorszky
Copy link

zsjavorszky commented Dec 7, 2024

Nexus Username

TheSupremeSky

Extension URL

https://www.nexusmods.com/site/mods/1117

Game URL

https://www.nexusmods.com/carmechanicsimulator2021

Existing Extension URL

No response

New features

There is no vortex extension exists for CMS2021. The implementation is a simple basic loader.
I created a collection. It contains the melon loaded and 3 of the most popular mods.
https://next.nexusmods.com/carmechanicsimulator2021/collections/jrp9jm

Information

  • I confirm the above is accurate

Packaging

  • This extension is packaged correctly

Testing

  • This game extension has been tested

Review Tasks

If a task fails, contact the author to request changes before continuing.

  • Double-check for existing extension
  • Is the extension packaged correctly?
  • Does it install into Vortex?
  • Does it correctly discover the game?
  • Does it successfully install a mod?
  • Does it successfully install a collection?
  • Does the game run correctly with the mods installed? ​

When reviewed and passed, please complete the following tasks:

  • Run the GitHub Actions to add to manifest
  • Contact author
  • Ask Community to enable the Vortex button for the game
  • Update the #vortex-announcements channel on Discord
@zsjavorszky zsjavorszky changed the title Review: Car Mechanic SImulator 2021 Review: Car Mechanic Simulator 2021 Dec 7, 2024
@IDCs
Copy link
Contributor

IDCs commented Dec 18, 2024

Hi there @zsjavorszky,

I reviewed your game extension and I'm afraid that it's currently too basic to fully support the game. We usually apply a 80/20 mod support rule when reviewing an extension, and I'm afraid that the lack of information presented to the user regarding the MelonLoader hard requirement is causing the extension to fall below the mark.

To add your extension to the Vortex extensions manifest you would have to either notify the user of this requirement and point him to a download location, or even better, download and install the requirement during your game registration's setup.

Have a look at the automatic downloader for Stardew Valley's SMAPI as an example.
https://github.com/Nexus-Mods/vortex-games/blob/master/game-stardewvalley/SMAPI.ts

@zsjavorszky
Copy link
Author

Hello @IDCs ,

Thank you for taking the time to review my game extension.

I utilized the collection system of Vortex to bundle MelonLoader with the mods, eliminating the need for manual installation. MelonLoader is a required dependency and it configured directly within the collection. I see the Vortex collection system as a very basic package manager, although I’m not certain of the developers' original intentions.

You can test the extension alongside the collection here:
https://next.nexusmods.com/carmechanicsimulator2021/collections/jrp9jm/mods

I understand you have established policies, standards, and workflows that guided your current system design. However, I’d like to highlight a few areas where I believe there are opportunities for improvement.

Game Registration

  • Current Issue: Overlapping responsibilities in the extension.
  • Suggested Change: Game registration should be a separate, simplified module.

Benefits:

  • Allows for small, simple and independent community contribution, development and release cycles.
  • Allows automatic server-side validation (e.g., image type, size).
  • Enables users to handle registration directly through the Vortex UI with client-side validation.

Game Detection

  • Current Issue: Overlapping responsibilities in the extension.
  • Suggested Change: Game detection should be a separate, simplified module.

Benefits:

  • Allows for small, simple and independent community contribution, development and release cycles.
  • Supports implementation of multiple independent detectors (e.g., Steam, GOG) under a standardized approach.
  • Enables server-side validation (e.g., ID length).
  • Enables users to configure detection via the Vortex UI.

Game Mod Installation

  • Current Issue: Integrated too deeply with other extension features.
  • Suggested Change: Separate mod installation as an independent module

Benefits:

  • Allows for small, simple and independent community contribution, development and release cycles.
  • Facilitates describing and managing dependencies between modules.

I appreciate your work, but by by decoupling game registration, detection, and mod installation into smaller, independent modules, you could achieve a simpler development process, faster release cycles, and greater community involvement.

Best regards,
Sky

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants