Skip to content

eng(build): support analyzer >=10.0.0 <15.0.0 - #679

Closed
PhuCG wants to merge 1 commit into
GetDutchie:mainfrom
PhuCG:eng/analyzer-14
Closed

PhuCG wants to merge 1 commit into
GetDutchie:mainfrom
PhuCG:eng/analyzer-14

Conversation

@PhuCG

@PhuCG PhuCG commented Sep 29, 2026 •

Copy link
Copy Markdown

Closes #676

Lets the build packages use analyzer up to 14, so Brick works with current freezed, json_serializable and mockito.

  • brick_build: isSynthetic was removed in analyzer 11:
    • getter → isOriginVariable
    • field → isOriginGetterSetter
  • All build packages: analyzer: ">=10.0.0 <15.0.0"
    • ⚠️ The floor goes from 8 to 10 because the isOrigin* getters only exist from 10.
  • brick_sqlite_generators: 2 fixtures updated for dart_style 3.1.13 line wrapping. Output is otherwise unchanged.

Tested on analyzer 14.4:

🤖 Generated with Claude Code

`FieldElement.isSynthetic` / `PropertyAccessorElement.isSynthetic` were
removed in analyzer 11. Replace them with the `isOrigin*` getters added in
analyzer 10:

- getter induced by a field: `isSynthetic` -> `isOriginVariable`
- field induced by a getter/setter: `isSynthetic` -> `isOriginGetterSetter`

Since those getters do not exist before 10.0.0, the minimum analyzer
version is raised to 10.0.0 in every build package.

The newer analyzer resolves dart_style 3.1.13, which wraps long method
chains differently; two sqlite generator fixtures are updated to match.

Closes GetDutchie#676

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@PhuCG
PhuCG requested a review from tshedor as a code owner September 29, 2026 02:32
@dutchie-stale

dutchie-stale Bot commented Oct 10, 2026

Copy link
Copy Markdown

This PR has been automatically marked as stale due to inactivity.
It will be closed if no further activity occurs.

To keep this PR open, you can:

  • Push new changes or leave a comment
  • Add the not-stale or pinned label to exempt it from this process

@dutchie-stale dutchie-stale Bot added the stale label Oct 10, 2026
@dutchie-stale

dutchie-stale Bot commented Oct 13, 2026

Copy link
Copy Markdown

This PR has been automatically closed due to inactivity.
Feel free to reopen it if you plan to continue working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Outdated analyzer version for brick_offline_first_with_supabase_build

1 participant