Skip to content

Updated code linting reports#2619

Merged
robgruen merged 28 commits into
mainfrom
dev/robgruen/code_complexity
Jul 8, 2026
Merged

Updated code linting reports#2619
robgruen merged 28 commits into
mainfrom
dev/robgruen/code_complexity

Conversation

@robgruen

@robgruen robgruen commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

This pull request introduces several improvements and cleanups to the code quality tooling and tests, focusing on making code debt gating more robust and maintainable. It adds support for baseline exceptions in code quality gates, improves skipped/focused test detection, re-enables a previously skipped grammar test, and replaces a dead test with a more meaningful one. There are also small updates to documentation and npm scripts to reflect these changes.

Code quality tooling enhancements:

  • Added support for baseline exception files (--exceptions-file) to the code-lint, code-complexity, code-debt, and code-circular tools, allowing known issues to be grandfathered and not block the ratchet/gate. Exception file format and usage are documented in README.md. [1] [2] [3] [4] [5] [6] [7]
  • Improved detection of skipped and focused tests in code-debt tooling: now recognizes .skip.each and .only.each, and ignores placeholder stubs (empty () => {} bodies). [1] [2] [3]

Test and CI improvements:

  • Re-enabled the CalendarDate grammar loader test in dynamicGrammarLoader.spec.ts now that the required converter exists, and updated its expected output to match the new slot value format. [1] [2]
  • Removed the dead, no-op api/test/api.spec.ts and replaced it with a real web server smoke test in api.test.ts that exercises static file serving and 404 handling. [1] [2]
  • Updated the code-lint and lint:fix npm scripts to use the new ratchet/gate mechanisms and only fix changed files. [1] [2]

Documentation updates:

  • Updated README.md to reflect new test debt numbers (skipped tests down to 2, both intentional), improved explanations of the ratchet/gate process, and documented the new baseline exception mechanism. [1] [2] [3] [4]

Other minor improvements:

  • Adjusted reporting in circularDepsReport.ts and complexityReport.ts to show all relevant results instead of slicing to a top-N, and improved output for baseline exceptions. [1] [2]

These changes collectively make code quality enforcement more flexible and accurate, while reducing false positives and technical debt noise.

robgruen and others added 28 commits July 2, 2026 21:38
…ibute sanitization'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ist to each report/gate/check.

pnpm lint now runs these checks in addition to prettier
@robgruen robgruen marked this pull request as ready for review July 8, 2026 03:45
@robgruen robgruen added this pull request to the merge queue Jul 8, 2026
Merged via the queue into main with commit 3548f69 Jul 8, 2026
21 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants