Skip to content

Add SUPER|Brie+ Factory pop#342

Merged
AardWolf merged 3 commits intotsitu:masterfrom
hymccord:sb-factory
Mar 4, 2026
Merged

Add SUPER|Brie+ Factory pop#342
AardWolf merged 3 commits intotsitu:masterfrom
hymccord:sb-factory

Conversation

@hymccord
Copy link
Contributor

@hymccord hymccord commented Mar 4, 2026

Copilot AI review requested due to automatic review settings March 4, 2026 22:14
@hymccord hymccord marked this pull request as draft March 4, 2026 22:18
Copy link
Contributor

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

Adds SUPER|brie+ Factory support to the bookmarklet “phase” detection and to the population (pop) data generator, enabling room-specific attraction-rate exports for that location.

Changes:

  • Add SUPER|brie+ Factory sublocation/phase detection (Boss vs room vs Any Room) in bookmarklet code.
  • Add a new pop setup definition for SUPER|brie+ Factory and commit the generated CSV.
  • Add a pop:sbf npm script to generate the new CSV from the pop setup.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/bookmarklet/bm-setup-fields.js Adds SUPER
src/bookmarklet/bm-cre.js Adds SUPER
package.json Adds pop:sbf script to generate the SBF population CSV.
data/pop-js/super-brie-factory.js Defines room/cheese/charm query series and post-process Boss row for SBF pop generation.
data/pop-csv/super-brie-factory.csv Adds generated population CSV output for SBF.

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

@hymccord hymccord marked this pull request as ready for review March 4, 2026 22:36
@AardWolf AardWolf merged commit 546d9a8 into tsitu:master Mar 4, 2026
2 checks passed
@hymccord hymccord deleted the sb-factory branch March 5, 2026 16:13
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.

3 participants