-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Feat/point in time revamp 2710 #3555
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: feat/point-in-time-revamp
Are you sure you want to change the base?
Feat/point in time revamp 2710 #3555
Conversation
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This PR introduces significant changes to implement point-in-time recovery functionality for secretsV2. Here's a concise summary of the key changes:
- Added new database tables and schemas for tracking folder commits, checkpoints, and changes to support point-in-time recovery functionality
- Implemented folder commit service to track all folder/secret operations with actor metadata and change history
- Added checkpoint resources to maintain state snapshots for recovery purposes
- Created tree checkpoint functionality to handle hierarchical folder structures
- Added proper transaction handling and database schema migrations for the new features
Key points to review:
- The migration file
20250505194916_add-pit-revamp-tables.ts
should be scrutinized for proper index creation and foreign key relationships - The folder commit service implementation could benefit from batching operations for better performance with large change sets
- The recursion depth limit of 20 in folder path generation should be validated for real-world folder structures
- Column name ambiguity in SQL joins should be addressed using table qualifiers consistently
- The actor type validation could be strengthened in the commit creation process
34 file(s) reviewed, 17 comment(s)
Edit PR Review Bot Settings | Greptile
backend/src/db/migrations/20250505194916_add-pit-revamp-tables.ts
Outdated
Show resolved
Hide resolved
backend/src/services/folder-tree-checkpoint/folder-tree-checkpoint-dal.ts
Show resolved
Hide resolved
backend/src/services/folder-tree-checkpoint/folder-tree-checkpoint-dal.ts
Show resolved
Hide resolved
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) |
Description 📣
Add new commit logic on every folder/secret operation (only for secretsV2, secretsV1 are not going to be added in this feature)
Type ✨
Tests 🛠️
# Here's some code block to paste some code snippets