Skip to content

feat: SDK update for version 26.0.0 - #329

Merged
ChiragAgg5k merged 13 commits into
mainfrom
dev
Aug 24, 2026
Merged

feat: SDK update for version 26.0.0#329
ChiragAgg5k merged 13 commits into
mainfrom
dev

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Aug 24, 2026

Copy link
Copy Markdown
Member

This PR contains updates to the SDK for version 26.0.0.

What's Changed

  • Breaking: Execution.functionId replaced by resourceId and resourceType, now that executions cover both functions and sites
  • Added: avatars.getPhoto returning the user's best available profile photo
  • Added: huggingface OAuth provider
  • Added: custom authentication factor
  • Added: ExecutionResourceType enum and resourceType attribute on the Execution model
  • Fixed: OAuth2 failures now surface the error returned in the callback URL
  • Fixed: empty JSON error responses no longer throw FormatException
  • Updated: relaxed device_info_plus to <14.0.0 and package_info_plus to <11.0.0
  • Updated: minimum Dart SDK is now 3.7.0 and minimum Flutter 3.29.0

@ChiragAgg5k ChiragAgg5k changed the title feat: Flutter SDK update for version 25.5.0 feat: SDK update for version 25.5.0 Aug 24, 2026
@greptile-apps

greptile-apps Bot commented Aug 24, 2026

Copy link
Copy Markdown

Greptile Summary

The SDK is updated for version 26.0.0, including the intentional breaking evolution of executions from function-specific identifiers to resource identifiers and types.

  • Adds Execution.resourceId, Execution.resourceType, and the public ExecutionResourceType enum.
  • Adds avatar photo support and new authentication/OAuth enum values.
  • Updates transport error handling, dependency constraints, SDK requirements, generated services, models, tests, and examples.

Confidence Score: 5/5

The PR appears safe to merge because the previously reported source-compatibility break is now intentionally covered by a major version and explicit migration notice.

No blocking failure remains.

Important Files Changed

Filename Overview
lib/src/models/execution.dart Consistently replaces functionId with required resourceId and resourceType fields across construction and map serialization.
lib/src/enums/execution_resource_type.dart Defines the function and site resource types required by the expanded execution model.
lib/enums.dart Exposes ExecutionResourceType through the SDK's public enum library.
pubspec.yaml Versions the package as 26.0.0 and updates the supported SDK and dependency constraints.
CHANGELOG.md Clearly documents the execution field replacement as a breaking 26.0.0 change.
lib/src/client_mixin.dart Updates shared response and error processing, including handling for empty JSON error bodies.

Reviews (2): Last reviewed commit: "chore: remove auto-formatting workflow, ..." | Re-trigger Greptile

Comment thread lib/src/models/execution.dart
@ChiragAgg5k ChiragAgg5k changed the title feat: SDK update for version 25.5.0 feat: Flutter SDK update for version 26.0.0 Aug 24, 2026
@ChiragAgg5k ChiragAgg5k changed the title feat: Flutter SDK update for version 26.0.0 feat: SDK update for version 26.0.0 Aug 24, 2026
@ChiragAgg5k

Copy link
Copy Markdown
Member Author

Looked through this as a maintainer pass on HEAD 11e368fd.

Greptile’s P1 was right on the 25.5.0 snapshot — Execution.functionIdresourceId/resourceType is source-breaking — but that’s now a real 26.0.0 major. pubspec, CHANGELOG, README, and x-sdk-version all agree. Dart sibling is 28.0.0 with the same model change.

CI test is green. I wouldn’t block.

Notes, none of them merge-blocking:

  • OAuth callback error parsing is in ClientIO.webAuth only. Flutter web (ClientBrowser.webAuth) still doesn’t surface error.
  • No unit tests for the empty-JSON FormatException fix or the OAuth error path.
  • README still doesn’t mention the Dart 3.7 / Flutter 3.29 floor (CHANGELOG does). Worth a generator tweak later.
  • device_info_plus <14 can include 13.x (Flutter 3.41.6 / Dart 3.11). Solver won’t pick that on 3.29 apps; the 3.7 floor is actually a fix for the old >=2.17.0 vs device_info_plus >=11.5.0 mismatch.

Fine to merge once required reviews land.

@ChiragAgg5k
ChiragAgg5k merged commit 03306bb into main Aug 24, 2026
2 checks passed
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.

2 participants