Executive summary
⚠ A critical security finding caps this grade — resolve it before relying on the score below; see the Security lens.
Template / example. This repo declares itself a template, kata, sample or demo — code to read or copy, not operate — so the ship-it and operate-it dimensions (CI/CD, observability, ADRs, architecture docs, deployment security) are N/A and what remains is judged accordingly.
The platform is a large, complex asset currently rated at risk, with an overall health score of 68%. While the underlying code structure and team maturity are strong, the system’s operational readiness and accessibility compliance are lagging. This gap creates a tangible business risk: the system is difficult to maintain securely and excludes a significant portion of users, potentially impacting revenue and brand reputation. The sheer scale—over 320,000 lines of production code—means that technical debt accumulates quickly, and the estimated cost to rebuild from scratch would be approximately €730,000, underscoring the value of improving the existing foundation rather than replacing it.
The primary concern is operational fragility. Despite a solid architectural base, the system’s readiness for production is compromised by low scores in security and performance visibility. This lack of observability and security rigor increases the likelihood of undetected defects and potential exposure to threats. When issues do arise, the cost in terms of outage duration and remediation effort will be high. Furthermore, the absence of measured test coverage for key JavaScript and TypeScript suites means we cannot confidently assert the stability of the user-facing components, leaving the business vulnerable to regression errors that could disrupt critical customer journeys.
A secondary, yet equally urgent, theme is accessibility exclusion. The system currently fails to meet basic standards for keyboard navigation and screen reader support. This is not merely a compliance issue; it directly limits the addressable market and exposes the company to legal risk. The effort to rectify these issues is relatively low compared to the impact, making it a high-leverage area for immediate improvement. Ignoring this drift signals a lack of inclusivity and can damage customer trust.
On the positive side, the codebase is well-organized, with high maturity and strong architectural patterns. This provides a stable foundation for future growth. To maximize leverage, the team should first focus on making custom controls keyboard-operable and ensuring proper semantic HTML. This single action addresses the most critical accessibility gaps with minimal effort, delivering immediate value and reducing risk. The picture remains partial in areas like performance and domain modeling, which were not measured, but the current data is sufficient to prioritize these urgent fixes.
No single dominant problem — the weakest areas are close, so progress on any of them moves the score.
Rebuild cost & value ~ Modeled — €240,000–€1,200,000
| Cost to rebuild | €240,000–€1,200,000 |
|---|---|
| Domain complexity | Standard |
| Quality factor | 1.0× (at 68% quality) |
| Size & shape | Large · effort split not classified (source measured from disk; the effort-tier breakdown is a C#-only syntax walk) |
| Dead frontend code | ~131 LoC unreachable (1 file(s)) |
This codebase represents roughly ~5.0 person-years of build effort (about ~€730,000 to rebuild). Its weakest lens is Accessibility at 65% — the part of that asset most exposed by the findings below.
Top priorities
Diagnosis — what's actually going on
Architecture — module dependency matrix
| depends on → | 1 .github.scripts | 2 openbank-admin-ui.src.app.api.approvals.pending.route | 3 openbank-admin-ui.src.app.api.devops.dora.route | 4 openbank-admin-ui.src.app.api.finops.ai-costs.route | 5 openbank-admin-ui.src.app.api.loyalty.route | 6 openbank-admin-ui.src.app.api.reporting.[queryId].route | 7 openbank-admin-ui.src.app.api.services.health.route | 8 openbank-admin-ui.src.app.campaigns.new.page | 9 openbank-admin-ui.src.app.docs.adr.[slug].page | 10 openbank-admin-ui.src.app.docs.cloud-architecture.page | 11 openbank-admin-ui.src.app.docs.service-map.page | 12 openbank-admin-ui.src.components.agent.AgentDiagnostics | 13 openbank-admin-ui.src.components.agent.AgentInsightsPanel | 14 openbank-admin-ui.src.components.approvals.AgentIdentityBadge | 15 openbank-admin-ui.src.components.feedback.DataUnavailable | 16 openbank-admin-ui.src.components.ui.tone | 17 openbank-admin-ui.src.lib.auth | 18 openbank-admin-ui.src.lib.cards | 19 openbank-admin-ui.src.lib.context | 20 openbank-admin-ui.src.lib.finops | 21 openbank-admin-ui.src.lib.governance.caseDecisionBrief | 22 openbank-admin-ui.src.lib.security | 23 openbank-admin-ui.src.lib.services.bff | 24 openbank-admin-ui.src.types | 25 openbank-admin-ui.src.app.api.loyalty.party.[partyId].route | 26 openbank-admin-ui.src.app.campaigns.new | 27 openbank-admin-ui.src.app.devops | 28 openbank-admin-ui.src.app.docs.cloud-architecture | 29 openbank-admin-ui.src.app.docs.customer-app.page | 30 openbank-admin-ui.src.app.system.tests | 31 openbank-admin-ui.src.components.agent | 32 openbank-admin-ui.src.components.party | 33 openbank-admin-ui.src.lib.services | 34 openbank-admin-ui.src.lib.types.test-intelligence | 35 openbank-admin-ui.src.app.api.app-status.route | 36 openbank-admin-ui.src.app.docs.adr.[slug] | 37 openbank-admin-ui.src.lib | 38 openbank-admin-ui.src.lib.governance | 39 openbank-admin-ui.src.lib.docs.sensors | 40 openbank-admin-ui.src.lib.docs |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 1 .github.scripts | ||||||||||||||||||||||||||||||||||||||||
| 2 openbank-admin-ui.src.app.api.approvals.pending.route | ||||||||||||||||||||||||||||||||||||||||
| 3 openbank-admin-ui.src.app.api.devops.dora.route | ||||||||||||||||||||||||||||||||||||||||
| 4 openbank-admin-ui.src.app.api.finops.ai-costs.route | ||||||||||||||||||||||||||||||||||||||||
| 5 openbank-admin-ui.src.app.api.loyalty.route | ||||||||||||||||||||||||||||||||||||||||
| 6 openbank-admin-ui.src.app.api.reporting.[queryId].route | ||||||||||||||||||||||||||||||||||||||||
| 7 openbank-admin-ui.src.app.api.services.health.route | ||||||||||||||||||||||||||||||||||||||||
| 8 openbank-admin-ui.src.app.campaigns.new.page | ||||||||||||||||||||||||||||||||||||||||
| 9 openbank-admin-ui.src.app.docs.adr.[slug].page | ||||||||||||||||||||||||||||||||||||||||
| 10 openbank-admin-ui.src.app.docs.cloud-architecture.page | ||||||||||||||||||||||||||||||||||||||||
| 11 openbank-admin-ui.src.app.docs.service-map.page | ||||||||||||||||||||||||||||||||||||||||
| 12 openbank-admin-ui.src.components.agent.AgentDiagnostics | ||||||||||||||||||||||||||||||||||||||||
| 13 openbank-admin-ui.src.components.agent.AgentInsightsPanel | ||||||||||||||||||||||||||||||||||||||||
| 14 openbank-admin-ui.src.components.approvals.AgentIdentityBadge | ||||||||||||||||||||||||||||||||||||||||
| 15 openbank-admin-ui.src.components.feedback.DataUnavailable | ||||||||||||||||||||||||||||||||||||||||
| 16 openbank-admin-ui.src.components.ui.tone | ||||||||||||||||||||||||||||||||||||||||
| 17 openbank-admin-ui.src.lib.auth | ||||||||||||||||||||||||||||||||||||||||
| 18 openbank-admin-ui.src.lib.cards | ||||||||||||||||||||||||||||||||||||||||
| 19 openbank-admin-ui.src.lib.context | ||||||||||||||||||||||||||||||||||||||||
| 20 openbank-admin-ui.src.lib.finops | ||||||||||||||||||||||||||||||||||||||||
| 21 openbank-admin-ui.src.lib.governance.caseDecisionBrief | ||||||||||||||||||||||||||||||||||||||||
| 22 openbank-admin-ui.src.lib.security | ||||||||||||||||||||||||||||||||||||||||
| 23 openbank-admin-ui.src.lib.services.bff | ||||||||||||||||||||||||||||||||||||||||
| 24 openbank-admin-ui.src.types | ||||||||||||||||||||||||||||||||||||||||
| 25 openbank-admin-ui.src.app.api.loyalty.party.[partyId].route | 1 | |||||||||||||||||||||||||||||||||||||||
| 26 openbank-admin-ui.src.app.campaigns.new | 1 | |||||||||||||||||||||||||||||||||||||||
| 27 openbank-admin-ui.src.app.devops | 1 | |||||||||||||||||||||||||||||||||||||||
| 28 openbank-admin-ui.src.app.docs.cloud-architecture | 1 | |||||||||||||||||||||||||||||||||||||||
| 29 openbank-admin-ui.src.app.docs.customer-app.page | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 30 openbank-admin-ui.src.app.system.tests | 1 | 1 | 1 | |||||||||||||||||||||||||||||||||||||
| 31 openbank-admin-ui.src.components.agent | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 32 openbank-admin-ui.src.components.party | 1 | |||||||||||||||||||||||||||||||||||||||
| 33 openbank-admin-ui.src.lib.services | 1 | |||||||||||||||||||||||||||||||||||||||
| 34 openbank-admin-ui.src.lib.types.test-intelligence | 10 | |||||||||||||||||||||||||||||||||||||||
| 35 openbank-admin-ui.src.app.api.app-status.route | 1 | |||||||||||||||||||||||||||||||||||||||
| 36 openbank-admin-ui.src.app.docs.adr.[slug] | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 37 openbank-admin-ui.src.lib | 1 | 2 | 1 | 1 | ||||||||||||||||||||||||||||||||||||
| 38 openbank-admin-ui.src.lib.governance | 3 | 1 | ||||||||||||||||||||||||||||||||||||||
| 39 openbank-admin-ui.src.lib.docs.sensors | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 40 openbank-admin-ui.src.lib.docs | 1 | 1 |
25→5 openbank-admin-ui.src.app.api.loyalty.party.[partyId].route depends on openbank-admin-ui.src.app.api.loyalty.route✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.api.loyalty.party.[partyId].route → type in openbank-admin-ui.src.app.api.loyalty.route) reference.
- Which types
openbank-admin-ui.src.app.api.loyalty.party.[partyId].route.LoyaltyPartyResponse→openbank-admin-ui.src.app.api.loyalty.route.LoyaltyState
- Direction
- openbank-admin-ui.src.app.api.loyalty.party.[partyId].route uses openbank-admin-ui.src.app.api.loyalty.route. Changing openbank-admin-ui.src.app.api.loyalty.route can break openbank-admin-ui.src.app.api.loyalty.party.[partyId].route, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
26→8 openbank-admin-ui.src.app.campaigns.new depends on openbank-admin-ui.src.app.campaigns.new.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.campaigns.new → type in openbank-admin-ui.src.app.campaigns.new.page) reference.
- Which types
openbank-admin-ui.src.app.campaigns.new.page→openbank-admin-ui.src.app.campaigns.new.page.CampaignTemplate
- Direction
- openbank-admin-ui.src.app.campaigns.new uses openbank-admin-ui.src.app.campaigns.new.page. Changing openbank-admin-ui.src.app.campaigns.new.page can break openbank-admin-ui.src.app.campaigns.new, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
27→13 openbank-admin-ui.src.app.devops depends on openbank-admin-ui.src.components.agent.AgentInsightsPanel✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.devops → type in openbank-admin-ui.src.components.agent.AgentInsightsPanel) reference.
- Which types
openbank-admin-ui.src.app.devops.page→openbank-admin-ui.src.components.agent.AgentInsightsPanel.AgentFinding
- Direction
- openbank-admin-ui.src.app.devops uses openbank-admin-ui.src.components.agent.AgentInsightsPanel. Changing openbank-admin-ui.src.components.agent.AgentInsightsPanel can break openbank-admin-ui.src.app.devops, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
28→10 openbank-admin-ui.src.app.docs.cloud-architecture depends on openbank-admin-ui.src.app.docs.cloud-architecture.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.docs.cloud-architecture → type in openbank-admin-ui.src.app.docs.cloud-architecture.page) reference.
- Which types
openbank-admin-ui.src.app.docs.cloud-architecture.page→openbank-admin-ui.src.app.docs.cloud-architecture.page.Status
- Direction
- openbank-admin-ui.src.app.docs.cloud-architecture uses openbank-admin-ui.src.app.docs.cloud-architecture.page. Changing openbank-admin-ui.src.app.docs.cloud-architecture.page can break openbank-admin-ui.src.app.docs.cloud-architecture, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
29→10 openbank-admin-ui.src.app.docs.customer-app.page depends on openbank-admin-ui.src.app.docs.cloud-architecture.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.docs.customer-app.page → type in openbank-admin-ui.src.app.docs.cloud-architecture.page) reference.
- Which types
openbank-admin-ui.src.app.docs.customer-app.page.Capability→openbank-admin-ui.src.app.docs.cloud-architecture.page.Status
- Direction
- openbank-admin-ui.src.app.docs.customer-app.page uses openbank-admin-ui.src.app.docs.cloud-architecture.page. Changing openbank-admin-ui.src.app.docs.cloud-architecture.page can break openbank-admin-ui.src.app.docs.customer-app.page, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
29→35 openbank-admin-ui.src.app.docs.customer-app.page depends on openbank-admin-ui.src.app.api.app-status.route cycle✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.docs.customer-app.page → type in openbank-admin-ui.src.app.api.app-status.route) reference.
- Which types
openbank-admin-ui.src.app.docs.customer-app.page.Capability→openbank-admin-ui.src.app.api.app-status.route.BiText
- Direction
- openbank-admin-ui.src.app.docs.customer-app.page uses openbank-admin-ui.src.app.api.app-status.route. Changing openbank-admin-ui.src.app.api.app-status.route can break openbank-admin-ui.src.app.docs.customer-app.page, not the reverse.
- Position
- Above the diagonal — a cycle. Neither module can be changed, tested or deployed independently until one of these dependencies goes.
30→11 openbank-admin-ui.src.app.system.tests depends on openbank-admin-ui.src.app.docs.service-map.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.system.tests → type in openbank-admin-ui.src.app.docs.service-map.page) reference.
- Which types
openbank-admin-ui.src.app.system.tests.page→openbank-admin-ui.src.app.docs.service-map.page.EvidenceState
- Direction
- openbank-admin-ui.src.app.system.tests uses openbank-admin-ui.src.app.docs.service-map.page. Changing openbank-admin-ui.src.app.docs.service-map.page can break openbank-admin-ui.src.app.system.tests, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
30→12 openbank-admin-ui.src.app.system.tests depends on openbank-admin-ui.src.components.agent.AgentDiagnostics✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.system.tests → type in openbank-admin-ui.src.components.agent.AgentDiagnostics) reference.
- Which types
openbank-admin-ui.src.app.system.tests.page→openbank-admin-ui.src.components.agent.AgentDiagnostics.Language
- Direction
- openbank-admin-ui.src.app.system.tests uses openbank-admin-ui.src.components.agent.AgentDiagnostics. Changing openbank-admin-ui.src.components.agent.AgentDiagnostics can break openbank-admin-ui.src.app.system.tests, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
30→16 openbank-admin-ui.src.app.system.tests depends on openbank-admin-ui.src.components.ui.tone✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.system.tests → type in openbank-admin-ui.src.components.ui.tone) reference.
- Which types
openbank-admin-ui.src.app.system.tests.page→openbank-admin-ui.src.components.ui.tone.Tone
- Direction
- openbank-admin-ui.src.app.system.tests uses openbank-admin-ui.src.components.ui.tone. Changing openbank-admin-ui.src.components.ui.tone can break openbank-admin-ui.src.app.system.tests, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
31→12 openbank-admin-ui.src.components.agent depends on openbank-admin-ui.src.components.agent.AgentDiagnostics✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.components.agent → type in openbank-admin-ui.src.components.agent.AgentDiagnostics) reference.
- Which types
openbank-admin-ui.src.components.agent.AgentDiagnostics→openbank-admin-ui.src.components.agent.AgentDiagnostics.Language
- Direction
- openbank-admin-ui.src.components.agent uses openbank-admin-ui.src.components.agent.AgentDiagnostics. Changing openbank-admin-ui.src.components.agent.AgentDiagnostics can break openbank-admin-ui.src.components.agent, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
31→13 openbank-admin-ui.src.components.agent depends on openbank-admin-ui.src.components.agent.AgentInsightsPanel✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.components.agent → type in openbank-admin-ui.src.components.agent.AgentInsightsPanel) reference.
- Which types
openbank-admin-ui.src.components.agent.AgentInsightsPanel→openbank-admin-ui.src.components.agent.AgentInsightsPanel.AgentInsightsPanelProps
- Direction
- openbank-admin-ui.src.components.agent uses openbank-admin-ui.src.components.agent.AgentInsightsPanel. Changing openbank-admin-ui.src.components.agent.AgentInsightsPanel can break openbank-admin-ui.src.components.agent, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
32→14 openbank-admin-ui.src.components.party depends on openbank-admin-ui.src.components.approvals.AgentIdentityBadge✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.components.party → type in openbank-admin-ui.src.components.approvals.AgentIdentityBadge) reference.
- Which types
openbank-admin-ui.src.components.party.PartySearch→openbank-admin-ui.src.components.approvals.AgentIdentityBadge.Props
- Direction
- openbank-admin-ui.src.components.party uses openbank-admin-ui.src.components.approvals.AgentIdentityBadge. Changing openbank-admin-ui.src.components.approvals.AgentIdentityBadge can break openbank-admin-ui.src.components.party, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
33→23 openbank-admin-ui.src.lib.services depends on openbank-admin-ui.src.lib.services.bff✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib.services → type in openbank-admin-ui.src.lib.services.bff) reference.
- Which types
openbank-admin-ui.src.lib.services.bff→openbank-admin-ui.src.lib.services.bff.BffFailure
- Direction
- openbank-admin-ui.src.lib.services uses openbank-admin-ui.src.lib.services.bff. Changing openbank-admin-ui.src.lib.services.bff can break openbank-admin-ui.src.lib.services, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
34→11 openbank-admin-ui.src.lib.types.test-intelligence depends on openbank-admin-ui.src.app.docs.service-map.page✕
- Type pairs
- 10 distinct (type in openbank-admin-ui.src.lib.types.test-intelligence → type in openbank-admin-ui.src.app.docs.service-map.page) references.
- Which types
openbank-admin-ui.src.lib.types.test-intelligence.AiEvalAssurance→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.ContractEvidence→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.CoverageObservation→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.EvidenceObservation→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.MutationEvidence→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.PerformanceEvidence→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.PerformanceHistoryPoint→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.RequiredTestControl→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.SyntheticJourneyEvidence→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.types.test-intelligence.TestRunHistoryPoint→openbank-admin-ui.src.app.docs.service-map.page.EvidenceState
- Direction
- openbank-admin-ui.src.lib.types.test-intelligence uses openbank-admin-ui.src.app.docs.service-map.page. Changing openbank-admin-ui.src.app.docs.service-map.page can break openbank-admin-ui.src.lib.types.test-intelligence, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
35→29 openbank-admin-ui.src.app.api.app-status.route depends on openbank-admin-ui.src.app.docs.customer-app.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.api.app-status.route → type in openbank-admin-ui.src.app.docs.customer-app.page) reference.
- Which types
openbank-admin-ui.src.app.api.app-status.route.ResolvedAdr→openbank-admin-ui.src.app.docs.customer-app.page.AdrStatus
- Direction
- openbank-admin-ui.src.app.api.app-status.route uses openbank-admin-ui.src.app.docs.customer-app.page. Changing openbank-admin-ui.src.app.docs.customer-app.page can break openbank-admin-ui.src.app.api.app-status.route, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
36→9 openbank-admin-ui.src.app.docs.adr.[slug] depends on openbank-admin-ui.src.app.docs.adr.[slug].page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.docs.adr.[slug] → type in openbank-admin-ui.src.app.docs.adr.[slug].page) reference.
- Which types
openbank-admin-ui.src.app.docs.adr.[slug].page→openbank-admin-ui.src.app.docs.adr.[slug].page.PageProps
- Direction
- openbank-admin-ui.src.app.docs.adr.[slug] uses openbank-admin-ui.src.app.docs.adr.[slug].page. Changing openbank-admin-ui.src.app.docs.adr.[slug].page can break openbank-admin-ui.src.app.docs.adr.[slug], not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
36→29 openbank-admin-ui.src.app.docs.adr.[slug] depends on openbank-admin-ui.src.app.docs.customer-app.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.app.docs.adr.[slug] → type in openbank-admin-ui.src.app.docs.customer-app.page) reference.
- Which types
openbank-admin-ui.src.app.docs.adr.[slug].page→openbank-admin-ui.src.app.docs.customer-app.page.AdrStatus
- Direction
- openbank-admin-ui.src.app.docs.adr.[slug] uses openbank-admin-ui.src.app.docs.customer-app.page. Changing openbank-admin-ui.src.app.docs.customer-app.page can break openbank-admin-ui.src.app.docs.adr.[slug], not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
37→7 openbank-admin-ui.src.lib depends on openbank-admin-ui.src.app.api.services.health.route✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib → type in openbank-admin-ui.src.app.api.services.health.route) reference.
- Which types
openbank-admin-ui.src.lib.api→openbank-admin-ui.src.app.api.services.health.route.ServiceHealthEntry
- Direction
- openbank-admin-ui.src.lib uses openbank-admin-ui.src.app.api.services.health.route. Changing openbank-admin-ui.src.app.api.services.health.route can break openbank-admin-ui.src.lib, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
37→11 openbank-admin-ui.src.lib depends on openbank-admin-ui.src.app.docs.service-map.page✕
- Type pairs
- 2 distinct (type in openbank-admin-ui.src.lib → type in openbank-admin-ui.src.app.docs.service-map.page) references.
- Which types
openbank-admin-ui.src.lib.test-intelligence-freshness→openbank-admin-ui.src.app.docs.service-map.page.EvidenceStateopenbank-admin-ui.src.lib.test-intelligence-state→openbank-admin-ui.src.app.docs.service-map.page.EvidenceState
- Direction
- openbank-admin-ui.src.lib uses openbank-admin-ui.src.app.docs.service-map.page. Changing openbank-admin-ui.src.app.docs.service-map.page can break openbank-admin-ui.src.lib, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
37→24 openbank-admin-ui.src.lib depends on openbank-admin-ui.src.types✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib → type in openbank-admin-ui.src.types) reference.
- Which types
openbank-admin-ui.src.lib.api→openbank-admin-ui.src.types.ServiceSnapshot
- Direction
- openbank-admin-ui.src.lib uses openbank-admin-ui.src.types. Changing openbank-admin-ui.src.types can break openbank-admin-ui.src.lib, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
37→34 openbank-admin-ui.src.lib depends on openbank-admin-ui.src.lib.types.test-intelligence✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib → type in openbank-admin-ui.src.lib.types.test-intelligence) reference.
- Which types
openbank-admin-ui.src.lib.test-intelligence-freshness→openbank-admin-ui.src.lib.types.test-intelligence.TestIntelligenceReport
- Direction
- openbank-admin-ui.src.lib uses openbank-admin-ui.src.lib.types.test-intelligence. Changing openbank-admin-ui.src.lib.types.test-intelligence can break openbank-admin-ui.src.lib, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
38→21 openbank-admin-ui.src.lib.governance depends on openbank-admin-ui.src.lib.governance.caseDecisionBrief✕
- Type pairs
- 3 distinct (type in openbank-admin-ui.src.lib.governance → type in openbank-admin-ui.src.lib.governance.caseDecisionBrief) references.
- Which types
openbank-admin-ui.src.lib.governance.caseDecisionBrief→openbank-admin-ui.src.lib.governance.caseDecisionBrief.CaseDecisionBriefopenbank-admin-ui.src.lib.governance.caseDecisionBrief→openbank-admin-ui.src.lib.governance.caseDecisionBrief.CaseThreadForBriefopenbank-admin-ui.src.lib.governance.caseStatusPresentation→openbank-admin-ui.src.lib.governance.caseDecisionBrief.CaseStatus
- Direction
- openbank-admin-ui.src.lib.governance uses openbank-admin-ui.src.lib.governance.caseDecisionBrief. Changing openbank-admin-ui.src.lib.governance.caseDecisionBrief can break openbank-admin-ui.src.lib.governance, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
38→29 openbank-admin-ui.src.lib.governance depends on openbank-admin-ui.src.app.docs.customer-app.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib.governance → type in openbank-admin-ui.src.app.docs.customer-app.page) reference.
- Which types
openbank-admin-ui.src.lib.governance.docs→openbank-admin-ui.src.app.docs.customer-app.page.AdrStatus
- Direction
- openbank-admin-ui.src.lib.governance uses openbank-admin-ui.src.app.docs.customer-app.page. Changing openbank-admin-ui.src.app.docs.customer-app.page can break openbank-admin-ui.src.lib.governance, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
39→10 openbank-admin-ui.src.lib.docs.sensors depends on openbank-admin-ui.src.app.docs.cloud-architecture.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib.docs.sensors → type in openbank-admin-ui.src.app.docs.cloud-architecture.page) reference.
- Which types
openbank-admin-ui.src.lib.docs.sensors.SensorEntry→openbank-admin-ui.src.app.docs.cloud-architecture.page.Status
- Direction
- openbank-admin-ui.src.lib.docs.sensors uses openbank-admin-ui.src.app.docs.cloud-architecture.page. Changing openbank-admin-ui.src.app.docs.cloud-architecture.page can break openbank-admin-ui.src.lib.docs.sensors, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
39→35 openbank-admin-ui.src.lib.docs.sensors depends on openbank-admin-ui.src.app.api.app-status.route✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib.docs.sensors → type in openbank-admin-ui.src.app.api.app-status.route) reference.
- Which types
openbank-admin-ui.src.lib.docs.sensors.SensorEntry→openbank-admin-ui.src.app.api.app-status.route.BiText
- Direction
- openbank-admin-ui.src.lib.docs.sensors uses openbank-admin-ui.src.app.api.app-status.route. Changing openbank-admin-ui.src.app.api.app-status.route can break openbank-admin-ui.src.lib.docs.sensors, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
40→10 openbank-admin-ui.src.lib.docs depends on openbank-admin-ui.src.app.docs.cloud-architecture.page✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib.docs → type in openbank-admin-ui.src.app.docs.cloud-architecture.page) reference.
- Which types
openbank-admin-ui.src.lib.docs.sensors→openbank-admin-ui.src.app.docs.cloud-architecture.page.Status
- Direction
- openbank-admin-ui.src.lib.docs uses openbank-admin-ui.src.app.docs.cloud-architecture.page. Changing openbank-admin-ui.src.app.docs.cloud-architecture.page can break openbank-admin-ui.src.lib.docs, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
40→39 openbank-admin-ui.src.lib.docs depends on openbank-admin-ui.src.lib.docs.sensors✕
- Type pairs
- 1 distinct (type in openbank-admin-ui.src.lib.docs → type in openbank-admin-ui.src.lib.docs.sensors) reference.
- Which types
openbank-admin-ui.src.lib.docs.sensors→openbank-admin-ui.src.lib.docs.sensors.SensorFamily
- Direction
- openbank-admin-ui.src.lib.docs uses openbank-admin-ui.src.lib.docs.sensors. Changing openbank-admin-ui.src.lib.docs.sensors can break openbank-admin-ui.src.lib.docs, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
At a glance — Code Health
At a glance — Architecture
At a glance — Maturity
At a glance — Readiness
At a glance — Security
At a glance — Accessibility
Security & Compliance — OWASP Top-10 mapping
| OWASP category | Findings | Severity |
|---|---|---|
| A05:2021 — Security Misconfiguration | 1399 | REDACTED / Critical |
| A03:2021 — Injection | 75 | REDACTED / Critical |
| A06:2021 — Vulnerable & Outdated Components | 58 | REDACTED / Critical |
| A02:2021 — Cryptographic Failures | 2 | REDACTED / Critical |
Roadmap
First, ensure all custom controls are fully keyboard-operable and anchors have valid hrefs to fix interactive element issues. Next, establish proper page structure by declaring language, title, and main landmarks, while ensuring every form control and button has a programmatic label. Finally, enforce accessibility standards in the development toolchain via linting and CI checks, and refactor unreachable entry-point modules to be directly importable for proper test coverage.
| Do this | Helps | Effort | Dimension |
|---|---|---|---|
| Make custom controls keyboard-operable (role + tabindex + key handler), drop positive tabindex, and give anchors a real href. | +3.4 pts | REDACTED | Keyboard semantics |
| Declare <html lang>, a document <title> and a <main> landmark, keep headings in order, leave zoom enabled, title iframes and drop meta-refresh. | +3.2 pts | REDACTED | Page structure |
| Give every control a programmatic label (a <label for> / wrapping <label> / aria-label) and every button text — a placeholder is not a label. | +3.1 pts | REDACTED | Forms & labels |
| Enforce accessibility in the toolchain: add eslint-plugin-jsx-a11y, then run the suite that already asserts accessibility in CI so a regression blocks the merge. | +3.0 pts | REDACTED | A11y enforcement |
| 17 of the 18 unreached modules are entry points — the browser loads some as a top-level script and something else executes the rest by path, so no import specifier addresses them and no public entry publishes them. An importing test cannot be written for them as they ship, and two different things are worth doing here — they are not the same thing. To EXERCISE them, a browser or end-to-end suite that loads the page, or a test that runs the script the way its caller does; that is worth having, but it will NOT clear this row, because reachability is measured over the import graph and running a file by path adds no edge to it. To CLEAR it, make them addressable: lift the logic into a module a test can import and leave the entry points thin shims over it, or put their bodies behind a main-guard (`import.meta.url === argv[1]`) and export them. Add a test that imports the remaining one — directly or through its public entry. | +2.3 pts | REDACTED | Test Coverage |
| Remove unused dependencies, declare unlisted imports explicitly, and demote type-/test-only packages to devDependencies. | +2.3 pts | REDACTED | Dependency Hygiene |
| Bump outdated dependencies to current versions to limit upgrade debt. | +1.7 pts | REDACTED | Dependency Freshness |
| Break down the listed branch-heavy functions; aim P95 cyclomatic ≤ 5. | +0.9 pts | REDACTED | Cyclomatic Complexity |
File quality
| File | Score | Band | Worst signal |
|---|---|---|---|
| REDACTED | 0.0 | Slop | Dependency Vulnerabilities: Critical CVE: REDACTED |
| REDACTED | 3.0 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.4 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.4 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.4 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.4 | Mixed | IaC & Container Security: REDACTED IaC: REDACTED |
| REDACTED | 4.4 | Mixed | IaC & Container Security: REDACTED IaC: REDACTED |
| REDACTED | 4.4 | Mixed | IaC & Container Security: REDACTED IaC: REDACTED |
| REDACTED | 4.4 | Mixed | IaC & Container Security: REDACTED IaC: REDACTED |
| REDACTED | 4.4 | Mixed | IaC & Container Security: REDACTED IaC: REDACTED |
| REDACTED | 4.5 | Mixed | IaC & Container Security: REDACTED IaC: REDACTED |
| REDACTED | 4.7 | Mixed | IaC & Container Security: REDACTED IaC: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
How the grades work
A definite problem that already costs you something and drags the score down: a missing authorisation check, a dependency with a known exploit, a build that does not reproduce. Failure here tends to cause failures elsewhere.
Likely wrong, but not failing yet. It degrades the codebase over a longer horizon and can cause failures elsewhere — not urgent this week, not something to carry for two years either.
Recorded, with no effect on how the codebase functions. Present so the survey is complete, not because it needs doing.
Something this survey could not settle from the outside, and which could be critical or serious. Either a control was required and no positive evidence of it exists in the repository — a backup job that nothing shows was ever restored from proves nothing about restores — or our own analysis could not run over that part of the tree. This is not a clean result. These are excluded from the score rather than awarded a pass, so the number on the cover neither rewards nor penalises them: if you act on this survey without resolving them, you carry that risk yourself. Each one is named under Limitations.
Methodology & how to trust this report
Watchdog is a deep, periodic assessment — run each sprint, monthly, or quarterly, taking the time to go wider and deeper than a quick check and surfacing in one coherent report what you'd otherwise piece together from a dozen separate tools. It scores deterministically: the same commit yields the same score, every run. 44 of 49 evaluated dimensions are computed purely by tools and static analysis (confidence 1.0); 5 documentation/naming judgement(s) are LLM-assisted and labelled advisory. Overall confidence is 0.8 — the weighted average across measured dimensions; it falls as more of the score leans on LLM-assisted judgement and rises when it's fully tool-backed.
What we checked — 49 dimensions across the health lenses
- Can you open the finding? Real findings cite a repo-relative file and line you can open at the cited line — never an absolute scratch path. Here, 1698 of 1708 do; the remainder are repo-wide signals — a dimension-level measurement, not a single line.
- Is there a tool behind the number? Every score below names the method that produced it — Roslyn, git, a scanner, or (for a handful of documentation/naming dimensions) an LLM labelled sampled · advisory — not a narrative.
- Does re-running give the same result? Run it again on the same commit and the score — and this report, byte for byte — is identical. A report whose numbers move between runs is describing the run, not the code.
Tools & methods
| Method | Backs | Version | Evaluator |
|---|---|---|---|
| Roslyn static analysis | Complexity, cohesion, coupling, dead code, API surface, layering | 5.3.0 | ✓ deterministic |
| Native secret scanner | Hardcoded secrets / credentials | 1.0.0 | ✓ deterministic |
| Watchdog duplication detector (in-process) | Code duplication | 1.0.0 | ✓ deterministic |
| Coverage (coverlet / dotnet-coverage) | Line & branch coverage | 10.0.400 | ✓ deterministic |
| NuGet / dotnet | Outdated, vulnerable & deprecated dependencies | 10.0.400 | ✓ deterministic |
| git / LibGit2Sharp | Churn hotspots, knowledge concentration, history | 2.43.0 · 0.31.0 | ✓ deterministic |
| gitleaks · semgrep · trivy · checkov | Secrets in history, SAST, CVEs, IaC & container, PII / GDPR | 1.86.0 · 0.69.3 · 3.2.533 | ✓ deterministic |
| LLM (sampled · advisory) | Documentation quality, ADR conformance, naming — sampled over a bounded sample; advisory, never a deterministic measurement | Local LLM | ◐ LLM · sampled · advisory |
Run transparency — what happened this run
- D1 Cyclomatic Complexity — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Most of this repository's production source (.kt) had no cyclomatic complexity computed for it: no method bodies were exposed for those file kinds by any language model this pass could load.
- D2 Cognitive Complexity — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Most of this repository's production source (.kt) had no cognitive complexity computed for it: no method bodies were exposed for those file kinds by any language model this pass could load.
- D4 Code Duplication — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Most of this repository's production source (.kt, .py) was not read by duplication detection: no source of those file kinds was exposed to the token comparison by any language model this pass could load. Duplication in .ts, .tsx is measured by R10 Code Duplication and is not part of this gap.
- D10 Test Quality — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. The 2,320 test(s) behind this row are the ones the JavaScript/TypeScript census could read, and this repository also carries at least 1,798 test source file(s) (.kt, .py) that it cannot: it reads JavaScript/TypeScript test declarations off disk, so a JUnit/pytest-style suite is invisible to it. Skipped tests, zero-assertion tests and the other quality signals on this row are UNMEASURED in that suite — their absence from the counts above is a gap in this analyzer's language coverage, not a finding that those tests are sound.
- D11 Test Reliability — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Test reliability not included — the JavaScript/TypeScript suite (vitest via `npm ci --ignore-scripts` in openbank-admin-ui/ (3175 tests)) did not finish re-running within its budget: too large to re-run within its budget.
- D12 Dependency Hygiene — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Dependency Hygiene ran out of its 5-minute budget before it had finished, so what it reports here is a floor rather than a complete count. The rows above are real and stand; what is not known is how many more there are. This is a limit of the analysis run, not a finding about this repository.
- D14 License Compliance — evaluation did not complete — License Compliance not included (check did not complete) — excluded from the score.
- D15 Churn × Complexity Hotspots — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. A hotspot is churn × complexity. Churn was measured (1456590 line(s) across the 90-day window), but no complexity could be computed for .kt, .py, which is most of this repository's production code — so every churned file would score as complexity 0 and the hotspot list would be empty no matter how tangled the code is. Not scored — this is a gap in the analysis run, not a finding about this repository.
- D16 Bus Factor — not measured this run — This is a true statement about the repository that carries nothing for its owner to act on, so it is reported here rather than as a defect in their code. Single-contributor repository — bus factor is not applicable (1 contributor(s) across 8284 commit(s) sampled, automation and bot accounts excluded). Concentration needs a team to concentrate: with one contributor there is nobody to spread the knowledge to, so there is nothing here for the owner to act on.
- D22 Internal API Consistency — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. D22 identifies the intentionally-exposed surface from `IsPackable` and `.Contracts` project names, MSBuild conventions read off the loaded project set. This target exposed no such projects, so the probe never ran; this says nothing about whether the repository has a public API. A language sidecar this run needed FAILED, so the code model the non-MSBuild public-API collector reads was never built — this is a DEGRADED run, not a settled gap, and the surface may well be collectable on a re-run: The Kotlin sidecar EXITED with code 1 after 77.5s, so the Kotlin code model degraded to null. This is the sidecar's own failure, not a budget of ours.
- D32 Data Compliance (PII/GDPR) — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. `openbank-admin-ui/src/app/api/customer-360/[partyId]/route.ts`, `openbank-admin-ui/src/app/api/entities/resolve/route.ts`, `openbank-admin-ui/src/app/api/finops/ai-costs/route.ts`, `openbank-admin-ui/src/app/api/fx/history/route.ts`, `openbank-admin-ui/src/app/api/governance/agent-identities/route.ts`, … (+12 more) produced a parse error, so every rule in this engine's `gdpr.yml` was absent there. That absence is NOT a clean result: these rules detect personal data crossing a boundary into a log sink, a URL or browser storage, and a file that was never parsed cannot report any of the three. The rest of the tree analysed normally and its rows above stand; only these files are unaccounted for. You can widen what we reach: fix the syntax error (or exclude the file deliberately) and re-scan to cover it.
- AX3 Project dependency cycles — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over which project references which — facts that live in .csproj references. This repository either commits no MSBuild project at all (its C# may still have been parsed as syntax-only projects, which carry no references between them) or its projects failed to load, so there was no graph to read. That is a gap in this analyzer's reach — not a finding that the repository is free of what this check looks for.
- AX4 Dependency direction — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over the direction each project reference points — facts that live in .csproj references. This repository either commits no MSBuild project at all (its C# may still have been parsed as syntax-only projects, which carry no references between them) or its projects failed to load, so there was no graph to read. That is a gap in this analyzer's reach — not a finding that the repository is free of what this check looks for.
- AX6 Interface segregation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over the public interfaces this run's compilations declare, and none was loaded, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- AX8 Test isolation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over which projects are test projects, and what they reference — facts that live in .csproj references. This repository either commits no MSBuild project at all (its C# may still have been parsed as syntax-only projects, which carry no references between them) or its projects failed to load, so there was no graph to read. That is a gap in this analyzer's reach — not a finding that the repository is free of what this check looks for.
- C1 Data Protection — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These personal data controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks personal data controls.
- C2 Access Controls — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These authorization controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks authorization controls.
- C3 Audit Trail — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These audit controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks audit controls.
- C4 Data Retention — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These retention controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks retention controls.
- C5 Data-Subject Rights — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These data-subject rights controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks data-subject rights controls.
- DM1 Aggregate boundaries — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: a class holding another class reads the same whether the inner type is an aggregate or a value object, so this cannot be decided from source without guessing — reported as guidance rather than measured.
- DM2 Strongly-typed ids — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: branded ids (`type Id = string & { __brand }`) are an uncommon idiom, so a bare-string id is not on its own evidence of a missing typed id — reported as guidance rather than measured.
- DM3 Integration-event coupling — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: a domain type used across packages is indistinguishable in source from a deliberate shared-kernel package, so this is reported as guidance rather than measured.
- DM4 Rich vs anemic model — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: telling a rich domain entity from an anemic data holder needs the behaviour a source-only read cannot always attribute (components, DTOs and readonly value objects are all legitimately data-shaped), so this is reported as guidance rather than measured.
- DM5 Encapsulated state — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: the language already steers state behind #private/private/readonly, so a mutable public field is rare enough that we report this as guidance rather than measuring it.
- DM7 Repository granularity — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: deciding whether a repository belongs to an aggregate root needs the aggregate structure, which source alone does not state — reported as guidance rather than measured.
- ED5 Idempotency — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check finds retry-prone mutations by walking the repository's declared types, and NONE was loaded on this run, so it had nothing to look at. That is a limit of the analyzer's reach — it reads .NET projects — not a finding that this repository has no command handlers or message consumers.
- GD1 Unfinished & placeholder code — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- IC1 Incompleteness & stubs — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- R10 Code Duplication — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. 304 further occurrence(s) are not listed individually; the score already reflects all 344.
- S1 Web-Security Posture — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These web-security controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks web-security controls.
- X1 Async correctness — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X12 Unreachable branch — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X13 Undrained process stream — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X14 Bypassable address classification — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X15 Unvalidated length from an untrusted reader — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X16 Unfloored truncation loop — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X17 Uncapped recursion over a caller-supplied document — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X18 Disposal-pattern correctness — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X19 Unrestored process-global state — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X2 Cancellation propagation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X20 Mistyped argument guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X21 Side-effecting pattern guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X22 Contradicted release guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X23 Unguarded diagnostic materialisation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X24 Document value interpolated into markup unescaped — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X25 Inert configuration knob — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X26 Unsynchronised callback handoff — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X27 Collection changed while being enumerated — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X28 Index access outside its own emptiness guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X29 Per-element action decided by a fixed element — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X3 Exception handling — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X30 Support guard that admits what it rejects — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X32 Type resolved by simple name across every loaded assembly — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X4 Structured logging — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X5 Nullable reference types — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- Production source was not modelled — run is Degraded — No code model read .kt, so the complexity, duplication and architecture dimensions were scored over a minority of this repository rather than its product, and dimensions that abstained did so for want of a model rather than for want of findings. The grade is NOT representative of this repository; diagnostics.md records the exact cause. The Kotlin sidecar EXITED with code 1 after 77.5s, so the Kotlin code model degraded to null. This is the sidecar's own failure, not a budget of ours.
Limitations & what we did not check
Watchdog assesses the repository exactly as committed, and only the repository. By design it does not reach outside the source tree: the live cloud account, the running CI/CD pipeline, the host's branch-protection and approval rules, the production configuration, or a restore actually exercised against a backup are all out of scope. That boundary is a feature, not a gap — a repo-relative, deterministic scan re-runs identically on any commit and every finding opens at a real file and line, where a live audit can neither be reproduced nor traced. The visible consequence is that controls which leave no in-repo evidence are reported as "not evidenced" and excluded from the score rather than awarded a number a static scan cannot justify.
Per-dimension blind spots
- D7 Architectural Integrity: Layering is checked against detected/declared rules — an architecture whose boundaries live in convention or in code review, not in a rule a scanner can read, is not enforced here.
- D9 Test Distribution: The test-pyramid shape is inferred from project/folder naming and references, with a single test host bucketed per-file by its path tier and content signals — a suite that names tiers unconventionally and gives no per-file signal can still be mis-bucketed.
- D10 Test Quality: Assertion density is structural — it cannot tell a meaningful behavioural assertion from a trivial one, only that an assertion is present.
- D13 REDACTED Scanning: REDACTED detection is signature- and entropy-based on the current tree — a secret that does not match a known pattern, or one already rotated, will not be flagged (a clean scan is "nothing matched", not "no secrets exist").
- D17 Explicit Debt: Acknowledged-debt signals (TODO/FIXME, suppressions, dead code) are textual — undocumented debt that nobody marked, and debt that lives in design rather than annotations, is invisible. Committed machine-written code (scaffolded migrations, designer/codegen output, generated stubs) is excluded — it is never the team's dead code to delete.
- D19 Documentation Quality: Documentation quality is judged by an LLM over a bounded sample of docs — it reads what is written, not whether the docs match the running system, and it is advisory, not a measurement. Its critique rows are drawn from a closed category vocabulary and each row means the same thing in every run, so two scans can be compared row by row; the SET that fires is still a sample, and does not repeat exactly. Measured on one frozen input, six scans at one engine SHA: 2-5 critique rows per scan, 8 distinct rows across the six, 3 of those 8 seen in only one scan. So a D19 row is evidence about the documentation, but a COUNT of D19 rows is not a quantity — never read a change in it as an improvement or a regression.
- D20 ADR Quality: ADR quality is an LLM read of the decision records present — it cannot know about decisions made and never recorded, and its verdict is sampled and advisory.
- D21 Naming Consistency: Naming quality is an LLM judgement over a bounded sample — it assesses clarity/consistency of the names it sees, not domain-correctness, and is advisory.
- D25 ADR Conformance: ADR conformance is the LLM-scored fraction of sampled code that follows recorded decisions — it checks the decisions that were written down and the slices it sampled, not unrecorded rules or the whole tree.
- D28 Secrets (history): Secret-history scanning sweeps the git log for known patterns — a secret that predates the available history, or never matched a signature, is not found (clean means "nothing matched in the history we can see").
- D29 Static Analysis (SAST): SAST findings are pattern-based (semgrep) — it finds classes of bug it has rules for; logic flaws, auth/authorization gaps and issues needing runtime context are out of reach (and clean means "no rule matched").
- D30 Dependency Vulnerabilities: CVE matching depends on accurate package/version metadata and on the advisory databases — a vulnerability with no published advisory, or in code not declared as a dependency, is not seen. Coverage needs a RESOLVED graph: an unpinned requirements.txt, or a pom without a resolved build, yields partial coverage rather than a clean verdict. An ecosystem the analyzer cannot scan is reported as unmeasured, never as clean.
- D31 IaC & Container Security: IaC scanning checks Dockerfiles/Terraform/Kubernetes against best-practice rules — it cannot see the live cloud account, runtime configuration, or drift between the committed config and what is actually deployed.
- D34 Knowledge Freshness: Freshness is decayed commit RECENCY, not comprehension — code read often but rarely committed reads as orphaned, and stable code that genuinely needs no changes is penalised the same as forgotten code; bot/squash commits distort it like the bus factor.
- D35 Change Coupling: Change coupling is co-change in COMMITS — files split across separate commits, or coupled only through a shared config/build step, read as uncoupled, and a sweeping commit (rename/format) is excluded so it doesn't couple everything. It shows that files change together, not WHY: a high coupling can be a healthy cohesive pair as readily as a hidden leak.
- D40 Network Egress Confinement: Egress confinement is read from committed Kubernetes manifests — a policy applied out-of-band (cluster-default deny, a service mesh, or a cloud firewall/security group off-repo) is invisible, and a present NetworkPolicy is declared config, not proof the cluster admission-controller actually enforces it at runtime.
- D41 Kernel & Syscall Confinement: Syscall/MAC confinement is read from committed manifests — a profile applied by a cluster-wide PodSecurity default or a mutating webhook off-repo isn't seen, and a declared seccomp/AppArmor profile is config presence, not proof the node's kernel actually loaded and enforced it.
- D42 Runtime Threat Enforcement: Runtime enforcement is read from committed policy files — a Tetragon/Falco/Kyverno stack installed cluster-wide (Helm release, platform add-on) with no in-repo trace can't be credited, and a committed policy is declared intent, not proof the engine is running and blocking in the live cluster.
- D43 Malicious Dependencies: Only packages some vulnerability database has already NAMED as malicious are seen — a compromise published in the last hours, or never reported at all, is invisible here, and this dimension reading 10 is not evidence that a dependency is trustworthy. There is no typosquat or dependency-confusion analysis: a package nobody has reported is simply absent from the feeds. Coverage is the dependency scan's: an ecosystem that could not be scanned is disclosed as unmeasured, never as clean.
- D44 Platform End-of-Life: The support table is FROZEN, so it goes out of date by losing RECALL: a release that ended support after the table was written is missed until the table is refreshed, and this dimension reading 10 is not evidence that a platform is current. Only platforms the repository DECLARES in a place this pass reads are seen — a runtime named only in a REDACTED (D31's subject), in a CI workflow (D29's), or in a file this pass does not parse (go.mod, a Gemfile ruby directive) is invisible here, which is why a repository declaring none of them abstains rather than scoring. Only frameworks with a PUBLISHED support policy are tracked: React, Flask and Express publish none, so their age cannot be judged and their absence from a report is not a statement that they are supported.
- AC1 Text alternatives: Alt-text is detected structurally — the scan sees that an alternative EXISTS, not whether it meaningfully describes the image, and decorative-vs-missing is judged by attribute shape; runtime-injected images and a non-role=img decorative svg are out of scope. This is accessibility readiness, never a WCAG conformance claim.
- AC2 Forms & labels: Label association is read from static markup — a label wired up at runtime (JS-set aria-labelledby, framework-injected ids) reads as missing, a present label says nothing about whether its text is correct. A known UI-library field component (e.g. a JSX <TextField>) is now checked conservatively — flagged only when it carries NO label/aria-label/aria-labelledby/id/name — but wrapper/context-labelled libraries (Chakra/Radix FormControl+FormLabel) aren't statically visible (possible false positive) and non-JSX lowercased components are still skipped. A click handler on a plain element is now asked for a name too (it is a control the author declared), but the subtree test that answers it is deliberately generous: any DYNAMIC text expression in the subtree counts as a name, so an icon chosen by a ternary ({cond ? <IconA/> : <IconB/>}) reads as named, and a glyph component from a library the icon-import list does not know still names its parent. A clean result is "no unlabelled control found", not a labelling proof.
- AC3 Page structure: Page structure is read from the static markup tree — landmarks, headings and lang injected at runtime aren't seen, heading ORDER is checked structurally (not against the rendered visual hierarchy), and lang/title/main fire only on full documents, never partials, and the data-table check sees header-cell presence (a <th> exists), not whether each header correctly associates with its cells. Static readiness, not conformance.
- AC4 Keyboard semantics: Keyboard semantics are inferred from markup attributes — interactivity wired purely in script, focus managed at runtime, and component-level handlers are invisible. A clean result means "no static keyboard-trap shape", not a keyboard-operability proof.
- AC5 ARIA correctness: ARIA correctness is checked against the static role/attribute shape — roles/attributes set dynamically aren't seen, a valid role says nothing about whether it matches the element's real behaviour, and required-state checks are suppressed when a JSX spread could supply them. The two-branch toggle check (a control whose state is conveyed only by which of two mutually exclusive branches renders) reads CONDITIONALS THAT ARE ATTRIBUTES — Vue v-if/v-else/v-show and Alpine x-if/x-show — so the same toggle written as a Svelte {#if} block or a JSX ternary is control flow the markup model never projects as a branch and is not seen at all.
- AC6 Visual & motion safety: Contrast and motion safety are PARTIAL by construction — literal colours (hex/rgb/hsl/named) in inline styles, in-repo <style> blocks, in-repo .css files, var() tokens, Tailwind neutral utilities and CSS-in-JS top-level declarations are read (same-rule/same-element colour+background pairs only); computed/runtime/theme colour, external-CDN stylesheets, CSS-in-JS dynamic (${…}) and nested-selector colours, cross-element pairs and image contrast stay out of reach, so a clean result is bounded by what the static CSS itself shows.
- AC7 A11y enforcement: Enforcement is scored from in-repo config/CI evidence only — an a11y gate enforced in external tooling with no in-repo trace can't be credited, and a configured linter is presence, not proof the rules actually run or block a merge.
- AX10 Code composition: Role is inferred from namespace/folder convention, not semantics — a domain concept living in a folder named "Services" reads as application, and the split is lines-of-code, not business value. The business-logic-share score is a SOFT, FLOORED signal: it contributes to the Architecture lens but is floored at the Critical gate, so an infrastructure-heavy design (a gateway, an ETL, a driver) is legitimately low without being nuked to zero.
- M4 Documentation accuracy: Onboarding quality is an LLM read of the docs/setup present — it cannot run the onboarding or measure how long a real new joiner takes; the verdict is sampled and advisory.
- P4 Deployment & Rollback: Approval/branch-protection rules live in repository settings the scan cannot see — only their in-repo evidence (config files, workflows) is checked, so a control enforced purely in the host's settings reads as "not evidenced".
- P5 DR & Backup: Backup/restore and disaster-recovery readiness is judged from in-repo evidence — a config that exists is not a tested restore, so the absence of positive evidence is reported as "not evidenced", never scored as present.
- P6 Release Hygiene: Rollback/observability controls are inferred from repo artefacts (pipelines, dashboards-as-code) — controls configured in external tooling, with no in-repo trace, cannot be credited.
The LLM boundary
Dimensions
D7 · Architectural Integrity
All 154 mechanizable ADR(s) are enforced: 81 by analyzers, 73 by tests. Dependency cycles not checked (no project-reference graph; where this repository's language has an import-cycle lens, cycles are reported there).
D9 · Test Distribution
2516 test methods: 2317 unit, 0 integration, 0 BDD, 199 e2e. The JavaScript/TypeScript suite contributes 2336 `it`/`test` case(s) across 707 test file(s) declaring at least one; its tier split is read from package names and paths only. The Python suite contributes 180 test function(s) across 15 file(s) declaring at least one — every `def test…` in a file pytest or unittest would collect, which is those frameworks' own definition of a case; a parametrize table counts once, so this is a floor. Its tier split is read from file names and paths only.
What to do
- Improve Test Distribution — currently 9.0/10. — 2516 test methods: 2317 unit, 0 integration, 0 BDD, 199 e2e. The JavaScript/TypeScript suite contributes 2336 `it`/`test` case(s) across 707 test file(s) declaring at least one; its tier split is read from package names and paths only. The Python suite contributes 180 test function(s) across 15 file(s) declaring at least one — every `def test…` in a file pytest or unittest would collect, which is those frameworks' own definition of a case; a parametrize table counts once, so this is a floor. Its tier split is read from file names and paths only.
D10 · Test Quality
2 skipped (2 with a documented reason), 1 zero-assertion, no mocking-framework packages referenced (hand-written doubles or no mocking) across 2320 tests. Measured on the JavaScript/TypeScript suite only — at least 1,798 test source file(s) (.kt, .py) went unread, so its test quality is unmeasured and is not in these counts.
What to do
- Resolve the 1 No assertions finding(s) in Test Quality — start with lending-risk-rbac.guard.test.ts. — One of this dimension's main actionable groups (1 warning-level).
D13 · REDACTED Scanning
REDACTED scan ran and found no leaked secrets.
D17 · Explicit Debt
9 deducted task-comment markers across 321738 LoC (0.0/KLoC) → score 10.0. Task comments only: this repository's language is read without a compiler, so D17's suppression, dead-code and commented-out-code arms did not run and this score counts fewer marker kinds than a .NET repository's would.
What to do
- Resolve the 9 TodoComment finding(s) in Explicit Debt — start with GovernanceReadAdapter.kt (2), GitHubReadAdapter.kt (2), GitHubOpenPrReadAdapter.kt (2). — One of this dimension's main actionable groups (9 warning-level).
D19 · Documentation Quality
This is a comprehensive documentation set for the OpenBank OSS project: 31 README files (including one pure-JVM DST harness and standalone PDF-renderer) plus architecture/Docs markdown files totaling 1074 documents. The READMEs are well written with clear branding, status, use cases, and runnable code examples; each document is clearly outlined in its own heading and the full outline exists for every named file (e.g. 'Project Status', 'What works right now', 'What is NOT there yet'). There is no single missing section across all documents. The documentation is comprehensive and well-structured for an infrastructure-as-code project: the READMEs describe each component's purpose (OpenTofu substrate, OSS-Fuzz integration, AP2 mandate verification), show layout details (two runners split by architecture), give first-apply steps, and document security requirements (deterministic demo key, trust-listed public keys). A dedicated strategy directory explains what the project is, its strategic rationale, a compliance index mapping regulators to controls, and governance rules for adding new ADRs. The main gap is that none of the documents explicitly states how to install or run the codebase from source (no build/run commands), so an 'missing-installation' finding would apply only to the READMEs. The documentation is clear, complete, and well-structured for an open-source governance project: every README file in the governance/evals/prompts/infra/gitops/dr-restore-templates subdirectories (31 + 1074) is present and fully visible. The prompt-registry document explains why a versioned in-repo content store exists over SaaS, with explicit rules on live deployment and immutable edits; the evals registry covers the coverage-ratchet pattern applied to agent scenarios; the landing page provides local-run instructions, file-table structure, and a transparent logo build pipeline; DR restore templates describe status, apply-time substitution, and deliberately omitted Kafka/OIDC/Redis; and key-ceremony runbooks are provider-neutral with pre-check phases and dry-run requirements. The visible content is exemplary.
D20 · ADR Quality
Evaluated 305 ADR(s) individually; mean quality 9.0/10 (consistently complete and clear). 3 flagged with a specific gap.
What to do
- Resolve the 1 Context and trade-offs are absent (only a summary + amendment list… finding(s) in ADR Quality — start with 0077-observability-three-pillar-strategy.md. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 No context/problem and no consequences; only an allowlist with a… finding(s) in ADR Quality — start with known-repos.txt. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 The body is a bare list of tags and ADR domain categories with no… finding(s) in ADR Quality — start with tags.txt. — One of this dimension's main actionable groups (1 warning-level).
D21 · Naming Consistency
0 naming inconsistencies across 0 sampled symbols.
D25 · ADR Conformance
37 conform / 5 violate across 305 ADRs.
What to do
- Resolve the 5 ADR not followed finding(s) in ADR Conformance — start with 0009-postgres-per-service.md, 0021-sca-decoupled-device-approval-no-auto-approve.md, 0071-governance-manifest-as-derived-data.md. — One of this dimension's main actionable groups (5 issue-level).
D28 · Secrets (history)
1 finding(s): 0 critical, 1 high, 0 medium, 0 low. Remediation for historically-committed secrets is credential rotation — they remain in history regardless of later deletion.
What to do
- Resolve the 1 REDACTED finding(s) in Secrets (history) — start with REDACTED. — One of this dimension's main actionable groups (1 issue-level).
- Resolve the 1 Rotate the exposed credentials finding(s) in Secrets (history). — One of this dimension's main actionable groups (1 recommendation-level).
D29 · Static Analysis (SAST)
75 finding(s): 0 critical, 33 high, 39 medium, 3 low. 1 unpinned-GitHub-Actions row(s) are reported here but scored by D36 (supply-chain provenance), which measures that posture as `pinned_actions` — one pinning decision is charged once, not once per lens. semgrep hit a parse error in 82 file(s) — `.github/scripts/can-i-deploy-gate.sh` (line 52, line 53), `.github/scripts/check-adr-registry.sh`, `.github/scripts/check-agent-review-happened.sh` (line 54), `.github/scripts/check-fleet-attestations.sh` (line 213), `.github/scripts/classify-verification-sweep-failure.sh` (line 30), … (+77 more) — so no absence of findings in the named regions is evidence of anything; rows reported elsewhere in those files are real. Fix the syntax error (or exclude the file deliberately) and re-scan to cover them. Separately, one or more rules could not re-parse an embedded snippet in 30 file(s) (e.g. a workflow `run:` block read as shell). Those files WERE scanned and their other rows are unaffected; only those rules' view of those snippets is missing.
What to do
- Resolve the 39 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED (17), REDACTED (5), REDACTED (3). — One of this dimension's main actionable groups (39 warning-level).
- Resolve the 32 REDACTED finding(s) charged to Static Analysis (SAST) — the other 1 are reported here at file:line but scored by D36 (supply-chain provenance), which charges them once. — One of this dimension's main actionable groups (33 issue-level, 32 of them charged here).
- Resolve the 3 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED, REDACTED, REDACTED. — One of this dimension's main actionable groups (3 recommendation-level).
D30 · Dependency Vulnerabilities
58 finding(s): 4 critical, 29 high, 23 medium, 2 low. 33 package override(s) declared in osv-scanner.toml.
What to do
- Resolve the 27 REDACTED CVE finding(s) in Dependency Vulnerabilities — start with REDACTED (27). — One of this dimension's main actionable groups (27 issue-level).
- Resolve the 4 Critical CVE finding(s) in Dependency Vulnerabilities — start with REDACTED (4). — One of this dimension's main actionable groups (4 issue-level).
- Resolve the 2 REDACTED vulnerability finding(s) in Dependency Vulnerabilities — start with REDACTED (2). — One of this dimension's main actionable groups (2 issue-level).
D31 · IaC & Container Security
1399 finding(s): 0 critical, 1 high, 1144 medium, 254 low.
What to do
- Resolve the 1144 REDACTED IaC finding(s) in IaC & Container Security — start with REDACTED (318), REDACTED (138), REDACTED (31). — One of this dimension's main actionable groups (1144 warning-level).
- Resolve the 1 REDACTED IaC finding(s) in IaC & Container Security — start with REDACTED. — One of this dimension's main actionable groups (1 issue-level).
- Resolve the 254 REDACTED IaC finding(s) in IaC & Container Security — start with REDACTED (60), REDACTED (25), REDACTED (18). — One of this dimension's main actionable groups (254 recommendation-level).
D34 · Knowledge Freshness
Every significant source file has living knowledge — recently and meaningfully worked. Counted over 1993 of the 3167 production source files in this repository: the rest are under the ~2,400-byte size floor this dimension measures over.
D35 · Change Coupling
Strongest change-coupling: REDACTED↔page.tsx 90%; PartyPort.kt↔PartyRepositoryImpl.kt 64%; StepEditor.tsx↔Campaign.kt 60%
What to do
- Resolve the 3 Change coupling finding(s) in Change Coupling — start with PartyPort.kt, StepEditor.tsx, collect-test-run-evidence.py. — One of this dimension's main actionable groups (3 warning-level).
- Resolve the 1 Change-coupling hub finding(s) in Change Coupling — start with page.tsx. — One of this dimension's main actionable groups (1 warning-level).
D36 · Supply-chain Provenance & Signing
3/4 supply-chain integrity signals present (provenance, signing, SBOM, pinned actions).
What to do
- Resolve the 3 REDACTED finding(s) in Supply-chain Provenance & Signing — start with REDACTED, REDACTED, REDACTED. — One of this dimension's main actionable groups (3 warning-level).
- Resolve the 1 REDACTED finding(s) in Supply-chain Provenance & Signing. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 REDACTED finding(s) in Supply-chain Provenance & Signing. — One of this dimension's main actionable groups (1 warning-level).
D37 · Vulnerability-disclosure Policy
A vulnerability-disclosure policy (SECURITY.md) is published with a reporting contact.
D40 · Network Egress Confinement
Kubernetes workloads are covered by a network policy that restricts egress.
D41 · Kernel & Syscall Confinement
1/2 syscall-confinement controls present (seccomp, AppArmor/SELinux).
What to do
- Resolve the 1 No AppArmor/SELinux confinement finding(s) in Kernel & Syscall Confinement. — One of this dimension's main actionable groups (1 recommendation-level).
D42 · Runtime Threat Enforcement
The deployment wires runtime threat detection and an admission-control policy gate.
D43 · Malicious Dependencies
No dependency in any ecosystem this repository declares is published as malicious. 33 package override(s) declared in osv-scanner.toml.
D44 · Platform End-of-Life
0 end-of-life runtime(s) and 0 end-of-life framework(s), read from 0 platform declaration(s) and 14 dependency declaration(s). This dimension reads what the repository says about ITSELF — a pinned target framework, a version file, a capped requires-python, a framework major a constraint cannot move off. A FLOOR is deliberately never charged: `requires-python = ">=3.8"` states what the package SUPPORTS, not what it runs on, and a well-maintained library declares exactly that while running its own CI on a current release. The end-of-life facts are FROZEN and dated, so this dimension needs no network and answers identically inside a closed scan fence; as the table ages it loses recall and never precision, because a statement about support that ended in the past cannot become false. The OS layer of a container image is D31's question and the toolchain a CI workflow installs is D29's; this row is neither.
Frontend & cross-cutting dimensions
AC1 · Text alternatives
AC2 · Forms & labels
- This control has no associated label. Add a <label htmlFor> / wrapping <label> / aria-label / aria-labelledby so assistive tech can name it. (×2) — openbank-admin-ui/src/app/campaigns/new/page.tsx:1146, openbank-admin-ui/src/app/product-studio/page.tsx:514
- A fieldset groups related controls but has no <legend> to name the group. Add a <legend> as its first child — or, if the group already has a visible caption beside it (or the fieldset is there only to disable its subtree and carries no group box), point aria-labelledby at that caption's id instead, which names the group without rendering a second one. (×2) — openbank-admin-ui/src/app/document-templates/page.tsx:672, openbank-admin-ui/src/app/pid/page.tsx:323
- This <label> has no htmlFor and wraps no form control, so it names nothing: assistive tech never announces it, and clicking the caption focuses no field. Note that an id on the label is not an association — it is the TARGET of one, so a control still has to point at it. Give the label htmlFor="<the control's id>", move the control inside the label, or point the control's aria-labelledby at this label's id. (×3) — openbank-admin-ui/src/app/sanctions/page.tsx:965, openbank-admin-ui/src/app/system/agent/page.tsx:394, openbank-admin-ui/src/components/cards/ConfirmTransitionDialog.tsx:109
What to do
- Give every control a programmatic label (a <label for> / wrapping <label> / aria-label) and every button text — a placeholder is not a label.
AC3 · Page structure
- A <table> with data cells but no <th> gives assistive technology no way to associate cells with their headers. Mark the header row/column cells as <th> (with a scope). — openbank-admin-ui/src/app/docs/compliance/page.tsx:216
- A page with no <title> gives no name in the tab, history or screen-reader page list. Declare it the way this router does — Next App Router `export const metadata = { title }`, React Router/Remix `export function meta`, TanStack Start `head`, or a Helmet/Head component. — openbank-admin-ui/src/app/global-error.tsx:24
- No <main> (or role="main") means no "skip to content" target and a weaker landmark map. Wrap the primary content in <main>. (×4) — openbank-admin-ui/src/app/global-error.tsx:24, openbank-developer-portal/site/api/redoc.standalone.js:2, openbank-infra/web/landing/index.html:2, …
- The page declares no language, so assistive tech can't pick the right pronunciation. Add lang (e.g. lang="en"). — openbank-developer-portal/site/api/redoc.standalone.js:2
- A page with no <title> gives no name in the tab, history or screen-reader page list. Add a descriptive <title> in <head>. — openbank-developer-portal/site/api/redoc.standalone.js:2
- Skipping heading levels breaks the document outline assistive tech relies on. Don't jump levels — increase by at most one. — openbank-infra/web/landing/platform.html:285
What to do
- Declare <html lang>, a document <title> and a <main> landmark, keep headings in order, leave zoom enabled, title iframes and drop meta-refresh.
AC4 · Keyboard semantics
- This <tr> binds mouse-enter and mouse-leave and nothing else — no click, no key handler, no focus handler, and no tabIndex={0}, so it cannot even take focus. A keyboard user cannot hover, so whatever the pair reveals or highlights is reachable with a mouse only. Bind the same state to focus/blur as well, and add tabIndex={0}="0" so the element can receive that focus. (×10) — openbank-admin-ui/src/app/aml/page.tsx:154, openbank-admin-ui/src/app/clearing/page.tsx:124, openbank-admin-ui/src/app/fx/page.tsx:460, …
- A click handler on a plain element isn't keyboard-operable. Use a <button>, or add role + tabIndex={0} + a key handler. (×3) — openbank-admin-ui/src/app/docs/lineage/page.tsx:282, openbank-admin-ui/src/components/flow/StageBoard.tsx:178, openbank-admin-ui/src/components/lending/OriginationPipeline.tsx:197
- This <div> binds mouse-enter and mouse-leave and nothing else — no click, no key handler, no focus handler, and no tabIndex={0}, so it cannot even take focus. A keyboard user cannot hover, so whatever the pair reveals or highlights is reachable with a mouse only. Bind the same state to focus/blur as well, and add tabIndex={0}="0" so the element can receive that focus. (×2) — openbank-admin-ui/src/app/docs/page.tsx:233, openbank-admin-ui/src/app/temporal/page.tsx:842
- A click handler on a plain element with no role and no tabindex: even where another element's key handler can reach it, assistive tech can neither focus it nor announce what it is. Use a <button>, or add role + tabIndex={0}. — openbank-admin-ui/src/components/party/CustomerContextGraph.tsx:174
- The repo's own CSS styles .node with `cursor: pointer`, so this <span> is a control — but it has no role, no tabindex="0" and no key handler, so it can only be reached with a mouse. Use a <button>, or add role + tabindex="0" + a key handler. (×21) — openbank-infra/web/landing/platform.html:237, openbank-infra/web/landing/platform.html:238, openbank-infra/web/landing/platform.html:249, …
What to do
- Make custom controls keyboard-operable (role + tabindex + key handler), drop positive tabindex, and give anchors a real href.
AC5 · ARIA correctness
AC6 · Visual & motion safety
AC7 · A11y enforcement
- Accessibility checks run in tests (Verified) but aren't gated in CI — run the suite that holds those checks in your pipeline so an accessibility regression blocks the merge (Prevented). What was searched, so you can tell an absence from a miss: the 238 markup file(s) this pass actually assessed, the linter configuration checked in beside them, and this repository's test and CI files — matched by name against the accessibility checkers this dimension carries. An audit run outside the repository, a hosted scanner, or a check whose name is not one of those, is not seen here.
What to do
- Enforce accessibility in the toolchain: add eslint-plugin-jsx-a11y, then run the suite that already asserts accessibility in CI so a regression blocks the merge.
AX10 · Code composition
What to do
- The domain core is a small share of production code, but most of the rest matched no layer vocabulary at all — so this is not yet an anemic-domain finding. The namespace/path convention could not place that code, which makes the composition above a statement about the naming, not about the design. Name the layers (or check that the repository's conventions differ from the ones this check knows) before reading a thin domain into it.
M1 · Documentation (README)
What to do
- Add a 'Testing' section to the root README — how to run the test suite.
- Add a README to the 74 of 76 project(s) that lack one — worth up to 1.9 pts.
M2 · Architecture documentation
M3 · Folder & project structure
M4 · Documentation accuracy
P1 · CI/CD gates
P3 · Security & performance tooling
What to do
- Dependabot is configured but does not watch `pip` — add that `package-ecosystem` entry to REDACTED so those dependencies get the same automatic update and advisory pressure as the ones it already covers.
P4 · Deployment & Rollback
P5 · DR & Backup
P6 · Release Hygiene
R1 · Type Safety
What to do
- Migrate the remaining .js/.jsx files to TypeScript.
R10 · Code Duplication
- 194 duplicated blocks under openbank-admin-ui/src/app/ have copies in at least two of the sibling directories accounts, aml, api, approvals, audit, auth (+47 more sibling(s) not listed) — 105 of them are reported below, and 89 are counted here but not reported individually: those copies match on shape but no longer clear R10's bar for an individually reported row — either they kept neither their own names nor their values, or what was copied is too small to stand on its own (it reports near-exact duplication only, and only of substantial extent). What this row states is the concentration, which the detector measured over all 194 and which does not depend on how exactly each block's copies still match. That concentration is one structural fact, not 194 local ones: the siblings replicate behaviour none of them owns, which is the shape of a missing shared module — a common library every sibling imports — rather than 194 separate extractions. Check first whether the siblings are deliberately standalone deliverables (scaffold templates, demo apps that must stay copy-pasteable); where they are, the duplication is the design and the per-block rows are the ones to act on. — openbank-admin-ui/src/app/aml/page.tsx:18
- 68 duplicated blocks under openbank-admin-ui/src/ have copies in at least two of the sibling directories app, components, lib — 27 of them are reported below, and 41 are counted here but not reported individually: those copies match on shape but no longer clear R10's bar for an individually reported row — either they kept neither their own names nor their values, or what was copied is too small to stand on its own (it reports near-exact duplication only, and only of substantial extent). What this row states is the concentration, which the detector measured over all 68 and which does not depend on how exactly each block's copies still match. That concentration is one structural fact, not 68 local ones: the siblings replicate behaviour none of them owns, which is the shape of a missing shared module — a common library every sibling imports — rather than 68 separate extractions. Check first whether the siblings are deliberately standalone deliverables (scaffold templates, demo apps that must stay copy-pasteable); where they are, the duplication is the design and the per-block rows are the ones to act on. — openbank-admin-ui/src/app/kyc/page.tsx:91
- 29 duplicated blocks under openbank-admin-ui/src/app/api/ have copies in at least two of the sibling directories agent, app-status, approvals, audiences, campaigns, catalog (+26 more sibling(s) not listed) — 22 of them are reported below, and 7 are counted here but not reported individually: those copies match on shape but no longer clear R10's bar for an individually reported row — either they kept neither their own names nor their values, or what was copied is too small to stand on its own (it reports near-exact duplication only, and only of substantial extent). What this row states is the concentration, which the detector measured over all 29 and which does not depend on how exactly each block's copies still match. That concentration is one structural fact, not 29 local ones: the siblings replicate behaviour none of them owns, which is the shape of a missing shared module — a common library every sibling imports — rather than 29 separate extractions. Check first whether the siblings are deliberately standalone deliverables (scaffold templates, demo apps that must stay copy-pasteable); where they are, the duplication is the design and the per-block rows are the ones to act on. — openbank-admin-ui/src/app/api/finops/right-sizing/route.ts:61
- 25 duplicated blocks under openbank-admin-ui/src/lib/ have copies in at least two of the sibling directories accounts, aml, approvals, audit, cards, clearing (+22 more sibling(s) not listed) — 16 of them are reported below, and 9 are counted here but not reported individually: those copies match on shape but no longer clear R10's bar for an individually reported row — either they kept neither their own names nor their values, or what was copied is too small to stand on its own (it reports near-exact duplication only, and only of substantial extent). What this row states is the concentration, which the detector measured over all 25 and which does not depend on how exactly each block's copies still match. That concentration is one structural fact, not 25 local ones: the siblings replicate behaviour none of them owns, which is the shape of a missing shared module — a common library every sibling imports — rather than 25 separate extractions. Check first whether the siblings are deliberately standalone deliverables (scaffold templates, demo apps that must stay copy-pasteable); where they are, the duplication is the design and the per-block rows are the ones to act on. — openbank-admin-ui/src/lib/closings/upstream.ts:64
- 11 duplicated blocks under openbank-admin-ui/src/components/ have copies in at least two of the sibling directories cards, closings, delegations, devops, finops, flow (+8 more sibling(s) not listed) — 5 of them are reported below, and 6 are counted here but not reported individually: those copies match on shape but no longer clear R10's bar for an individually reported row — either they kept neither their own names nor their values, or what was copied is too small to stand on its own (it reports near-exact duplication only, and only of substantial extent). What this row states is the concentration, which the detector measured over all 11 and which does not depend on how exactly each block's copies still match. That concentration is one structural fact, not 11 local ones: the siblings replicate behaviour none of them owns, which is the shape of a missing shared module — a common library every sibling imports — rather than 11 separate extractions. Check first whether the siblings are deliberately standalone deliverables (scaffold templates, demo apps that must stay copy-pasteable); where they are, the duplication is the design and the per-block rows are the ones to act on. — openbank-admin-ui/src/components/flow/StageBoard.tsx:84
- openbank-admin-ui/src/app/kyc/page.tsx:91 · openbank-admin-ui/src/components/notifications/NotificationsPage.tsx:147 — the two spans are one implementation copied and then locally edited — 1150 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/kyc/page.tsx:91
- openbank-admin-ui/src/components/flow/StageBoard.tsx:84 · openbank-admin-ui/src/components/lending/OriginationPipeline.tsx:80 — the two spans are one implementation copied and then locally edited — 1103 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/components/flow/StageBoard.tsx:84
- openbank-admin-ui/src/app/aml/page.tsx:18 · openbank-admin-ui/src/app/clearing/page.tsx:19 — the two spans are one implementation copied and then locally edited — 1088 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/aml/page.tsx:18
- openbank-admin-ui/src/app/consents/page.tsx:206 · openbank-admin-ui/src/app/customer-360/page.tsx:142 — the two spans are one implementation copied and then locally edited — 522 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/consents/page.tsx:206
- openbank-admin-ui/src/app/devops/page.tsx:169 · openbank-admin-ui/src/app/finops/page.tsx:335 — the two spans are one implementation copied and then locally edited — 534 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/devops/page.tsx:169
- openbank-admin-ui/src/app/iaops/cases/[caseId]/page.tsx:64 · openbank-admin-ui/src/app/iaops/cases/page.tsx:33 — the two spans are one implementation copied and then locally edited — 535 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/iaops/cases/[caseId]/page.tsx:64
- openbank-admin-ui/src/app/api/finops/right-sizing/route.ts:61 · openbank-admin-ui/src/app/api/temporal/status/route.ts:45 — the two spans are one implementation copied and then locally edited — 380 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/api/finops/right-sizing/route.ts:61
- openbank-admin-ui/src/app/onboarding/page.tsx:278 · openbank-admin-ui/src/app/parties/page.tsx:263 — the two spans are one implementation copied and then locally edited — 445 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/onboarding/page.tsx:278
- openbank-admin-ui/src/app/api/domestic-payments/route.ts:43 · openbank-admin-ui/src/app/api/sepa-payments/route.ts:44 — the 2 copies are spread across 2 directories, so the shared home is a decision rather than an obvious spot: check first whether one of them already owns this behaviour, and otherwise put the extracted module somewhere all of the sites already reach rather than making one of them depend on another. — openbank-admin-ui/src/app/api/domestic-payments/route.ts:43
- openbank-admin-ui/src/app/api/catalog/governance/route.ts:33 · openbank-admin-ui/src/app/api/catalog/services/route.ts:29 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/api/catalog/governance/route.ts:33
- openbank-admin-ui/src/app/iaops/agents/[agentId]/page.tsx:147 · openbank-admin-ui/src/app/iaops/cases/[caseId]/page.tsx:94 — the two spans are one implementation copied and then locally edited — 299 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/iaops/agents/[agentId]/page.tsx:147
- openbank-admin-ui/scripts/generate-infra-lifecycle.mjs:21 · REDACTED:21 — the 2 copies sit in sibling files in one directory, so check first whether one of them (or an existing module there) already owns this behaviour and the others should call it; otherwise extract it into one module in that directory and have each site call it. — openbank-admin-ui/scripts/generate-infra-lifecycle.mjs:21
- openbank-admin-ui/src/app/api/prometheus/[...path]/route.ts:14 · openbank-admin-ui/src/app/api/tempo/[...path]/route.ts:23 — the 2 copies are spread across 2 directories, so the shared home is a decision rather than an obvious spot: check first whether one of them already owns this behaviour, and otherwise put the extracted module somewhere all of the sites already reach rather than making one of them depend on another. — openbank-admin-ui/src/app/api/prometheus/[...path]/route.ts:14
- openbank-admin-ui/src/app/api/context/complaints/[reference]/route.ts:9 · openbank-admin-ui/src/app/api/context/incidents/[reference]/impact/route.ts:9 — the 2 copies are spread across 2 directories, so the shared home is a decision rather than an obvious spot: check first whether one of them already owns this behaviour, and otherwise put the extracted module somewhere all of the sites already reach rather than making one of them depend on another. — openbank-admin-ui/src/app/api/context/complaints/[reference]/route.ts:9
- openbank-admin-ui/src/app/business-onboarding/page.tsx:455 · openbank-admin-ui/src/app/identity-cases/page.tsx:343 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/business-onboarding/page.tsx:455
- openbank-admin-ui/src/app/communication/[personaId]/page.tsx:83 · openbank-admin-ui/src/app/communication/page.tsx:93 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/communication/[personaId]/page.tsx:83
- openbank-admin-ui/src/app/api/campaigns/[id]/journey/route.ts:23 · openbank-admin-ui/src/app/api/segments/route.ts:22 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/api/campaigns/[id]/journey/route.ts:23
- openbank-admin-ui/src/app/payments/page.tsx:506 · openbank-admin-ui/src/app/payments/page.tsx:570 — both copies are in the same file, so extract the block into one function there and call it from each site — the copies drift apart the first time only one of them is edited. — openbank-admin-ui/src/app/payments/page.tsx:506
- openbank-admin-ui/src/components/campaigns/StepEditor.tsx:167 · openbank-admin-ui/src/components/campaigns/StepEditor.tsx:220 — all 2 copies are in the same file, and the SHAPE of this repetition could not be determined. It is not a run of declarations, a listing, a declaration header, a type body or a slice through a construct — and it was not measured as a run of executable statements either, so this row cannot tell you whether a function can stand where these lines are. Read the two spans before acting, because the move is opposite in the two cases. Where they are statements, the ordinary answer holds: give the shared part one home and call it from each site. Where they turn out to be declarations, a literal's entries, or the cases of an enumeration, there is no call site to call anything from, and collapsing them would delete what each copy pins — a shared base type, a generated set, or one exported constant each site refers to is the move instead, and sometimes the honest answer is that there is nothing to extract at all. Reported because the copies drift apart the first time only one of them is edited, which is true whichever of those they are. — openbank-admin-ui/src/components/campaigns/StepEditor.tsx:167
- openbank-admin-ui/src/lib/api.ts:215 · openbank-admin-ui/src/lib/api.ts:278 — all 2 copies are in the same file, and the SHAPE of this repetition could not be determined. It is not a run of declarations, a listing, a declaration header, a type body or a slice through a construct — and it was not measured as a run of executable statements either, so this row cannot tell you whether a function can stand where these lines are. Read the two spans before acting, because the move is opposite in the two cases. Where they are statements, the ordinary answer holds: give the shared part one home and call it from each site. Where they turn out to be declarations, a literal's entries, or the cases of an enumeration, there is no call site to call anything from, and collapsing them would delete what each copy pins — a shared base type, a generated set, or one exported constant each site refers to is the move instead, and sometimes the honest answer is that there is nothing to extract at all. Reported because the copies drift apart the first time only one of them is edited, which is true whichever of those they are. — openbank-admin-ui/src/lib/api.ts:215
- openbank-admin-ui/src/app/payments/page.tsx:848 · openbank-admin-ui/src/app/payments/page.tsx:985 — all 2 copies are in the same file, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/payments/page.tsx:848
- openbank-admin-ui/src/app/api/agent/proposals/route.ts:13 · openbank-admin-ui/src/app/api/iaops/agents/[agentId]/route.ts:23 — the two spans are one implementation copied and then locally edited — 166 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — openbank-admin-ui/src/app/api/agent/proposals/route.ts:13
- openbank-admin-ui/src/app/api/campaigns/route.ts:110 · openbank-admin-ui/src/app/api/segments/route.ts:14 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/api/campaigns/route.ts:110
- openbank-admin-ui/src/app/api/feedback/screen-feedback/route.ts:82 · openbank-admin-ui/src/app/api/onboarding/funnel-analytics/route.ts:31 — the 2 copies are spread across 2 directories, so the shared home is a decision rather than an obvious spot: check first whether one of them already owns this behaviour, and otherwise put the extracted module somewhere all of the sites already reach rather than making one of them depend on another. — openbank-admin-ui/src/app/api/feedback/screen-feedback/route.ts:82
- openbank-admin-ui/src/app/clearing/page.tsx:66 · openbank-admin-ui/src/app/swift/page.tsx:70 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/clearing/page.tsx:66
- openbank-admin-ui/src/app/payments/[id]/page.tsx:215 · openbank-admin-ui/src/app/swift/[id]/page.tsx:166 — the 2 copies are spread across 2 files, and the SHAPE of this repetition could not be determined. It is not a run of declarations, a listing, a declaration header, a type body or a slice through a construct — and it was not measured as a run of executable statements either, so this row cannot tell you whether a function can stand where these lines are. Read the two spans before acting, because the move is opposite in the two cases. Where they are statements, the ordinary answer holds: give the shared part one home and call it from each site. Where they turn out to be declarations, a literal's entries, or the cases of an enumeration, there is no call site to call anything from, and collapsing them would delete what each copy pins — a shared base type, a generated set, or one exported constant each site refers to is the move instead, and sometimes the honest answer is that there is nothing to extract at all. Reported because the copies drift apart the first time only one of them is edited, which is true whichever of those they are. — openbank-admin-ui/src/app/payments/[id]/page.tsx:215
- openbank-admin-ui/src/components/party/DevicesPanel.tsx:28 · openbank-admin-ui/src/components/party/DocumentsPanel.tsx:13 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/components/party/DevicesPanel.tsx:28
- openbank-admin-ui/src/app/api/campaigns/[id]/route.ts:80 · openbank-admin-ui/src/app/api/campaigns/[id]/route.ts:100 · openbank-admin-ui/src/app/api/campaigns/[id]/route.ts:120 — all 3 copies are in the same file, and the SHAPE of this repetition could not be determined. It is not a run of declarations, a listing, a declaration header, a type body or a slice through a construct — and it was not measured as a run of executable statements either, so this row cannot tell you whether a function can stand where these lines are. Read the two spans before acting, because the move is opposite in the two cases. Where they are statements, the ordinary answer holds: give the shared part one home and call it from each site. Where they turn out to be declarations, a literal's entries, or the cases of an enumeration, there is no call site to call anything from, and collapsing them would delete what each copy pins — a shared base type, a generated set, or one exported constant each site refers to is the move instead, and sometimes the honest answer is that there is nothing to extract at all. Reported because the copies drift apart the first time only one of them is edited, which is true whichever of those they are. — openbank-admin-ui/src/app/api/campaigns/[id]/route.ts:80
- openbank-admin-ui/src/app/api/delegations/[id]/audit/route.ts:21 · openbank-admin-ui/src/app/api/delegations/approvals/[id]/route.ts:16 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/api/delegations/[id]/audit/route.ts:21
- openbank-admin-ui/src/app/document-templates/page.tsx:66 · openbank-admin-ui/src/app/product-catalog/page.tsx:60 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/document-templates/page.tsx:66
- openbank-admin-ui/src/app/finops/page.tsx:326 · openbank-admin-ui/src/app/iaops/page.tsx:94 — the 2 copies are spread across 2 files, and the SHAPE of this repetition could not be determined. It is not a run of declarations, a listing, a declaration header, a type body or a slice through a construct — and it was not measured as a run of executable statements either, so this row cannot tell you whether a function can stand where these lines are. Read the two spans before acting, because the move is opposite in the two cases. Where they are statements, the ordinary answer holds: give the shared part one home and call it from each site. Where they turn out to be declarations, a literal's entries, or the cases of an enumeration, there is no call site to call anything from, and collapsing them would delete what each copy pins — a shared base type, a generated set, or one exported constant each site refers to is the move instead, and sometimes the honest answer is that there is nothing to extract at all. Reported because the copies drift apart the first time only one of them is edited, which is true whichever of those they are. — openbank-admin-ui/src/app/finops/page.tsx:326
- openbank-admin-ui/src/lib/cards/money.ts:76 · openbank-admin-ui/src/lib/cards/money.ts:102 — both copies are in the same file, so extract the block into one function there and call it from each site — the copies drift apart the first time only one of them is edited. — openbank-admin-ui/src/lib/cards/money.ts:76
- openbank-admin-ui/src/app/aml/page.tsx:70 · openbank-admin-ui/src/app/clearing/page.tsx:66 · openbank-admin-ui/src/app/swift/page.tsx:70 — the 3 copies are spread across 3 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/aml/page.tsx:70
- openbank-admin-ui/src/app/api/agent/mcp/route.ts:50 · openbank-admin-ui/src/app/api/agent/obo-mcp/route.ts:137 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/api/agent/mcp/route.ts:50
- openbank-admin-ui/src/app/api/campaigns/[id]/actions/route.ts:42 · openbank-admin-ui/src/app/api/campaigns/route.ts:89 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — openbank-admin-ui/src/app/api/campaigns/[id]/actions/route.ts:42
What to do
- Act on each finding's own remediation rather than one rule: the move depends on what recurs. Where the copies are executable blocks, give the shared part one home and call it from each site; where they are declarations, a listing, a specialisation already delegating to its base, or one shape repeated per entity, there is no call site and the move is a shared type, a generated set or a factory — sometimes there is nothing to extract.
R2 · Cyclomatic Complexity
- CampaignDetailPage has cyclomatic complexity 98 and cognitive complexity 83; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/campaigns/[id]/page.tsx:159
- CreditRiskConsole has cyclomatic complexity 83 and cognitive complexity 74; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/lending/risk/page.tsx:110
- PaymentsContent has cyclomatic complexity 75 and cognitive complexity 72; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/payments/page.tsx:297
- ProductDetailPanel has cyclomatic complexity 69 and cognitive complexity 61; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/product-catalog/page.tsx:116
- ProductStudioPage has cyclomatic complexity 58 and cognitive complexity 50; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/product-studio/page.tsx:92
- NewCampaignPage has cyclomatic complexity 58 and cognitive complexity 46; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/campaigns/new/page.tsx:134
- SanctionsPage has cyclomatic complexity 57 and cognitive complexity 49; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/sanctions/page.tsx:205
- TemporalPage has cyclomatic complexity 49 and cognitive complexity 47; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/temporal/page.tsx:258
- buildCustomerGraph has cyclomatic complexity 47 and cognitive complexity 63; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/lib/context/customerGraph.ts:271
- (anonymous) has cyclomatic complexity 47 and cognitive complexity 46; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/api/infra/lifecycle/route.ts:157
- CampaignsPage has cyclomatic complexity 47 and cognitive complexity 41; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/campaigns/page.tsx:104
- (anonymous) has cyclomatic complexity 46 and cognitive complexity 44; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/scripts/collect-test-intelligence.mjs:729
- IssueCardDialog has cyclomatic complexity 46 and cognitive complexity 43; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/components/cards/IssueCardDialog.tsx:107
- LoyaltyPage has cyclomatic complexity 46 and cognitive complexity 36; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/loyalty/page.tsx:40
- FinOpsContent has cyclomatic complexity 44 and cognitive complexity 43; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/finops/page.tsx:347
- DocumentTemplatesPage has cyclomatic complexity 44 and cognitive complexity 39; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/document-templates/page.tsx:193
- ServiceCard has cyclomatic complexity 42 and cognitive complexity 40; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/system/health/page.tsx:156
- LifecycleStrip has cyclomatic complexity 42 and cognitive complexity 38; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/components/infra/LifecycleStrip.tsx:60
- deriveCapabilityStatus has cyclomatic complexity 42 and cognitive complexity 36; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/scripts/generate-app-status.mjs:161
- CardDetailPage has cyclomatic complexity 39 and cognitive complexity 36; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — openbank-admin-ui/src/app/cards/[id]/page.tsx:76
What to do
- Break down the listed branch-heavy functions; aim P95 cyclomatic ≤ 5.
R3 · Large Files
- 46 file(s) over 400 lines (counted as significant lines — blank lines excluded — over production source only, tests excluded), largest first: openbank-admin-ui/src/app/finops/page.tsx (1226), openbank-admin-ui/src/app/payments/page.tsx (1205), openbank-admin-ui/src/app/campaigns/new/page.tsx (1167), openbank-admin-ui/src/app/sanctions/page.tsx (1138), openbank-admin-ui/src/app/docs/service-map/page.tsx (1099), openbank-admin-ui/src/app/campaigns/[id]/page.tsx (1074) (+40 more).
What to do
- Split each oversized file along the responsibilities already in it, into smaller focused modules in the same package.
R4 · Test Coverage
- No test imports this module directly or transitively. Import reachability cannot see a test that executes a file by path instead of importing it, nor one that drives it through a running browser by navigating to a URL — if neither does, no test reaches this one. (×18) — openbank-admin-ui/scripts/generate-service-graph.mjs, openbank-admin-ui/scripts/collect-gate-health.mjs, openbank-admin-ui/scripts/generate-security-graph.mjs, …
What to do
- 17 of the 18 unreached modules are entry points — the browser loads some as a top-level script and something else executes the rest by path, so no import specifier addresses them and no public entry publishes them. An importing test cannot be written for them as they ship, and two different things are worth doing here — they are not the same thing. To EXERCISE them, a browser or end-to-end suite that loads the page, or a test that runs the script the way its caller does; that is worth having, but it will NOT clear this row, because reachability is measured over the import graph and running a file by path adds no edge to it. To CLEAR it, make them addressable: lift the logic into a module a test can import and leave the entry points thin shims over it, or put their bodies behind a main-guard (`import.meta.url === argv[1]`) and export them. Add a test that imports the remaining one — directly or through its public entry.
R5 · Dependency Freshness
What to do
- Bump outdated dependencies to current versions to limit upgrade debt.
R6 · Tooling
R7 · Dead Code
- Unreachable from the 321 application, 13 tooling and 1256 test entry point(s) detected in this repo. Gate removals on `npm run build` — an undetected custom entry would make these reachable.
- no import path from any entry point (321 application, 13 tooling, 1256 test roots considered), and no other file in the scanned tree imports it — nothing in-repo names this module at all, which is the strongest form of this claim the import graph can make — openbank-admin-ui/scripts/admin-login-synthetic.mjs
R8 · Dependency Hygiene
- Declared in openbank-admin-ui/package.json but never imported anywhere in that package or its workspace members — no static import reaches it. Usually that is dead weight and attack surface, but two shapes are indistinguishable from source and are NOT dead: an optional or native peer that another dependency loads dynamically at runtime, and a package a build, docs or test step installs and invokes separately. Confirm which of the three this is before removing it. (×3)
- Only test files import it — move it to devDependencies. — openbank-admin-ui/src/test/mermaid-labels.test.ts:6
What to do
- Remove unused dependencies, declare unlisted imports explicitly, and demote type-/test-only packages to devDependencies.
R9 · Circular Imports
WCAG coverage — what static analysis assessed
| Dimension | WCAG 2.2 A/AA criteria | Coverage |
|---|---|---|
| AC1 · Text alternatives | 1.1.1, 1.2.2, 1.2.5 | Partial signal |
| AC2 · Forms & labels | 1.3.1, 3.3.2, 4.1.2 | Partial signal |
| AC3 · Page structure | 1.4.4, 2.2.1, 2.4.1, 2.4.2, 3.1.1, 4.1.2 | Partial signal |
| AC4 · Keyboard semantics | 2.1.1, 2.4.3 | Partial signal |
| AC5 · ARIA correctness | 4.1.2 | Partial signal |
| AC6 · Visual & motion safety | 1.4.3, 2.4.7 | Partial — literal CSS only |
| AC7 · A11y enforcement | enforcement — no page criterion | Enforcement posture (process) |
Reference — by lens
| Lens | Score | Rating | Impact |
|---|---|---|---|
| Code Health | 77% | Strong | Solid. |
| Architecture | 81% | Strong | Solid. |
| Maturity | 89% | Exemplary | Strongest area. |
| Readiness | 67% | Adequate — gated by R8 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Security | 68% | Adequate — gated by D29, D30, D31 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Accessibility | 65% | Adequate — gated by AC4 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
Not evidenced — 3 control(s) we could not find positive evidence for
- C3 Audit Trail — Not assessed: these audit controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks audit controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- C4 Data Retention — Not assessed: these retention controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks retention controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- C5 Data-Subject Rights — Not assessed: these data-subject rights controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks data-subject rights controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
Not included — 81 check(s) not relevant to this codebase
- AX1 Captive dependencies — no DI registrations detected
- AX2 Stateful singletons — no singleton implementations detected
- AX3 Project dependency cycles — not assessed — project cycles and dependency direction are computed over a project-reference graph that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX4 Dependency direction — not assessed — project cycles and dependency direction are computed over a project-reference graph that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX5 Architecture & structure — not assessed — architecture style/structure is computed from a project graph (projects, types, module namespaces) that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX6 Interface segregation — not assessed — interface segregation is computed over a type surface that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX7 Slice cohesion — not applicable — not a vertical-slice architecture
- AX8 Test isolation — not assessed — test isolation is computed from a project graph (which projects are test projects, and what they reference) that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX9 CQS / query purity — no CQRS query handlers detected — query purity is not applicable to this codebase
- AXB2 Runtime readiness — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- C1 Data Protection — Not assessed: these personal data controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks personal data controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- C2 Access Controls — Not assessed: these authorization controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks authorization controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- D1 Cyclomatic Complexity — cyclomatic complexity not measured — .kt exposed no member bodies
- D11 Test Reliability — Test reliability not included
- D12 Dependency Hygiene — Dependency Hygiene incomplete (time budget)
- D14 License Compliance — License Compliance not included (check did not complete)
- D15 Churn × Complexity Hotspots — complexity unreadable for .kt, .py — churn × complexity hotspots could not be measured
- D16 Bus Factor — single-contributor repository — bus factor is not applicable
- D18 Solution Shape — D18 scores the shape of a .NET solution; this repository has no .NET solution or project files, so the dimension does not apply.
- D2 Cognitive Complexity — cognitive complexity not measured — .kt exposed no member bodies
- D22 Internal API Consistency — The exposed public-API surface could not be collected — no C#/VB projects loaded.
- D23 Boundary Type-Coupling — Production source is present (.kt, .py, .ts, .tsx) but bounded contexts are resolved over the C#/VB project set, which exposed none, so context scope could not be assessed. Not scored — this is a gap in the analyzer, not a verdict about this repository. Declaring the codebase's bounded contexts (≥2) would let cross-boundary type coupling be assessed — see the recommendation on this dimension for where. Declare them in `.codehealth/config.yaml` at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it — e.g. `architecture:` → `contexts:` → `Billing: ["src/billing", "Acme.Billing"]`, `Catalog: ["src/catalog", "Acme.Catalog"]`.
- D24 Comment Value — No inline comments to assess — comment value is not applicable here.
- D26 Project Cohesion — Project cohesion is assessed over the .NET project set; this target exposed no projects, so project size and spread could not be assessed. Not scored — this is a gap in the analyzer's reach, not a verdict about this repository.
- D27 Navigability — Most of this repository's production source (.kt, .py) was not read by navigability analysis, so tracing effort was not assessed — whatever else resolved (another language's projects) is not this repository's navigability. Not scored — this is a gap in the analyzer's reach, not a verdict about this repository.
- D3 God Classes — Most of this repository's production source (.kt) was not read by god-class detection, so class size was not assessed for the languages that are the product — whatever else this pass did read is not this repository's class size. Not scored — this is a gap in the analyzer, not a verdict about this repository.
- D32 Data Compliance (PII/GDPR) — 17 file(s) were not parsed by semgrep — the PII/GDPR ruleset never ran over them
- D39 IL Efficiency — D39 measures the IL emitted by a .NET build; this repository has no .NET solution or project files, so the dimension does not apply.
- D4 Code Duplication — Duplication not measured — .kt, .py not exposed to the token comparison
- D5 Coupling — Inter-project coupling could not be assessed — no analyzable project graph was found for this repository. Not scored: a gap in the analyzer's reach, not a verdict about this repository. (Coupling here is Martin afferent/efferent/instability plus reference cycles across a project-reference graph, read today from .NET project files; other ecosystems' module graphs are not read yet.)
- D6 Cohesion (LCOM4) — Cohesion (LCOM4) is measured over a CS/VB/GO/SCALA/SWIFT/DART class graph, and this repository's production source is .kt, .py, .ts, .tsx, which this pass does not read — so no class could be assessed. Not scored — this is a gap in the analyzer, not a finding about this repository.
- D8 Code Coverage — Coverage NOT MEASURED: the JavaScript/TypeScript half could not be measured — the vitest suite in openbank-admin-ui/ ran and passed but wrote no lcov report. Coverage is excluded from the score rather than counted as a near-zero. The named suite step is one the repository's maintainers can perform; once it passes, the real number is measured on the next scan. Alternatively, commit the lcov/Cobertura report your CI produces and it is read without a re-run.
- DM1 Aggregate boundaries — not scored for TypeScript: a class holding another class reads the same whether the inner type is an aggregate or a value object, so this cannot be decided from source without guessing — reported as guidance rather than measured
- DM2 Strongly-typed ids — not scored for TypeScript: branded ids (`type Id = string & { __brand }`) are an uncommon idiom, so a bare-string id is not on its own evidence of a missing typed id — reported as guidance rather than measured
- DM3 Integration-event coupling — not scored for TypeScript: a domain type used across packages is indistinguishable in source from a deliberate shared-kernel package, so this is reported as guidance rather than measured
- DM4 Rich vs anemic model — not scored for TypeScript: telling a rich domain entity from an anemic data holder needs the behaviour a source-only read cannot always attribute (components, DTOs and readonly value objects are all legitimately data-shaped), so this is reported as guidance rather than measured
- DM5 Encapsulated state — not scored for TypeScript: the language already steers state behind #private/private/readonly, so a mutable public field is rare enough that we report this as guidance rather than measuring it
- DM6 Domain ↔ infrastructure boundary — this TypeScript/JavaScript repository maps no type to a persistence framework (TypeORM/Prisma/MikroORM/Sequelize/Mongoose), so DM6's domain↔infrastructure fusion read has no population to be taken over
- DM7 Repository granularity — not scored for TypeScript: deciding whether a repository belongs to an aggregate root needs the aggregate structure, which source alone does not state — reported as guidance rather than measured
- ED1 Event-Driven — not scored — this repository shows none of the 3 signals this lens looks for
- ED5 Idempotency — This check finds retry-prone mutations (command handlers and message/event consumers) by walking the repository's declared types, and none was loaded here, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- ES1 Event Sourcing — not scored — this repository shows none of the 3 signals this lens looks for
- GD1 Unfinished & placeholder code — no source files were read — this check reads C# syntax, and none was loaded for this repository. That is a limit of the analyzer, not a finding about your code.
- IC1 Incompleteness & stubs — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- P12 CI test-gate honesty — Reported, not scored — this card publishes what the CI gate does with the test inventory rather than grading it. The findings above are its output.
- P2 Observability — Observability was not assessed: this check reads a source model that does not carry this repository's product — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of a logging idiom this check recognises is NOT evidence that this repo lacks structured logging (it may log through its own ecosystem's logger). This is a gap in the analyzer, not a finding about this repository.
- P7 Outbound HTTP resilience — not measured — the application kind could not be determined for this repo
- P8 Schema migrations — not assessed — schema-migration practice is read from a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- P9 Domain vs controller coverage — no coverage report found on disk — produce a coverage report in a standard format (JaCoCo XML — the Gradle `jacocoTestReport` task) and commit it — a hosted scan measures a clone of the repository, so a report that exists only in a working tree, a CI runner's or your own, never reaches it; the artefact is commonly gitignored, so `git add -f` that one file (or un-ignore its path) and commit it alongside the code it measures, or wire coverage collection into CI, to enable this cross-layer check
- PF1 Benchmark discipline — Performance was not assessed: this lens reads a source model that was not loaded for this repository, because the repository is written in a language this lens does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository — in particular it is NOT a statement that this repo is unpackaged or performance-careless.
- PF2 Allocation hygiene — Performance was not assessed: this lens reads a source model that was not loaded for this repository, because the repository is written in a language this lens does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository — in particular it is NOT a statement that this repo is unpackaged or performance-careless.
- PF3 Async & latency hygiene — Performance was not assessed: this lens reads a source model that was not loaded for this repository, because the repository is written in a language this lens does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository — in particular it is NOT a statement that this repo is unpackaged or performance-careless.
- R11 Import Boundaries — No recognizable feature-sliced/layered src layout — boundary rules not applicable.
- S1 Web-Security Posture — Not assessed: these web-security controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks web-security controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- SC1 Supply-chain hygiene — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X1 Async correctness — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X12 Unreachable branch — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X13 Undrained process stream — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X14 Bypassable address classification — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X15 Unvalidated length from an untrusted reader — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X16 Unfloored truncation loop — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X17 Uncapped recursion over a caller-supplied document — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X18 Disposal-pattern correctness — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X19 Unrestored process-global state — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X2 Cancellation propagation — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X20 Mistyped argument guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X21 Side-effecting pattern guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X22 Contradicted release guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X23 Unguarded diagnostic materialisation — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X24 Document value interpolated into markup unescaped — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X25 Inert configuration knob — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X26 Unsynchronised callback handoff — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X27 Collection changed while being enumerated — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X28 Index access outside its own emptiness guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X29 Per-element action decided by a fixed element — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X3 Exception handling — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X30 Support guard that admits what it rejects — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X32 Type resolved by simple name across every loaded assembly — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X4 Structured logging — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X5 Nullable reference types — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X9 Subsumed condition operand — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
Appendix A — Findings (grouped)
Critical — 118 finding(s)
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:237
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:238
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:249
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:250
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:261
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:262
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:320
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:321
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:328
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:329
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:330
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:331
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:339
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:343
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:344
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:348
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:349
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:350
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:354
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:355
- Clickable element isn't keyboard-operable openbank-infra/web/landing/platform.html:356
- Hover-only interaction on <tr> openbank-admin-ui/src/app/aml/page.tsx:154
- Hover-only interaction on <tr> openbank-admin-ui/src/app/clearing/page.tsx:124
- Hover-only interaction on <tr> openbank-admin-ui/src/app/fx/page.tsx:460
- Hover-only interaction on <tr> openbank-admin-ui/src/app/fx/page.tsx:508
- Hover-only interaction on <tr> openbank-admin-ui/src/app/fx/page.tsx:587
- Hover-only interaction on <tr> openbank-admin-ui/src/app/fx/page.tsx:827
- Hover-only interaction on <tr> openbank-admin-ui/src/app/interest/page.tsx:220
- Hover-only interaction on <tr> openbank-admin-ui/src/app/payments/page.tsx:771
- Hover-only interaction on <tr> openbank-admin-ui/src/app/sanctions/page.tsx:729
- Hover-only interaction on <tr> openbank-admin-ui/src/app/standing-orders/page.tsx:130
- ADR not followed: 9. Postgres per service (database-per-service) docs/adr/0009-postgres-per-service.md
- ADR not followed: 21. SCA push/biometric: decoupled device approval, never auto-approve docs/adr/0021-sca-decoupled-device-approval-no-auto-approve.md
- ADR not followed: Governance manifest as derived data (per-service governance.yaml) docs/adr/0071-governance-manifest-as-derived-data.md
- ADR not followed: 104. Production-faithful payment rails: real ISO 20022 messages + scheme simulator docs/adr/0104-production-faithful-payment-rails-iso-20022-and-scheme-simulator.md
- ADR not followed: ADR-0205 — Marketing consent forwarder shape and SCA-exempt activation docs/adr/0205-marketing-consent-forwarder-and-sca-exempt-activation.md
- Click handler on a non-interactive <g> openbank-admin-ui/src/app/docs/lineage/page.tsx:282
- Click handler on a non-interactive <g> openbank-admin-ui/src/components/flow/StageBoard.tsx:178
- Click handler on a non-interactive <g> openbank-admin-ui/src/components/lending/OriginationPipeline.tsx:197
- Click handler on a non-interactive <g> openbank-admin-ui/src/components/party/CustomerContextGraph.tsx:174
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- <label> that labels no control openbank-admin-ui/src/app/sanctions/page.tsx:965
- <label> that labels no control openbank-admin-ui/src/app/system/agent/page.tsx:394
- <label> that labels no control openbank-admin-ui/src/components/cards/ConfirmTransitionDialog.tsx:109
- Document without a <title> openbank-admin-ui/src/app/global-error.tsx:24
- Document without a <title> openbank-developer-portal/site/api/redoc.standalone.js:2
- Hover-only interaction on <div> openbank-admin-ui/src/app/docs/page.tsx:233
- Hover-only interaction on <div> openbank-admin-ui/src/app/temporal/page.tsx:842
- REDACTED
- REDACTED
- <input> without a programmatic label openbank-admin-ui/src/app/campaigns/new/page.tsx:1146
- <textarea> without a programmatic label openbank-admin-ui/src/app/product-studio/page.tsx:514
- <html> without a lang openbank-developer-portal/site/api/redoc.standalone.js:2
- REDACTED
- REDACTED
Serious — 1320 finding(s)
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- No test reaches this file openbank-admin-ui/scripts/generate-service-graph.mjs
- No test reaches this file openbank-admin-ui/scripts/collect-gate-health.mjs
- No test reaches this file openbank-admin-ui/scripts/generate-security-graph.mjs
- No test reaches this file openbank-admin-ui/scripts/generate-events.mjs
- No test reaches this file openbank-admin-ui/scripts/generate-infra-lifecycle.mjs
- No test reaches this file openbank-admin-ui/scripts/generate-catalog.mjs
- No test reaches this file openbank-infra/web/landing/main.js
- No test reaches this file openbank-admin-ui/scripts/admin-login-synthetic.mjs
- No test reaches this file openbank-admin-ui/scripts/generate-origination-graph.mjs
- No test reaches this file REDACTED
- No test reaches this file openbank-admin-ui/scripts/collect-dora.mjs
- No test reaches this file openbank-admin-ui/scripts/collect-aws-costs.mjs
- No test reaches this file openbank-admin-ui/scripts/generate-product-catalog-v2-types.mjs
- No test reaches this file openbank-admin-ui/scripts/collect-cost-footprints.mjs
- No test reaches this file openbank-admin-ui/scripts/collect-test-results.mjs
- No test reaches this file openbank-admin-ui/otel-bootstrap.cjs
- No test reaches this file openbank-infra/web/landing/platform.js
- No test reaches this file openbank-admin-ui/scripts/list-test-services.mjs
- TodoComment openbank-control-liveness-sentinel/src/main/kotlin/com/openbank/liveness/infrastructure/adapter/GovernanceReadAdapter.kt:29
- TodoComment openbank-control-liveness-sentinel/src/main/kotlin/com/openbank/liveness/infrastructure/adapter/GovernanceReadAdapter.kt:35
- TodoComment openbank-document-service/src/main/kotlin/com/openbank/document/application/usecase/DocumentTemplateService.kt:46
- TodoComment openbank-document-service/src/main/kotlin/com/openbank/document/infrastructure/delivery/LoggingStatementDeliveryAdapter.kt:12
- TodoComment openbank-governance-auditor/src/main/kotlin/com/openbank/govaudit/infrastructure/adapter/GovernanceRulesReadAdapter.kt:30
- TodoComment openbank-governance-auditor/src/main/kotlin/com/openbank/govaudit/infrastructure/adapter/GitHubReadAdapter.kt:34
- TodoComment openbank-governance-auditor/src/main/kotlin/com/openbank/govaudit/infrastructure/adapter/GitHubReadAdapter.kt:43
- TodoComment openbank-release-steward/src/main/kotlin/com/openbank/releasesteward/infrastructure/adapter/GitHubOpenPrReadAdapter.kt:34
- TodoComment openbank-release-steward/src/main/kotlin/com/openbank/releasesteward/infrastructure/adapter/GitHubOpenPrReadAdapter.kt:42
- Duplicated block (19 lines × 2 locations) openbank-admin-ui/src/app/api/campaigns/route.ts:110
- Duplicated block (19 lines × 2 locations) openbank-admin-ui/src/app/api/feedback/screen-feedback/route.ts:82
- Duplicated block (19 lines × 2 locations) openbank-admin-ui/src/app/clearing/page.tsx:66
- Duplicated block (19 lines × 2 locations) openbank-admin-ui/src/app/payments/[id]/page.tsx:215
- Duplicated block (19 lines × 2 locations) openbank-admin-ui/src/components/party/DevicesPanel.tsx:28
- Page without a main landmark openbank-admin-ui/src/app/global-error.tsx:24
- Page without a main landmark openbank-developer-portal/site/api/redoc.standalone.js:2
- Page without a main landmark openbank-infra/web/landing/index.html:2
- Page without a main landmark openbank-infra/web/landing/platform.html:2
- Duplicated block (18 lines × 2 locations) openbank-admin-ui/src/app/api/delegations/[id]/audit/route.ts:21
- Duplicated block (18 lines × 2 locations) openbank-admin-ui/src/app/document-templates/page.tsx:66
- Duplicated block (18 lines × 2 locations) openbank-admin-ui/src/app/finops/page.tsx:326
- Duplicated block (18 lines × 2 locations) openbank-admin-ui/src/lib/cards/money.ts:76
- Change coupling: PartyPort.kt ↔ PartyRepositoryImpl.kt openbank-party-service/src/main/kotlin/com/openbank/party/application/port/in/PartyPort.kt
- Change coupling: StepEditor.tsx ↔ Campaign.kt openbank-admin-ui/src/components/campaigns/StepEditor.tsx
- Change coupling: collect-test-run-evidence.py ↔ test-intelligence.ts .github/scripts/collect-test-run-evidence.py
- REDACTED
- REDACTED
- REDACTED
- <fieldset> without a <legend> openbank-admin-ui/src/app/document-templates/page.tsx:672
- <fieldset> without a <legend> openbank-admin-ui/src/app/pid/page.tsx:323
- Duplicated block (26 lines × 2 locations) openbank-admin-ui/scripts/generate-infra-lifecycle.mjs:21
- Duplicated block (26 lines × 2 locations) openbank-admin-ui/src/app/api/prometheus/[...path]/route.ts:14
- Duplicated block (25 lines × 2 locations) openbank-admin-ui/src/app/api/context/complaints/[reference]/route.ts:9
- Duplicated block (25 lines × 2 locations) openbank-admin-ui/src/app/business-onboarding/page.tsx:455
- Duplicated block (23 lines × 2 locations) openbank-admin-ui/src/app/api/campaigns/[id]/journey/route.ts:23
- Duplicated block (23 lines × 2 locations) openbank-admin-ui/src/app/payments/page.tsx:506
- Duplicated block (22 lines × 2 locations) openbank-admin-ui/src/components/campaigns/StepEditor.tsx:167
- Duplicated block (22 lines × 2 locations) openbank-admin-ui/src/lib/api.ts:215
- Duplicated block (17 lines × 2 locations) openbank-admin-ui/src/app/api/agent/mcp/route.ts:50
- Duplicated block (17 lines × 2 locations) openbank-admin-ui/src/app/api/campaigns/[id]/actions/route.ts:42
- Data table has no header cells openbank-admin-ui/src/app/docs/compliance/page.tsx:216
- Heading level jumps from h2 to h4 openbank-infra/web/landing/platform.html:285
- No assertions: grants the UI exactly the roles CreditRiskResource admits — no wider openbank-admin-ui/src/test/lending-risk-rbac.guard.test.ts:25
- Context and trade-offs are absent (only a summary + amendment list appears) docs/adr/0077-observability-three-pillar-strategy.md
- No context/problem and no consequences; only an allowlist with a one-line check-adr-registry.sh rule docs/adr/known-repos.txt
- The body is a bare list of tags and ADR domain categories with no context/problem or explicit decision docs/adr/tags.txt
- Change-coupling hub: page.tsx → REDACTED, page.tsx, StepEditor.tsx openbank-admin-ui/src/app/campaigns/[id]/page.tsx
- REDACTED
- REDACTED
- Coverage not measured — JavaScript/TypeScript suite
- Duplication concentrated across 53 sibling directories (194 clone groups) openbank-admin-ui/src/app/aml/page.tsx:18
- Duplication concentrated across 3 sibling directories (68 clone groups) openbank-admin-ui/src/app/kyc/page.tsx:91
- Duplication concentrated across 32 sibling directories (29 clone groups) openbank-admin-ui/src/app/api/finops/right-sizing/route.ts:61
- Duplication concentrated across 28 sibling directories (25 clone groups) openbank-admin-ui/src/lib/closings/upstream.ts:64
- Duplication concentrated across 14 sibling directories (11 clone groups) openbank-admin-ui/src/components/flow/StageBoard.tsx:84
- Duplicated block with local edits (154 matched lines × 2 locations) openbank-admin-ui/src/app/kyc/page.tsx:91
- Duplicated block with local edits (106 matched lines × 2 locations) openbank-admin-ui/src/components/flow/StageBoard.tsx:84
- Duplicated block with local edits (93 matched lines × 2 locations) openbank-admin-ui/src/app/aml/page.tsx:18
- Duplicated block with local edits (68 matched lines × 2 locations) openbank-admin-ui/src/app/consents/page.tsx:206
- Duplicated block with local edits (66 matched lines × 2 locations) openbank-admin-ui/src/app/devops/page.tsx:169
- Duplicated block with local edits (61 matched lines × 2 locations) openbank-admin-ui/src/app/iaops/cases/[caseId]/page.tsx:64
- Duplicated block with local edits (49 matched lines × 2 locations) openbank-admin-ui/src/app/api/finops/right-sizing/route.ts:61
- Duplicated block with local edits (46 matched lines × 2 locations) openbank-admin-ui/src/app/onboarding/page.tsx:278
- Duplicated block (38 lines × 2 locations) openbank-admin-ui/src/app/api/domestic-payments/route.ts:43
- Duplicated block (32 lines × 2 locations) openbank-admin-ui/src/app/api/catalog/governance/route.ts:33
- Duplicated block with local edits (27 matched lines × 2 locations) openbank-admin-ui/src/app/iaops/agents/[agentId]/page.tsx:147
- Duplicated block (24 lines × 2 locations) openbank-admin-ui/src/app/communication/[personaId]/page.tsx:83
- Duplicated block (21 lines × 2 locations) openbank-admin-ui/src/app/payments/page.tsx:848
- Duplicated block with local edits (20 matched lines × 2 locations) openbank-admin-ui/src/app/api/agent/proposals/route.ts:13
- Duplicated block (18 lines × 3 locations) openbank-admin-ui/src/app/api/campaigns/[id]/route.ts:80
- Duplicated block (17 lines × 3 locations) openbank-admin-ui/src/app/aml/page.tsx:70
- Complex function CampaignDetailPage (cyclomatic 98, cognitive 83) openbank-admin-ui/src/app/campaigns/[id]/page.tsx:159
- Complex function CreditRiskConsole (cyclomatic 83, cognitive 74) openbank-admin-ui/src/app/lending/risk/page.tsx:110
- Complex function PaymentsContent (cyclomatic 75, cognitive 72) openbank-admin-ui/src/app/payments/page.tsx:297
- Complex function ProductDetailPanel (cyclomatic 69, cognitive 61) openbank-admin-ui/src/app/product-catalog/page.tsx:116
- Complex function ProductStudioPage (cyclomatic 58, cognitive 50) openbank-admin-ui/src/app/product-studio/page.tsx:92
- Complex function NewCampaignPage (cyclomatic 58, cognitive 46) openbank-admin-ui/src/app/campaigns/new/page.tsx:134
- Complex function SanctionsPage (cyclomatic 57, cognitive 49) openbank-admin-ui/src/app/sanctions/page.tsx:205
- Complex function TemporalPage (cyclomatic 49, cognitive 47) openbank-admin-ui/src/app/temporal/page.tsx:258
- Complex function buildCustomerGraph (cyclomatic 47, cognitive 63) openbank-admin-ui/src/lib/context/customerGraph.ts:271
- Complex function (anonymous) (cyclomatic 47, cognitive 46) openbank-admin-ui/src/app/api/infra/lifecycle/route.ts:157
- Complex function CampaignsPage (cyclomatic 47, cognitive 41) openbank-admin-ui/src/app/campaigns/page.tsx:104
- Complex function (anonymous) (cyclomatic 46, cognitive 44) openbank-admin-ui/scripts/collect-test-intelligence.mjs:729
- Complex function IssueCardDialog (cyclomatic 46, cognitive 43) openbank-admin-ui/src/components/cards/IssueCardDialog.tsx:107
- Complex function LoyaltyPage (cyclomatic 46, cognitive 36) openbank-admin-ui/src/app/loyalty/page.tsx:40
- Complex function FinOpsContent (cyclomatic 44, cognitive 43) openbank-admin-ui/src/app/finops/page.tsx:347
- Complex function DocumentTemplatesPage (cyclomatic 44, cognitive 39) openbank-admin-ui/src/app/document-templates/page.tsx:193
- Complex function ServiceCard (cyclomatic 42, cognitive 40) openbank-admin-ui/src/app/system/health/page.tsx:156
- Complex function LifecycleStrip (cyclomatic 42, cognitive 38) openbank-admin-ui/src/components/infra/LifecycleStrip.tsx:60
- Complex function deriveCapabilityStatus (cyclomatic 42, cognitive 36) openbank-admin-ui/scripts/generate-app-status.mjs:161
- Complex function CardDetailPage (cyclomatic 39, cognitive 36) openbank-admin-ui/src/app/cards/[id]/page.tsx:76
- Large Files
- Dead file (~131 LoC) openbank-admin-ui/scripts/admin-login-synthetic.mjs
- Unused dependency '@hookform/resolvers'
- Unused dependency '@radix-ui/react-dropdown-menu'
- Unused dependency '@radix-ui/react-label'
Minor — 268 finding(s)
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Documentation: no installation or build instructions README.md
- Documentation: no usage examples README.md
- Documentation: no installation or build instructions openbank-infra/aws/README.md
- Documentation: no usage examples openbank-infra/aws/README.md
- Documentation: hard to navigate openbank-infra/web/landing/README.md
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Accessibility enforcement below the top rung
- REDACTED
- No AppArmor/SELinux confinement
- Test-only dependency 'dompurify' in production deps openbank-admin-ui/src/test/mermaid-labels.test.ts:6
Minor — 2 finding(s)
- Skipped (documented): ${rel} — not mountable: ${CANNOT_MOUNT.get(rel)} openbank-admin-ui/src/test/render-smoke.test.tsx:190
- Skipped (documented): (unnamed test) openbank-admin-ui/src/test/bff-tracing.test.ts:113
Appendix B — Reproduction & audit trail
| Dimension | Tool | Version | Command | Findings | Raw output |
|---|---|---|---|---|---|
| D28 · Secrets (history) | gitleaks | — | gitleaks detect --no-banner --config /opt/gitleaks-rules/watchdog-gitleaks.toml --report-format json --report-path /tmp/watchdog-gitleaks-8a7f18dbde3f407389265c12ea3edc81/history.json --exit-code 0 --source . | 1 | artifacts/raw/gitleaks-history.json |
| D28 · Secrets (history) | gitleaks | — | gitleaks detect --no-git --no-banner --config /opt/gitleaks-rules/watchdog-gitleaks.toml --report-format json --report-path /tmp/watchdog-gitleaks-8a7f18dbde3f407389265c12ea3edc81/tree.json --exit-code 0 --source . | 1 | artifacts/raw/gitleaks-tree.json |
| D29 · Static Analysis (SAST) | semgrep | — | semgrep --config /opt/semgrep-rules/security-audit.yml --config /opt/semgrep-rules/owasp-top-ten.yml --config /opt/semgrep-rules/watchdog-sast.yml --json --quiet --timeout 10 --timeout-threshold 3 --metrics off . | 75 | artifacts/raw/semgrep.json |
| D30 · Dependency Vulnerabilities | trivy | — | trivy fs --scanners vuln --format json --quiet --severity CRITICAL,HIGH,MEDIUM,LOW --skip-dirs **/bin/** --skip-dirs **/obj/** . --skip-db-update | 58 | artifacts/raw/trivy-fs.json |
| D31 · IaC & Container Security | trivy | — | trivy config --format json --quiet . | 1399 | artifacts/raw/trivy-config.json |
| D32 · Data Compliance (PII/GDPR) | semgrep | — | semgrep: not applicable — No personal data was found crossing a boundary the PII/GDPR ruleset checks — nothing written to a log or console sink, placed in a URL or query string, or persisted to browser storage. That is a clean result for the LEAK surface only: this ruleset detects personal data escaping, it does not inventory the personal data a repository holds, so it is not evidence that this repository has no personal-data surface. The personal-data map (Appendix C) and the C1-C5 compliance cards are what speak to that. semgrep could not parse 17 file(s) — `openbank-admin-ui/src/app/api/customer-360/[partyId]/route.ts`, `openbank-admin-ui/src/app/api/entities/resolve/route.ts`, `openbank-admin-ui/src/app/api/finops/ai-costs/route.ts`, `openbank-admin-ui/src/app/api/fx/history/route.ts`, `openbank-admin-ui/src/app/api/governance/agent-identities/route.ts`, … (+12 more) — so the PII/GDPR sweep did not cover the unparsed regions of them; rows reported elsewhere in those files are real. | 0 | — |
| D43 · Malicious Dependencies | trivy | — | trivy fs --scanners vuln --format json --quiet --severity CRITICAL,HIGH,MEDIUM,LOW --skip-dirs **/bin/** --skip-dirs **/obj/** . --skip-db-update | 0 | artifacts/raw/trivy-fs.json |