Ultimate-Multisite / Ultimate-Multisite/ultimate-multisite
Eliminate ESLint warnings and legacy overrides
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 214
- Forks
- 86
- Avg merge
- 5h 29m
- Merged PRs (30d)
- 60
Description
What
Eliminate the nine remaining ESLint warnings and retire the narrow legacy-file overrides added in #1805 by modernizing the affected source files safely.
Why
The current gate passes, but warnings and per-file suppressions reduce signal and leave legacy equality, expression, JSDoc, and operator-precedence debt.
Tier
tier:standard — edits are bounded to named JavaScript files but some loose comparisons and expression statements require behavior-preserving decisions.
Files
.eslintrc.jsassets/js/activity-stream.jsassets/js/command-palette.jsassets/js/coupon-code.jsassets/js/dashboard-statistics.jsassets/js/dns-management.jsassets/js/settings-loader.jsassets/js/webhook-page.js- Legacy override targets listed in
.eslintrc.js
Reference pattern
Edit unminified source only, remove each override rule only after its target files pass, then regenerate minified assets with pnpm run build:dev.
Implementation steps
- Resolve the nine warnings without changing runtime behavior.
- Replace loose comparisons and ambiguous expressions in small reviewed batches.
- Correct JSDoc and localized-global declarations at their source where practical.
- Remove obsolete override entries and rebuild generated assets.
Acceptance criteria
-
pnpm run lint:jsreports zero errors and zero warnings. - Legacy overrides are reduced or removed.
- No minified file is hand-edited.
-
pnpm run build:devandpnpm run qualitypass. - Relevant browser flows still work.
Verification
pnpm run lint:jspnpm run build:devpnpm run quality- Browser smoke checks for affected admin and checkout flows
Dependency
Blocked by #1805; begin after PR #1806 merges.
aidevops.sh v3.32.317 plugin for OpenCode v1.18.29 with gpt-5.6-sol spent 4h 26m and 539,830 tokens on this with the user in an interactive session.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Begin after PR #1806 merges. Read .eslintrc.js and the listed assets/js files, then run pnpm run lint:js to identify the nine warnings and their legacy override targets. Update only unminified sources, remove overrides after their targets pass, run pnpm run build:dev and pnpm run quality, and verify the affected admin and checkout browser flows.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- eslint, javascript
- Domain
- frontend, tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 64/100