Skip to content

imp(): get rid of duplicated releases#462

Merged
e11sy merged 2 commits intomasterfrom
fix-duplicated-releases
Sep 23, 2025
Merged

imp(): get rid of duplicated releases#462
e11sy merged 2 commits intomasterfrom
fix-duplicated-releases

Conversation

@e11sy
Copy link
Copy Markdown
Member

@e11sy e11sy commented Sep 22, 2025

Problem

Releases with same projectId and release, but with different file sets
when we tries to parse source-map not all of the files were included to the release record, so some of the stack frames were not parsed due to source-map file leak

Solution

  • add unique index on projectId-release pair
  • in this migration consolidate all of the records that have duplicated projectId-release pair
  • add special handling to the release worker - it not tries to insert new record, and catches DUPLICATE_KEY_ERROR from the db, after that waits and calls findOneAndUpdate method

@e11sy e11sy merged commit 1ec0662 into master Sep 23, 2025
4 checks passed
@e11sy e11sy deleted the fix-duplicated-releases branch September 23, 2025 20:11
e11sy added a commit that referenced this pull request Sep 23, 2025
* chore: added try-catch for sending report (#457)

* chore: added try-catch for sending report

* added res log

* chore(license): adopt BSL 1.1 (#460)

* chore: update license to BUSL-1.1 in package.json and README.md

* chore: update licensed work reference in LICENSE file

* imp(): get rid of duplicated releases (#462)

* imp(): get rid of duplicated releases

* chore(): eslint fix

---------

Co-authored-by: Vyacheslav Chernyshev <81693471+slaveeks@users.noreply.github.com>
Co-authored-by: Dobrunia Kostrigin <48620984+Dobrunia@users.noreply.github.com>
Co-authored-by: e11sy <130844513+e11sy@users.noreply.github.com>
Dobrunia added a commit that referenced this pull request Oct 1, 2025
* chore: added try-catch for sending report (#457)

* chore: added try-catch for sending report

* added res log

* chore(license): adopt BSL 1.1 (#460)

* chore: update license to BUSL-1.1 in package.json and README.md

* chore: update licensed work reference in LICENSE file

* imp(): get rid of duplicated releases (#462)

* imp(): get rid of duplicated releases

* chore(): eslint fix

* fix(grouper): skip notifications for muted errors (#459)

* feat(grouper): add event ignored check before notifying worker

* Implemented a new method to check if an event is marked as ignored.
* Updated the notifier task addition to only proceed if the event is not ignored.

* Removed the isEventIgnored method as its functionality is now integrated directly into the task addition logic.

* Removed unnecessary line break

* update @hawk.so/types dependency to version 0.1.35

* add tests for event marks handling in NotifierWorker

* fix

---------

Co-authored-by: Vyacheslav Chernyshev <81693471+slaveeks@users.noreply.github.com>
Co-authored-by: Dobrunia Kostrigin <48620984+Dobrunia@users.noreply.github.com>
Co-authored-by: e11sy <130844513+e11sy@users.noreply.github.com>
e11sy added a commit that referenced this pull request Oct 5, 2025
* chore: added try-catch for sending report (#457)

* chore: added try-catch for sending report

* added res log

* chore(license): adopt BSL 1.1 (#460)

* chore: update license to BUSL-1.1 in package.json and README.md

* chore: update licensed work reference in LICENSE file

* imp(): get rid of duplicated releases (#462)

* imp(): get rid of duplicated releases

* chore(): eslint fix

* fix(grouper): skip notifications for muted errors (#459)

* feat(grouper): add event ignored check before notifying worker

* Implemented a new method to check if an event is marked as ignored.
* Updated the notifier task addition to only proceed if the event is not ignored.

* Removed the isEventIgnored method as its functionality is now integrated directly into the task addition logic.

* Removed unnecessary line break

* update @hawk.so/types dependency to version 0.1.35

* add tests for event marks handling in NotifierWorker

* fix

* chore(release): check mapContent file field (#464)

---------

Co-authored-by: Vyacheslav Chernyshev <81693471+slaveeks@users.noreply.github.com>
Co-authored-by: Dobrunia Kostrigin <48620984+Dobrunia@users.noreply.github.com>
Co-authored-by: e11sy <130844513+e11sy@users.noreply.github.com>
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.

3 participants