Skip to content

Conversation

@MarcosSpessatto
Copy link

@MarcosSpessatto MarcosSpessatto marked this pull request as ready for review July 29, 2025 15:13
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 42.25352% with 41 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.29%. Comparing base (b47d0d8) to head (93474ff).

Files with missing lines Patch % Lines
...ages/federation-sdk/src/services/config.service.ts 42.25% 41 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
+ Coverage   50.03%   50.29%   +0.26%     
==========================================
  Files          77       77              
  Lines        8295     8305      +10     
==========================================
+ Hits         4150     4177      +27     
+ Misses       4145     4128      -17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ricardogarim ricardogarim merged commit 56c0724 into main Jul 29, 2025
2 checks passed
ricardogarim pushed a commit that referenced this pull request Jul 29, 2025
* refactor: move dot-env dep to the callers (out of fed-sdk)

* fix: lint
ricardogarim pushed a commit that referenced this pull request Jul 30, 2025
* refactor: move dot-env dep to the callers (out of fed-sdk)

* fix: lint
ricardogarim added a commit that referenced this pull request Jul 31, 2025
* refactor: move dot-env dep to the callers (out of fed-sdk) (#97)

* refactor: move dot-env dep to the callers (out of fed-sdk)

* fix: lint

* commit before sleeping :p

* commit everything to not lose

* with invites

* ci

* ci 2

* more small

* ci 3

* ci: do not lint js or generated files

* ci: no emit files during build

* better typings

* migrate test cases

* Revert "conflict -n"

This reverts commit 778a2ce.

* Revert "Revert "conflict -n""

This reverts commit d1319cf.

* maybe

* this is why i don't like magic

* chore: better zod typings and misc fixes on PersistentEventBase wrapper (#75)

* .

* remove the test that doesn't have the required changes in this pr

* chore!: remove v1 and v2 room version support (#76)

* .

* remove the test that doesn't have the required changes in this pr

* remove v1 and v2

* don't forget it deb

* less diff

* reset lint changes

* no generator use

* lost changes

* forgotten

* ...

* send stripped state too

* send message yeah (#94)

* feat: adds set and unset reactions events

* chore: fixes biome format

* fix: general fixes where the _id references were wrong

* fix: prev_events should now look to eventId instead of _id

* feat: makes set and unset reactions compliant with new state approach

* x

* reverts files

* removes eventId mentions by _id

* reverts files

* reverts files

* reverts files

* reverts files

* reverts files

* reverts files

* adds reaction event publishing via emitter

* chore: simplifies configs attrs and keep just one config service

* chore: adjusts unseting reactions from rc

* fix: fixes queue processing

* chore: makes persistTimelineEvent sendEventToAllServersInRoom void calls

* makes persistTimelineEvent to be awaited

---------

Co-authored-by: Marcos Spessatto Defendi <marcos.defendi@rocket.chat>
Co-authored-by: Debdut Chakraborty <debdutdeb@outlook.com>
Co-authored-by: Guilherme Gazzo <guilhermegazzo@gmail.com>
ricardogarim added a commit that referenced this pull request Aug 3, 2025
* refactor: move dot-env dep to the callers (out of fed-sdk) (#97)

* refactor: move dot-env dep to the callers (out of fed-sdk)

* fix: lint

* commit before sleeping :p

* commit everything to not lose

* with invites

* ci

* ci 2

* more small

* ci 3

* ci: do not lint js or generated files

* ci: no emit files during build

* better typings

* migrate test cases

* Revert "conflict -n"

This reverts commit 778a2ce.

* Revert "Revert "conflict -n""

This reverts commit d1319cf.

* maybe

* this is why i don't like magic

* chore: better zod typings and misc fixes on PersistentEventBase wrapper (#75)

* .

* remove the test that doesn't have the required changes in this pr

* chore!: remove v1 and v2 room version support (#76)

* .

* remove the test that doesn't have the required changes in this pr

* remove v1 and v2

* don't forget it deb

* less diff

* reset lint changes

* no generator use

* lost changes

* forgotten

* ...

* send stripped state too

* send message yeah (#94)

* feat: adds set and unset reactions events

* chore: fixes biome format

* fix: general fixes where the _id references were wrong

* fix: prev_events should now look to eventId instead of _id

* feat: makes set and unset reactions compliant with new state approach

* x

* reverts files

* removes eventId mentions by _id

* reverts files

* reverts files

* reverts files

* reverts files

* reverts files

* reverts files

* adds reaction event publishing via emitter

* chore: simplifies configs attrs and keep just one config service

* chore: adjusts unseting reactions from rc

* fix: fixes queue processing

* chore: makes persistTimelineEvent sendEventToAllServersInRoom void calls

* feat: adds set and unset reactions events

* use new server-discovery everywhere

* get help from typescript

* chore: adds leave and kick events

* x

* x

* x

* x

* x

* brings back removed files

* adjusts room service to comply with new state format

* publishes membership event to rocketchat

* adjusts internal routes to remove targetServer from input body

* makes newMembershipEvent accepts reason as optional param

---------

Co-authored-by: Marcos Spessatto Defendi <marcos.defendi@rocket.chat>
Co-authored-by: Debdut Chakraborty <debdutdeb@outlook.com>
Co-authored-by: Guilherme Gazzo <guilhermegazzo@gmail.com>
@sampaiodiego sampaiodiego deleted the refactor/move-dotenv branch September 26, 2025 19:39
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