Skip to content

Conversation

@coratgerl
Copy link
Contributor

@coratgerl coratgerl commented Nov 23, 2025

Pull Request

Remove unused dependencies or move to depDependencies

Summary by CodeRabbit

  • Chores
    • Updated project dependencies for improved development workflow and optimized bundle size.
    • Reorganized build tooling dependencies.
    • Removed unused dependencies.

✏️ Tip: You can customize this high-level summary in your review settings.

@parse-github-assistant
Copy link

I will reformat the title to use the proper commit message syntax.

@parse-github-assistant parse-github-assistant bot changed the title fix: remove unused dependencies fix: Remove unused dependencies Nov 23, 2025
@parse-github-assistant
Copy link

parse-github-assistant bot commented Nov 23, 2025

🚀 Thanks for opening this pull request!

@coderabbitai
Copy link

coderabbitai bot commented Nov 23, 2025

📝 Walkthrough

Walkthrough

Reorganized package dependencies by moving @babel/eslint-parser from dependencies to devDependencies, and removing graphql-tag, router, flow-bin, and husky packages entirely.

Changes

Cohort / File(s) Summary
Dependency reorganization
package.json
Moved @babel/eslint-parser from dependencies to devDependencies; removed graphql-tag and router from dependencies; removed flow-bin and husky from devDependencies

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

  • Verify that @babel/eslint-parser is appropriate as a devDependency (linting tool)
  • Confirm that graphql-tag and router are not used in the codebase or have been replaced
  • Ensure flow-bin and husky removals don't break existing development workflows

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete. It lacks the Issue section with a link/reference, detailed Approach description, and Tasks checklist required by the template. Complete the description by filling in the 'Closes' field with an issue reference, providing a detailed Approach section explaining the rationale for dependency changes, and checking applicable Tasks.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title 'perf: Remove unused dependencies' accurately describes the main change of removing and reorganizing dependencies, though the conventional commit type 'perf' may be slightly imprecise for a dependency restructuring task.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@parseplatformorg
Copy link
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov
Copy link

codecov bot commented Nov 23, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.08%. Comparing base (12beb8f) to head (6dbe25e).
⚠️ Report is 9 commits behind head on alpha.

Additional details and impacted files
@@            Coverage Diff             @@
##            alpha    #9943      +/-   ##
==========================================
+ Coverage   93.07%   93.08%   +0.01%     
==========================================
  Files         188      188              
  Lines       15298    15298              
  Branches      177      177              
==========================================
+ Hits        14238    14240       +2     
+ Misses       1048     1046       -2     
  Partials       12       12              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

Copy link
Member

@mtrezza mtrezza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find!

@mtrezza mtrezza changed the title fix: Remove unused dependencies refactor: Remove unused dependencies Nov 23, 2025
@mtrezza mtrezza changed the title refactor: Remove unused dependencies perf: Remove unused dependencies Nov 23, 2025
@mtrezza mtrezza merged commit d4c6de0 into parse-community:alpha Nov 23, 2025
27 of 28 checks passed
parseplatformorg pushed a commit that referenced this pull request Nov 23, 2025
# [8.5.0-alpha.15](8.5.0-alpha.14...8.5.0-alpha.15) (2025-11-23)

### Performance Improvements

* Remove unused dependencies ([#9943](#9943)) ([d4c6de0](d4c6de0))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 8.5.0-alpha.15

@parseplatformorg parseplatformorg added the state:released-alpha Released as alpha version label Nov 23, 2025
parseplatformorg pushed a commit that referenced this pull request Dec 1, 2025
# [8.5.0](8.4.0...8.5.0) (2025-12-01)

### Bug Fixes

* `GridFSBucketAdapter` throws when using some Parse Server specific options in MongoDB database options ([#9915](#9915)) ([d3d4003](d3d4003))
* Deprecation warning logged at server launch for nested Parse Server option even if option is explicitly set ([#9934](#9934)) ([c22cb0a](c22cb0a))
* Parse Server option `rateLimit.zone` does not use default value `ip` ([#9941](#9941)) ([12beb8f](12beb8f))
* Queries with object field `authData.provider.id` are incorrectly transformed to `_auth_data_provider.id` for custom classes ([#9932](#9932)) ([7b9fa18](7b9fa18))
* Race condition can cause multiple Apollo server initializations under load ([#9929](#9929)) ([7d5e9fc](7d5e9fc))
* Server internal error details leaking in error messages returned to clients ([#9937](#9937)) ([50edb5a](50edb5a))

### Features

* Add `beforePasswordResetRequest` hook ([#9906](#9906)) ([94cee5b](94cee5b))
* Add MongoDB client event logging via database option `logClientEvents` ([#9914](#9914)) ([b760733](b760733))
* Add Parse Server option `allowPublicExplain` to allow `Parse.Query.explain` without master key ([#9890](#9890)) ([4456b02](4456b02))
* Add Parse Server option `enableSanitizedErrorResponse` to remove detailed error messages from responses sent to clients ([#9944](#9944)) ([4752197](4752197))
* Add support for MongoDB driver options `serverSelectionTimeoutMS`, `maxIdleTimeMS`, `heartbeatFrequencyMS` ([#9910](#9910)) ([1b661e9](1b661e9))
* Add support for more MongoDB driver options ([#9911](#9911)) ([cff451e](cff451e))
* Allow option `publicServerURL` to be set dynamically as asynchronous function ([#9803](#9803)) ([460a65c](460a65c))
* Upgrade to parse 7.1.1 ([#9954](#9954)) ([fa57d69](fa57d69))
* Upgrade to parse 7.1.2 ([#9955](#9955)) ([5c644a5](5c644a5))

### Performance Improvements

* `Parse.Query.include` now fetches pointers at same level in parallel ([#9861](#9861)) ([dafea21](dafea21))
* Remove unused dependencies ([#9943](#9943)) ([d4c6de0](d4c6de0))
* Upgrade MongoDB driver to 6.20.0 ([#9887](#9887)) ([3c9af48](3c9af48))
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 8.5.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:released Released as stable version state:released-alpha Released as alpha version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants