[HIGH] Variable shadowing causes fallback notification to use wrong target — `let Som

[ironclaw-ci]

[HIGH:100] Issue Found by Staging CI Review

Severity: HIGH
Confidence: 100/100
PR comment: #1279 (comment)

Description

Variable shadowing causes fallback notification to use wrong target — let Some(user) = fallback_user at line 181 shadows the primary user from resolve_routine_notification_target(), causing fallback broadcast to use owner_scope-only target instead of extension-manager-aware target.

---

Auto-created by staging-ci Claude Code review

[henrypark133]

Triage verdict: STALE — already fixed

Fixed in commit bcc38ce ("Fix notification fallback scopes"). The code now correctly maintains separate variables:

• fallback_user: computed via resolve_owner_scope_notification_user() — owner-scope fallback only
• user: computed via resolve_routine_notification_target() — includes extension-manager-aware resolution

No variable shadowing occurs — the fallback broadcast at line ~615 uses fallback_user, not shadowing user.