Skip to content

Conversation

@StefH
Copy link
Owner

@StefH StefH commented Mar 11, 2025

No description provided.

@StefH StefH self-assigned this Mar 11, 2025
@StefH StefH changed the title Reafctor code to use Incremental Generator Reafctor code to implement IIncrementalGenerator Mar 11, 2025
@StefH StefH changed the title Reafctor code to implement IIncrementalGenerator Refactor code to implement IncrementalGenerator Mar 11, 2025
@StefH StefH added the enhancement New feature or request label Jun 24, 2025
@StefH StefH requested a review from Copilot August 24, 2025 06:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the FluentBuilderSourceGenerator to implement the IIncrementalGenerator interface instead of the older ISourceGenerator pattern. This modernizes the source generator architecture to use the newer, more efficient incremental generation model introduced in Roslyn.

  • Updates from ISourceGenerator to IIncrementalGenerator pattern for better performance and caching
  • Refactors syntax receiver logic to use incremental syntax providers
  • Adds diagnostic reporting with specific error/info codes for better user feedback

Reviewed Changes

Copilot reviewed 83 out of 83 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
FluentBuilderSourceGenerator.cs Core refactor from ISourceGenerator to IIncrementalGenerator with new pipeline setup
FluentBuilderSourceGeneratorTests.cs Updates test methods to work with new generator interface and adds diagnostic validation
SyntaxReceiver files Converts from receiver pattern to static methods for syntax node processing
Wrapper files Removes context wrapper abstraction in favor of direct compilation helper
Various generator files Updates to support new architecture and remove context dependencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@StefH StefH merged commit b604ce9 into main Aug 24, 2025
2 checks passed
@StefH StefH deleted the incremental branch August 24, 2025 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants