Skip to content

feat(rpc_generator): upgrade to analyzer v9#24

Merged
Nicobond07 merged 1 commit intomainfrom
update-analyzer
Feb 3, 2026
Merged

feat(rpc_generator): upgrade to analyzer v9#24
Nicobond07 merged 1 commit intomainfrom
update-analyzer

Conversation

@Nicobond07
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

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

Updates rpc_generator to be compatible with analyzer v9 (and associated build/source_gen upgrades), adjusting generator code and tests to match the updated analyzer/source_gen APIs.

Changes:

  • Upgraded analyzer, build, source_gen, and related dev tooling; bumped rpc_generator version to 1.2.0.
  • Migrated generator code from *Element2 APIs to legacy *Element APIs (element.dart, .fields, .methods, etc.).
  • Reworked generator tests to use build_test + PartBuilder, and updated the expected generated output.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
rpc_generator/lib/src/generator.dart Migrates generator element model types/APIs for analyzer v9 compatibility.
rpc_generator/test/generator_test.dart Replaces generator_test harness with build_test-based builder tests.
rpc_generator/test/src/expected/with_call_adapter.g.dart Updates fixture to match newly generated output (call adapter annotations).
rpc_generator/pubspec.yaml Bumps package version and updates dependency constraints.
pubspec.lock Locks updated dependency graph after the analyzer/build/source_gen upgrade.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown

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

Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Nicobond07 Nicobond07 merged commit 5427c3f into main Feb 3, 2026
10 checks passed
@Nicobond07 Nicobond07 deleted the update-analyzer branch February 3, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants