feat(@aws-amplify/datastore): selective sync enhancements

[iartemiev]

• support returning Predicates.ALL from syncExpression - allows clearing the syncExpression at runtime
• persist lastSyncPredicate to ModelMetadata in order to better detect when the predicates have changed, e.g., if developer stops application, changes the syncExpression and then restart their application.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[lgtm-com]

This pull request introduces 1 alert when merging da4b9e4 into cf511b8 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

[codecov]

Codecov Report

Merging #7083 into main will decrease coverage by 0.03%.
The diff coverage is 4.54%.

@@            Coverage Diff             @@
##             main    #7083      +/-   ##
==========================================
- Coverage   72.98%   72.95%   -0.04%     
==========================================
  Files         213      213              
  Lines       13293    13298       +5     
  Branches     2603     2510      -93     
==========================================
- Hits         9702     9701       -1     
- Misses       3395     3432      +37     
+ Partials      196      165      -31     

Impacted Files	Coverage Δ
packages/datastore/src/sync/index.ts	14.53% <0.00%> (-0.54%)	⬇️
packages/datastore/src/datastore/datastore.ts	78.42% <20.00%> (+0.64%)	⬆️
packages/auth/src/OAuth/OAuth.ts	56.11% <0.00%> (ø)
packages/core/src/Credentials.ts	31.51% <0.00%> (ø)
packages/analytics/src/Analytics.ts	63.58% <0.00%> (ø)
packages/datastore/src/sync/outbox.ts	24.00% <0.00%> (ø)
packages/datastore/src/storage/storage.ts	71.66% <0.00%> (ø)
packages/core/src/OAuthHelper/GoogleOAuth.ts	33.33% <0.00%> (ø)
packages/core/src/Util/Reachability.native.ts	37.50% <0.00%> (ø)
... and 11 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cf511b8...4166d8b. Read the comment docs.

[lgtm-com]

This pull request introduces 1 alert when merging a17a614 into cf511b8 - view on LGTM.com

new alerts:

• 1 for Unused variable, import, function or class

[elorzafe]

Thanks Ivan, this looks good! 🌮
Good catch by persisting the predicate!

[github-actions]

This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels or Discussions for those types of questions.