docs: Document code testing and coverage requirements for doc-detective-common#149
docs: Document code testing and coverage requirements for doc-detective-common#149promptless[bot] wants to merge 3 commits intomainfrom
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
![promptless[bot]](https://avatars.githubusercontent.com/in/979718?s=60&v=4){kind=small}
|
|
||
| This repo doesn't depend on any other Doc Detective repos. No newline at end of file | ||
| This repo doesn't depend on any other Doc Detective repos. | ||
|
|
There was a problem hiding this comment.
Citation: Based on testing infrastructure added in PR #151, which introduced comprehensive code coverage tooling including c8, coverage-thresholds.json with 100% baseline thresholds, scripts/check-coverage-ratchet.js for ratchet enforcement, new test files (test/validate.test.js, test/resolvePaths.test.js), and npm scripts (test:coverage, test:coverage:ratchet, test:coverage:html). Also references CI/CD changes in .github/workflows/auto-dev-release.yml and .github/workflows/npm-test.yml.
View source
| This repo doesn't depend on any other Doc Detective repos. No newline at end of file | ||
| This repo doesn't depend on any other Doc Detective repos. | ||
|
|
||
| ## TypeScript codebase |
There was a problem hiding this comment.
Citation: TypeScript migration PR describing the new codebase structure. Source files moved from .js to .ts, build process generates CommonJS, ESM, and TypeScript declarations in dist/. Tests remain JavaScript and run against compiled output. See PR description for build pipeline details and tsconfig.json for TypeScript configuration.
View source
| | `src/validate.ts` | `test/validate.test.js` | | ||
| | `src/resolvePaths.ts` | `test/resolvePaths.test.js` | | ||
| | `src/files.ts` | `test/files.test.js` | | ||
|
|
There was a problem hiding this comment.
Citation: Added src/schemas/index.ts → test/schema.test.js mapping based on the merged TypeScript migration PR. The diff shows test/schema.test.js was updated alongside src/schemas/index.ts.
View source
Open this suggestion in Promptless to view citations and reasoning process
Updates contributor documentation for
doc-detective-commonto reflect the new testing and code coverage requirements introduced in PR #151. Contributors now need to maintain 100% code coverage across all metrics (lines, statements, functions, branches) with automated ratchet checks enforcing these thresholds in CI. The documentation includes test file locations, commands for running tests with coverage, and TDD workflow guidance.Trigger Events
Help us improve Promptless — If this suggestion missed the mark, please share quick feedback.
If you want Promptless to make further changes on this PR, feel free to leave a comment tagging Promptless (It won't show up in the user drop down but Promptless will get it!)