Skip to content

docs: improve packages jsdoc docs#957

Merged
omermorad merged 10 commits into
masterfrom
packages-docs
Dec 9, 2025
Merged

docs: improve packages jsdoc docs#957
omermorad merged 10 commits into
masterfrom
packages-docs

Conversation

@iddan

@iddan iddan commented Dec 8, 2025

Copy link
Copy Markdown
Collaborator

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Documentation: Adds comprehensive JSDoc documentation across all public API packages with practical examples, cross-references, and links to suites.dev documentation.

What is the current behavior?

Public API types, methods, and interfaces across all Suites packages lack comprehensive JSDoc documentation. This makes it difficult for developers to understand the purpose, usage, and benefits of each API without referring to external documentation.

Issue Number: N/A

What is the new behavior?

  • Comprehensive JSDoc documentation added to all public API elements across packages:
    • @suites/unit - TestBed API, types, and error classes
    • @suites/core - UnitReference, TestBedBuilder, MockOverride interfaces
    • @suites/doubles.jest, @suites/doubles.sinon, @suites/doubles.vitest - Adapter exports and type augmentations
    • @suites/di.nestjs, @suites/di.inversify - DI adapter documentation
    • @suites/types.common, @suites/types.doubles - Base types and error documentation
  • Practical examples with imports in all JSDoc blocks
  • @see links to relevant suites.dev documentation pages
  • @since version tags for API versioning
  • @throws documentation for error conditions
  • Fixed broken TSDoc links across all packages
  • Added TSDoc link validation script (scripts/validate-tsdoc-links.ts)
  • Improved JSDoc style guide with templates and examples

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

This PR also includes minor fixes discovered during documentation work:

  • Fixed exported stub variable in doubles packages
  • Corrected type names in unit.d.ts declaration files

The new TSDoc validation script can be run to check for broken {@link} URLs in documentation comments across all packages.

@codecov

codecov Bot commented Dec 8, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.35%. Comparing base (150500d) to head (5223a20).
⚠️ Report is 10 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #957      +/-   ##
==========================================
+ Coverage   92.94%   93.35%   +0.41%     
==========================================
  Files          26       26              
  Lines         595      632      +37     
  Branches       99       99              
==========================================
+ Hits          553      590      +37     
  Misses         33       33              
  Partials        9        9              
Flag Coverage Δ
core.unit 93.44% <ø> (ø)
di.inversify 93.61% <ø> (ø)
di.nestjs 87.25% <ø> (ø)
doubles.jest 100.00% <ø> (ø)
doubles.sinon 100.00% <ø> (ø)
doubles.vitest 100.00% <100.00%> (ø)
unit 84.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@iddan iddan force-pushed the packages-docs branch 3 times, most recently from 14269dc to 58a6c5f Compare December 8, 2025 17:25
@iddan iddan added bug Something isn't working documentation Improvements or additions to documentation labels Dec 8, 2025
@iddan iddan self-assigned this Dec 8, 2025
@omermorad omermorad self-requested a review December 9, 2025 21:22
@omermorad omermorad force-pushed the packages-docs branch 2 times, most recently from 6d0cf64 to 5c1f860 Compare December 9, 2025 21:47
@omermorad omermorad changed the title Fix Package Docs docs: improve packages jsdoc docs Dec 9, 2025
@omermorad omermorad merged commit 2a06991 into master Dec 9, 2025
50 checks passed
@omermorad omermorad deleted the packages-docs branch December 9, 2025 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants