parse-server icon indicating copy to clipboard operation
parse-server copied to clipboard

refactor: Remove Javascript SDK as a dependency

Open dplewis opened this issue 8 months ago • 3 comments

Pull Request

Issue

Closes: https://github.com/parse-community/parse-server/issues/8787

Approach

Tasks

  • [ ] Add tests
  • [ ] Add changes to documentation (guides, repository pages, code comments)
  • [ ] Add security check
  • [ ] Add new Parse Error codes to Parse JS SDK

dplewis avatar Apr 16 '25 18:04 dplewis

🚀 Thanks for opening this pull request!

Codecov Report

Attention: Patch coverage is 75.80872% with 172 lines in your changes missing coverage. Please review.

Project coverage is 92.74%. Comparing base (ed69e03) to head (00e16f8). Report is 1 commits behind head on alpha.

Files with missing lines Patch % Lines
src/Utils.js 0.00% 49 Missing :warning:
src/Adapters/Storage/Mongo/MongoTransform.js 71.15% 15 Missing :warning:
src/Controllers/DatabaseController.js 57.14% 12 Missing :warning:
src/RestWrite.js 83.01% 9 Missing :warning:
src/Adapters/Auth/twitter.js 40.00% 6 Missing :warning:
src/RestQuery.js 45.45% 6 Missing :warning:
src/GraphQL/helpers/objectsQueries.js 28.57% 5 Missing :warning:
src/Adapters/Auth/facebook.js 60.00% 4 Missing :warning:
src/Adapters/Auth/gcenter.js 42.85% 4 Missing :warning:
src/Adapters/Auth/vkontakte.js 33.33% 4 Missing :warning:
... and 35 more
Additional details and impacted files
@@            Coverage Diff             @@
##            alpha    #9721      +/-   ##
==========================================
- Coverage   93.01%   92.74%   -0.28%     
==========================================
  Files         187      188       +1     
  Lines       15081    15189     +108     
  Branches      174      174              
==========================================
+ Hits        14028    14087      +59     
- Misses       1041     1090      +49     
  Partials       12       12              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Apr 16 '25 18:04 codecov[bot]

@mtrezza The goal is to remove import * as Parse from '../../ClientSDK'; from every file.

This means eliminating all these exports

https://github.com/parse-community/parse-server/pull/9721/files#diff-1c0699bcc374b849308efa6b59c447c2666170d00c470a86a834a1aa8a351726

dplewis avatar Apr 16 '25 18:04 dplewis