[ChuckBuilds]

Why

NFL and NCAA FB share ten team abbreviations (CAR CIN DAL DEN HOU LAC MIA NE TB TBD), and one scroll strip carries both leagues. The classic logo cache was already scoped by logo directory (_logo_scope, see test_logo_cache_league_scope.py). The adaptive layout path (layout_mode: adaptive) keeps two caches of its own, and both were still keyed by abbreviation alone:

• _raw_logo_cache (unresized logos), keyed "MIA";
• the LayoutContext fit, cached under cache_key="logo:MIA".

Whichever Miami was drawn first won: the Hurricanes' card showed the Dolphins' logo, or the other way round. This was found during a core performance investigation (LEDMatrix #716–#719).

What

• Both keys now include self._logo_scope(logo_path), the same scoping the classic cache uses.
• Classic layout is untouched, and the adaptive golden images are unchanged.
• Bumps football-scoreboard to 3.18.2, with a new versions[] entry and a CHANGELOG entry. plugins.json was updated via update_registry.py.

Tests

test_adaptive_logo_league_scope.py (new, standalone like the other adaptive tests; needs LEDMATRIX_CORE) renders an NFL MIA card and then an NCAA MIA card on one adaptive renderer, in both orders, and checks:

• each card shows only its own logo colour;
• the same directory and abbreviation share one cache entry.

On origin/main all 3 checks fail: the second card shows 1,936 px of the other league's logo and none of its own. With the fix, 3 pass.

Existing tests also pass: test_logo_cache_league_scope.py (8), test_adaptive_score_clears_logos.py (68 checks), and test_adaptive_layout_mode / text_colors / upcoming_stacks / scorebug_settings (27, including goldens).

Guards: check_version_bump football-scoreboard PASS, check_manifests_ascii, check_min_core_version, check_module_collisions and check_sports_helpers_parity all PASS.

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 628e2d6c-68cc-4965-9335-ceacbba7df23

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

---

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.}

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 19 complexity

Metric	Results
Complexity	19

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}