feat: scope hint overhaul — popover UX, EAV form coverage, product grid pills, extension API

[adexandros]

Overview

This branch consolidates work across 7 areas:

1. CI baseline — GitHub Actions matrix (PHP 7.4/8.1/8.2/8.3), PHPStan level 5, updated composer constraints
2. Override detection service — extracted ScopeOverrideDetector replacing per-field scopeConfig->getValue() calls
3. Select label resolution — shows human-readable labels (Yes/No, Enabled/Disabled) instead of raw 1/0 values (closes Support select field in configuration #1)
4. Broader field-type coverage — formatter pool handles datetime, password/encrypted, file/image, WYSIWYG, serialised arrays, multiselect
5. Popover UX — replaces hover tooltip with click-triggered, website-grouped expandable panel; legacy <br>-list mode preserved behind dev/debug/scopehint_legacy_tooltip flag
6. Category edit form — scope hints on category EAV fields (closes Support For Categories #26, supersedes Add scopehint on the category edit form #27 — thanks @thedotwriter for the original approach)
7. Product grid pills — per-attribute columns show store-override values inline as click-to-inspect pills; portaled panel avoids grid cell overflow:hidden clipping

Extension API (Api/OverrideSourceInterface)

Third-party modules can register custom override sources without forking this module:

<type name="AvS\ScopeHint\Service\OverrideSourcePool">
  <arguments>
    <argument name="sources" xsi:type="array">
      <item name="my_custom" xsi:type="object" sortOrder="50">Vendor\Module\ScopeHint\MyOverrideSource</item>
    </argument>
  </arguments>
</type>

See docs/extending.md for a worked example.

What changed

Area	Before	After
System config tooltip	Hover ? bubble with <br>-joined store list	Click-triggered popover, website-grouped, collapsible, searchable
Select field values	Raw 1 / 0	Resolved option labels via source_model
Field type coverage	Basic text only	datetime, password, file, WYSIWYG, serialised arrays
Product EAV form	tooltip.description hover bubble	additionalInfo slot popover, MutationObserver-bound after KO render
Category EAV form	No hints	Same popover, detects overrides per store
Product grid	No indicator	Per-attribute pills showing overridden value per store/website
Override detection	Per-field scopeConfig->getValue() loop	Centralised ScopeOverrideDetector with memoisation
Extension surface	None	OverrideSourceInterface pool for custom sources

Tests

• 143 unit tests, 227 assertions — all passing
• Playwright admin E2E (8 scenarios) covering pill render, click→open, no row navigation, outside-click close, portal-to-body, panel restore

Closes

• Support select field in configuration #1 (select label resolution)
• Add support to the product overview #12 (product grid override marker)
• Support For Categories #26 / supersedes Add scopehint on the category edit form #27 (category form hints)
• Support Page Builder Attributes #36 (Page Builder) — planned as a follow-up PR

[avstudnitz]

Thanks a lot! That's a huge rewrite which makes the module genuinely better. A few things popped up, but all in all, quality is really good.

[avstudnitz]

Ignoring this is dangerous - are you really sure it works with a parameter? Perhaps we replace it with a more generic version which doesn't use a parameter, see next comments.

[avstudnitz]

Config\Model\Config\Structure\AbstractElement::getData()
takes no arguments and returns the whole $_data array. The extra argument is silently
ignored, so $backendModel is an array: it fails both is_object() and is_string(), and
isSerializedBackend() always returns false. The "N rows configured" formatter therefore
never runs in production — serialized fields fall through to DefaultFormatter and show raw
JSON. The unit test passes only because it mocks getData($key) with a per-key callback
(Test/Unit/.../SerializedArrayFormatterTest.php:35), i.e. it asserts the intended contract
rather than the real one.

$backendModel = $field->getData()['backend_model'] ?? null;

[avstudnitz]

See comment in SerializedArrayFormatter.php: This is an array as $field->getData() accepts no parameters.

$frontendModel = $field->getData()['frontend_model'] ?? null;

[avstudnitz]

See comment in ValueFormatter/SerializedArrayFormatter.php: This is an array as $field->getData() accepts no parameters.

$sourceModel = $field->getData()['source_model'] ?? null;

[avstudnitz]

Multiselect values render as the literal string Array.
Eav\...\Source\Table::getOptionText() returns an array of labels when the value contains
a comma, and line 218 does (string) $optionText. In production you get Array; in developer
mode the converted warning is swallowed by the \Throwable catch and you get raw IDs. The old
code only resolved labels for dataType == 'select', so this is a regression for multiselects.
Same at Service/OverrideSource/EavCategoryOverrideSource.php:222.

if (is_array($optionText)) {
    $valueAsString = implode(', ', $optionText);
} elseif ($optionText !== false && $optionText !== null) {
    $valueAsString = (string) $optionText;
}

[avstudnitz]

Grid pills show raw option IDs while the form popover shows labels —
formatValue($value, AbstractAttribute $attribute) never uses $attribute and just returns
(string) $value. A status override renders as 2 instead of Disabled, which contradicts
the PR's "human-readable labels" goal for the grid. Either resolve labels here (with the
is_array() guard from the multiselect finding) or drop the unused parameter so the gap is
explicit.

[avstudnitz]

Can we limit the loaded products to those actually used in the grid? The way it is, we load all attributes including the big text fields which is a lot of unnecessary data.

[avstudnitz]

This missesattributes
that have a store row but no default row, so the grid shows no pill while the form popover does.

joinLeft plus the existing NULL-safe NOT (t.value <=> admin.value) handles it correctly.

[avstudnitz]

Can we use Magento\Config\Model\Config\SourceFactory instead of the object manager?

[avstudnitz]

Add escaping her in order to prevent XSS:

$this->escaper->escapeHtml((string) $storeValue)