Repository navigation
Conversation
Signed-off-by: Denis Ćorić <denis85@gmail.com>
Signed-off-by: Denis Ćorić <denis85@gmail.com>
Signed-off-by: Denis Ćorić <denis85@gmail.com>
Signed-off-by: Denis Ćorić <denis85@gmail.com>
Signed-off-by: Denis Ćorić <denis85@gmail.com>
✅ Deploy Preview for endearing-brigadeiros-63f9d0 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This was referenced Oct 6, 2026
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1801 +/- ##
==========================================
+ Coverage 87.05% 88.62% +1.57%
==========================================
Files 113 123 +10
Lines 6482 6876 +394
Branches 1260 1305 +45
==========================================
+ Hits 5643 6094 +451
+ Misses 536 487 -49
+ Partials 303 295 -8 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Signed-off-by: Denis Ćorić <denis85@gmail.com>
Signed-off-by: Denis Ćorić <denis85@gmail.com>
Signed-off-by: Denis Ćorić <denis85@gmail.com>
Signed-off-by: Denis Ćorić <denis85@gmail.com>
14 of 21 tasks
Refresh against finos/main while retaining the repository activity summaries and current push diff/list behavior. Add shared sink regression coverage for detail payload preservation and list projections across review actions and activity rebuilds. Signed-off-by: Denis Ćorić <denis85@gmail.com>
Contributor
Author
|
@kriswest, a bit of context: this PR covers the repository activity optimisation from #1691. I split it out of #1795 following Juan’s suggestion to review it separately from the audit-storage changes. #1795 reduces duplicate audit data, while this PR maintains activity summaries so repository-page loads don’t need to recalculate them from the full push history. |
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
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.
Description
Repository pages currently scan push history to calculate counts and recent activity. Maintain summaries for the file, MongoDB, and PostgreSQL sinks instead, preserving status precedence, canonical URL matching, and latest push/pending timestamps.
File summaries update in memory. MongoDB persists revisions and retries interrupted refreshes. PostgreSQL migration 9 records audit changes transactionally and refreshes the affected summaries. Existing audits are preserved, and deployment documentation covers backfill and rebuilding derived data.
Startup migration
20261013-repository-activitybackfills summaries before either server accepts requests. MongoDB checks for duplicate push IDs without deleting audits, and concurrent dashboard reads share one refresh per process while revision checks protect against stale work from other instances. Rebuilding is available through the shared database API for all three sinks.Review actions also invalidate the relevant UI caches, and a shared helper handles numeric and serialized approval/rejection timestamps.
Related Issue
Activity portion of #1691 and PostgreSQL parity in #1794, following the merge of #1687. Audit storage and list/detail payload changes are isolated in #1795. Both PRs are needed to complete the agreed scope. The Action.diff work in #1693 and #1701 remains separate.
Validation
On a local synthetic PostgreSQL history of 10,000 pushes across 10 repositories, five-run median warm reads were 2.44 ms for summaries versus 11.27 ms for the previous history query. Initial backfill took 178.51 ms. These are workload examples, not general performance guarantees.
Checklist
No configuration schema changes.