merge - #166
Closed
sjoerd2025 wants to merge 40 commits into
Closed
merge#166sjoerd2025 wants to merge 40 commits into
sjoerd2025 wants to merge 40 commits into
Conversation
…on-1476546773550709856 ⚡ Optimize webhook performance with batch operations
Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
- Batched repository upserts in `syncInstallation` to reduce N+1 queries. - Replaced iterative deletion with `deleteMany` in `cleanupSuspendedInstallations`. - Added `src/lib/installation-service.test.ts` to verify behavior. Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Adds angular-standards (testing & docs enforcement), angular-best-practices, angular-migration, and ng-update from sjoerd2025/cli-docs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…y-upsert-1394745518843308883 Refactor: extract duplicate repository upsert logic
…082335678021 ⚡ Optimize retryAllFlaggedTasks to avoid redundant DB fetch
…020874637062 🧪 Add tests for getInstallationStatus
…tar-check-7064352253284591806 🔒 Security: Fix DoS vulnerability in checkIfUserStarredRepository
…service-10460793182449338768 ⚡ Bolt: Optimize Installation Service Sync and Cleanup
…8792540152619938779 Refactor: remove leftover debug logging in label setup handler
…17552834892220776312 refactor: remove debug logging in installation status handler
…96648701832816 🧪 Add tests for src/lib/number.ts
…ts-4789700094769845220 🧪 Testing Improvement: Add tests for crypto.ts
…484581071 Secure tasks router endpoints with adminProcedure
…ts-12691864612353580486 🧪 Testing Improvement: Add tests for crypto utility
…1252433783763 🔒 Fix timing attack in cron secret verification
…ueries Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
…797207216585041 ⚡ Bolt: optimize rate-limiter.ts to prevent TOCTOU race condition
Fixes N+1 API querying and sequentially-blocking operations inside `syncAllInstallations` by pre-fetching the necessary data globally and executing batched synchronization chunks concurrently. Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Replaced manual findUnique and conditionally update/create pattern with a single Prisma upsert call in src/lib/jules.ts. This reduces database round-trips from two to one. Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
Refactored the repository removal handler in \`src/app/api/webhooks/github-app/route.ts\` to use a single \`updateMany\` query with Prisma's \`in\` operator instead of executing multiple separate queries within a \`Promise.all\`. This eliminates N+1 query bottlenecks during bulk uninstallation or repository access removal events. Co-authored-by: sjoerd2025 <244033076+sjoerd2025@users.noreply.github.com>
…sk-12040283002622463515 ⚡ Bolt: Refactor manual upsert to use Prisma native upsert for JulesTask
…al-11189785954534146822 ⚡ Bolt: Optimize bulk repository removal
…ions-3088437305465949765 ⚡ Bolt: Optimize syncAllInstallations performance
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
sjoerd2025
marked this pull request as draft
September 27, 2026 16:26
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.