Commit Graph

176 Commits

Author SHA1 Message Date
Marko Gaćeša 26d5e3b1b0 feat: [CODE-4774]: close PR if merge-base is non-unique (#4738)
* c1dd3a merge direct to target sha

* afbe46 close PR if merge-base is non-unique
2025-11-19 11:04:57 +00:00
Enver Biševac 9e6cc3813a feat: [CODE-4627]: add modernize tool (#4668)
* 3aef45 rebase with main

* 0e1fbb use omitzero go1.24 feature for non pointer structs

* f7254b add modernize tool
2025-10-29 14:06:35 +00:00
Marko Gaćeša 959040fd30 feat: [CODE-4589]: support for fork repo delete, pt2 (#4656)
* 4c4088 handling deleted forks - follow up
2025-10-24 10:39:22 +00:00
Marko Gaćeša 0aac751751 feat: [CODE-4589]: support for fork repo delete (#4644)
* c8c800 support for fork repo delete
2025-10-24 09:17:43 +00:00
Abhinav Singh 765f9a2160 fix: [CODE-4585]: fix default reviewers with codeowner file error (#4634)
* ec8660 Apply suggestion from code review

* b67640 Apply suggestion from code review

* 82cad0 fix: [CODE-4585]: fix default reviewers with codeowner file error
2025-10-17 06:33:32 +00:00
Marko Gaćeša e0a1c40289 feat: [CODE-4456]: PR change branch; PR target SHA update (#4590)
* 61a561 PR change branch; PR target sha update
2025-10-03 07:42:32 +00:00
Marko Gaćeša 475b6e1d74 feat: [CODE-4528]: pull requests from forks (#4574)
Pull requests from forks
2025-09-30 11:53:33 +00:00
Marko Gaćeša 3d31181453 feat: [CODE-4429]: remove unused fields from merge input (#4549)
* 7f7d70 remove unused fields from merge input
2025-09-22 09:38:10 +00:00
Darko Draskovic d41290d6d7 feat: [CODE-4402]: Use the most recent reviewer SHA in case no PR latest SHA reviews (#4461)
* 980261 Use the most recent reviewer SHA in case no PR latest SHA reviews
2025-09-18 11:36:28 +00:00
Karan Saraswat 3fc62cc7a0 feat: [CODE-4433]: Add users and user group evaluations to dry run merge call (#4505)
* 53e37d fixed lint issue

* 0e575e fixed tests

* d1dc29 fixed lint and tests

* eabcc6 addressed feedback comments

* aeeca5 fixed tests

* 88803b feat: [CODE-4433]: Add users and user group evaluations to dry run merge call (#4531)

* 0690d9 feat: [CODE-4433]: Removed user group evaluations and started sending effective evaluations

* cbd4e0 formatted the code

* a5eee1 feat: [CODE-4433]: Add users and user group evaluations to dry run merge call (#4518)

* e53488 feat: [CODE-4433]: Add users and user group evaluations to dry run merge call (#4506)

* c66c55 took out common part

* 719b7d feat: [CODE-4433]: Add users and user group evaluations to dry run merge call
2025-09-17 11:41:36 +00:00
Johannes Batzill 6ed8c1213f fix: [CODE-4411]: Do not updated 'updated' timestamp when pr merge check metadata is updated (#4469) 2025-09-08 17:42:45 +00:00
Darko Draskovic ceb8839d72 feat: [CODE-4402]: Enhance user group reviewer decision logic with SHA-aware prioritization (#4447)
* e5c471 Merge remote-tracking branch 'origin/main' into dd/group-reviewer-decision-sha

* c7fd06 Add userGroupReviewerDecisions helper

* 246724 Add user decisions to user group reviwer add output

* 5496f1 Enhance user group reviewer decision logic with SHA-aware prioritization
2025-09-03 18:09:58 +00:00
Darko Draskovic e15b4ed2df feat: [CODE-4314]: Add default user group reviewers (#4405)
* ddc523 Fix pullreq verify var name

* 50116b Merge remote-tracking branch 'origin/main' into dd/default-usergroup-reviewers

* fbdd12 Merge remote-tracking branch 'origin/main' into dd/default-usergroup-reviewers

* a38ddf Refactor usr grp store method to return int group map

* 68d5bf Add default user group reviewers
2025-09-01 11:57:00 +00:00
Karan Saraswat 383740b0d8 feat: [CODE-4384]: Add user group activities on PR page (#4407)
* 35c720 feat: [CODE-4384]: Add user group activities on PR page
2025-08-28 12:00:14 +00:00
Karan Saraswat 2a9463b7d3 feat: [CODE-4090]: Add users group reviewers activity with mentions (#4401)
* afe521 addressed review comments

* d5f1c7 feat: [CODE-4090]: Add users group reviewers activity with mentions (#4402)

* f61277 feat: [CODE-4090]: Add users group reviewers activity with mentions
2025-08-27 18:04:03 +00:00
Darko Draskovic 5c8e154e43 feat: [CODE-4315]: Add feature to add user group reviewers on pr creation (#4333)
* 44862f Remove decision from add/remove user group reviewer activity

* 421eed Merge remote-tracking branch 'origin/main' into dd/pr-create-usergroup-reviewers

* 37ef4f Merge remote-tracking branch 'origin/main' into dd/pr-create-usergroup-reviewers

* 2c5e59 Add PullRequestActivityPayloadUserGroupReviewerDelete

* aeb74f Add PullRequestActivityPayloadUserGroupReviewerAdd

* 976dcf Merge remote-tracking branch 'origin/main' into dd/pr-create-usergroup-reviewers

* 22ab2c Add added user group reviewers on pr creation
2025-08-21 11:57:59 +00:00
Abhinav Singh 81dbdad72a fix: [CODE-4337]: make pr branch banner not show up for non deleted pr branches (#4315)
* 7c5e60 fix: [CODE-4338]: address code comment

* 6e2d54 Merge remote-tracking branch 'origin' into abhinav/CODE-4337

* bf14da Merge remote-tracking branch 'origin' into abhinav/CODE-4337

* 78a271 fix: [CODE-4338]: address code comment

* ced409 fix: [CODE-4338]: address code comment

* 1bbe60 fix: [CODE-4338]: address code comment

* 3534de fix: [CODE-4338]: address code comment

* 8a8733 fix: [CODE-4338]: address code comment

* 7c11f3 Merge remote-tracking branch 'origin' into abhinav/CODE-4337

* 3955fd fix: [CODE-4338]: address code comment

* 683b50 fix: [CODE-4338]: fix lint errors

* 3dfacd fix: [CODE-4337]: make pr branch banner not show up for non deleted pr branches

* 4be4a6 fix: [CODE-4337]: make pr branch banner not show up for non deleted pr branches

* a368fe fix: [CODE-4337]: make pr branch banner not show up for non deleted pr branches
2025-08-18 20:23:39 +00:00
Darko Draskovic 60e1e54f5f feat: [CODE-4295]: Add proper user groups in CODEOWNERS validation (#4220)
* 739404 Merge remote-tracking branch 'origin/main' into dd/usergoup-codeowners

* 343314 Avoid if nesting on ErrNotFound

* 303cb9 Merge remote-tracking branch 'origin/main' into dd/usergoup-codeowners

* 79a906 Merge remote-tracking branch 'origin/main' into dd/usergoup-codeowners

* 614de6 Remove prepareRuleReviewers helper

* 4787ea Merge remote-tracking branch 'origin/main' into dd/usergoup-codeowners

* f0f12b Add createPullReqVerify helper to pullreq controller

* 63e2ea Streamline pr create reviewer prepare and create

* f5864f Make log and error msgs more consistent and accurate

* f1e90b Rename store.UserGroupReviewersStore to store.UserGroupReviewerStore for consistency

* 05fd57 Remove reduntant 'owner' pre/in-fix from codeowners package structs

* 73bd49 Merge remote-tracking branch 'origin/main' into dd/usergoup-codeowners

* a02a05 Merge remote-tracking branch 'origin/main' into dd/usergoup-codeowners

* aa7eb2 Merge remote-tracking branch 'origin/main' into dd/usergoup-
2025-08-18 11:14:45 +00:00
Johannes Batzill 8c98031653 fix: [CODE-4304]: Ensure user facing error messages are upper case (#4287) 2025-08-12 22:09:18 +00:00
Darko Draskovic 40d8af7dfe feat: [CODE-4055]: Add option to include/exclude repos in account/org/project rules (#3978)
* 094fa1 Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* d0f6b8 Renumber db migration

* 95cc4b Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* 528b64 Fix bool return func names

* 476a1b Rename RepoTargetFilter fields

* 70c41b Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* e8442a Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* bd7745 Fix migrations

* b336f3 Refactor forEachRuleMatchBranch to match repo name

* f8d446 Rename match branch funcs to match ref and use repo identifier instead of git uid

* 6ff336 Fix tests

* c0fee2 Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* 141b03 Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* 2832c6 Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* f74c2d Merge remote-tracking branch 'origin/main' into dd/exclude-repos

* 236a09 Refactor RepoTarget Matches function and add unit tests

* 5a1675 Merge remote-trac
2025-08-01 15:30:18 +00:00
Karan Saraswat d547a06fb0 feat: [CODE-4230]: Fix and add audit logs for rules and tag (#4125)
* feat: [CODE-4230]: Fix and add audit logs for rules and tag (#4126)

* harness-auto-fix created this fix

* harness-auto-fix created this fix

* harness-auto-fix created this fix

* feat: [CODE-4230]: Fix and add audit logs for rules and tag

feat: [CODE-4230]: Fix and add audit logs for rules and tag
2025-07-25 11:22:34 +00:00
Karan Saraswat d95fdc1d5d feat: [CODE-4110]: Add dry run rules param to create tag API (#4039)
* feat: [CODE-4110]: add violations to the CreateBranchOutput

* addressed review comment, fixed status code to 200 and added to openapi

* Apply suggestion from code review

* feat: [CODE-4110]: Add dry run rules param to create tag API
2025-07-18 03:47:52 +00:00
Johannes Batzill 54bec56832 feat: [CODE-4062]: Add 'delete_source_branch' flag to merge API call (#3985) 2025-07-07 21:05:59 +00:00
Abhinav Singh 8745233118 feat: [CODE-4038]: fix pr banner query for pq (#3949) 2025-06-30 20:02:33 +00:00
Darko Draskovic 0b3ceb0f5e feat: [CODE-3981]: Add possibility to add user groups to bypass list (#3882)
* Merge remote-tracking branch 'origin/main' into dd/usergroups-bypass-list
* Add possibility to add user groups to bypass list
2025-06-16 16:39:00 +00:00
Marko Gaćeša 01968a8f85 feat: [CODE-2548]: refactor git layer commit handling (#3870)
* refactor git layer commit handling
2025-06-11 10:15:00 +00:00
Darko Draskovic 8ebc5af8ac fix: [CODE-2191]: Rename ReviewDecision to Decision in types.UserGroupReviewerDecision (#3863)
* Rename ReviewDecision to Decision in types.UserGroupReviewerDecision
2025-06-06 16:32:37 +00:00
Darko Draskovic bd4a2623bd fix: [CODE-2191]: Refactor add user group reviewer to fix, optimize and make consistent (#3812)
* Merge remote-tracking branch 'origin/main' into dd/usergroup-reviewers
* Remove redundant ListUserIDsByGroupID
* Move space find from user group svc user group ctrl
* Merge remote-tracking branch 'origin/main' into dd/usergroup-reviewers
* Check map key exists
* Merge remote-tracking branch 'origin/main' into dd/usergroup-reviewers
* Add getHighestOrderDecision helper and remove pointers from resp struct
* Remove ListUserIDsByGroupID and add MapGroupIDsToPrincipals
* Add logic to derive user group reviewer uniqe decision
* Merge remote-tracking branch 'origin/main' into dd/usergroup-reviewers
* Merge remote-tracking branch 'origin/main' into dd/usergroup-reviewers
* Merge remote-tracking branch 'origin/main' into dd/usergroup-reviewers
* Add proper userGroupReviewerDecisions to UserGroupReviewers
* Rename serach service to service and search to list
* Merge remote-tracking branch 'origin/main' into dd/usergroup-reviewers
* Refactor SearchService
* Merge remote-tracking branch 'origin
2025-06-06 15:43:18 +00:00
Abhinav Singh d7b60af902 feat: [CODE-3841]: branches table and populate it (#3779)
* feat: [CODE-3837]: PR banner
* Apply suggestions from code review
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* Merge remote-tracking branch 'origin' into abhinav/pr-banner
* feat: [CODE-3837]: PR banner
* Merge remote-tracking branch 'origin' into abhinav/pr-banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* Update app/api/request/common.go
* Update app/api/request/common.go
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner (#3784)

* feat: [CODE-3837]: PR banner
* feat: [CODE-3837]: PR banner
* Merge remote-tracking branch 'origin/abhinav/pr-banner' into abhinav/pr-banner

# Conflicts:
#	app/store/
2025-05-29 21:19:52 +00:00
Darko Draskovic 60bc0a7d79 feat: [CODE-3788]: Add tag rule type (#3761)
* Merge remote-tracking branch 'origin/main' into dd/tag-rules
* Use strings builder
* Add rule type string slice flatten for sqlite
* Merge remote-tracking branch 'origin/main' into dd/tag-rules
* Revert to force update
* Merge remote-tracking branch 'origin/main' into dd/tag-rules
* Add separate life cycle rules for tags and branches
* Merge remote-tracking branch 'origin/main' into dd/tag-rules
* Remove reflection
* Merge remote-tracking branch 'origin/main' into dd/tag-rules
* Rename GetRuleInfos to GetBranchRuleInfos
* Add update tag pre-receive violation
* Merge remote-tracking branch 'origin/main' into dd/tag-rules
* Merge remote-tracking branch 'origin/main' into dd/tag-rules
* Add unit tests for tagRuleSet
* Add pre-receive tag protection rule
* Rename funcs, export interfaces and further generalize/reuse rule set funcs
* Extract common rule set funcs in set_common.go
* Use ruleTypes ...types.RuleType in ListAllRepoRules store func
* Merge remote-tracking branch 'origin/main'
2025-05-20 15:24:30 +00:00
Darko Draskovic 988f6eff9b feat: [CODE-3646]: Show if PR is bypassed (#3725)
* Add pullreq_merge_violations_bypassed to store list funcs
* Merge remote-tracking branch 'origin/main' into dd/pr-bypassed
* Rename pullreq_merge_bypassed to pullreq_merge_violations_bypassed
* Add false as state
* Merge remote-tracking branch 'origin/main' into dd/pr-bypassed
* Add merge- prefix
* Merge remote-tracking branch 'origin/main' into dd/pr-bypassed
* Add bypassed field to pullreq struct
* Show if PR is bypassed
2025-04-30 14:01:42 +00:00
Johannes Batzill dbb193e834 feat: [CODE-3514]: Track git writes per account (#3671) 2025-04-22 00:51:51 +00:00
Johannes Batzill 6b50985f20 fix: [CODE-3524]: Fix cache eviction for repo move operation, improve PR revert commit title (#3686) 2025-04-16 00:19:45 +00:00
Marko Gaćeša 5e1b1b407e feat: [CODE-3279]: revert PR API (#3657)
* pass the patch filename as a param to git
* revert doesn't return conflict
* revert PR API
2025-04-10 11:16:55 +00:00
Darko Draskovic 8d7fc35548 fix: [CODE-3014]: Return 401 in case of unauthorized call without auth data (#3634)
* Remove IsAuthErr helper
* Merge remote-tracking branch 'origin/main' into dd/auth
* Fix registry tests
* Add IsAuthErr app api auth package
* Add hasAccess helper method
* Merge remote-tracking branch 'origin/main' into dd/auth
* Merge remote-tracking branch 'origin/main' into dd/auth
* Merge remote-tracking branch 'origin/main' into dd/auth
* Clean up controllers and handle anonymous in Check
* Return 401 in case of unauthorized call without auth data
2025-04-09 15:24:54 +00:00
Darko Draskovic cc64066d9a fix: [CODE-1277]: Fix response pr ret val on target branch change not updated (#3639)
* Fix response pr ret val on target branch change not updated
2025-04-04 14:09:47 +00:00
Darko Draskovic 60a182e515 feat: [CODE-1277]: Add support for PR target branch change (#3579)
* Merge remote-tracking branch 'origin/main' into dd/change-target-branch
* Merge remote-tracking branch 'origin/main' into dd/change-target-branch
* Add openapi spec for opChangeTargetBranch
* Fix updateMergeData args
* Refactor ChangeTargetBranch to use verifyBranchExistence and git.GetRefPath
* Merge remote-tracking branch 'origin/main' into dd/change-target-branch
* Refactor instumentation and event payload
* Change instrumentation and event payload
* Add target-branch-changed event
* Get new merge base and clear merge related fields
* Merge remote-tracking branch 'origin/main' into dd/change-target-branch
* Merge remote-tracking branch 'origin/main' into dd/change-target-branch
* Add merge check to target branch change
* Merge remote-tracking branch 'origin/main' into dd/change-target-branch
* Add new PullReqActivityType and EventType for audit track
* Add support for PR target branch change
2025-03-28 16:04:15 +00:00
Darko Draskovic cc2a63b797 feat: [CODE-3293]: Add create compound label activity on PR creation label assignments (#3603)
* Add PullRequestActivityLabelBase
* Merge remote-tracking branch 'origin/main' into dd/compound-label-activity
* Add create compound label activity on PR creation label assignments
2025-03-28 15:29:01 +00:00
Johannes Batzill 91c4ccf3b7 fix: [CODE-3393]: Fix PR creation when no codeowner file exists for being adding as reviewers (#3583) 2025-03-21 03:37:39 +00:00
Johannes Batzill 9d5ceb0033 fix: [CODE-3379]: Remove PR author from default reviewer evaluation (#3578) 2025-03-20 19:33:56 +00:00
Johannes Batzill bcc9fc50f1 fix: [CODE-3338]: remove pr author from list of reviewers in activities (#3536) 2025-03-12 16:26:28 +00:00
Marko Gaćeša f365ba6713 fix: [CODE-3272]: revert delete of source branch in merge git call (#3542)
* revert delete of source branch in merge git call
2025-03-12 14:48:32 +00:00
Darko Draskovic bb82438965 feat: [CODE-3278]: Add PR activity for reviewers that were added on PR creation (#3499)
* Reuse PullRequestActivityPayloadReviewerAdd and amend it with PrinciaplIDs
* Remove unused and redundant activity (sub)type
* Merge remote-tracking branch 'origin/main' into dd/pr-reviewer-creation-activity
* Replace store individual activity per reviewer added with store batch
* Add PR activity for reviewers that were added on PR creation
2025-03-03 16:07:01 +00:00
Darko Draskovic cad0fbdf98 feat: [CODE-3256]: Add merge verify rule info related to minimum required approvals count for default reviewers (#3483)
* Merge remote-tracking branch 'origin/main' into dd/merge-verify-rules
* Replace princiapl ids with infos in merge verify response
* Populate DefaultReviewerApprovalsResponse with principal ids and current count
* Add MergeVerifyRulesResponse type
2025-02-26 17:56:59 +00:00
Darko Draskovic df7ac77361 feat: [CODE-3233]: Automatically add default reviewers based on rules (#3464)
* Fix tests
* Amend on protection rules verify to include the entire rule output data
* Merge remote-tracking branch 'origin/main' into dd/rule-reviewers
* Fix protection.Branch init in mapToBranchRules
* Amend MergeVerify to include number of default reviewer approvals
* Merge remote-tracking branch 'origin/main' into dd/rule-reviewers
* Refactor add code owners as reviewers to include add default reviewers
* Automatically add default reviewers based on rules
2025-02-25 15:52:59 +00:00
Marko Gaćeša 53142b18a8 techdebt: [CODE-3218]: use context.WithoutCancel in contextutil (#3457) 2025-02-20 19:20:48 +00:00
Darko Draskovic e39ae83e78 feat: [CODE-2987]: Add codeowners as reviewers (#3439)
* Handle violations in branch CreatePullReqVerify
* Merge remote-tracking branch 'origin/main' into dd/codeowner-reviewers
* Add PullReq infix to Create rule interface and dedup reviewers in PR create
* Add create field to pullreq field in rule def
* Add codeowners as reviewers
2025-02-20 16:03:06 +00:00
Darko Draskovic 13f6558ca3 techdebt: [CODE-3169]: Move unnecessary DB calls related to reviewer creation out of the create PR tx (#3420)
* Move activity creation out of tx and reset pr act seq after pr creation
* Add storeCreateReviewerActivity
* Move unnecessary DB calls related to reviewer creation out of the create PR tx
2025-02-13 11:14:36 +00:00
Marko Gaćeša a906259562 feat: [CODE-3148]: refactor refcache (#3390)
* linter issues fix
* reintroduce repofinder to aiagent
* Merge remote-tracking branch 'origin/main' into mg/cache/space-and-repo-finder
* removed refcache deps from aiagent
* pr comments
* fix linter issues
* refactor refcache
2025-02-12 12:42:12 +00:00
Darko Draskovic 72e7a3dde9 techdebt: [CODE-3169]: Move unnecessary DB calls related to label assignment out of the create PR tx (#3402)
* Merge remote-tracking branch 'origin/main' into dd/remove-from-transaction
* Move storeLabelAssignActivity to tx
* Move pr backfill with label info and activity storing out of the tx
* Replace slices containing label/value info with an assignment map
* Move unnecessary DB calls related to label assignment out of the create PR tx
2025-02-11 18:44:17 +00:00