4 Commits

Author	SHA1	Message	Date
Danijel Martinek	9663c82624	fix(errors): set this.name in every domain error constructor (R6) ... Spec reviewer caught a systemic gap during Task 6 (navigation) review: Plan 8 left every feature's domain error class with a constructor that didn't set this.name = '<ClassName>'. Plan 9 spec R6 requires it. Fixed across all 10 error files: - auth: AuthenticationError, UnauthenticatedError, UnauthorizedError, InputParseError - blog: ArticleNotFoundError, InputParseError - marketing-pages: PageNotFoundError, InputParseError - navigation: HeaderNotFoundError, InputParseError - media: MediaNotFoundError, InputParseError Functionally a no-op — defineErrorMiddleware uses instanceof, not name — but ensures correct serialization, stack traces, and JSON inspection. Refactor log: §7 Spec: R6	2026-05-06 15:33:28 +02:00
Danijel Martinek	a4c4ca6b6e	refactor(features): split entities into models/ + errors/ subdirs ... All 5 features (auth, blog, marketing-pages, navigation; media has no entities yet) now follow Lazar's pattern: - entities/<x>.ts → entities/models/<x>.ts - entities/errors.ts → entities/errors/<domain>.ts + errors/common.ts Updates all import paths across factories, contracts, tests, use cases, controllers, repositories, integrations, and src/index.ts exports. navigation divergence: had no errors.ts; errors/header.ts + errors/common.ts added as new forward-looking stubs. Refactor log: docs/superpowers/refactor-logs/2026-05-05-lazar-pattern-conformance.md Spec: §5, §9.3 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-05 23:34:32 +02:00
Danijel Martinek	9415eb1c5a	test: enforce per-directory coverage thresholds ... entities + use-cases + controllers must hit 100% (95% branches). Project-wide baseline remains 80/75/80/80. Tightening these directories reflects the architectural intent: these are the pure-logic layers and should be exhaustively tested. Added @vitest/coverage-v8@^3 to core-typescript, auth, blog, marketing-pages, and navigation devDependencies. Excluded from coverage (legitimately untestable): - src/di/bind-production.ts — InversifyJS startup bootstrap - src/application/repositories/** — pure TypeScript interfaces - src/application/services/** — pure TypeScript interfaces (auth) - src/integrations/cms/** — declarative Payload CMS config - src/entities/cookie.ts — pure type aliases (auth) - src/ui/** — React Query helpers, integration-tested in apps Tests added (6 new files / 1 extended): - auth/src/entities/errors.test.ts — UnauthenticatedError, UnauthorizedError, AuthenticationError, InputParseError constructors - blog/src/entities/errors.test.ts — ArticleNotFoundError (default + custom message), InputParseError - marketing-pages/src/entities/errors.test.ts — PageNotFoundError, InputParseError - marketing-pages/src/entities/site-settings.test.ts — siteSettingsSchema (valid, no description, empty name rejection) - navigation/src/entities/header.test.ts — headerItemSchema and headerSchema validation paths - navigation/src/.../payload-header.repository.test.ts — logo relation-object, scalar-id, null, and null-item-field branches for full branch coverage Spec: §6.9	2026-05-05 19:51:34 +02:00
Danijel Martinek	692ee58365	feat(blog): add Article entity with rich-text content + domain errors	2026-05-04 22:11:08 +02:00