Skip to content

Feature/cloud adapter#10218

Draft
dblythy wants to merge 5 commits intoparse-community:alphafrom
dblythy:feature/cloud-adapter
Draft

Feature/cloud adapter#10218
dblythy wants to merge 5 commits intoparse-community:alphafrom
dblythy:feature/cloud-adapter

Conversation

@dblythy
Copy link
Member

@dblythy dblythy commented Mar 16, 2026

Pull Request

Issue

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

dblythy added 5 commits March 16, 2026 21:16
Defines the architecture for a next-generation Parse.Cloud system
that replaces triggers.js with a CloudCodeManager, supports BYO
adapters, and enables multi-language cloud code via three built-in
adapter types (Legacy, InProcess, ExternalProcess).
- Add beforePasswordResetRequest to TriggerName
- Replace file/config trigger names with virtual className pattern
  (@file, @config, @connect) matching triggers.js internals
- Add applicationId scoping (one CloudCodeManager per app on Config)
- Add GlobalConfigRouter.js to consumer migration table
- Add missing API methods (triggerExists, getJobs, runQueryTrigger,
  runFileTrigger, runGlobalConfigTrigger, runLiveQueryEventHandlers)
- Clarify validators/rate-limiting only for LegacyAdapter
- Document crash recovery ownership (manager calls unregisterAll)
- Rename getValidator param to key (covers both functions and triggers)
- Note runTrigger subsumes maybeRunAfterFindTrigger
- Make runLiveQueryEventHandlers synchronous to match existing behavior
- Add defineTrigger validation rules (enforced for all adapters)
- Add getRequestFileObject to utility function list
Comprehensive implementation plan addressing all review feedback:
- LegacyAdapter loads files only (no Parse.Cloud patching)
- triggers.js facade delegates reads AND writes with correct signatures
- AppCache persistence survives Config.put() overwrites
- _unregisterAll uses synchronous clearAll() for test cleanup
- getJobsObject() for facade backwards compatibility
- @parse-lite/cloud usage example in plan header
@parse-github-assistant
Copy link

🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review.

Tip

  • Keep pull requests small. Large PRs will be rejected. Break complex features into smaller, incremental PRs.
  • Use Test Driven Development. Write failing tests before implementing functionality. Ensure tests pass.
  • Group code into logical blocks. Add a short comment before each block to explain its purpose.
  • We offer conceptual guidance. Coding is up to you. PRs must be merge-ready for human review.
  • Our review focuses on concept, not quality. PRs with code issues will be rejected. Use an AI agent.
  • Human review time is precious. Avoid review ping-pong. Inspect and test your AI-generated code.

Note

Please respond to review comments from AI agents just like you would to comments from a human reviewer. Let the reviewer resolve their own comments, unless they have reviewed and accepted your commit, or agreed with your explanation for why the feedback was incorrect.

Caution

Pull requests must be written using an AI agent with human supervision. Pull requests written entirely by a human will likely be rejected, because of lower code quality, higher review effort and the higher risk of introducing bugs. Please note that AI review comments on this pull request alone do not satisfy this requirement.

@coderabbitai
Copy link

coderabbitai bot commented Mar 16, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0ecdc290-7ac6-4a93-99b3-75b668f86289

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

You can enable review details to help with troubleshooting, context usage and more.

Enable the reviews.review_details setting to include review details such as the model used, the time taken for each step and more in the review comments.

@parseplatformorg
Copy link
Contributor

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

Status Scan Engine 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.

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