Skip to content

Modularization Risk Radar (Community Research)#4374

Open
SzymonMrozek wants to merge 2 commits intowireapp:developfrom
SzymonMrozek:develop
Open

Modularization Risk Radar (Community Research)#4374
SzymonMrozek wants to merge 2 commits intowireapp:developfrom
SzymonMrozek:develop

Conversation

@SzymonMrozek
Copy link
Copy Markdown
Contributor

@SzymonMrozek SzymonMrozek commented Feb 26, 2026

Context and intent

Hello Wire team, and thank you for all the ongoing work on modularization.
As a past contributor and someone who values Wire's open-source approach, I've been experimenting with a coding agent (Codex 5.3 extra high) as a research tool for relatively large codebases.
I noticed the ongoing modularization effort and thought this was a good moment to run that experiment and share the findings.
This is an independent, volunteer architecture review.

It is shared in a collaborative spirit:

  • findings may be useful, partially useful, or not useful for your roadmap;
  • some conclusions may be wrong and should be validated by maintainers (it was constrained to a 8h spike);
  • there are no expectations attached to this report.

If helpful, this can be treated as an early risk scan while modularization is still at a stage where directional corrections are relatively low-cost.

Wire iOS Issues are currently disabled, so this pull request is used as the discussion thread for architecture feedback.
This is a docs-only contribution with one central report and linked subreports.

What is in scope

No production code, no build logic, and no runtime behavior was changed.

TL;DR

  • Modularization progress is real and meaningful.
  • The biggest early risks are around cross-feature boundaries and integration shape, not feature extraction itself.
  • Priority order in the report:
    • Top: interface consistency, routing capabilities, app-boundary DI cost, tooling/DX.
    • Mid: internal layer leakage, broad shared-module ownership drift.
    • Low: assembly re-exports, data-model ownership split.

Impact Areas

wire-impact-areas-diagram

Recommended Sequence

wire-recommended-sequence-diagram

Reading path

Feedback welcome (if useful)

  • Validate or challenge evidence quality in the master report (file/line references).
  • Confirm whether the top-priority ordering matches current team pain.
  • Confirm whether the proposed sequence is realistic for current roadmap constraints.

Non-goals

  • Not a rewrite proposal.
  • Not a demand for immediate tooling migration.
  • Not a claim that all findings are final or complete.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Feb 26, 2026

CLA assistant check
All committers have signed the CLA.

@sonarqubecloud
Copy link
Copy Markdown

@SzymonMrozek
Copy link
Copy Markdown
Contributor Author

cc @johnxnguyen @netbe (not sure if you noticed this 'PR' 😄)

@netbe
Copy link
Copy Markdown
Collaborator

netbe commented Mar 4, 2026

cc @johnxnguyen @netbe (not sure if you noticed this 'PR' 😄)

@SzymonMrozek I noticed, but didn't take the time yet to read through :)

@johnxnguyen
Copy link
Copy Markdown
Collaborator

cc @johnxnguyen @netbe (not sure if you noticed this 'PR' 😄)

Hey @SzymonMrozek ! Sorry the delay. I hope that you have been well. Thanks for your interest in sharing these insights with us, we don't get much external engagement but it's always nice to see it. I haven't had a chance to look at your findings yet but it's on my list and I look forward to it.

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.

4 participants