Skip to content

docs: subsystem analysis for uPortal 6/7 modernization#2917

Merged
bjagg merged 1 commit into
uPortal-Project:masterfrom
bjagg:docs/subsystem-analysis
Mar 24, 2026
Merged

docs: subsystem analysis for uPortal 6/7 modernization#2917
bjagg merged 1 commit into
uPortal-Project:masterfrom
bjagg:docs/subsystem-analysis

Conversation

@bjagg

@bjagg bjagg commented Feb 14, 2026

Copy link
Copy Markdown
Member

Summary

  • Comprehensive analysis of all 39 Gradle submodules identifying 10 logical subsystems
  • Inter-module dependency mapping with coupling hubs identified
  • Portlet API blast radius analysis (~179 files with javax.portlet imports) tiered by removal effort
  • Frontend inventory: 111 JSPs, Fluid Infusion 1.5.0, jQuery 1.12.4, Bootstrap 3 — all need modernization
  • Persistence architecture review: recommends shared utility library (not centralized subsystem) for microservice compatibility
  • Rendering pipeline breakdown: 14 primary stages, 4 portlet-specific
  • Ordered first moves for uPortal 6 modularization

Purpose

Team review document to align on subsystem boundaries before starting modularization work. This is a living document — expect iterations based on team feedback.

Test plan

  • Team reviews accuracy of dependency claims against their domain knowledge
  • Discuss proposed subsystem boundaries in architecture meeting
  • Validate persistence separation approach with DB team

🤖 Generated with Claude Code

Comprehensive analysis of the codebase identifying 10 logical subsystems,
inter-module dependencies, portlet coupling blast radius, frontend/JSP
inventory, and persistence architecture. Includes recommended subsystem
boundaries and ordered first moves for modularization.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@bjagg bjagg merged commit d24c942 into uPortal-Project:master Mar 24, 2026
8 checks passed
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.

1 participant