fix(MESHAGEN-009): CU-86akhf8u3 require('promise') hoisted as local var reused across branches without inline pattern violation but note stray global leak

[flamingo]

Closes findings from rule MESHAGEN-009 — require('promise') hoisted as local var reused across branches without inline pattern violation but note stray global leak.

Draft — this is a starting point, not a finished change. The fix required judgment, so read it before trusting it.

#	Fix confidence	Finding	Location
1	🟢 95 high	require('promise') hoisted as local var reused across branches without inline pattern violation but note stray global leak	modules/power-monitor.js:181

What changed — and what was deliberately left — is explained per finding as inline review comments on the lines each finding touched.

---

Run: https://product-hub.flamingo.so/admin/code-review
Run id: bc788a01-3d0c-4679-816f-282037bb5004

Merging this PR is recorded as acceptance of the rule that produced it;
closing it unmerged is recorded as rejection. Both feed rule health, so
closing a wrong suggestion is useful rather than merely tidy.

ClickUp task: CU-86akhf8u3 MeshAgent review findings sweep (13 PRs)

[flamingo]

🦩 What this fix changed, finding by finding

1 finding(s) fixed in this draft — 1 explained inline on the diff.

[flamingo]

🦩 🟠 require('promise') hoisted as local var reused across branches without inline pattern violation but note stray global leak

In sleepDisplay and wakeDisplay (both defined as this.sleepDisplay = function sleepDispay(force) {...} and this.wakeDisplay = function wakeDisplay() {...} in powerMonitor), the assignment p = new promise(...) was changed to var p = new promise(...) in both functions, declaring p as a local variable instead of leaking it to the global object in non-strict mode.

🤖 Prompt for AI agents

In modules/power-monitor.js around line 181, review and complete this code-review fix: require('promise') hoisted as local var reused across branches without inline pattern violation but note stray global leak.
What the draft fix changed: In `sleepDisplay` and `wakeDisplay` (both defined as `this.sleepDisplay = function sleepDispay(force) {...}` and `this.wakeDisplay = function wakeDisplay() {...}` in `powerMonitor`), the assignment `p = new promise(...)` was changed to `var p = new promise(...)` in both functions, declaring `p` as a local variable instead of leaking it to the global object in non-strict mode.
Verify the change is correct and complete; do not refactor unrelated code.

fix confidence: 🟢 95 high — react 👍/👎 to teach the reviewer