Skip to content

Red Brinstar Fireflea Room: low energy spark to block - #2972

Open
dragonsinth wants to merge 3 commits into
vg-json-data:masterfrom
dragonsinth:rbfleaspark
Open

Red Brinstar Fireflea Room: low energy spark to block#2972
dragonsinth wants to merge 3 commits into
vg-json-data:masterfrom
dragonsinth:rbfleaspark

Conversation

@dragonsinth

@dragonsinth dragonsinth commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Copilot AI lite review requested due to automatic review settings September 9, 2026 02:07

Copilot AI 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.

🟢 Approval recommended

The changes are self-contained data additions with consistent schema usage (new strats without IDs, notable added with updated nextNotableId) and no detected structural issues.

Pull request overview

Adds a new “Low Energy Spark to Block” technique/strat option for Red Brinstar Fireflea Room, expanding the room’s shinespark routing logic to account for intentionally low-energy horizontal midair sparks.

Changes:

  • Added two new strats for reaching the block via a low-energy midair horizontal shinespark when entering shinecharged or shinecharging.
  • Added a corresponding notable (“Low Energy Spark to Block”) and advanced nextNotableId.
File summaries
File Description
region/brinstar/red/Red Brinstar Fireflea Room.json Adds new low-energy shinespark strats and a notable to document/enable the technique.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kjbranch kjbranch 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.

This needs a much more detailed description.

  • where to spark from
  • mention that being higher gives more wiggle room when falling

{"shineChargeFrames": 42},
"canHorizontalMidairShinespark",
{"resourceAtMost": [{"type": "RegularEnergy", "count": 99}]},
{"shinespark": {"frames": 70, "excessFrames": 4}}

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.

I think with these excess frames, you would need to start in a different position, which is then canControllShinesparkEnd.

Suggested change
{"shinespark": {"frames": 70, "excessFrames": 4}}
{"shinespark": {"frames": 70, "excessFrames": 0}}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

When I was testing this out, I was normalizing the start position to be close to the ceiling, and 2-3 tiles right of the end of the platform. So then from that start, with 95-99 hp I was close enough to the block to fall into it by holding left or right. So this line was me trying to reflect what I tested, but I'm not dogmatic about it, we could just say you need exactly 99.

@dragonsinth dragonsinth Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Re-testing, at 95 you need a pretty big jump forward. So I think your inclination to just normalize at 99 is probably right. With 99 energy, the startup position is incredibly lenient, anywhere from directly over the last platform tile, up to 6 tiles out, works as long as you're close to the ceiling.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Basically there's a 7-tile window that works, because ending the spark anywhere from -3 to +3 allows you to fall onto to the block.

@dragonsinth

Copy link
Copy Markdown
Contributor Author

This needs a much more detailed description.

  • where to spark from
  • mention that being higher gives more wiggle room when falling

True, in my head I was imagining that it's paired with a video; but I'll call this out in more detail.

@dragonsinth

Copy link
Copy Markdown
Contributor Author

Updated the description, removed the excess frames.

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