group-invalid not applying properly with tailwindcss v4?

[devlzcode]

This is the resolved CSS:

.group-invalid\:border-negative-900 {
    border-color: var(--color-negative-900) /* #ff9581 */;
}

For comparison this is without group-:

.invalid\:border-negative-900 {
    &:where([data-rac])[data-invalid] {
        border-color: var(--color-negative-900) /* #ff9581 */;
    }
    &:where(:not([data-rac])):invalid {
        border-color: var(--color-negative-900) /* #ff9581 */;
    }
}

Group modifier does work with other states, just not this one. Here is group-disabled

.group-disabled\:cursor-not-allowed {
    :merge(.group):where([data-rac])[data-disabled] & {
        cursor: not-allowed;
    }
    :merge(.group):where(:not([data-rac])):disabled & {
        cursor: not-allowed;
    }
}

[devlzcode]

Looking into it I'm now curious why the plugin uses matchVariant over addVariant. What modifier edge case are you accounting for? My issue is fixed with the following change:

addVariant(
    `group-${variantName}`,
    mapSelector(selectors, (selector) =>
      wrapSelector(selector, (s) => `:merge(.group)${s.slice(1)} &`),
    ),
  );
  addVariant(
    `peer-${variantName}`,
    mapSelector(selectors, (selector) =>
      wrapSelector(selector, (s) => `:merge(.peer)${s.slice(1)} ~ &`),
    ),
  );

[devlzcode]

I don't even think adding the group/peer variants are necessary anymore in modern tailwind?

[devlzcode]

Ah it was to support named groups. While the above comment is true, the css is prepended by :is and I'm not sure if that'll affect anything or not.
This should suffice (while fixing the issue):

  matchVariant(
    `group-${variantName}`,
    (_, { modifier }) =>
      modifier
        ? mapSelector(selectors, (selector) =>
            wrapSelector(
              selector,
              (s) => `:merge(.group\\/${modifier})${s.slice(1)} &`,
            ),
          )
        : mapSelector(selectors, (selector) =>
            wrapSelector(selector, (s) => `:merge(.group)${s.slice(1)} &`),
          ),
    { values: { DEFAULT: "" } },
  );
  matchVariant(
    `peer-${variantName}`,
    (_, { modifier }) =>
      modifier
        ? mapSelector(selectors, (selector) =>
            wrapSelector(
              selector,
              (s) => `:merge(.peer\\/${modifier})${s.slice(1)} ~ &`,
            ),
          )
        : mapSelector(selectors, (selector) =>
            wrapSelector(selector, (s) => `:merge(.peer)${s.slice(1)} ~ &`),
          ),
    { values: { DEFAULT: "" } },
  );