Problem
tools/check_docs.ts only validates a single file:
await checkDocs([
import.meta.resolve("../packages/fresh/src/error.ts"),
]);The checkDocs() function in check_docs_lib.ts is fully capable of checking multiple specifiers — it runs deno doc --lint plus custom assertions on @param, @return, @example tags, etc. But the tool is only pointed at error.ts.
What's missing
The public API surface defined in packages/fresh/src/mod.ts exports:
App (class)createDefine (function)csrf, cors, csp, ipFilter (middlewares)staticFiles, trailingSlashes (middlewares)page (handler)HttpError (class) ← only this is checked
None of the other 8+ public symbols have their JSDoc validated by CI.
Suggested Fix
Expand the specifier list to cover all public API entry points, or at minimum the key ones defined in mod.ts. The checkDocs library already supports multiple specifiers with no additional work needed.
Problem
tools/check_docs.tsonly validates a single file:The
checkDocs()function incheck_docs_lib.tsis fully capable of checking multiple specifiers — it runsdeno doc --lintplus custom assertions on@param,@return,@exampletags, etc. But the tool is only pointed aterror.ts.What's missing
The public API surface defined in
packages/fresh/src/mod.tsexports:App(class)createDefine(function)csrf,cors,csp,ipFilter(middlewares)staticFiles,trailingSlashes(middlewares)page(handler)HttpError(class) ← only this is checkedNone of the other 8+ public symbols have their JSDoc validated by CI.
Suggested Fix
Expand the specifier list to cover all public API entry points, or at minimum the key ones defined in
mod.ts. ThecheckDocslibrary already supports multiple specifiers with no additional work needed.