Skip to content

Conversation

@ManlyMarco
Copy link
Collaborator

No description provided.

@ManlyMarco ManlyMarco requested a review from Copilot October 18, 2025 18:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for sardine mods, which are loaded by the SardineHead plugin, expanding the mod manager's capabilities to handle this new mod format alongside existing zipmods and plugins.

Key changes:

  • Implementation of sardine mod loading infrastructure with SardineModLoader and SardineModInfo classes
  • Addition of UI components for browsing and managing sardine mods
  • Integration of sardine mod support into the existing mod update and management workflows

Reviewed Changes

Copilot reviewed 21 out of 24 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
SardineModLoader.cs Core loader implementation for scanning and parsing sardine mod files (.stp/.st_)
SardineModInfo.cs Data model representing sardine mod metadata and properties
SardineModsWindow.cs UI window for browsing and managing sardine mods with enable/disable/delete functionality
SardineViewer.cs Property viewer for displaying sardine mod details, images, and contents
MainWindow.cs Integration of sardine mod browser into main window menu and refresh logic
InstallDirectoryHelper.cs Addition of sardines directory path configuration
UpdateSourceManager.cs Extension of update ignore list to include disabled sardine mods
Various .resx files Localization resources for Japanese, Russian, and Chinese languages
Files not reviewed (3)
  • src/KKManager/Windows/Content/SardineModsWindow.Designer.cs: Language not supported
  • src/KKManager/Windows/MainWindow.Designer.cs: Language not supported
  • src/KKManager/Windows/ToolWindows/Properties/Viewers/SardineViewer.Designer.cs: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

ManlyMarco and others added 3 commits October 18, 2025 20:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ManlyMarco ManlyMarco merged commit 975f0f1 into master Oct 18, 2025
2 checks passed
@ManlyMarco ManlyMarco deleted the sardines branch October 18, 2025 19:01
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