Skip to content

Conversation

@Kathund
Copy link
Member

@Kathund Kathund commented Nov 22, 2025

Description

Related Issue(s)

Fixes #

How to test

Release Notes

Documentation

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 refactors the bundle management system by separating the update and generation processes into distinct scripts. The main changes include creating a new generate-bundles.sh script to handle bundle generation, simplifying update-bundles.sh to only perform updates, adding package management configuration with pnpm, and updating metadata across numerous mod configuration files with proper IDs and override sections.

Key Changes:

  • Split bundle update and generation into separate scripts for better separation of concerns
  • Added Node.js package management with pnpm configuration
  • Updated mod metadata across multiple .pw.toml files with correct IDs and override information
  • Generated new .mrpack bundle files
  • Removed deprecated mods (tnt-time, redaction, screenshot-to-clipboard) and added replacements (screencopy, modernfix-mvus)

Reviewed changes

Copilot reviewed 77 out of 101 changed files in this pull request and generated no comments.

Show a summary per file
File Description
oneclient/update-bundles.sh Simplified to only update packages, removed bundling logic
oneclient/generate-bundles.sh New script dedicated to generating .mrpack bundles
package.json Added Node.js package configuration with pnpm setup
pnpm-lock.yaml Added pnpm lockfile for @iarna/toml dependency
oneclient/bundles.json Minor formatting fix (added newline at EOF)
oneclient/mrpacks/**/mods/*.pw.toml Updated mod IDs and added override sections with metadata
oneclient/mrpacks/**/index.toml Updated hashes to reflect metadata changes
oneclient/mrpacks/**/pack.toml Updated index hashes
oneclient/generated/*.mrpack Generated bundle files
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Wyvest Wyvest merged commit e74257a into Polyfrost:main Nov 22, 2025
6 checks passed
@Kathund Kathund deleted the fix/bundles branch November 22, 2025 15: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