Draw Omen's Paranoia at its in-game size, 28 m by 9 m - #222
Conversation
Players saw Paranoia stop about 3 m short of the game. We drew the 25 m patch-note travel distance; the game's minimap draws MaxDistance + Targeting Box Offset (28 m) by MinimapWidth (9 m), per the 13.05 Ability_Wraith_Q_NearsightMissile blueprint. Data version 104 keeps each saved Paranoia's Omen end in place on both sides and in lineups. Migrations 39, 97 and 98 reconstruct the old 25 m size so old files are not corrected twice. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (9)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughOmen’s Paranoia dimensions change from 4.3 by 25 meters to 9 by 28 meters. A version-104 migration adjusts saved placements, and strategy migration applies it to current-version and legacy data. ChangesParanoia range migration
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Bug fix Sequence Diagram(s)sequenceDiagram
participant StrategyProvider
participant SunsetScaleMigration
participant ParanoiaRangeMigration
participant AgentWeaponMigration
StrategyProvider->>SunsetScaleMigration: migrate strategy
SunsetScaleMigration-->>StrategyProvider: sunset-migrated strategy
StrategyProvider->>ParanoiaRangeMigration: migrate Paranoia placements
ParanoiaRangeMigration-->>StrategyProvider: Paranoia-migrated strategy
StrategyProvider->>AgentWeaponMigration: migrate strategy
Merge Risk: ⚪ Minimal · up to Saved Paranoia placements retain their Omen-side endpoint while displaying the longer range. No identified issue prevents merging after normal checks. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The observed migration is confined to saved strategy placements and has guards against reapplying during normal reloads. No new access-control or remote-service boundary is evident. Security coverage is incomplete, so the risk is low rather than minimal. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Resilience and Maintainability Implications
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Players reported Omen's Paranoia comes up short compared to the game: MATRIX on Split B Heaven, Stazor on the Haven C retake from Garage. In game, the rectangle reaches Heaven's back wall. Icarus stopped about 3 m short.
Cause. We drew Paranoia at its patch-note travel distance, 25 m by 8.6 m. The rectangle the game shows on the minimap is bigger. I pulled Paranoia's assets from the 13.05 paks (
Ability_Wraith_Q_NearsightMissile) and decompiledUpdateMinimapSize:So Riot draws 28 m by 9 m when aiming level. The projectile itself travels 25 m at 20 m/s with a 4.3 m hit radius; the extra 3 m roughly covers that radius. Map scale is not the cause: every map's scale is within about 1% of Riot's minimap multipliers (Haven draws 25 m as 24.9 m, Split as 25.1 m).
Change. Paranoia is now 28 m by 9 m, matching what players see in game.
Saved abilities store only agent and slot, so the new size applies to every saved Paranoia. Data version 104 keeps each saved Paranoia's Omen end where it was, and the far end grows 3 m. It covers attack and defense pages and lineups, leaves deleted flags alone, and does not touch strategies without a Paranoia (no
lastEditedbump). No Hive fields or adapters changed.The migrations for versions 39, 97 and 98 compute anchors from the ability's size.
abilityDataBeforeVersion104gives them the 25 m size they ran with, the same waymapScaleBeforeVersion98handles Sunset, so old files are not corrected twice.Before / after (Haven, Omen in Garage firing across C; the Omen end is fixed):
Tests
test/paranoia_range_migration_test.dart: pins 28 m by 9 m and the old 25 m by 8.6 m; a v103 strategy keeps the Omen end on both sides and in a lineup, including a deleted Paranoia; the far end moves exactly 3 m; strategies without Paranoia keep identical pages andlastEdited; zip export and import doesn't shift it twice.test/sunset_scale_migration_test.dartnow builds the saved-data anchor from the pre-104 size. Its "other maps unchanged" case now calls the Sunset step directly, since the full chain now moves Paranoia on every map.flutter analyze --no-fatal-infosclean; fullflutter test --no-pub: 638 passed, 4 skipped.Not covered
b44db08) had no migration, so Paranoias saved before it probably moved 7.5 m then. This PR doesn't try to undo that.icarus-cloud.🤖 Generated with Claude Code
Summary by CodeRabbit
Gameplay Data
Bug Fixes
Safe to merge based on the reviewed changes.
What we checked:
Summary
The PR updates Omen’s Paranoia indicator to 28 m by 9 m and adds a versioned migration for saved placements. No actionable issue was found.
Reviews (1) · Last reviewed commit: "Draw Omen's Paranoia at its in-game size..."