ScoutEnhancer::enhanceBuilder() improvements to be more similar to …

[LastDragon-ru]

…ArgBuilderDirective: process all ScoutBuilderDirective directives and directives filter support.

• Added or updated tests
• Documented user facing changes
• Updated CHANGELOG.md

Changes

Not sure why, but ScoutEnhancer::enhanceBuilder() use only first directive

lighthouse/src/Scout/ScoutEnhancer.php

Lines 73 to 76 in 0f46c72

	foreach ($this->argumentsWithScoutBuilderDirectives as $argument) {
	$scoutBuilderDirective = $argument
	->directives
	->first(Utils::instanceofMatcher(ScoutBuilderDirective::class));

This is contrary to how ArgBuilderDirective/FieldBuilderDirective directives works: ResolveInfo::enhanceBuilder() process all of them. So the PR makes ScoutEnhancer::enhanceBuilder() to process all ScoutBuilderDirective directives.

The PR also fix directives filter - the filter will be applied to ScoutBuilderDirective too.

Breaking changes

If someone passes $directiveFilter into ResolveInfo::enhanceBuilder() it may be need to update its declaration to accept \Nuwave\Lighthouse\Support\Contracts\ArgBuilderDirective|\Nuwave\Lighthouse\Scout\ScoutBuilderDirective.

[spawnia]

Good change.

[LastDragon-ru]

Thanks, I will update PR tomorrow :)

[LastDragon-ru]

Seems done

[spawnia]

Thank you, released with https://github.com/nuwave/lighthouse/releases/tag/v6.16.0.

[LastDragon-ru]

Cool, thank you :)