Code scene: tracking 03/04/2026

[quantranhong1999]

As of 03/04/2026, tracking the code class with health below < 9, for potential improvement:

• Dav Client (ongoing PR => Refactor DavCLient #2300)

• IdentityProvisionListener
  https://codescene.io/projects/75877/jobs/6025075/results/code/hotspots/biomarkers?name=tmail-backend%2Ftmail-backend%2Fjmap%2Fextensions%2Fsrc%2Fmain%2Fjava%2Fcom%2Flinagora%2Ftmail%2Fjames%2Fjmap%2Fevent%2FIdentityProvisionListener.java

  Mostly code duplication

• TmailMailboxMappingFactory
  https://codescene.io/projects/75877/jobs/6025075/results/code/hotspots/biomarkers?name=tmail-backend%2Ftmail-backend%2Fmailbox%2Fopensearch%2Fsrc%2Fmain%2Fjava%2Fcom%2Flinagora%2Ftmail%2Fmailbox%2Fopensearch%2FTmailMailboxMappingFactory.java

  You have 1 functions that exceed the threshold.

  generateProperties(LoC = 230 lines)
  Why does this problem occur?
  Overly long functions make the code harder to read. The recommended maximum function length for the Java language is 70 lines of code. Severity: Brain Method - Complex Method - Long Method.

• RabbitMQAndRedisEventBus
  File Code Health Impact Categories Improved
  RabbitMQAndRedisEventBus.java 8.82 → 9.39 Code Duplication

  Was improved to 9.39 after the RabbitMQ partitioning PR

• TeamMailboxManagementRoutes
  https://codescene.io/projects/75877/jobs/6025075/results/code/hotspots/biomarkers?name=tmail-backend%2Ftmail-backend%2Fwebadmin%2Fwebadmin-team-mailboxes%2Fsrc%2Fmain%2Fjava%2Fcom%2Flinagora%2Ftmail%2Fwebadmin%2FTeamMailboxManagementRoutes.java

  Code duplication, setSubAddressing method complexity

• TmailEventSerializer
  https://codescene.io/projects/75877/jobs/6025075/results/code/hotspots/biomarkers?name=tmail-backend%2Ftmail-backend%2Fguice%2Fdistributed%2Fsrc%2Fmain%2Fjava%2Fcom%2Flinagora%2Ftmail%2Fevent%2FTmailEventSerializer.java

Otherwise, most of our "unhealthy" code by CodeScene is tests, which are supposed to be explicit and accept code duplication.

[coderabbitai]

🔗 Related PRs

#2294 - JAMES-4193 Bind LegacyEncryptionModule [merged]
#2298 - Update James SHA1 [merged]

---

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

• Create Plan

---

🧪 Issue enrichment is currently in open beta.

You can configure auto-planning by selecting labels in the issue_enrichment configuration.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!

[Arsnael]

Good initiative, thanks @quantranhong1999 !

[chibenwa]

TmailMailboxMappingFactory => IMO we can silent this as we are describing a JSON