Executive summary
Varagos/MinasPhone scores 36 out of 100, but it is rated At risk while a critical security finding is open; the score itself is unchanged. The project hasn't declared a stable release yet, so its lenses are rated against a lower bar than a released product's, while the score itself is worked out as for any repository.
The weakest lens counts most in the score, and here that is Production Readiness, which scores 21 and is rated Weak. Its Test Distribution dimension scores 0 out of 10, and until that reaches 4, Production Readiness can't be rated higher than Adequate. The first actions are to resolve the 1 Inverted test pyramid finding(s) in Test Distribution and to resolve the 81 Deprecated finding(s) in Dependency Hygiene, and if Production Readiness rose to 70, the score would rise from 36 to about 49.
133 findings are critical: 100 in Dependency Vulnerabilities, 13 in Dependency Hygiene, nine in Static Analysis (SAST), seven in Circular Imports, two in IaC & Container Security, one in Keyboard semantics and one in Secrets (history).
Watchdog couldn't measure Navigability, Platform End-of-Life and Runtime accessibility in this code, so those dimensions are left out of the score. The product code is 30,846 lines, which would take an estimated 0.3 person-years, around €44,000, to rebuild.
If Production Readiness rose from 21 to 70, the lowest score rated Strong, the score would rise from 36 to about 49.
New since the last survey: 49 findings
- Dependency Hygiene · Deprecated: recharts admin/package.json
- Dependency Hygiene · Deprecated: uuid REDACTED
- Dependency Hygiene · Deprecated: eslint REDACTED
- Dependency Hygiene · Deprecated: eslint server/package.json
- Dependency Hygiene · Deprecated: supertest server/package.json
- Secrets (history) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Static Analysis (SAST) · REDACTED
- Dependency Vulnerabilities · REDACTED
- Dependency Vulnerabilities · REDACTED
- Dependency Vulnerabilities · REDACTED
- Supply-chain Provenance & Signing · REDACTED
- Large Files · admin/src/api/server/api.ts: 1885 lines of code admin/src/api/server/api.ts
- Large Files · client/src/api/server/api.ts: 1284 lines of code client/src/api/server/api.ts
- Large Files · client/src/app/[locale]/information/privacy-policy/page.tsx: 527 lines of code client/src/app/[locale]/information/privacy-policy/page.tsx
- Large Files · REDACTED: 406 lines of code REDACTED
- Dependency Hygiene · Unlisted import '@radix-ui/react-visually-hidden' client/src/components/Navbar/AppDrawer/AppDrawer.tsx
- Dependency Hygiene · Unlisted import 'use-intl' client/src/app/[locale]/cart/page.tsx
- Dependency Hygiene · Unlisted import 'cookiejar' server/tests/test-utils/ApiClient.ts
- Dependency Hygiene · Unlisted import 'express' REDACTED
- Incompleteness & stubs · Both branches are identical — the condition decides nothing server/src/logger/winston.logger.ts
- Visual & motion safety · Animation without a prefers-reduced-motion guard client/src/components/Spinner/Spinner.tsx
- Visual & motion safety · Focus outline removed inline client/src/components/ui/input.tsx
Fixed since the last survey: 200 findings
- Secrets (history) · Secret: generic-api-key a.html
- Dependency Vulnerabilities · Critical vulnerability: REDACTED REDACTED
- Dependency Vulnerabilities · Medium vulnerability: [GHSA redacted] REDACTED
- Supply-chain Provenance & Signing REDACTED
- Code Duplication · Duplicated block (16 lines × 2 locations) REDACTED
- Code Duplication · Duplicated block (16 lines × 2 locations) server/src/modules/product-catalog/application/attributes/queries/find-attributes/find-attributes.handler.ts
- Code Duplication · Duplicated block with local edits (16 matched lines × 2 locations) server/src/modules/product-catalog/application/products/commands/reduce-products-stock/reduce-products-stock.handler.ts
- Code Duplication · Duplicated block (15 lines × 2 locations) admin/src/resources/attributes/AttributeList.tsx
- Code Duplication · Duplicated block (15 lines × 2 locations) REDACTED
- Code Duplication · Duplicated block (15 lines × 2 locations) server/src/modules/orders/application/orders/commands/checkout-order/checkout-order.handler.ts
- Code Duplication · Duplicated block (14 lines × 2 locations) client/src/app/[locale]/checkout/components/CheckoutController/PaymentForm.tsx
- Code Duplication · Duplicated block (14 lines × 4 locations) REDACTED
- Code Duplication · Duplicated block (14 lines × 4 locations) server/src/modules/orders/application/orders/queries/find-order-by-id/find-order-by-id.handler.ts
- Code Duplication · Duplicated block with local edits (14 matched lines × 2 locations) server/src/modules/orders/infra/database/order.repository.ts
- Code Duplication · Duplicated block (14 lines × 2 locations) server/src/modules/product-catalog/application/images/event-handlers/delete-image-after-product-deletion.domain-event-handler.ts
- Code Duplication · Duplicated block (13 lines × 8 locations) REDACTED
- Code Duplication · Duplicated block (13 lines × 4 locations) REDACTED
- Code Duplication · Duplicated block with local edits (13 matched lines × 2 locations) server/src/modules/product-catalog/application/categories/queries/find-categories/find-categories.handler.ts
- Code Duplication · Duplicated block (13 lines × 2 locations) server/src/modules/product-catalog/application/product-types/dtos/product-type.response.dto.ts
- Code Duplication · Duplicated block (12 lines × 2 locations) admin/src/resources/categories/categories.tsx
- Code Duplication · Duplicated block (12 lines × 2 locations) client/src/utils/cookieConsentHelpers.ts
- Code Duplication · Duplicated block (12 lines × 2 locations) REDACTED
- Code Duplication · Duplicated block (12 lines × 2 locations) REDACTED
- Code Duplication · Duplicated block (12 lines × 3 locations) server/src/modules/orders/application/carts/commands/add-line-item/add-cart-line-item.handler.ts
- Code Duplication · Duplicated block (11 lines × 3 locations) client/src/app/[locale]/components/PopularCategories/index.tsx
- Code Duplication · Duplicated block (11 lines × 3 locations) client/src/app/[locale]/information/return-policy/page.tsx
- Code Duplication · Duplicated block (11 lines × 2 locations) REDACTED
- Code Duplication · Duplicated block (11 lines × 2 locations) server/src/modules/product-catalog/application/attributes/commands/create-attribute/create-attribute.request.dto.ts
- Code Duplication · Duplicated block (11 lines × 3 locations) server/src/modules/product-catalog/application/categories/commands/update-category/update-category.handler.ts
- Code Duplication · Duplicated block (10 lines × 2 locations) admin/src/resources/products/components/AttributesSubformEdit.tsx
- Code Duplication · Duplicated block (10 lines × 2 locations) client/src/app/[locale]/information/user-terms/page.tsx
- Code Duplication · Duplicated block (10 lines × 3 locations) client/src/components/ui/dialog.tsx
- Code Duplication · Duplicated block (10 lines × 2 locations) server/src/modules/product-catalog/application/product-types/commands/create-product-type/create-product-type.handler.ts
- Code Duplication · Duplicated block (10 lines × 2 locations) server/src/modules/product-catalog/application/product-types/queries/find-product-type/find-product-type.handler.ts
- Code Duplication · Duplicated block (9 lines × 2 locations) admin/src/pages/dashboard/Dashboard.tsx
- Code Duplication · Duplicated block (9 lines × 2 locations) admin/src/resources/categories/categories.tsx
- Code Duplication · Duplicated block (9 lines × 2 locations) client/src/api/server/cart.ts
- Code Duplication · Duplicated block with local edits (9 matched lines × 2 locations) client/src/api/server/config/index.ts
- Code Duplication · Duplicated block (9 lines × 2 locations) server/src/modules/orders/domain/order.entity.ts
- Code Duplication · Duplicated block (8 lines × 3 locations) admin/src/resources/products/components/AttributesSubform.tsx
- Code Duplication · Duplicated block (8 lines × 3 locations) client/src/app/[locale]/information/privacy-policy/page.tsx
- Code Duplication · Duplicated block (8 lines × 3 locations) REDACTED
- Code Duplication · Duplicated block (8 lines × 2 locations) server/src/libs/api/query-params.request.ts
- Code Duplication · Duplicated block (8 lines × 3 locations) server/src/modules/orders/application/orders/sagas/checkout-order/order-processing.saga.ts
- Code Duplication · Duplicated block (7 lines × 2 locations) admin/src/data/dataProvider.ts
- Code Duplication · Duplicated block (7 lines × 2 locations) client/src/app/[locale]/information/privacy-policy/page.tsx
- Code Duplication · Duplicated block (7 lines × 3 locations) server/src/libs/db/sql-repository.base.ts
- Code Duplication · Duplicated block (7 lines × 2 locations) server/src/modules/orders/application/orders/commands/confirm-order/confirm-order.handler.ts
- Code Duplication · Duplicated block (7 lines × 3 locations) server/src/modules/orders/infra/database/order.repository.ts
- Code Duplication · Duplicated block (7 lines × 3 locations) server/src/modules/product-catalog/application/attributes/queries/find-attribute/find-attribute.handler.ts
- Code Duplication · Duplicated block (7 lines × 2 locations) server/src/modules/product-catalog/infra/database/product-type.repository.ts
- Code Duplication · Duplicated block with local edits (6 matched lines × 2 locations) client/src/api/mock/cart.ts
- Code Duplication · Duplicated block with local edits (5 matched lines × 2 locations) server/src/modules/product-catalog/infra/database/attribute.repository.ts
- Code Duplication · Duplicated block with local edits (5 matched lines × 2 locations) server/src/modules/product-catalog/infra/database/category.repository.ts
- Test Coverage · No test reaches this file admin/src/resources/attributes/AttributeQuickCreateForm.tsx
- Test Coverage · No test reaches this file client/src/api/mock/products.ts
- Test Coverage · No test reaches this file admin/src/resources/productTypes/ProductTypeEdit.tsx
- Test Coverage · No test reaches this file admin/src/pages/Login/Login.tsx
- Test Coverage · No test reaches this file admin/src/resources/products/Aside.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/checkout/components/CheckoutController/Account.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/sitemap.ts
- Test Coverage · No test reaches this file client/src/app/[locale]/checkout/components/CheckoutController/Shipping.tsx
- Test Coverage · No test reaches this file client/src/components/ui/breadcrumb.tsx
- Test Coverage · No test reaches this file client/src/components/Products/ProductsLayout.tsx
- Test Coverage · No test reaches this file client/src/components/Navbar/AppDrawer/AppDrawer.tsx
- Test Coverage · No test reaches this file admin/src/layout/themes.tsx
- Test Coverage · No test reaches this file client/src/context/ToastProvider.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/layout.tsx
- Test Coverage · No test reaches this file client/src/utils/cookieConsentHelpers.ts
- Test Coverage · No test reaches this file client/src/app/[locale]/cart/page.tsx
- Test Coverage · No test reaches this file client/src/components/Products/Filter/Filter.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/not-found.tsx
- Test Coverage · No test reaches this file client/src/components/ui/card.tsx
- Test Coverage · No test reaches this file admin/src/pages/dashboard/OldDashboard/DashboardContent.tsx
- Test Coverage · No test reaches this file admin/src/data/addUploadCapabilitiesbase64.tsx
- Test Coverage · No test reaches this file admin/src/resources/orders/MobileGrid.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/components/Services/data.tsx
- Test Coverage · No test reaches this file client/src/components/Navbar/LanguageSelector.tsx
- Test Coverage · No test reaches this file client/src/components/common/BreadcrumbNav.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/order-tracking/page.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/components/CategoryItem/CategoryItem.tsx
- Test Coverage · No test reaches this file REDACTED
- Test Coverage · No test reaches this file admin/src/resources/categories/categories.tsx
- Test Coverage · No test reaches this file client/src/api/server/products.ts
- Test Coverage · No test reaches this file admin/src/pages/dashboard/OldDashboard/Chart.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/components/HeroSection/index.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/checkout/components/CheckoutController/FormInput.tsx
- Test Coverage · No test reaches this file admin/src/pages/Login/signInHandler.tsx
- Test Coverage · No test reaches this file admin/src/resources/products/ProductList.tsx
- Test Coverage · No test reaches this file client/src/api/mock/categories.ts
- Test Coverage · No test reaches this file admin/src/pages/dashboard/Welcome.tsx
- Test Coverage · No test reaches this file client/src/components/Products/ProductCard/AddToCartProductButton.tsx
- Test Coverage · No test reaches this file admin/src/data/addUploadCapabilities.tsx
- Test Coverage · No test reaches this file admin/src/pages/configuration/Configuration.tsx
- Test Coverage · No test reaches this file client/src/app/[locale]/components/PopularCategories/index.tsx
- Test Coverage · No test reaches this file admin/src/resources/products/GridList.tsx
- Test Coverage · No test reaches this file client/src/components/CookieConsent/CookieConsent.tsx
- Test Coverage · No test reaches this file client/src/components/ui/alert.tsx
- Test Coverage · No test reaches this file admin/src/resources/orders/OrderItem.tsx
- Test Coverage · No test reaches this file client/src/components/common/IconLinkButton.tsx
Rebuild cost: about €44,000 · Estimate
| Cost to rebuild | €14,000 to €73,000, for 241 to 766 hours of work, about 0.3 person-years for one engineer |
|---|---|
| Difficulty of the problem | High, judged from the kind of software and how it is built. Code that solves a harder problem, such as payments or medical bookings, costs more per line to write. |
| Quality factor | ×0.7 at a score of 36. Better code takes more work to write, so the factor is ×1.0 at a score of 70, lower below it and higher above it. |
| Size | Medium: 30,846 lines of product code |
Top priorities
Diagnosis: what the measurements show together
Architecture: which parts of the code depend on which
| uses → | 1 admin.src.api.server.api | 2 admin.src.dto.base | 3 admin.src.types | 4 client.src.api.types.products | 5 server.src.libs.ddd.domain-event_base | 6 server.src.libs.ddd.entity_base | 7 server.src.libs.ddd.mapper_interface | 8 server.src.libs.ddd.query_base | 9 server.src.libs.ddd.value-object_base | 10 server.src.libs.exceptions.exception_base | 11 server.src.modules.product-catalog.domain.ports.attribute_repository_port | 12 server.src.modules.product-catalog.domain.ports.category_repository_port | 13 server.src.modules.product-catalog.domain.ports.product-type_repository_port | 14 server.src.modules.product-catalog.domain.ports.product_repository_port | 15 server.src.modules.product-catalog.infra.services.image-search.google-image_search_service | 16 admin.src.dto.order | 17 client.src.api | 18 server.src.libs.ddd.aggregate-root_base | 19 server.src.libs.ddd.repository_port | 20 server.src.libs.exceptions.exceptions | 21 server.src.modules.product-catalog.infra.mappers.category_mapper | 22 server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface | 23 server.tests.test-utils.TestContext | 24 client.src.api.server.orders | 25 server.src.api.controllers.orders_http_controller | 26 server.src.libs.api.paginated_response_base | 27 server.src.libs.db.sql-repository_base | 28 server.src.modules.orders.domain.order_entity | 29 server.src.modules.product-catalog.domain.attribute_entity | 30 server.src.modules.product-catalog.domain.product-type_entity | 31 server.src.modules.product-catalog.domain.product_entity | 32 server.src.modules.orders.domain.ports.order_repository_port | 33 server.src.modules.orders.infra.mappers.order_mapper | 34 server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler | 35 server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler | 36 server.src.modules.product-catalog.infra.database.attribute_repository | 37 server.src.modules.product-catalog.infra.database.product_repository | 38 server.src.modules.product-catalog.infra.mappers.product-type_mapper | 39 server.src.modules.orders.infra.database.order_repository | 40 server.src.modules.product-catalog.infra.database.product-type_repository |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 1 admin.src.api.server.api | ||||||||||||||||||||||||||||||||||||||||
| 2 admin.src.dto.base | ||||||||||||||||||||||||||||||||||||||||
| 3 admin.src.types | ||||||||||||||||||||||||||||||||||||||||
| 4 client.src.api.types.products | ||||||||||||||||||||||||||||||||||||||||
| 5 server.src.libs.ddd.domain-event_base | ||||||||||||||||||||||||||||||||||||||||
| 6 server.src.libs.ddd.entity_base | ||||||||||||||||||||||||||||||||||||||||
| 7 server.src.libs.ddd.mapper_interface | ||||||||||||||||||||||||||||||||||||||||
| 8 server.src.libs.ddd.query_base | ||||||||||||||||||||||||||||||||||||||||
| 9 server.src.libs.ddd.value-object_base | ||||||||||||||||||||||||||||||||||||||||
| 10 server.src.libs.exceptions.exception_base | ||||||||||||||||||||||||||||||||||||||||
| 11 server.src.modules.product-catalog.domain.ports.attribute_repository_port | ||||||||||||||||||||||||||||||||||||||||
| 12 server.src.modules.product-catalog.domain.ports.category_repository_port | ||||||||||||||||||||||||||||||||||||||||
| 13 server.src.modules.product-catalog.domain.ports.product-type_repository_port | ||||||||||||||||||||||||||||||||||||||||
| 14 server.src.modules.product-catalog.domain.ports.product_repository_port | ||||||||||||||||||||||||||||||||||||||||
| 15 server.src.modules.product-catalog.infra.services.image-search.google-image_search_service | ||||||||||||||||||||||||||||||||||||||||
| 16 admin.src.dto.order | 1 | |||||||||||||||||||||||||||||||||||||||
| 17 client.src.api | 1 | |||||||||||||||||||||||||||||||||||||||
| 18 server.src.libs.ddd.aggregate-root_base | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 19 server.src.libs.ddd.repository_port | 1 | |||||||||||||||||||||||||||||||||||||||
| 20 server.src.libs.exceptions.exceptions | 6 | |||||||||||||||||||||||||||||||||||||||
| 21 server.src.modules.product-catalog.infra.mappers.category_mapper | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 22 server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface | 1 | |||||||||||||||||||||||||||||||||||||||
| 23 server.tests.test-utils.TestContext | 1 | |||||||||||||||||||||||||||||||||||||||
| 24 client.src.api.server.orders | 1 | 1 | 1 | |||||||||||||||||||||||||||||||||||||
| 25 server.src.api.controllers.orders_http_controller | 4 | 1 | 1 | |||||||||||||||||||||||||||||||||||||
| 26 server.src.libs.api.paginated_response_base | 1 | |||||||||||||||||||||||||||||||||||||||
| 27 server.src.libs.db.sql-repository_base | 1 | 2 | ||||||||||||||||||||||||||||||||||||||
| 28 server.src.modules.orders.domain.order_entity | 1 | 1 | ||||||||||||||||||||||||||||||||||||||
| 29 server.src.modules.product-catalog.domain.attribute_entity | 1 | |||||||||||||||||||||||||||||||||||||||
| 30 server.src.modules.product-catalog.domain.product-type_entity | 1 | |||||||||||||||||||||||||||||||||||||||
| 31 server.src.modules.product-catalog.domain.product_entity | 1 | |||||||||||||||||||||||||||||||||||||||
| 32 server.src.modules.orders.domain.ports.order_repository_port | 1 | |||||||||||||||||||||||||||||||||||||||
| 33 server.src.modules.orders.infra.mappers.order_mapper | 1 | 1 | 1 | |||||||||||||||||||||||||||||||||||||
| 34 server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler | 1 | 1 | 1 | 1 | ||||||||||||||||||||||||||||||||||||
| 35 server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler | 1 | 1 | 1 | 1 | ||||||||||||||||||||||||||||||||||||
| 36 server.src.modules.product-catalog.infra.database.attribute_repository | 1 | 1 | 1 | |||||||||||||||||||||||||||||||||||||
| 37 server.src.modules.product-catalog.infra.database.product_repository | 1 | 1 | 1 | |||||||||||||||||||||||||||||||||||||
| 38 server.src.modules.product-catalog.infra.mappers.product-type_mapper | 1 | 1 | 1 | |||||||||||||||||||||||||||||||||||||
| 39 server.src.modules.orders.infra.database.order_repository | 1 | 1 | 1 | 1 | ||||||||||||||||||||||||||||||||||||
| 40 server.src.modules.product-catalog.infra.database.product-type_repository | 1 | 1 | 1 | 1 |
16→3 admin.src.dto.order uses admin.src.types✕
- Connections
- 1 pair of types, where a type in admin.src.dto.order uses a type in admin.src.types.
- The types involved
admin.src.dto.order.OrderResponseDto→admin.src.types.OrderStatus
- Direction
- admin.src.dto.order uses admin.src.types, so a change to admin.src.types can break admin.src.dto.order, while a change to admin.src.dto.order can't break admin.src.types.
- Position
- Below the diagonal: admin.src.dto.order uses a module listed above it, which is the healthy direction.
17→4 client.src.api uses client.src.api.types.products✕
- Connections
- 1 pair of types, where a type in client.src.api uses a type in client.src.api.types.products.
- The types involved
client.src.api.Api→client.src.api.types.products.IProductsApi
- Direction
- client.src.api uses client.src.api.types.products, so a change to client.src.api.types.products can break client.src.api, while a change to client.src.api can't break client.src.api.types.products.
- Position
- Below the diagonal: client.src.api uses a module listed above it, which is the healthy direction.
18→5 server.src.libs.ddd.aggregate-root_base uses server.src.libs.ddd.domain-event_base✕
- Connections
- 1 pair of types, where a type in server.src.libs.ddd.aggregate-root_base uses a type in server.src.libs.ddd.domain-event_base.
- The types involved
server.src.libs.ddd.aggregate-root_base.AggregateRoot→server.src.libs.ddd.domain-event_base.DomainEvent
- Direction
- server.src.libs.ddd.aggregate-root_base uses server.src.libs.ddd.domain-event_base, so a change to server.src.libs.ddd.domain-event_base can break server.src.libs.ddd.aggregate-root_base, while a change to server.src.libs.ddd.aggregate-root_base can't break server.src.libs.ddd.domain-event_base.
- Position
- Below the diagonal: server.src.libs.ddd.aggregate-root_base uses a module listed above it, which is the healthy direction.
18→6 server.src.libs.ddd.aggregate-root_base uses server.src.libs.ddd.entity_base✕
- Connections
- 1 pair of types, where a type in server.src.libs.ddd.aggregate-root_base uses a type in server.src.libs.ddd.entity_base.
- The types involved
server.src.libs.ddd.aggregate-root_base.AggregateRoot→server.src.libs.ddd.entity_base.Entity
- Direction
- server.src.libs.ddd.aggregate-root_base uses server.src.libs.ddd.entity_base, so a change to server.src.libs.ddd.entity_base can break server.src.libs.ddd.aggregate-root_base, while a change to server.src.libs.ddd.aggregate-root_base can't break server.src.libs.ddd.entity_base.
- Position
- Below the diagonal: server.src.libs.ddd.aggregate-root_base uses a module listed above it, which is the healthy direction.
19→6 server.src.libs.ddd.repository_port uses server.src.libs.ddd.entity_base✕
- Connections
- 1 pair of types, where a type in server.src.libs.ddd.repository_port uses a type in server.src.libs.ddd.entity_base.
- The types involved
server.src.libs.ddd.repository_port.RepositoryPort→server.src.libs.ddd.entity_base.Entity
- Direction
- server.src.libs.ddd.repository_port uses server.src.libs.ddd.entity_base, so a change to server.src.libs.ddd.entity_base can break server.src.libs.ddd.repository_port, while a change to server.src.libs.ddd.repository_port can't break server.src.libs.ddd.entity_base.
- Position
- Below the diagonal: server.src.libs.ddd.repository_port uses a module listed above it, which is the healthy direction.
20→10 server.src.libs.exceptions.exceptions uses server.src.libs.exceptions.exception_base✕
- Connections
- 6 pairs of types, where a type in server.src.libs.exceptions.exceptions uses a type in server.src.libs.exceptions.exception_base.
- The types involved
server.src.libs.exceptions.exceptions.ArgumentInvalidException→server.src.libs.exceptions.exception_base.ExceptionBaseserver.src.libs.exceptions.exceptions.ArgumentNotProvidedException→server.src.libs.exceptions.exception_base.ExceptionBaseserver.src.libs.exceptions.exceptions.ArgumentOutOfRangeException→server.src.libs.exceptions.exception_base.ExceptionBaseserver.src.libs.exceptions.exceptions.ConflictException→server.src.libs.exceptions.exception_base.ExceptionBaseserver.src.libs.exceptions.exceptions.InternalServerErrorException→server.src.libs.exceptions.exception_base.ExceptionBaseserver.src.libs.exceptions.exceptions.NotFoundException→server.src.libs.exceptions.exception_base.ExceptionBase
- Direction
- server.src.libs.exceptions.exceptions uses server.src.libs.exceptions.exception_base, so a change to server.src.libs.exceptions.exception_base can break server.src.libs.exceptions.exceptions, while a change to server.src.libs.exceptions.exceptions can't break server.src.libs.exceptions.exception_base.
- Position
- Below the diagonal: server.src.libs.exceptions.exceptions uses a module listed above it, which is the healthy direction.
21→1 server.src.modules.product-catalog.infra.mappers.category_mapper uses admin.src.api.server.api✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.mappers.category_mapper uses a type in admin.src.api.server.api.
- The types involved
server.src.modules.product-catalog.infra.mappers.category_mapper.CategoryMapper→admin.src.api.server.api.CategoryResponseDto
- Direction
- server.src.modules.product-catalog.infra.mappers.category_mapper uses admin.src.api.server.api, so a change to admin.src.api.server.api can break server.src.modules.product-catalog.infra.mappers.category_mapper, while a change to server.src.modules.product-catalog.infra.mappers.category_mapper can't break admin.src.api.server.api.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.mappers.category_mapper uses a module listed above it, which is the healthy direction.
21→7 server.src.modules.product-catalog.infra.mappers.category_mapper uses server.src.libs.ddd.mapper_interface✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.mappers.category_mapper uses a type in server.src.libs.ddd.mapper_interface.
- The types involved
server.src.modules.product-catalog.infra.mappers.category_mapper.CategoryMapper→server.src.libs.ddd.mapper_interface.Mapper
- Direction
- server.src.modules.product-catalog.infra.mappers.category_mapper uses server.src.libs.ddd.mapper_interface, so a change to server.src.libs.ddd.mapper_interface can break server.src.modules.product-catalog.infra.mappers.category_mapper, while a change to server.src.modules.product-catalog.infra.mappers.category_mapper can't break server.src.libs.ddd.mapper_interface.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.mappers.category_mapper uses a module listed above it, which is the healthy direction.
22→15 server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface uses server.src.modules.product-catalog.infra.services.image-search.google-image_search_service✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface uses a type in server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.
- The types involved
server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface.ICloudStorageServicePort→server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.Image
- Direction
- server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface uses server.src.modules.product-catalog.infra.services.image-search.google-image_search_service, so a change to server.src.modules.product-catalog.infra.services.image-search.google-image_search_service can break server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface, while a change to server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface can't break server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.services.cloud-storage.cloud-storage_interface uses a module listed above it, which is the healthy direction.
23→15 server.tests.test-utils.TestContext uses server.src.modules.product-catalog.infra.services.image-search.google-image_search_service✕
- Connections
- 1 pair of types, where a type in server.tests.test-utils.TestContext uses a type in server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.
- The types involved
server.tests.test-utils.TestContext.TestContext→server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.Context
- Direction
- server.tests.test-utils.TestContext uses server.src.modules.product-catalog.infra.services.image-search.google-image_search_service, so a change to server.src.modules.product-catalog.infra.services.image-search.google-image_search_service can break server.tests.test-utils.TestContext, while a change to server.tests.test-utils.TestContext can't break server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.
- Position
- Below the diagonal: server.tests.test-utils.TestContext uses a module listed above it, which is the healthy direction.
24→1 client.src.api.server.orders uses admin.src.api.server.api✕
- Connections
- 1 pair of types, where a type in client.src.api.server.orders uses a type in admin.src.api.server.api.
- The types involved
client.src.api.server.orders.OrdersApi→admin.src.api.server.api.Api
- Direction
- client.src.api.server.orders uses admin.src.api.server.api, so a change to admin.src.api.server.api can break client.src.api.server.orders, while a change to client.src.api.server.orders can't break admin.src.api.server.api.
- Position
- Below the diagonal: client.src.api.server.orders uses a module listed above it, which is the healthy direction.
24→3 client.src.api.server.orders uses admin.src.types✕
- Connections
- 1 pair of types, where a type in client.src.api.server.orders uses a type in admin.src.types.
- The types involved
client.src.api.server.orders.OrdersApi→admin.src.types.Order
- Direction
- client.src.api.server.orders uses admin.src.types, so a change to admin.src.types can break client.src.api.server.orders, while a change to client.src.api.server.orders can't break admin.src.types.
- Position
- Below the diagonal: client.src.api.server.orders uses a module listed above it, which is the healthy direction.
24→16 client.src.api.server.orders uses admin.src.dto.order✕
- Connections
- 1 pair of types, where a type in client.src.api.server.orders uses a type in admin.src.dto.order.
- The types involved
client.src.api.server.orders.OrdersApi→admin.src.dto.order.OrderResponseDto
- Direction
- client.src.api.server.orders uses admin.src.dto.order, so a change to admin.src.dto.order can break client.src.api.server.orders, while a change to client.src.api.server.orders can't break admin.src.dto.order.
- Position
- Below the diagonal: client.src.api.server.orders uses a module listed above it, which is the healthy direction.
25→1 server.src.api.controllers.orders_http_controller uses admin.src.api.server.api✕
- Connections
- 4 pairs of types, where a type in server.src.api.controllers.orders_http_controller uses a type in admin.src.api.server.api.
- The types involved
server.src.api.controllers.orders_http_controller.OrdersHttpController→admin.src.api.server.api.CheckoutOrderRequestDtoserver.src.api.controllers.orders_http_controller.OrdersHttpController→admin.src.api.server.api.OrderCreatedResponseDtoserver.src.api.controllers.orders_http_controller.OrdersHttpController→admin.src.api.server.api.OrderPaginatedResponseDtoserver.src.api.controllers.orders_http_controller.OrdersHttpController→admin.src.api.server.api.UpdateOrderStatusRequestDto
- Direction
- server.src.api.controllers.orders_http_controller uses admin.src.api.server.api, so a change to admin.src.api.server.api can break server.src.api.controllers.orders_http_controller, while a change to server.src.api.controllers.orders_http_controller can't break admin.src.api.server.api.
- Position
- Below the diagonal: server.src.api.controllers.orders_http_controller uses a module listed above it, which is the healthy direction.
25→15 server.src.api.controllers.orders_http_controller uses server.src.modules.product-catalog.infra.services.image-search.google-image_search_service✕
- Connections
- 1 pair of types, where a type in server.src.api.controllers.orders_http_controller uses a type in server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.
- The types involved
server.src.api.controllers.orders_http_controller.OrdersHttpController→server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.Request
- Direction
- server.src.api.controllers.orders_http_controller uses server.src.modules.product-catalog.infra.services.image-search.google-image_search_service, so a change to server.src.modules.product-catalog.infra.services.image-search.google-image_search_service can break server.src.api.controllers.orders_http_controller, while a change to server.src.api.controllers.orders_http_controller can't break server.src.modules.product-catalog.infra.services.image-search.google-image_search_service.
- Position
- Below the diagonal: server.src.api.controllers.orders_http_controller uses a module listed above it, which is the healthy direction.
25→16 server.src.api.controllers.orders_http_controller uses admin.src.dto.order✕
- Connections
- 1 pair of types, where a type in server.src.api.controllers.orders_http_controller uses a type in admin.src.dto.order.
- The types involved
server.src.api.controllers.orders_http_controller.OrdersHttpController→admin.src.dto.order.OrderResponseDto
- Direction
- server.src.api.controllers.orders_http_controller uses admin.src.dto.order, so a change to admin.src.dto.order can break server.src.api.controllers.orders_http_controller, while a change to server.src.api.controllers.orders_http_controller can't break admin.src.dto.order.
- Position
- Below the diagonal: server.src.api.controllers.orders_http_controller uses a module listed above it, which is the healthy direction.
26→19 server.src.libs.api.paginated_response_base uses server.src.libs.ddd.repository_port✕
- Connections
- 1 pair of types, where a type in server.src.libs.api.paginated_response_base uses a type in server.src.libs.ddd.repository_port.
- The types involved
server.src.libs.api.paginated_response_base.PaginatedResponseDto→server.src.libs.ddd.repository_port.Paginated
- Direction
- server.src.libs.api.paginated_response_base uses server.src.libs.ddd.repository_port, so a change to server.src.libs.ddd.repository_port can break server.src.libs.api.paginated_response_base, while a change to server.src.libs.api.paginated_response_base can't break server.src.libs.ddd.repository_port.
- Position
- Below the diagonal: server.src.libs.api.paginated_response_base uses a module listed above it, which is the healthy direction.
27→7 server.src.libs.db.sql-repository_base uses server.src.libs.ddd.mapper_interface✕
- Connections
- 1 pair of types, where a type in server.src.libs.db.sql-repository_base uses a type in server.src.libs.ddd.mapper_interface.
- The types involved
server.src.libs.db.sql-repository_base.SqlRepositoryBase→server.src.libs.ddd.mapper_interface.Mapper
- Direction
- server.src.libs.db.sql-repository_base uses server.src.libs.ddd.mapper_interface, so a change to server.src.libs.ddd.mapper_interface can break server.src.libs.db.sql-repository_base, while a change to server.src.libs.db.sql-repository_base can't break server.src.libs.ddd.mapper_interface.
- Position
- Below the diagonal: server.src.libs.db.sql-repository_base uses a module listed above it, which is the healthy direction.
27→19 server.src.libs.db.sql-repository_base uses server.src.libs.ddd.repository_port✕
- Connections
- 2 pairs of types, where a type in server.src.libs.db.sql-repository_base uses a type in server.src.libs.ddd.repository_port.
- The types involved
server.src.libs.db.sql-repository_base.SqlRepositoryBase→server.src.libs.ddd.repository_port.Paginatedserver.src.libs.db.sql-repository_base.SqlRepositoryBase→server.src.libs.ddd.repository_port.RepositoryPort
- Direction
- server.src.libs.db.sql-repository_base uses server.src.libs.ddd.repository_port, so a change to server.src.libs.ddd.repository_port can break server.src.libs.db.sql-repository_base, while a change to server.src.libs.db.sql-repository_base can't break server.src.libs.ddd.repository_port.
- Position
- Below the diagonal: server.src.libs.db.sql-repository_base uses a module listed above it, which is the healthy direction.
28→3 server.src.modules.orders.domain.order_entity uses admin.src.types✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.domain.order_entity uses a type in admin.src.types.
- The types involved
server.src.modules.orders.domain.order_entity.OrderProps→admin.src.types.OrderStatus
- Direction
- server.src.modules.orders.domain.order_entity uses admin.src.types, so a change to admin.src.types can break server.src.modules.orders.domain.order_entity, while a change to server.src.modules.orders.domain.order_entity can't break admin.src.types.
- Position
- Below the diagonal: server.src.modules.orders.domain.order_entity uses a module listed above it, which is the healthy direction.
28→18 server.src.modules.orders.domain.order_entity uses server.src.libs.ddd.aggregate-root_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.domain.order_entity uses a type in server.src.libs.ddd.aggregate-root_base.
- The types involved
server.src.modules.orders.domain.order_entity.OrderEntity→server.src.libs.ddd.aggregate-root_base.AggregateRoot
- Direction
- server.src.modules.orders.domain.order_entity uses server.src.libs.ddd.aggregate-root_base, so a change to server.src.libs.ddd.aggregate-root_base can break server.src.modules.orders.domain.order_entity, while a change to server.src.modules.orders.domain.order_entity can't break server.src.libs.ddd.aggregate-root_base.
- Position
- Below the diagonal: server.src.modules.orders.domain.order_entity uses a module listed above it, which is the healthy direction.
29→18 server.src.modules.product-catalog.domain.attribute_entity uses server.src.libs.ddd.aggregate-root_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.domain.attribute_entity uses a type in server.src.libs.ddd.aggregate-root_base.
- The types involved
server.src.modules.product-catalog.domain.attribute_entity.AttributeEntity→server.src.libs.ddd.aggregate-root_base.AggregateRoot
- Direction
- server.src.modules.product-catalog.domain.attribute_entity uses server.src.libs.ddd.aggregate-root_base, so a change to server.src.libs.ddd.aggregate-root_base can break server.src.modules.product-catalog.domain.attribute_entity, while a change to server.src.modules.product-catalog.domain.attribute_entity can't break server.src.libs.ddd.aggregate-root_base.
- Position
- Below the diagonal: server.src.modules.product-catalog.domain.attribute_entity uses a module listed above it, which is the healthy direction.
30→18 server.src.modules.product-catalog.domain.product-type_entity uses server.src.libs.ddd.aggregate-root_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.domain.product-type_entity uses a type in server.src.libs.ddd.aggregate-root_base.
- The types involved
server.src.modules.product-catalog.domain.product-type_entity.ProductTypeEntity→server.src.libs.ddd.aggregate-root_base.AggregateRoot
- Direction
- server.src.modules.product-catalog.domain.product-type_entity uses server.src.libs.ddd.aggregate-root_base, so a change to server.src.libs.ddd.aggregate-root_base can break server.src.modules.product-catalog.domain.product-type_entity, while a change to server.src.modules.product-catalog.domain.product-type_entity can't break server.src.libs.ddd.aggregate-root_base.
- Position
- Below the diagonal: server.src.modules.product-catalog.domain.product-type_entity uses a module listed above it, which is the healthy direction.
31→18 server.src.modules.product-catalog.domain.product_entity uses server.src.libs.ddd.aggregate-root_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.domain.product_entity uses a type in server.src.libs.ddd.aggregate-root_base.
- The types involved
server.src.modules.product-catalog.domain.product_entity.ProductEntity→server.src.libs.ddd.aggregate-root_base.AggregateRoot
- Direction
- server.src.modules.product-catalog.domain.product_entity uses server.src.libs.ddd.aggregate-root_base, so a change to server.src.libs.ddd.aggregate-root_base can break server.src.modules.product-catalog.domain.product_entity, while a change to server.src.modules.product-catalog.domain.product_entity can't break server.src.libs.ddd.aggregate-root_base.
- Position
- Below the diagonal: server.src.modules.product-catalog.domain.product_entity uses a module listed above it, which is the healthy direction.
32→28 server.src.modules.orders.domain.ports.order_repository_port uses server.src.modules.orders.domain.order_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.domain.ports.order_repository_port uses a type in server.src.modules.orders.domain.order_entity.
- The types involved
server.src.modules.orders.domain.ports.order_repository_port.OrderRepositoryPort→server.src.modules.orders.domain.order_entity.OrderEntity
- Direction
- server.src.modules.orders.domain.ports.order_repository_port uses server.src.modules.orders.domain.order_entity, so a change to server.src.modules.orders.domain.order_entity can break server.src.modules.orders.domain.ports.order_repository_port, while a change to server.src.modules.orders.domain.ports.order_repository_port can't break server.src.modules.orders.domain.order_entity.
- Position
- Below the diagonal: server.src.modules.orders.domain.ports.order_repository_port uses a module listed above it, which is the healthy direction.
33→7 server.src.modules.orders.infra.mappers.order_mapper uses server.src.libs.ddd.mapper_interface✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.infra.mappers.order_mapper uses a type in server.src.libs.ddd.mapper_interface.
- The types involved
server.src.modules.orders.infra.mappers.order_mapper.OrderMapper→server.src.libs.ddd.mapper_interface.Mapper
- Direction
- server.src.modules.orders.infra.mappers.order_mapper uses server.src.libs.ddd.mapper_interface, so a change to server.src.libs.ddd.mapper_interface can break server.src.modules.orders.infra.mappers.order_mapper, while a change to server.src.modules.orders.infra.mappers.order_mapper can't break server.src.libs.ddd.mapper_interface.
- Position
- Below the diagonal: server.src.modules.orders.infra.mappers.order_mapper uses a module listed above it, which is the healthy direction.
33→16 server.src.modules.orders.infra.mappers.order_mapper uses admin.src.dto.order✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.infra.mappers.order_mapper uses a type in admin.src.dto.order.
- The types involved
server.src.modules.orders.infra.mappers.order_mapper.OrderMapper→admin.src.dto.order.OrderResponseDto
- Direction
- server.src.modules.orders.infra.mappers.order_mapper uses admin.src.dto.order, so a change to admin.src.dto.order can break server.src.modules.orders.infra.mappers.order_mapper, while a change to server.src.modules.orders.infra.mappers.order_mapper can't break admin.src.dto.order.
- Position
- Below the diagonal: server.src.modules.orders.infra.mappers.order_mapper uses a module listed above it, which is the healthy direction.
33→28 server.src.modules.orders.infra.mappers.order_mapper uses server.src.modules.orders.domain.order_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.infra.mappers.order_mapper uses a type in server.src.modules.orders.domain.order_entity.
- The types involved
server.src.modules.orders.infra.mappers.order_mapper.OrderMapper→server.src.modules.orders.domain.order_entity.OrderEntity
- Direction
- server.src.modules.orders.infra.mappers.order_mapper uses server.src.modules.orders.domain.order_entity, so a change to server.src.modules.orders.domain.order_entity can break server.src.modules.orders.infra.mappers.order_mapper, while a change to server.src.modules.orders.infra.mappers.order_mapper can't break server.src.modules.orders.domain.order_entity.
- Position
- Below the diagonal: server.src.modules.orders.infra.mappers.order_mapper uses a module listed above it, which is the healthy direction.
34→11 server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.ports.attribute_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a type in server.src.modules.product-catalog.domain.ports.attribute_repository_port.
- The types involved
server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler.CreateProductCommandHandler→server.src.modules.product-catalog.domain.ports.attribute_repository_port.AttributeRepositoryPort
- Direction
- server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.ports.attribute_repository_port, so a change to server.src.modules.product-catalog.domain.ports.attribute_repository_port can break server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler can't break server.src.modules.product-catalog.domain.ports.attribute_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a module listed above it, which is the healthy direction.
34→13 server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.ports.product-type_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a type in server.src.modules.product-catalog.domain.ports.product-type_repository_port.
- The types involved
server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler.CreateProductCommandHandler→server.src.modules.product-catalog.domain.ports.product-type_repository_port.ProductTypeRepositoryPort
- Direction
- server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.ports.product-type_repository_port, so a change to server.src.modules.product-catalog.domain.ports.product-type_repository_port can break server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler can't break server.src.modules.product-catalog.domain.ports.product-type_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a module listed above it, which is the healthy direction.
34→14 server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.ports.product_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a type in server.src.modules.product-catalog.domain.ports.product_repository_port.
- The types involved
server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler.CreateProductCommandHandler→server.src.modules.product-catalog.domain.ports.product_repository_port.ProductRepositoryPort
- Direction
- server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.ports.product_repository_port, so a change to server.src.modules.product-catalog.domain.ports.product_repository_port can break server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler can't break server.src.modules.product-catalog.domain.ports.product_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a module listed above it, which is the healthy direction.
34→31 server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.product_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a type in server.src.modules.product-catalog.domain.product_entity.
- The types involved
server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler.CreateProductCommandHandler→server.src.modules.product-catalog.domain.product_entity.ProductAttributeValuesMap
- Direction
- server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses server.src.modules.product-catalog.domain.product_entity, so a change to server.src.modules.product-catalog.domain.product_entity can break server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler can't break server.src.modules.product-catalog.domain.product_entity.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.create-product.create-product_handler uses a module listed above it, which is the healthy direction.
35→11 server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.ports.attribute_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a type in server.src.modules.product-catalog.domain.ports.attribute_repository_port.
- The types involved
server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler.UpdateProductCommandHandler→server.src.modules.product-catalog.domain.ports.attribute_repository_port.AttributeRepositoryPort
- Direction
- server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.ports.attribute_repository_port, so a change to server.src.modules.product-catalog.domain.ports.attribute_repository_port can break server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler can't break server.src.modules.product-catalog.domain.ports.attribute_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a module listed above it, which is the healthy direction.
35→13 server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.ports.product-type_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a type in server.src.modules.product-catalog.domain.ports.product-type_repository_port.
- The types involved
server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler.UpdateProductCommandHandler→server.src.modules.product-catalog.domain.ports.product-type_repository_port.ProductTypeRepositoryPort
- Direction
- server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.ports.product-type_repository_port, so a change to server.src.modules.product-catalog.domain.ports.product-type_repository_port can break server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler can't break server.src.modules.product-catalog.domain.ports.product-type_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a module listed above it, which is the healthy direction.
35→14 server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.ports.product_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a type in server.src.modules.product-catalog.domain.ports.product_repository_port.
- The types involved
server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler.UpdateProductCommandHandler→server.src.modules.product-catalog.domain.ports.product_repository_port.ProductRepositoryPort
- Direction
- server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.ports.product_repository_port, so a change to server.src.modules.product-catalog.domain.ports.product_repository_port can break server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler can't break server.src.modules.product-catalog.domain.ports.product_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a module listed above it, which is the healthy direction.
35→31 server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.product_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a type in server.src.modules.product-catalog.domain.product_entity.
- The types involved
server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler.UpdateProductCommandHandler→server.src.modules.product-catalog.domain.product_entity.ProductAttributeValuesMap
- Direction
- server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses server.src.modules.product-catalog.domain.product_entity, so a change to server.src.modules.product-catalog.domain.product_entity can break server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler, while a change to server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler can't break server.src.modules.product-catalog.domain.product_entity.
- Position
- Below the diagonal: server.src.modules.product-catalog.application.products.commands.update-product.update-product_handler uses a module listed above it, which is the healthy direction.
36→11 server.src.modules.product-catalog.infra.database.attribute_repository uses server.src.modules.product-catalog.domain.ports.attribute_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.attribute_repository uses a type in server.src.modules.product-catalog.domain.ports.attribute_repository_port.
- The types involved
server.src.modules.product-catalog.infra.database.attribute_repository.AttributeRepository→server.src.modules.product-catalog.domain.ports.attribute_repository_port.AttributeRepositoryPort
- Direction
- server.src.modules.product-catalog.infra.database.attribute_repository uses server.src.modules.product-catalog.domain.ports.attribute_repository_port, so a change to server.src.modules.product-catalog.domain.ports.attribute_repository_port can break server.src.modules.product-catalog.infra.database.attribute_repository, while a change to server.src.modules.product-catalog.infra.database.attribute_repository can't break server.src.modules.product-catalog.domain.ports.attribute_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.attribute_repository uses a module listed above it, which is the healthy direction.
36→27 server.src.modules.product-catalog.infra.database.attribute_repository uses server.src.libs.db.sql-repository_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.attribute_repository uses a type in server.src.libs.db.sql-repository_base.
- The types involved
server.src.modules.product-catalog.infra.database.attribute_repository.AttributeRepository→server.src.libs.db.sql-repository_base.SqlRepositoryBase
- Direction
- server.src.modules.product-catalog.infra.database.attribute_repository uses server.src.libs.db.sql-repository_base, so a change to server.src.libs.db.sql-repository_base can break server.src.modules.product-catalog.infra.database.attribute_repository, while a change to server.src.modules.product-catalog.infra.database.attribute_repository can't break server.src.libs.db.sql-repository_base.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.attribute_repository uses a module listed above it, which is the healthy direction.
36→29 server.src.modules.product-catalog.infra.database.attribute_repository uses server.src.modules.product-catalog.domain.attribute_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.attribute_repository uses a type in server.src.modules.product-catalog.domain.attribute_entity.
- The types involved
server.src.modules.product-catalog.infra.database.attribute_repository.AttributeRepository→server.src.modules.product-catalog.domain.attribute_entity.AttributeEntity
- Direction
- server.src.modules.product-catalog.infra.database.attribute_repository uses server.src.modules.product-catalog.domain.attribute_entity, so a change to server.src.modules.product-catalog.domain.attribute_entity can break server.src.modules.product-catalog.infra.database.attribute_repository, while a change to server.src.modules.product-catalog.infra.database.attribute_repository can't break server.src.modules.product-catalog.domain.attribute_entity.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.attribute_repository uses a module listed above it, which is the healthy direction.
37→14 server.src.modules.product-catalog.infra.database.product_repository uses server.src.modules.product-catalog.domain.ports.product_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.product_repository uses a type in server.src.modules.product-catalog.domain.ports.product_repository_port.
- The types involved
server.src.modules.product-catalog.infra.database.product_repository.ProductRepository→server.src.modules.product-catalog.domain.ports.product_repository_port.ProductRepositoryPort
- Direction
- server.src.modules.product-catalog.infra.database.product_repository uses server.src.modules.product-catalog.domain.ports.product_repository_port, so a change to server.src.modules.product-catalog.domain.ports.product_repository_port can break server.src.modules.product-catalog.infra.database.product_repository, while a change to server.src.modules.product-catalog.infra.database.product_repository can't break server.src.modules.product-catalog.domain.ports.product_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.product_repository uses a module listed above it, which is the healthy direction.
37→27 server.src.modules.product-catalog.infra.database.product_repository uses server.src.libs.db.sql-repository_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.product_repository uses a type in server.src.libs.db.sql-repository_base.
- The types involved
server.src.modules.product-catalog.infra.database.product_repository.ProductRepository→server.src.libs.db.sql-repository_base.SqlRepositoryBase
- Direction
- server.src.modules.product-catalog.infra.database.product_repository uses server.src.libs.db.sql-repository_base, so a change to server.src.libs.db.sql-repository_base can break server.src.modules.product-catalog.infra.database.product_repository, while a change to server.src.modules.product-catalog.infra.database.product_repository can't break server.src.libs.db.sql-repository_base.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.product_repository uses a module listed above it, which is the healthy direction.
37→31 server.src.modules.product-catalog.infra.database.product_repository uses server.src.modules.product-catalog.domain.product_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.product_repository uses a type in server.src.modules.product-catalog.domain.product_entity.
- The types involved
server.src.modules.product-catalog.infra.database.product_repository.ProductRepository→server.src.modules.product-catalog.domain.product_entity.ProductEntity
- Direction
- server.src.modules.product-catalog.infra.database.product_repository uses server.src.modules.product-catalog.domain.product_entity, so a change to server.src.modules.product-catalog.domain.product_entity can break server.src.modules.product-catalog.infra.database.product_repository, while a change to server.src.modules.product-catalog.infra.database.product_repository can't break server.src.modules.product-catalog.domain.product_entity.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.product_repository uses a module listed above it, which is the healthy direction.
38→1 server.src.modules.product-catalog.infra.mappers.product-type_mapper uses admin.src.api.server.api✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.mappers.product-type_mapper uses a type in admin.src.api.server.api.
- The types involved
server.src.modules.product-catalog.infra.mappers.product-type_mapper.ProductTypeMapper→admin.src.api.server.api.IdResponse
- Direction
- server.src.modules.product-catalog.infra.mappers.product-type_mapper uses admin.src.api.server.api, so a change to admin.src.api.server.api can break server.src.modules.product-catalog.infra.mappers.product-type_mapper, while a change to server.src.modules.product-catalog.infra.mappers.product-type_mapper can't break admin.src.api.server.api.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.mappers.product-type_mapper uses a module listed above it, which is the healthy direction.
38→7 server.src.modules.product-catalog.infra.mappers.product-type_mapper uses server.src.libs.ddd.mapper_interface✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.mappers.product-type_mapper uses a type in server.src.libs.ddd.mapper_interface.
- The types involved
server.src.modules.product-catalog.infra.mappers.product-type_mapper.ProductTypeMapper→server.src.libs.ddd.mapper_interface.Mapper
- Direction
- server.src.modules.product-catalog.infra.mappers.product-type_mapper uses server.src.libs.ddd.mapper_interface, so a change to server.src.libs.ddd.mapper_interface can break server.src.modules.product-catalog.infra.mappers.product-type_mapper, while a change to server.src.modules.product-catalog.infra.mappers.product-type_mapper can't break server.src.libs.ddd.mapper_interface.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.mappers.product-type_mapper uses a module listed above it, which is the healthy direction.
38→30 server.src.modules.product-catalog.infra.mappers.product-type_mapper uses server.src.modules.product-catalog.domain.product-type_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.mappers.product-type_mapper uses a type in server.src.modules.product-catalog.domain.product-type_entity.
- The types involved
server.src.modules.product-catalog.infra.mappers.product-type_mapper.ProductTypeMapper→server.src.modules.product-catalog.domain.product-type_entity.ProductTypeEntity
- Direction
- server.src.modules.product-catalog.infra.mappers.product-type_mapper uses server.src.modules.product-catalog.domain.product-type_entity, so a change to server.src.modules.product-catalog.domain.product-type_entity can break server.src.modules.product-catalog.infra.mappers.product-type_mapper, while a change to server.src.modules.product-catalog.infra.mappers.product-type_mapper can't break server.src.modules.product-catalog.domain.product-type_entity.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.mappers.product-type_mapper uses a module listed above it, which is the healthy direction.
39→27 server.src.modules.orders.infra.database.order_repository uses server.src.libs.db.sql-repository_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.infra.database.order_repository uses a type in server.src.libs.db.sql-repository_base.
- The types involved
server.src.modules.orders.infra.database.order_repository.OrderRepository→server.src.libs.db.sql-repository_base.SqlRepositoryBase
- Direction
- server.src.modules.orders.infra.database.order_repository uses server.src.libs.db.sql-repository_base, so a change to server.src.libs.db.sql-repository_base can break server.src.modules.orders.infra.database.order_repository, while a change to server.src.modules.orders.infra.database.order_repository can't break server.src.libs.db.sql-repository_base.
- Position
- Below the diagonal: server.src.modules.orders.infra.database.order_repository uses a module listed above it, which is the healthy direction.
39→28 server.src.modules.orders.infra.database.order_repository uses server.src.modules.orders.domain.order_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.infra.database.order_repository uses a type in server.src.modules.orders.domain.order_entity.
- The types involved
server.src.modules.orders.infra.database.order_repository.OrderRepository→server.src.modules.orders.domain.order_entity.OrderEntity
- Direction
- server.src.modules.orders.infra.database.order_repository uses server.src.modules.orders.domain.order_entity, so a change to server.src.modules.orders.domain.order_entity can break server.src.modules.orders.infra.database.order_repository, while a change to server.src.modules.orders.infra.database.order_repository can't break server.src.modules.orders.domain.order_entity.
- Position
- Below the diagonal: server.src.modules.orders.infra.database.order_repository uses a module listed above it, which is the healthy direction.
39→32 server.src.modules.orders.infra.database.order_repository uses server.src.modules.orders.domain.ports.order_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.infra.database.order_repository uses a type in server.src.modules.orders.domain.ports.order_repository_port.
- The types involved
server.src.modules.orders.infra.database.order_repository.OrderRepository→server.src.modules.orders.domain.ports.order_repository_port.OrderRepositoryPort
- Direction
- server.src.modules.orders.infra.database.order_repository uses server.src.modules.orders.domain.ports.order_repository_port, so a change to server.src.modules.orders.domain.ports.order_repository_port can break server.src.modules.orders.infra.database.order_repository, while a change to server.src.modules.orders.infra.database.order_repository can't break server.src.modules.orders.domain.ports.order_repository_port.
- Position
- Below the diagonal: server.src.modules.orders.infra.database.order_repository uses a module listed above it, which is the healthy direction.
39→33 server.src.modules.orders.infra.database.order_repository uses server.src.modules.orders.infra.mappers.order_mapper✕
- Connections
- 1 pair of types, where a type in server.src.modules.orders.infra.database.order_repository uses a type in server.src.modules.orders.infra.mappers.order_mapper.
- The types involved
server.src.modules.orders.infra.database.order_repository.OrderRepository→server.src.modules.orders.infra.mappers.order_mapper.OrderMapper
- Direction
- server.src.modules.orders.infra.database.order_repository uses server.src.modules.orders.infra.mappers.order_mapper, so a change to server.src.modules.orders.infra.mappers.order_mapper can break server.src.modules.orders.infra.database.order_repository, while a change to server.src.modules.orders.infra.database.order_repository can't break server.src.modules.orders.infra.mappers.order_mapper.
- Position
- Below the diagonal: server.src.modules.orders.infra.database.order_repository uses a module listed above it, which is the healthy direction.
40→13 server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.modules.product-catalog.domain.ports.product-type_repository_port✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.product-type_repository uses a type in server.src.modules.product-catalog.domain.ports.product-type_repository_port.
- The types involved
server.src.modules.product-catalog.infra.database.product-type_repository.ProductTypeRepository→server.src.modules.product-catalog.domain.ports.product-type_repository_port.ProductTypeRepositoryPort
- Direction
- server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.modules.product-catalog.domain.ports.product-type_repository_port, so a change to server.src.modules.product-catalog.domain.ports.product-type_repository_port can break server.src.modules.product-catalog.infra.database.product-type_repository, while a change to server.src.modules.product-catalog.infra.database.product-type_repository can't break server.src.modules.product-catalog.domain.ports.product-type_repository_port.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.product-type_repository uses a module listed above it, which is the healthy direction.
40→27 server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.libs.db.sql-repository_base✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.product-type_repository uses a type in server.src.libs.db.sql-repository_base.
- The types involved
server.src.modules.product-catalog.infra.database.product-type_repository.ProductTypeRepository→server.src.libs.db.sql-repository_base.SqlRepositoryBase
- Direction
- server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.libs.db.sql-repository_base, so a change to server.src.libs.db.sql-repository_base can break server.src.modules.product-catalog.infra.database.product-type_repository, while a change to server.src.modules.product-catalog.infra.database.product-type_repository can't break server.src.libs.db.sql-repository_base.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.product-type_repository uses a module listed above it, which is the healthy direction.
40→30 server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.modules.product-catalog.domain.product-type_entity✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.product-type_repository uses a type in server.src.modules.product-catalog.domain.product-type_entity.
- The types involved
server.src.modules.product-catalog.infra.database.product-type_repository.ProductTypeRepository→server.src.modules.product-catalog.domain.product-type_entity.ProductTypeEntity
- Direction
- server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.modules.product-catalog.domain.product-type_entity, so a change to server.src.modules.product-catalog.domain.product-type_entity can break server.src.modules.product-catalog.infra.database.product-type_repository, while a change to server.src.modules.product-catalog.infra.database.product-type_repository can't break server.src.modules.product-catalog.domain.product-type_entity.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.product-type_repository uses a module listed above it, which is the healthy direction.
40→38 server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.modules.product-catalog.infra.mappers.product-type_mapper✕
- Connections
- 1 pair of types, where a type in server.src.modules.product-catalog.infra.database.product-type_repository uses a type in server.src.modules.product-catalog.infra.mappers.product-type_mapper.
- The types involved
server.src.modules.product-catalog.infra.database.product-type_repository.ProductTypeRepository→server.src.modules.product-catalog.infra.mappers.product-type_mapper.ProductTypeMapper
- Direction
- server.src.modules.product-catalog.infra.database.product-type_repository uses server.src.modules.product-catalog.infra.mappers.product-type_mapper, so a change to server.src.modules.product-catalog.infra.mappers.product-type_mapper can break server.src.modules.product-catalog.infra.database.product-type_repository, while a change to server.src.modules.product-catalog.infra.database.product-type_repository can't break server.src.modules.product-catalog.infra.mappers.product-type_mapper.
- Position
- Below the diagonal: server.src.modules.product-catalog.infra.database.product-type_repository uses a module listed above it, which is the healthy direction.
At a glance — Code Health
At a glance — Architecture
At a glance — Maturity
At a glance — Readiness
At a glance — Security
At a glance — Domain Modelling
At a glance — Accessibility
At a glance — Performance
Security & Compliance — OWASP Top-10 mapping
| OWASP category | Findings | Severity |
|---|---|---|
| A03:2025 — Software Supply Chain Failures | 173 | High / Critical |
| A05:2025 — Injection | 42 | High / Critical |
| A02:2025 — Security Misconfiguration | 7 | High / Critical |
| A07:2025 — Authentication Failures | 2 | High / Critical |
Roadmap
Begin by resolving the 81 deprecated dependencies in REDACTED and addressing the inverted test pyramid to stabilize the foundation. Next, improve code coverage by targeting key files such as index.ts and constants.ts, while integrating a SAST step into CI to prevent security regressions. Finally, link this repository to the operational documentation for production backups and disaster recovery to ensure clear visibility into data protection and recovery procedures.
| Do this | Helps | Effort | Dimension |
|---|---|---|---|
| Resolve the 1 Inverted test pyramid finding(s) in Test Distribution. | +10.7 pts | Low | Test Distribution |
| Resolve the 81 Deprecated finding(s) in Dependency Hygiene — start with REDACTED (81). | +12.6 pts | Medium | Dependency Hygiene |
| Resolve the 1 No ADRs found finding(s) in ADR Quality. | +2.2 pts | Low | ADR Quality |
| Resolve the 244 Low coverage finding(s) in Code Coverage — start with index.ts (4), constants.ts (2), app.module.ts. | +8.3 pts | High | Code Coverage |
| 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. | +3.4 pts | Medium | Security & performance tooling |
| Point at where production persistence is operated — a link from this repo to the ops/platform repo that holds its backup, geo-recovery and RTO/RPO controls — so the DR posture is reviewable from here; the dev-stack volume is not the thing to protect. | +3.4 pts | Medium | DR & Backup |
| Remove unused dependencies, declare unlisted imports explicitly, and demote type-/test-only packages to devDependencies. | +3.4 pts | Medium | Dependency Hygiene |
| Make custom controls keyboard-operable (role + tabindex + key handler), drop positive tabindex, and give anchors a real href. | +2.5 pts | Medium | Keyboard semantics |
File quality
| File | Score | Band | Worst signal |
|---|---|---|---|
| REDACTED | 0.0 | Slop | Dependency Vulnerabilities: Critical CVE: REDACTED |
| REDACTED | 0.0 | Slop | Dependency Vulnerabilities: Critical CVE: REDACTED |
| REDACTED | 0.0 | Slop | Dependency Vulnerabilities: Critical CVE: REDACTED |
| REDACTED | 0.5 | Slop | Static Analysis (SAST): High: REDACTED |
| REDACTED | 3.4 | Mixed | IaC & Container Security: High IaC: REDACTED |
| REDACTED | 4.0 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 4.0 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 4.4 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 4.4 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 4.5 | Mixed | Dependency Vulnerabilities: Critical vulnerability: REDACTED |
| REDACTED | 4.6 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 4.8 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 5.1 | Mixed | IaC & Container Security: High IaC: REDACTED |
| REDACTED | 5.4 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 5.4 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| REDACTED | 5.4 | Mixed | Static Analysis (SAST): Medium: REDACTED |
| server/src/modules/orders/domain/order.entity.ts | 6.0 | Mixed | Explicit Debt: TodoComment |
| server/src/modules/user-management/domain/user.types.ts | 6.0 | Mixed | Explicit Debt: TodoComment |
| server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts | 6.3 | Mixed | Cyclomatic Complexity: FindProductsQueryHandler.execute (cyclomatic 18) |
| REDACTED | 6.4 | Mixed | Static Analysis (SAST): Medium: REDACTED |
How the grades work
A definite problem that already costs you something and drags the score down: a missing authorisation check, a dependency with a known exploit, a build that does not reproduce. Failure here tends to cause failures elsewhere.
Likely wrong, but not failing yet. It degrades the codebase over a longer horizon and can cause failures elsewhere — not urgent this week, not something to carry for two years either.
Recorded, with no effect on how the codebase functions. Present so the survey is complete, not because it needs doing.
Something this survey could not settle from the outside, and which could be critical or serious. Either a control was required and no positive evidence of it exists in the repository — a backup job that nothing shows was ever restored from proves nothing about restores — or our own analysis could not run over that part of the tree. This is not a clean result. These are excluded from the score rather than awarded a pass, so the number on the cover neither rewards nor penalises them: if you act on this survey without resolving them, you carry that risk yourself. Each one is named under Limitations.
Methodology & how to trust this report
Watchdog is a deep, periodic assessment — run each sprint, monthly, or quarterly, taking the time to go wider and deeper than a quick check and surfacing in one coherent report what you'd otherwise piece together from a dozen separate tools. It scores deterministically: the same commit yields the same score, every run. 90 of 95 evaluated dimensions are computed purely by tools and static analysis (confidence 1.0); 5 documentation/naming judgement(s) are LLM-assisted and labelled advisory. Overall confidence is 1.0 — 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 — 95 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, 934 of 1010 do; the remainder are repo-wide signals — a dimension-level measurement, not a single line.
- Is there a tool behind the number? Every score below names the method that produced it — Roslyn, git, a scanner, or (for a handful of documentation/naming dimensions) an LLM labelled sampled · advisory — not a narrative.
- Does re-running give the same result? Run it again on the same commit and the score — and this report, byte for byte — is identical. A report whose numbers move between runs is describing the run, not the code.
Tools & methods
| Method | Backs | Version | Evaluator |
|---|---|---|---|
| Roslyn static analysis | Complexity, cohesion, coupling, dead code, API surface, layering | 5.3.0 | ✓ deterministic |
| Native secret scanner | Hardcoded secrets / credentials | 1.0.0 | ✓ deterministic |
| Watchdog duplication detector (in-process) | Code duplication | 1.0.0 | ✓ deterministic |
| Coverage (coverlet / dotnet-coverage) | Line & branch coverage | 10.0.400 | ✓ deterministic |
| NuGet / dotnet | Outdated, vulnerable & deprecated dependencies | 10.0.400 | ✓ deterministic |
| git / LibGit2Sharp | Churn hotspots, knowledge concentration, history | 2.43.0 · 0.31.0 | ✓ deterministic |
| gitleaks · semgrep · trivy · checkov | Secrets in history, SAST, CVEs, IaC & container, PII / GDPR | 1.86.0 · 0.69.3 · 3.2.533 | ✓ deterministic |
| LLM (sampled · advisory) | Documentation quality, ADR conformance, naming — sampled over a bounded sample; advisory, never a deterministic measurement | Local LLM | ◐ LLM · sampled · advisory |
Run transparency — what happened this run
- D16 Bus Factor — not measured this run — This is a true statement about the repository that carries nothing for its owner to act on, so it is reported here rather than as a defect in their code. Single-contributor repository — bus factor is not applicable (1 contributor(s) across 184 commit(s) sampled, automation and bot accounts excluded). Concentration needs a team to concentrate: with one contributor there is nobody to spread the knowledge to, so there is nothing here for the owner to act on.
- D22 Internal API Consistency — not measured this run — 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. This repository's source is written only in languages whose packages are published through a manifest, and no manifest in the tree makes a publication act: either there is no package manifest of any ecosystem (no REDACTED, pyproject.toml, setup.py, gemspec, Cargo.toml, pom.xml, build.gradle, composer.json, pubspec.yaml or mix.exs), or every one present declares in its package manager's own terms that it is not published (an npm package marked "private": true, a Cargo package with publish = false, a pubspec with publish_to: none, a Composer "project", a Poetry project with package-mode = false, or the "Private :: Do Not Upload" classifier). With no publication act there is no intentionally-exposed surface for API consistency to be judged over.
- D27 Navigability — 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. Navigability is taken over call sites whose callee resolved. For C# that population includes calls into referenced assemblies; for this language only calls declared inside the repository count, so the abstention threshold reads the repository's use of its standard library as a resolution failure. That is a limit of this analyzer, not a build problem in this repository.
- D32 Data Compliance (PII/GDPR) — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. `client/src/api/types/types.ts` produced a parse error, so every rule in this engine's `gdpr.yml` was absent there. That absence is NOT a clean result: these rules detect personal data crossing a boundary into a log sink, a URL or browser storage, and a file that was never parsed cannot report any of the three. The rest of the tree analysed normally and its rows above stand; only these files are unaccounted for. You can widen what we reach: fix the syntax error (or exclude the file deliberately) and re-scan to cover it.
- D44 Platform End-of-Life — not measured this run — Watchdog could not measure this here. That is a gap on our side — a collector, parser or image we have not built yet — and it is neither a defect in this repository nor evidence that the check would have passed. This dimension reads a project's own statement about the platform it runs on: a TargetFramework in a .NET project file, a .nvmrc or .python-version, a capped requires-python, a Rust toolchain file or Cargo.toml rust-version, a .go-version, .java-version, .ruby-version, .tool-versions or .sdkmanrc, a go.mod go directive, a Maven or Gradle Java level or toolchain, a Gemfile's ruby directive, a mix.exs elixir requirement, a rebar.config minimum_otp_vsn, a pubspec.yaml SDK constraint, a build.sbt scalaVersion, or a framework major pinned by a dependency constraint. This repository carries none of them, so nothing about its platform was established. That is a gap in this analyzer's coverage, NOT a finding that the platform is supported — a language whose runtime is declared elsewhere (Package.swift, a REDACTED) is simply not read here yet.
- AXR1 Runtime accessibility — 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. Render strategy: boot via DockerCompose (REDACTED). compose up failed (exit 15 — the compose project is invalid) — time="2026-10-22T13:32:02Z" level=warning msg="The \"DB_USERNAME\" variable is not set. Defaulting to a blank string." error while interpolating services.db.environment.POSTGRES_DB: required variable DB_NAME is missing a value: err; runtime evidence skipped.
- DM9 Scattered domain decisions — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. Arm two tells a construction that PRODUCES a measurement from one that merely passes it along, and this repository's frontend does not compute that fact. A measurement built independently in two places would not appear above.
- P1 CI/CD gates — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. A CI pipeline exists and the word "test" appears, but no explicit test-runner invocation (your stack's test command, or a test job) was matched — so either the gate runs tests through a step this pass could not recognise, or "test" is incidental here (a path, "latest", a reporter). Which of the two it is cannot be settled from this dimension's evidence; the coverage dimensions report whether a suite exists at all. You can widen what we reach: name the test runner explicitly in the pipeline step (your stack's test command, or a job named for the suite) so the gate is unambiguous to a reader and to this pass.
- P2 Observability — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. 5 further occurrence(s) are not listed individually; the score already reflects all 45.
- R10 Code Duplication — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. 50 further occurrence(s) are not listed individually; the score already reflects all 90.
- R4 Test Coverage — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. 146 further occurrence(s) are not listed individually; the score already reflects all 186.
- R7 Dead Code — measured, with a gap in what it reached — Watchdog measured this, but not all of it. What it did not reach is a gap on our side — a collector, parser or image we have not built yet — so the numbers on that dimension cover less than the repository, and the part left out is not evidence that it would have passed. 12 further occurrence(s) are not listed individually; the score already reflects all 52.
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.
- D5 Coupling: Coupling is measured between projects/assemblies — runtime coupling through DI, reflection, messaging or shared databases is invisible to a static reference graph.
- D6 Cohesion (LCOM4): LCOM4 cohesion is syntactic — it infers connectivity from which methods touch which fields/methods by name, not from real runtime behaviour or intent.
- D8 Code Coverage: Coverage is measured by building and running the test suite 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.
- 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 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").
- D14 License Compliance: License compatibility is checked against declared package metadata and a policy — mislabelled or missing license metadata, and obligations that depend on how you distribute, are not resolved here.
- D15 Churn × Complexity Hotspots: Churn hotspots come from git history — a freshly imported or squashed repository has no churn signal, and recent rewrites can mask a historically risky file.
- D17 Explicit Debt: Acknowledged-debt signals (TODO/FIXME, suppressions, dead code) are textual — undocumented debt that nobody marked, and debt that lives in design rather than annotations, is invisible. Committed machine-written code (scaffolded migrations, designer/codegen output, generated stubs) is excluded — it is never the team's dead code to delete.
- D19 Documentation Quality: Documentation quality is judged by an LLM over a bounded sample of docs — it reads what is written, not whether the docs match the running system, and it is advisory, not a measurement. Its critique rows are drawn from a closed category vocabulary and each row means the same thing in every run, so two scans can be compared row by row; the SET that fires is still a sample, and does not repeat exactly. Measured on one frozen input, six scans at one engine SHA: 2-5 critique rows per scan, 8 distinct rows across the six, 3 of those 8 seen in only one scan. So a D19 row is evidence about the documentation, but a COUNT of D19 rows is not a quantity — never read a change in it as an improvement or a regression.
- D20 ADR Quality: ADR quality is an LLM read of the decision records present — it cannot know about decisions made and never recorded, and its verdict is sampled and advisory.
- D21 Naming Consistency: Naming quality is an LLM judgement over a bounded sample — it assesses clarity/consistency of the names it sees, not domain-correctness, and is advisory.
- D26 Project Cohesion: Project focus is sized from members/namespaces per project — a project that is broad by deliberate design reads the same as one that has sprawled.
- D28 Secrets (history): Secret-history scanning sweeps the git log for known patterns — a secret that predates the available history, or never matched a signature, is not found (clean means "nothing matched in the history we can see").
- D29 Static Analysis (SAST): SAST findings are pattern-based (semgrep) — it finds classes of bug it has rules for; logic flaws, auth/authorization gaps and issues needing runtime context are out of reach (and clean means "no rule matched").
- D30 Dependency Vulnerabilities: CVE matching depends on accurate package/version metadata and on the advisory databases — a vulnerability with no published advisory, or in code not declared as a dependency, is not seen. Coverage needs a RESOLVED graph: an unpinned requirements.txt, or a pom without a resolved build, yields partial coverage rather than a clean verdict. An ecosystem the analyzer cannot scan is reported as unmeasured, never as clean.
- D31 IaC & Container Security: IaC scanning checks Dockerfiles/Terraform/Kubernetes against best-practice rules — it cannot see the live cloud account, runtime configuration, or drift between the committed config and what is actually deployed.
- D34 Knowledge Freshness: Freshness is decayed commit RECENCY, not comprehension — code read often but rarely committed reads as orphaned, and stable code that genuinely needs no changes is penalised the same as forgotten code; bot/squash commits distort it like the bus factor.
- D35 Change Coupling: Change coupling is co-change in COMMITS — files split across separate commits, or coupled only through a shared config/build step, read as uncoupled, and a sweeping commit (rename/format) is excluded so it doesn't couple everything. It shows that files change together, not WHY: a high coupling can be a healthy cohesive pair as readily as a hidden leak.
- D43 Malicious Dependencies: Only packages some vulnerability database has already NAMED as malicious are seen — a compromise published in the last hours, or never reported at all, is invisible here, and this dimension reading 10 is not evidence that a dependency is trustworthy. There is no typosquat or dependency-confusion analysis: a package nobody has reported is simply absent from the feeds. Coverage is the dependency scan's: an ecosystem that could not be scanned is disclosed as unmeasured, never as clean.
- AC1 Text alternatives: Alt-text is detected structurally — the scan sees that an alternative EXISTS, not whether it meaningfully describes the image, and decorative-vs-missing is judged by attribute shape; runtime-injected images and a non-role=img decorative svg are out of scope. This is accessibility readiness, never a WCAG conformance claim.
- AC2 Forms & labels: Label association is read from static markup — a label wired up at runtime (JS-set aria-labelledby, framework-injected ids) reads as missing, a present label says nothing about whether its text is correct. A known UI-library field component (e.g. a JSX <TextField>) is now checked conservatively — flagged only when it carries NO label/aria-label/aria-labelledby/id/name — but wrapper/context-labelled libraries (Chakra/Radix FormControl+FormLabel) aren't statically visible (possible false positive) and non-JSX lowercased components are still skipped. A click handler on a plain element is now asked for a name too (it is a control the author declared), but the subtree test that answers it is deliberately generous: any DYNAMIC text expression in the subtree counts as a name, so an icon chosen by a ternary ({cond ? <IconA/> : <IconB/>}) reads as named, and a glyph component from a library the icon-import list does not know still names its parent. A clean result is "no unlabelled control found", not a labelling proof.
- AC3 Page structure: Page structure is read from the static markup tree — landmarks, headings and lang injected at runtime aren't seen, heading ORDER is checked structurally (not against the rendered visual hierarchy), and lang/title/main fire only on full documents, never partials, and the data-table check sees header-cell presence (a <th> exists), not whether each header correctly associates with its cells. Static readiness, not conformance.
- AC4 Keyboard semantics: Keyboard semantics are inferred from markup attributes — interactivity wired purely in script, focus managed at runtime, and component-level handlers are invisible. A clean result means "no static keyboard-trap shape", not a keyboard-operability proof.
- AC5 ARIA correctness: ARIA correctness is checked against the static role/attribute shape — roles/attributes set dynamically aren't seen, a valid role says nothing about whether it matches the element's real behaviour, and required-state checks are suppressed when a JSX spread could supply them. The two-branch toggle check (a control whose state is conveyed only by which of two mutually exclusive branches renders) reads CONDITIONALS THAT ARE ATTRIBUTES — Vue v-if/v-else/v-show and Alpine x-if/x-show — so the same toggle written as a Svelte {#if} block or a JSX ternary is control flow the markup model never projects as a branch and is not seen at all.
- AC6 Visual & motion safety: Contrast and motion safety are PARTIAL by construction — literal colours (hex/rgb/hsl/named) in inline styles, in-repo <style> blocks, in-repo .css files, var() tokens, Tailwind neutral utilities and CSS-in-JS top-level declarations are read (same-rule/same-element colour+background pairs only); computed/runtime/theme colour, external-CDN stylesheets, CSS-in-JS dynamic (${…}) and nested-selector colours, cross-element pairs and image contrast stay out of reach, so a clean result is bounded by what the static CSS itself shows.
- AC7 A11y enforcement: Enforcement is scored from in-repo config/CI evidence only — an a11y gate enforced in external tooling with no in-repo trace can't be credited, and a configured linter is presence, not proof the rules actually run or block a merge.
- AX10 Code composition: Role is inferred from namespace/folder convention, not semantics — a domain concept living in a folder named "Services" reads as application, and the split is lines-of-code, not business value. The business-logic-share score is a SOFT, FLOORED signal: it contributes to the Architecture lens but is floored at the Critical gate, so an infrastructure-heavy design (a gateway, an ETL, a driver) is legitimately low without being nuked to zero.
- AX9 CQS / query purity: Handlers are found by interface/name convention — a query handler using neither is not seen. Mutation is a resolved write/publish invocation (SaveChanges/repository/bus), so a write hidden behind a hand-rolled wrapper, reflection, or a string-keyed service locator resolves to a non-persistence type and isn't flagged; it detects that a query writes state, not whether the write is a legitimate read-side cache update. Clean means "no resolved write/publish in a query body", not a proof of CQS purity.
- DM4 Rich vs anemic model: Behaviour is detected as state mutation inside a method body — a method that enforces an invariant by validating-and-throwing without mutating reads as a query, and mutation delegated through an interface the scan can't resolve isn't credited; entities with zero public properties still drop out of the population. It detects that state changes, not whether the rule is correct.
- DM6 Domain ↔ infrastructure boundary: Infrastructure reached through a hand-rolled wrapper, a domain-named facade, reflection, or a string-keyed service locator resolves to a non-infra type and isn't seen; the body scan is symbol resolution over syntax, not full dataflow. A clean result means "no resolved infra reference in a domain body", not a proof of purity.
- ED5 Idempotency: Idempotency is judged from the handler body's visible writes and guards — a guard enforced by a database unique constraint, a broker's exactly-once delivery, or a domain method whose no-op-when-applied logic the scan can't follow may read as at-risk; the at-risk candidates are confirmed by a SAMPLED LLM verdict (advisory, not exhaustive) and degrade to heuristic-only when no model is configured. It flags the at-least-once double-apply SHAPE, not a runtime proof of a duplicate effect.
- M4 Documentation accuracy: Onboarding quality is an LLM read of the docs/setup present — it cannot run the onboarding or measure how long a real new joiner takes; the verdict is sampled and advisory.
- P4 Deployment & Rollback: Approval/branch-protection rules live in repository settings the scan cannot see — only their in-repo evidence (config files, workflows) is checked, so a control enforced purely in the host's settings reads as "not evidenced".
- P5 DR & Backup: Backup/restore and disaster-recovery readiness is judged from in-repo evidence — a config that exists is not a tested restore, so the absence of positive evidence is reported as "not evidenced", never scored as present.
- P6 Release Hygiene: Rollback/observability controls are inferred from repo artefacts (pipelines, dashboards-as-code) — controls configured in external tooling, with no in-repo trace, cannot be credited.
The LLM boundary
Dimensions
D1 · Cyclomatic Complexity
4 method(s) exceeded the cyclomatic complexity threshold of 15; the worst was stepper.Stepper at 37.
What to do
- Bring the 4 bodies over 15 down to 15 or less in Cyclomatic Complexity — start with stepper.Stepper (cyclomatic 37), AttributesSubformEdit.AttributesSubformEdit (cyclomatic 19), FindProductsQueryHandler.execute (cyclomatic 18). — This score is capped by its worst body, so a finding fixed alone moves it by almost nothing — the next one down takes its place. Refactoring these 4 together lifts Cyclomatic Complexity from 8.7 to about 10.0/10, projected with the scoring formula itself and assuming each lands exactly at 15.
- 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
5 method(s) exceeded the cognitive complexity threshold of 15; the worst was stepper.Stepper at 25.
What to do
- Bring the 5 bodies over 15 down to 15 or less in Cognitive Complexity — start with stepper.Stepper (cognitive 25), AttributeEntity.productValueConforms (cognitive 24), FindProductsQueryHandler.execute (cognitive 21). — This score is capped by its worst body, so a finding fixed alone moves it by almost nothing — the next one down takes its place. Refactoring these 5 together lifts Cognitive Complexity from 9.1 to about 10.0/10, projected with the scoring formula itself and assuming each lands exactly at 15.
- 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
14 over-large unit(s) detected — types, modules or files that carry too much.
What to do
- Resolve the 13 FunctionTooLong finding(s) in God Classes — start with REDACTED (6), Navbar.tsx, Footer.tsx. — One of this dimension's main actionable groups (13 warning-level).
- Resolve the 1 FileTooLong finding(s) in God Classes — start with REDACTED. — One of this dimension's main actionable groups (1 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.
D5 · Coupling
3 production modules (npm), 0 dependency cycle(s), 0 unstable depended-on module(s). Read from the build's own module declarations; 0 module(s) off the main sequence.
D6 · Cohesion (LCOM4)
0 of 31 classes have LCOM4 above 3.
D8 · Code Coverage
Line coverage 10.3% — 244 file(s) below 50%. Measured from the JavaScript/TypeScript suite (jest in client/: not run; jest via `yarn install --frozen-lockfile --ignore-scripts` in server/).
What to do
- Resolve the 244 Low coverage finding(s) in Code Coverage — start with index.ts (4), constants.ts (2), app.module.ts. — One of this dimension's main actionable groups (244 warning-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
46 test methods: 3 unit, 0 integration, 30 BDD, 13 e2e. The JavaScript/TypeScript suite contributes 46 `it`/`test` case(s) across 16 test file(s) declaring at least one; its tier split is read from package names and paths only.
What to do
- Resolve the 1 Inverted test pyramid finding(s) in Test Distribution. — One of this dimension's main actionable groups (1 recommendation-level).
D10 · Test Quality
0 skipped, 3 zero-assertion, no mocking-framework packages referenced (hand-written doubles or no mocking) across 46 tests.
What to do
- Resolve the 3 No assertions finding(s) in Test Quality — start with add-to-cart.e2e.spec.ts, create-category.e2e.spec.ts, create-product.e2e.spec.ts. — One of this dimension's main actionable groups (3 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 (jest via `yarn install --frozen-lockfile --ignore-scripts` in client/ (1 tests); jest via `yarn install --frozen-lockfile --ignore-scripts` in server/ (38 tests)): measured (0 flaky).
D12 · Dependency Hygiene
56 outdated direct production npm dependency(ies) of 66 graded, 3 pinning defect(s), across 3 REDACTED (3 of them the product) and 3 committed lockfile(s). Development dependencies are deliberately not graded for currency: that grade is about what SHIPS. 4 direct dependency(ies) — production, development or optional — are marked DEPRECATED by their publisher (the registry's per-version mark, or the lockfile's copy of it when the registry could not be asked); deprecated transitive packages are listed but not charged. Whether any is UNMAINTAINED is not graded — the registry publishes no maintenance status, and release age does not stand in for one. 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.
What to do
- Resolve the 81 Deprecated finding(s) in Dependency Hygiene — start with REDACTED (81). — One of this dimension's main actionable groups (81 recommendation-level).
- Resolve the 5 Deprecated finding(s) in Dependency Hygiene — start with REDACTED (5). — One of this dimension's main actionable groups (5 warning-level).
- Resolve the 3 One package at two majors across the workspace finding(s) in Dependency Hygiene. — One of this dimension's main actionable groups (3 warning-level).
- Enforce Dependency Hygiene in CI to reach Verified (currently Documented). — Hardens enforcement from Documented toward Verified — provenance only; does not change the score.
D13 · Secret Scanning
Secret scan ran and found no leaked secrets.
D14 · License Compliance
0 of 66 shipped npm package(s) use a banned license. Licences were resolved from registry.npmjs.org over the 66 production dependency(ies) this repository's committed lockfile resolves, across 3 product REDACTED manifest(s). Its 63 `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
No churn × complexity hotspots in the window.
D17 · Explicit Debt
35 deducted task-comment markers across 30821 LoC (0.1/KLoC) → score 9.8. 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 35 TodoComment finding(s) in Explicit Debt — start with REDACTED (5), user.types.ts (3), order.entity.ts (2). — One of this dimension's main actionable groups (35 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 repository's root README gives a strong overview of MinasPhone as a modular monolith built with NestJS, CQRS, and DDD principles (architecture overview, core domains, planned modules, key features, order-processing saga flow), plus links to architecture docs. The admin/README is a Create React App starter guide covering available scripts, eject notes, and deployment; the client README is a Next.js bootstrapped project with getting-started commands and resources. The server README is an exhaustive NestJS starter repository document (installation, running Docker env, test coverage, first-time setup, migration/seed steps) plus a clipped outline that confirms all sections exist. All four READMEs are well-structured but the admin/README's Create React App content is focused on build tool configuration rather than usage of the app itself, and the server README's 'Notes' section lists bounded-context modules without explaining what each one does.
What to do
- Improve Documentation Quality — currently 8.0/10. — The repository's root README gives a strong overview of MinasPhone as a modular monolith built with NestJS, CQRS, and DDD principles (architecture overview, core domains, planned modules, key features, order-processing saga flow), plus links to architecture docs. The admin/README is a Create React App starter guide covering available scripts, eject notes, and deployment; the client README is a Next.js bootstrapped project with getting-started commands and resources. The server README is an exhaustive NestJS starter repository document (installation, running Docker env, test coverage, first-time setup, migration/seed steps) plus a clipped outline that confirms all sections exist. All four READMEs are well-structured but the admin/README's Create React App content is focused on build tool configuration rather than usage of the app itself, and the server README's 'Notes' section lists bounded-context modules without explaining what each one does.
D20 · ADR Quality
No architecture decision records were found.
What to do
- Resolve the 1 No ADRs found finding(s) in ADR Quality. — One of this dimension's main actionable groups (1 recommendation-level).
D21 · Naming Consistency
0 naming inconsistencies across 0 sampled symbols.
D26 · Project Cohesion
3 of 3 build units (npm) flagged as possibly oversized/incoherent.
What to do
- Resolve the 1 Split admin finding(s) in Project Cohesion. — One of this dimension's main actionable groups (1 recommendation-level).
- Resolve the 1 Split server finding(s) in Project Cohesion. — One of this dimension's main actionable groups (1 recommendation-level).
D28 · Secrets (history)
1 finding(s): 0 critical, 1 high, 0 medium, 0 low. Remediation for historically-committed secrets is credential rotation — they remain in history regardless of later deletion.
What to do
- Resolve the 1 High secret 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)
42 finding(s): 0 critical, 9 high, 33 medium, 0 low. 5 unpinned-GitHub-Actions row(s) are reported here but scored by D36 (supply-chain provenance), which measures that posture as `pinned_actions` — one pinning decision is charged once, not once per lens. semgrep hit a parse error in 1 file(s) — `client/src/api/types/types.ts` (line 1, line 2, line 3, …) — so no absence of findings in the named regions is evidence of anything; rows reported elsewhere in those files are real. Fix the syntax error (or exclude the file deliberately) and re-scan to cover them. Separately, one or more rules could not re-parse an embedded snippet in 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 23 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED (5), REDACTED (5), REDACTED (3). — One of this dimension's main actionable groups (23 warning-level).
- Resolve the 4 REDACTED finding(s) in Static Analysis (SAST) — start with REDACTED (4). — One of this dimension's main actionable groups (4 issue-level).
- No action in Static Analysis (SAST) — all 5 REDACTED finding(s) are reported here at file:line but scored by D36 (supply-chain provenance), so none is charged to this dimension. — One of this dimension's main actionable groups (5 issue-level, 0 of them charged here).
D30 · Dependency Vulnerabilities
167 finding(s): 13 critical, 87 high, 52 medium, 15 low.
What to do
- Resolve the 86 High CVE finding(s) in Dependency Vulnerabilities — start with REDACTED (86). — One of this dimension's main actionable groups (86 issue-level).
- Resolve the 12 Critical CVE finding(s) in Dependency Vulnerabilities — start with REDACTED (12). — One of this dimension's main actionable groups (12 issue-level).
- Resolve the 1 Critical vulnerability finding(s) in Dependency Vulnerabilities — start with REDACTED. — One of this dimension's main actionable groups (1 issue-level).
D31 · IaC & Container Security
7 finding(s): 0 critical, 2 high, 4 medium, 1 low.
What to do
- Resolve the 2 High IaC finding(s) in IaC & Container Security — start with REDACTED, REDACTED. — One of this dimension's main actionable groups (2 issue-level).
- Resolve the 4 Medium IaC finding(s) in IaC & Container Security — start with REDACTED (2), REDACTED (2). — One of this dimension's main actionable groups (4 warning-level).
- Resolve the 1 Low IaC finding(s) in IaC & Container Security — start with REDACTED. — One of this dimension's main actionable groups (1 recommendation-level).
D34 · Knowledge Freshness
Every significant source file has living knowledge — recently and meaningfully worked. Counted over 122 of the 484 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
0/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 — start with REDACTED. — 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
AC1 · Text alternatives
AC2 · Forms & labels
- This UI-library field component has no label / aria-label / aria-labelledby / id — and neither does anything it renders (a name prop is the form-submission key, not a label) — so it likely renders an unlabelled control. Name it whichever way this library supports: a label prop, an aria-label, or an id on the rendered control with a <label htmlFor> pointing at it. For a group of controls, name the group itself (aria-label, or a fieldset with a legend) — labelling each item leaves the set unnamed. (×15) — admin/src/resources/categories/categories.tsx:43, admin/src/resources/orders/MobileGrid.tsx:48, admin/src/resources/orders/MobileGrid.tsx:53, …
- This UI-library field component has no label / aria-label / aria-labelledby / id, and nothing it renders carries one either (a name prop is the form-submission key, not a label). The text inside it does not stand in for one: these compound APIs render a trigger with role="combobox" (or role="button"), and those roles take their name from the AUTHOR, not from their content — so the words on screen never reach the accessibility tree as a name. Name it whichever way this library supports: a label prop, an aria-label, or an id on the rendered control with a <label htmlFor> pointing at it. For a group of controls, name the group itself (aria-label, or a fieldset with a legend) — labelling each item leaves the set unnamed. (×2) — client/src/components/Navbar/LanguageSelector.tsx:48, client/src/components/Products/Filter/PriceFilter.tsx:134
What to do
- Give every control a programmatic label (a <label for> / wrapping <label> / aria-label) and every button text — a placeholder is not a label.
AC3 · Page structure
- Skipping heading levels breaks the document outline assistive tech relies on. Don't jump levels — increase by at most one. (×2) — client/src/app/[locale]/information/cookie-policy/page.tsx:157, client/src/app/[locale]/information/user-terms/page.tsx:25
What to do
- Declare <html lang>, a document <title> and a <main> landmark, keep headings in order, leave zoom enabled, title iframes and drop meta-refresh.
AC4 · Keyboard semantics
- A click handler on a plain element isn't keyboard-operable. Use a <button>, or add role + tabIndex={0} + a key handler. — client/src/components/CookieConsent/CookieSettings.tsx:23
What to do
- Make custom controls keyboard-operable (role + tabindex + key handler), drop positive tabindex, and give anchors a real href.
AC5 · ARIA correctness
AC6 · Visual & motion safety
- <div> moves — its `animate-spin` utility runs with no motion-safe:/motion-reduce: variant — and nothing reads prefers-reduced-motion for it, so a motion-sensitive user cannot turn it off (WCAG 2.3.3). Gate it on @media (prefers-reduced-motion: no-preference) or motion-safe:. — client/src/components/Spinner/Spinner.tsx:17
- <input> takes keyboard focus, and its outline-none utility removes the focus ring with no focus-state ring, shadow, border or outline utility in its place. Add one (e.g. focus-visible:ring-2). — client/src/components/ui/input.tsx:7
What to do
- Keep a visible focus style (don't remove the outline without a replacement), guard animation with prefers-reduced-motion, and raise low-contrast colour pairs to at least 4.5:1.
AC7 · A11y enforcement
- No accessibility enforcement found — no a11y linter (eslint-plugin-jsx-a11y) and no axe/pa11y/Lighthouse in tests or CI. Start with the linter to catch issues at author time. What was searched, so you can tell an absence from a miss: the 119 markup file(s) this pass actually assessed, the linter configuration checked in beside them, and this repository's test and CI files — matched by name against the accessibility checkers this dimension carries. An audit run outside the repository, a hosted scanner, or a check whose name is not one of those, is not seen here.
What to do
- Enforce accessibility in the toolchain: add eslint-plugin-jsx-a11y, then assert with jest-axe in tests, then gate axe/pa11y/Lighthouse in CI.
AX10 · Code composition
What to do
- The domain core is a small share of production code, but most of the rest matched no layer vocabulary at all — so this is not yet an anemic-domain finding. The namespace/path convention could not place that code, which makes the composition above a statement about the naming, not about the design. Name the layers (or check that the repository's conventions differ from the ones this check knows) before reading a thin domain into it.
AX3 · Project dependency cycles
AX4 · Dependency direction
AX5 · Architecture & structure
AX6 · Interface segregation
AX9 · CQS / query purity
C2 · Access Controls
- Authorization IS enforced here (by route guards, auth middleware or checks that refuse a caller) — this is NOT a claim that endpoints are unprotected. What's missing is a NAMED rule set: who may do what is encoded inline where it is checked, so the rules can't be listed, reviewed or tested apart from the code that applies them. Recommendation, not a defect.
What to do
- Name the access rules — roles/permissions declared once (a @Roles/@Permissions decorator with its guard, a CASL ability, a casbin model, or Supabase CREATE POLICY) and consulted where access is checked, so the rule set is reviewable and testable on its own.
DM1 · Aggregate boundaries
DM11 · Constructible invalid state
DM2 · Strongly-typed ids
- `CartLineItemEntity.productId` is a raw `string` — give it a strongly-typed id: a dedicated single-field type wrapping the `string`, in whatever form your language spells that. — server/src/modules/orders/domain/cart-line-item.entity.ts:21
- `OrderLineItemEntity.productId` is a raw `string` — give it a strongly-typed id: a dedicated single-field type wrapping the `string`, in whatever form your language spells that. — server/src/modules/orders/domain/order-line-item.entity.ts:20
- `ProductsStockReservedDomainEvent.causationOrderId` is a raw `string` — give it a strongly-typed id: a dedicated single-field type wrapping the `string`, in whatever form your language spells that. — server/src/modules/product-catalog/domain/events/products-stock-reserved.domain-event.ts:4
- `StockReservationFailedDomainEvent.causationOrderId` is a raw `string` — give it a strongly-typed id: a dedicated single-field type wrapping the `string`, in whatever form your language spells that. — server/src/modules/product-catalog/domain/events/stock-reservation-failed.domain-event.ts:4
- `ProductEntity.categoryId` is a raw `string` — give it a strongly-typed id: a dedicated single-field type wrapping the `string`, in whatever form your language spells that. — server/src/modules/product-catalog/domain/product.entity.ts:86
What to do
- Adopt strongly-typed ids across the domain — finish the migration or document the boundary; primitive ids invite transposed-argument bugs.
DM4 · Rich vs anemic model
- `CartLineItemEntity` is an aggregate/entity with 3 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/orders/domain/cart-line-item.entity.ts:14
- `CartEntity` is an aggregate/entity with 2 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/orders/domain/cart.entity.ts:18
- `OrderLineItemEntity` is an aggregate/entity with 7 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/orders/domain/order-line-item.entity.ts:13
- `OrderEntity` is an aggregate/entity with 7 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/orders/domain/order.entity.ts:35
- `AttributeValueEntity` is an aggregate/entity with 4 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/product-catalog/domain/attribute-value.entity.ts:21
- `AttributeEntity` is an aggregate/entity with 7 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/product-catalog/domain/attribute.entity.ts:48
- `CategoryEntity` is an aggregate/entity with 4 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/product-catalog/domain/category.entity.ts:17
- `ProductTypeEntity` is an aggregate/entity with 4 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/product-catalog/domain/product-type.entity.ts:21
- `ProductEntity` is an aggregate/entity with 13 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/product-catalog/domain/product.entity.ts:48
- `UserEntity` is an aggregate/entity with 1 data propert(ies) but no state-changing behaviour (only data and queries) — the business logic lives in a service. — server/src/modules/user-management/domain/user.entity.ts:12
What to do
- Move business rules onto the aggregates/entities they govern so invariants are enforced at the source, not in anemic services.
DM5 · Encapsulated state
DM6 · Domain ↔ infrastructure boundary
DM9 · Scattered domain decisions
ED5 · Idempotency
GD1 · Unfinished & placeholder code
- A shipped function's whole body still throws a "not implemented" error — generated scaffolding that was never completed. Implement it or remove the dead surface. (×11) — client/src/api/mock/cart.ts:4, client/src/api/mock/cart.ts:7, client/src/api/mock/cart.ts:10, …
What to do
- Finish or delete the functions that still throw "not implemented", and replace placeholder text before shipping.
IC1 · Incompleteness & stubs
- `CartApi` has 5 unfinished members out of 5 declared members (methods and accessors, whether or not each carries a body) — a scaffolded class that was never implemented. — client/src/api/mock/cart.ts:3
- `retrieveCart` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/cart.ts:4
- `clearCart` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/cart.ts:7
- `addToCart` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/cart.ts:10
- `removeFromCart` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/cart.ts:13
- `updateLineItem` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/cart.ts:16
- `CategoriesApi` has 2 unfinished members out of 4 declared members (methods and accessors, whether or not each carries a body) — a scaffolded class that was never implemented. — client/src/api/mock/categories.ts:10
- `findCategoryBySlug` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/categories.ts:11
- `findMany` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. (×2) — client/src/api/mock/categories.ts:14, client/src/api/mock/products.ts:15
- `ProductsApi` has 4 unfinished members out of 6 declared members (methods and accessors, whether or not each carries a body) — a scaffolded class that was never implemented. — client/src/api/mock/products.ts:11
- `findAllProductSlugs` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/products.ts:12
- `findManyByCategorySlug` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/products.ts:18
- `findOneById` throws an exception whose message says it isn't finished — a not-implemented stub wearing an ordinary exception. Implement it or remove it. — client/src/api/mock/products.ts:25
- `clearCart` is declared `async` but never awaits anything, so it runs synchronously while pretending to be asynchronous. Drop `async` or do the real async work. — REDACTED:285
- Both arms of this conditional expression are the same, so no input can make the condition matter. Implement the distinction that was intended, or drop the condition. — server/src/logger/winston.logger.ts:58
- `handle` is declared `async` but never awaits anything, so it runs synchronously while pretending to be asynchronous. Drop `async` or do the real async work. — server/src/modules/user-management/application/events/listeners/user-registered.listener.ts:21
- A bare `eslint-disable` or `@ts-nocheck` with no rule list switches every lint rule or the whole type checker off for the rest of the file — hidden debt. Disable the specific rule on the specific line, with a justification, instead. (×4) — admin/src/api/server/api.ts:1, admin/src/api/server/api.ts:3, client/src/api/server/api.ts:1, …
What to do
- Finish or delete the unfinished stubs (functions that throw "not implemented", or whose body is empty or a bare constant) — they are dead surface that looks live.
- Clear the softer debt: re-enable or delete skipped tests, decide whether each empty function is a deliberate no-op (and say so in a comment), and replace blanket `eslint-disable` / `@ts-nocheck` with targeted, justified suppressions.
M1 · Documentation (README)
What to do
- Explain in the README how to run the tests. Add a section called Testing to the README in the repository's top folder, saying how to run the test suite.
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 your significant design decisions. Write one short document per decision, giving its date, the situation, the decision and its consequences, and keep them together with your other design documents. A docs/adr folder with each file named by number and title is the easiest place to find them.
M3 · Folder & project structure
M4 · Documentation accuracy
- README.md links to `./docs/adr/`, but `docs/adr` does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every REDACTED, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents' own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong. — README.md:66
- README.md links to `./docs/domains/`, but `docs/domains` does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every REDACTED, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents' own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong. — README.md:67
- README.md links to `./docs/api/`, but `docs/api` does not exist. Checked without a model against the repository: its files and folders, the scripts and dependencies of every REDACTED, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents' own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong. — README.md:68
- server/README.md names `.env.development`, but no file or folder `.env.development` exists. Checked without a model against the repository: its files and folders, the scripts and dependencies of every REDACTED, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents' own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong. — server/README.md:53
- server/README.md names `.env.prod`, but no file or folder `.env.prod` exists. Checked without a model against the repository: its files and folders, the scripts and dependencies of every REDACTED, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents' own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong. — server/README.md:54
- server/README.md names `.env.test`, but no file or folder `.env.test` exists. Checked without a model against the repository: its files and folders, the scripts and dependencies of every REDACTED, the projects, namespaces, package references and target frameworks of the .NET build, the SDK and Node pins, and the identifiers of every file that is not a document; the documents' own text never counts. A footprint generated at build time, read only by a platform, or living outside this repository is not seen, and this row is then wrong. — server/README.md:55
What to do
- Reconcile the README with reality: README.md links to `./docs/adr/`, but `docs/adr` does not exist; README.md links to `./docs/domains/`, but `docs/domains` does not exist; README.md links to `./docs/api/`, but `docs/api` does not exist; server/README.md names `.env.development`, but no file or folder `.env.development` exists; server/README.md names `.env.prod`, but no file or folder `.env.prod` exists.
P1 · CI/CD gates
What to do
- Run the test suite in CI via an explicit runner step for your stack, and gate merges on it.
P11 · BDD / executable specs
P2 · Observability
- Only 1/2 runnable modules use logging (modules with no entry point or server are excluded — they are libraries a runnable module hosts). Silent: `client`.
- `server/database/migrate.ts` reports through `console.*` at line 7 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/database/migrate.ts:7
- `server/database/getMigrator.ts` reports through `console.*` at line 26 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/database/getMigrator.ts:26
- `server/database/getMigrator.ts` reports through `console.*` at line 33 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/database/getMigrator.ts:33
- `server/database/getMigrator.ts` reports through `console.*` at line 34 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/database/getMigrator.ts:34
- `REDACTED` reports through `console.*` at line 240 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — REDACTED:240
- `REDACTED` reports through `console.*` at line 254 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — REDACTED:254
- `REDACTED` reports through `console.*` at line 188 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — REDACTED:188
- `REDACTED` reports through `console.*` at line 214 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — REDACTED:214
- `REDACTED` reports through `console.*` at line 250 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — REDACTED:250
- `server/src/configs/dotenv.ts` reports through `console.*` at line 4 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/configs/dotenv.ts:4
- `server/src/configs/dotenv.ts` reports through `console.*` at line 17 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/configs/dotenv.ts:17
- `server/src/libs/api/query-params.request.ts` reports through `console.*` at line 17 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/libs/api/query-params.request.ts:17
- `server/src/libs/api/query-params.request.ts` reports through `console.*` at line 24 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/libs/api/query-params.request.ts:24
- `server/src/libs/application/interceptors/exception.interceptor.ts` reports through `console.*` at line 35 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/libs/application/interceptors/exception.interceptor.ts:35
- `server/src/modules/notifications/application/services/email.service.ts` reports through `console.*` at line 31 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/notifications/application/services/email.service.ts:31
- `server/src/modules/notifications/application/services/email.service.ts` reports through `console.*` at line 53 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/notifications/application/services/email.service.ts:53
- `server/src/modules/orders/application/orders/commands/confirm-order/confirm-order.handler.ts` reports through `console.*` at line 30 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/orders/application/orders/commands/confirm-order/confirm-order.handler.ts:30
- `server/src/modules/orders/application/orders/commands/update-order-status/update-order-status.handler.ts` reports through `console.*` at line 29 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/orders/application/orders/commands/update-order-status/update-order-status.handler.ts:29
- `server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts` reports through `console.*` at line 47 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:47
- `server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts` reports through `console.*` at line 100 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:100
- `server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts` reports through `console.*` at line 103 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:103
- `server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts` reports through `console.*` at line 114 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:114
- `server/src/modules/orders/domain/order.entity.ts` reports through `console.*` at line 64 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/orders/domain/order.entity.ts:64
- `server/src/modules/product-catalog/application/attributes/commands/update-attribute/update-attribute.handler.ts` reports through `console.*` at line 57 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/attributes/commands/update-attribute/update-attribute.handler.ts:57
- `server/src/modules/product-catalog/application/images/commands/delete-image/delete-image.handler.ts` reports through `console.*` at line 24 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/images/commands/delete-image/delete-image.handler.ts:24
- `server/src/modules/product-catalog/application/images/commands/delete-image/delete-image.handler.ts` reports through `console.*` at line 27 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/images/commands/delete-image/delete-image.handler.ts:27
- `server/src/modules/product-catalog/application/images/commands/delete-image/delete-image.handler.ts` reports through `console.*` at line 28 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/images/commands/delete-image/delete-image.handler.ts:28
- `server/src/modules/product-catalog/application/images/event-handlers/delete-old-image-after-product-image-update.domain-event-handler.ts` reports through `console.*` at line 19 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/images/event-handlers/delete-old-image-after-product-image-update.domain-event-handler.ts:19
- `server/src/modules/product-catalog/application/images/event-handlers/delete-image-after-product-deletion.domain-event-handler.ts` reports through `console.*` at line 16 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/images/event-handlers/delete-image-after-product-deletion.domain-event-handler.ts:16
- `server/src/modules/product-catalog/application/products/commands/create-product/create-product.handler.ts` reports through `console.*` at line 89 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/products/commands/create-product/create-product.handler.ts:89
- `server/src/modules/product-catalog/application/products/commands/update-product/update-product.handler.ts` reports through `console.*` at line 122 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/products/commands/update-product/update-product.handler.ts:122
- `server/src/modules/product-catalog/application/products/queries/find-all-product-slugs/find-all-product-slugs.handler.ts` reports through `console.*` at line 62 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/products/queries/find-all-product-slugs/find-all-product-slugs.handler.ts:62
- `server/src/modules/product-catalog/application/products/queries/find-products-by-category-slug/find-products-by-category-slug.handler.ts` reports through `console.*` at line 72 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/products/queries/find-products-by-category-slug/find-products-by-category-slug.handler.ts:72
- `server/src/modules/product-catalog/application/products/queries/find-products-by-category-slug/find-products-by-category-slug.handler.ts` reports through `console.*` at line 74 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/products/queries/find-products-by-category-slug/find-products-by-category-slug.handler.ts:74
- `server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts` reports through `console.*` at line 57 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts:57
- `server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts` reports through `console.*` at line 157 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts:157
- `server/src/modules/product-catalog/infra/services/cloud-storage/google-cloud-storage.service.ts` reports through `console.*` at line 156 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/infra/services/cloud-storage/google-cloud-storage.service.ts:156
- `server/src/modules/product-catalog/infra/services/cloud-storage/google-cloud-storage.service.ts` reports through `console.*` at line 171 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/product-catalog/infra/services/cloud-storage/google-cloud-storage.service.ts:171
- `server/src/modules/user-management/application/events/listeners/user-registered.listener.ts` reports through `console.*` at line 23 although other files in its module log through a logger. Console lines carry no level, fields or correlation and never reach the log pipeline. — server/src/modules/user-management/application/events/listeners/user-registered.listener.ts:23
What to do
- Extend structured logging to the remaining runnable modules so everything you run is diagnosable in production.
- Route service-code `console.*` output through the module's structured logger (with named fields), so delivery outcomes and failures reach the log pipeline with a level.
- Add OpenTelemetry tracing/metrics (@opentelemetry/sdk-node) so requests are traceable across the system, not just health-probable.
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 2417 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
What to do
- Add an approval/environment gate (required reviewers / protection rules) before production promotion.
P5 · DR & Backup
- The only persistence guard found is a data volume in a stack this repo names a DEVELOPER stack, and no backup, geo-recovery or RTO/RPO controls were evidenced anywhere. A dev volume holds no production data, so there may be nothing here to recover — if this system's data lives in a platform or ops repo we can't see, that is where the DR evidence belongs, and this row is about what this repository can show rather than about that volume.
What to do
- Point at where production persistence is operated — a link from this repo to the ops/platform repo that holds its backup, geo-recovery and RTO/RPO controls — so the DR posture is reviewable from here; the dev-stack volume is not the thing to protect.
P6 · Release Hygiene
P7 · Outbound HTTP resilience
- `fetch(` makes an outbound HTTP call, and nothing bounds it: no timeout, deadline, retry or circuit breaker is set for it here, and the client has no process-wide default. A slow or failing dependency will hold this service's request, thread or connection until the call gives up on its own — or never, for a client with no default timeout. — server/src/modules/product-catalog/infra/services/image-search/google-image.search.service.ts:110
What to do
- Bound every outbound call: pass `signal: AbortSignal.timeout(ms)` to `fetch`, set `timeout` on the axios instance (`axios.create({ timeout })`), and add retries with back-off (`axios-retry`, `p-retry`) and a breaker (`opossum`, `cockatiel`) around dependencies that fail.
P9 · Domain vs controller coverage
- Domain coverage is 35%, against a bar of 50% for this row — the layer carrying the business rules is the least covered part of the codebase that matters most. Measured over the types this analyzer classifies as domain (aggregates, value objects, domain services), so code the classifier does not place in the domain is neither counted for nor against this figure.
What to do
- Raise unit-test coverage of the domain (aggregates, value objects, domain services) — that's where correctness matters most.
PF3 · Async & latency hygiene
- `fs.readFileSync` in `callback to fs.readdir` blocks the event loop until it finishes — a blocking call inside async code stalls the event loop — every other request waits for as long as it runs. — server/database/seed.ts:22
What to do
- TypeScript/JavaScript: use the promise APIs (fs/promises, a promisified child_process.execFile, the async zlib/crypto functions) and await them instead of the *Sync variants.
R1 · Type Safety
R10 · Code Duplication
- 19 duplicated blocks under server/src/modules/ have copies in at least two of the sibling directories orders, product-catalog, user-management — 7 of them are reported below, and 12 are counted here but not reported individually: those copies match on shape but no longer clear R10's bar for an individually reported row — either they kept neither their own names nor their values, or what was copied is too small to stand on its own (it reports near-exact duplication only, and only of substantial extent). What this row states is the concentration, which the detector measured over all 19 and which does not depend on how exactly each block's copies still match. That concentration is one structural fact, not 19 local ones: the siblings replicate behaviour none of them owns, which is the shape of a missing shared module — a common library every sibling imports — rather than 19 separate extractions. Check first whether the siblings are deliberately standalone deliverables (scaffold templates, demo apps that must stay copy-pasteable); where they are, the duplication is the design and the per-block rows are the ones to act on. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:21
- 11 duplicated blocks under server/src/modules/product-catalog/application/ have copies in at least two of the sibling directories attributes, categories, product-types, products — 7 of them are reported below, and 4 are counted here but not reported individually: those copies match on shape but no longer clear R10's bar for an individually reported row — either they kept neither their own names nor their values, or what was copied is too small to stand on its own (it reports near-exact duplication only, and only of substantial extent). What this row states is the concentration, which the detector measured over all 11 and which does not depend on how exactly each block's copies still match. That concentration is one structural fact, not 11 local ones: the siblings replicate behaviour none of them owns, which is the shape of a missing shared module — a common library every sibling imports — rather than 11 separate extractions. Check first whether the siblings are deliberately standalone deliverables (scaffold templates, demo apps that must stay copy-pasteable); where they are, the duplication is the design and the per-block rows are the ones to act on. — server/src/modules/product-catalog/application/attributes/commands/create-attribute/create-attribute.handler.ts:22
- admin/src/resources/products/components/AttributesSubform.tsx:50 · admin/src/resources/products/components/AttributesSubformEdit.tsx:62 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — admin/src/resources/products/components/AttributesSubform.tsx:50
- client/src/components/ui/dropdown-menu.tsx:9 · client/src/components/ui/select.tsx:9 — the two spans are one implementation copied and then locally edited — 504 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. — client/src/components/ui/dropdown-menu.tsx:9
- REDACTED:157 · REDACTED:99 — the two spans are one implementation copied and then locally edited — 493 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. — REDACTED:157
- server/src/modules/product-catalog/application/products/commands/create-product/create-product.handler.ts:46 · server/src/modules/product-catalog/application/products/commands/update-product/update-product.handler.ts:39 — the two spans are one implementation copied and then locally edited — 407 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. — server/src/modules/product-catalog/application/products/commands/create-product/create-product.handler.ts:46
- admin/src/resources/productTypes/ProductTypeCreate.tsx:1 · admin/src/resources/productTypes/ProductTypeEdit.tsx:1 — these 2 files are line-for-line copies of one another — 72 lines are identical, in the same order, in every one of them — so this is one fact about the file set, not a block to extract. An edit made to one file and not the others changes behaviour silently, which is the failure a wholesale copy guarantees. Pick one file as the single source and derive the others from it (re-export it, spread it into the local overrides each variant genuinely needs, or generate the copies at build time) — the few lines that differ between the files are exactly the part each variant should still own. Check first whether the copies are deliberately standalone deliverables (a translation file seeded from its sibling and waiting to be translated); where they are, the duplication is the design, and the honest move is to mark the seeded file as untranslated rather than to let it pass as done. — admin/src/resources/productTypes/ProductTypeCreate.tsx:1
- admin/src/resources/products/ProductCreate.tsx:207 · admin/src/resources/products/ProductEdit.tsx:91 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — admin/src/resources/products/ProductCreate.tsx:207
- client/src/components/CartItem/CartItem.tsx:68 · client/src/components/CartItem/CartItem.tsx:142 — the two spans are one implementation copied and then locally edited — 269 tokens are still identical, in the same order in both spans, 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. — client/src/components/CartItem/CartItem.tsx:68
- admin/src/resources/products/ProductCreate.tsx:132 · admin/src/resources/products/ProductEdit.tsx:28 — the 2 copies sit in sibling files in one directory, so check first whether one of them (or an existing module there) already owns this behaviour and the others should call it; otherwise extract it into one module in that directory and have each site call it. — admin/src/resources/products/ProductCreate.tsx:132
- REDACTED:236 · REDACTED:232 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — REDACTED:236
- server/src/modules/product-catalog/infra/database/attribute.repository.ts:179 · server/src/modules/product-catalog/infra/database/product-type.repository.ts:123 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — server/src/modules/product-catalog/infra/database/attribute.repository.ts:179
- server/src/modules/product-catalog/infra/mappers/attribute.mapper.ts:12 · server/src/modules/product-catalog/infra/mappers/product.mapper.ts:10 — the two spans are one implementation copied and then locally edited — 204 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. — server/src/modules/product-catalog/infra/mappers/attribute.mapper.ts:12
- server/src/modules/product-catalog/application/attributes/commands/create-attribute/create-attribute.handler.ts:22 · server/src/modules/product-catalog/application/product-types/commands/create-product-type/create-product-type.handler.ts:15 — the two spans are one implementation copied and then locally edited — 180 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. — server/src/modules/product-catalog/application/attributes/commands/create-attribute/create-attribute.handler.ts:22
- REDACTED:214 · REDACTED:317 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — REDACTED:214
- server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:9 · server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:14 — the two spans are one implementation copied and then locally edited — 160 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. — server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:9
- client/src/components/ui/tabs.tsx:43 · client/src/components/ui/tabs.tsx:95 — 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. — client/src/components/ui/tabs.tsx:43
- server/src/modules/product-catalog/application/attributes/commands/update-attribute/update-attribute.handler.ts:25 · server/src/modules/product-catalog/application/products/commands/update-product/update-product.handler.ts:44 — the two spans are one implementation copied and then locally edited — 169 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. — server/src/modules/product-catalog/application/attributes/commands/update-attribute/update-attribute.handler.ts:25
- REDACTED:131 · REDACTED:186 — both copies are in the same file, so extract the block into one function there and call it from each site — the copies drift apart the first time only one of them is edited. — REDACTED:131
- admin/src/resources/attributes/AttributeCreate.tsx:32 · admin/src/resources/attributes/AttributeEdit.tsx:32 — the 2 copies sit in sibling files in one directory, so check first whether one of them (or an existing module there) already owns this behaviour and the others should call it; otherwise extract it into one module in that directory and have each site call it. — admin/src/resources/attributes/AttributeCreate.tsx:32
- client/src/api/mock/categories.ts:46 · client/src/api/mock/products.ts:92 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — client/src/api/mock/categories.ts:46
- server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:21 · server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts:29 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:21
- REDACTED:79 · REDACTED:136 — both copies are in the same file, so extract the block into one function there and call it from each site — the copies drift apart the first time only one of them is edited. — REDACTED:79
- REDACTED:151 · REDACTED:348 · REDACTED:140 — the 3 copies are spread across 3 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — REDACTED:151
- server/src/modules/product-catalog/domain/attribute.entity.ts:143 · server/src/modules/product-catalog/domain/attribute.entity.ts:170 — both copies are in the same file, so extract the block into one function there and call it from each site — the copies drift apart the first time only one of them is edited. — server/src/modules/product-catalog/domain/attribute.entity.ts:143
- server/src/modules/orders/application/carts/commands/add-line-item/add-cart-line-item.handler.ts:17 · server/src/modules/orders/application/carts/commands/update-cart/update-cart-line-item.handler.ts:19 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — server/src/modules/orders/application/carts/commands/add-line-item/add-cart-line-item.handler.ts:17
- REDACTED:81 · REDACTED:177 — 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. — REDACTED:81
- server/src/modules/orders/domain/order.errors.ts:6 · server/src/modules/product-catalog/domain/product.errors.ts:16 — the 2 copies are spread across 2 files, and each CITED SPAN is a SPECIALISATION DECLARATION — a type whose `extends` clause names a base and whose body hands that base its own values through `super(…)`. The shared module this dimension usually asks you to extract already exists: it is that base, every one of these sites already reaches it, and the `super(…)` call this row matched on is where each site passes its differences in. So do not read this as an extract-a-helper row — there is no missing helper, and what is left at each site is the declaration of one distinct specialisation, which cannot be deleted without deleting the thing it declares. Read what actually differs between the spans, because two different answers follow. Where the sites differ only in the values they hand the base — a type argument, a service, an alias, a cache — the repetition is a TEMPLATE, and the only moves that collapse it are to generate these declarations from the set they enumerate or to replace the repeated construction with one factory each site calls with its own values; where that set is the point, each site pinning one distinct thing, the repetition IS the enumeration and there is nothing to remove. Where instead a span carries logic OUTSIDE the delegation that is the same at every site, that logic is the part to move, and its home is the base type rather than a new module. Reported because the copies still drift apart the first time only one of them is edited. — server/src/modules/orders/domain/order.errors.ts:6
- admin/src/resources/orders/OrderList.tsx:107 · admin/src/resources/orders/OrderList.tsx:127 — all 2 copies are in the same file, and the SHAPE of this repetition could not be determined. It is not a run of declarations, a listing, a declaration header, a type body or a slice through a construct — and it was not measured as a run of executable statements either, so this row cannot tell you whether a function can stand where these lines are. Read the two spans before acting, because the move is opposite in the two cases. Where they are statements, the ordinary answer holds: give the shared part one home and call it from each site. Where they turn out to be declarations, a literal's entries, or the cases of an enumeration, there is no call site to call anything from, and collapsing them would delete what each copy pins — a shared base type, a generated set, or one exported constant each site refers to is the move instead, and sometimes the honest answer is that there is nothing to extract at all. Reported because the copies drift apart the first time only one of them is edited, which is true whichever of those they are. — admin/src/resources/orders/OrderList.tsx:107
- server/src/libs/api/paginated-query.request.dto.ts:5 · server/src/modules/product-catalog/application/categories/queries/find-categories/find-categories.request.dto.ts:45 — the 2 copies are spread across 2 files, and the CITED SPAN is a run of DECLARATIONS inside a construct — the members of a type, or the entries of an object or array literal — with no statement anywhere in it. Do not read this as an extract-a-helper row: nothing here executes, so there is no call site, and a call expression cannot stand where a member declaration or a literal's entry was. What repeats is a SHAPE, and which shape decides the move. Where the copies declare the same members, the repetition is a missing common ancestor: give it a base type, an interface both extend, a generic instantiated twice, or — where the copies are a literal's entries rather than a type's members — one shared constant each site spreads or refers to, so the set is written once. Where they declare DIFFERENT members and only the form is shared — a decorator and its options, an annotation, a registration table's keys — the form is prescribed by a framework or a schema rather than copied, and the only collapse available is to generate the declarations from that schema; where you do not own the generator, this is the cost of the framework and there is nothing to extract. Check which of the two it is before acting: these copies still drift apart the first time only one of them is edited, which is why the repetition is reported, but the sentence to act on is not the same in both cases. — server/src/libs/api/paginated-query.request.dto.ts:5
- server/src/modules/product-catalog/application/categories/queries/find-category-by-id/find-category-by-id.handler.ts:12 · server/src/modules/product-catalog/application/products/queries/find-image/find-image.handler.ts:12 — the 2 copies are spread across 2 files, and the SHAPE of this repetition could not be determined. It is not a run of declarations, a listing, a declaration header, a type body or a slice through a construct — and it was not measured as a run of executable statements either, so this row cannot tell you whether a function can stand where these lines are. Read the two spans before acting, because the move is opposite in the two cases. Where they are statements, the ordinary answer holds: give the shared part one home and call it from each site. Where they turn out to be declarations, a literal's entries, or the cases of an enumeration, there is no call site to call anything from, and collapsing them would delete what each copy pins — a shared base type, a generated set, or one exported constant each site refers to is the move instead, and sometimes the honest answer is that there is nothing to extract at all. Reported because the copies drift apart the first time only one of them is edited, which is true whichever of those they are. — server/src/modules/product-catalog/application/categories/queries/find-category-by-id/find-category-by-id.handler.ts:12
- server/src/modules/product-catalog/domain/attribute.entity.ts:265 · server/src/modules/product-catalog/domain/product-type.entity.ts:159 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — server/src/modules/product-catalog/domain/attribute.entity.ts:265
- client/src/components/ui/pagination.tsx:68 · client/src/components/ui/pagination.tsx:86 — both copies are in the same file, so extract the block into one function there and call it from each site — the copies drift apart the first time only one of them is edited. — client/src/components/ui/pagination.tsx:68
- client/src/components/ui/select.tsx:138 · client/src/components/ui/select.tsx:156 — both copies are in the same file, so extract the block into one function there and call it from each site — the copies drift apart the first time only one of them is edited. — client/src/components/ui/select.tsx:138
- REDACTED:92 · REDACTED:133 · REDACTED:335 · REDACTED:127 · +1 more site(s) not listed — the 5 copies are spread across 3 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — REDACTED:92
- server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:96 · server/src/modules/product-catalog/application/products/queries/find-products-by-category-slug/find-products-by-category-slug.handler.ts:56 — the 2 copies are spread across 2 directories, so the shared home is a decision rather than an obvious spot: check first whether one of them already owns this behaviour, and otherwise put the extracted module somewhere all of the sites already reach rather than making one of them depend on another. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:96
- server/src/modules/product-catalog/application/product-types/commands/delete-product-type/delete-product-type.handler.ts:9 · server/src/modules/product-catalog/application/product-types/commands/update-product-type/update-product-type.handler.ts:11 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — server/src/modules/product-catalog/application/product-types/commands/delete-product-type/delete-product-type.handler.ts:9
- client/src/app/[locale]/categories/[categorySlug]/_client-page.tsx:18 · client/src/app/[locale]/products/_clientpage.tsx:13 — the 2 copies are spread across 2 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — client/src/app/[locale]/categories/[categorySlug]/_client-page.tsx:18
- client/src/components/ui/stepper.tsx:97 · client/src/components/ui/stepper.tsx:116 — all 2 copies are in the same file, and the CITED SPAN is a run of DECLARATIONS inside a construct — the members of a type, or the entries of an object or array literal — with no statement anywhere in it. Do not read this as an extract-a-helper row: nothing here executes, so there is no call site, and a call expression cannot stand where a member declaration or a literal's entry was. What repeats is a SHAPE, and which shape decides the move. Where the copies declare the same members, the repetition is a missing common ancestor: give it a base type, an interface both extend, a generic instantiated twice, or — where the copies are a literal's entries rather than a type's members — one shared constant each site spreads or refers to, so the set is written once. Where they declare DIFFERENT members and only the form is shared — a decorator and its options, an annotation, a registration table's keys — the form is prescribed by a framework or a schema rather than copied, and the only collapse available is to generate the declarations from that schema; where you do not own the generator, this is the cost of the framework and there is nothing to extract. Check which of the two it is before acting: these copies still drift apart the first time only one of them is edited, which is why the repetition is reported, but the sentence to act on is not the same in both cases. — client/src/components/ui/stepper.tsx:97
- REDACTED:56 · REDACTED:105 · REDACTED:151 · REDACTED:61 · +4 more site(s) not listed — the 8 copies are spread across 4 files, and the CITED SPAN is not a self-contained block — it runs from inside one construct into the next (the tail of a branch plus the head of the following one, a run of switch arms, the end of a declaration plus the list that follows it) rather than covering a whole unit. So do not lift these lines literally: no call can be substituted for a half-open construct. Extract the enclosing repeated UNIT instead — the whole function, component or branch these lines sit in — and where the repetition IS the construct (a run of switch arms, a stack of near-identical declarations) replace it with one table or registry looked up by key rather than a helper each arm calls. The copies still drift apart the first time only one of them is edited, which is why this is reported. — REDACTED:56
What to do
- Act on each finding's own remediation rather than one rule: the move depends on what recurs. Where the copies are executable blocks, give the shared part one home and call it from each site; where they are declarations, a listing, a specialisation already delegating to its base, or one shape repeated per entity, there is no call site and the move is a shared type, a generated set or a factory — sometimes there is nothing to extract.
R2 · Cyclomatic Complexity
- (anonymous) has cyclomatic complexity 29 and cognitive complexity 19; 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. — client/src/components/ui/stepper.tsx:62
- (top-level) has cyclomatic complexity 23 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. — server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts:1
- execute has cyclomatic complexity 16 and cognitive complexity 20; 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. — server/src/modules/product-catalog/application/products/commands/update-product/update-product.handler.ts:48
- AttributesSubformEdit has cyclomatic complexity 13 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. — admin/src/resources/products/components/AttributesSubformEdit.tsx:177
- validate has cyclomatic complexity 12 and cognitive complexity 22; 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. — server/src/libs/api/query-params.request.ts:64
- (top-level) has cyclomatic complexity 12 and cognitive complexity 15; 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. — server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:1
- isEmpty has cyclomatic complexity 12 and cognitive complexity 13; 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. — server/src/libs/guard.ts:5
- Err 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. (×2) — REDACTED:79, REDACTED:136
- (top-level) 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. (×2) — server/src/libs/ddd/entity.base.ts:1, server/src/modules/product-catalog/domain/value-objects/product-type-attribute-config.value-object.ts:1
What to do
- Break down the listed branch-heavy functions; aim P95 cyclomatic ≤ 5.
R3 · Large Files
- admin/src/api/server/api.ts has 1885 significant lines (blank lines excluded), over the 400-line bar, and contains logic — a file this long is usually several responsibilities in one module. — admin/src/api/server/api.ts:1
- client/src/api/server/api.ts has 1284 significant lines (blank lines excluded), over the 400-line bar, and contains logic — a file this long is usually several responsibilities in one module. — client/src/api/server/api.ts:1
- client/src/app/[locale]/information/privacy-policy/page.tsx has 527 significant lines (blank lines excluded), over the 400-line bar, and contains logic — a file this long is usually several responsibilities in one module. — client/src/app/[locale]/information/privacy-policy/page.tsx:1
- REDACTED has 406 significant lines (blank lines excluded), over the 400-line bar, and contains logic — a file this long is usually several responsibilities in one module. — REDACTED:1
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. (×40) — client/src/app/[locale]/information/privacy-policy/page.tsx:1, client/src/app/[locale]/information/user-terms/page.tsx:1, admin/src/resources/products/components/AttributesSubformEdit.tsx:1, …
What to do
- Add tests that import the unreached modules (directly or through their public entry).
R6 · Tooling
- test ✓ · lint ✓ · typecheck ✗ — read from this repository's REDACTED scripts and corroborated against its CI workflows. A script counts when its name or command matches the step: `test` for the suite, `lint` or `prettier` for linting, `typecheck`/`type-check`/`tsc` for type checking. ✗ therefore means no script or CI step under those names was found, NOT that the step is absent from your pipeline — a task invoked by a runner this check does not read, or named something else entirely, is not seen and is worth confirming before acting on a cross. A ✓ means the wiring is DECLARED — a script or CI step under those names exists. It is not a statement that the step passes, or that it runs at all: nothing here installs a dependency or executes a suite.
What to do
- Add `tsc --noEmit` as REDACTED scripts and run them in CI.
R7 · Dead Code
- Unreachable from the 24 application, 12 tooling and 31 test entry point(s) detected in this repo. Gate removals on `npm run build` — an undetected custom entry would make these reachable.
- no import path from any entry point (24 application, 12 tooling, 31 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 (×14) — admin/src/resources/orders/MobileGrid.tsx, server/src/modules/product-catalog/application/products/queries/find-image/find-product-image.request.dto.ts, client/src/api/mock/cart.ts, …
- no import path from any entry point (24 application, 12 tooling, 31 test roots considered), but 1 in-repo import(s) from 1 other file(s) do name it (server/src/libs/decorators/index.ts) — every one of those referrers is itself unreachable, so this file is dead only as a member of that cluster: if any referrer is in fact alive, this row falls with it (×2) — server/src/libs/decorators/final.decorator.ts, server/src/libs/decorators/frozen.decorator.ts
- Nothing imports this binding — it is safe to review for removal. (×18) — admin/src/types.ts:3, admin/src/types.ts:62, admin/src/types.ts:67, …
- Nothing ultimately consumes this binding: its only referrer is the re-export in server/src/libs/types/index.ts, whose re-exported name has no consumer either. Removing it is therefore not a one-file change — drop the re-export line in server/src/libs/types/index.ts in the same commit, or the barrel will import a name that no longer exists. (×3) — server/src/libs/types/non-function-properties.type.ts:9, server/src/libs/types/require-one.type.ts:4, server/src/libs/types/require-one.type.ts:13
- This top-level declaration is not exported and its name occurs nowhere else in its module — not called, read, typed against, re-exported or named by a string. Nothing outside the module can reach a non-exported binding, so this is the whole search: remove it, or use it. (×2) — admin/src/pages/dashboard/Dashboard.tsx:16, admin/src/pages/dashboard/Dashboard.tsx:22
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 REDACTED — installs work only by hoisting accident. (×13) — admin/src/App.tsx:4, admin/src/layout/AppBar.tsx:4, admin/src/pages/dashboard/Dashboard.tsx:3, …
- Declared in admin/package.json but never imported anywhere in that package or its workspace members — no static import reaches it. Usually that is dead weight and attack surface, but two shapes are indistinguishable from source and are NOT dead: an optional or native peer that another dependency loads dynamically at runtime, and a package a build, docs or test step installs and invokes separately. Confirm which of the three this is before removing it. (×2)
- Declared in REDACTED but never imported anywhere in that package or its workspace members — no static import reaches it. Usually that is dead weight and attack surface, but two shapes are indistinguishable from source and are NOT dead: an optional or native peer that another dependency loads dynamically at runtime, and a package a build, docs or test step installs and invokes separately. Confirm which of the three this is before removing it.
What to do
- Remove unused dependencies, declare unlisted imports explicitly, and demote type-/test-only packages to devDependencies.
R9 · Circular Imports
- server/src/libs/ddd/entity.base.ts → server/src/libs/utils/index.ts → server/src/libs/utils/convert-props-to-object.util.ts → server/src/libs/ddd/entity.base.ts (one verified cycle inside a mutually-dependent group of 4 files) — server/src/libs/ddd/entity.base.ts:7
- server/src/libs/exceptions/exceptions.ts → server/src/libs/exceptions/index.ts → server/src/libs/exceptions/exceptions.ts — server/src/libs/exceptions/exceptions.ts:1
- server/src/modules/orders/infra/database/order.repository.ts → server/src/modules/orders/infra/mappers/order.mapper.ts → server/src/modules/orders/infra/database/order.repository.ts — server/src/modules/orders/infra/database/order.repository.ts:8
- server/src/modules/product-catalog/infra/database/attribute.repository.ts → server/src/modules/product-catalog/infra/mappers/attribute.mapper.ts → server/src/modules/product-catalog/infra/database/attribute.repository.ts — server/src/modules/product-catalog/infra/database/attribute.repository.ts:17
- server/src/modules/product-catalog/infra/database/category.repository.ts → server/src/modules/product-catalog/infra/mappers/category.mapper.ts → server/src/modules/product-catalog/infra/database/category.repository.ts — server/src/modules/product-catalog/infra/database/category.repository.ts:9
- server/src/modules/product-catalog/infra/database/product-type.repository.ts → server/src/modules/product-catalog/infra/mappers/product-type.mapper.ts → server/src/modules/product-catalog/infra/database/product-type.repository.ts — server/src/modules/product-catalog/infra/database/product-type.repository.ts:9
- server/src/modules/product-catalog/infra/database/product.repository.ts → server/src/modules/product-catalog/infra/mappers/product.mapper.ts → server/src/modules/product-catalog/infra/database/product.repository.ts — server/src/modules/product-catalog/infra/database/product.repository.ts:9
What to do
- Break each cycle by extracting the shared piece into a module both sides can import.
S1 · Web-Security Posture
- `app.listen(…)` serves plain HTTP on a network address, and nothing in its package enforces HTTPS: no TLS served (`https.createServer`, Fastify `https`, Nest `httpsOptions`), no redirect of plain requests to `https://`, no `Strict-Transport-Security` (helmet sends it by default), and no TLS-terminating proxy configuration committed beside it — so a browser may talk to it in clear text. (−2.0 on this card.) — REDACTED:75
What to do
- Enforce HTTPS on each listed server: serve TLS itself, or redirect plain requests to `https://` and send `Strict-Transport-Security` (`helmet()` does by default) behind the TLS-terminating proxy — and commit that proxy's configuration beside the service. A server only this machine reaches should bind `127.0.0.1`.
X1 · Async correctness
X12 · Unreachable branch
X13 · Undrained process stream
X14 · Bypassable address classification
X15 · Unvalidated length from an untrusted reader
X16 · Unfloored truncation loop
X17 · Uncapped recursion over a caller-supplied document
X18 · Disposal-pattern correctness
X19 · Unrestored process-global state
X2 · Cancellation propagation
- Only 0/9 async functions that make a cancellable request (fetch, axios, ky, ofetch) accept an AbortSignal or pass one on, so a request keeps running after whatever wanted it has given up — a page navigated away from, an unmounted component, a timed-out job. Take a `signal` (or an options object that carries one) and hand it to the request; where nothing could ever cancel the call, omitting it is a deliberate choice — judge against how the function is used.
- `findMany` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. (×2) — client/src/api/server/products.ts:38, client/src/api/server/categories.ts:21
- `findOneById` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. — client/src/api/server/products.ts:61
- `checkoutOrder` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. — client/src/api/server/orders.ts:15
- `clearCart` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. — client/src/api/server/cart.ts:49
- `addToCart` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. — client/src/api/server/cart.ts:63
- `removeFromCart` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. — client/src/api/server/cart.ts:95
- `updateLineItem` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. — client/src/api/server/cart.ts:107
- `searchImages` awaits `fetch(…)` but neither accepts an AbortSignal nor passes one on, so its caller cannot stop the request once it has started. Accept `{ signal }` and pass it through: `fetch(url, { signal })`. — server/src/modules/product-catalog/infra/services/image-search/google-image.search.service.ts:101
What to do
- Thread an AbortSignal through async functions that make requests, so the work stops when its caller gives up.
X21 · Side-effecting pattern guard
X22 · Contradicted release guard
X23 · Unguarded diagnostic materialisation
X24 · Document value interpolated into markup unescaped
- `JSON.stringify(productSchema)` is text this program did not author, and it is written into the text of an element through `dangerouslySetInnerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — REDACTED:171
- `product.description` is text this program did not author, and it is written into the text of an element through `dangerouslySetInnerHTML` with nothing encoding it for that place. Whoever controls the value controls the markup: it can close the element or attribute it sits in and open a script of its own. Encode it for HTML at this site, render it through the framework's escaping output, or sanitise it if it must stay markup. — REDACTED:218
What to do
- Each finding names the expression, the attribute it lands in and the line. Confirm the value's origin from the read this analyzer traced, then escape it at the sink or validate it against the shape the attribute accepts before interpolating.
X25 · Inert configuration knob
X28 · Index access outside its own emptiness guard
X29 · Per-element action decided by a fixed element
X3 · Exception handling
X30 · Support guard that admits what it rejects
X4 · Structured logging
- `log` at line 17 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/api/middleware/request-logger.middleware.ts:17
- `debug` at line 25 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/application/interceptors/exception.interceptor.ts:25
- `debug` at line 89 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/db/sql-repository.base.ts:89
- `debug` at line 117 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/db/sql-repository.base.ts:117
- `debug` at line 146 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/db/sql-repository.base.ts:146
- `debug` at line 179 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/db/sql-repository.base.ts:179
- `debug` at line 295 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/db/sql-repository.base.ts:295
- `debug` at line 304 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/db/sql-repository.base.ts:304
- `debug` at line 309 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/db/sql-repository.base.ts:309
- `debug` at line 28 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/libs/ddd/aggregate-root.base.ts:28
- `log` at line 20 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:20
- `log` at line 47 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:47
- `warn` at line 52 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.warn({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.warn('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:52
- `error` at line 57 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.error({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.error('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. (×2) — server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:57, server/src/modules/notifications/application/services/email.service.ts:57
- `log` at line 15 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:15
- `log` at line 34 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:34
- `warn` at line 38 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.warn({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.warn('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:38
- `error` at line 43 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.error({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.error('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:43
- `log` at line 32 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/services/email.service.ts:32
- `error` at line 35 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.error({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.error('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/services/email.service.ts:35
- `log` at line 54 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/notifications/application/services/email.service.ts:54
- `log` at line 85 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.log({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.log('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/orders/application/orders/commands/checkout-order/checkout-order.handler.ts:85
- `debug` at line 58 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.debug({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.debug('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/product-catalog/application/products/commands/reserve-products-stocks/reserve-products-stock.handler.ts:58
- `error` at line 65 is handed a template literal with `${…}` substitutions as its message, so the logger receives one finished string — the values in it cannot be queried, filtered or aggregated as fields, and every occurrence reads as a different message. Pass the values beside a constant message instead: `logger.error({ orderId, status }, 'order status updated')` for pino and bunyan, `logger.error('order status updated', { orderId, status })` for winston. If these calls go through a NestJS `Logger` routed into pino, nestjs-pino forwards an object first argument as fields in the same way. — server/src/modules/product-catalog/application/products/commands/reserve-products-stocks/reserve-products-stock.handler.ts:65
What to do
- Interpolated log message defeats structured logging
X5 · Nullable reference types
- ~0.4 `!` null assertions per 1k TypeScript source lines — 14 assertion(s) across the 34295 source line(s) of the 475 TypeScript file(s) read, which is the only code the operator asserts anything in (a `!` in a `.js` file is not this operator and was not counted, and one in a file whose null checking is off silences nothing). Densest: admin/src/resources/orders/OrderList.tsx (3), server/src/modules/product-catalog/domain/attribute.entity.ts (3), server/src/modules/product-catalog/infra/database/attribute.repository.ts (2). Each one tells the compiler to trust the author about null, suppressing the very diagnostic `strictNullChecks` exists to produce.
What to do
- The findings name the configuration each number was taken from and, for the assertions, the files that hold them. Turn null checking on where it is off — `"strict": true` in the base tsconfig that the others extend switches `strictNullChecks` on with the rest of the family — and then resolve the assertions rather than adding to them. A `!` on a value the checker could show to be null is a promise the author cannot check from where they are standing, and each one is a `TypeError: Cannot read properties of null` waiting for the path that makes it true; where the value genuinely cannot be null, say something the checker can see instead (narrow it with a test, make the property required, drop the `?`).
WCAG coverage — what static analysis assessed
| Dimension | WCAG 2.2 A/AA criteria | Coverage |
|---|---|---|
| AC1 · Text alternatives | 1.1.1, 1.2.2, 1.2.5 | Partial signal |
| AC2 · Forms & labels | 1.3.1, 3.3.2, 4.1.2 | Partial signal |
| AC3 · Page structure | 1.4.4, 2.2.1, 2.4.1, 2.4.2, 3.1.1, 4.1.2 | Partial signal |
| AC4 · Keyboard semantics | 2.1.1, 2.4.3 | Partial signal |
| AC5 · ARIA correctness | 4.1.2 | Partial signal |
| AC6 · Visual & motion safety | 1.4.3, 2.4.7 | Partial — literal CSS only |
| AC7 · A11y enforcement | enforcement — no page criterion | Enforcement posture (process) |
Reference — by lens
| Lens | Score | Rating | Impact |
|---|---|---|---|
| Code Health | 58% | Adequate — gated by X4 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Architecture | 61% | Adequate — gated by D26, R9 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Maturity | 46% | Adequate — gated by M2, M4 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Readiness | 21% | Weak — gated by D8, D9, R8, P3, P5, P7 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Security | 63% | Adequate — gated by D29, D30, D36 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Domain Modelling | 63% | Adequate — gated by DM4, DM2 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Accessibility | 42% | Weak — gated by AC2, AC4 | Capped at Fair by a Critical contributor — resolve it before relying on this lens. |
| Performance | 85% | Exemplary | Strongest area. |
Unscored — 1 check(s) recorded observations but carry no score
- X10 Duplicated predicate — 12 observation(s) recorded · Advisory — this card reports evidence and never carries a score.
Not included — 45 check(s) not relevant to this codebase
- AX1 Captive dependencies — Not applicable: NestJS makes a provider that injects a request-scoped one request-scoped itself (scope bubbles up the injection chain), and gives every consumer its own transient instance by contract.
- AX2 Stateful singletons — Not applicable: TypeScript/JavaScript runs each process's requests on one event loop, so no two requests write a shared object at the same instant (interleaving across an await is a different defect).
- AX7 Slice cohesion — not applicable — not a vertical-slice architecture
- AX8 Test isolation — no test/production split to check
- AXB2 Runtime readiness — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- AXR1 Runtime accessibility — compose up failed (exit 15 — the compose project is invalid) — time="2026-10-22T13:32:02Z" level=warning msg="The \"DB_USERNAME\" variable is not set. Defaulting to a blank string." error while interpolating services.db.environment.POSTGRES_DB: required variable DB_NAME is missing a value: err; runtime evidence skipped This is a statement about this run, not a statement about your application: nothing here says the surface is inaccessible, only that it was never rendered.
- C1 Data Protection — No personal data detected in a persisted data model — no PII-named field (email, firstName, dateOfBirth, phoneNumber, …) or stored credential on a TypeORM/MikroORM/sequelize-typescript/NestJS-Mongoose entity, a Mongoose schema, a Sequelize or Drizzle table, a Knex migration or a Prisma model — and no database or data-store client in the source either, so this repository keeps no data at rest for these controls to protect. If it does persist personal data (through a hosted backend configured outside this repository, for instance), the controls belong to wherever that data is stored.
- C3 Audit Trail — No personal data detected in a persisted data model — no PII-named field (email, firstName, dateOfBirth, phoneNumber, …) or stored credential on a TypeORM/MikroORM/sequelize-typescript/NestJS-Mongoose entity, a Mongoose schema, a Sequelize or Drizzle table, a Knex migration or a Prisma model — and no database or data-store client in the source either, so this repository keeps no sensitive data whose changes an audit trail would record. If it does (through a hosted backend configured outside this repository, for instance), the trail belongs to wherever that data is stored.
- C4 Data Retention — No personal data detected in a persisted data model — no PII-named field (email, firstName, dateOfBirth, phoneNumber, …) or stored credential on a TypeORM/MikroORM/sequelize-typescript/NestJS-Mongoose entity, a Mongoose schema, a Sequelize or Drizzle table, a Knex migration or a Prisma model — and no database or data-store client in the source either, so this repository keeps no personal data whose lifetime a retention limit would bound. If it does (through a hosted backend configured outside this repository, for instance), the limit belongs wherever that data is stored.
- C5 Data-Subject Rights — No personal data detected in a persisted data model — no PII-named field (email, firstName, dateOfBirth, phoneNumber, …) or stored credential on a TypeORM/MikroORM/sequelize-typescript/NestJS-Mongoose entity, a Mongoose schema, a Sequelize or Drizzle table, a Knex migration or a Prisma model — and no database or data-store client in the source either, so this repository keeps no personal data for a data subject to erase, export or consent over. If it does (through a hosted backend configured outside this repository, for instance), the rights are served wherever that data is stored.
- D16 Bus Factor — single-contributor repository — bus factor is not applicable
- D18 Solution Shape — D18 scores the shape of a .NET solution; this repository has no .NET solution or project files, so the dimension does not apply.
- D22 Internal API Consistency — No intentionally-exposed public API to evaluate for consistency.
- D23 Boundary Type-Coupling — No bounded-context organisation was detected either — neither a context-shaped layout, nor 2+ sibling source directories each declaring an aggregate root, nor 2+ projects each declaring an entity of their own. Aggregate roots were found in only one source directory (server), which is a layered codebase, not two contexts. Declaring this codebase's bounded contexts (≥2) would let cross-boundary type coupling be assessed. Declare them in `.codehealth/config.yaml` at the repository root (create it if absent), mapping each context name to the module-path or namespace prefixes that belong to it — e.g. `architecture:` → `contexts:` → `Billing: ["src/billing", "Acme.Billing"]`, `Catalog: ["src/catalog", "Acme.Catalog"]`.
- D24 Comment Value — No inline comments to assess — comment value is not applicable here.
- D25 ADR Conformance — no ADRs to check
- D27 Navigability — D27 counts a call into the standard library or a dependency as unresolved for this repository's language (.ts,.tsx), so too few call sites counted as resolved to assess navigability. Not scored: this is a gap in the analyzer, not a verdict about this repository.
- D32 Data Compliance (PII/GDPR) — 1 file(s) were not parsed by semgrep — the PII/GDPR ruleset never ran over them
- 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 (.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
- 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
- DM10 One transaction, one aggregate — no repository writes detected — no operation to judge against the one-aggregate rule
- DM3 Integration-event coupling — no integration events detected — coupling check not applicable
- DM7 Repository granularity — no repository abstraction detected (e.g. uses a document session)
- ED1 Event-Driven — not scored — this repository shows only 1 of the 3 signals this lens looks for (30 CQRS handler(s))
- ES1 Event Sourcing — not scored — this repository shows none of the 3 signals this lens looks for
- P12 CI test-gate honesty — Reported, not scored — and nothing was matched here. The coverage check applies to any stack, and the automatic-re-run check to any GitHub-Actions workflow, but the checks for excluded tests, skipped tests and sleep-based synchronisation currently recognise only some ecosystems' test-runner idioms, so on a repository built with another stack the zeros below mean 'not checked', not 'clean'.
- P8 Schema migrations — no ORM, schema-migration tool or schema auto-create was found in this repository's dependency manifests or source, so there is no database schema for this check to judge
- PF1 Benchmark discipline — Not applicable: no benchmark suite was found. This check searched for tinybench, mitata, benchmark.js, benny or vitest `bench(...)` calls in files that import them (or `*.bench.*` files), or one of those in a REDACTED, and for a `*benchmark*` script that this repository's CI runs. Benchmarks are credited as a bonus, so their absence is neither scored nor deducted.
- PF2 Allocation hygiene — Not applicable: TypeScript/JavaScript runs on a garbage-collected runtime that gives a program no allocation-control idiom to choose on a hot path — no pools, stack allocation or value types — so allocation awareness is not something this code can be rated on.
- R11 Import Boundaries — No recognizable feature-sliced/layered src layout — boundary rules not applicable.
- R5 Dependency Freshness — uses a yarn lockfile — dependency freshness not measured here; JS/npm CVEs are scored in D30 (Dependency Vulnerabilities), which answers every ecosystem
- SC1 Supply-chain hygiene — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X20 Mistyped argument guard — This check asks whether an argument guard reports an EMPTY value as a NULL one, which needs a language that throws a null-specific argument exception — .NET's ArgumentNullException, the JVM's NullPointerException, Dart's ArgumentError.notNull. This repository contains none of those languages: the ones it is written in have a single exception for a bad argument, so there is no pair of exceptions to confuse and nothing here for it to assess. Not a gap in the analyzer and not a finding about your code.
- X26 Unsynchronised callback handoff — Not applicable: this check looks for a collection written by a callback on one thread while the body waiting on it touches it on another, and in this repository's languages no collection is reachable from two threads at once. TypeScript/JavaScript runs every callback on the one thread that owns its objects: a callback runs only when the body waiting on it has yielded, never alongside it, and a worker thread receives a COPY of what it is sent. A SharedArrayBuffer carries raw bytes, never an Array, Map or Set, so no collection is reachable from two threads at once. Not a gap in the analyzer and not a finding about your code.
- X27 Collection changed while being enumerated — Not applicable: this check looks for a loop whose body adds to or removes from the very collection the loop is walking, invalidating the enumerator it holds, and in this repository's languages that cannot happen. JavaScript and TypeScript have no iterator that a change can invalidate: an `Array` is walked by index against its live length, and a `Map` or `Set` by the ECMAScript specification's live entry list, so a body that adds to, removes from or empties what it walks raises nothing — every loop form (`for…of`, `for…in`, `forEach`, `map`, `filter`, `reduce`, `entries()`) on every built-in collection (`Array`, `Map`, `Set`, plain objects, `URLSearchParams`, `Headers`, `FormData`) runs to completion. Such a loop can still skip an element or never end, which is a different defect and not the one this check asks about. Not a gap in the analyzer and not a finding about your code.
- X32 Type resolved by simple name across every loaded assembly — This check is about how a .NET program searches the assemblies loaded into its process for a type, and this repository contains no .NET source, so there is nothing here for it to assess. Not a gap in the analyzer and not a finding about your code.
- X6 Hand-rolled structured-format parsing — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X7 Silent fallback defaults — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
- X9 Subsumed condition operand — Advisory — this card reports evidence and never carries a score, so there is nothing missing here.
Appendix A — Findings (grouped)
Critical — 133 finding(s)
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Import cycle (2 files) server/src/libs/exceptions/exceptions.ts:1
- Import cycle (2 files) server/src/modules/orders/infra/database/order.repository.ts:8
- Import cycle (2 files) server/src/modules/product-catalog/infra/database/attribute.repository.ts:17
- Import cycle (2 files) server/src/modules/product-catalog/infra/database/category.repository.ts:9
- Import cycle (2 files) server/src/modules/product-catalog/infra/database/product-type.repository.ts:9
- Import cycle (2 files) server/src/modules/product-catalog/infra/database/product.repository.ts:9
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Click handler on a non-interactive <span> client/src/components/CookieConsent/CookieSettings.tsx:23
- REDACTED
- REDACTED
- REDACTED
- Unlisted import '@mui/icons-material' admin/src/App.tsx:4
- Unlisted import '@mui/material' admin/src/layout/AppBar.tsx:4
- Unlisted import 'date-fns' admin/src/pages/dashboard/Dashboard.tsx:3
- Unlisted import 'query-string' admin/src/data/dataProvider.ts:2
- Unlisted import 'ra-i18n-polyglot' admin/src/i18n/index.ts:1
- Unlisted import 'ra-language-english' admin/src/i18n/en.ts:1
- Unlisted import 'react-hook-form' admin/src/resources/products/ProductCreate.tsx:19
- Unlisted import 'react-query' admin/src/resources/productTypes/ProductTypeCreate.tsx:21
- Unlisted import 'react-router-dom' admin/src/App.tsx:2
- Unlisted import '@radix-ui/react-visually-hidden' client/src/components/Navbar/AppDrawer/AppDrawer.tsx:15
- Unlisted import 'use-intl' client/src/app/[locale]/cart/page.tsx:8
- Unlisted import 'cookiejar' server/tests/test-utils/ApiClient.ts:2
- Unlisted import 'express' REDACTED:15
- Import cycle (3 files) server/src/libs/ddd/entity.base.ts:7
Serious — 594 finding(s)
- Low coverage: server/src/app.module.ts server/src/app.module.ts:1
- Low coverage: server/src/instrument.ts server/src/instrument.ts:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: server/src/api/api.module.ts server/src/api/api.module.ts:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: server/src/api/controllers/product-images.http.controller.ts server/src/api/controllers/product-images.http.controller.ts:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: REDACTED REDACTED:1
- Low coverage: server/src/api/controllers/users.controller.ts server/src/api/controllers/users.controller.ts:1
- Low coverage: server/src/configs/app.routes.ts server/src/configs/app.routes.ts:1
- Low coverage: server/src/configs/auth.config.ts server/src/configs/auth.config.ts:1
- Low coverage: server/src/configs/database.config.ts server/src/configs/database.config.ts:1
- Low coverage: server/src/configs/dotenv.ts server/src/configs/dotenv.ts:1
- Low coverage: server/src/configs/email.config.ts server/src/configs/email.config.ts:1
- Low coverage: server/src/configs/services.config.ts server/src/configs/services.config.ts:1
- Low coverage: server/src/configs/storage.config.ts server/src/configs/storage.config.ts:1
- Low coverage: server/src/libs/guard.ts server/src/libs/guard.ts:1
- Low coverage: server/src/libs/api/api-error.response.ts server/src/libs/api/api-error.response.ts:1
- Low coverage: server/src/libs/api/id.response.dto.ts server/src/libs/api/id.response.dto.ts:1
- Low coverage: server/src/libs/api/paginated-query.request.dto.ts server/src/libs/api/paginated-query.request.dto.ts:1
- Low coverage: server/src/libs/api/paginated.response.base.ts server/src/libs/api/paginated.response.base.ts:1
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- No test reaches this file client/src/app/[locale]/information/privacy-policy/page.tsx:1
- No test reaches this file client/src/app/[locale]/information/user-terms/page.tsx:1
- No test reaches this file admin/src/resources/products/components/AttributesSubformEdit.tsx:1
- No test reaches this file admin/src/resources/products/ProductCreate.tsx:1
- No test reaches this file admin/src/resources/products/components/AttributesSubform.tsx:1
- No test reaches this file client/src/app/[locale]/information/cookie-policy/page.tsx:1
- No test reaches this file client/src/components/ui/dropdown-menu.tsx:1
- No test reaches this file REDACTED:1
- No test reaches this file client/src/components/ui/stepper.tsx:1
- No test reaches this file client/src/components/Navbar/Navbar.tsx:1
- No test reaches this file client/src/components/CartItem/CartItem.tsx:1
- No test reaches this file admin/src/i18n/gr.ts:1
- No test reaches this file admin/src/i18n/ar.ts:1
- No test reaches this file admin/src/i18n/en.ts:1
- No test reaches this file admin/src/resources/orders/OrderEdit.tsx:1
- No test reaches this file admin/src/i18n/default-messages/greek.ts:1
- No test reaches this file admin/src/i18n/default-messages/arabic.ts:1
- No test reaches this file client/src/app/[locale]/information/return-policy/page.tsx:1
- No test reaches this file client/src/components/ui/select.tsx:1
- No test reaches this file client/src/components/Footer/Footer.tsx:1
- No test reaches this file client/src/app/[locale]/checkout/components/CheckoutController/PaymentForm.tsx:1
- No test reaches this file admin/src/data/dataProvider.ts:1
- No test reaches this file admin/src/resources/products/ProductEdit.tsx:1
- No test reaches this file client/src/components/ui/navigation-menu.tsx:1
- No test reaches this file client/src/components/ui/form.tsx:1
- TodoComment admin/src/authProvider.ts:16
- TodoComment client/src/api/mock/products.ts:10
- TodoComment client/src/api/mock/categories.ts:9
- TodoComment client/src/app/[locale]/cart/page.tsx:71
- TodoComment client/src/app/[locale]/checkout/page.tsx:52
- TodoComment client/src/app/[locale]/products/page.tsx:22
- TodoComment client/src/app/[locale]/products/page.tsx:29
- TodoComment REDACTED:55
- TodoComment client/src/components/Navbar/Navbar.tsx:61
- TodoComment REDACTED:63
- TodoComment server/src/libs/db/sql-repository.base.ts:205
- TodoComment server/src/modules/orders/application/orders/commands/confirm-order/confirm-order.handler.ts:43
- TodoComment server/src/modules/orders/domain/order.entity.ts:139
- TodoComment server/src/modules/orders/domain/order.entity.ts:156
- TodoComment server/src/modules/orders/domain/order-line-item.entity.ts:44
- TodoComment server/src/modules/orders/domain/cart.entity.ts:30
- TodoComment server/src/modules/orders/domain/cart-line-item.entity.ts:29
- TodoComment server/src/modules/product-catalog/domain/category.entity.ts:36
- TodoComment server/src/modules/product-catalog/application/attributes/commands/update-attribute/update-attribute.handler.ts:47
- TodoComment server/src/modules/product-catalog/application/categories/commands/create-category/create-category.handler.ts:31
- TodoComment server/src/modules/product-catalog/application/categories/commands/update-category/update-category.handler.ts:28
- TodoComment server/src/modules/product-catalog/application/product-types/commands/create-product-type/create-product-type.handler.ts:36
- TodoComment server/src/modules/product-catalog/application/product-types/commands/update-product-type/update-product-type.handler.ts:38
- TodoComment server/src/modules/product-catalog/application/products/commands/create-product/create-product.handler.ts:161
- TodoComment server/src/modules/product-catalog/application/products/event-handlers/update-stock-after-order-creation.domain-event-handler.ts:9
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- FunctionTooLong: page.PrivacyPolicy client/src/app/[locale]/information/privacy-policy/page.tsx:3
- FunctionTooLong: page.TermsAndConditions client/src/app/[locale]/information/user-terms/page.tsx:18
- FunctionTooLong: page.CookiePolicy client/src/app/[locale]/information/cookie-policy/page.tsx:34
- FunctionTooLong: page.ReturnPolicy client/src/app/[locale]/information/return-policy/page.tsx:3
- FunctionTooLong: page.ProductPage REDACTED:50
- FunctionTooLong: Navbar.NewNavbar client/src/components/Navbar/Navbar.tsx:43
- FunctionTooLong: Footer.Footer client/src/components/Footer/Footer.tsx:15
- FunctionTooLong: page.Order client/src/app/[locale]/orders/[orderSlug]/page.tsx:12
- FunctionTooLong: stepper.Stepper client/src/components/ui/stepper.tsx:15
- FunctionTooLong: Menu.Menu admin/src/layout/Menu.tsx:19
- FunctionTooLong: AttributesSubformEdit.AttributesSubformEdit admin/src/resources/products/components/AttributesSubformEdit.tsx:177
- FunctionTooLong: PaymentForm.PaymentForm client/src/app/[locale]/checkout/components/CheckoutController/PaymentForm.tsx:31
- FunctionTooLong: PriceFilter.PriceFilter client/src/components/Products/Filter/PriceFilter.tsx:19
- Unfinished stub — throws a "not implemented" error client/src/api/mock/cart.ts:4
- Unfinished stub — throws a "not implemented" error client/src/api/mock/cart.ts:7
- Unfinished stub — throws a "not implemented" error client/src/api/mock/cart.ts:10
- Unfinished stub — throws a "not implemented" error client/src/api/mock/cart.ts:13
- Unfinished stub — throws a "not implemented" error client/src/api/mock/cart.ts:16
- Unfinished stub — throws a "not implemented" error client/src/api/mock/categories.ts:11
- Unfinished stub — throws a "not implemented" error client/src/api/mock/categories.ts:14
- Unfinished stub — throws a "not implemented" error client/src/api/mock/products.ts:12
- Unfinished stub — throws a "not implemented" error client/src/api/mock/products.ts:15
- Unfinished stub — throws a "not implemented" error client/src/api/mock/products.ts:18
- Unfinished stub — throws a "not implemented" error client/src/api/mock/products.ts:25
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/cart.ts:4
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/cart.ts:7
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/cart.ts:10
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/cart.ts:13
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/cart.ts:16
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/categories.ts:11
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/categories.ts:14
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/products.ts:12
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/products.ts:15
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/products.ts:18
- Unfinished stub — throws a "not implemented" exception client/src/api/mock/products.ts:25
- Anemic entity: CartLineItemEntity server/src/modules/orders/domain/cart-line-item.entity.ts:14
- Anemic entity: CartEntity server/src/modules/orders/domain/cart.entity.ts:18
- Anemic entity: OrderLineItemEntity server/src/modules/orders/domain/order-line-item.entity.ts:13
- Anemic entity: OrderEntity server/src/modules/orders/domain/order.entity.ts:35
- Anemic entity: AttributeValueEntity server/src/modules/product-catalog/domain/attribute-value.entity.ts:21
- Anemic entity: AttributeEntity server/src/modules/product-catalog/domain/attribute.entity.ts:48
- Anemic entity: CategoryEntity server/src/modules/product-catalog/domain/category.entity.ts:17
- Anemic entity: ProductTypeEntity server/src/modules/product-catalog/domain/product-type.entity.ts:21
- Anemic entity: ProductEntity server/src/modules/product-catalog/domain/product.entity.ts:48
- Anemic entity: UserEntity server/src/modules/user-management/domain/user.entity.ts:12
- Async function without AbortSignal: findMany client/src/api/server/products.ts:38
- Async function without AbortSignal: findOneById client/src/api/server/products.ts:61
- Async function without AbortSignal: checkoutOrder client/src/api/server/orders.ts:15
- Async function without AbortSignal: findMany client/src/api/server/categories.ts:21
- Async function without AbortSignal: clearCart client/src/api/server/cart.ts:49
- Async function without AbortSignal: addToCart client/src/api/server/cart.ts:63
- Async function without AbortSignal: removeFromCart client/src/api/server/cart.ts:95
- Async function without AbortSignal: updateLineItem client/src/api/server/cart.ts:107
- Async function without AbortSignal: searchImages server/src/modules/product-catalog/infra/services/image-search/google-image.search.service.ts:101
- <NumberField> field component without a label admin/src/resources/orders/MobileGrid.tsx:53
- <NumberField> field component without a label admin/src/resources/orders/OrderEdit.tsx:163
- <NumberField> field component without a label admin/src/resources/orders/OrderList.tsx:111
- <NumberField> field component without a label admin/src/resources/orders/OrderList.tsx:130
- <NumberField> field component without a label admin/src/resources/orders/OrderList.tsx:149
- Deprecated: recharts admin/package.json:12
- Deprecated: uuid REDACTED:48
- Deprecated: eslint REDACTED:73
- Deprecated: eslint server/package.json:80
- Deprecated: supertest server/package.json:88
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- <TextField> field component without a label admin/src/resources/orders/MobileGrid.tsx:64
- <TextField> field component without a label admin/src/resources/orders/OrderList.tsx:124
- <TextField> field component without a label admin/src/resources/orders/OrderList.tsx:143
- <TextField> field component without a label admin/src/resources/users/users.tsx:8
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Duplicated block (17 lines × 2 locations) client/src/components/ui/pagination.tsx:68
- Duplicated block (17 lines × 2 locations) client/src/components/ui/select.tsx:138
- Duplicated block (17 lines × 2 locations) server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:96
- Duplicated block (17 lines × 2 locations) server/src/modules/product-catalog/application/product-types/commands/delete-product-type/delete-product-type.handler.ts:9
- Dead file (~1 LoC) server/src/modules/analytics/entities/analytics.entity.ts
- Dead file (~1 LoC) server/src/modules/marketing/entities/marketing.entity.ts
- Dead file (~1 LoC) server/src/modules/shipping/entities/shipping.entity.ts
- Dead file (~1 LoC) server/src/modules/user-management/application/events/index.ts
- <DateField> field component without a label admin/src/resources/orders/MobileGrid.tsx:48
- <DateField> field component without a label admin/src/resources/orders/OrderList.tsx:123
- <DateField> field component without a label admin/src/resources/orders/OrderList.tsx:142
- No assertions: Adding a product with invalid data to the cart server/tests/e2e/cart/add-to-cart/add-to-cart.e2e.spec.ts:123
- No assertions: I try to create a category with invalid data server/tests/e2e/category/create-category/create-category.e2e.spec.ts:59
- No assertions: I try to create a product with invalid data server/tests/e2e/product/create-product/create-product.e2e.spec.ts:68
- One package at two majors across the workspace: nanoid
- One package at two majors across the workspace: react
- One package at two majors across the workspace: react-dom
- Skeleton type — most members are unfinished client/src/api/mock/cart.ts:3
- Skeleton type — most members are unfinished client/src/api/mock/categories.ts:10
- Skeleton type — most members are unfinished client/src/api/mock/products.ts:11
- Duplicated block (19 lines × 2 locations) admin/src/resources/orders/OrderList.tsx:107
- Duplicated block (19 lines × 2 locations) server/src/libs/api/paginated-query.request.dto.ts:5
- Duplicated block (19 lines × 2 locations) server/src/modules/product-catalog/application/categories/queries/find-category-by-id/find-category-by-id.handler.ts:12
- Dead file (~11 LoC) client/src/global.ts
- Dead file (~11 LoC) server/src/libs/decorators/frozen.decorator.ts
- Dead file (~11 LoC) server/src/modules/user-management/domain/user.errors.ts
- Dead file (~0 LoC) server/src/libs/api/graphql/paginated.graphql-response.base.ts
- Dead file (~0 LoC) server/src/modules/product-catalog/application/products/event-handlers/update-stock-after-order-creation.domain-event-handler.ts
- Dead file (~0 LoC) server/src/modules/user-management/application/commands/delete-user/delete-user.handler.ts
- <TextInput> field component without a label admin/src/resources/categories/categories.tsx:43
- <TextInput> field component without a label admin/src/resources/products/ProductEdit.tsx:161
- Heading level jumps from h1 to h3 client/src/app/[locale]/information/cookie-policy/page.tsx:157
- Heading level jumps from h1 to h3 client/src/app/[locale]/information/user-terms/page.tsx:25
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Fake-async — async method never awaits REDACTED:285
- Fake-async — async method never awaits server/src/modules/user-management/application/events/listeners/user-registered.listener.ts:21
- Duplicated block (41 lines × 2 locations) REDACTED:236
- Duplicated block (41 lines × 2 locations) server/src/modules/product-catalog/infra/database/attribute.repository.ts:179
- Duplicated block (25 lines × 2 locations) admin/src/resources/attributes/AttributeCreate.tsx:32
- Duplicated block (25 lines × 2 locations) client/src/api/mock/categories.ts:46
- Duplicated block (20 lines × 2 locations) REDACTED:81
- Duplicated block (20 lines × 2 locations) server/src/modules/orders/domain/order.errors.ts:6
- Duplicated block (16 lines × 2 locations) client/src/app/[locale]/categories/[categorySlug]/_client-page.tsx:18
- Duplicated block (16 lines × 2 locations) client/src/components/ui/stepper.tsx:97
- Complex function Err (cyclomatic 12, cognitive 11) REDACTED:79
- Complex function Err (cyclomatic 12, cognitive 11) REDACTED:136
- Complex function (top-level) (cyclomatic 11, cognitive 10) server/src/libs/ddd/entity.base.ts:1
- Complex function (top-level) (cyclomatic 11, cognitive 10) server/src/modules/product-catalog/domain/value-objects/product-type-attribute-config.value-object.ts:1
- Untrusted text written into markup unescaped REDACTED:171
- Untrusted text written into markup unescaped REDACTED:218
- <Select> field component without a label client/src/components/Navbar/LanguageSelector.tsx:48
- <Slider> field component without a label client/src/components/Products/Filter/PriceFilter.tsx:123
- <RadioGroup> field component without a label client/src/components/Products/Filter/PriceFilter.tsx:134
- Animation without a prefers-reduced-motion guard client/src/components/Spinner/Spinner.tsx:17
- Focus outline removed inline client/src/components/ui/input.tsx:7
- Accessibility enforcement below the top rung
- stepper.Stepper (cyclomatic 37) client/src/components/ui/stepper.tsx:15
- AttributesSubformEdit.AttributesSubformEdit (cyclomatic 19) admin/src/resources/products/components/AttributesSubformEdit.tsx:177
- FindProductsQueryHandler.execute (cyclomatic 18) server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts:51
- AttributeEntity.productValueConforms (cyclomatic 18) server/src/modules/product-catalog/domain/attribute.entity.ts:94
- stepper.Stepper (cognitive 25) client/src/components/ui/stepper.tsx:15
- AttributeEntity.productValueConforms (cognitive 24) server/src/modules/product-catalog/domain/attribute.entity.ts:94
- FindProductsQueryHandler.execute (cognitive 21) server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts:51
- CustomFilterParam.validate (cognitive 21) server/src/libs/api/query-params.request.ts:64
- UpdateProductCommandHandler.execute (cognitive 19) server/src/modules/product-catalog/application/products/commands/update-product/update-product.handler.ts:48
- REDACTED
- FileTooLong: privacy-policy/page.tsx client/src/app/[locale]/information/privacy-policy/page.tsx
- REDACTED
- REDACTED
- REDACTED
- Both branches are identical — the condition decides nothing server/src/logger/winston.logger.ts:58
- No DR/backup evidence
- Outbound HTTP without resilience server/src/modules/product-catalog/infra/services/image-search/google-image.search.service.ts:110
- Low domain coverage
- Sync-over-async blocking server/database/seed.ts:22
- Duplication concentrated across 3 sibling directories (19 clone groups) server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:21
- Duplication concentrated across 4 sibling directories (11 clone groups) server/src/modules/product-catalog/application/attributes/commands/create-attribute/create-attribute.handler.ts:22
- Duplicated block (144 lines × 2 locations) admin/src/resources/products/components/AttributesSubform.tsx:50
- Duplicated block with local edits (128 matched lines × 2 locations) client/src/components/ui/dropdown-menu.tsx:9
- Duplicated block with local edits (94 matched lines × 2 locations) REDACTED:157
- Duplicated block with local edits (89 matched lines × 2 locations) server/src/modules/product-catalog/application/products/commands/create-product/create-product.handler.ts:46
- Wholesale file copy (72 identical lines × 2 files) admin/src/resources/productTypes/ProductTypeCreate.tsx:1
- Duplicated block (66 lines × 2 locations) admin/src/resources/products/ProductCreate.tsx:207
- Duplicated block with local edits (60 matched lines × 2 locations) client/src/components/CartItem/CartItem.tsx:68
- Duplicated block (44 lines × 2 locations) admin/src/resources/products/ProductCreate.tsx:132
- Duplicated block with local edits (41 matched lines × 2 locations) server/src/modules/product-catalog/infra/mappers/attribute.mapper.ts:12
- Duplicated block with local edits (37 matched lines × 2 locations) server/src/modules/product-catalog/application/attributes/commands/create-attribute/create-attribute.handler.ts:22
- Duplicated block (34 lines × 2 locations) REDACTED:214
- Duplicated block with local edits (34 matched lines × 2 locations) server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:9
- Duplicated block (30 lines × 2 locations) client/src/components/ui/tabs.tsx:43
- Duplicated block with local edits (30 matched lines × 2 locations) server/src/modules/product-catalog/application/attributes/commands/update-attribute/update-attribute.handler.ts:25
- Duplicated block (27 lines × 2 locations) REDACTED:131
- Duplicated block (24 lines × 2 locations) server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:21
- Duplicated block (23 lines × 2 locations) REDACTED:79
- Duplicated block (23 lines × 3 locations) REDACTED:151
- Duplicated block (22 lines × 2 locations) server/src/modules/product-catalog/domain/attribute.entity.ts:143
- Duplicated block (21 lines × 2 locations) server/src/modules/orders/application/carts/commands/add-line-item/add-cart-line-item.handler.ts:17
- Duplicated block (18 lines × 2 locations) server/src/modules/product-catalog/domain/attribute.entity.ts:265
- Duplicated block (17 lines × 5 locations) REDACTED:92
- Duplicated block (16 lines × 8 locations) REDACTED:56
- Complex function (anonymous) (cyclomatic 29, cognitive 19) client/src/components/ui/stepper.tsx:62
- Complex function (top-level) (cyclomatic 23, cognitive 29) server/src/modules/product-catalog/application/products/queries/find-products/find-products.handler.ts:1
- Complex function execute (cyclomatic 16, cognitive 20) server/src/modules/product-catalog/application/products/commands/update-product/update-product.handler.ts:48
- Complex function AttributesSubformEdit (cyclomatic 13, cognitive 10) admin/src/resources/products/components/AttributesSubformEdit.tsx:177
- Complex function validate (cyclomatic 12, cognitive 22) server/src/libs/api/query-params.request.ts:64
- Complex function (top-level) (cyclomatic 12, cognitive 15) server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:1
- Complex function isEmpty (cyclomatic 12, cognitive 13) server/src/libs/guard.ts:5
- admin/src/api/server/api.ts: 1885 lines of code admin/src/api/server/api.ts:1
- client/src/api/server/api.ts: 1284 lines of code client/src/api/server/api.ts:1
- client/src/app/[locale]/information/privacy-policy/page.tsx: 527 lines of code client/src/app/[locale]/information/privacy-policy/page.tsx:1
- REDACTED: 406 lines of code REDACTED:1
- No typecheck script
- Dead file (~84 LoC) admin/src/resources/orders/MobileGrid.tsx
- Dead file (~22 LoC) server/src/modules/product-catalog/application/products/queries/find-image/find-product-image.request.dto.ts
- Dead file (~19 LoC) client/src/api/mock/cart.ts
- Dead file (~18 LoC) server/src/libs/decorators/final.decorator.ts
- Dead file (~16 LoC) client/src/i18n/request.ts
- Dead file (~2 LoC) server/src/libs/decorators/index.ts
- Unused dependency '@testing-library/react'
- Unused dependency '@testing-library/user-event'
- Unused dependency '@fontsource/roboto'
Minor — 233 finding(s)
- Deprecated: @babel/plugin-proposal-async-generator-functions REDACTED:467
- Deprecated: @babel/plugin-proposal-class-properties REDACTED:476
- Deprecated: @babel/plugin-proposal-class-static-block REDACTED:484
- Deprecated: @babel/plugin-proposal-dynamic-import REDACTED:505
- Deprecated: @babel/plugin-proposal-export-namespace-from REDACTED:513
- Deprecated: @babel/plugin-proposal-export-namespace-from REDACTED:521
- Deprecated: @babel/plugin-proposal-json-strings REDACTED:529
- Deprecated: @babel/plugin-proposal-logical-assignment-operators REDACTED:537
- Deprecated: @babel/plugin-proposal-nullish-coalescing-operator REDACTED:545
- Deprecated: @babel/plugin-proposal-numeric-separator REDACTED:553
- Deprecated: @babel/plugin-proposal-object-rest-spread REDACTED:561
- Deprecated: @babel/plugin-proposal-optional-catch-binding REDACTED:572
- Deprecated: @babel/plugin-proposal-optional-chaining REDACTED:580
- Deprecated: @babel/plugin-proposal-private-methods REDACTED:589
- Deprecated: @babel/plugin-proposal-private-property-in-object REDACTED:597
- Deprecated: @babel/plugin-proposal-unicode-property-regex REDACTED:607
- Deprecated: @humanwhocodes/config-array REDACTED:1527
- Deprecated: @humanwhocodes/object-schema REDACTED:1536
- Deprecated: @mui/base REDACTED:1884
- Deprecated: abab REDACTED:3153
- Deprecated: acorn-import-assertions REDACTED:3174
- Deprecated: babel-merge REDACTED:3498
- Deprecated: core-js-pure REDACTED:4183
- Deprecated: core-js REDACTED:4188
- Deprecated: domexception REDACTED:4789
- Console output used as the logging channel server/database/migrate.ts:7
- Console output used as the logging channel server/database/getMigrator.ts:26
- Console output used as the logging channel server/database/getMigrator.ts:33
- Console output used as the logging channel server/database/getMigrator.ts:34
- Console output used as the logging channel REDACTED:240
- Console output used as the logging channel REDACTED:254
- Console output used as the logging channel REDACTED:188
- Console output used as the logging channel REDACTED:214
- Console output used as the logging channel REDACTED:250
- Console output used as the logging channel server/src/configs/dotenv.ts:4
- Console output used as the logging channel server/src/configs/dotenv.ts:17
- Console output used as the logging channel server/src/libs/api/query-params.request.ts:17
- Console output used as the logging channel server/src/libs/api/query-params.request.ts:24
- Console output used as the logging channel server/src/libs/application/interceptors/exception.interceptor.ts:35
- Console output used as the logging channel server/src/modules/notifications/application/services/email.service.ts:31
- Console output used as the logging channel server/src/modules/notifications/application/services/email.service.ts:53
- Console output used as the logging channel server/src/modules/orders/application/orders/commands/confirm-order/confirm-order.handler.ts:30
- Console output used as the logging channel server/src/modules/orders/application/orders/commands/update-order-status/update-order-status.handler.ts:29
- Console output used as the logging channel server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:47
- Console output used as the logging channel server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:100
- Console output used as the logging channel server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:103
- Console output used as the logging channel server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:114
- Console output used as the logging channel server/src/modules/orders/domain/order.entity.ts:64
- Console output used as the logging channel server/src/modules/product-catalog/application/attributes/commands/update-attribute/update-attribute.handler.ts:57
- Console output used as the logging channel server/src/modules/product-catalog/application/images/commands/delete-image/delete-image.handler.ts:24
- Interpolated log message defeats structured logging server/src/libs/api/middleware/request-logger.middleware.ts:17
- Interpolated log message defeats structured logging server/src/libs/application/interceptors/exception.interceptor.ts:25
- Interpolated log message defeats structured logging server/src/libs/db/sql-repository.base.ts:89
- Interpolated log message defeats structured logging server/src/libs/db/sql-repository.base.ts:117
- Interpolated log message defeats structured logging server/src/libs/db/sql-repository.base.ts:146
- Interpolated log message defeats structured logging server/src/libs/db/sql-repository.base.ts:179
- Interpolated log message defeats structured logging server/src/libs/db/sql-repository.base.ts:295
- Interpolated log message defeats structured logging server/src/libs/db/sql-repository.base.ts:304
- Interpolated log message defeats structured logging server/src/libs/db/sql-repository.base.ts:309
- Interpolated log message defeats structured logging server/src/libs/ddd/aggregate-root.base.ts:28
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:20
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:47
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:52
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/send-customer-order-confirmation-email.domain-event-handler.ts:57
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:15
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:34
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:38
- Interpolated log message defeats structured logging server/src/modules/notifications/application/event-handlers/order-created-notification.domain-event-handler.ts:43
- Interpolated log message defeats structured logging server/src/modules/notifications/application/services/email.service.ts:32
- Interpolated log message defeats structured logging server/src/modules/notifications/application/services/email.service.ts:35
- Interpolated log message defeats structured logging server/src/modules/notifications/application/services/email.service.ts:54
- Interpolated log message defeats structured logging server/src/modules/notifications/application/services/email.service.ts:57
- Interpolated log message defeats structured logging server/src/modules/orders/application/orders/commands/checkout-order/checkout-order.handler.ts:85
- Interpolated log message defeats structured logging server/src/modules/product-catalog/application/products/commands/reserve-products-stocks/reserve-products-stock.handler.ts:58
- Interpolated log message defeats structured logging server/src/modules/product-catalog/application/products/commands/reserve-products-stocks/reserve-products-stock.handler.ts:65
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Duplicated predicate server/src/modules/product-catalog/domain/attribute.entity.ts:275
- Duplicated predicate admin/src/data/addUploadCapabilities.tsx:52
- Duplicated predicate admin/src/resources/products/components/AttributesSubform.tsx:193
- Duplicated predicate admin/src/resources/attributes/AttributeCreate.tsx:44
- Duplicated predicate server/src/libs/ddd/command.base.ts:50
- Duplicated predicate server/src/modules/orders/application/orders/queries/find-orders/find-orders.handler.ts:46
- Duplicated predicate REDACTED:245
- Duplicated predicate REDACTED:253
- Duplicated predicate REDACTED:246
- Duplicated predicate admin/src/resources/products/ProductCreate.tsx:175
- Duplicated predicate admin/src/resources/products/ProductCreate.tsx:136
- Duplicated predicate admin/src/resources/attributes/AttributeQuickCreateForm.tsx:97
- Outdated (npm): @nestjs/event-emitter server/package.json:41
- Outdated (npm): @nestjs/mapped-types server/package.json:42
- Outdated (npm): slonik server/package.json:63
- Outdated (npm): supertokens-node server/package.json:64
- Outdated (npm): uuid REDACTED:48
- Outdated (npm): web-vitals admin/package.json:14
- README/code drift README.md:66
- README/code drift README.md:67
- README/code drift README.md:68
- README/code drift server/README.md:53
- README/code drift server/README.md:54
- README/code drift server/README.md:55
- Primitive id on a domain type: CartLineItemEntity.productId server/src/modules/orders/domain/cart-line-item.entity.ts:21
- Primitive id on a domain type: OrderLineItemEntity.productId server/src/modules/orders/domain/order-line-item.entity.ts:20
- Primitive id on a domain type: ProductsStockReservedDomainEvent.causationOrderId server/src/modules/product-catalog/domain/events/products-stock-reserved.domain-event.ts:4
- Primitive id on a domain type: StockReservationFailedDomainEvent.causationOrderId server/src/modules/product-catalog/domain/events/stock-reservation-failed.domain-event.ts:4
- Primitive id on a domain type: ProductEntity.categoryId server/src/modules/product-catalog/domain/product.entity.ts:86
- Blanket warning suppression admin/src/api/server/api.ts:1
- Blanket warning suppression admin/src/api/server/api.ts:3
- Blanket warning suppression client/src/api/server/api.ts:1
- Blanket warning suppression client/src/api/server/api.ts:3
- No named authorization policies
- No ADRs found
- Split admin
- Split server
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- REDACTED
- Inverted test pyramid
- No ADRs
- Logging is not universal
- No SAST
- Unused export 'ThemeName' admin/src/types.ts:3
- Unused export 'BasketItem' admin/src/types.ts:62
- Unused export 'Invoice' admin/src/types.ts:67
- Unused export 'Filters' client/src/components/Products/Filter/definitions.ts:18
- Unused export 'Step' client/src/components/ui/stepper.tsx:168
- Unused export 'StepLabel' client/src/components/ui/stepper.tsx:184
- Unused export 'StepContent' client/src/components/ui/stepper.tsx:208
- Unused export 'getPathname' client/src/i18n/navigation.ts:11
- Unused export 'redirect' client/src/i18n/navigation.ts:11
- Unused export 'isCategoryAccepted' client/src/utils/cookieConsentHelpers.ts:12
- Unused export 'showCookieConsentModal' client/src/utils/cookieConsentHelpers.ts:38
- Unused export 'getConsentStatus' client/src/utils/cookieConsentHelpers.ts:75
- Unused export 'NonFunctionProperties' server/src/libs/types/non-function-properties.type.ts:9
- Unused export 'RequireAtLeastOne' server/src/libs/types/require-one.type.ts:4
- Unused export 'RequireOnlyOne' server/src/libs/types/require-one.type.ts:13
- Unused export 'CreateContactInfoProps' server/src/modules/orders/domain/value-objects/contact-info.value-object.ts:11
- Unused export 'FindAllProductSlugsRequestDto' server/src/modules/product-catalog/application/products/queries/find-all-product-slugs/find-all-product-slugs.request.dto.ts:10
- Unused export 'FindCategoryParams' server/src/modules/product-catalog/domain/ports/category.repository.port.ts:4
- Unused export 'FindProductParams' server/src/modules/product-catalog/domain/ports/product.repository.port.ts:4
- Unused export 'ProductTypeAttributeModel' server/src/modules/product-catalog/infra/database/product-type.repository.ts:23
- Unused export 'ImagePublicAccessException' server/src/modules/product-catalog/infra/services/cloud-storage/exceptions.ts:9
- Unused module-local declaration 'OrderStats' admin/src/pages/dashboard/Dashboard.tsx:16
- Unused module-local declaration 'State' admin/src/pages/dashboard/Dashboard.tsx:22
- No app-layer HTTPS enforcement detected REDACTED:75
- Not all async functions that make a request accept an AbortSignal
- Null assertions (`!`) reduce the strict null checking score
Minor — 50 finding(s)
- Outdated (npm): @fontsource/roboto
- Outdated (npm): @google-cloud/storage
- Outdated (npm): @hookform/resolvers
- Outdated (npm): @nestjs/common
- Outdated (npm): @nestjs/core
- Outdated (npm): @nestjs/cqrs
- Outdated (npm): @nestjs/platform-express
- Outdated (npm): @nestjs/swagger
- Outdated (npm): @radix-ui/react-dialog
- Outdated (npm): @radix-ui/react-dropdown-menu
- Outdated (npm): @radix-ui/react-label
- Outdated (npm): @radix-ui/react-navigation-menu
- Outdated (npm): @radix-ui/react-radio-group
- Outdated (npm): @radix-ui/react-select
- Outdated (npm): @radix-ui/react-separator
- Outdated (npm): @radix-ui/react-slider
- Outdated (npm): @radix-ui/react-slot
- Outdated (npm): @radix-ui/react-tabs
- Outdated (npm): @radix-ui/react-tooltip
- Outdated (npm): @sendgrid/mail
- Outdated (npm): @sentry/cli
- Outdated (npm): @sentry/nestjs
- Outdated (npm): @slonik/migrator
- Outdated (npm): class-validator
- Outdated (npm): date-fns
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-0d59df9848494dd0adf9a1daaea29872/history.json --exit-code 0 --source . | 0 | 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-0d59df9848494dd0adf9a1daaea29872/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 . | 42 | 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 | 167 | artifacts/raw/trivy-fs.json |
| D31 · IaC & Container Security | trivy | — | trivy config --format json --quiet . | 7 | artifacts/raw/trivy-config.json |
| D32 · Data Compliance (PII/GDPR) | semgrep | — | semgrep: not applicable — No personal data was found crossing a boundary the PII/GDPR ruleset checks — nothing written to a log or console sink, placed in a URL or query string, or persisted to browser storage. That is a clean result for the LEAK surface only: this ruleset detects personal data escaping, it does not inventory the personal data a repository holds, so it is not evidence that this repository has no personal-data surface. The personal-data map (Appendix C) and the C1-C5 compliance cards are what speak to that. semgrep could not parse 1 file(s) — `client/src/api/types/types.ts` — so the PII/GDPR sweep did not cover the unparsed regions of them; rows reported elsewhere in those files are real. | 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 |