Executive summary
akoken/onion-architecture scores 39 out of 100 and is rated Weak, while four of its six lenses score higher still: Code Health at 98 and Security and Compliance at 93, both rated Exemplary, Architecture at 80, rated Strong, and Production Readiness at 45, rated Weak.
The score is lower than those four because the weakest lens counts most, and here that is Accessibility, which scores 23 and is rated Critical. Its Text alternatives dimension scores 1 out of 10, and until that reaches 4, Accessibility can't be rated higher than Adequate. The first actions are to resolve the 8 Low XML-doc coverage finding(s) in Documentation Quality and to keep a visible focus style (don't remove the outline without a replacement), guard animation with prefers-reduced-motion, and raise low-contrast colour pairs to at least 4.5:1, and if Accessibility rose to 70, the score would rise from 39 to about 52.
13 findings are critical: five in Forms & labels, three in Page structure, two in Dependency Vulnerabilities, one in Explicit Debt, one in Project dependency cycles and one in Secrets (history).
Watchdog couldn't measure Dependency Hygiene, Platform End-of-Life and Runtime evidence locked — no reproducible boot in this code, so those dimensions are left out of the score. The product code is 720 lines.
If Accessibility rose from 23 to 70, the lowest score rated Strong, the score would rise from 39 to about 52.
New since the last survey: 17 findings
- Static Analysis (SAST) · REDACTED
- Dependency Vulnerabilities · REDACTED
- Dependency Vulnerabilities · REDACTED
- Dependency Vulnerabilities · REDACTED
- Dependency Vulnerabilities · REDACTED
- Visual & motion safety · Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css
- Visual & motion safety · Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css
Rebuild cost: about €450 · Estimate
| Cost to rebuild | €160 to €740, for 3 to 8 hours of work, about 0.1 person-years for one engineer |
|---|---|
| Difficulty of the problem | High, judged from the kind of software and how it is built. Code that solves a harder problem, such as payments or medical bookings, costs more per line to write. |
| Quality factor | ×0.7 at a score of 39. Better code takes more work to write, so the factor is ×1.0 at a score of 70, lower below it and higher above it. |
| Size | Very small: 720 lines of product code, of which 73% is declarations and boilerplate, 22% straight logic and 5% logic with branches |
| Comment lines | The estimate prices 582 lines. Comment lines count towards the size of 720 lines but aren't priced as work. |
Top priorities
Diagnosis: what the measurements show together
Architecture — bounded-context dependency graph
Architecture: which parts of the code depend on which
| uses → | 1 OnionArchitecture.Core.ApplicationService | 2 OnionArchitecture.Core.Domain | 3 OnionArchitecture.DependencyResolution | 4 Web | 5 src.Client.Web.Scripts | 6 OnionArchitecture.Core.DomainService.Queries | 7 OnionArchitecture.Core.DomainService.Services | 8 OnionArchitecture.Core.DomainService.Validations | 9 OnionArchitecture.Infrastructure.Logging | 10 OnionArchitecture.Infrastructure.Persistence.EntityFramework | 11 OnionArchitecture.Core.DomainService.CommandBase | 12 OnionArchitecture.Infrastructure.Persistence.Queries | 13 OnionArchitecture.Infrastructure.Service | 14 Web.Controllers | 15 OnionArchitecture.Core.DomainService.Commands | 16 OnionArchitecture.Infrastructure.Persistence | 17 OnionArchitecture.Infrastructure.Persistence.Commands |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 1 OnionArchitecture.Core.ApplicationService | |||||||||||||||||
| 2 OnionArchitecture.Core.Domain | |||||||||||||||||
| 3 OnionArchitecture.DependencyResolution | |||||||||||||||||
| 4 Web | |||||||||||||||||
| 5 src.Client.Web.Scripts | |||||||||||||||||
| 6 OnionArchitecture.Core.DomainService.Queries | 1 | ||||||||||||||||
| 7 OnionArchitecture.Core.DomainService.Services | 2 | ||||||||||||||||
| 8 OnionArchitecture.Core.DomainService.Validations | 1 | ||||||||||||||||
| 9 OnionArchitecture.Infrastructure.Logging | 1 | ||||||||||||||||
| 10 OnionArchitecture.Infrastructure.Persistence.EntityFramework | 4 | ||||||||||||||||
| 11 OnionArchitecture.Core.DomainService.CommandBase | 1 | ||||||||||||||||
| 12 OnionArchitecture.Infrastructure.Persistence.Queries | 1 | 1 | 1 | ||||||||||||||
| 13 OnionArchitecture.Infrastructure.Service | 2 | 1 | 1 | ||||||||||||||
| 14 Web.Controllers | 1 | ||||||||||||||||
| 15 OnionArchitecture.Core.DomainService.Commands | 1 | 1 | |||||||||||||||
| 16 OnionArchitecture.Infrastructure.Persistence | 7 | ||||||||||||||||
| 17 OnionArchitecture.Infrastructure.Persistence.Commands | 1 | 1 |
6→2 OnionArchitecture.Core.DomainService.Queries uses OnionArchitecture.Core.Domain✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Core.DomainService.Queries uses a type in OnionArchitecture.Core.Domain.
- The types involved
OnionArchitecture.Core.DomainService.Queries.IGetCategoriesQuery→OnionArchitecture.Core.Domain.Category
- Direction
- OnionArchitecture.Core.DomainService.Queries uses OnionArchitecture.Core.Domain, so a change to OnionArchitecture.Core.Domain can break OnionArchitecture.Core.DomainService.Queries, while a change to OnionArchitecture.Core.DomainService.Queries can't break OnionArchitecture.Core.Domain.
- Position
- Below the diagonal: OnionArchitecture.Core.DomainService.Queries uses a module listed above it, which is the healthy direction.
7→2 OnionArchitecture.Core.DomainService.Services uses OnionArchitecture.Core.Domain✕
- Connections
- 2 pairs of types, where a type in OnionArchitecture.Core.DomainService.Services uses a type in OnionArchitecture.Core.Domain.
- The types involved
OnionArchitecture.Core.DomainService.Services.IProductService→OnionArchitecture.Core.Domain.CategoryOnionArchitecture.Core.DomainService.Services.IProductService→OnionArchitecture.Core.Domain.Product
- Direction
- OnionArchitecture.Core.DomainService.Services uses OnionArchitecture.Core.Domain, so a change to OnionArchitecture.Core.Domain can break OnionArchitecture.Core.DomainService.Services, while a change to OnionArchitecture.Core.DomainService.Services can't break OnionArchitecture.Core.Domain.
- Position
- Below the diagonal: OnionArchitecture.Core.DomainService.Services uses a module listed above it, which is the healthy direction.
8→11 OnionArchitecture.Core.DomainService.Validations uses OnionArchitecture.Core.DomainService.CommandBase part of a loop✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Core.DomainService.Validations uses a type in OnionArchitecture.Core.DomainService.CommandBase.
- The types involved
OnionArchitecture.Core.DomainService.Validations.AddNewCategoryValidator→OnionArchitecture.Core.DomainService.CommandBase.ICommandValidator
- Direction
- OnionArchitecture.Core.DomainService.Validations uses OnionArchitecture.Core.DomainService.CommandBase, so a change to OnionArchitecture.Core.DomainService.CommandBase can break OnionArchitecture.Core.DomainService.Validations, while a change to OnionArchitecture.Core.DomainService.Validations can't break OnionArchitecture.Core.DomainService.CommandBase.
- Position
- Above the diagonal, so it is part of a loop. Neither module can be changed, tested or released on its own until one of these dependencies is removed.
9→1 OnionArchitecture.Infrastructure.Logging uses OnionArchitecture.Core.ApplicationService✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Logging uses a type in OnionArchitecture.Core.ApplicationService.
- The types involved
OnionArchitecture.Infrastructure.Logging.LoggingService→OnionArchitecture.Core.ApplicationService.ILoggingService
- Direction
- OnionArchitecture.Infrastructure.Logging uses OnionArchitecture.Core.ApplicationService, so a change to OnionArchitecture.Core.ApplicationService can break OnionArchitecture.Infrastructure.Logging, while a change to OnionArchitecture.Infrastructure.Logging can't break OnionArchitecture.Core.ApplicationService.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Logging uses a module listed above it, which is the healthy direction.
10→2 OnionArchitecture.Infrastructure.Persistence.EntityFramework uses OnionArchitecture.Core.Domain✕
- Connections
- 4 pairs of types, where a type in OnionArchitecture.Infrastructure.Persistence.EntityFramework uses a type in OnionArchitecture.Core.Domain.
- The types involved
OnionArchitecture.Infrastructure.Persistence.EntityFramework.IStoreContext→OnionArchitecture.Core.Domain.CategoryOnionArchitecture.Infrastructure.Persistence.EntityFramework.IStoreContext→OnionArchitecture.Core.Domain.ProductOnionArchitecture.Infrastructure.Persistence.EntityFramework.StoreContext→OnionArchitecture.Core.Domain.CategoryOnionArchitecture.Infrastructure.Persistence.EntityFramework.StoreContext→OnionArchitecture.Core.Domain.Product
- Direction
- OnionArchitecture.Infrastructure.Persistence.EntityFramework uses OnionArchitecture.Core.Domain, so a change to OnionArchitecture.Core.Domain can break OnionArchitecture.Infrastructure.Persistence.EntityFramework, while a change to OnionArchitecture.Infrastructure.Persistence.EntityFramework can't break OnionArchitecture.Core.Domain.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Persistence.EntityFramework uses a module listed above it, which is the healthy direction.
11→8 OnionArchitecture.Core.DomainService.CommandBase uses OnionArchitecture.Core.DomainService.Validations✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Core.DomainService.CommandBase uses a type in OnionArchitecture.Core.DomainService.Validations.
- The types involved
OnionArchitecture.Core.DomainService.CommandBase.ICommandValidator→OnionArchitecture.Core.DomainService.Validations.ValidationResult
- Direction
- OnionArchitecture.Core.DomainService.CommandBase uses OnionArchitecture.Core.DomainService.Validations, so a change to OnionArchitecture.Core.DomainService.Validations can break OnionArchitecture.Core.DomainService.CommandBase, while a change to OnionArchitecture.Core.DomainService.CommandBase can't break OnionArchitecture.Core.DomainService.Validations.
- Position
- Below the diagonal: OnionArchitecture.Core.DomainService.CommandBase uses a module listed above it, which is the healthy direction.
12→2 OnionArchitecture.Infrastructure.Persistence.Queries uses OnionArchitecture.Core.Domain✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Persistence.Queries uses a type in OnionArchitecture.Core.Domain.
- The types involved
OnionArchitecture.Infrastructure.Persistence.Queries.GetCategoriesQuery→OnionArchitecture.Core.Domain.Category
- Direction
- OnionArchitecture.Infrastructure.Persistence.Queries uses OnionArchitecture.Core.Domain, so a change to OnionArchitecture.Core.Domain can break OnionArchitecture.Infrastructure.Persistence.Queries, while a change to OnionArchitecture.Infrastructure.Persistence.Queries can't break OnionArchitecture.Core.Domain.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Persistence.Queries uses a module listed above it, which is the healthy direction.
12→6 OnionArchitecture.Infrastructure.Persistence.Queries uses OnionArchitecture.Core.DomainService.Queries✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Persistence.Queries uses a type in OnionArchitecture.Core.DomainService.Queries.
- The types involved
OnionArchitecture.Infrastructure.Persistence.Queries.GetCategoriesQuery→OnionArchitecture.Core.DomainService.Queries.IGetCategoriesQuery
- Direction
- OnionArchitecture.Infrastructure.Persistence.Queries uses OnionArchitecture.Core.DomainService.Queries, so a change to OnionArchitecture.Core.DomainService.Queries can break OnionArchitecture.Infrastructure.Persistence.Queries, while a change to OnionArchitecture.Infrastructure.Persistence.Queries can't break OnionArchitecture.Core.DomainService.Queries.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Persistence.Queries uses a module listed above it, which is the healthy direction.
12→10 OnionArchitecture.Infrastructure.Persistence.Queries uses OnionArchitecture.Infrastructure.Persistence.EntityFramework✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Persistence.Queries uses a type in OnionArchitecture.Infrastructure.Persistence.EntityFramework.
- The types involved
OnionArchitecture.Infrastructure.Persistence.Queries.GetCategoriesQuery→OnionArchitecture.Infrastructure.Persistence.EntityFramework.IStoreContext
- Direction
- OnionArchitecture.Infrastructure.Persistence.Queries uses OnionArchitecture.Infrastructure.Persistence.EntityFramework, so a change to OnionArchitecture.Infrastructure.Persistence.EntityFramework can break OnionArchitecture.Infrastructure.Persistence.Queries, while a change to OnionArchitecture.Infrastructure.Persistence.Queries can't break OnionArchitecture.Infrastructure.Persistence.EntityFramework.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Persistence.Queries uses a module listed above it, which is the healthy direction.
13→2 OnionArchitecture.Infrastructure.Service uses OnionArchitecture.Core.Domain✕
- Connections
- 2 pairs of types, where a type in OnionArchitecture.Infrastructure.Service uses a type in OnionArchitecture.Core.Domain.
- The types involved
OnionArchitecture.Infrastructure.Service.ProductService→OnionArchitecture.Core.Domain.CategoryOnionArchitecture.Infrastructure.Service.ProductService→OnionArchitecture.Core.Domain.Product
- Direction
- OnionArchitecture.Infrastructure.Service uses OnionArchitecture.Core.Domain, so a change to OnionArchitecture.Core.Domain can break OnionArchitecture.Infrastructure.Service, while a change to OnionArchitecture.Infrastructure.Service can't break OnionArchitecture.Core.Domain.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Service uses a module listed above it, which is the healthy direction.
13→7 OnionArchitecture.Infrastructure.Service uses OnionArchitecture.Core.DomainService.Services✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Service uses a type in OnionArchitecture.Core.DomainService.Services.
- The types involved
OnionArchitecture.Infrastructure.Service.ProductService→OnionArchitecture.Core.DomainService.Services.IProductService
- Direction
- OnionArchitecture.Infrastructure.Service uses OnionArchitecture.Core.DomainService.Services, so a change to OnionArchitecture.Core.DomainService.Services can break OnionArchitecture.Infrastructure.Service, while a change to OnionArchitecture.Infrastructure.Service can't break OnionArchitecture.Core.DomainService.Services.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Service uses a module listed above it, which is the healthy direction.
13→10 OnionArchitecture.Infrastructure.Service uses OnionArchitecture.Infrastructure.Persistence.EntityFramework✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Service uses a type in OnionArchitecture.Infrastructure.Persistence.EntityFramework.
- The types involved
OnionArchitecture.Infrastructure.Service.ProductService→OnionArchitecture.Infrastructure.Persistence.EntityFramework.IStoreContext
- Direction
- OnionArchitecture.Infrastructure.Service uses OnionArchitecture.Infrastructure.Persistence.EntityFramework, so a change to OnionArchitecture.Infrastructure.Persistence.EntityFramework can break OnionArchitecture.Infrastructure.Service, while a change to OnionArchitecture.Infrastructure.Service can't break OnionArchitecture.Infrastructure.Persistence.EntityFramework.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Service uses a module listed above it, which is the healthy direction.
14→7 Web.Controllers uses OnionArchitecture.Core.DomainService.Services✕
- Connections
- 1 pair of types, where a type in Web.Controllers uses a type in OnionArchitecture.Core.DomainService.Services.
- The types involved
Web.Controllers.HomeController→OnionArchitecture.Core.DomainService.Services.IProductService
- Direction
- Web.Controllers uses OnionArchitecture.Core.DomainService.Services, so a change to OnionArchitecture.Core.DomainService.Services can break Web.Controllers, while a change to Web.Controllers can't break OnionArchitecture.Core.DomainService.Services.
- Position
- Below the diagonal: Web.Controllers uses a module listed above it, which is the healthy direction.
15→2 OnionArchitecture.Core.DomainService.Commands uses OnionArchitecture.Core.Domain✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Core.DomainService.Commands uses a type in OnionArchitecture.Core.Domain.
- The types involved
OnionArchitecture.Core.DomainService.Commands.AddNewCategoryCommand→OnionArchitecture.Core.Domain.Category
- Direction
- OnionArchitecture.Core.DomainService.Commands uses OnionArchitecture.Core.Domain, so a change to OnionArchitecture.Core.Domain can break OnionArchitecture.Core.DomainService.Commands, while a change to OnionArchitecture.Core.DomainService.Commands can't break OnionArchitecture.Core.Domain.
- Position
- Below the diagonal: OnionArchitecture.Core.DomainService.Commands uses a module listed above it, which is the healthy direction.
15→11 OnionArchitecture.Core.DomainService.Commands uses OnionArchitecture.Core.DomainService.CommandBase✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Core.DomainService.Commands uses a type in OnionArchitecture.Core.DomainService.CommandBase.
- The types involved
OnionArchitecture.Core.DomainService.Commands.AddNewCategoryCommand→OnionArchitecture.Core.DomainService.CommandBase.ICommand
- Direction
- OnionArchitecture.Core.DomainService.Commands uses OnionArchitecture.Core.DomainService.CommandBase, so a change to OnionArchitecture.Core.DomainService.CommandBase can break OnionArchitecture.Core.DomainService.Commands, while a change to OnionArchitecture.Core.DomainService.Commands can't break OnionArchitecture.Core.DomainService.CommandBase.
- Position
- Below the diagonal: OnionArchitecture.Core.DomainService.Commands uses a module listed above it, which is the healthy direction.
16→11 OnionArchitecture.Infrastructure.Persistence uses OnionArchitecture.Core.DomainService.CommandBase✕
- Connections
- 7 pairs of types, where a type in OnionArchitecture.Infrastructure.Persistence uses a type in OnionArchitecture.Core.DomainService.CommandBase.
- The types involved
OnionArchitecture.Infrastructure.Persistence.CommandDispatcher→OnionArchitecture.Core.DomainService.CommandBase.ICommandOnionArchitecture.Infrastructure.Persistence.CommandDispatcher→OnionArchitecture.Core.DomainService.CommandBase.ICommandDispatcherOnionArchitecture.Infrastructure.Persistence.CommandDispatcher→OnionArchitecture.Core.DomainService.CommandBase.ICommandHandlerOnionArchitecture.Infrastructure.Persistence.CommandDispatcher→OnionArchitecture.Core.DomainService.CommandBase.ICommandValidatorOnionArchitecture.Infrastructure.Persistence.CommandExecutor→OnionArchitecture.Core.DomainService.CommandBase.ICommandOnionArchitecture.Infrastructure.Persistence.CommandExecutor→OnionArchitecture.Core.DomainService.CommandBase.ICommandDispatcherOnionArchitecture.Infrastructure.Persistence.CommandExecutor→OnionArchitecture.Core.DomainService.CommandBase.ICommandExecutor
- Direction
- OnionArchitecture.Infrastructure.Persistence uses OnionArchitecture.Core.DomainService.CommandBase, so a change to OnionArchitecture.Core.DomainService.CommandBase can break OnionArchitecture.Infrastructure.Persistence, while a change to OnionArchitecture.Infrastructure.Persistence can't break OnionArchitecture.Core.DomainService.CommandBase.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Persistence uses a module listed above it, which is the healthy direction.
17→10 OnionArchitecture.Infrastructure.Persistence.Commands uses OnionArchitecture.Infrastructure.Persistence.EntityFramework✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Persistence.Commands uses a type in OnionArchitecture.Infrastructure.Persistence.EntityFramework.
- The types involved
OnionArchitecture.Infrastructure.Persistence.Commands.AddNewCategoryCommandHandler→OnionArchitecture.Infrastructure.Persistence.EntityFramework.IStoreContext
- Direction
- OnionArchitecture.Infrastructure.Persistence.Commands uses OnionArchitecture.Infrastructure.Persistence.EntityFramework, so a change to OnionArchitecture.Infrastructure.Persistence.EntityFramework can break OnionArchitecture.Infrastructure.Persistence.Commands, while a change to OnionArchitecture.Infrastructure.Persistence.Commands can't break OnionArchitecture.Infrastructure.Persistence.EntityFramework.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Persistence.Commands uses a module listed above it, which is the healthy direction.
17→11 OnionArchitecture.Infrastructure.Persistence.Commands uses OnionArchitecture.Core.DomainService.CommandBase✕
- Connections
- 1 pair of types, where a type in OnionArchitecture.Infrastructure.Persistence.Commands uses a type in OnionArchitecture.Core.DomainService.CommandBase.
- The types involved
OnionArchitecture.Infrastructure.Persistence.Commands.AddNewCategoryCommandHandler→OnionArchitecture.Core.DomainService.CommandBase.ICommandHandler
- Direction
- OnionArchitecture.Infrastructure.Persistence.Commands uses OnionArchitecture.Core.DomainService.CommandBase, so a change to OnionArchitecture.Core.DomainService.CommandBase can break OnionArchitecture.Infrastructure.Persistence.Commands, while a change to OnionArchitecture.Infrastructure.Persistence.Commands can't break OnionArchitecture.Core.DomainService.CommandBase.
- Position
- Below the diagonal: OnionArchitecture.Infrastructure.Persistence.Commands uses a module listed above it, which is the healthy direction.
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 |
|---|---|---|
| A03:2025 — Software Supply Chain Failures | 4 | High / Critical |
| A07:2025 — Authentication Failures | 2 | High / Critical |
| A05:2025 — Injection | 1 | Medium |
Roadmap
First, address visual and motion safety by ensuring visible focus styles, respecting reduced motion preferences, and fixing low-contrast colors. Next, add appropriate text alternatives for all images, SVGs, canvases, and videos to improve accessibility. Then, document significant design decisions in a dedicated folder and resolve low XML documentation coverage in the core project files. Finally, improve the value of code comments by removing redundant ones and enhancing the clarity of existing notes.
| Do this | Helps | Effort | Dimension |
|---|---|---|---|
| Resolve the 8 Low XML-doc coverage finding(s) in Documentation Quality — start with Domain.csproj, DomainService.csproj, ApplicationService.csproj. | +4.0 pts | Low | Documentation Quality |
| Keep a visible focus style (don't remove the outline without a replacement), guard animation with prefers-reduced-motion, and raise low-contrast colour pairs to at least 4.5:1. | +6.3 pts | Medium | Visual & motion safety |
| Add a text alternative — alt on images (alt="" for purely decorative ones), a title or aria-label on meaningful svg, an aria-label or inner fallback content on canvas, and a captions <track> on video. | +6.3 pts | Medium | Text alternatives |
| Record your significant design decisions. Write one short document per decision, giving its date, the situation, the decision and its consequences, and keep them together with your other design documents. A docs/adr folder with each file named by number and title is the easiest place to find them. | +4.7 pts | Medium | Architecture documentation |
| Improve Comment Value — currently 3.0/10. | +4.0 pts | Medium | Comment Value |
| Expand the README with getting-started, architecture overview and a project map. | +3.3 pts | Medium | Documentation (README) |
| Give every control a programmatic label (a <label for> / wrapping <label> / aria-label) and every button text — a placeholder is not a label. | +3.2 pts | Medium | Forms & labels |
| Add a SAST step to CI running what this repository's stack ships: CodeQL's csharp pack, or a .NET security analyzer package — or `semgrep --config=auto`, which runs on any language — so a security regression fails the build instead of landing. | +3.1 pts | Medium | Security & performance tooling |
File quality
| File | Score | Band | Worst signal |
|---|---|---|---|
| REDACTED | 2.7 | Slop | Dependency Vulnerabilities: High CVE: REDACTED |
| src/DependencyResolution/Bootstrapper/Bootstrapper.csproj | 7.0 | Mixed | Coupling: Unstable project Bootstrapper |
| src/Core/Domain/Domain.csproj | 7.0 | Mixed | Coupling: Off the main sequence: Domain |
| REDACTED | 7.2 | Mixed | Secrets (history): REDACTED: REDACTED |
| test/Infrastructure.Tests/GetCategoriesQueryTests.cs | 7.6 | Mixed | Explicit Debt: WriteOnlyPrivateField |
| REDACTED | 7.9 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| OnionArchitecture.sln | 8.5 | Near-clean | Solution Shape: Solution references a project that does not exist |
| src/Core/DomainService/DomainService.csproj | 8.5 | Near-clean | Documentation Quality: Low XML-doc coverage: DomainService |
| src/Core/ApplicationService/ApplicationService.csproj | 8.5 | Near-clean | Documentation Quality: Low XML-doc coverage: ApplicationService |
| src/Infrastructure/Persistence/Persistence.csproj | 8.5 | Near-clean | Documentation Quality: Low XML-doc coverage: Persistence |
| src/Infrastructure/Logging/Logging.csproj | 8.5 | Near-clean | Documentation Quality: Low XML-doc coverage: Logging |
| src/Infrastructure/Service/Service.csproj | 8.5 | Near-clean | Documentation Quality: Low XML-doc coverage: Service |
| src/Client/Web/Web.csproj | 8.5 | Near-clean | Documentation Quality: Low XML-doc coverage: Web |
| README.md | 9.5 | Near-clean | Documentation Quality: Documentation: no installation or build instructions |
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. 61 of 65 evaluated dimensions are computed purely by tools and static analysis (confidence 1.0); 4 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 — 65 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, 54 of 66 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 | Secrets in history, SAST, CVEs, IaC & container, PII / GDPR | 1.86.0 · 0.69.3 | ✓ 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
- 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 MEASURED: the test run produced no results for any test tier, so no test ever ran and flakiness could not be exercised. The cause could not be attributed, so it is excluded from the score rather than read as an absence of tests.
- 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. `dotnet list package` did not return usable output (restore or query failed); no outdated/vulnerable/deprecated data to assess. The tool said: `dotnet list package` exited 1 with no output.
- 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 76 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.
- D17 Explicit Debt — 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 C# workspace loaded this repository's projects but reported a load failure, and the dead-code reference search is unreliable over a graph that did not load cleanly, so it was skipped. The other debt markers on this row were read; dead symbols were not, and their absence from the score is not a finding that the code has none. First load diagnostic: Failure: solution references project 'Test\Infrastructure.Tests\Infrastructure.Tests.csproj' but the on-disk path is 'test\Infrastructure.Tests\Infrastructure.Tests.csproj' (separator/casing mismatch); on a case-sensitive file system MSBuildWorkspace would load 0 of these projects. Loading a corrected copy of the solution.
- D18 Solution Shape — 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 build did not finish within its budget on this run; structural metrics are reported, build/warning status is unknown.
- D22 Internal API Consistency — 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. The loaded project set declares no packable project and no `.Contracts` project, so there is no intentionally-exposed surface for API consistency to be judged over.
- D30 Dependency Vulnerabilities — measured, with a gap in what it reached — 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. The solution did not restore on the analyzer's .NET SDK (an SDK/target-framework/restore mismatch, common for an older codebase), so there was no restored dependency graph to scan. 1 of 2 declared ecosystem(s) (nuget REDACTED) WERE scanned and every vulnerability they reported is included in this result; nuget was not, so this dimension's score covers less than the dependency surface this repository declares, and nothing here is evidence that nuget is free of known-vulnerable dependencies.
- D34 Knowledge Freshness — 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. All 37 production source file(s) in this repository were read, and every one is under the ~2,400-byte (~100-line) size floor this dimension measures over (or declares itself generated), so there is no significant file to judge and knowledge freshness is not applicable. This is the size of the repository's files, not a file the engine failed to read.
- D39 IL Efficiency — 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. `dotnet build` exited 1 with a compiler/MSBuild diagnostic, so there were no assemblies to read IL from. The build itself is D18's row, with the ask; nothing further is charged here.
- D44 Platform End-of-Life — 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 dimension reads a project's own statement about the platform it runs on: a TargetFramework in a .NET project file, a .nvmrc or .python-version, a capped requires-python, a Rust toolchain file or Cargo.toml rust-version, a .go-version, .java-version, .ruby-version, .tool-versions or .sdkmanrc, a go.mod go directive, a Maven or Gradle Java level or toolchain, a Gemfile's ruby directive, a mix.exs elixir requirement, a rebar.config minimum_otp_vsn, a pubspec.yaml SDK constraint, a build.sbt scalaVersion, or a framework major pinned by a dependency constraint. This repository carries none of them, so nothing about its platform was established. That is a gap in this analyzer's coverage, NOT a finding that the platform is supported — a language whose runtime is declared elsewhere (Package.swift, a Dockerfile) is simply not read here yet.
- AXB1 Runtime evidence locked — no reproducible boot — 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. Subject: 8 markup document(s) — HTML, Razor, JSX, Vue, Svelte or server templates. The Runtime Evidence tier boots an app only via docker-compose, an Aspire AppHost, a Dockerfile, an npm dev script, a web service's own toolchain entry point (cargo, go, dotnet, Python, Spring Boot), or an Android app's own Gradle wrapper (run on an emulator); an Apple or Flutter app needs a host it lacks. None applied, so no live runtime a11y/egress/header evidence was collected. You can widen what we reach: add a docker-compose.yml (or an Aspire AppHost) that brings the app up with its dependencies. Watchdog then boots it in an isolated sandbox and gathers real runtime evidence — you change nothing in your pipeline (no CI step, no SDK).
- P5 DR & Backup — 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. No backup/snapshot/replication config, RTO/RPO or restore-procedure documentation was found — and no production persistence was detected either (no data-access packages, no data-store services, no database resources), so there is nothing in this repository whose loss a DR control would recover. If this system's data lives in a platform or ops repo we can't see, that's where the DR evidence belongs.
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
- D1 Cyclomatic Complexity: Cyclomatic complexity counts branches statically — it cannot tell an essential decision tree from accidental tangle, nor see complexity that lives in data or configuration (large switch-case token tables, DSL lexers/parsers, data-as-code rule tables) rather than control flow: a tokenizer's many single-character cases read as high complexity though each branch is trivial.
- D2 Cognitive Complexity: Cognitive-complexity heuristics approximate how hard code is to follow; genuine domain difficulty and well-named intent that eases reading are not captured.
- D3 God Classes: "God class" is sized by members and responsibilities visible in the type — a deliberately broad facade over a coherent subsystem can read the same as an accidental grab-bag. For front-end JS the file-length check is cohesion-aware (a single-responsibility module — one class/IIFE — earns a 3× threshold), but cohesion is approximated from top-level declarations, not true dependency structure.
- D4 Code Duplication: Duplication is token-similarity — an in-process token-stream comparison over sliding windows, with type-aware normalization — so it finds copy-paste, not semantic duplication expressed differently. Committed machine-written code (scaffolded migrations, designer/codegen output, protobuf/OpenAPI stubs, model snapshots) is EXCLUDED — its repetition is the tool's, not the team's — so the score reflects hand-written duplication only.
- D5 Coupling: Coupling is measured between projects/assemblies — runtime coupling through DI, reflection, messaging or shared databases is invisible to a static reference graph.
- D6 Cohesion (LCOM4): LCOM4 cohesion is syntactic — it infers connectivity from which methods touch which fields/methods by name, not from real runtime behaviour or intent.
- 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").
- D14 License Compliance: License compatibility is checked against declared package metadata and a policy — mislabelled or missing license metadata, and obligations that depend on how you distribute, are not resolved here.
- D15 Churn × Complexity Hotspots: Churn hotspots come from git history — a freshly imported or squashed repository has no churn signal, and recent rewrites can mask a historically risky file.
- 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.
- D18 Solution Shape: Build integrity reflects whether the solution compiled in this environment — a build that needs a private feed, a specific SDK, or a generated file absent from the repo can read as broken when it is merely unreproducible here.
- 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.
- 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.
- D24 Comment Value: Comment value (WHY vs WHAT) is an LLM judgement over a bounded sample — it is advisory and cannot weigh a comment against the precise code change it was written to explain.
- D26 Project Cohesion: Project focus is sized from members/namespaces per project — a project that is broad by deliberate design reads the same as one that has sprawled.
- 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.
- 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.
- 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.
- 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.
- 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.
- 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
D1 · Cyclomatic Complexity
0 method(s) exceeded the cyclomatic complexity threshold of 15.
D2 · Cognitive Complexity
0 method(s) exceeded the cognitive complexity threshold of 15.
D3 · God Classes
0 god class(es) detected.
D4 · Code Duplication
0 duplicated block group(s) detected.
D5 · Coupling
9 projects, 0 dependency cycle(s), 1 unstable depended-on project(s).
What to do
- Resolve the 1 Unstable project Bootstrapper finding(s) in Coupling — start with Bootstrapper.csproj. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 Off the main sequence finding(s) in Coupling — start with Domain.csproj. — One of this dimension's main actionable groups (1 warning-level).
- Enforce Coupling in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D6 · Cohesion (LCOM4)
0 of 1 classes have LCOM4 above 3.
D9 · Test Distribution
1 test methods: 1 unit, 0 integration, 0 BDD, 0 e2e.
D10 · Test Quality
0 skipped, 0 zero-assertion, no mocking-framework packages referenced (hand-written doubles or no mocking) across 1 tests.
D13 · REDACTED Scanning
REDACTED scan ran and found no leaked secrets.
D14 · License Compliance
0 of 168 packages use a banned license. The depth of this set was not established for this repository: the graded packages are what this pass resolved, and no lockfile was read, so it is not certified to be the full transitive closure.
D15 · Churn × Complexity Hotspots
No churn × complexity hotspots in the window.
D17 · Explicit Debt
1 deducted debt markers + unmeasured dead symbols across 720 LoC in the .NET projects (0.4/KLoC) → score 9.3. Dead code not measured: the solution did not load cleanly, so the reference search was skipped.
What to do
- Resolve the 1 WriteOnlyPrivateField finding(s) in Explicit Debt — start with GetCategoriesQueryTests.cs. — One of this dimension's main actionable groups (1 issue-level).
- Enforce Explicit Debt in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D18 · Solution Shape
9 projects, 42 source files, 784 hand-written lines of code (720 production / 64 test — the split is derived per file: test is what its project, its own path, or a compile-guarded region marks as test, and a file carrying no test signal counts as production), 17 inter-project edges (build status unknown — did not finish).
What to do
- Resolve the 1 Solution references a project that does not exist finding(s) in Solution Shape — start with OnionArchitecture.sln. — One of this dimension's main actionable groups (1 warning-level).
D19 · Documentation Quality
The repository's root README is a one-line project banner with no overview, installation, usage, or contribution guidance. It states the project name and its purpose but gives no description of what it does, how to install/run it, or where to contribute.
What to do
- Resolve the 8 Low XML-doc coverage finding(s) in Documentation Quality — start with Domain.csproj, DomainService.csproj, ApplicationService.csproj. — One of this dimension's main actionable groups (8 warning-level).
- Resolve the 1 Documentation finding(s) in Documentation Quality — start with README.md. — One of this dimension's main actionable groups (1 recommendation-level).
D21 · Naming Consistency
0 naming inconsistencies across 133 sampled symbols.
D24 · Comment Value
2 valuable / 0 redundant across 35 sampled comments.
What to do
- Improve Comment Value — currently 3.0/10. — 2 valuable / 0 redundant across 35 sampled comments.
D26 · Project Cohesion
0 of 9 projects flagged as possibly oversized/incoherent.
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)
1 finding(s): 0 critical, 0 high, 1 medium, 0 low.
What to do
- Resolve the 1 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED. — One of this dimension's main actionable groups (1 warning-level).
D30 · Dependency Vulnerabilities
4 finding(s): 0 critical, 2 high, 2 medium, 0 low. Partial dependency scan: 1 of 2 declared ecosystem(s) were scanned (nuget REDACTED), and the findings above are real and complete for them. nuget was not scanned (nuget: the solution did not restore on the analyzer's .NET SDK (an SDK/target-framework/restore mismatch, common for an older codebase), so there was no restored dependency graph to scan), so this is not the whole dependency surface and the result is reported at reduced confidence.
What to do
- Resolve the 2 High CVE finding(s) in Dependency Vulnerabilities — start with REDACTED (2). — One of this dimension's main actionable groups (2 issue-level).
- Resolve the 2 Medium CVE finding(s) in Dependency Vulnerabilities — start with REDACTED (2). — One of this dimension's main actionable groups (2 warning-level).
D35 · Change Coupling
No strong hidden change-coupling between production files.
D43 · Malicious Dependencies
No dependency in any ecosystem this repository declares is published as malicious.
Frontend & cross-cutting dimensions
AC1 · Text alternatives
- A <object> with no aria-label/title and no inner fallback content can't be described by assistive tech. Add a name or fallback content. — src/Client/Web/Scripts/jquery-1.10.2.js:6034
What to do
- Add a text alternative — alt on images (alt="" for purely decorative ones), a title or aria-label on meaningful svg, an aria-label or inner fallback content on canvas, and a captions <track> on video.
AC2 · Forms & labels
- This control has no associated label. Add a <label for> / wrapping <label> / aria-label / aria-labelledby so assistive tech can name it. (×4) — src/Client/Web/Scripts/jquery-1.10.2.js:1615, src/Client/Web/Scripts/jquery-1.10.2.js:2959, src/Client/Web/Scripts/jquery-1.10.2.js:3340, …
- A link with no text, aria-label or labelled child (e.g. an icon-only link) has no accessible name, so assistive tech can't say where it goes. Add visible text or an aria-label. — src/Client/Web/Scripts/jquery-1.10.2.js:2946
- 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. — src/Client/Web/Scripts/jquery-1.10.2.js:6032
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). — src/Client/Web/Scripts/jquery-1.10.2.js:3503
- An iframe with no title (or an empty one) is announced only as "frame". Add a non-empty title describing its content, or an aria-label. — src/Client/Web/Scripts/jquery-1.10.2.js:7291
- The page declares no language, so assistive tech can't pick the right pronunciation. Add lang (e.g. lang="en"). (×2) — src/Client/Web/Views/Shared/Error.cshtml:2, src/Client/Web/Views/Shared/_Layout.cshtml:2
- No <main> (or role="main") means no "skip to content" target and a weaker landmark map. Wrap the primary content in <main>. (×2) — src/Client/Web/Views/Shared/Error.cshtml:2, src/Client/Web/Views/Shared/_Layout.cshtml:2
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.
AC6 · Visual & motion safety
- `.btn-primary` sets color: #ffffff on background-color: #428bca — 3.6:1, below the 4.5:1 WCAG AA minimum for normal text. Darken or lighten one of them. — src/Client/Web/Content/bootstrap.css:2092
- `.btn.active` sets the outline to none/0 and no visible :focus / :focus-visible style for that element replaces it in this stylesheet, so a keyboard user cannot see where focus is. Draw a focus ring (outline, box-shadow or border) under :focus-visible. — src/Client/Web/Content/bootstrap.css:2035
- `.dropdown-toggle:focus` sets the outline to none/0 and no visible :focus / :focus-visible style for that element replaces it in this stylesheet, so a keyboard user cannot see where focus is. Draw a focus ring (outline, box-shadow or border) under :focus-visible. — src/Client/Web/Content/bootstrap.css:3232
- `.dropdown-menu > .active > a` sets the outline to none/0 and no visible :focus / :focus-visible style for that element replaces it in this stylesheet, so a keyboard user cannot see where focus is. Draw a focus ring (outline, box-shadow or border) under :focus-visible. — src/Client/Web/Content/bootstrap.css:3291
- `.open > a` sets the outline to none/0 and no visible :focus / :focus-visible style for that element replaces it in this stylesheet, so a keyboard user cannot see where focus is. Draw a focus ring (outline, box-shadow or border) under :focus-visible. — src/Client/Web/Content/bootstrap.css:3314
- `.btn-group > .btn:focus` sets the outline to none/0 and no visible :focus / :focus-visible style for that element replaces it in this stylesheet, so a keyboard user cannot see where focus is. Draw a focus ring (outline, box-shadow or border) under :focus-visible. — src/Client/Web/Content/bootstrap.css:3410
- `.btn-group.open .dropdown-toggle` sets the outline to none/0 and no visible :focus / :focus-visible style for that element replaces it in this stylesheet, so a keyboard user cannot see where focus is. Draw a focus ring (outline, box-shadow or border) under :focus-visible. — src/Client/Web/Content/bootstrap.css:3491
- `.modal-content` sets the outline to none/0 and no visible :focus / :focus-visible style for that element replaces it in this stylesheet, so a keyboard user cannot see where focus is. Draw a focus ring (outline, box-shadow or border) under :focus-visible. — src/Client/Web/Content/bootstrap.css:5655
- `.collapsing` moves (`transition: height 0.35s ease`) and nothing reads prefers-reduced-motion for it — no @media (prefers-reduced-motion) rule for this element, no reset this project loads — so a motion-sensitive user cannot turn it off (WCAG 2.3.3). Declare it under @media (prefers-reduced-motion: no-preference), or neutralise it under reduce. — src/Client/Web/Content/bootstrap.css:2394
- `.progress-bar` moves (`transition: width 0.6s ease`) and nothing reads prefers-reduced-motion for it — no @media (prefers-reduced-motion) rule for this element, no reset this project loads — so a motion-sensitive user cannot turn it off (WCAG 2.3.3). Declare it under @media (prefers-reduced-motion: no-preference), or neutralise it under reduce. — src/Client/Web/Content/bootstrap.css:5151
- `.progress.active .progress-bar` moves (`animation: progress-bar-stripes 2s linear infinite`) and nothing reads prefers-reduced-motion for it — no @media (prefers-reduced-motion) rule for this element, no reset this project loads — so a motion-sensitive user cannot turn it off (WCAG 2.3.3). Declare it under @media (prefers-reduced-motion: no-preference), or neutralise it under reduce. — src/Client/Web/Content/bootstrap.css:5167
- `.modal.fade .modal-dialog` moves (`transition: transform 0.3s ease-out`) and nothing reads prefers-reduced-motion for it — no @media (prefers-reduced-motion) rule for this element, no reset this project loads — so a motion-sensitive user cannot turn it off (WCAG 2.3.3). Declare it under @media (prefers-reduced-motion: no-preference), or neutralise it under reduce. — src/Client/Web/Content/bootstrap.css:5632
- `.carousel-inner > .item` moves (`transition: 0.6s ease-in-out left`) and nothing reads prefers-reduced-motion for it — no @media (prefers-reduced-motion) rule for this element, no reset this project loads — so a motion-sensitive user cannot turn it off (WCAG 2.3.3). Declare it under @media (prefers-reduced-motion: no-preference), or neutralise it under reduce. — src/Client/Web/Content/bootstrap.css:6020
What to do
- Keep a visible focus style (don't remove the outline without a replacement), guard animation with prefers-reduced-motion, and raise low-contrast colour pairs to at least 4.5:1.
AC7 · A11y enforcement
- No accessibility enforcement found — no automated accessibility check runs over the HTML your app renders. Assert the accessibility invariants over that HTML in the test suite you already have (parse the output and assert, or drive a browser), and gate that test in CI so a regression blocks the merge. What was searched, so you can tell an absence from a miss: the 8 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 test suite you already have: assert the accessibility invariants over the HTML your app renders — parse the rendered output in an existing test, or drive a real browser from one — and gate that test 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.
AX3 · Project dependency cycles
- The namespaces OnionArchitecture.Core.DomainService.CommandBase ↔ OnionArchitecture.Core.DomainService.Validations depend on each other. The lightest dependency in the cycle is OnionArchitecture.Core.DomainService.CommandBase → OnionArchitecture.Core.DomainService.Validations (OnionArchitecture.Core.DomainService.CommandBase.ICommandValidator → OnionArchitecture.Core.DomainService.Validations.ValidationResult, 1 type pair(s)); cut that one first, e.g. by moving the shared type to a namespace both can use or inverting it behind an interface. — src/Core/DomainService/CommandBase/ICommandValidator.cs:1
What to do
- Break namespace cycles inside a project by moving the shared type to a namespace both sides can use, or by inverting the back edge behind an interface.
AX4 · Dependency direction
AX5 · Architecture & structure
AX6 · Interface segregation
GD1 · Unfinished & placeholder code
IC1 · Incompleteness & stubs
M1 · Documentation (README)
- The root README is 15 words, against a bar of 120. Of the three newcomer-critical sections this check looks for by heading, it found no a build/run or getting-started section, no a testing section. Sections are matched on HEADING text only, so material written under a heading this check does not recognise — or with no heading at all — is not seen and this row may understate what the document covers.
What to do
- Expand the README with getting-started, architecture overview and a project map.
- Add a build/run (quick start) section to the root README — the first thing a newcomer needs.
- Explain in the README how to run the tests. Add a section called Testing to the README in the repository's top folder, saying how to run the test suite.
- Add a README to the 9 of 9 project(s) that lack one — worth up to 2 pts.
M2 · Architecture documentation
- No Architecture Decision Records found — no conventional ADR directory, no numbered `NNNN-title` documents in any markup this check reads, and nothing ADR-shaped by content. Design rationale recorded elsewhere (a design-notes tree, a mailing list, pull-request discussion) is not visible to this check and is not re-findable per decision, so a future maintainer cannot ask why one choice was made and get an answer.
- No C4/Structurizr/PlantUML/Mermaid/Graphviz/D2 diagram, no drawn diagram named for the architecture, no file named `architecture` or `design` in any markup this check reads, and nothing in the README, docs or contributor guides that announces the shape — no `## Architecture` heading, no "architecture overview"/"high-level design" phrasing, no "the architecture is …" introduction, no guided code tour. A shape laid out in prose that never names itself as the architecture is not visible to this check, and neither is one kept outside the repository, so this row reports the absence of a re-findable shape document — not evidence that nobody wrote the shape down.
What to do
- Record your significant design decisions. Write one short document per decision, giving its date, the situation, the decision and its consequences, and keep them together with your other design documents. A docs/adr folder with each file named by number and title is the easiest place to find them.
- Add a C4 context/container diagram (Structurizr, PlantUML or Mermaid) or an architecture.md overview.
M3 · Folder & project structure
- Only 1/9 projects share a common root namespace — the code's module identity is inconsistent.
What to do
- Adopt a consistent root-namespace convention (a shared prefix, e.g. Acme.*); short project-file/directory names are fine as long as the RootNamespace is uniform.
M4 · Documentation accuracy
P1 · CI/CD gates
- A CI pipeline exists but no test-runner invocation (your stack's test command, or a test job) was found — changes may merge without the suite running.
What to do
- Run the test suite in CI via an explicit runner step for your stack, and gate merges on it.
P2 · Observability
What to do
- Consider OpenTelemetry tracing/metrics and a health-check endpoint for operability.
P3 · Security & performance tooling
- No static application security testing detected. For this repository's stack, add CodeQL's csharp pack, or a .NET security analyzer package (or `semgrep --config=auto`, which runs on any language) as a CI step. What was searched, so you can tell an absence from a miss: the 310 CI workflow file(s) in this repository, and the scanner and linter configuration checked in beside them. A scan that runs outside CI, one configured in your forge's web UI rather than in a committed file, or a tool whose name is none of those this check carries, is not seen — if that is your case the row is wrong, and saying so is more useful than adding a second scanner.
What to do
- Add a SAST step to CI running what this repository's stack ships: CodeQL's csharp pack, or a .NET security analyzer package — or `semgrep --config=auto`, which runs on any language — so a security regression fails the build instead of landing.
- Enable Dependabot/Renovate or a dependency-review gate.
- Add gitleaks/trufflehog in CI to block PRs that introduce committed secrets.
P6 · Release Hygiene
- No CHANGELOG/HISTORY/RELEASES file — what shipped when isn't easy to reconstruct for support or audit. (Versioning/tagging makes releases traceable, but a changelog records the what.)
What to do
- Keep a changelog (e.g. Keep-a-Changelog) recording what shipped in each release.
X1 · Async correctness
X12 · Unreachable branch
X13 · Undrained process stream
X14 · Bypassable address classification
X15 · Unvalidated length from an untrusted reader
X16 · Unfloored truncation loop
X17 · Uncapped recursion over a caller-supplied document
X18 · Disposal-pattern correctness
X19 · Unrestored process-global state
X20 · Mistyped argument guard
X21 · Side-effecting pattern guard
X22 · Contradicted release guard
X23 · Unguarded diagnostic materialisation
X24 · Document value interpolated into markup unescaped
- `value` is text this program did not author, and it is written into the text of an element through `innerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/jquery-1.10.2.js:6177
- `src.outerHTML` is text this program did not author, and it is written into the text of an element through `outerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/jquery-1.10.2.js:6404
- `src.innerHTML` is text this program did not author, and it is written into the text of an element through `innerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/jquery-1.10.2.js:6412
- `elem.outerHTML` is text this program did not author, and it is written into the text of an element through `innerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/jquery-1.10.2.js:6505
- `wrap[1]+elem.replace(rxhtmlTag,"<$1></$2>")+wrap[2]` is text this program did not author, and it is written into the text of an element through `innerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/jquery-1.10.2.js:6583
- `style` is text this program did not author, and it is written into the text of an element through `innerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/modernizr-2.6.2.js:148
- `cssText` is text this program did not author, and it is written into the text of an element through `innerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/modernizr-2.6.2.js:1094
- `q` is text this program did not author, and it is written into a quoted attribute value through `innerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — src/Client/Web/Scripts/respond.js:33
What to do
- Each finding names the expression, the attribute it lands in and the line. Confirm the value's origin from the read this analyzer traced, then escape it at the sink or validate it against the shape the attribute accepts before interpolating.
X25 · Inert configuration knob
X26 · Unsynchronised callback handoff
X27 · Collection changed while being enumerated
X28 · Index access outside its own emptiness guard
X29 · Per-element action decided by a fixed element
X3 · Exception handling
X30 · Support guard that admits what it rejects
X32 · Type resolved by simple name across every loaded assembly
X4 · Structured logging
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 |
| 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 | 98% | Exemplary | Strongest area. |
| Architecture | 80% | Strong | Solid. |
| Maturity | 39% | Weak — gated by M1, M2 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Readiness | 45% | Weak — gated by P3 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Security | 93% | Exemplary | Solid. |
| Accessibility | 23% | Critical — gated by AC1, AC2, AC6 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
Unscored — 1 check(s) recorded observations but carry no score
- SC1 Supply-chain hygiene — 1 observation(s) recorded · Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
Not evidenced — 2 control(s) we could not find positive evidence for
- P4 Deployment & Rollback — not evidenced — no deploy/rollback/approval signal in the repo; absence of evidence is not evidence of a manual release
- P5 DR & Backup — not evidenced — repo shows no backup/RTO/RPO controls; absence of evidence is not evidence of a working control
Not included — 52 check(s) not relevant to this codebase
- AC4 Keyboard semantics — No interactive element found in the parsed markup — AC4 not applicable here.
- AC5 ARIA correctness — No ARIA usage found in the parsed markup — AC5 not applicable here.
- AX1 Captive dependencies — no DI registrations detected
- AX2 Stateful singletons — no singleton implementations detected
- AX7 Slice cohesion — not applicable — not a vertical-slice architecture
- AX8 Test isolation — no test/production split to check
- AX9 CQS / query purity — no CQRS query handlers detected — query purity is not applicable to this codebase
- AXB1 Runtime evidence locked — no reproducible boot — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- AXB2 Runtime readiness — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- C1 Data Protection — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- C2 Access Controls — No access-control surface detected in the analyzed source — no web/app surface to authorize (no HTTP API or web-UI project) and no authorization code at all (no [Authorize]/policies, no imperative guard methods). Access control is therefore N/A here — this is a library/CLI, which is authorized by its CALLER, not by itself. If this codebase grows request handlers, the dimension reactivates and a default-deny posture is expected then.
- C3 Audit Trail — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- C4 Data Retention — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- C5 Data-Subject Rights — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- D11 Test Reliability — Test reliability not measured — no test run produced results
- D12 Dependency Hygiene — Dependency hygiene not measured
- D16 Bus Factor — single-contributor repository — bus factor is not applicable
- D20 ADR Quality — N/A — ADRs are expected on deployable products with a user-facing host, not consumed libraries; no ADR log is required here.
- D22 Internal API Consistency — No intentionally-exposed public API to evaluate for consistency.
- D23 Boundary Type-Coupling — small single-context codebase (720 production LoC in the .NET projects) — bounded contexts are not needed at this size No bounded-context organisation was detected either — neither a context-shaped layout, nor 2+ sibling source directories each declaring an aggregate root, nor 2+ projects each declaring an entity of their own. Declaring this codebase's bounded contexts (≥2) would let cross-boundary type coupling be assessed. 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"]`.
- D25 ADR Conformance — no ADRs to check
- D27 Navigability — symbol resolution incomplete — navigability not assessed
- D31 IaC & Container Security — No Infrastructure-as-Code or container manifests found (Dockerfile, Docker Compose, Terraform, Kubernetes/Helm, CloudFormation, ARM, Bicep, Ansible); nothing to scan.
- D32 Data Compliance (PII/GDPR) — 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.
- D34 Knowledge Freshness — every production source file is under the ~100-line size floor — too little code for knowledge freshness to be judged
- D36 Supply-chain Provenance & Signing — The CI pipeline builds and tests but publishes no released artifact — no package publish, container push, GitHub release or deployment step. Supply-chain provenance, signing and SBOM attest RELEASED artifacts, so there is nothing to attest here. Add them to the release pipeline when this repo starts shipping artifacts (a published package, a container image, a deployed service or a tagged release). Build integrity and workflow-token hygiene are reported below: they describe what the CI runs and the token it runs with, neither of which is affected by whether the pipeline ships an artifact.
- D37 Vulnerability-disclosure Policy — No vulnerability-disclosure policy file found (SECURITY.md/.markdown/.rst/.txt at root or under .github/.forgejo/.gitea/docs, .well-known/security.txt). A coordinated-disclosure policy may live off-repo, so this is not evidenced rather than failed.
- D39 IL Efficiency — IL not measured — the target did not build
- D40 Network Egress Confinement — No Kubernetes/orchestration workloads found in the repository manifests; network egress policy is a cluster-native control that may live at the platform/firewall layer, so there is nothing to assess here.
- D41 Kernel & Syscall Confinement — No Kubernetes/orchestration workloads found in the repository manifests; seccomp/AppArmor/SELinux confinement is a workload-level control, so there is nothing to assess here.
- D42 Runtime Threat Enforcement — No Kubernetes/orchestration workloads found in the repository manifests; runtime threat-detection and admission-control policy are cluster-level controls, so there is nothing to assess here.
- D44 Platform End-of-Life — Platform end-of-life not assessed — this repository declares no platform this pass reads
- D7 Architectural Integrity — no checkable ADRs and no dependency cycles — architectural integrity not assessed
- D8 Code Coverage — Coverage NOT MEASURED: `--collect:"XPlat Code Coverage"` names a data collector that ships in the `coverlet.collector` package, and this repository wires up none — no test project references it and no runsettings declares one. The absence of coverage here is therefore not evidence about the suite or about our analyzer environment: without a collector, `--collect` produces nothing even from a suite that builds and passes. Add a `coverlet.collector` PackageReference to the test project(s) (or commit the Cobertura/OpenCover/lcov report your CI produces) and real coverage will be measured. It is excluded from the score rather than counted as a near-zero defect.
- DM1 Domain Modelling — not scored — this repository shows only 1 of the 3 signals this lens looks for (a Domain/Aggregates/ValueObjects layer)
- ED1 Event-Driven — not scored — this repository shows only 1 of the 3 signals this lens looks for (3 CQRS handler(s))
- ED5 Idempotency — 3 mutating command handler(s)/consumer(s) detected and 1 handler method(s) walked, but no persistent write was recognised in any of them — idempotency not assessed
- ES1 Event Sourcing — not scored — this repository shows none of the 3 signals this lens looks for
- P12 CI test-gate honesty — Reported, not scored — and nothing was matched here. The coverage check applies to any stack, and the automatic-re-run check to any GitHub-Actions workflow, but the checks for excluded tests, skipped tests and sleep-based synchronisation currently recognise only some ecosystems' test-runner idioms, so on a repository built with another stack the zeros below mean 'not checked', not 'clean'.
- P7 Outbound HTTP resilience — no outbound HTTP usage detected — no HttpClient/IHttpClientFactory/AddHttpClient in source, and no Refit/RestSharp/Flurl.Http namespace or package reference
- P8 Schema migrations — no EF Core usage detected
- P9 Domain vs controller coverage — no coverage report found on disk — produce a coverage report in a standard format (Cobertura — `dotnet test --collect:"XPlat Code Coverage"` with a `coverlet.collector` PackageReference) 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 is assessed only for perf-relevant repos — a packaged library, one that ships benchmarks, or one already using allocation-aware APIs. This repo isn't one, so the Performance lens is not applicable and is excluded from the score.
- PF2 Allocation hygiene — Performance is assessed only for perf-relevant repos — a packaged library, one that ships benchmarks, or one already using allocation-aware APIs. This repo isn't one, so the Performance lens is not applicable and is excluded from the score.
- PF3 Async & latency hygiene — Performance is assessed only for perf-relevant repos — a packaged library, one that ships benchmarks, or one already using allocation-aware APIs. This repo isn't one, so the Performance lens is not applicable and is excluded from the score.
- S1 Web-Security Posture — No web surface detected in the analyzed source — no HTTP API or web-UI project (no controllers/minimal-API endpoints, no Razor/Blazor views) and no web middleware (HTTPS redirection, HSTS, security headers, cookies). Transport security, security headers, secure cookies, CSRF/input-validation and middleware-order controls are therefore N/A here — this is a library/CLI/worker, not a web app. Crypto hygiene was still checked and found nothing to flag. If this codebase becomes web-facing, the dimension reactivates automatically.
- X10 Duplicated predicate — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X2 Cancellation propagation — no async method does work a token could cancel
- X5 Nullable reference types — no NRT-eligible projects
- X6 Hand-rolled structured-format parsing — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X7 Silent fallback defaults — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- 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 — 13 finding(s)
- <select> without a programmatic label src/Client/Web/Scripts/jquery-1.10.2.js:1615
- <select> without a programmatic label src/Client/Web/Scripts/jquery-1.10.2.js:6031
- <input> without a programmatic label src/Client/Web/Scripts/jquery-1.10.2.js:2959
- <input> without a programmatic label src/Client/Web/Scripts/jquery-1.10.2.js:3340
- <html> without a lang src/Client/Web/Views/Shared/Error.cshtml:2
- <html> without a lang src/Client/Web/Views/Shared/_Layout.cshtml:2
- REDACTED
- REDACTED
- Link with no accessible name src/Client/Web/Scripts/jquery-1.10.2.js:2946
- <iframe> without a title src/Client/Web/Scripts/jquery-1.10.2.js:7291
- Namespace dependency cycle src/Core/DomainService/CommandBase/ICommandValidator.cs:1
- WriteOnlyPrivateField test/Infrastructure.Tests/GetCategoriesQueryTests.cs:16
- REDACTED
Serious — 43 finding(s)
- Low XML-doc coverage: Domain src/Core/Domain/Domain.csproj
- Low XML-doc coverage: DomainService src/Core/DomainService/DomainService.csproj
- Low XML-doc coverage: ApplicationService src/Core/ApplicationService/ApplicationService.csproj
- Low XML-doc coverage: Persistence src/Infrastructure/Persistence/Persistence.csproj
- Low XML-doc coverage: Logging src/Infrastructure/Logging/Logging.csproj
- Low XML-doc coverage: Service src/Infrastructure/Service/Service.csproj
- Low XML-doc coverage: Web src/Client/Web/Web.csproj
- Low XML-doc coverage: Bootstrapper src/DependencyResolution/Bootstrapper/Bootstrapper.csproj
- Untrusted text written into markup unescaped src/Client/Web/Scripts/jquery-1.10.2.js:6177
- Untrusted text written into markup unescaped src/Client/Web/Scripts/jquery-1.10.2.js:6404
- Untrusted text written into markup unescaped src/Client/Web/Scripts/jquery-1.10.2.js:6412
- Untrusted text written into markup unescaped src/Client/Web/Scripts/jquery-1.10.2.js:6505
- Untrusted text written into markup unescaped src/Client/Web/Scripts/jquery-1.10.2.js:6583
- Untrusted text written into markup unescaped src/Client/Web/Scripts/modernizr-2.6.2.js:148
- Untrusted text written into markup unescaped src/Client/Web/Scripts/modernizr-2.6.2.js:1094
- Untrusted text written into markup unescaped src/Client/Web/Scripts/respond.js:33
- Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css:2035
- Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css:3232
- Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css:3291
- Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css:3314
- Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css:3410
- Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css:3491
- Focus outline removed in CSS without a :focus replacement src/Client/Web/Content/bootstrap.css:5655
- Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css:2394
- Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css:5151
- Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css:5167
- Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css:5632
- Animation without a prefers-reduced-motion guard src/Client/Web/Content/bootstrap.css:6020
- Page without a main landmark src/Client/Web/Views/Shared/Error.cshtml:2
- Page without a main landmark src/Client/Web/Views/Shared/_Layout.cshtml:2
- REDACTED
- REDACTED
- <object> without a text alternative src/Client/Web/Scripts/jquery-1.10.2.js:6034
- <fieldset> without a <legend> src/Client/Web/Scripts/jquery-1.10.2.js:6032
- Data table has no header cells src/Client/Web/Scripts/jquery-1.10.2.js:3503
- Low contrast colour pair in CSS (3.6:1) src/Client/Web/Content/bootstrap.css:2092
- Accessibility enforcement below the top rung
- Solution references a project that does not exist OnionArchitecture.sln
- REDACTED
- Unstable project Bootstrapper src/DependencyResolution/Bootstrapper/Bootstrapper.csproj:1
- Off the main sequence: Domain src/Core/Domain/Domain.csproj:1
- Coverage not measured — no coverage collector is wired up
- NuGet dependencies are not locked
Minor — 10 finding(s)
- Documentation: no installation or build instructions README.md
- REDACTED
- REDACTED
- Thin README
- No ADRs
- No architecture diagram/doc
- Inconsistent root namespaces
- CI test execution not evidenced
- No SAST
- No changelog
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-4d5001f5e2184c599bc431ddb2bd7780/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-4d5001f5e2184c599bc431ddb2bd7780/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 . | 1 | artifacts/raw/semgrep.json |
| D30 · Dependency Vulnerabilities | nuget | — | nuget: not applicable — nuget: the solution did not restore on the analyzer's .NET SDK (an SDK/target-framework/restore mismatch, common for an older codebase), so there was no restored dependency graph to scan | 0 | — |
| D30 · Dependency Vulnerabilities | osv-scanner | — | osv-scanner --format json --recursive --experimental-disable-plugins transitivedependency/requirements --experimental-disable-plugins dotnet/csproj --experimental-disable-plugins dotnet/nugetcpm . | 4 | artifacts/raw/osv-scanner.json |
| D31 · IaC & Container Security | trivy | — | trivy: not applicable — No Infrastructure-as-Code or container manifests found (Dockerfile, Docker Compose, Terraform, Kubernetes/Helm, CloudFormation, ARM, Bicep, Ansible); nothing to scan. | 0 | — |
| 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. | 0 | — |
| D37 · Vulnerability-disclosure Policy | disclosure | — | disclosure: not applicable — No vulnerability-disclosure policy file found (SECURITY.md/.markdown/.rst/.txt at root or under .github/.forgejo/.gitea/docs, .well-known/security.txt). A coordinated-disclosure policy may live off-repo, so this is not evidenced rather than failed. | 0 | — |
| D40 · Network Egress Confinement | runtime-hardening | — | runtime-hardening: not applicable — No Kubernetes/orchestration workloads found in the repository manifests; network egress policy is a cluster-native control that may live at the platform/firewall layer, so there is nothing to assess here. | 0 | — |
| D41 · Kernel & Syscall Confinement | runtime-hardening | — | runtime-hardening: not applicable — No Kubernetes/orchestration workloads found in the repository manifests; seccomp/AppArmor/SELinux confinement is a workload-level control, so there is nothing to assess here. | 0 | — |
| D42 · Runtime Threat Enforcement | runtime-hardening | — | runtime-hardening: not applicable — No Kubernetes/orchestration workloads found in the repository manifests; runtime threat-detection and admission-control policy are cluster-level controls, so there is nothing to assess here. | 0 | — |
| D43 · Malicious Dependencies | osv-scanner | — | osv-scanner --format json --recursive --experimental-disable-plugins transitivedependency/requirements --experimental-disable-plugins dotnet/csproj --experimental-disable-plugins dotnet/nugetcpm . | 0 | artifacts/raw/osv-scanner.json |