Code Quality
Type safety, test validity, naming conventions, dead code, documentation accuracy, and general maintainability of code that exists and resolves.
The correctness-of-existing-code layer: is the code that does exist and does resolve also maintainable, type-safe, and honestly tested?
In scope. Type safety (any-type leakage, unchecked casts, @ts-ignore without justification), test validity (assertion-free tests, over-mocked suites, coverage theater), naming conventions and consistency, dead code and unused exports, documentation accuracy (docs vs. behavior drift), cyclomatic complexity and duplication, maintainability hygiene.
Not in scope. Plural tooling doing the same job (that's dependency-coherence). References that don't resolve (that's reference-integrity). Stubs or scaffolding leaked into production (that's placeholder-hygiene).
Distinct because. Concerns correctness and maintainability of code that exists and resolves. The neighboring taxons carve off specific defect families (reference failure, plural tooling, scaffold leakage) that would otherwise overwhelm this taxon. A pattern about "test file has no expect() calls" is code-quality. A pattern about "import points at module that doesn't exist" is reference-integrity.
Conceptual sub-structure. Type safety, test validity, naming, dead code, documentation accuracy, maintainability. Formal sub-taxon paths likely warranted once catalog passes ~3,000 patterns in this taxon.
Patterns in this taxon (177)
- lowAPI changelog maintainedab-000368 · api-design
- lowAPI deprecation strategy definedab-002215 · saas-api-design
- lowAPI documentation covers exportsab-002384 · sdk-package-quality
- highAPI responses use a consistent error formatab-002291 · saas-error-handling
- mediumAPI routes or public interfaces are documentedab-000675 · code-maintainability
- criticalAPI schema definition existsab-000358 · api-design
- infoAPI surface changes documentedab-002154 · code-quality-essentials
- mediumAPI versioning strategy definedab-002165 · saas-api-design
- criticalAPI versioning strategy existsab-000365 · api-design
- mediumAssertion density is reasonable per testab-000294 · ai-slop-test-theater
- lowAsync error handling is consistent across the codebaseab-002313 · saas-error-handling
- highAsync hooks supportedab-002078 · plugin-extension-architecture
- mediumAsync operations return Promisesab-002356 · sdk-package-quality
- lowAsync/await used consistentlyab-002163 · code-quality-essentials
- lowAttribution window is configurable, not hard-codedab-000592 · campaign-analytics-attribution
- highAuthentication setup is documentedab-001001 · developer-documentation
- lowAuthorization Logic Centralizedab-002248 · saas-authorization
- mediumBefore/after hook patternsab-002080 · plugin-extension-architecture
- lowBoolean fields use descriptive prefixesab-000357 · api-design
- highBreaking change policy documentedab-000366 · api-design
- highBreaking changes follow semverab-002089 · plugin-extension-architecture
- infoCalculation logic unit-tested with edge casesab-001456 · finserv-form-validation
- highCHANGELOG with entries per versionab-002379 · sdk-package-quality
- criticalClear default entry pointab-002362 · sdk-package-quality
- criticalClear hook registration APIab-002076 · plugin-extension-architecture
- highCode examples accompany major API methodsab-000998 · developer-documentation
- highCode examples use real imports and are runnableab-001003 · developer-documentation
- highCode is free of obfuscation and minification that hides logicab-001349 · extension-store-readiness
- infoCode review checklist documentedab-002164 · code-quality-essentials
- mediumComponents follow single-responsibility principleab-000665 · code-maintainability
- lowConfigurable decay thresholdsab-000868 · data-quality-list-hygiene
- lowConfiguration via constructor, not global stateab-002390 · sdk-package-quality
- lowConflicting plugins detectedab-002092 · plugin-extension-architecture
- highConsistent API naming conventionab-002167 · saas-api-design
- highConsistent field naming conventionab-000353 · api-design
- lowConsistent naming convention across tables and columnsab-000902 · database-design-operations
- mediumConsistent response envelope formatab-002166 · saas-api-design
- highConsistent subcommand patternab-000641 · cli-quality
- mediumConsistent terminology throughoutab-001005 · developer-documentation
- lowContributing guide existsab-000996 · developer-documentation
- criticalCorrect exit codesab-000651 · cli-quality
- mediumCoverage configuration is presentab-000306 · ai-slop-test-theater
- criticalCritical source files have at least one corresponding testab-000300 · ai-slop-test-theater
- highCustom error classes exportedab-002358 · sdk-package-quality
- lowCustom error classes extend Errorab-002161 · code-quality-essentials
- lowDead code and unused exports eliminatedab-002160 · code-quality-essentials
- mediumDeprecated endpoints/fields markedab-000367 · api-design
- lowDev dependencies separatedab-002151 · code-quality-essentials
- lowDiscount logic correctly applies percentage or fixed amountsab-001124 · ecommerce-catalog
- mediumDiscriminated unions usedab-002156 · code-quality-essentials
- highDocumentation matches current versionab-001008 · developer-documentation
- highDual ESM/CJS format or clear ESM-only stanceab-002391 · sdk-package-quality
- highE2E tests exist for the primary user flowab-000301 · ai-slop-test-theater
- lowEmpty collections return empty arrayab-000375 · api-design
- mediumEnum values use consistent formatab-000354 · api-design
- highEnvironment setup is documentedab-000995 · developer-documentation
- highEnvironment variables are documented in .env.exampleab-000674 · code-maintainability
- highError codes and types are documentedab-001000 · developer-documentation
- lowError handling and boundary testsab-002146 · code-quality-essentials
- criticalError responses follow one structureab-000364 · api-design
- highError responses include codes and messagesab-002214 · saas-api-design
- mediumESLint and Prettier are configuredab-000671 · code-maintainability
- lowESLint configuredab-002162 · code-quality-essentials
- highESP abstracted behind a common interfaceab-002462 · sending-pipeline-infrastructure
- lowEvent names follow a consistent naming conventionab-001714 · marketing-analytics
- criticalEvery test file contains at least one assertionab-000291 · ai-slop-test-theater
- lowExample plugin providedab-002096 · plugin-extension-architecture
- highExecution order definedab-002077 · plugin-extension-architecture
- lowFake timers are paired with explicit clock controlab-000298 · ai-slop-test-theater
- criticalFile and folder structure follows framework conventionsab-000663 · code-maintainability
- lowFile naming convention consistencyab-000233 · ai-slop-code-drift
- highFiles field restricts published contentsab-002361 · sdk-package-quality
- highFramework dependencies listed as peerDependenciesab-002360 · sdk-package-quality
- mediumGeneric type constraints appliedab-002155 · code-quality-essentials
- lowGeneric types for flexibilityab-002389 · sdk-package-quality
- lowGit pre-commit hooks are configured for lintingab-000686 · code-maintainability
- mediumHardcoded service identifiers are scoped to config filesab-000045 · ai-slop-hallucinations
- highHook payloads typedab-002079 · plugin-extension-architecture
- highHTTP methods used correctlyab-002169 · saas-api-design
- highHTTP status codes used consistentlyab-000370 · api-design
- highIdempotent operations and idempotency keysab-000372 · api-design
- lowImport alias usage consistencyab-000234 · ai-slop-code-drift
- lowImport organization is consistentab-000667 · code-maintainability
- criticalInstallation instructions are completeab-000992 · developer-documentation
- mediumIntegration tests cover critical workflowsab-002144 · code-quality-essentials
- lowInterest/APY rounded consistentlyab-001450 · finserv-form-validation
- mediumInvalid input handlingab-000654 · cli-quality
- mediumKey functions and complex logic have inline comments or JSDocab-000673 · code-maintainability
- mediumLicense field and LICENSE file presentab-002363 · sdk-package-quality
- lowLICENSE file exists and is stated in READMEab-001013 · developer-documentation
- lowMaximum context window is explicitly configured per modelab-000314 · ai-token-optimization
- highMigration guides for major versionsab-001011 · developer-documentation
- mediumMinimal runtime dependenciesab-002381 · sdk-package-quality
- highMissing argument handlingab-000653 · cli-quality
- lowMocking strategy documentedab-002147 · code-quality-essentials
- highMocks do not swallow the system under testab-000296 · ai-slop-test-theater
- criticalModern exports field definedab-002371 · sdk-package-quality
- highNamed exports used for tree-shakingab-002355 · sdk-package-quality
- mediumNaming conventions are consistentab-000669 · code-maintainability
- lowNaming conventions consistentab-002159 · code-quality-essentials
- lowNavigation event listeners properly cleanup on unmountab-001871 · mobile-navigation-linking
- lowNegative Authorization Tests Presentab-002238 · saas-authorization
- highNew fields are additive with defaultsab-000369 · api-design
- mediumNext.js page/layout convention exports are in correct file typesab-000033 · ai-slop-hallucinations
- highNo @ts-ignore comments without explanationab-002139 · code-quality-essentials
- highNo `.skip` patterns in shipping tests without justificationab-000302 · ai-slop-test-theater
- mediumNo `if (false)` / `if (true)` debug blocksab-000268 · ai-slop-half-finished
- highNo bloated dependenciesab-002388 · sdk-package-quality
- highNo catch-all try/catch blocks that swallow errorsab-000293 · ai-slop-test-theater
- criticalNo circular dependenciesab-000664 · code-maintainability
- highNo circular module dependenciesab-002150 · code-quality-essentials
- lowNo commented-out code blocksab-000682 · code-maintainability
- lowNo console.log in production codeab-002145 · code-quality-essentials
- lowNo console.log or console.debug in production code pathsab-000683 · code-maintainability
- mediumNo dead code — unused exports, components, and functionsab-000681 · code-maintainability
- lowNo hardcoded `process.env.NODE_ENV === 'development'` blocks with empty elseab-000273 · ai-slop-half-finished
- criticalNo implicit any typesab-002137 · code-quality-essentials
- lowNo magic numbers or strings without named constantsab-000670 · code-maintainability
- mediumNo orphaned features detectedab-001544 · goal-alignment
- criticalNo side effects on importab-002357 · sdk-package-quality
- criticalNo significant scope creep detectedab-001543 · goal-alignment
- criticalNo tautological assertionsab-000292 · ai-slop-test-theater
- mediumNo TODO or FIXME comments in critical business logicab-000684 · code-maintainability
- mediumNo unused dependencies in package.jsonab-000676 · code-maintainability
- highNon-functional requirements are addressedab-001548 · goal-alignment
- criticalnoUncheckedIndexedAccess enabledab-002136 · code-quality-essentials
- mediumNullable fields explicitly markedab-000362 · api-design
- highPackage name follows conventionsab-002359 · sdk-package-quality
- highPagination pattern defined and consistentab-000363 · api-design
- lowPlugin API changelog maintainedab-002098 · plugin-extension-architecture
- highPlugin API versionedab-002088 · plugin-extension-architecture
- highPlugin authoring guide existsab-002094 · plugin-extension-architecture
- lowPlugin dependency resolutionab-002091 · plugin-extension-architecture
- criticalPlugin lifecycle hooks definedab-002075 · plugin-extension-architecture
- highPlugin loading mechanism documentedab-002095 · plugin-extension-architecture
- highPlugin manifest requiredab-002090 · plugin-extension-architecture
- mediumPolymorphic Feed Item Renderingab-000768 · community-social-engagement
- highPrerequisites are documentedab-000994 · developer-documentation
- criticalPublic API is documentedab-000997 · developer-documentation
- mediumPublic changelog existsab-001012 · developer-documentation
- mediumPublic types exported and JSDoc-commentedab-002157 · code-quality-essentials
- mediumQuery parameter naming is consistentab-000355 · api-design
- mediumRate limits and quotas are documentedab-001002 · developer-documentation
- criticalREADME exists and describes the projectab-000991 · developer-documentation
- highREADME exists and is adequately maintainedab-000672 · code-maintainability
- lowRepository metadata fields presentab-002387 · sdk-package-quality
- highRequest and response examples for API endpointsab-000999 · developer-documentation
- highResource naming follows REST conventionsab-000352 · api-design
- highResponse objects are typedab-000361 · api-design
- mediumRoute handler files only export valid Next.js convention namesab-000031 · ai-slop-hallucinations
- criticalSchema matches implementationab-000359 · api-design
- highScoring model documented and versionedab-000614 · campaign-orchestration-sequencing
- lowSequence templates reusable across campaignsab-000603 · campaign-orchestration-sequencing
- mediumSetup and teardown are balancedab-000297 · ai-slop-test-theater
- mediumShared logic is extracted into reusable modulesab-000666 · code-maintainability
- mediumSort/filter fields documentedab-000371 · api-design
- criticalStdout/stderr separationab-000645 · cli-quality
- highStrict null checks activeab-002141 · code-quality-essentials
- mediumTest bodies are not empty or TODO-onlyab-000303 · ai-slop-test-theater
- highTest files exist for critical application pathsab-000685 · code-maintainability
- mediumTest files present and organizedab-002142 · code-quality-essentials
- lowTest names are descriptiveab-000295 · ai-slop-test-theater
- lowTest naming convention consistentab-002158 · code-quality-essentials
- lowTest utilities for pluginsab-002097 · plugin-extension-architecture
- infoTests use a fake clock when time is involvedab-000307 · ai-slop-test-theater
- infoTransactional and marketing emails properly distinguishedab-001253 · email-sms-compliance
- highTree-shaking enabledab-002392 · sdk-package-quality
- criticalTypeScript compilation produces no errorsab-000680 · code-maintainability
- criticalTypeScript strict mode enabledab-002135 · code-quality-essentials
- criticalTypeScript strict mode or equivalent linting is enabledab-000668 · code-maintainability
- highTypeScript type coverage above 85%ab-002140 · code-quality-essentials
- criticalTypeScript type declarations shippedab-002370 · sdk-package-quality
- mediumUnit test coverage above 60%ab-002143 · code-quality-essentials
- highURL segments use one casing styleab-000356 · api-design
- highVersion flag outputs version stringab-000640 · cli-quality
- highVersion history shows semver complianceab-002383 · sdk-package-quality
- infoVersion information is documented or accessibleab-001954 · mobile-store-readiness