apache / fineract
Code health
This codebase scores 7.1 out of 10 for code health, which we rate good. It also scores maintainability 8.1 and static performance 9.8 out of 10. The three are scored separately and never blended into one number. Risk is concentrated, as it usually is: 199 of 6,930 files are git hotspots, and they average 3.9 — which is where the fixes pay off most.
Do next
4 things worth doing this quarter.
Worth planning
Break up execute (CCN 24, 91 lines)
fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/jobs/executeemail/ExecuteEmailTasklet.java:75
execute nests its control flow deeply; 1 file imports it, changed 1 time in 90 days.
- gain
- up to +1.9 health on this file(inferred)
- finding
- Nested complexity
- size
- CCN 24, 91 lines, nests 9 deep
- severity
- critical
Medium effort, worth a checkBreak up fromParsedJson (CCN 34, 147 lines)
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanChargeAssembler.java:67
fromParsedJson nests its control flow deeply; 5 files import it.
- gain
- up to +1.4 health on this file(inferred)
- finding
- Nested complexity
- size
- CCN 34, 147 lines, nests 11 deep
- severity
- critical
Medium effort, worth a checkDelete 149 unused symbols and files (2,122 lines)
Nothing in the graph reaches them, across 122 files; every reader and agent pays to skip them.
- symbols
- 149
- lines
- 2,122
- safe to delete
- Yes(inferred)
Medium effortBreak up modifyCalendar (CCN 25, 86 lines)
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanApplicationWritePlatformServiceJpaRepositoryImpl.java:369
modifyCalendar has many independent paths through it; 11 files import it.
- gain
- up to +1.2 health on this file(inferred)
- finding
- Complex method
- size
- CCN 25, 86 lines, nests 8 deep
- severity
- critical
Medium effort, worth a check
Improve what Repowise can see
Add a test coverage report
Without one, Repowise cannot tell tested code from untested code, so every test-related action says “unknown”.
- line coverage
- Unknown, no report
Small effort
- Lines of code
- 624.6K
Recent activity
All commitsCommits
- b0bfd56fFINERACT-2455: Add integration testing for monetary transaction after charge-off for WC loans - E2E tests2mo ago
- 35f53873FINERACT-2455: WC - Breach reset: Split current period - E2E tests2mo ago
- a920e6f2FINERACT-2730: add loan and savings transaction search2mo ago
- 4907881eFINERACT-2747: Add unit tests for FrequencyTypeUtil2mo ago
- 35a592fcFINERACT-2455: Add integration testing for monetary transaction after charge-off for WC loans2mo ago
- 9b9b9046FINERACT-2745: Deprecate and remove obsolete MIX XBRL reporting module2mo ago
Decisions
- FINERACT-1495-interesest-posting-performance-improvementproposed · 2mo ago
- Log email campaign JSON parsing failures instead of silently swallowing themproposed · 2mo ago
- Use explicit JDBC/JPA REQUIRES_NEW templates and remove batch JDBC manager splitproposed · 2mo ago
- Migrate ClientLoanAccountLockIntegrationTest from RestAssured to fineract-client-feignproposed · 2mo ago
- Replace SQL string concatenation with parameterized queriesproposed · 2mo ago
- Integration test clean up: remove Google Truth dependency; extract Fineract client from IntegrationTest classproposed · 2mo ago
Everything we found
10,674 open
Every detector's open count by area, each with its worst example. An inventory, not a to-do list: the actions above are the part worth doing.
- worst item is critical severity. Code health6,356Untested hotspot · fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/service/SavingsAccountReadPlatformServiceImpl.java20 findings · 9.9 deducted
- worst item is medium severity. Refactoring990filterChain+1.5 health, L effort · fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/config/SecurityConfig.java
- worst item is medium severity. Decisions205fineract-e2e-tests-core/src/test/java/org/apache/fineract/test/stepdef/loan/WorkingCapitalLoanAccountStepDef.java6 awaiting review · 199 hotspots ungoverned
- worst item is medium severity. Ownership2,966integration-tests/src/test/java/org/apache/fineract/integrationtests/client/feign/FeignLoanTestBase.java83% single-owner concentration, 21 commits in 90d
- worst item is low severity. Dead code157fineract-core/src/main/java/org/apache/fineract/organisation/monetary/serialization/MoneyDeserializer.javaUnreachable symbol (30 lines)
Composition
Open the graphExplore this codebase
- Docs9,862 pages across 39 modules
- ChatAsk this codebase a question and get an answer with its sources.
- Files6,930 files with per-file docs, health and history
- ArchitectureDependency graph, layers, and 2,253 entry points
- Code healthPer-file scores, 7,878 open findings, coverage and refactoring targets
- Knowledge graphEntities, communities, and the paths between them
- Change couplingFiles that keep changing together, mined from commit history
- CommitsChange-risk ranked history with agent provenance
- ContributorsBus factor, per-file maintainers, and the human/agent split
- StatsSize class, origin, lifetime churn, rhythm and records
- CostsWhat indexing this snapshot cost, by model and by run.