roslyn icon indicating copy to clipboard operation
roslyn copied to clipboard

[Source Generators] ISyntaxReceiver instances should be able to access AdditionalFiles

Open raffaeler opened this issue 1 year ago • 1 comments

Version Used: 4.10.0

Steps to Reproduce:

  1. Create a generator
  2. Create a SyntaxReceiver and register it in the generator
  3. Add a file in the AdditionalFiles
  4. Try to access the AdditionalFiles from the SyntaxReceiver

Expected Behavior: I would expect to be able to access the AdditionalFiles

Actual Behavior: I am not able to access the AdditionalFiles

Rationale: The standard workflow for code generators is to use a SyntaxReceiver to identify the portions of the syntax trees useful to generate new code. In many cases, we can use a generated attribute to annotate the portions of code that a SyntaxReceiver should capture. In other cases, it is not possible to annotate the code because it has been generated using T4Templates or because it comes from external tools that can generate file.

In these cases, it would make sense to access the AdditionalFiles from the SyntaxReceiver in order to minimize the amount of data provided to the generator.

Since the AdditionalFiles are not available in a SyntaxReceiver, the generator must do all the processing inside the Execute method which forces processing all the SyntaxTrees instead of just the ones that are captured by a SyntaxReceiver. It's not even possible to make the pre-processing in the Initialize method as the AdditionalFiles are not accessible there.

raffaeler avatar Jun 26 '24 18:06 raffaeler

Create a generator

What kind of generator: incremental, ISourceGenerator?

Create a SyntaxReceiver and register it in the generator

Can you show us a sample of whhere you are trying to access this?

jaredpar avatar Jun 27 '24 23:06 jaredpar

Closing due to lack of response.

jaredpar avatar Jul 11 '24 15:07 jaredpar

Sorry for the late response @jaredpar. One week in a conference, another sick :-(

  • I am using Source Generators

Here is the example of the SyntaxReceiver. I have other cases where the _rootInterfaceName cannot be hardcoded but should be configured at project level. AdditionalFiles look like a greate place to put this string, but apparently I can't access the AdditionalFiles from a ISyntaxReceiver

internal class SiblingConfigurationSyntaxReceiver : ISyntaxReceiver
{
    private const string _rootInterfaceName = "IEntityTypeConfiguration";

    public List<ClassDeclarationSyntax> ConfigurationClasses = new();

    public SiblingConfigurationSyntaxReceiver()
    {
    }

    public void OnVisitSyntaxNode(SyntaxNode syntaxNode)
    {
        BaseTypeDeclarationSyntax? typeDeclaration = GeneratorsCommon.GetTypeDeclaration(syntaxNode);

        // dbcontext configurations are always classes
        if (typeDeclaration is not ClassDeclarationSyntax classDeclaration) return;

        //((classDeclaration.BaseList?.Types[0] as BaseTypeSyntax).Type as SimpleNameSyntax).Identifier.Text
        bool isConfiguration = classDeclaration.BaseList?.Types
            .Any(t => (t?.Type as SimpleNameSyntax)?.Identifier.Text == _rootInterfaceName) ?? false;

        if(!isConfiguration) return;

        ConfigurationClasses.Add(classDeclaration);
    }
}

raffaeler avatar Jul 15 '24 09:07 raffaeler