McRitchie Studio

Scout report: merge-ready - Light review found no blockers; shiny badge polish matches acceptance and keeps sprite behavior intact. Reporter: jasper Outcome: merge-ready Findings: - No blocker: avatar badge now renders as a large raw sparkle without bg/border/rounded/shadow classes. - No blocker: stacked crew render forces under-stack shiny badges off while leaving the visible lead badge eligible. - Non-blocker merge safety: PR #443 also edits _stage_agent_avatars; read-only merge-tree shows a conflict, so #443 should preserve this PR's shiny: false-under-stack arg when rebased or assembled. Questions: - none Checks: - PR #441 is open against release, non-draft, mergeable clean, and GitHub CI scan_ruby/scan_js/lint/test are successful. - Task metadata acceptance/checks match the PR body; reported full-suite/rubocop tree hash matches HEAD tree 12d5598a. - Diff read against origin/release: 4 files, no whitespace errors; no on-chain/Jasper checklist surface touched.

Scout report: merge-ready - Merge-ready: badge polish matches acceptance; DOR, cert, CI, focused Rails, and E2E checks are green. Reporter: shannon Outcome: merge-ready Findings: - No blockers. The diff renders a raw backgroundless sparkle at the avatar northeast edge and suppresses lower stacked badges without changing shiny sprite URLs. - Merge-safety note: draft PR #443 also edits app/views/components/_stage_agent_avatars.html.erb and will need a rebase if this lands first; #441 is clean against release. Questions: - none Checks: - bin/dor-check shiny-badge-polish passed from the feature worktree; full-suite and rubocop cert evidence is fresh at 12d5598a67d7 and CI is 4/4 green. - Focused Rails check passed with the documented Homebrew Ruby PATH: bin/rails test test/views/agent_avatar_component_test.rb test/integration/board_card_shiny_mascot_test.rb; 5 runs, 28 assertions. - E2E_PORT=3098 npx playwright test e2e/shiny_mascot.spec.js passed: 1 test. The first 3018 rerun reused a stale server missing the shiny seed, so I reran on a clean port. - Reviewed task metadata, PR body, diff, helper flow, CI, screenshots, and open PR overlap.

pr-review moved shiny-badge-polish to reviewed after shannon + jasper reported merge-ready. Left for Steffon QA. Reports: - shannon: merge-ready - Merge-ready: badge polish matches acceptance; DOR, cert, CI, focused Rails, and E2E checks are green. - jasper: merge-ready - Light review found no blockers; shiny badge polish matches acceptance and keeps sprite behavior intact.