Executive summary
The system holds an adequate standing with a health score of 52%, indicating a workable asset that carries real operational risk. While the codebase is small and relatively inexpensive to rebuild, its current state imposes a significant velocity tax on every change. This drag on delivery speed compounds as the team grows, making immediate attention to foundational quality and operational readiness essential for sustainable growth.
The most pressing concern is the high cost of change driven by poor code quality. Complexity and duplication in the core logic act as a hidden tax, plausibly increasing effort by 15–34% for every modification. This inefficiency creates a recurring annual cost that far exceeds the one-time investment required to fix the underlying issues. Addressing this technical debt offers a rapid return on investment, paying for itself within months by freeing up engineering capacity for new features rather than maintenance.
A second critical theme is the gap between development and production readiness. With a readiness score of 47%, the system lacks the safety nets needed for reliable operation. The absence of automated security checks in the build pipeline exposes the business to potential vulnerabilities slipping into production. Strengthening these operational guardrails is the highest-leverage move, as it provides immediate protection against outages and security regressions with minimal effort.
The codebase is genuinely small, with a rebuild cost of approximately €41,000, which limits the total financial exposure. However, the concentration of risk in the weakest areas means that neglecting these gaps could lead to disproportionate delays and defects. The assessment is partial, as several key lenses such as domain modeling and performance were not measured, leaving some aspects of the system’s health unknown.
Focus first on integrating automated security scanning into the continuous integration pipeline. This single action secures the build process against regressions and establishes a baseline for operational safety. It is the most effective step to reduce immediate risk and improve confidence in the system’s reliability, allowing the team to address deeper code quality issues with greater stability.
Raise Readiness 47 → 70 (the Healthy floor) ⇒ headline 52 → ~56.
New since the last scan (83+)
- D1 · debug.initDebug (cyclomatic 104) debug/src/debug.js
- D1 · diff (cyclomatic 94) src/diff/index.js
- D1 · index.diffElementNodes (cyclomatic 67) src/diff/index.js
- D1 · render.handleDomVNode (cyclomatic 45) compat/src/render.js
- D1 · initDebug::diffed (cyclomatic 44) debug/src/debug.js
- D1 · props.setProperty (cyclomatic 40) src/diff/props.js
- D1 · children.constructNewChildrenArray (cyclomatic 36) src/diff/children.js
- D1 · initDebug::_diff (cyclomatic 24) debug/src/debug.js
- D1 · children.diffChildren (cyclomatic 18) src/diff/children.js
- D1 · children.insert (cyclomatic 18) src/diff/children.js
- D1 · index.jsxAttr (cyclomatic 17) jsx-runtime/src/index.js
- D1 · children.findMatchingIndex (cyclomatic 16) src/diff/children.js
- D1 · index.unmount (cyclomatic 16) src/diff/index.js
- D2 · diff (cognitive 232) src/diff/index.js
- D2 · initDebug::diffed (cognitive 67) debug/src/debug.js
- D2 · debug.initDebug (cognitive 31) debug/src/debug.js
- D2 · children.insert (cognitive 30) src/diff/children.js
- D2 · initDebug::_diff (cognitive 29) debug/src/debug.js
- D2 · useReducer (cognitive 28) hooks/src/index.js
- D2 · index.unmount (cognitive 26) src/diff/index.js
- D2 · children.diffChildren (cognitive 24) src/diff/children.js
- D2 · index.options.unmount (cognitive 21) hooks/src/index.js
- D2 · suspense.detachedClone (cognitive 19) compat/src/suspense.js
- D2 · catch-error._catchError (cognitive 18) src/diff/catch-error.js
- D2 · flushAfterPaintEffects (cognitive 17) hooks/src/index.js
- D2 · render.options.vnode (cognitive 16) compat/src/render.js
- D2 · babel-plugin-fast-rest.fastRestTransform (cognitive 16) scripts/babel-plugin-fast-rest.mjs
- D2 · initSuspenseHooks (cognitive 16) compat/src/suspense.js
- D3 · FunctionTooLong: debug.initDebug debug/src/debug.js
- D3 · FunctionTooLong: index.diff src/diff/index.js
- D3 · FunctionTooLong: index.diffElementNodes src/diff/index.js
- D3 · FunctionTooLong: children.constructNewChildrenArray src/diff/children.js
- D8 · Coverage not measured — JavaScript/TypeScript suite
- D10 · No assertions: should handle a promise thrown from a layout effect compat/test/browser/suspense.test.jsx
- D10 · No assertions: should not crash when effect throws and component is unmounted by render(null) during flush hooks/test/browser/useEffect.test.jsx
- D15 · Hotspot: jsx-runtime/src/index.js jsx-runtime/src/index.js
- D17 · TodoComment test/ts/Component.test.tsx
- D28 · REDACTED
- D29 · REDACTED
- D29 · REDACTED
- D29 · REDACTED
- D29 · REDACTED
- D29 · REDACTED
- D30 · REDACTED
- D36 · REDACTED
- D36 · REDACTED
- R10 · Duplicated block with local edits (14 matched lines × 2 locations) compat/client.js
- R10 · Duplicated block (7 lines × 2 locations) src/create-element.js
- R11 · Boundary violation [package:relative-cross-package] compat/src/index.d.ts
- R11 · Boundary violation [package:relative-cross-package] compat/src/internal.d.ts
- R11 · Boundary violation [package:relative-cross-package] compat/src/internal.d.ts
- R11 · Boundary violation [package:relative-cross-package] debug/src/index.d.ts
- R11 · Boundary violation [package:relative-cross-package] debug/src/internal.d.ts
- R11 · Boundary violation [package:relative-cross-package] hooks/src/index.d.ts
- R11 · Boundary violation [package:relative-cross-package] hooks/src/internal.d.ts
- R11 · Boundary violation [package:relative-cross-package] jsx-runtime/src/index.d.ts
- R11 · Boundary violation [package:relative-cross-package] jsx-runtime/src/index.d.ts
- R11 · Boundary violation [package:relative-cross-package] jsx-runtime/src/index.d.ts
- R2 · Complex function diff (cyclomatic 92, cognitive 224) src/diff/index.js
- R2 · Complex function diffed (cyclomatic 44, cognitive 67) debug/src/debug.js
- R2 · Complex function _diff (cyclomatic 24, cognitive 29) debug/src/debug.js
- R2 · Complex function insert (cyclomatic 18, cognitive 29) src/diff/children.js
- R2 · Complex function diffChildren (cyclomatic 18, cognitive 24) src/diff/children.js
- R2 · Complex function jsxAttr (cyclomatic 17, cognitive 23) jsx-runtime/src/index.js
- R2 · Complex function unmount (cyclomatic 16, cognitive 26) src/diff/index.js
- R2 · Complex function findMatchingIndex (cyclomatic 16, cognitive 17) src/diff/children.js
- R2 · Complex function serializeDomTree (cyclomatic 15, cognitive 28) test/_util/helpers.jsx
- R2 · Complex function loadMangle (cyclomatic 12, cognitive 11) scripts/build.mjs
- R2 · Complex function (anonymous) (cyclomatic 11, cognitive 10) test/browser/focus.test.jsx
- R3 · Large Files
- R4 · No test reaches this file config/compat-entries.js
- R4 · No test reaches this file compat/client.mjs
- R4 · No test reaches this file compat/scheduler.mjs
- R4 · No test reaches this file compat/jsx-dev-runtime.js
- R4 · No test reaches this file compat/jsx-dev-runtime.mjs
- R4 · No test reaches this file compat/jsx-runtime.js
- R4 · No test reaches this file compat/jsx-runtime.mjs
- R4 · No test reaches this file compat/test-utils.js
- R7 · Dead file (~1 LoC) compat/test-utils.js
- R7 · Dead file (~1 LoC) compat/test-utils.mjs
- R9 · Import cycle (2 files) compat/src/index.js
- R9 · Import cycle (4 files) src/component.js
- P12 · Coverage collected but not gated
Rebuild cost & value ~ Modeled — €13,000–€68,000
| Cost to rebuild | €13,000–€68,000 |
|---|---|
| Domain complexity | Standard |
| Quality factor | 0.7× (at 52% quality) |
| Size & shape | Small · effort split not classified (source measured from disk; the effort-tier breakdown is a C#-only syntax walk) |
This codebase represents roughly ~0.3 person-years of build effort (about ~€41,000 to rebuild). Its weakest lens is Readiness at 47% — the part of that asset most exposed by the findings below.
Top priorities
Diagnosis — what's actually going on
Architecture — module dependency matrix
| depends on → | 1 (global) | 2 compat | 3 compat.src | 4 compat.test | 5 compat.test.browser | 6 compat.test.browser.suspense-utils | 7 compat.test.ts.lazy | 8 compat.test.ts.memo | 9 compat.test.ts.react-default | 10 compat.test.ts.react-star | 11 compat.test.ts.suspense | 12 debug.src | 13 demo | 14 demo.context | 15 demo.devtools | 16 demo.fragments | 17 demo.key_bug | 18 demo.logger | 19 demo.mobx | 20 demo.nested-suspense | 21 demo.people | 22 demo.people.profile | 23 demo.profiler | 24 demo.pythagoras | 25 demo.redux | 26 demo.reduxUpdate | 27 demo.reorder | 28 demo.spiral | 29 demo.suspense | 30 demo.suspense-router | 31 demo.todo | 32 devtools | 33 devtools.src | 34 hooks | 35 test.shared.isValidElementTests | 36 test.ts.jsx-namespace_test-d | 37 test.ts.preact | 38 compat.test.ts | 39 test.ts | 40 test.ts.refs |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 1 (global) | ||||||||||||||||||||||||||||||||||||||||
| 2 compat | ||||||||||||||||||||||||||||||||||||||||
| 3 compat.src | ||||||||||||||||||||||||||||||||||||||||
| 4 compat.test | ||||||||||||||||||||||||||||||||||||||||
| 5 compat.test.browser | ||||||||||||||||||||||||||||||||||||||||
| 6 compat.test.browser.suspense-utils | ||||||||||||||||||||||||||||||||||||||||
| 7 compat.test.ts.lazy | ||||||||||||||||||||||||||||||||||||||||
| 8 compat.test.ts.memo | ||||||||||||||||||||||||||||||||||||||||
| 9 compat.test.ts.react-default | ||||||||||||||||||||||||||||||||||||||||
| 10 compat.test.ts.react-star | ||||||||||||||||||||||||||||||||||||||||
| 11 compat.test.ts.suspense | ||||||||||||||||||||||||||||||||||||||||
| 12 debug.src | ||||||||||||||||||||||||||||||||||||||||
| 13 demo | ||||||||||||||||||||||||||||||||||||||||
| 14 demo.context | ||||||||||||||||||||||||||||||||||||||||
| 15 demo.devtools | ||||||||||||||||||||||||||||||||||||||||
| 16 demo.fragments | ||||||||||||||||||||||||||||||||||||||||
| 17 demo.key_bug | ||||||||||||||||||||||||||||||||||||||||
| 18 demo.logger | ||||||||||||||||||||||||||||||||||||||||
| 19 demo.mobx | ||||||||||||||||||||||||||||||||||||||||
| 20 demo.nested-suspense | ||||||||||||||||||||||||||||||||||||||||
| 21 demo.people | ||||||||||||||||||||||||||||||||||||||||
| 22 demo.people.profile | ||||||||||||||||||||||||||||||||||||||||
| 23 demo.profiler | ||||||||||||||||||||||||||||||||||||||||
| 24 demo.pythagoras | ||||||||||||||||||||||||||||||||||||||||
| 25 demo.redux | ||||||||||||||||||||||||||||||||||||||||
| 26 demo.reduxUpdate | ||||||||||||||||||||||||||||||||||||||||
| 27 demo.reorder | ||||||||||||||||||||||||||||||||||||||||
| 28 demo.spiral | ||||||||||||||||||||||||||||||||||||||||
| 29 demo.suspense | ||||||||||||||||||||||||||||||||||||||||
| 30 demo.suspense-router | ||||||||||||||||||||||||||||||||||||||||
| 31 demo.todo | ||||||||||||||||||||||||||||||||||||||||
| 32 devtools | ||||||||||||||||||||||||||||||||||||||||
| 33 devtools.src | ||||||||||||||||||||||||||||||||||||||||
| 34 hooks | ||||||||||||||||||||||||||||||||||||||||
| 35 test.shared.isValidElementTests | ||||||||||||||||||||||||||||||||||||||||
| 36 test.ts.jsx-namespace_test-d | ||||||||||||||||||||||||||||||||||||||||
| 37 test.ts.preact | ||||||||||||||||||||||||||||||||||||||||
| 38 compat.test.ts | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 39 test.ts | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 40 test.ts.refs | 1 |
38→7 compat.test.ts depends on compat.test.ts.lazy✕
- Type pairs
- 1 distinct (type in compat.test.ts → type in compat.test.ts.lazy) reference.
- Which types
compat.test.ts.suspense→compat.test.ts.lazy.LazyProps
- Direction
- compat.test.ts uses compat.test.ts.lazy. Changing compat.test.ts.lazy can break compat.test.ts, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
38→8 compat.test.ts depends on compat.test.ts.memo✕
- Type pairs
- 1 distinct (type in compat.test.ts → type in compat.test.ts.memo) reference.
- Which types
compat.test.ts.memo→compat.test.ts.memo.MemoProps
- Direction
- compat.test.ts uses compat.test.ts.memo. Changing compat.test.ts.memo can break compat.test.ts, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
39→36 test.ts depends on test.ts.jsx-namespace_test-d✕
- Type pairs
- 1 distinct (type in test.ts → type in test.ts.jsx-namespace_test-d) reference.
- Which types
test.ts.preact→test.ts.jsx-namespace_test-d.Element
- Direction
- test.ts uses test.ts.jsx-namespace_test-d. Changing test.ts.jsx-namespace_test-d can break test.ts, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
39→37 test.ts depends on test.ts.preact✕
- Type pairs
- 1 distinct (type in test.ts → type in test.ts.preact) reference.
- Which types
test.ts.preact→test.ts.preact.DummerComponentProps
- Direction
- test.ts uses test.ts.preact. Changing test.ts.preact can break test.ts, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
40→35 test.ts.refs depends on test.shared.isValidElementTests✕
- Type pairs
- 1 distinct (type in test.ts.refs → type in test.shared.isValidElementTests) reference.
- Which types
test.ts.refs.CreateRefComponent→test.shared.isValidElementTests.Bar
- Direction
- test.ts.refs uses test.shared.isValidElementTests. Changing test.shared.isValidElementTests can break test.ts.refs, not the reverse.
- Position
- Below the diagonal — points down the layering, which is what you want.
At a glance — Code Health
At a glance — Architecture
At a glance — Maturity
At a glance — Readiness
At a glance — Security
Security & Compliance — OWASP Top-10 mapping
| OWASP category | Findings | Severity |
|---|---|---|
| A03:2021 — Injection | 5 | REDACTED / Critical |
| A02:2021 — Cryptographic Failures | 2 | REDACTED / Critical |
| A06:2021 — Vulnerable & Outdated Components | 1 | REDACTED / Critical |
Roadmap
First, integrate static analysis into the CI pipeline to block security regressions before they land. Next, expand test coverage by importing unreached production modules and migrate remaining JavaScript files to TypeScript to ensure type safety. Finally, maintain a changelog for release hygiene and split large files into smaller, focused modules to improve code manageability.
| Do this | Helps | Effort | Dimension |
|---|---|---|---|
| Add a SAST step to CI running what this repository's stack ships: CodeQL's javascript-typescript pack, `semgrep --config=p/typescript`, or eslint-plugin-security — so a security regression fails the build instead of landing. | +6.4 pts | Medium | Security & performance tooling |
| Add tests that import the unreached modules (directly or through their public entry). | +6.4 pts | Medium | Test Coverage |
| Keep a changelog (e.g. Keep-a-Changelog) recording what shipped in each release. | +6.2 pts | Medium | Release Hygiene |
| Migrate the remaining .js/.jsx files to TypeScript. | +4.0 pts | Medium | Type Safety |
| Split each oversized file along the responsibilities already in it, into smaller focused modules in the same package. | +3.3 pts | Medium | Large Files |
| Resolve the 1 Documentation finding(s) in Documentation Quality — start with README.md. | +1.6 pts | Low | Documentation Quality |
| Fix the listed violations: import through public entries (package exports / slice index), never reach into another layer or package's internals. | +2.9 pts | Medium | Import Boundaries |
| Break each cycle by extracting the shared piece into a module both sides can import. | +2.7 pts | Medium | Circular Imports |
File quality
| File | Score | Band | Worst signal |
|---|---|---|---|
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| src/diff/index.js | 7.0 | Mixed | Cyclomatic Complexity: diff (cyclomatic 94) |
| src/diff/children.js | 7.0 | Mixed | Cyclomatic Complexity: children.constructNewChildrenArray (cyclomatic 36) |
| debug/src/debug.js | 7.0 | Mixed | Cyclomatic Complexity: debug.initDebug (cyclomatic 104) |
| REDACTED | 7.2 | Mixed | Secrets (history): REDACTED: REDACTED |
| REDACTED | 7.2 | Mixed | Static Analysis (SAST): REDACTED: REDACTED |
| REDACTED | 7.2 | Mixed | Dependency Vulnerabilities: REDACTED CVE: REDACTED |
| compat/src/render.js | 7.4 | Mixed | Cyclomatic Complexity: render.handleDomVNode (cyclomatic 45) |
| hooks/src/index.js | 7.4 | Mixed | Cognitive Complexity: useReducer (cognitive 28) |
| src/diff/props.js | 7.8 | Mixed | Cyclomatic Complexity: props.setProperty (cyclomatic 40) |
| jsx-runtime/src/index.js | 7.8 | Mixed | Cyclomatic Complexity: index.jsxAttr (cyclomatic 17) |
| compat/src/suspense.js | 7.8 | Mixed | Cognitive Complexity: suspense.detachedClone (cognitive 19) |
| test/ts/Component.test.tsx | 8.2 | Near-clean | Explicit Debt: TodoComment |
| src/diff/catch-error.js | 8.5 | Near-clean | Cognitive Complexity: catch-error._catchError (cognitive 18) |
| scripts/babel-plugin-fast-rest.mjs | 8.5 | Near-clean | Cognitive Complexity: babel-plugin-fast-rest.fastRestTransform (cognitive 16) |
| compat/test/browser/suspense.test.jsx | 8.5 | Near-clean | Test Quality: No assertions: should handle a promise thrown from a layout effect |
| hooks/test/browser/useEffect.test.jsx | 8.5 | Near-clean | Test Quality: No assertions: should not crash when effect throws and component is unmounted by render(null) during flush |
| 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. 37 of 40 evaluated dimensions are computed purely by tools and static analysis (confidence 1.0); 3 documentation/naming judgement(s) are LLM-assisted and labelled advisory. Overall confidence is 0.9 — 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 — 40 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, 124 of 157 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
- D22 Internal API Consistency — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. D22 identifies the intentionally-exposed surface from `IsPackable` and `.Contracts` project names, MSBuild conventions read off the loaded project set. This target exposed no such projects, so the probe never ran; this says nothing about whether the repository has a public API. This repository commits no C#/VB source at all, so there was never an MSBuild project set to read these conventions off. That is OUR side and it is a COLLECTOR gap, not an environment fault: D22 has no public-API collector for any other ecosystem, and the remedy is to write one — no change to the scan image can close it.
- 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, 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 (go.mod, a Gemfile's ruby directive, a Dockerfile) is simply not read here yet.
- AX3 Project dependency cycles — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over which project references which — facts that live in .csproj references. This repository either commits no MSBuild project at all (its C# may still have been parsed as syntax-only projects, which carry no references between them) or its projects failed to load, so there was no graph to read. That is a gap in this analyzer's reach — not a finding that the repository is free of what this check looks for.
- AX4 Dependency direction — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over the direction each project reference points — facts that live in .csproj references. This repository either commits no MSBuild project at all (its C# may still have been parsed as syntax-only projects, which carry no references between them) or its projects failed to load, so there was no graph to read. That is a gap in this analyzer's reach — not a finding that the repository is free of what this check looks for.
- AX6 Interface segregation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over the public interfaces this run's compilations declare, and none was loaded, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- AX8 Test isolation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is computed over which projects are test projects, and what they reference — facts that live in .csproj references. This repository either commits no MSBuild project at all (its C# may still have been parsed as syntax-only projects, which carry no references between them) or its projects failed to load, so there was no graph to read. That is a gap in this analyzer's reach — not a finding that the repository is free of what this check looks for.
- C1 Data Protection — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These personal data controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks personal data controls.
- C2 Access Controls — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These authorization controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks authorization controls.
- C3 Audit Trail — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These audit controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks audit controls.
- C4 Data Retention — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These retention controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks retention controls.
- C5 Data-Subject Rights — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These data-subject rights controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks data-subject rights controls.
- DM1 Aggregate boundaries — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: a class holding another class reads the same whether the inner type is an aggregate or a value object, so this cannot be decided from source without guessing — reported as guidance rather than measured.
- DM2 Strongly-typed ids — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: branded ids (`type Id = string & { __brand }`) are an uncommon idiom, so a bare-string id is not on its own evidence of a missing typed id — reported as guidance rather than measured.
- DM3 Integration-event coupling — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: a domain type used across packages is indistinguishable in source from a deliberate shared-kernel package, so this is reported as guidance rather than measured.
- DM4 Rich vs anemic model — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: telling a rich domain entity from an anemic data holder needs the behaviour a source-only read cannot always attribute (components, DTOs and readonly value objects are all legitimately data-shaped), so this is reported as guidance rather than measured.
- DM5 Encapsulated state — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: the language already steers state behind #private/private/readonly, so a mutable public field is rare enough that we report this as guidance rather than measuring it.
- DM7 Repository granularity — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. Not scored for TypeScript: deciding whether a repository belongs to an aggregate root needs the aggregate structure, which source alone does not state — reported as guidance rather than measured.
- ED5 Idempotency — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check finds retry-prone mutations by walking the repository's declared types, and NONE was loaded on this run, so it had nothing to look at. That is a limit of the analyzer's reach — it reads .NET projects — not a finding that this repository has no command handlers or message consumers.
- GD1 Unfinished & placeholder code — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- IC1 Incompleteness & stubs — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- 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 none applies: this repository publishes a library and deploys nothing (no container, IaC or deployment manifest), so it holds no runtime state of its own. Any data-access dependency it declares is the store it is a CLIENT for, not one it operates. The DR control belongs to whoever runs that data.
- S1 Web-Security Posture — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. These web-security controls are read from declarative annotations, request middleware, entity/column names and guard methods in a C# source model, and none was loaded on this run, so there was nothing to gather. That is a gap in this analyzer's language reach — not a finding that the repository lacks web-security controls.
- X1 Async correctness — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X12 Unreachable branch — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X13 Undrained process stream — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X14 Bypassable address classification — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X15 Unvalidated length from an untrusted reader — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X16 Unfloored truncation loop — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X17 Uncapped recursion over a caller-supplied document — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X18 Disposal-pattern correctness — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X19 Unrestored process-global state — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X2 Cancellation propagation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X20 Mistyped argument guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X21 Side-effecting pattern guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X22 Contradicted release guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X23 Unguarded diagnostic materialisation — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X24 Document value interpolated into markup unescaped — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X25 Inert configuration knob — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X26 Unsynchronised callback handoff — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X27 Collection changed while being enumerated — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X28 Index access outside its own emptiness guard — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X29 Per-element action decided by a fixed element — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X3 Exception handling — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X30 Support guard that admits what it rejects — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X32 Type resolved by simple name across every loaded assembly — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X4 Structured logging — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
- X5 Nullable reference types — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This check is implemented over the C# syntax tree, and no C# was loaded on this run, so it had nothing to read. That is a gap in this analyzer's language reach — not a finding that the repository is free of what this check looks for.
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.
- 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.
- D11 Test Reliability: Flakiness is inferred from history/markers — Watchdog runs the suite once (for coverage), not the repeated runs under varied conditions that reveal nondeterminism, so a flaky test never recorded as failing is invisible here.
- D12 Dependency Hygiene: Dependency health reads manifests and lockfiles — a vulnerability in a vendored/copied dependency, or risk from how a dependency is actually used, is outside this view.
- 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.
- D16 Bus Factor: Bus-factor is a time-decayed model of commit attribution (who has recently, repeatedly worked a file), not comprehension — pairing, review and reading-without-committing spread knowledge it can't see; bot commits and shared accounts still distort it.
- D17 Explicit Debt: Acknowledged-debt signals (TODO/FIXME, suppressions, dead code) are textual — undocumented debt that nobody marked, and debt that lives in design rather than annotations, is invisible. Committed machine-written code (scaffolded migrations, designer/codegen output, generated stubs) is excluded — it is never the team's dead code to delete.
- D19 Documentation Quality: Documentation quality is judged by an LLM over a bounded sample of docs — it reads what is written, not whether the docs match the running system, and it is advisory, not a measurement. Its critique rows are drawn from a closed category vocabulary and each row means the same thing in every run, so two scans can be compared row by row; the SET that fires is still a sample, and does not repeat exactly. Measured on one frozen input, six scans at one engine SHA: 2-5 critique rows per scan, 8 distinct rows across the six, 3 of those 8 seen in only one scan. So a D19 row is evidence about the documentation, but a COUNT of D19 rows is not a quantity — never read a change in it as an improvement or a regression.
- 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.
- 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.
- D34 Knowledge Freshness: Freshness is decayed commit RECENCY, not comprehension — code read often but rarely committed reads as orphaned, and stable code that genuinely needs no changes is penalised the same as forgotten code; bot/squash commits distort it like the bus factor.
- D35 Change Coupling: Change coupling is co-change in COMMITS — files split across separate commits, or coupled only through a shared config/build step, read as uncoupled, and a sweeping commit (rename/format) is excluded so it doesn't couple everything. It shows that files change together, not WHY: a high coupling can be a healthy cohesive pair as readily as a hidden leak.
- 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.
- AX10 Code composition: Role is inferred from namespace/folder convention, not semantics — a domain concept living in a folder named "Services" reads as application, and the split is lines-of-code, not business value. The business-logic-share score is a SOFT, FLOORED signal: it contributes to the Architecture lens but is floored at the Critical gate, so an infrastructure-heavy design (a gateway, an ETL, a driver) is legitimately low without being nuked to zero.
- M4 Documentation accuracy: Onboarding quality is an LLM read of the docs/setup present — it cannot run the onboarding or measure how long a real new joiner takes; the verdict is sampled and advisory.
- P4 Deployment & Rollback: Approval/branch-protection rules live in repository settings the scan cannot see — only their in-repo evidence (config files, workflows) is checked, so a control enforced purely in the host's settings reads as "not evidenced".
- 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
23 method(s) exceeded the cyclomatic complexity threshold of 15; the worst was debug.initDebug at 104.
What to do
- Resolve the 2 initDebug finding(s) in Cyclomatic Complexity — start with debug.js (2). — One of this dimension's main actionable groups (2 warning-level).
- Resolve the 1 debug.initDebug (cyclomatic 104) finding(s) in Cyclomatic Complexity — start with debug.js. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 diff (cyclomatic 94) finding(s) in Cyclomatic Complexity — start with index.js. — One of this dimension's main actionable groups (1 warning-level).
- Enforce Cyclomatic Complexity in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D2 · Cognitive Complexity
36 method(s) exceeded the cognitive complexity threshold of 15; the worst was diff at 232.
What to do
- Resolve the 2 initDebug finding(s) in Cognitive Complexity — start with debug.js (2). — One of this dimension's main actionable groups (2 warning-level).
- Resolve the 1 diff (cognitive 232) finding(s) in Cognitive Complexity — start with index.js. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 diffElementNodes (cognitive 106) finding(s) in Cognitive Complexity — start with index.js. — One of this dimension's main actionable groups (1 warning-level).
- Enforce Cognitive Complexity in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D3 · God Classes
4 over-large unit(s) detected — types, modules or files that carry too much.
What to do
- Resolve the 4 FunctionTooLong finding(s) in God Classes — start with index.js (2), debug.js, children.js. — One of this dimension's main actionable groups (4 warning-level).
- Enforce God Classes in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D9 · Test Distribution
1315 test methods: 1315 unit, 0 integration, 0 BDD, 0 e2e. The JavaScript/TypeScript suite contributes 1315 `it`/`test` case(s) across 105 test file(s) declaring at least one; its tier split is read from package names and paths only.
D10 · Test Quality
24 skipped (24 with a documented reason), 2 zero-assertion, no mocking-framework packages referenced (hand-written doubles or no mocking) across 1315 tests.
What to do
- Resolve the 2 No assertions finding(s) in Test Quality — start with suspense.test.jsx, useEffect.test.jsx. — One of this dimension's main actionable groups (2 warning-level).
- Enforce Test Quality in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D11 · Test Reliability
0 flaky across 1 measured tier(s). JavaScript/TypeScript (vitest via `pnpm install --frozen-lockfile --ignore-scripts` in the repository root (26 tests)): measured (0 flaky).
D12 · Dependency Hygiene
15 outdated direct production npm dependency(ies) of 16 graded, 0 pinning defect(s), across 2 package.json (1 of them the product) and 1 committed lockfile(s). Development dependencies are deliberately not graded: this dimension grades what SHIPS. Whether any of these packages is DEPRECATED or UNMAINTAINED is not graded — registry.npmjs.org's latest-version answer carries neither, and release age does not stand in for a maintenance status. Whether any is UNUSED is not graded either: that is a source question, and the frontend dependency lens (R8) answers it in this same run. Known CVEs in this dependency graph are D30's question, read from the manifest there.
D13 · REDACTED Scanning
REDACTED scan ran and found no leaked secrets.
D14 · License Compliance
0 of 16 shipped npm package(s) use a banned license. Licences were resolved from registry.npmjs.org over the 16 production dependency(ies) this repository's committed lockfile resolves, across 1 product package.json manifest(s). Its 28 `devDependencies` declaration(s) are excluded: a consumer installs none of them. ★ DEPTH: this is the DIRECT production set the lockfile resolves, NOT the transitive closure — only one of the four lock dialects this pass reads states a full graph, so a banned licence pulled in only by a dependency's OWN dependencies is outside this verdict, exactly as the JVM arm's declaration-site verdict is. ★ Each licence is the one the registry publishes for the package's CURRENT release rather than for the pinned version, which is the same caveat the Hex and RubyGems arms carry. This repository publishes itself under MIT, which is its own choice and is not judged here.
D15 · Churn × Complexity Hotspots
Top hotspots: src/diff/index.js (29×93=2697); src/diff/children.js (22×36=792); compat/src/render.js (12×45=540)
What to do
- Resolve the 5 Hotspot finding(s) in Churn × Complexity Hotspots — start with index.js (2), children.js, render.js. — One of this dimension's main actionable groups (5 warning-level).
D16 · Bus Factor
3 source file(s) have their living knowledge concentrated in one author (≥90% of recent, decayed contribution). The largest is src/diff/children.js. Counted over 15 of the 51 production source files in this repository: the rest are under the ~2,400-byte size floor this dimension measures over.
What to do
- Resolve the 1 Off-boarding risk finding(s) in Bus Factor. — One of this dimension's main actionable groups (1 recommendation-level).
D17 · Explicit Debt
1 deducted task-comment markers across 319 LoC (0.0/KLoC) → score 9.9. Task comments only: this repository's language is read without a compiler, so D17's suppression, dead-code and commented-out-code arms did not run and this score counts fewer marker kinds than a .NET repository's would.
What to do
- Resolve the 1 TodoComment finding(s) in Explicit Debt — start with Component.test.tsx. — One of this dimension's main actionable groups (1 warning-level).
- Enforce Explicit Debt in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D19 · Documentation Quality
The README is a strong single document that describes Preact as the 4kB alternative to React with the same modern API and lists its features (ES6 Class, hooks, VDOM, preact/compat alias, hydration, SSR, diff algorithm, async rendering). It also provides links to the Preact website, npm badge, Slack community, OpenCollective backers/sponsors, coveralls, and gzip/brotli size. The document is clipped mid-Getting Started section (the note about not needing ES2015) before any installation or usage content appears; the outline lists Tutorial: Building UI with Preact as present but unshown in the visible text.
What to do
- 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 0 sampled symbols.
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)
5 finding(s): 0 critical, 5 high, 0 medium, 0 low. Separately, one or more rules could not re-parse an embedded snippet in 1 file(s) (e.g. a workflow `run:` block read as shell). Those files WERE scanned and their other rows are unaffected; only those rules' view of those snippets is missing.
What to do
- Resolve the 5 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED (4), REDACTED. — One of this dimension's main actionable groups (5 issue-level).
D30 · Dependency Vulnerabilities
1 finding(s): 0 critical, 1 high, 0 medium, 0 low.
What to do
- Resolve the 1 REDACTED CVE finding(s) in Dependency Vulnerabilities — start with REDACTED. — One of this dimension's main actionable groups (1 issue-level).
D34 · Knowledge Freshness
Every significant source file has living knowledge — recently and meaningfully worked. Counted over 15 of the 51 production source files in this repository: the rest are under the ~2,400-byte size floor this dimension measures over.
D35 · Change Coupling
No strong hidden change-coupling between production files.
D36 · Supply-chain Provenance & Signing
3/4 supply-chain integrity signals present (provenance, signing, SBOM, pinned actions).
What to do
- Resolve the 1 REDACTED finding(s) in Supply-chain Provenance & Signing. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 REDACTED finding(s) in Supply-chain Provenance & Signing. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 REDACTED finding(s) in Supply-chain Provenance & Signing. — One of this dimension's main actionable groups (1 recommendation-level).
D43 · Malicious Dependencies
No dependency in any ecosystem this repository declares is published as malicious.
Frontend & cross-cutting dimensions
AX10 · Code composition
What to do
- The domain core is a small share of production code, but most of the rest matched no layer vocabulary at all — so this is not yet an anemic-domain finding. The namespace/path convention could not place that code, which makes the composition above a statement about the naming, not about the design. Name the layers (or check that the repository's conventions differ from the ones this check knows) before reading a thin domain into it.
M1 · Documentation (README)
What to do
- Add a 'Testing' section to the root README — how to run the test suite.
- Add an 'Architecture' / 'How it works' section to the root README — the high-level shape.
- Add a README to the 7 of 7 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.
What to do
- Record significant decisions one document per decision — dated, stating the context, the decision and its consequences — and keep them together wherever your design docs already live (a conventional `docs/adr/` tree, with each file named `NNNN-title` in whatever markup those docs already use, is the most discoverable form).
M3 · Folder & project structure
M4 · Documentation accuracy
- README omits the jsx-runtime project (the JSX runtime) — reported by the model that read the README against this repository; no term search was run for this one, so nothing here has been checked against the tree. Treat it as a reading to confirm, not as a measured contradiction: verify it against the code before acting on it, and if the footprint it describes does exist, this row is wrong.
- README omits the hooks project — reported by the model that read the README against this repository; no term search was run for this one, so nothing here has been checked against the tree. Treat it as a reading to confirm, not as a measured contradiction: verify it against the code before acting on it, and if the footprint it describes does exist, this row is wrong.
- README omits the devtools bridge project — reported by the model that read the README against this repository; no term search was run for this one, so nothing here has been checked against the tree. Treat it as a reading to confirm, not as a measured contradiction: verify it against the code before acting on it, and if the footprint it describes does exist, this row is wrong.
- README omits the compat compatibility layer — reported by the model that read the README against this repository; no term search was run for this one, so nothing here has been checked against the tree. Treat it as a reading to confirm, not as a measured contradiction: verify it against the code before acting on it, and if the footprint it describes does exist, this row is wrong.
What to do
- Reconcile the README with reality: README omits the jsx-runtime project (the JSX runtime); README omits the hooks project; README omits the devtools bridge project; README omits the compat compatibility layer.
P1 · CI/CD gates
P3 · Security & performance tooling
- No static application security testing detected. For this repository's stack, add CodeQL's javascript-typescript pack, `semgrep --config=p/typescript`, or eslint-plugin-security as a CI step. What was searched, so you can tell an absence from a miss: the 21904 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 javascript-typescript pack, `semgrep --config=p/typescript`, or eslint-plugin-security — 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.
P4 · Deployment & Rollback
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.
R1 · Type Safety
- 18 typed · 164 plain JS — the untyped files are compat/client.js, compat/client.mjs, compat/jsx-dev-runtime.js, compat/jsx-dev-runtime.mjs, compat/jsx-runtime.js, compat/jsx-runtime.mjs (+158 more).
What to do
- Migrate the remaining .js/.jsx files to TypeScript.
R10 · Code Duplication
- compat/client.js:3 · compat/client.mjs:3 — the two spans are one implementation copied and then locally edited — 59 tokens are still identical, in the same order in both files, with only local edits between them. The copies have already begun to drift, which is this row's finding: an edit made to one and not the other changes behaviour silently. Diff the two spans first to learn what genuinely differs, then extract the shared core into one module both sites use, passing the differences in as parameters — or, if one copy exists only because the other could not be imported from its context, make one of them the single source the other is generated or re-exported from. If one copy is no longer reachable, delete it rather than letting it shadow the live one. — compat/client.js:3
- src/create-element.js:24 · src/create-element.js:53 — all 2 copies are in the same file, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — src/create-element.js:24
What to do
- Act on each finding's own remediation rather than one rule: the move depends on what recurs. Where the copies are executable blocks, give the shared part one home and call it from each site; where they are declarations, a listing, a specialisation already delegating to its base, or one shape repeated per entity, there is no call site and the move is a shared type, a generated set or a factory — sometimes there is nothing to extract.
R11 · Import Boundaries
- compat/client.d.ts:1 reaches into another package with a relative path (../src/index) — import the package by name instead. — compat/client.d.ts:1
- compat/src/index.d.ts:1 reaches into another package with a relative path (../../src/index) — import the package by name instead. — compat/src/index.d.ts:1
- compat/src/index.d.ts:2 reaches into another package with a relative path (../../src/jsx) — import the package by name instead. — compat/src/index.d.ts:2
- compat/src/index.d.ts:3 reaches into another package with a relative path (../../hooks) — import the package by name instead. — compat/src/index.d.ts:3
- compat/src/internal.d.ts:1 reaches into another package with a relative path (../../src/internal) — import the package by name instead. — compat/src/internal.d.ts:1
- compat/src/internal.d.ts:9 reaches into another package with a relative path (../..) — import the package by name instead. — compat/src/internal.d.ts:9
- compat/src/suspense.d.ts:1 reaches into another package with a relative path (../../src/index) — import the package by name instead. — compat/src/suspense.d.ts:1
- compat/src/suspense.js:2 reaches into another package with a relative path (../../src/constants) — import the package by name instead. — compat/src/suspense.js:2
- debug/src/index.d.ts:1 reaches into another package with a relative path (../../src/index) — import the package by name instead. — debug/src/index.d.ts:1
- debug/src/internal.d.ts:1 reaches into another package with a relative path (../../src/internal) — import the package by name instead. — debug/src/internal.d.ts:1
- hooks/src/index.d.ts:1 reaches into another package with a relative path (../../src/index) — import the package by name instead. — hooks/src/index.d.ts:1
- hooks/src/internal.d.ts:1 reaches into another package with a relative path (../../src/internal) — import the package by name instead. — hooks/src/internal.d.ts:1
- jsx-runtime/src/index.d.ts:1 reaches into another package with a relative path (../../src/index) — import the package by name instead. — jsx-runtime/src/index.d.ts:1
- jsx-runtime/src/index.d.ts:2 reaches into another package with a relative path (../../src/index) — import the package by name instead. — jsx-runtime/src/index.d.ts:2
- jsx-runtime/src/index.d.ts:11 reaches into another package with a relative path (../../src/jsx) — import the package by name instead. — jsx-runtime/src/index.d.ts:11
What to do
- Fix the listed violations: import through public entries (package exports / slice index), never reach into another layer or package's internals.
R2 · Cyclomatic Complexity
- diff has cyclomatic complexity 92 and cognitive complexity 224; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/index.js:57
- diffElementNodes has cyclomatic complexity 67 and cognitive complexity 106; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/index.js:521
- diffed has cyclomatic complexity 44 and cognitive complexity 67; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — debug/src/debug.js:326
- handleDomVNode has cyclomatic complexity 44 and cognitive complexity 61; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — compat/src/render.js:169
- setProperty has cyclomatic complexity 40 and cognitive complexity 71; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/props.js:43
- constructNewChildrenArray has cyclomatic complexity 36 and cognitive complexity 79; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/children.js:163
- _diff has cyclomatic complexity 24 and cognitive complexity 29; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — debug/src/debug.js:142
- insert has cyclomatic complexity 18 and cognitive complexity 29; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/children.js:388
- diffChildren has cyclomatic complexity 18 and cognitive complexity 24; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/children.js:46
- jsxAttr has cyclomatic complexity 17 and cognitive complexity 23; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — jsx-runtime/src/index.js:113
- unmount has cyclomatic complexity 16 and cognitive complexity 26; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/index.js:759
- findMatchingIndex has cyclomatic complexity 16 and cognitive complexity 17; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — src/diff/children.js:467
- serializeDomTree has cyclomatic complexity 15 and cognitive complexity 28; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive is at or above cyclomatic here, so the branching is nested or entangled rather than laid out side by side — extracting each decision into its own named function is the change that pays. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — test/_util/helpers.jsx:92
- loadMangle has cyclomatic complexity 12 and cognitive complexity 11; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — scripts/build.mjs:57
- (anonymous) has cyclomatic complexity 11 and cognitive complexity 10; this row is raised above a cyclomatic bar of 10. The two numbers answer different questions and the gap between them is what decides whether to act: cyclomatic counts the independent arms through the body, cognitive counts what it costs to hold them in your head, so nesting and mixed boolean chains raise it while a flat run of independent arms does not. Cognitive sits below cyclomatic here, so much of the count is breadth — arms side by side rather than stacked — and splitting per arm would leave a function per arm; group the work between the checks into named steps instead. Measured by this repository's own parse of the file, so a body assembled at runtime, or generated, is counted as written rather than as it executes. — test/browser/focus.test.jsx:279
What to do
- Break down the listed branch-heavy functions; aim P95 cyclomatic ≤ 5.
R3 · Large Files
- 4 file(s) over 400 lines (counted as significant lines — blank lines excluded — over production source only, tests excluded), largest first: src/diff/index.js (715), debug/src/debug.js (523), hooks/src/index.js (507), src/diff/children.js (473).
What to do
- Split each oversized file along the responsibilities already in it, into smaller focused modules in the same package.
R4 · Test Coverage
- No test imports this module directly or transitively. Import reachability cannot see a test that executes a file by path instead of importing it, nor one that drives it through a running browser by navigating to a URL — if neither does, no test reaches this one. (×10) — scripts/build.mjs, scripts/babel-plugin-fast-rest.mjs, config/compat-entries.js, …
What to do
- Add tests that import the unreached modules (directly or through their public entry).
R6 · Tooling
R7 · Dead Code
- 9 file(s) (~1370 LoC) were excluded from dead-code analysis — declare main/module/exports or a conventional entry (src/index.*, an index.html script) so reachability can see this package. — test/ts
- Unreachable from the 34 application, 20 tooling and 167 test entry point(s) detected in this repo. Gate removals on `pnpm run build` — an undetected custom entry would make these reachable.
- no import path from any entry point (34 application, 20 tooling, 167 test roots considered), and no other file in the scanned tree imports it — nothing in-repo names this module at all, which is the strongest form of this claim the import graph can make (×2) — compat/test-utils.js, compat/test-utils.mjs
R8 · Dependency Hygiene
R9 · Circular Imports
- compat/src/index.js → compat/src/render.js → compat/src/index.js — compat/src/index.js
- src/component.js → src/create-element.js → src/options.js → src/diff/catch-error.js → src/component.js (one verified cycle inside a mutually-dependent group of 7 files) — src/component.js
What to do
- Break each cycle by extracting the shared piece into a module both sides can import.
Reference — by lens
| Lens | Score | Rating | Impact |
|---|---|---|---|
| Code Health | 52% | Adequate — gated by D1, D2, D3, R1 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Architecture | 56% | Adequate — gated by R11 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Maturity | 58% | Adequate — gated by M2 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Readiness | 47% | Weak — gated by R4, P3 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Security | 80% | Strong | Strongest area. |
Unscored — 1 check(s) recorded observations but carry no score
- P12 CI test-gate honesty — 1 observation(s) recorded · Reported, not scored — this card publishes what the CI gate does with the test inventory rather than grading it. The findings above are its output.
Not evidenced — 4 control(s) we could not find positive evidence for
- C3 Audit Trail — Not assessed: these audit controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks audit controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- C4 Data Retention — Not assessed: these retention controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks retention controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- C5 Data-Subject Rights — Not assessed: these data-subject rights controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks data-subject rights controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- P5 DR & Backup — not evidenced — repo shows no backup/RTO/RPO controls; absence of evidence is not evidence of a working control
Not included — 88 check(s) not relevant to this codebase
- AC1 Text alternatives — No web markup found — accessibility is not applicable to this repository.
- AC2 Forms & labels — No web markup found — accessibility is not applicable to this repository.
- AC3 Page structure — No web markup found — accessibility is not applicable to this repository.
- AC4 Keyboard semantics — No web markup found — accessibility is not applicable to this repository.
- AC5 ARIA correctness — No web markup found — accessibility is not applicable to this repository.
- AC6 Visual & motion safety — No web markup found — accessibility is not applicable to this repository.
- AC7 A11y enforcement — No web markup found — accessibility is not applicable to this repository.
- AX1 Captive dependencies — no DI registrations detected
- AX2 Stateful singletons — no singleton implementations detected
- AX3 Project dependency cycles — not assessed — project cycles and dependency direction are computed over a project-reference graph that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX4 Dependency direction — not assessed — project cycles and dependency direction are computed over a project-reference graph that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX5 Architecture & structure — not assessed — architecture style/structure is computed from a project graph (projects, types, module namespaces) that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX6 Interface segregation — not assessed — interface segregation is computed over a type surface that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX7 Slice cohesion — not applicable — not a vertical-slice architecture
- AX8 Test isolation — not assessed — test isolation is computed from a project graph (which projects are test projects, and what they reference) that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- AX9 CQS / query purity — no CQRS query handlers detected — query purity is not applicable to this codebase
- AXB2 Runtime readiness — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- C1 Data Protection — Not assessed: these personal data controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks personal data controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- C2 Access Controls — Not assessed: these authorization controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks authorization controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- D18 Solution Shape — D18 scores the shape of a .NET solution; this repository has no .NET solution or project files, so the dimension does not apply.
- 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 — The exposed public-API surface could not be collected — no C#/VB projects loaded.
- D23 Boundary Type-Coupling — Production source is present (.ts, .tsx) but bounded contexts are resolved over the C#/VB project set, which exposed none, so context scope could not be assessed. Not scored — this is a gap in the analyzer, not a verdict about this repository. Declaring the codebase's bounded contexts (≥2) would let cross-boundary type coupling be assessed — see the recommendation on this dimension for where. Declare them in `.codehealth/config.yaml` at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it — e.g. `architecture:` → `contexts:` → `Billing: ["src/billing", "Acme.Billing"]`, `Catalog: ["src/catalog", "Acme.Catalog"]`.
- D24 Comment Value — No inline comments to assess — comment value is not applicable here.
- D25 ADR Conformance — no ADRs to check
- D26 Project Cohesion — Project cohesion is assessed over the .NET project set; this target exposed no projects, so project size and spread could not be assessed. Not scored — this is a gap in the analyzer's reach, not a verdict about this repository.
- D27 Navigability — 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.
- 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 — D39 measures the IL emitted by a .NET build; this repository has no .NET solution or project files, so the dimension does not apply.
- D4 Code Duplication — This repository's production source (.js, .jsx, .mjs, .ts, .tsx) is not read by D4's token comparison, which compares .NET source: duplication in it is measured by R10 Code Duplication, the frontend lens's card running the same clone algorithm over the JS/TS token stream. Not scored here — read the R10 card for this repository's duplication.
- 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
- D5 Coupling — Inter-project coupling could not be assessed — no analyzable project graph was found for this repository. Not scored: a gap in the analyzer's reach, not a verdict about this repository. (Coupling here is Martin afferent/efferent/instability plus reference cycles across a project-reference graph, read today from .NET project files; other ecosystems' module graphs are not read yet.)
- D6 Cohesion (LCOM4) — Cohesion (LCOM4) is measured over a CS/VB/GO/SCALA/SWIFT/DART class graph, and this repository's production source is .ts, .tsx, which this pass does not read — so no class could be assessed. Not scored — this is a gap in the analyzer, not a finding about this repository.
- D7 Architectural Integrity — no checkable ADRs, and no project-reference graph for the cycle pass to read — so this dimension makes no claim about dependency cycles in either direction (where this repository's language has an import-cycle lens, cycles are reported there). Architectural integrity not assessed
- D8 Code Coverage — Coverage NOT MEASURED: the JavaScript/TypeScript half could not be measured — the vitest suite in the repository root ran and passed but wrote no lcov report. Coverage is excluded from the score rather than counted as a near-zero. The named suite step is one the repository's maintainers can perform; once it passes, the real number is measured on the next scan. Alternatively, commit the lcov/Cobertura report your CI produces and it is read without a re-run.
- DM1 Aggregate boundaries — not scored for TypeScript: a class holding another class reads the same whether the inner type is an aggregate or a value object, so this cannot be decided from source without guessing — reported as guidance rather than measured
- DM2 Strongly-typed ids — not scored for TypeScript: branded ids (`type Id = string & { __brand }`) are an uncommon idiom, so a bare-string id is not on its own evidence of a missing typed id — reported as guidance rather than measured
- DM3 Integration-event coupling — not scored for TypeScript: a domain type used across packages is indistinguishable in source from a deliberate shared-kernel package, so this is reported as guidance rather than measured
- DM4 Rich vs anemic model — not scored for TypeScript: telling a rich domain entity from an anemic data holder needs the behaviour a source-only read cannot always attribute (components, DTOs and readonly value objects are all legitimately data-shaped), so this is reported as guidance rather than measured
- DM5 Encapsulated state — not scored for TypeScript: the language already steers state behind #private/private/readonly, so a mutable public field is rare enough that we report this as guidance rather than measuring it
- DM6 Domain ↔ infrastructure boundary — this TypeScript/JavaScript repository maps no type to a persistence framework (TypeORM/Prisma/MikroORM/Sequelize/Mongoose), so DM6's domain↔infrastructure fusion read has no population to be taken over
- DM7 Repository granularity — not scored for TypeScript: deciding whether a repository belongs to an aggregate root needs the aggregate structure, which source alone does not state — reported as guidance rather than measured
- ED1 Event-Driven — not scored — this repository shows none of the 3 signals this lens looks for
- ED5 Idempotency — This check finds retry-prone mutations (command handlers and message/event consumers) by walking the repository's declared types, and none was loaded here, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- ES1 Event Sourcing — not scored — this repository shows none of the 3 signals this lens looks for
- GD1 Unfinished & placeholder code — no source files were read — this check reads C# syntax, and none was loaded for this repository. That is a limit of the analyzer, not a finding about your code.
- IC1 Incompleteness & stubs — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- P2 Observability — Observability was not assessed: this check reads a source model that does not carry this repository's product — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of a logging idiom this check recognises is NOT evidence that this repo lacks structured logging (it may log through its own ecosystem's logger). This is a gap in the analyzer, not a finding about this repository.
- P7 Outbound HTTP resilience — not measured — the application kind could not be determined for this repo
- P8 Schema migrations — not assessed — schema-migration practice is read from a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- P9 Domain vs controller coverage — no coverage report found on disk — produce a coverage report in a standard format (lcov — `vitest --coverage --coverage.reporter=lcovonly`) and commit it — a hosted scan measures a clone of the repository, so a report that exists only in a working tree, a CI runner's or your own, never reaches it; the artefact is commonly gitignored, so `git add -f` that one file (or un-ignore its path) and commit it alongside the code it measures, or wire coverage collection into CI, to enable this cross-layer check
- PF1 Benchmark discipline — Performance was not assessed: this lens reads a source model that was not loaded for this repository, because the repository is written in a language this lens does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository — in particular it is NOT a statement that this repo is unpackaged or performance-careless.
- PF2 Allocation hygiene — Performance was not assessed: this lens reads a source model that was not loaded for this repository, because the repository is written in a language this lens does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository — in particular it is NOT a statement that this repo is unpackaged or performance-careless.
- PF3 Async & latency hygiene — Performance was not assessed: this lens reads a source model that was not loaded for this repository, because the repository is written in a language this lens does not yet model or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository — in particular it is NOT a statement that this repo is unpackaged or performance-careless.
- R5 Dependency Freshness — uses a pnpm lockfile — dependency freshness not measured here; JS/npm CVEs are scored in D30 (Dependency Vulnerabilities), which answers every ecosystem
- S1 Web-Security Posture — Not assessed: these web-security controls are read from a source model (declarative annotations, request middleware, entity/column names, guard methods) that was not loaded for this repository — because the repository is written in a language this check does not yet model, or because its projects failed to load. Absence of an idiom this check recognises is NOT evidence that this repository lacks web-security controls: it may implement them entirely in its own ecosystem. This is a gap in the analyzer's language coverage, not a finding about this repository.
- SC1 Supply-chain hygiene — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X1 Async correctness — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X12 Unreachable branch — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X13 Undrained process stream — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X14 Bypassable address classification — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X15 Unvalidated length from an untrusted reader — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X16 Unfloored truncation loop — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X17 Uncapped recursion over a caller-supplied document — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X18 Disposal-pattern correctness — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X19 Unrestored process-global state — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X2 Cancellation propagation — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X20 Mistyped argument guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X21 Side-effecting pattern guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X22 Contradicted release guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X23 Unguarded diagnostic materialisation — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X24 Document value interpolated into markup unescaped — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X25 Inert configuration knob — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X26 Unsynchronised callback handoff — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X27 Collection changed while being enumerated — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X28 Index access outside its own emptiness guard — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X29 Per-element action decided by a fixed element — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X3 Exception handling — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X30 Support guard that admits what it rejects — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X32 Type resolved by simple name across every loaded assembly — This check reads C# syntax; no C# was loaded for this repository, so it has nothing to report. That is a limit of the analyzer, not a finding about your code.
- X4 Structured logging — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X5 Nullable reference types — not analysed — these correctness checks read a source model that was not loaded for this repository, because the repository is written in a language this check does not yet model, or because its projects failed to load. This is a gap in the analyzer, not a finding about this repository
- X9 Subsumed condition operand — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
Appendix A — Findings (grouped)
Critical — 9 finding(s)
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Import cycle (2 files) compat/src/index.js
- Import cycle (4 files) src/component.js
Serious — 96 finding(s)
- Boundary violation [package:relative-cross-package] compat/client.d.ts:1
- Boundary violation [package:relative-cross-package] compat/src/index.d.ts:1
- Boundary violation [package:relative-cross-package] compat/src/index.d.ts:2
- Boundary violation [package:relative-cross-package] compat/src/index.d.ts:3
- Boundary violation [package:relative-cross-package] compat/src/internal.d.ts:1
- Boundary violation [package:relative-cross-package] compat/src/internal.d.ts:9
- Boundary violation [package:relative-cross-package] compat/src/suspense.d.ts:1
- Boundary violation [package:relative-cross-package] compat/src/suspense.js:2
- Boundary violation [package:relative-cross-package] debug/src/index.d.ts:1
- Boundary violation [package:relative-cross-package] debug/src/internal.d.ts:1
- Boundary violation [package:relative-cross-package] hooks/src/index.d.ts:1
- Boundary violation [package:relative-cross-package] hooks/src/internal.d.ts:1
- Boundary violation [package:relative-cross-package] jsx-runtime/src/index.d.ts:1
- Boundary violation [package:relative-cross-package] jsx-runtime/src/index.d.ts:2
- Boundary violation [package:relative-cross-package] jsx-runtime/src/index.d.ts:11
- No test reaches this file scripts/build.mjs
- No test reaches this file scripts/babel-plugin-fast-rest.mjs
- No test reaches this file config/compat-entries.js
- No test reaches this file compat/client.mjs
- No test reaches this file compat/scheduler.mjs
- No test reaches this file compat/jsx-dev-runtime.js
- No test reaches this file compat/jsx-dev-runtime.mjs
- No test reaches this file compat/jsx-runtime.js
- No test reaches this file compat/jsx-runtime.mjs
- No test reaches this file compat/test-utils.js
- Hotspot: src/diff/index.js src/diff/index.js:57
- Hotspot: src/diff/children.js src/diff/children.js:163
- Hotspot: compat/src/render.js compat/src/render.js:169
- Hotspot: src/diff/props.js src/diff/props.js:43
- Hotspot: jsx-runtime/src/index.js jsx-runtime/src/index.js:113
- FunctionTooLong: debug.initDebug debug/src/debug.js:56
- FunctionTooLong: index.diff src/diff/index.js:57
- FunctionTooLong: index.diffElementNodes src/diff/index.js:521
- FunctionTooLong: children.constructNewChildrenArray src/diff/children.js:163
- initDebug::diffed (cyclomatic 44) debug/src/debug.js:326
- initDebug::_diff (cyclomatic 24) debug/src/debug.js:142
- No assertions: should handle a promise thrown from a layout effect compat/test/browser/suspense.test.jsx:423
- No assertions: should not crash when effect throws and component is unmounted by render(null) during flush hooks/test/browser/useEffect.test.jsx:714
- initDebug::diffed (cognitive 67) debug/src/debug.js:326
- initDebug::_diff (cognitive 29) debug/src/debug.js:142
- Dead file (~1 LoC) compat/test-utils.js
- Dead file (~1 LoC) compat/test-utils.mjs
- debug.initDebug (cyclomatic 104) debug/src/debug.js:56
- diff (cyclomatic 94) src/diff/index.js:57
- index.diffElementNodes (cyclomatic 67) src/diff/index.js:521
- render.handleDomVNode (cyclomatic 45) compat/src/render.js:169
- props.setProperty (cyclomatic 40) src/diff/props.js:43
- children.constructNewChildrenArray (cyclomatic 36) src/diff/children.js:163
- children.diffChildren (cyclomatic 18) src/diff/children.js:46
- children.insert (cyclomatic 18) src/diff/children.js:388
- index.jsxAttr (cyclomatic 17) jsx-runtime/src/index.js:113
- children.findMatchingIndex (cyclomatic 16) src/diff/children.js:467
- index.unmount (cyclomatic 16) src/diff/index.js:759
- TodoComment test/ts/Component.test.tsx:151
- diff (cognitive 232) src/diff/index.js:57
- diffElementNodes (cognitive 106) src/diff/index.js:521
- constructNewChildrenArray (cognitive 79) src/diff/children.js:163
- setProperty (cognitive 71) src/diff/props.js:43
- handleDomVNode (cognitive 65) compat/src/render.js:169
- debug.initDebug (cognitive 31) debug/src/debug.js:56
- children.insert (cognitive 30) src/diff/children.js:388
- useReducer (cognitive 28) hooks/src/index.js:213
- index.unmount (cognitive 26) src/diff/index.js:759
- children.diffChildren (cognitive 24) src/diff/children.js:46
- jsxAttr (cognitive 23) jsx-runtime/src/index.js:113
- index.options.unmount (cognitive 21) hooks/src/index.js:127
- suspense.detachedClone (cognitive 19) compat/src/suspense.js:46
- catch-error._catchError (cognitive 18) src/diff/catch-error.js:18
- findMatchingIndex (cognitive 17) src/diff/children.js:467
- flushAfterPaintEffects (cognitive 17) hooks/src/index.js:448
- render.options.vnode (cognitive 16) compat/src/render.js:285
- babel-plugin-fast-rest.fastRestTransform (cognitive 16) scripts/babel-plugin-fast-rest.mjs:6
- initSuspenseHooks (cognitive 16) compat/src/suspense.js:11
- REDACTED
- REDACTED
- Coverage not measured — JavaScript/TypeScript suite
- Coverage collected but not gated
- Type Safety
- Duplicated block with local edits (14 matched lines × 2 locations) compat/client.js:3
- Duplicated block (7 lines × 2 locations) src/create-element.js:24
- Complex function diff (cyclomatic 92, cognitive 224) src/diff/index.js:57
- Complex function diffElementNodes (cyclomatic 67, cognitive 106) src/diff/index.js:521
- Complex function diffed (cyclomatic 44, cognitive 67) debug/src/debug.js:326
- Complex function handleDomVNode (cyclomatic 44, cognitive 61) compat/src/render.js:169
- Complex function setProperty (cyclomatic 40, cognitive 71) src/diff/props.js:43
- Complex function constructNewChildrenArray (cyclomatic 36, cognitive 79) src/diff/children.js:163
- Complex function _diff (cyclomatic 24, cognitive 29) debug/src/debug.js:142
- Complex function insert (cyclomatic 18, cognitive 29) src/diff/children.js:388
- Complex function diffChildren (cyclomatic 18, cognitive 24) src/diff/children.js:46
- Complex function jsxAttr (cyclomatic 17, cognitive 23) jsx-runtime/src/index.js:113
- Complex function unmount (cyclomatic 16, cognitive 26) src/diff/index.js:759
- Complex function findMatchingIndex (cyclomatic 16, cognitive 17) src/diff/children.js:467
- Complex function serializeDomTree (cyclomatic 15, cognitive 28) test/_util/helpers.jsx:92
- Complex function loadMangle (cyclomatic 12, cognitive 11) scripts/build.mjs:57
- Complex function (anonymous) (cyclomatic 11, cognitive 10) test/browser/focus.test.jsx:279
- Large Files
Minor — 13 finding(s)
- README/code drift
- README/code drift
- README/code drift
- README/code drift
- Off-boarding risk: anonymized user #1
- Documentation: no installation or build instructions README.md
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- No ADRs
- No SAST
- No changelog
Minor — 39 finding(s)
- Skipped (documented): should allow suspended children to update compat/test/browser/suspense.test.jsx:2056
- Skipped (documented): should allow multiple suspended children to update compat/test/browser/suspense.test.jsx:2122
- Skipped (documented): should allow suspended children children to update compat/test/browser/suspense.test.jsx:2189
- Skipped (documented): should have $$typeof property compat/test/browser/forwardRef.test.jsx:39
- Skipped (documented): should return DOM Node if render is not false nor null compat/test/browser/findDOMNode.test.jsx:24
- Skipped (documented): should run effects child-first even for children separated by memoization hooks/test/browser/combinations.test.jsx:307
- Skipped (documented): (unnamed test) test/browser/render.test.jsx:489
- Skipped (documented): 2 test/browser/render.test.jsx:499
- Skipped (documented): (unnamed test) test/browser/render.test.jsx:500
- Skipped (documented): 3 test/browser/render.test.jsx:506
- Skipped (documented): (unnamed test) test/browser/render.test.jsx:507
- Skipped (documented): 4 test/browser/render.test.jsx:513
- Skipped (documented): (unnamed test) test/browser/render.test.jsx:514
- Skipped (documented): should not swap unkeyed chlildren test/browser/render.test.jsx:1916
- Skipped (documented): should not swap unkeyed chlildren test/browser/render.test.jsx:1947
- Skipped (documented): should maintain focus when swapping elements test/browser/focus.test.jsx:158
- Skipped (documented): (unnamed test) test/browser/components.test.jsx:606
- Skipped (documented): (unnamed test) test/browser/components.test.jsx:610
- Skipped (documented): (unnamed test) test/browser/components.test.jsx:611
- Skipped (documented): (unnamed test) test/browser/components.test.jsx:612
- Skipped (documented): should successfully unmount constantly throwing ref test/browser/lifecycles/getDerivedStateFromError.test.jsx:641
- Skipped (documented): should successfully unmount constantly throwing ref test/browser/lifecycles/componentDidCatch.test.jsx:854
- Skipped (documented): can be used with an object test/ts/Component.test.tsx:152
- Skipped (documented): can be used with a function test/ts/Component.test.tsx:156
- Outdated (npm): @material-ui/core
- Outdated (npm): d3-scale
- Outdated (npm): d3-selection
- Outdated (npm): htm
- Outdated (npm): mobx
- Outdated (npm): mobx-react
- Outdated (npm): mobx-state-tree
- Outdated (npm): preact-render-to-string
- Outdated (npm): preact-router
- Outdated (npm): react-redux
- Outdated (npm): react-router
- Outdated (npm): react-router-dom
- Outdated (npm): redux
- Outdated (npm): styled-components
- Outdated (npm): zustand
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-7e3acad9f0204cc29d9574da8f918bf4/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-7e3acad9f0204cc29d9574da8f918bf4/tree.json --exit-code 0 --source . | 0 | 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 . | 5 | artifacts/raw/semgrep.json |
| D30 · Dependency Vulnerabilities | trivy | — | trivy fs --scanners vuln --format json --quiet --severity CRITICAL,HIGH,MEDIUM,LOW --skip-dirs **/bin/** --skip-dirs **/obj/** . --skip-db-update | 1 | artifacts/raw/trivy-fs.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 | trivy | — | trivy fs --scanners vuln --format json --quiet --severity CRITICAL,HIGH,MEDIUM,LOW --skip-dirs **/bin/** --skip-dirs **/obj/** . --skip-db-update | 0 | artifacts/raw/trivy-fs.json |