]> Repositorios git - classgraph.git/commit
Revert per-pivot reduction in replaceFamilyApp
authorJavier Sagredo <[email protected]>
Thu, 7 May 2026 01:08:27 +0000 (03:08 +0200)
committerJavier Sagredo <[email protected]>
Thu, 7 May 2026 01:08:27 +0000 (03:08 +0200)
commit5306040244edaad2d04675fb498b91e462e82680
tree4f66ae40f55c93c12a0902856c4710228959b59d
parentd8c99ff73b6cb908584fdab5db689b5a56a5ce1e
Revert per-pivot reduction in replaceFamilyApp

The previous change ('Reduce pivot's sub-args before unifying in
replaceFamilyApp') traded "F (G a) won't resolve" for "every
concrete fam-instance of F now matches once G a unwraps to a
TyVarRef". That blew up real cases — a ChainDepState use site
intended for Praos / TPraos started matching the BFT and PBFT
data instances too, because biUnify wildcards both sides on
TyVarRef.

Roll back to the conservative behaviour: don't pre-reduce the
pivot's sub-args. Use sites with nested families that we can't
unify directly take the placeholder path. INTERNALS.md updated
to describe the trade-off as a deliberate non-feature.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
data/viewer.js
docs/INTERNALS.md