SAM Launcher Defense Radius

[wnusair]

Description:

SAM Launcher defense radius visible

Please complete the following:

• I have added screenshots for all UI updates
• I process any text displayed to the user through translateText() and I've added it to the en.json file
• I have added relevant tests to the test directory
• I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced

Please put your Discord username so you can be contacted if a bug or regression is found:

mayfly404

[coderabbitai]

Walkthrough

Adds a Prettier import-sorting plugin and new dependencies, introduces an exported SAMRadiusLayer class for drawing SAM launcher radii, and updates GameRenderer imports and constructor fields (many made readonly). GameRenderer instantiates SAMRadiusLayer but the layer is not added to the rendered layers array in this diff.

Changes

Cohort / File(s)	Summary
Tooling: Prettier & deps \.prettierrc, package.json	Enables @trivago/prettier-plugin-sort-imports in Prettier overrides and adds several new deps/devDeps (including @trivago/prettier-plugin-sort-imports, @stylistic/eslint-plugin, @swc/core, @swc/jest, @total-typescript/ts-reset, intl-messageformat, seedrandom, @types/seedrandom, eslint-plugin-jest), and removes ts-jest. Scripts unchanged.
Rendering: SAM radius layer & renderer updates src/client/graphics/GameRenderer.ts, src/client/graphics/layers/SAMRadiusLayer.ts	Adds exported SAMRadiusLayer implementing Layer (overlay canvas, tick/redraw, draws dashed yellow radius using defaultSamRange). Updates GameRenderer imports and constructor/fields to readonly, initializes a SAMRadiusLayer instance (but does not add it to the layers array in this patch), adjusts emoji table init and UILayer constructor usage.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Frame as "Frame tick"
  participant GameRenderer
  participant SAMRadiusLayer
  participant GameView
  participant TransformHandler
  participant OverlayCanvas as "Overlay Canvas"
  participant MainCanvas as "Main Canvas"

  Note over GameRenderer,SAMRadiusLayer: setup
  GameRenderer->>SAMRadiusLayer: new(game, eventBus, transformHandler)
  GameRenderer->>SAMRadiusLayer: init()
  SAMRadiusLayer->>OverlayCanvas: create canvas + getContext('2d')
  SAMRadiusLayer->>SAMRadiusLayer: redraw() — populate samLaunchers

  loop per frame
    Frame->>GameRenderer: tick frame
    GameRenderer->>SAMRadiusLayer: tick()
    SAMRadiusLayer->>GameView: updatesSinceLastTick()
    SAMRadiusLayer->>SAMRadiusLayer: update samLaunchers, set needsRedraw
    alt needsRedraw or transform changed
      SAMRadiusLayer->>SAMRadiusLayer: redraw() — draw dashed/yellow circles
    end
    GameRenderer->>SAMRadiusLayer: renderLayer(mainCtx)
    SAMRadiusLayer->>MainCanvas: draw OverlayCanvas (centered)
  end

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

• Display FPS monitor overview #1573 — related modifications to GameRenderer and FPSDisplay integration overlap with this PR's GameRenderer edits.
• Fix all strict errors in /client #1489 — changes to Layer interface/typings that affect SAMRadiusLayer and GameRenderer.
• Enable eslint rule @stylistic/ts/indent #1779 — adds/enables @stylistic ESLint plugin and lint config changes related to newly added devDependency.

Suggested labels

Feature - Frontend, UI/UX

Suggested reviewers

• evanpelle
• drillskibo
• scottanderson

Pre-merge checks (2 passed, 1 warning)

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title "SAM Launcher Defense Radius" succinctly and accurately describes the primary change — adding a visible SAM launcher defense radius to the UI — and is concise and specific enough for teammates scanning PR history.
Description Check	✅ Passed	The pull request description is directly related to the changeset, includes a screenshot of the UI change, and lists checklist items confirming tests and i18n updates, so it meets the lenient description requirements.

Poem

Bright circles bloom on overlayed skies,
Yellow dashed rings mark watchful eyes.
Imports tidy, layers newly born,
Canvas echoes at early morn.
Small changes stack — the game adorns.

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
7 out of 25 committers have signed the CLA.

✅ DevelopingTom
✅ Baguette8ot
✅ woodydrn
✅ evanpelle
✅ abdallahbahrawi1
✅ TheGiraffe3
✅ Boostry123
❌ scottanderson
❌ pierrebrtr
❌ Aotumuri
❌ TylerHanavan
❌ dependabot[bot]
❌ djmrFunnyMan
❌ Kipstz
❌ Cameron Clark
❌ TLS15
❌ Killersoren
❌ fantom845
❌ ElMelchizedek
❌ PilkeySEK
❌ mokizzz
❌ Bornasm
❌ wnusair
❌ VariableVince
❌ aaa4xu

---

Cameron Clark seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (9)

package.json (1)

36-36: Import-sorting plugin is added but not actually used; consider removing or enabling it globally (pick one).

Right now, "@trivago/prettier-plugin-sort-imports" is only installed but not referenced in root plugins. Also, the Dockerfile override in .prettierrc won’t use it (that plugin doesn’t handle Dockerfiles). Keep just one import sorter to avoid churn with "prettier-plugin-organize-imports".

Option A — remove unused plugin:

-    "@trivago/prettier-plugin-sort-imports": "^5.2.2",

Option B — switch to trivago plugin globally (and drop organize-imports to avoid conflicts):

   "devDependencies": {
+    "@trivago/prettier-plugin-sort-imports": "^5.2.2",
     "prettier": "^3.5.3",
-    "prettier-plugin-organize-imports": "^4.1.0",

…and in .prettierrc root:

-  "plugins": ["prettier-plugin-organize-imports", "prettier-plugin-sh"]
+  "plugins": ["@trivago/prettier-plugin-sort-imports", "prettier-plugin-sh"],
+  "importOrder": ["^@?(\\w|/)", "^\\.", "^\\u0000"], 
+  "importOrderSeparation": true,
+  "importOrderSortSpecifiers": true

.prettierrc (1)

12-12: Wrong plugin for Dockerfile; use a Dockerfile formatter plugin instead.

"@trivago/prettier-plugin-sort-imports" won’t parse Dockerfiles. If the intent is to format Dockerfiles, use "prettier-plugin-docker".

Apply:

   {
     "files": "Dockerfile",
     "options": {
-      "plugins": ["@trivago/prettier-plugin-sort-imports"]
+      "plugins": ["prettier-plugin-docker"]
     }
   }

And add the dependency:

   "devDependencies": {
+    "prettier-plugin-docker": "^0.2.0",
   }

src/client/graphics/GameRenderer.ts (1)

203-203: Constructor seems over-injected; SAMRadiusLayer doesn’t use eventBus.

Trim the dependency to reduce coupling.

-  const samRadiusLayer = new SAMRadiusLayer(game, eventBus, transformHandler);
+  const samRadiusLayer = new SAMRadiusLayer(game, transformHandler);

(See the matching constructor change in SAMRadiusLayer.ts below.)

src/client/graphics/layers/SAMRadiusLayer.ts (6)

17-30: Offscreen canvas sizing: watch out for memory and stale dimensions.

Sizing to full map once can be heavy on large maps and won’t react if map size changes. Consider lazy (re)size on first redraw and when map dims change.

-  ) {
-    this.canvas = document.createElement("canvas");
+  ) {
+    this.canvas = document.createElement("canvas");
     const ctx = this.canvas.getContext("2d");
     if (!ctx) {
       throw new Error("2d context not supported");
     }
     this.context = ctx;
-    this.canvas.width = this.game.width();
-    this.canvas.height = this.game.height();
+    // Defer sizing until first redraw to avoid allocating huge buffers too early.

And in redraw():

   redraw() {
+    // Ensure canvas matches current map size
+    if (this.canvas.width !== this.game.width() || this.canvas.height !== this.game.height()) {
+      this.canvas.width = this.game.width();
+      this.canvas.height = this.game.height();
+    }

---

32-36: Comment says “listen for updates” but no subscriptions exist.

Either wire an event, or reword the comment.

-  init() {
-    // Listen for game updates to detect SAM launcher changes
-    this.redraw();
-  }
+  init() {
+    // Initial draw; subsequent changes detected in tick()
+    this.redraw();
+  }

---

41-77: Avoid redrawing on pan/zoom; it’s unnecessary work.

Geometry doesn’t change with transform; only placement does. Keep redraws for SAM add/remove only.

-    // Redraw if transform changed or if we need to redraw
-    if (this.transformHandler.hasChanged() || this.needsRedraw) {
+    // Redraw only when content changed
+    if (this.needsRedraw) {
       this.redraw();
       this.needsRedraw = false;
     }

---

93-99: Drop redundant isActive filter.

GameView.units(UnitType.SAMLauncher) already returns active units.

-    const samLaunchers = this.game.units(UnitType.SAMLauncher).filter((unit) => unit.isActive());
+    const samLaunchers = this.game.units(UnitType.SAMLauncher);

---

106-141: Range source: prefer per-unit value, fallback to config.

If SAM range can vary (upgrades, tech, buffs), read from the unit first and default to config to keep visuals correct.

-    const radius = this.game.config().defaultSamRange(); // 70 pixels
+    const radius =
+      // @ts-expect-error optional property if available
+      (sam as any).range?.() ??
+      this.game.config().defaultSamRange();

Would you confirm whether UnitView exposes a range for SAMs?

---

1-21: Constructor: remove unused eventBus to simplify.

eventBus is not used; drop it here and at call site.

-import type { EventBus } from "../../../core/EventBus";
@@
-  constructor(
-    private readonly game: GameView,
-    private readonly eventBus: EventBus,
-    private readonly transformHandler: TransformHandler,
-  ) {
+  constructor(
+    private readonly game: GameView,
+    private readonly transformHandler: TransformHandler,
+  ) {

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 1ae7ebc and dd82cfc.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (4)

• .prettierrc (1 hunks)
• package.json (1 hunks)
• src/client/graphics/GameRenderer.ts (3 hunks)
• src/client/graphics/layers/SAMRadiusLayer.ts (1 hunks)

🧰 Additional context used

🧬 Code graph analysis (2)

src/client/graphics/layers/SAMRadiusLayer.ts (3)

src/client/graphics/layers/Layer.ts (1)

• Layer (1-7)

src/core/game/GameView.ts (3)

• GameView (364-696)
• unit (576-578)
• UnitView (47-178)

src/client/graphics/TransformHandler.ts (1)

• TransformHandler (15-296)

src/client/graphics/GameRenderer.ts (1)

src/client/graphics/layers/SAMRadiusLayer.ts (1)

• SAMRadiusLayer (11-141)

🔇 Additional comments (3)

src/client/graphics/GameRenderer.ts (2)

27-27: LGTM: import is correct and scoped.

---

237-237: Good layer ordering.

Placing the radius after StructureLayer (and before icons/units) gives clear visuals without covering units.

src/client/graphics/layers/SAMRadiusLayer.ts (1)

79-87: Verify drawImage offset assumptions.

The -width/2, -height/2 offset assumes world origin is map center. If origin is top-left in world space, this should be (0, 0) instead. Please confirm visually at extreme zoom/pan.

[BrewedCoffee]

Would it make sense to add this to the same view as hovering over SAM structures?

The merge-base changed after approval.

[coderabbitai]

Actionable comments posted: 3

🧹 Nitpick comments (5)

package.json (3)

32-36: Avoid running two import-sorting plugins (conflicts/churn).

You're adding @trivago/prettier-plugin-sort-imports while prettier-plugin-organize-imports is already present. Running both causes unstable diffs. Pick one (recommend the Trivago plugin) and drop the other.

Apply this diff to remove organize-imports:

-    "prettier-plugin-organize-imports": "^4.1.0",

---

122-122: intl-messageformat added—good for i18n.

Confirm tree-shaking and that translateText() uses a single shared formatter instance to avoid allocations per render.

---

128-128: seedrandom added—prefer native crypto when possible.

If you need reproducible seeds, seedrandom is fine. For non-deterministic RNG, use crypto.getRandomValues().

src/client/graphics/GameRenderer.ts (2)

232-234: Good callout on grouping by shouldTransform.

The comment helps avoid excessive save/restore. Consider batching all transformed layers together explicitly.

---

60-61: Typo in comment.

"dcoument" → "document".

-  // TODO maybe append this to dcoument instead of querying for them?
+  // TODO maybe append this to document instead of querying for them?

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9b3cceb and 68d5399.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (4)

• .prettierrc (1 hunks)
• package.json (4 hunks)
• src/client/graphics/GameRenderer.ts (9 hunks)
• src/client/graphics/layers/SAMRadiusLayer.ts (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

• .prettierrc
• src/client/graphics/layers/SAMRadiusLayer.ts

🧰 Additional context used

🧠 Learnings (1)

📚 Learning: 2025-06-09T02:20:43.637Z

Learnt from: VariableVince
PR: openfrontio/OpenFrontIO#1110
File: src/client/Main.ts:293-295
Timestamp: 2025-06-09T02:20:43.637Z
Learning: In src/client/Main.ts, during game start in the handleJoinLobby callback, UI elements are hidden using direct DOM manipulation with classList.add("hidden") for consistency. This includes modals, buttons, and error divs. The codebase follows this pattern rather than using component APIs for hiding elements during game transitions.

Applied to files:

• src/client/graphics/GameRenderer.ts

🧬 Code graph analysis (1)

src/client/graphics/GameRenderer.ts (7)

src/client/graphics/layers/SAMRadiusLayer.ts (1)

• SAMRadiusLayer (11-141)

src/client/graphics/layers/UILayer.ts (1)

• UILayer (27-370)

src/core/game/GameView.ts (1)

• GameView (364-696)

src/core/EventBus.ts (1)

• EventBus (8-46)

src/client/graphics/TransformHandler.ts (1)

• TransformHandler (15-296)

src/client/graphics/UIState.ts (1)

• UIState (1-3)

src/client/graphics/layers/Layer.ts (1)

• Layer (1-7)

🔇 Additional comments (4)

package.json (2)

67-67: Install missing dependency & re-run the eslint-plugin-jest check

ESLint aborted loading eslint.config.js: "eslint-config-prettier" not found, so the compatibility check did not run.

Actions:

• Install the missing package: npm install --save-dev eslint-config-prettier (or pnpm/yarn equivalent)
• Locate config files: rg -n --hidden --glob '!node_modules' 'eslint.config|.eslintrc'
• Re-run and confirm jest rules load (use a real source file):
  npx -y eslint --print-config <path/to/source-file> | head -n 60
  npx -y eslint --print-config <path/to/source-file> | rg -n 'jest' -C2

If jest rules are not present after fixing the import, check that eslint-plugin-jest v29 supports ESLint 9 flat config and update the config format accordingly.

---

33-35: Ensure Jest uses @swc/jest and remove ts-jest remnants

• Search output shows no "ts-jest" occurrences.
• Action: confirm Jest config uses a transform like { "^.+\.(t|j)sx?$": "@swc/jest" } and remove any "preset": "ts-jest" or "transform" references to ts-jest. Check: jest.config.{js,ts,cjs}, package.json "jest" key, CI/test configs, and tsconfig.jest.json.

src/client/graphics/GameRenderer.ts (2)

289-299: Immutability on renderer fields—nice.

readonly on core dependencies prevents accidental reassignment and clarifies ownership.

---

243-243: UILayer ctor change — signature confirmed (game, eventBus)

All occurrences (src/client/graphics/GameRenderer.ts and tests/client/graphics/UILayer.test.ts) call new UILayer(game, eventBus); none pass transformHandler.

[coderabbitai]

💡 Verification agent

🧩 Analysis chain

Wire up ts-reset early or via tsconfig types.

@total-typescript/ts-reset only works if imported first or added to tsconfig "types".

Run:

---

🏁 Script executed:

#!/bin/bash
# Option A: direct import at app entry (should be first import)
rg -nP '^\s*import\s+["\']@total-typescript/ts-reset["\']' -g '!**/node_modules/**' -C2

# Option B: tsconfig types entry
fd -a tsconfig*.json | xargs -I{} rg -nP '"types"\s*:\s*\[[^\]]*"@total-typescript/ts-reset"[^\]]*\]' {}

Length of output: 68

---

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== A) Search for any mention of @total-typescript/ts-reset =="
rg -n --hidden --no-ignore -S "@total-typescript/ts-reset" -g '!*node_modules*' || true

echo
echo "== B) Search for import lines that import ts-reset =="
rg -n --hidden --no-ignore -S "import.*@total-typescript/ts-reset" -g '!*node_modules*' -C2 || true

echo
echo "== C) Check tsconfig*.json files for compilerOptions.types =="
files=$(fd -a tsconfig*.json || true)
if [ -z "$files" ]; then
  files=$(find . -type f -name 'tsconfig*.json' 2>/dev/null || true)
fi

if [ -z "$files" ]; then
  echo "No tsconfig*.json files found."
  exit 0
fi

for f in $files; do
  echo
  echo "=== $f ==="
  python - <<PY
import json,sys
f="$f"
try:
    j=json.load(open(f))
except Exception as e:
    print(f"{f}: json parse error: {e}")
    sys.exit(0)
co = j.get("compilerOptions",{}) or {}
types = co.get("types")
if isinstance(types, list) and "@total-typescript/ts-reset" in types:
    print(f"{f}: contains @total-typescript/ts-reset in compilerOptions.types")
else:
    print(f"{f}: does NOT contain @total-typescript/ts-reset")
ext = j.get("extends")
if ext:
    print(f"{f}: extends -> {ext}")
PY
done

Length of output: 825

---

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== SEARCH: any mention of @total-typescript/ts-reset =="
rg -n --hidden --no-ignore -S "@total-typescript/ts-reset" --glob '!**/node_modules/**' --glob '!.git/**' || echo "No matches found."

echo
echo "== SEARCH: direct import or require lines =="
rg -n --hidden --no-ignore -S "^\s*(import\s+['\"]@total-typescript/ts-reset['\"]|require\(['\"]@total-typescript/ts-reset['\"]\))" --glob '!**/node_modules/**' --glob '!.git/**' -C2 || echo "No direct import/require found."

echo
echo "== FIND: tsconfig*.json files =="
tsfiles=$(find . -type f -name 'tsconfig*.json' -print 2>/dev/null || true)
if [ -z "$tsfiles" ]; then
  echo "No tsconfig*.json files found."
  exit 0
fi

for f in $tsfiles; do
  echo
  echo "=== $f ==="
  echo "--- head (first 200 lines) ---"
  sed -n '1,200p' "$f" || true
  echo "--- contains literal @total-typescript/ts-reset? ---"
  rg -n "@total-typescript/ts-reset" "$f" || echo "not found"
  echo "--- types array (if any) ---"
  rg -n '"types"\s*:\s*\[[^\]]*\]' "$f" -n -C1 || echo "no types array found"
done

Length of output: 1843

---

Wire up @total-typescript/ts-reset — import first or add to tsconfig types

• Observed: no import of "@total-typescript/ts-reset" in the repo; tsconfig.json (root) has no "types"; tsconfig.jest.json has "types": ["jest","node"] and extends ./tsconfig.json.
• Action (pick one):
  • Import it as the very first runtime import in your app entry (e.g. add at top of src/index.ts or src/main.ts): import '@total-typescript/ts-reset';
  • Or add "@total-typescript/ts-reset" to compilerOptions.types in tsconfig.json — and also add it to tsconfig.jest.json (it currently overrides types), or remove the override so the base applies.
• Files to change: tsconfig.json, tsconfig.jest.json, and your app entry file.

🤖 Prompt for AI Agents

In package.json around line 35, the repo added the dependency
"@total-typescript/ts-reset" but it's not wired in; either import it as the
first runtime import in your app entry (e.g., add import
'@total-typescript/ts-reset'; at the very top of src/index.ts or src/main.ts) or
register it in TypeScript config by adding "@total-typescript/ts-reset" to
compilerOptions.types in tsconfig.json and also ensure tsconfig.jest.json either
includes that type or stops overriding types (or add it there as well); update
tsconfig.json, tsconfig.jest.json, and the app entry file accordingly so the
reset is applied at runtime or compile-time.

[coderabbitai]

⚠️ Potential issue

Guard against null: emojiTable.init may throw if element missing.

The query may return null; calling init unconditionally can crash.

Apply:

-  emojiTable.init(transformHandler, game, eventBus);
+  if (emojiTable instanceof EmojiTable) {
+    emojiTable.init(transformHandler, game, eventBus);
+  } else {
+    console.error("EmojiTable element not found in the DOM");
+  }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	emojiTable.init(transformHandler, game, eventBus);
	if (emojiTable instanceof EmojiTable) {
	emojiTable.init(transformHandler, game, eventBus);
	} else {
	console.error("EmojiTable element not found in the DOM");
	}

🤖 Prompt for AI Agents

In src/client/graphics/GameRenderer.ts around line 65, calling
emojiTable.init(transformHandler, game, eventBus) unconditionally can throw if
the queried DOM element is null; modify the code to first obtain the element
reference, check for null/undefined, and only call emojiTable.init when the
element exists; if the element is missing, log a warning or early-return so
initialization is skipped safely (or provide a fallback element) to prevent
runtime crashes.

[coderabbitai]

⚠️ Potential issue

SAMRadiusLayer is never rendered (not added to layers).

You construct samRadiusLayer but don’t include it in layers, so the radius never shows.

Add it just before UILayer so it draws beneath UI but above FX/units as desired:

   new UnitLayer(game, eventBus, transformHandler),
   new FxLayer(game),
+  samRadiusLayer,
   new UILayer(game, eventBus),

If you prefer the radii under units, place it before UnitLayer instead.

Also applies to: 235-245

🤖 Prompt for AI Agents

In src/client/graphics/GameRenderer.ts around line 203 (and similarly for the
block at 235-245), you instantiate samRadiusLayer but never add it to the
renderer layers array so it never gets drawn; insert the samRadiusLayer into the
layers list just before the UILayer creation so it renders beneath UI but above
FX/units (or, if you want radii beneath units, insert it before the UnitLayer
instead). Ensure the layers array/order is updated accordingly and remove any
unused local variable warnings if needed.