Remove unused is_domain_loopback helper

[Copilot]

`Helpers::is_domain_loopback()` had no production call sites — it existed only alongside its own test file. Rather than cache the DNS lookup, the function and its tests are removed entirely.

Changes

• Deleted is_domain_loopback() from admin/class-helpers.php
• Deleted tests/phpunit/Admin/HelpersLoopbackTest.php

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Look into if we can cache the is_domain_loopback check to avoid DNS lookup on each page scan</issue_title>
<issue_description>We check if we are using a loopback connection when scanning pages. In some situations, like with the pro plugin, we can scan the same domain for many pages fairly quickly. If we can avoid having to look up the domain each time, that would be good. We can perhaps, but it is a short-lived transient.</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Look into if we can cache the is_domain_loopback check to avoid DNS lookup on each page scan #1436
• Fixes: https://linear.app/equalize-digital/issue/PRO-594

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[pattonwebz]

This is a good code cleanup, we should get this into the next release (1.40.0)

[Copilot]

Pull request overview

Removes an unused loopback-domain helper from the admin Helpers utility and deletes the now-orphaned PHPUnit test that only exercised that helper.

Changes:

• Removed EDAC\Admin\Helpers::is_domain_loopback() from admin/class-helpers.php.
• Deleted tests/phpunit/Admin/HelpersLoopbackTest.php.
• Updated package-lock.json (lockfile metadata churn).

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File	Description
admin/class-helpers.php	Deletes the unused loopback helper method.
tests/phpunit/Admin/HelpersLoopbackTest.php	Removes the PHPUnit test class that only covered the deleted helper.
package-lock.json	Contains lockfile-only changes unrelated to the helper/test removal.

[SteveJonesDev]

Hey @pattonwebz — I checked the pro plugin to verify Copilot's finding before this gets reviewed.

Result: safe to merge.

is_domain_loopback() has no callers in the pro plugin either (grepped the full accessibility-checker-pro repo, zero results). Combined with no production call sites in the free plugin, this function is genuinely dead code. Removing it is the right call.

A couple of things to double-check before requesting human review:

• The package-lock.json diff is unrelated lockfile churn — worth confirming that's expected or squashing it out to keep the PR clean.
• Confirm no other plugins/extensions outside of pro call this function.

Linear: https://linear.app/equalize-digital/issue/PRO-594

[SteveJonesDev]

@pattonwebz, I fixed the lock file issue and merged in develop. Please review and if it looks good, merge it in.