Executive summary
Read through the Production lens — the standard calibration. *Green* means good enough to run in production. The score is absolute and comparable across repos.
YMFE/yapi carries serious gaps (28%). Several issues below can materially affect correctness, security, or the cost of changing it — and propagate to everything that depends on it.
Most urgent: a critical security exposure was detected (see the Security & Compliance lens). Treat it as a priority regardless of the overall grade.
The area that most needs attention is Readiness (8%) — releases are harder to depend on — versioning, release notes and dependency hygiene are thin, so consumers can't easily tell what changed or trust an upgrade. Code Health (39%) is the next concern — changes there are slower and more error-prone.
Leadership focus, highest impact first: CI workflow that builds and runs the test suite on every push/PR (CI/CD gates); ILogger (or Serilog) and log at meaningful points across… (Observability); tests that import the unreached modules (directly or through… (Test Coverage).
For scale: — (~0 production lines); rebuilding it from scratch would take roughly — (—). Approximate, ±~30%.
Raise Readiness 8 → 70 (the Healthy floor) ⇒ headline 28 → ~49.
New since the last scan (77+)
- D2 · (anonymous) (cognitive 63) client/containers/Project/Interface/InterfaceList/InterfaceEditForm.js
- D2 · importPostman (cognitive 54) exts/yapi-plugin-import-postman/client.js
- D2 · handleAddInterface (cognitive 38) common/HandleImportData.js
- D2 · mockValidator (cognitive 37) server/middleware/mockServer.js
- D2 · render (cognitive 36) client/containers/Project/Interface/InterfaceList/View.js
- D2 · importHar (cognitive 35) exts/yapi-plugin-import-har/client.js
- D2 · render (cognitive 27) client/components/Postman/Postman.js
- D2 · matchApi (cognitive 20) server/middleware/mockServer.js
- D2 · createReqBody (cognitive 16) common/markdown.js
- D2 · getParamsKey (cognitive 16) exts/yapi-plugin-advanced-mock/MockCol/CaseDesModal.js
- D2 · onMouseDown (cognitive 16) client/components/EasyDragSort/EasyDragSort.js
- D3 · FileTooLong: utils/commons.js server/utils/commons.js
- D3 · FileTooLong: User/Profile.js client/containers/User/Profile.js
- D8 · No automated tests
- D18 · Dimension evaluation failed
- D19 · LLM evaluation failed
- D28 · Secret: generic-api-key static/doc/index.html
- D28 · Secret: generic-api-key static/doc/index.html
- D28 · Secret: generic-api-key static/doc/index.html
- D28 · Secret: generic-api-key doc/build/index.html
- D28 · Secret: generic-api-key doc/build/index.html
- D28 · Secret: generic-api-key doc/build/index.html
- D28 · Secret: generic-api-key doc/build/index.html
- D28 · Secret: generic-api-key doc/build/index.html
- D28 · Secret: generic-api-key doc/build/index.html
- D35 · Boundary-crossing change coupling: ProjectData.js ↔ index.js client/containers/Project/Setting/ProjectData/ProjectData.js
- D35 · Boundary-crossing change coupling: mockCol.js ↔ client.js client/reducer/modules/mockCol.js
- D35 · Boundary-crossing change coupling: mockCol.js ↔ CaseDesModal.js client/reducer/modules/mockCol.js
- D35 · Boundary-crossing change coupling: mockCol.js ↔ MockCol.js client/reducer/modules/mockCol.js
- D35 · Boundary-crossing change coupling: Intro.js ↔ Home.js client/components/Intro/Intro.js
- D35 · Boundary-crossing change coupling: InterfaceColMenu.js ↔ interfaceCol.js client/containers/Project/Interface/InterfaceCol/InterfaceColMenu.js
- D35 · Boundary-crossing change coupling: project.js ↔ project.js server/controllers/project.js
- D35 · Boundary-crossing change coupling: user.js ↔ user.js server/controllers/user.js
- D35 · Boundary-crossing change coupling: ProjectCard.js ↔ Follows.js client/components/ProjectCard/ProjectCard.js
- D35 · Boundary-crossing change coupling: ProjectCard.js ↔ follow.js client/components/ProjectCard/ProjectCard.js
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- D38 · Critical CVE: [GHSA redacted] package-lock.json
- R10 · Duplicated block (99 lines × 4 locations) client/plugin.js
- R10 · Duplicated block (82 lines × 3 locations) server/plugin.js
- R10 · Duplicated block (39 lines × 2 locations) server/controllers/interface.js
- R10 · Duplicated block (36 lines × 3 locations) exts/yapi-plugin-advanced-mock/controller.js
- R10 · Duplicated block (36 lines × 2 locations) server/controllers/interface.js
- R10 · Duplicated block (33 lines × 3 locations) server/controllers/interfaceCol.js
- R10 · Duplicated block (32 lines × 4 locations) exts/yapi-plugin-wiki/controller.js
- R10 · Duplicated block (32 lines × 2 locations) server/controllers/interfaceCol.js
- R2 · Complex function render (cyclomatic 35, cognitive 36) client/containers/Project/Interface/InterfaceList/View.js
- R2 · Complex function (anonymous) (cyclomatic 33, cognitive 63) client/containers/Project/Interface/InterfaceList/InterfaceEditForm.js
- R2 · Complex function importPostman (cyclomatic 29, cognitive 54) exts/yapi-plugin-import-postman/client.js
- R2 · Complex function render (cyclomatic 28, cognitive 27) client/components/Postman/Postman.js
- R2 · Complex function mockValidator (cyclomatic 27, cognitive 37) server/middleware/mockServer.js
- R2 · Complex function importHar (cyclomatic 23, cognitive 35) exts/yapi-plugin-import-har/client.js
- R7 · Dead file (~1357 LoC) client/containers/Project/Interface/InterfaceList/InterfaceEditForm.js
- R7 · Dead file (~1257 LoC) client/containers/Project/Interface/InterfaceCol/InterfaceColContent.js
- R7 · Dead file (~1051 LoC) client/components/Postman/Postman.js
- R7 · Dead file (~639 LoC) client/containers/Project/Interface/InterfaceList/InterfaceMenu.js
- R7 · Dead file (~630 LoC) client/containers/Project/Interface/InterfaceCol/InterfaceColMenu.js
- R7 · Dead file (~585 LoC) client/containers/Project/Interface/InterfaceList/View.js
- R7 · Dead file (~556 LoC) client/containers/User/Profile.js
- R8 · Unlisted import 'shelljs' npm-publish.js
- R8 · Unlisted import 'yapi-plugin-' common/plugin.js
- R8 · Unused dependency 'anujs'
- P2 · No structured logging
Top priorities
Diagnosis — what's actually going on
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 |
|---|---|---|
| A06:2021 — Vulnerable & Outdated Components | 50 | High / Critical |
| A02:2021 — Cryptographic Failures | 10 | High / Critical |
Roadmap
First, establish a continuous integration pipeline to automatically build and test every code change. Next, implement structured logging and health checks to improve system observability. Then, expand the test suite to cover currently unreachable production modules. Finally, clean up dependencies by removing unused packages and configure linting and type-checking in the build process.
| Do this | Helps | Effort | Dimension |
|---|---|---|---|
| Resolve the 1 No automated tests finding(s) in Code Coverage. | +19.8 pts | Low | Code Coverage |
| Resolve the 1 No tests found finding(s) in Test Distribution. | +19.8 pts | Low | Test Distribution |
| Add a CI workflow that builds and runs the test suite on every push/PR. | +23.3 pts | Medium | CI/CD gates |
| Adopt ILogger (or Serilog) and log at meaningful points across the projects. | +23.3 pts | Medium | Observability |
| Add tests that import the unreached modules (directly or through their public entry). | +23.3 pts | Medium | Test Coverage |
| Remove unused dependencies, declare unlisted imports explicitly, and demote type-/test-only packages to devDependencies. | +23.3 pts | Medium | Dependency Hygiene |
| Add the missing tooling (eslint, tsc) as package.json scripts and run them in CI. | +21.2 pts | Medium | Tooling |
| Bump outdated dependencies to current versions to limit upgrade debt. | +20.3 pts | Medium | Dependency Freshness |
File quality
| File | Score | Band | Worst signal |
|---|---|---|---|
| package-lock.json | 4.4 | Mixed | OSV Dependency Vulnerabilities: Critical CVE: [GHSA redacted] |
| doc/build/index.html | 4.5 | Mixed | Secrets (history): Secret: generic-api-key |
| static/doc/index.html | 5.1 | Mixed | Secrets (history): Secret: generic-api-key |
| server/controllers/project.js | 6.5 | Mixed | Change Coupling: Boundary-crossing change coupling: project.js ↔ project.js |
| server/controllers/user.js | 6.5 | Mixed | Change Coupling: Boundary-crossing change coupling: user.js ↔ user.js |
| client/containers/Project/Interface/InterfaceCol/InterfaceColMenu.js | 6.5 | Mixed | Change Coupling: Boundary-crossing change coupling: InterfaceColMenu.js ↔ interfaceCol.js |
| client/reducer/modules/mockCol.js | 6.5 | Mixed | Change Coupling: Boundary-crossing change coupling: mockCol.js ↔ client.js |
| client/components/ProjectCard/ProjectCard.js | 7.0 | Mixed | Change Coupling: Boundary-crossing change coupling: ProjectCard.js ↔ Follows.js |
| server/middleware/mockServer.js | 7.1 | Mixed | Cyclomatic Complexity: exports (cyclomatic 43) |
| server/controllers/interface.js | 7.2 | Mixed | Cyclomatic Complexity: up (cyclomatic 17) |
| exts/yapi-plugin-import-swagger/run.js | 7.4 | Mixed | Cyclomatic Complexity: handleSwagger (cyclomatic 19) |
| client/containers/User/Profile.js | 7.4 | Mixed | Cyclomatic Complexity: render (cyclomatic 18) |
| server/controllers/base.js | 7.4 | Mixed | Cyclomatic Complexity: getProjectRole (cyclomatic 17) |
| client/containers/Project/Interface/InterfaceList/View.js | 7.8 | Mixed | Cyclomatic Complexity: render (cyclomatic 35) |
| client/containers/Project/Interface/InterfaceList/InterfaceEditForm.js | 7.8 | Mixed | Cyclomatic Complexity: (anonymous) (cyclomatic 33) |
| exts/yapi-plugin-import-postman/client.js | 7.8 | Mixed | Cyclomatic Complexity: importPostman (cyclomatic 29) |
| client/components/Postman/Postman.js | 7.8 | Mixed | Cyclomatic Complexity: render (cyclomatic 28) |
| exts/yapi-plugin-import-har/client.js | 7.8 | Mixed | Cyclomatic Complexity: importHar (cyclomatic 23) |
| client/components/MockDoc/MockDoc.js | 7.8 | Mixed | Cyclomatic Complexity: mockToArr (cyclomatic 22) |
| common/HandleImportData.js | 7.8 | Mixed | Cyclomatic Complexity: handleAddInterface (cyclomatic 20) |
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. 42 of 43 evaluated dimensions are computed purely by tools and static analysis (confidence 1.0); 1 documentation/naming judgement(s) are LLM-assisted and labelled advisory. Overall confidence is 0.6 — 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 — 43 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, 125 of 132 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 |
| jscpd | Code duplication | — | ✓ deterministic |
| Coverage (coverlet / dotnet-coverage) | Line & branch coverage | 10.0.301 | ✓ deterministic |
| NuGet / dotnet | Outdated, vulnerable & deprecated dependencies | 10.0.301 | ✓ deterministic |
| git / LibGit2Sharp | Churn hotspots, knowledge concentration, history | 2.43.0 · 0.31.0 | ✓ deterministic |
| gitleaks · semgrep · trivy · checkov | Secrets in history, SAST, CVEs, IaC & container, PII / GDPR | 1.86.0 · 0.69.3 | ✓ 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
- D18 Solution Shape — evaluation did not complete — Dimension evaluation failed — excluded from the score.
- D19 Documentation Quality — LLM provider failed — The model provider returned an unusable result, so this LLM-assisted dimension fell back to a measurement gap (confidence 0) rather than a penalty. Re-run with a reachable provider to score it.
- D30 Dependency Vulnerabilities — scanner not present in this environment — The backing tool was not installed where this scan ran, so this dimension was not scored. Install the tool (or run in the hosted environment, where it is always present) for a graded result.
- D31 IaC & Container Security — scanner not present in this environment — The backing tool was not installed where this scan ran, so this dimension was not scored. Install the tool (or run in the hosted environment, where it is always present) for a graded result.
- D32 Data Compliance (PII/GDPR) — scanner not present in this environment — The backing tool was not installed where this scan ran, so this dimension was not scored. Install the tool (or run in the hosted environment, where it is always present) for a graded result.
- D36 Supply-chain Provenance & Signing — scanner not present in this environment — The backing tool was not installed where this scan ran, so this dimension was not scored. Install the tool (or run in the hosted environment, where it is always present) for a graded result.
Limitations & what we did not check
Watchdog assesses the repository exactly as committed, and only the repository. By design it does not reach outside the source tree: the live cloud account, the running CI/CD pipeline, the host's branch-protection and approval rules, the production configuration, or a restore actually exercised against a backup are all out of scope. That boundary is a feature, not a gap — a repo-relative, deterministic scan re-runs identically on any commit and every finding opens at a real file and line, where a live audit can neither be reproduced nor traced. The visible consequence is that controls which leave no in-repo evidence are reported as "not evidenced" and excluded from the score rather than awarded a number a static scan cannot justify.
Per-dimension blind spots
- D1 Cyclomatic Complexity: Cyclomatic complexity counts branches statically — it cannot tell an essential decision tree from accidental tangle, nor see complexity that lives in data or configuration (large switch-case token tables, DSL lexers/parsers, data-as-code rule tables) rather than control flow: a tokenizer's many single-character cases read as high complexity though each branch is trivial.
- D2 Cognitive Complexity: Cognitive-complexity heuristics approximate how hard code is to follow; genuine domain difficulty and well-named intent that eases reading are not captured.
- D3 God Classes: "God class" is sized by members and responsibilities visible in the type — a deliberately broad facade over a coherent subsystem can read the same as an accidental grab-bag. For front-end JS the file-length check is cohesion-aware (a single-responsibility module — one class/IIFE — earns a 3× threshold), but cohesion is approximated from top-level declarations, not true dependency structure.
- D4 Code Duplication: Duplication is token-similarity (jscpd) — it finds copy-paste, not semantic duplication expressed differently. Committed machine-written code (EF migration scaffolds, *.Designer.cs, model snapshots) is EXCLUDED — its repetition is the tool's, not the team's — so the score reflects hand-written duplication only; the generated footprint is reported separately under Solution Shape.
- D5 Coupling: Coupling is measured between projects/assemblies — runtime coupling through DI, reflection, messaging or shared databases is invisible to a static reference graph.
- D8 Code Coverage: Coverage is measured by building and running the suite (`dotnet test --collect`) inside Watchdog's isolated image — the target repo is never modified, and nothing on your systems runs. So coverage exists only when the suite builds and runs within the inline time budget; one that needs external services, can't build, or exceeds the budget yields no coverage (D8 then degrades to not-measured, not a low score). Line coverage also says nothing about assertion quality.
- 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.
- 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 Secret Scanning: Secret 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").
- 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.
- 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.
- D26 Project Cohesion: Project focus is sized from members/namespaces per project — a project that is broad by deliberate design reads the same as one that has sprawled.
- D27 Navigability: Indirection/navigability is structural — it measures hops to follow a call, not whether that indirection buys real flexibility or just ceremony.
- 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").
- D33 JS/npm Dependency Vulnerabilities: JS/npm CVE matching reads package manifests and lockfiles — risk from how a dependency is used, and advisories not yet published, fall outside this scan.
- 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.
- M4 Documentation accuracy: Onboarding quality is an LLM read of the docs/setup present — it cannot run the onboarding or measure how long a real new joiner takes; the verdict is sampled and advisory.
- P6 Release Hygiene: Rollback/observability controls are inferred from repo artefacts (pipelines, dashboards-as-code) — controls configured in external tooling, with no in-repo trace, cannot be credited.
The LLM boundary
Dimensions
D1 · Cyclomatic Complexity
18 method(s) exceeded the cyclomatic complexity threshold of 15; the worst was exports at 43.
What to do
- Resolve the 1 exports (cyclomatic 43) finding(s) in Cyclomatic Complexity — start with mockServer.js. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 render (cyclomatic 35) finding(s) in Cyclomatic Complexity — start with View.js. — One of this dimension's main actionable groups (1 warning-level).
- Resolve the 1 (anonymous) (cyclomatic 33) finding(s) in Cyclomatic Complexity — start with InterfaceEditForm.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 exports at 76.
What to do
- Resolve the 2 list (cognitive 18) finding(s) in Cognitive Complexity — start with interface.js, project.js. — One of this dimension's main actionable groups (2 warning-level).
- Resolve the 2 defaultExpandedKeys (cognitive 16) finding(s) in Cognitive Complexity — start with InterfaceColMenu.js, InterfaceMenu.js. — One of this dimension's main actionable groups (2 warning-level).
- Resolve the 1 exports (cognitive 76) finding(s) in Cognitive Complexity — start with mockServer.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
2 god class(es) detected.
What to do
- Resolve the 2 FileTooLong finding(s) in God Classes — start with Profile.js, commons.js. — One of this dimension's main actionable groups (2 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.
D4 · Code Duplication
0 duplicated block group(s) detected.
D5 · Coupling
0 projects, 0 dependency cycle(s), 0 unstable depended-on project(s).
What to do
- Enforce Coupling in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Prevented — provenance only; does not change the score.
D8 · Code Coverage
No automated tests — the solution has no test code.
What to do
- Resolve the 1 No automated tests finding(s) in Code Coverage. — One of this dimension's main actionable groups (1 issue-level).
- Enforce Code Coverage in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Verified — provenance only; does not change the score.
D9 · Test Distribution
No test projects found.
What to do
- Resolve the 1 No tests found finding(s) in Test Distribution. — One of this dimension's main actionable groups (1 recommendation-level).
D12 · Dependency Hygiene
0 outdated, 0 vulnerable, 0 deprecated packages.
D13 · Secret Scanning
Secret scan ran and found no leaked secrets.
D15 · Churn × Complexity Hotspots
No churn × complexity hotspots in the window.
D16 · Bus Factor
No source file's living knowledge is concentrated in a single author.
D21 · Naming Consistency
0 naming inconsistencies across 0 sampled symbols.
D26 · Project Cohesion
No projects to assess.
D27 · Navigability
Too little code to assess navigability.
D28 · Secrets (history)
28 finding(s): 0 critical, 28 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 9 Secret finding(s) in Secrets (history) — start with index.html (9). — One of this dimension's main actionable groups (9 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)
semgrep found no security issues.
D33 · JS/npm Dependency Vulnerabilities
No known-vulnerable JS/npm dependencies.
D34 · Knowledge Freshness
Every significant source file has living knowledge — recently and meaningfully worked.
D35 · Change Coupling
Strongest change-coupling: assets.js↔manifest@f2f4bd774d6c221b3d5f.js 73%; controller.js↔defaultTheme.js 73%; index.js↔plugin.js 70%
D37 · Vulnerability-disclosure Policy
A vulnerability-disclosure policy (SECURITY.md) is published with a reporting contact.
D38 · OSV Dependency Vulnerabilities
423 finding(s): 75 critical, 182 high, 134 medium, 32 low.
What to do
- Resolve the 50 Critical CVE finding(s) in OSV Dependency Vulnerabilities — start with package-lock.json (50). — One of this dimension's main actionable groups (50 issue-level).
Frontend & cross-cutting dimensions
AX5 · Architecture & structure
M1 · Documentation (README)
What to do
- Add a build/run (quick start) section to the root README — the first thing a newcomer needs.
- 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.
M2 · Architecture documentation
- No Architecture Decision Records found — decisions aren't captured for future maintainers.
- No C4/PlantUML/Mermaid diagram or architecture.md — the high-level shape isn't documented.
What to do
- Start an ADR log (docs/adr/) recording significant decisions and their rationale.
- Add a C4 context/container diagram (Structurizr, PlantUML or Mermaid) or an architecture.md overview.
M3 · Folder & project structure
M4 · Documentation accuracy
- README advertises Docker containerisation, but no Dockerfile/compose file exists
What to do
- Reconcile the README with reality: README advertises Docker containerisation, but no Dockerfile/compose file exists.
P1 · CI/CD gates
- No CI workflow found (.github/workflows, azure-pipelines.yml, .gitlab-ci.yml, …) — changes aren't gated by an automated build/test.
What to do
- Add a CI workflow that builds and runs the test suite on every push/PR.
P2 · Observability
- No ILogger/Serilog usage found — production issues will be hard to diagnose.
What to do
- Adopt ILogger (or Serilog) and log at meaningful points across the projects.
- Consider OpenTelemetry tracing/metrics and a health-check endpoint for operability.
P3 · Security & performance tooling
- No static application security testing (CodeQL / security analyzers / codehealth) detected.
What to do
- Add a SAST step (e.g. CodeQL) or a security analyzer package.
- Enable Dependabot/Renovate or a dependency-review gate.
- Add gitleaks/trufflehog in CI to block PRs that introduce committed secrets.
P6 · Release Hygiene
What to do
- Stamp a version (csproj <Version>, a VERSION file, or GitVersion) or tag releases with semver so builds and releases are traceable.
R1 · Type Safety
- 0 typed · 232 plain JS
What to do
- Migrate the remaining .js/.jsx files to TypeScript.
R10 · Code Duplication
- client/plugin.js:17 · client/plugin.js:73 · client/plugin.js:104 · client/plugin.js:203 — plugin.js:17
- server/plugin.js:25 · server/plugin.js:57 · server/plugin.js:89 — plugin.js:25
- server/controllers/interface.js:169 · server/controllers/interface.js:302 — interface.js:169
- exts/yapi-plugin-advanced-mock/controller.js:72 · server/controllers/interface.js:469 · server/controllers/interface.js:634 — controller.js:72
- server/controllers/interface.js:633 · server/controllers/interfaceCol.js:273 — interface.js:633
- server/controllers/interfaceCol.js:360 · server/controllers/interfaceCol.js:535 · server/controllers/user.js:472 — interfaceCol.js:360
- exts/yapi-plugin-wiki/controller.js:36 · server/controllers/interfaceCol.js:61 · server/controllers/interfaceCol.js:279 · server/controllers/interfaceCol.js:440 — controller.js:36
- server/controllers/interfaceCol.js:56 · server/controllers/interfaceCol.js:274 — interfaceCol.js:56
What to do
- Extract the duplicated blocks into shared functions/components.
R2 · Cyclomatic Complexity
- Branch-heavy code is where defects cluster — extract decisions into smaller functions. (×8) — mockServer.js:144, View.js:256, InterfaceEditForm.js:228, …
What to do
- Break down the listed branch-heavy functions; aim P95 cyclomatic ≤ 5.
R3 · Large Files
- 20 file(s) over 400 lines
What to do
- Split the oversized components into smaller, focused ones.
R4 · Test Coverage
- No test imports this module directly or transitively — its behavior is unverified. (×8) — InterfaceEditForm.js, interface.js, InterfaceColContent.js, …
What to do
- Add tests that import the unreached modules (directly or through their public entry).
R5 · Dependency Freshness
- 55 outdated package(s); JS/npm CVEs scored in D33
What to do
- Bump outdated dependencies to current versions to limit upgrade debt.
R6 · Tooling
- test ✓ · lint ✗ · typecheck ✗
What to do
- Add the missing tooling (eslint, tsc) as package.json scripts and run them in CI.
R7 · Dead Code
- Unreachable from the 3 application, 1 tooling and 6 test entry point(s) detected in this repo. Gate removals on your build/type-check — an undetected custom entry would make these reachable.
- no import path from any entry point (3 application, 1 tooling, 6 test roots considered) (×7) — InterfaceEditForm.js, InterfaceColContent.js, Postman.js, …
What to do
- Delete the dead files and unused exports — every line is maintenance cost and rebuild-estimate inflation with zero runtime value.
R8 · Dependency Hygiene
- Imported but not declared in any reachable package.json — installs work only by hoisting accident. (×3) — mongoose-auto-increment.js:3, npm-publish.js:1, plugin.js:8
- Declared in the root/package.json but never imported anywhere in that package or its workspace members — dead weight and attack surface. Verify against build tooling before removing. (×3)
What to do
- Remove unused dependencies, declare unlisted imports explicitly, and demote type-/test-only packages to devDependencies.
R9 · Circular Imports
X1 · Async correctness
X3 · Exception handling
X4 · Structured logging
Reference — by lens
| Lens | Score | Rating | Impact |
|---|---|---|---|
| Code Health | 39% | Weak — gated by R1, R7 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Maturity | 54% | Adequate — gated by M2 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Readiness | 8% | Critical — gated by D8, D9, R4, R6, R8, P1, P2, P3 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Security | 60% | Adequate — gated by D28, D38 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
Not included — 60 check(s) not relevant to this codebase
- AC1 Text alternatives — No user-facing web UI (the repo is a library/CLI/worker/headless service) — accessibility is not applicable.
- AC2 Forms & labels — No user-facing web UI (the repo is a library/CLI/worker/headless service) — accessibility is not applicable.
- AC3 Page structure — No user-facing web UI (the repo is a library/CLI/worker/headless service) — accessibility is not applicable.
- AC4 Keyboard semantics — No user-facing web UI (the repo is a library/CLI/worker/headless service) — accessibility is not applicable.
- AC5 ARIA correctness — No user-facing web UI (the repo is a library/CLI/worker/headless service) — accessibility is not applicable.
- AC6 Visual & motion safety — No user-facing web UI (the repo is a library/CLI/worker/headless service) — accessibility is not applicable.
- AC7 A11y enforcement — No user-facing web UI (the repo is a library/CLI/worker/headless service) — accessibility is not applicable.
- AX1 Captive dependencies — no DI registrations detected
- AX10 Code composition — no source files detected — code composition not applicable
- AX2 Stateful singletons — no singleton implementations detected
- AX3 Project dependency cycles — no csproj graph available
- AX4 Dependency direction — no csproj graph available
- AX6 Interface segregation — no public interfaces
- AX7 Slice cohesion — not applicable — not a vertical-slice architecture
- AX8 Test isolation — no test/production split to check
- AX9 CQS / query purity — no CQRS query handlers detected — query purity is not applicable to this codebase
- AXB2 Runtime readiness — no data
- C1 Data Protection — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- C2 Access Controls — No access-control surface detected in the analyzed source — no web/app surface to authorize (no HTTP API or web-UI project) and no authorization code at all (no [Authorize]/policies, no imperative guard methods). Access control is therefore N/A here — this is a library/CLI, which is authorized by its CALLER, not by itself. If this codebase grows request handlers, the dimension reactivates and a default-deny posture is expected then.
- C3 Audit Trail — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- C4 Data Retention — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- C5 Data-Subject Rights — No personal data detected in the analyzed source — no PII-typed entity/column names (Email, FirstName, DateOfBirth, …), no ASP.NET Identity / user-account model, and no stored user credentials. GDPR data-protection controls are therefore N/A here. If this is intentional, record the no-PII posture in an ADR; if the app does process personal data, name those fields conventionally so this dimension activates.
- D10 Test Quality — No tests in the analyzed solution to assess for quality.
- D11 Test Reliability — Test reliability not included
- D14 License Compliance — license scan produced no result — the tool ran but its JSON output could not be parsed; the offline NuGet fallback resolved nothing
- D17 Explicit Debt — the C# workspace loaded 0 projects, so explicit-debt density could not be measured
- D18 Solution Shape — Dimension evaluation failed
- D19 Documentation Quality — LLM evaluation failed
- D20 ADR Quality — N/A — ADRs are expected on deployable products with a user-facing host, not consumed libraries; no ADR log is required here.
- D22 Internal API Consistency — No exposed public API
- D23 Boundary Type-Coupling — Zero projects and zero LoC mean the codebase is trivial and has no structure to justify boundaries.
- D24 Comment Value — No inline comments to assess — comment value is not applicable here.
- D25 ADR Conformance — no ADRs to check
- D30 Dependency Vulnerabilities — No .NET solution found; no NuGet dependencies to scan for vulnerabilities.
- D31 IaC & Container Security — No Infrastructure-as-Code or container manifests found (Dockerfile, Terraform, Kubernetes/Helm, CloudFormation); nothing to scan.
- D32 Data Compliance (PII/GDPR) — No PII/GDPR ruleset is bundled (the public p/gdpr semgrep pack was retired) — data compliance is not assessed in this scan.
- D36 Supply-chain Provenance & Signing — No CI/build pipeline found (.github/workflows, .gitlab-ci.yml, azure-pipelines.yml, Jenkinsfile, .circleci); there is no build to attest provenance for.
- D39 IL Efficiency — The target did not build, so no IL was available to measure.
- D6 Cohesion (LCOM4) — No production classes were analyzable, so cohesion (LCOM4) was not measured (the solution likely failed to load or has no production code).
- D7 Architectural Integrity — no checkable ADRs and no dependency cycles — architectural integrity not assessed
- DM1 Domain Modelling — not run — 0/3 markers found
- ED1 Event-Driven — not run — 0/3 markers found
- ED5 Idempotency — no mutating command handlers or message consumers detected — idempotency check not applicable
- ES1 Event Sourcing — not run — 0/3 markers found
- GD1 Unfinished & placeholder code — no source files
- IC1 Incompleteness & stubs — no C# methods found
- P12 CI test-gate honesty — no CI workflow found
- P4 Deployment & Rollback — not evidenced — no deploy/rollback/approval signal in the repo; absence of evidence is not evidence of a manual release
- P5 DR & Backup — not evidenced — repo shows no backup/RTO/RPO controls; absence of evidence is not evidence of a working control
- P7 Outbound HTTP resilience — not applicable — this isn't a service/API/worker
- P8 Schema migrations — no EF Core usage detected
- P9 Domain vs controller coverage — no coverage report found on disk — run tests with `--collect:"XPlat Code Coverage"` (or in CI) to enable this cross-layer check
- PF1 Benchmark discipline — Performance is assessed only for perf-relevant repos — a packaged library, one that ships benchmarks, or one already using allocation-aware APIs. This repo isn't one, so the Performance lens is not applicable and is excluded from the score.
- PF2 Allocation hygiene — Performance is assessed only for perf-relevant repos — a packaged library, one that ships benchmarks, or one already using allocation-aware APIs. This repo isn't one, so the Performance lens is not applicable and is excluded from the score.
- PF3 Async & latency hygiene — Performance is assessed only for perf-relevant repos — a packaged library, one that ships benchmarks, or one already using allocation-aware APIs. This repo isn't one, so the Performance lens is not applicable and is excluded from the score.
- R11 Import Boundaries — No recognizable feature-sliced/layered src layout — boundary rules not applicable.
- S1 Web-Security Posture — No web surface detected in the analyzed source — no HTTP API or web-UI project (no controllers/minimal-API endpoints, no Razor/Blazor views) and no web middleware (HTTPS redirection, HSTS, security headers, cookies). Transport security, security headers, secure cookies, CSRF/input-validation and middleware-order controls are therefore N/A here — this is a library/CLI/worker, not a web app. Crypto hygiene was still checked and found nothing to flag. If this codebase becomes web-facing, the dimension reactivates automatically.
- SC1 Supply-chain hygiene — no data
- X2 Cancellation propagation — no async methods found
- X5 Nullable reference types — no NRT-eligible projects
Appendix A — Findings (grouped)
Issue — 71 finding(s)
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Critical CVE: [GHSA redacted] package-lock.json
- Boundary-crossing change coupling: ProjectData.js ↔ index.js client/containers/Project/Setting/ProjectData/ProjectData.js
- Boundary-crossing change coupling: mockCol.js ↔ client.js client/reducer/modules/mockCol.js
- Boundary-crossing change coupling: mockCol.js ↔ CaseDesModal.js client/reducer/modules/mockCol.js
- Boundary-crossing change coupling: mockCol.js ↔ MockCol.js client/reducer/modules/mockCol.js
- Boundary-crossing change coupling: Intro.js ↔ Home.js client/components/Intro/Intro.js
- Boundary-crossing change coupling: InterfaceColMenu.js ↔ interfaceCol.js client/containers/Project/Interface/InterfaceCol/InterfaceColMenu.js
- Boundary-crossing change coupling: project.js ↔ project.js server/controllers/project.js
- Boundary-crossing change coupling: user.js ↔ user.js server/controllers/user.js
- Boundary-crossing change coupling: ProjectCard.js ↔ Follows.js client/components/ProjectCard/ProjectCard.js
- Boundary-crossing change coupling: ProjectCard.js ↔ follow.js client/components/ProjectCard/ProjectCard.js
- Secret: generic-api-key static/doc/index.html:95
- Secret: generic-api-key static/doc/index.html:99
- Secret: generic-api-key static/doc/index.html:103
- Secret: generic-api-key doc/build/index.html:93
- Secret: generic-api-key doc/build/index.html:97
- Secret: generic-api-key doc/build/index.html:98
- Secret: generic-api-key doc/build/index.html:94
- Secret: generic-api-key doc/build/index.html:95
- Secret: generic-api-key doc/build/index.html:96
- Dimension evaluation failed
- No automated tests
Warning — 57 finding(s)
- list (cognitive 18) server/controllers/project.js:556
- list (cognitive 18) server/controllers/interface.js:489
- defaultExpandedKeys (cognitive 16) client/containers/Project/Interface/InterfaceList/InterfaceMenu.js:432
- defaultExpandedKeys (cognitive 16) client/containers/Project/Interface/InterfaceCol/InterfaceColMenu.js:414
- FileTooLong: utils/commons.js server/utils/commons.js:0
- FileTooLong: User/Profile.js client/containers/User/Profile.js:0
- exports (cyclomatic 43) server/middleware/mockServer.js:144
- render (cyclomatic 35) client/containers/Project/Interface/InterfaceList/View.js:256
- (anonymous) (cyclomatic 33) client/containers/Project/Interface/InterfaceList/InterfaceEditForm.js:228
- importPostman (cyclomatic 29) exts/yapi-plugin-import-postman/client.js:122
- render (cyclomatic 28) client/components/Postman/Postman.js:570
- mockValidator (cyclomatic 27) server/middleware/mockServer.js:90
- importHar (cyclomatic 23) exts/yapi-plugin-import-har/client.js:116
- mockToArr (cyclomatic 22) client/components/MockDoc/MockDoc.js:131
- handleAddInterface (cyclomatic 20) common/HandleImportData.js:60
- saveCase (cyclomatic 20) exts/yapi-plugin-advanced-mock/controller.js:94
- handleSwagger (cyclomatic 19) exts/yapi-plugin-import-swagger/run.js:149
- handleParams (cyclomatic 18) common/postmanLib.js:363
- exports (cyclomatic 18) common/diff-view.js:5
- (anonymous) (cyclomatic 18) client/reducer/modules/user.js:42
- render (cyclomatic 18) client/containers/User/Profile.js:207
- timeago (cyclomatic 17) common/utils.js:205
- up (cyclomatic 17) server/controllers/interface.js:668
- getProjectRole (cyclomatic 17) server/controllers/base.js:217
- LLM evaluation failed
- exports (cognitive 76) server/middleware/mockServer.js:144
- (anonymous) (cognitive 63) client/containers/Project/Interface/InterfaceList/InterfaceEditForm.js:228
- mockToArr (cognitive 55) client/components/MockDoc/MockDoc.js:131
- importPostman (cognitive 54) exts/yapi-plugin-import-postman/client.js:122
- handleAddInterface (cognitive 38) common/HandleImportData.js:60
- mockValidator (cognitive 37) server/middleware/mockServer.js:90
- render (cognitive 36) client/containers/Project/Interface/InterfaceList/View.js:256
- importHar (cognitive 35) exts/yapi-plugin-import-har/client.js:116
- parse (cognitive 33) common/mock-extra.js:27
- exports (cognitive 33) common/diff-view.js:5
- handleParams (cognitive 31) common/postmanLib.js:363
- getProjectRole (cognitive 27) server/controllers/base.js:217
- render (cognitive 27) client/components/Postman/Postman.js:570
- handleSwagger (cognitive 26) exts/yapi-plugin-import-swagger/run.js:149
- timeago (cognitive 23) common/utils.js:205
- render (cognitive 23) client/containers/User/Profile.js:207
- getCaseListByVariableParams (cognitive 21) server/controllers/interfaceCol.js:219
- up (cognitive 21) server/controllers/interface.js:668
- saveCase (cognitive 21) exts/yapi-plugin-advanced-mock/controller.js:94
- CompareObj (cognitive 20) common/lib.js:43
- matchApi (cognitive 20) server/middleware/mockServer.js:14
- init (cognitive 20) server/controllers/base.js:21
- runCaseScript (cognitive 18) server/utils/commons.js:529
- (anonymous) (cognitive 18) client/reducer/modules/user.js:42
- add (cognitive 17) server/controllers/interface.js:200
- createReqBody (cognitive 16) common/markdown.js:121
- project (cognitive 16) server/controllers/user.js:674
- list (cognitive 16) server/controllers/group.js:401
- handleResponse (cognitive 16) exts/yapi-plugin-import-swagger/run.js:293
- (anonymous) (cognitive 16) exts/yapi-plugin-export-swagger2-data/controller.js:158
- getParamsKey (cognitive 16) exts/yapi-plugin-advanced-mock/MockCol/CaseDesModal.js:128
- onMouseDown (cognitive 16) client/components/EasyDragSort/EasyDragSort.js:67
Recommendation — 3 finding(s)
- Test reliability not included
- Rotate the exposed credentials — git history can't be un-committed
- No tests found
Info — 1 finding(s)
- No exposed public API
Appendix B — Reproduction & audit trail
| Dimension | Tool | Version | Command | Findings | Raw output |
|---|---|---|---|---|---|
| D28 · Secrets (history) | gitleaks | — | gitleaks detect --no-banner --report-format json --report-path /dev/stdout --exit-code 0 --source . | 28 | artifacts/raw/gitleaks-history.json |
| D29 · Static Analysis (SAST) | semgrep | — | semgrep --config /opt/semgrep-rules/security-audit.yml --config /opt/semgrep-rules/owasp-top-ten.yml --json --quiet --timeout 0 --metrics off . | 0 | — |
| D30 · Dependency Vulnerabilities | dotnet (no .NET solution) | — | dotnet (no .NET solution): not present in this environment | 0 | — |
| D31 · IaC & Container Security | trivy | — | trivy: not applicable — No Infrastructure-as-Code or container manifests found (Dockerfile, Terraform, Kubernetes/Helm, CloudFormation); nothing to scan. | 0 | — |
| D32 · Data Compliance (PII/GDPR) | semgrep | — | semgrep: not applicable — No PII/GDPR ruleset is bundled (the public p/gdpr semgrep pack was retired) — data compliance is not assessed in this scan. | 0 | — |
| D33 · JS/npm Dependency Vulnerabilities | trivy | — | trivy fs --scanners vuln --format json --quiet --severity CRITICAL,HIGH,MEDIUM,LOW --skip-dirs **/bin/** --skip-dirs **/obj/** . --skip-db-update | 0 | — |
| D36 · Supply-chain Provenance & Signing | provenance | — | provenance: not applicable — No CI/build pipeline found (.github/workflows, .gitlab-ci.yml, azure-pipelines.yml, Jenkinsfile, .circleci); there is no build to attest provenance for. | 0 | — |
| D38 · OSV Dependency Vulnerabilities | osv-scanner | — | osv-scanner --format json --recursive . | 423 | artifacts/raw/osv-scanner.json |