dd-trace 6.15.0 → 6.17.0

package/ci/test-optimization-validation/generated-test-contract.js

@@ -32,7 +32,6 @@ const GENERATED_SCENARIOS = {
  *
  * @param {string} framework test framework
  * @param {string} filename generated ATR test file
- * @returns {string} retry-state path
  */
 function getGeneratedRetryStatePath (framework, filename) {
   const namespace = `dd-test-optimization-validation-${framework}-`
@@ -52,7 +51,6 @@ function getGeneratedRetryStatePath (framework, filename) {
  * @param {string} input.moduleSystem generated module system
  * @param {string} input.scenarioId generated scenario id
  * @param {string} [input.stateFile] persistent ATR state file
- * @returns {string} canonical generated source
  */
 function getGeneratedTestContent ({ framework, moduleSystem, scenarioId, stateFile }) {
   const scenario = GENERATED_SCENARIOS[scenarioId]
@@ -232,7 +230,6 @@ function getGeneratedTestContractError (framework) {
  * Describes adapter-specific files in generated-contract errors.
  *
  * @param {string} framework framework name
- * @returns {string} additional file description
  */
 function getAdditionalGeneratedFileDescription (framework) {
   if (framework === 'cucumber') return ' plus isolated Cucumber step definitions'
@@ -245,7 +242,6 @@ function getAdditionalGeneratedFileDescription (framework) {
  *
  * @param {string} root project root
  * @param {string} filename generated path
- * @returns {boolean} whether the path is contained
  */
 function isPathInside (root, filename) {
   const relative = path.relative(path.resolve(root), path.resolve(filename))
@@ -259,7 +255,6 @@ function isPathInside (root, filename) {
  * @param {string} input.moduleSystem generated module system
  * @param {string} input.assertion passing assertion
  * @param {string} input.stateFile persistent ATR state file
- * @returns {string} retry test body
  */
 function getAtrBody ({ moduleSystem, assertion, stateFile }) {
   if (typeof stateFile !== 'string' || !path.isAbsolute(stateFile)) {

package/ci/test-optimization-validation/generated-verifier.js

@@ -227,7 +227,6 @@ function getGeneratedRuntimeFileStatus (strategy) {
  * Formats a parsed test count for a customer-facing diagnosis.
  *
  * @param {number|null} count observed test count
- * @returns {string} formatted count
  */
 function formatObservedCount (count) {
   return count === null ? 'an unknown test count' : `${count} observed tests`

package/ci/test-optimization-validation/literal-environment.js

@@ -37,7 +37,6 @@ function parseLiteralEnvironmentPrefix (command) {
  * @param {string} command command text
  * @param {string} name environment variable
  * @param {string} [platform] target platform
- * @returns {string} command without empty assignments
  */
 function removeEmptyLiteralEnvironmentAssignments (command, name, platform = process.platform) {
   const source = String(command || '')

package/ci/test-optimization-validation/manifest-scaffold.js

@@ -746,7 +746,6 @@ function hasCucumberBrowserSupport (files, retainedInputs, projectRoot) {
  * @param {string} framework framework name
  * @param {string} projectRoot project root
  * @param {boolean} allowDirectoryConvention whether a literal runner selector owns this root
- * @returns {boolean} whether the file is a candidate
  */
 function isTestFile (filename, source, framework, projectRoot, allowDirectoryConvention) {
   const basename = path.basename(filename)
@@ -775,7 +774,6 @@ function isTestFile (filename, source, framework, projectRoot, allowDirectoryCon
  *
  * @param {string} source candidate source
  * @param {string} framework framework name
- * @returns {boolean} whether ownership is explicit
  */
 function hasExplicitFrameworkImport (source, framework) {
   if (framework === 'vitest') {
@@ -797,7 +795,6 @@ function hasExplicitFrameworkImport (source, framework) {
  * @param {string} source file source
  * @param {string} framework framework name
  * @param {string} packageName project package name
- * @returns {boolean} whether ownership is plausible
  */
 function hasFrameworkOwnership (filename, source, framework, packageName) {
   const normalized = filename.replaceAll('\\', '/').toLowerCase()
@@ -821,7 +818,6 @@ function hasFrameworkOwnership (filename, source, framework, packageName) {
  *
  * @param {string} source candidate source
  * @param {string} framework selected framework
- * @returns {boolean} whether source ownership conflicts
  */
 function hasConflictingFramework (source, framework) {
   const markers = {
@@ -840,7 +836,6 @@ function hasConflictingFramework (source, framework) {
  * @param {string} filename candidate filename
  * @param {string} source candidate source
  * @param {string} projectRoot project root
- * @returns {number} lower-is-better rank
  */
 function getTestRank (filename, source, projectRoot) {
   const relative = path.relative(projectRoot, filename).replaceAll('\\', '/').toLowerCase()
@@ -857,7 +852,6 @@ function getTestRank (filename, source, projectRoot) {
  * Counts static test declarations for preferring small representative files.
  *
  * @param {string} source test source
- * @returns {number} approximate test declaration count
  */
 function getStaticTestCount (source) {
   const direct = [...source.matchAll(/\b(?:it|test)((?:\.[A-Za-z]+)*)\s*\(\s*(['"`])/g)]
@@ -873,7 +867,6 @@ function getStaticTestCount (source) {
  *
  * @param {string} projectRoot detected project root
  * @param {string} repositoryRoot repository root
- * @returns {string} preferred representative root
  */
 function findPreferredRepresentativeRoot (projectRoot, repositoryRoot) {
   if (path.resolve(projectRoot) !== path.resolve(repositoryRoot)) return projectRoot
@@ -895,7 +888,6 @@ function findPreferredRepresentativeRoot (projectRoot, repositoryRoot) {
  * Normalizes repository and package names for exact identity comparison.
  *
  * @param {string} value package identity
- * @returns {string} normalized identity
  */
 function normalizeProjectIdentity (value) {
   const unscoped = String(value || '').toLowerCase().replace(/^@[^/]+\//, '')
@@ -983,7 +975,6 @@ function findPackageRoot (packageName, projectRoot, repositoryRoot) {
  * Returns a runner package name.
  *
  * @param {string} framework framework name
- * @returns {string} package name
  */
 function getRunnerPackageName (framework) {
   return {
@@ -1024,7 +1015,6 @@ function getGeneratedTestConvention (framework, representative, projectRoot) {
  *
  * @param {string} framework framework name
  * @param {string} representative representative test
- * @returns {string} generated test suffix
  */
 function getTestExtension (framework, representative) {
   if (framework === 'cucumber') return '.feature'
@@ -1102,7 +1092,6 @@ function rankCiReviewTargets (files) {
  * Returns a lower-is-better CI review rank.
  *
  * @param {string} filename CI path
- * @returns {number} rank
  */
 function getCiRank (filename) {
   const value = filename.toLowerCase()
@@ -1219,7 +1208,6 @@ function readJson (filename) {
  *
  * @param {string} framework framework id
  * @param {Set<string>} selected selected ids
- * @returns {boolean} selection result
  */
 function isSelected (framework, selected) {
   return selected.size === 0 || selected.has(framework)
@@ -1231,7 +1219,6 @@ function isSelected (framework, selected) {
  * @param {object} packageJson package metadata
  * @param {string} projectRoot project root
  * @param {string} repositoryRoot repository root
- * @returns {string} project id
  */
 function getProjectId (packageJson, projectRoot, repositoryRoot) {
   const value = packageJson.name || path.relative(repositoryRoot, projectRoot) || 'root'
@@ -1242,7 +1229,6 @@ function getProjectId (packageJson, projectRoot, repositoryRoot) {
  * Detects package-manager metadata for reporting only.
  *
  * @param {string} root repository root
- * @returns {string} package manager
  */
 function detectPackageManager (root) {
   if (fs.existsSync(path.join(root, 'pnpm-lock.yaml'))) return 'pnpm'
@@ -1255,7 +1241,6 @@ function detectPackageManager (root) {
  * Detects workspace metadata for reporting only.
  *
  * @param {string} root repository root
- * @returns {string} workspace manager
  */
 function detectWorkspaceManager (root) {
   if (fs.existsSync(path.join(root, 'pnpm-workspace.yaml'))) return 'pnpm'
@@ -1267,7 +1252,6 @@ function detectWorkspaceManager (root) {
  * Returns a manifest operating-system name.
  *
  * @param {string} platform Node.js platform
- * @returns {string} OS name
  */
 function getManifestOs (platform) {
   return platform === 'win32' ? 'windows' : platform
@@ -1278,7 +1262,6 @@ function getManifestOs (platform) {
  *
  * @param {string} root root path
  * @param {string} filename candidate path
- * @returns {boolean} whether the candidate is contained
  */
 function isPathInside (root, filename) {
   const relative = path.relative(path.resolve(root), path.resolve(filename))

package/ci/test-optimization-validation/manifest-schema.js

@@ -94,7 +94,6 @@ function validateManifest (manifest) {
  *
  * @param {object} manifest validation manifest
  * @param {{push: function(string): void, full: function(): boolean}} errors error collector
- * @returns {void}
  */
 function validateFrameworks (manifest, errors) {
   const ids = new Set()
@@ -164,7 +163,6 @@ function validateFrameworks (manifest, errors) {
  * @param {object} project project metadata
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function validateProject (repositoryRoot, project, prefix, errors) {
   if (!isObject(project)) return
@@ -192,7 +190,6 @@ function validateProject (repositoryRoot, project, prefix, errors) {
  * @param {string} prefix error prefix
  * @param {Map<string, string>} generatedPaths generated paths used by other frameworks
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function validateRunnableFramework (repositoryRoot, framework, prefix, generatedPaths, errors) {
   if (!isObject(framework.validation)) {
@@ -276,7 +273,6 @@ function validateRunnableFramework (repositoryRoot, framework, prefix, generated
  * @param {object} validation direct-runner validation data
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function validateFallbackTests (repositoryRoot, validation, prefix, errors) {
   if (validation.fallbackTests === undefined) return
@@ -322,7 +318,6 @@ function validateFallbackTests (repositoryRoot, validation, prefix, errors) {
  * @param {string} prefix framework error prefix
  * @param {Map<string, string>} generatedPaths generated paths used by other frameworks
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function validateGeneratedStrategy (repositoryRoot, framework, prefix, generatedPaths, errors) {
   const strategy = framework.generatedTestStrategy
@@ -406,7 +401,6 @@ function validateGeneratedStrategy (repositoryRoot, framework, prefix, generated
  * @param {object} scenario generated scenario
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function validateScenarioIdentity (repositoryRoot, scenario, prefix, errors) {
   if (!Array.isArray(scenario.testIdentities) || scenario.testIdentities.length !== 1) {
@@ -431,7 +425,6 @@ function validateScenarioIdentity (repositoryRoot, scenario, prefix, errors) {
  * @param {object|undefined} ciWiring CI evidence
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function validateCiWiring (repositoryRoot, ciWiring, prefix, errors) {
   if (ciWiring === undefined) return
@@ -468,7 +461,6 @@ function validateCiWiring (repositoryRoot, ciWiring, prefix, errors) {
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
  * @param {Set<string>} [allowed] explicitly inert keys
- * @returns {void}
  */
 function rejectExecutionFields (value, prefix, errors, allowed = new Set()) {
   if (!isObject(value)) return
@@ -487,7 +479,6 @@ function rejectExecutionFields (value, prefix, errors, allowed = new Set()) {
  * @param {string} prefix error prefix
  * @param {Map<string, string>} generatedPaths previously claimed paths
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function claimGeneratedPath (filename, owner, prefix, generatedPaths, errors) {
   if (typeof filename !== 'string' || !path.isAbsolute(filename)) return
@@ -505,7 +496,6 @@ function claimGeneratedPath (filename, owner, prefix, generatedPaths, errors) {
  * @param {unknown} value current value
  * @param {string} prefix error prefix
  * @param {{push: function(string): void, full: function(): boolean}} errors error collector
- * @returns {void}
  */
 function validateAllStrings (value, prefix, errors) {
   if (errors.full()) return
@@ -534,7 +524,6 @@ function validateAllStrings (value, prefix, errors) {
  * @param {string} field field name
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function requiredObject (object, field, prefix, errors) {
   if (!isObject(object[field])) errors.push(`${prefix}.${field} must be an object.`)
@@ -547,7 +536,6 @@ function requiredObject (object, field, prefix, errors) {
  * @param {string} field field name
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function requiredArray (object, field, prefix, errors) {
   if (!Array.isArray(object[field])) errors.push(`${prefix}.${field} must be an array.`)
@@ -560,7 +548,6 @@ function requiredArray (object, field, prefix, errors) {
  * @param {string} field field name
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function requiredString (object, field, prefix, errors) {
   if (typeof object?.[field] !== 'string' || object[field].trim() === '') {
@@ -576,7 +563,6 @@ function requiredString (object, field, prefix, errors) {
  * @param {Set<string>} values accepted values
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function enumString (object, field, values, prefix, errors) {
   if (!values.has(object?.[field])) {
@@ -591,7 +577,6 @@ function enumString (object, field, values, prefix, errors) {
  * @param {string} field field name
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function requiredAbsolutePath (object, field, prefix, errors) {
   absolutePathValue(object?.[field], `${prefix}.${field}`, errors)
@@ -603,7 +588,6 @@ function requiredAbsolutePath (object, field, prefix, errors) {
  * @param {unknown} value path value
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function absolutePathValue (value, prefix, errors) {
   if (typeof value !== 'string' || !path.isAbsolute(value)) errors.push(`${prefix} must be an absolute path.`)
@@ -616,7 +600,6 @@ function absolutePathValue (value, prefix, errors) {
  * @param {string} field field name
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function optionalAbsolutePathOrNull (object, field, prefix, errors) {
   if (object[field] !== null && object[field] !== undefined &&
@@ -632,7 +615,6 @@ function optionalAbsolutePathOrNull (object, field, prefix, errors) {
  * @param {string} field field name
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function optionalStringOrNull (object, field, prefix, errors) {
   if (object[field] !== null && object[field] !== undefined && typeof object[field] !== 'string') {
@@ -646,7 +628,6 @@ function optionalStringOrNull (object, field, prefix, errors) {
  * @param {unknown} value array value
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function validateStringArray (value, prefix, errors) {
   if (value === undefined) return
@@ -667,7 +648,6 @@ function validateStringArray (value, prefix, errors) {
  * @param {unknown} filename candidate path
  * @param {string} prefix error prefix
  * @param {{push: function(string): void}} errors error collector
- * @returns {void}
  */
 function containedPath (root, filename, prefix, errors) {
   if (typeof root !== 'string' || !path.isAbsolute(root) ||
@@ -682,7 +662,6 @@ function containedPath (root, filename, prefix, errors) {
  * Returns whether a value is a plain object.
  *
  * @param {unknown} value candidate value
- * @returns {boolean} whether the value is an object
  */
 function isObject (value) {
   return value !== null && typeof value === 'object' && !Array.isArray(value)

package/ci/test-optimization-validation/offline-fixtures.js

@@ -174,7 +174,6 @@ function getOfflineScenarioNames (requestedScenario) {
  *
  * @param {object} framework framework manifest entry
  * @param {string} scenarioName cache execution name
- * @returns {string} SHA-256 fixture recipe digest
  */
 function getFixtureRecipeDigest (framework, scenarioName) {
   const generatedScenarioId = {
@@ -276,7 +275,6 @@ function mergeSettings (settings = {}) {
  * Creates or verifies a private validator-owned fixture directory.
  *
  * @param {string} directory fixture base directory
- * @returns {void}
  */
 function ensurePrivateDirectory (directory) {
   try {
@@ -298,7 +296,6 @@ function ensurePrivateDirectory (directory) {
  *
  * @param {string} directory first candidate directory
  * @param {string} stop fixture base directory
- * @returns {void}
  */
 function removeEmptyParents (directory, stop) {
   let current = directory
@@ -320,7 +317,6 @@ function removeEmptyParents (directory, stop) {
  *
  * @param {string} root candidate parent
  * @param {string} filename candidate child
- * @returns {boolean} whether the child is inside the parent
  */
 function isPathInside (root, filename) {
   const relative = path.relative(root, filename)

package/ci/test-optimization-validation/package-check.js

@@ -88,7 +88,10 @@ function getFailureDetail (result) {
 }
 
 function sanitizeLine (value) {
-  return sanitizeString(String(value)).replaceAll(/\p{Cc}+/gu, ' ').trim().slice(0, 1000).replace(/\.+$/, '')
+  const line = sanitizeString(String(value)).replaceAll(/\p{Cc}+/gu, ' ').trim().slice(0, 1000)
+  let end = line.length
+  while (end > 0 && line[end - 1] === '.') end--
+  return line.slice(0, end)
 }
 
 module.exports = { checkInstalledPackage, getInstalledPackageFailure }

package/ci/test-optimization-validation/plan-writer.js

@@ -27,7 +27,6 @@ const SCENARIO_TO_GENERATED_ID = {
  * Produces the deterministic execution plan shown before live validation.
  *
  * @param {object} input plan inputs
- * @returns {string} Markdown execution plan
  */
 function formatExecutionPlan (input) {
   return formatExecutionPlanArtifacts(input).plan
@@ -104,7 +103,6 @@ function formatExecutionPlanArtifacts ({
  * @param {object|undefined} input.packageCheck installed package-load result
  * @param {string|null} input.requestedScenario selected scenario
  * @param {string[]} input.validatorArgv exact validator command
- * @returns {string} Markdown plan
  */
 function formatApprovalPlan ({
   approvalArtifacts,
@@ -449,7 +447,6 @@ function getSelectedSupportFiles (framework, selectedScenarios) {
  *
  * @param {object} framework framework entry
  * @param {object} scenario generated scenario
- * @returns {string} source
  */
 function getGeneratedSource (framework, scenario) {
   const filename = path.resolve(scenario.testIdentities[0].file)
@@ -461,7 +458,6 @@ function getGeneratedSource (framework, scenario) {
  * Returns the execution plan path.
  *
  * @param {string} out output directory
- * @returns {string} plan path
  */
 function getExecutionPlanPath (out) {
   return path.join(out, EXECUTION_PLAN_FILENAME)
@@ -472,7 +468,6 @@ function getExecutionPlanPath (out) {
  *
  * @param {object} framework framework entry
  * @param {string} root repository root
- * @returns {string} label
  */
 function getFrameworkLabel (framework, root) {
   const project = framework.project?.name || relative(root, framework.project?.root || root)
@@ -483,7 +478,6 @@ function getFrameworkLabel (framework, root) {
  * Formats a framework status.
  *
  * @param {string} status status id
- * @returns {string} status text
  */
 function formatStatus (status) {
   return {
@@ -498,7 +492,6 @@ function formatStatus (status) {
  *
  * @param {string} root repository root
  * @param {string} filename absolute path
- * @returns {string} relative display path
  */
 function relative (root, filename) {
   const value = path.relative(root, filename)
@@ -509,7 +502,6 @@ function relative (root, filename) {
  * Formats a safe Markdown code block.
  *
  * @param {string} value text
- * @returns {string} code block
  */
 function codeBlock (value) {
   return `\`\`\`text\n${plainMultiline(value).replaceAll('```', String.raw`\u0060\u0060\u0060`)}\n\`\`\``
@@ -519,7 +511,6 @@ function codeBlock (value) {
  * Formats safe inline code.
  *
  * @param {string} value text
- * @returns {string} inline code
  */
 function inline (value) {
   return `\`${plain(value).replaceAll('`', String.raw`\u0060`)}\``
@@ -529,7 +520,6 @@ function inline (value) {
  * Sanitizes one line of plan text.
  *
  * @param {unknown} value text
- * @returns {string} safe text
  */
 function plain (value) {
   return sanitizeString(String(value ?? '')).replaceAll(/\p{Cc}+/gu, ' ').trim()
@@ -539,7 +529,6 @@ function plain (value) {
  * Sanitizes multiline plan text.
  *
  * @param {unknown} value text
- * @returns {string} safe text
  */
 function plainMultiline (value) {
   return sanitizeString(String(value ?? '')).replaceAll('\r\n', '\n').replaceAll('\r', '\n').trim()

package/ci/test-optimization-validation/preflight-runner.js

@@ -141,7 +141,6 @@ async function runCandidate ({ framework, index, options, out, testFile }) {
  * @param {object|undefined} commandFailure classified blocker
  * @param {string[]} diagnosticSummary bounded diagnostic lines
  * @param {object} framework framework manifest entry
- * @returns {string} customer-facing diagnosis
  */
 function getFailureDiagnosis (result, observedTestCount, commandFailure, diagnosticSummary, framework) {
   const sharedPrerequisite = !commandFailure && framework.allCandidatesRequireLocalSocket

package/ci/test-optimization-validation/redaction.js

@@ -167,7 +167,6 @@ function sanitizeEnvValue (name, value) {
  * Redacts common inline secret forms from strings.
  *
  * @param {string} value string to sanitize
- * @returns {string} sanitized string
  */
 function sanitizeString (value) {
   const references = []
@@ -195,7 +194,6 @@ function sanitizeString (value) {
  * Detects default-ignorable Unicode characters that can conceal executable text or secret names.
  *
  * @param {unknown} value candidate text
- * @returns {boolean} true when the text contains a default-ignorable Unicode character
  */
 function hasUnicodeDefaultIgnorable (value) {
   return DEFAULT_IGNORABLE_TEST_PATTERN.test(String(value ?? ''))
@@ -214,7 +212,6 @@ function hasUnsafeExecutionCharacter (value) {
  * Makes untrusted text safe to print to an interactive terminal while preserving line breaks.
  *
  * @param {unknown} value console value
- * @returns {string} inert console text
  */
 function sanitizeConsoleText (value) {
   let result = ''
@@ -231,7 +228,6 @@ function sanitizeConsoleText (value) {
  * Returns whether a key or variable name usually carries secret values.
  *
  * @param {string} name key or environment variable name
- * @returns {boolean} true when values under this name should be redacted
  */
 function isSensitiveName (name) {
   const normalized = String(name || '').replaceAll(/[-.]/g, '_')

package/ci/test-optimization-validation/report-writer.js

@@ -33,7 +33,6 @@ const SCENARIO_NAMES = {
  * @param {string} input.out output directory
  * @param {object} [input.staticDiagnosis] static diagnosis artifacts
  * @param {object} [input.runSummary] run summary
- * @returns {void}
  */
 function writeReport ({ manifest, results, out, staticDiagnosis, runSummary = {} }) {
   const reportPath = path.join(out, REPORT_FILENAME)
@@ -60,7 +59,6 @@ function writeReport ({ manifest, results, out, staticDiagnosis, runSummary = {}
  * @param {object} input pending report inputs
  * @param {object} input.manifest normalized manifest
  * @param {string} input.out output directory
- * @returns {void}
  */
 function writePendingReport ({ manifest, out }) {
   const reportPath = path.join(out, REPORT_FILENAME)
@@ -91,7 +89,6 @@ function writePendingReport ({ manifest, out }) {
  * @param {object[]} input.results sanitized results
  * @param {object} input.runSummary run summary
  * @param {string|undefined} input.staticDiagnosisPath static diagnosis path
- * @returns {string} Markdown
  */
 function renderReport ({ manifest, out, reportPath, results, runSummary, staticDiagnosisPath }) {
   const lines = [
@@ -190,7 +187,6 @@ function renderReport ({ manifest, out, reportPath, results, runSummary, staticD
  *
  * @param {string} out validation output directory
  * @param {string|undefined} approvedPlanSha256 approved plan digest
- * @returns {boolean} whether the current plan is safe to label as approved
  */
 function hasCurrentExecutionPlan (out, approvedPlanSha256) {
   if (!APPROVAL_DIGEST_PATTERN.test(String(approvedPlanSha256 || ''))) return false
@@ -442,7 +438,6 @@ function compactEvidence (evidence = {}) {
  * Renders JSON without allowing evidence strings to close the Markdown fence.
  *
  * @param {object} value structured evidence
- * @returns {string} fenced JSON body
  */
 function fencedJson (value) {
   return JSON.stringify(value, null, 2).replaceAll('```', String.raw`\u0060\u0060\u0060`)
@@ -454,7 +449,6 @@ function fencedJson (value) {
  * @param {object[]} results validation results
  * @param {object} runSummary run summary
  * @param {string} reportPath report path
- * @returns {string} console text
  */
 function renderConsole (results, runSummary, reportPath) {
   const lines = [
@@ -478,7 +472,6 @@ function renderConsole (results, runSummary, reportPath) {
  * Returns whether a result lacks a conclusive pass/fail.
  *
  * @param {object|undefined} result validation result
- * @returns {boolean} whether incomplete
  */
 function isIncomplete (result) {
   return Boolean(result && (
@@ -492,7 +485,6 @@ function isIncomplete (result) {
  * Returns a display status.
  *
  * @param {object} result validation result
- * @returns {string} status
  */
 function getDisplayStatus (result) {
   if (result.scenario === 'ci-wiring' && result.status === 'fail') {
@@ -520,7 +512,6 @@ function getVisibleResults (results) {
  * Returns whether detailed debugging context should be shown.
  *
  * @param {object} result validation result
- * @returns {boolean} whether details are useful
  */
 function shouldRenderDiagnostics (result) {
   return result.status === 'fail' || result.status === 'error' || isIncomplete(result)
@@ -578,7 +569,6 @@ function getArtifactDirectory (artifacts = [], out) {
  *
  * @param {string} root root path
  * @param {string} filename candidate path
- * @returns {boolean} containment
  */
 function isPathInside (root, filename) {
   const value = path.relative(root, filename)
@@ -589,7 +579,6 @@ function isPathInside (root, filename) {
  * Formats an execution status.
  *
  * @param {string|undefined} status status id
- * @returns {string} display status
  */
 function formatExecutionStatus (status) {
   return String(status || 'incomplete').replaceAll('_', ' ').toUpperCase()
@@ -628,7 +617,6 @@ function formatCleanupStatus (cleanup) {
  * Returns a display scenario name.
  *
  * @param {string} scenario scenario id
- * @returns {string} display name
  */
 function getScenarioName (scenario) {
   return SCENARIO_NAMES[scenario] || scenario
@@ -638,7 +626,6 @@ function getScenarioName (scenario) {
  * Formats a Markdown table cell.
  *
  * @param {unknown} value cell value
- * @returns {string} escaped text
  */
 function cell (value) {
   return plain(value).replaceAll('|', String.raw`\|`)
@@ -648,7 +635,6 @@ function cell (value) {
  * Formats inline code.
  *
  * @param {unknown} value code value
- * @returns {string} Markdown code
  */
 function code (value) {
   return `\`${plain(value).replaceAll('`', String.raw`\u0060`)}\``
@@ -658,7 +644,6 @@ function code (value) {
  * Sanitizes one line of report text.
  *
  * @param {unknown} value text
- * @returns {string} safe text
  */
 function plain (value) {
   return sanitizeString(String(value ?? '')).replaceAll(/\p{Cc}+/gu, ' ').trim()
@@ -669,7 +654,6 @@ function plain (value) {
  *
  * @param {string} root base directory
  * @param {string} filename file path
- * @returns {string} relative path
  */
 function relative (root, filename) {
   return path.relative(root, filename) || '.'

package/ci/test-optimization-validation/runner-command.js

@@ -121,7 +121,6 @@ function getManifestInputFiles (manifest, { includeLocal = true } = {}) {
  *
  * @param {string} scenarioId generated scenario id
  * @param {string|null} requestedScenario selected validator scenario
- * @returns {boolean} whether to include the command
  */
 function shouldIncludeGeneratedScenario (scenarioId, requestedScenario) {
   if (!requestedScenario) return true
@@ -241,7 +240,6 @@ function getRunnerEnv (framework) {
  *
  * @param {Set<string>} files collected files
  * @param {string|undefined} filename candidate file
- * @returns {void}
  */
 function addExistingFile (files, filename) {
   if (typeof filename !== 'string') return
@@ -254,7 +252,6 @@ function addExistingFile (files, filename) {
  * Formats a framework name for command descriptions.
  *
  * @param {string} framework framework name
- * @returns {string} display name
  */
 function formatFrameworkName (framework) {
   return framework.charAt(0).toUpperCase() + framework.slice(1)