build(deps-dev): bump eslint-plugin-jsdoc from 63.3.3 to 64.2.1 - #928
build(deps-dev): bump eslint-plugin-jsdoc from 63.3.3 to 64.2.1#928dependabot[bot] wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
OpenCode cannot approve yet because required coverage evidence did not pass.
Review outcome
1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence
-
Problem: The required coverage-evidence job result was
failure, so OpenCode cannot establish approval sufficiency for this head. -
Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.
-
Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports
successwith required evidence or explicit no-source not-applicable evidence. -
Regression test: Keep the approval branch checking
needs.coverage-evidence.result == successbefore posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present. -
Result: REQUEST_CHANGES
-
Reason: coverage-evidence result was
failure, so required test/docstring evidence was not proven for current headf87c0fbdca3b69af444a617412e3d620171b2ee4. -
Head SHA:
f87c0fbdca3b69af444a617412e3d620171b2ee4 -
Workflow run: 32188602473
-
Workflow attempt: 1
Coverage evidence
Coverage evidence job did not run or did not publish coverage evidence.
Changed-File Evidence Map
flowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Changed file (2 files)"]
S1 --> I1["repository behavior"]
I1 --> R1["Review risk: Changed file (2 files)"]
R1 --> V1["required checks"]
OpenCode Review Overview
Pull request overviewOpenCode cannot approve yet because required coverage evidence did not pass. Review outcome1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence
Coverage evidenceCoverage evidence job did not run or did not publish coverage evidence. Changed-File Evidence Mapflowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Changed file (2 files)"]
S1 --> I1["repository behavior"]
I1 --> R1["Review risk: Changed file (2 files)"]
R1 --> V1["required checks"]
|
|
Converted this PR to Draft because its own upstream compatibility contract is stricter than BandScope's protected-base runtime contract. Exact head remains |
There was a problem hiding this comment.
Pull request overview
OpenCode cannot approve yet because required coverage evidence did not pass.
Review outcome
1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence
-
Problem: The required coverage-evidence job result was
failure, so OpenCode cannot establish approval sufficiency for this head. -
Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.
-
Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports
successwith required evidence or explicit no-source not-applicable evidence. -
Regression test: Keep the approval branch checking
needs.coverage-evidence.result == successbefore posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present. -
Result: REQUEST_CHANGES
-
Reason: coverage-evidence result was
failure, so required test/docstring evidence was not proven for current headf87c0fbdca3b69af444a617412e3d620171b2ee4. -
Head SHA:
f87c0fbdca3b69af444a617412e3d620171b2ee4 -
Workflow run: 32193263753
-
Workflow attempt: 1
Coverage evidence
Coverage evidence job did not run or did not publish coverage evidence.
Changed-File Evidence Map
flowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Changed file (2 files)"]
S1 --> I1["repository behavior"]
I1 --> R1["Review risk: Changed file (2 files)"]
R1 --> V1["required checks"]
seonghobae
left a comment
There was a problem hiding this comment.
Current exact head f87c0fbdca3b69af444a617412e3d620171b2ee4 is not ready to integrate. The v64 dependency graph itself declares node: ^22.22.2 || >=24.15.0 (including @es-joy/jsdoccomment), while protected develop still advertises/supports root Node >=22.13 <23 and CLAUDE.md documents the same floor. CI is green only because its runner is already pinned to Node 22.22.3; that does not prove the repository's declared 22.13–22.22.1 compatibility. This is a branch-introduced compatibility-contract gap, not the inherited #783 security baseline or central coverage tooling.
Canonical ownership already exists in #896 (build(node): coordinate Node 22.22.2 floor with jsdom 30), which raises the root floor to >=22.22.2 <23 and adds an exact-floor compatibility lane. Do not duplicate or hand-edit that generator-sensitive lock/toolchain authority here. Rebase/recreate this update only after #896 (itself ordered after #783) establishes the new protected-base Node contract, then require fresh exact-head install/lint/test/build evidence on Node 22.22.2. Until then this major dev-dependency bump is incompatible with the live protected-base support contract.
Bumps [eslint-plugin-jsdoc](https://github.com/gajus/eslint-plugin-jsdoc) from 63.3.3 to 64.2.1. - [Release notes](https://github.com/gajus/eslint-plugin-jsdoc/releases) - [Commits](gajus/eslint-plugin-jsdoc@v63.3.3...v64.2.1) --- updated-dependencies: - dependency-name: eslint-plugin-jsdoc dependency-version: 64.2.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
f87c0fb to
d325c6a
Compare
Bumps eslint-plugin-jsdoc from 63.3.3 to 64.2.1.
Release notes
Sourced from eslint-plugin-jsdoc's releases.
... (truncated)
Commits
dc99c3cfix: update jsdcoccomment, devDeps (#1746)8a37bd2feat(check-param-names): add order fixer and suggestions (#1732)7309d7dfeat: update jsdoccomment, devDeps.; use nodenext for module properties in TS4de1b19chore: update jsdoccomment, devDep.fbe2a2bfix: update jsdoccomment and devDep.343516afix: resume support for CJS via require(esm)7e94c04chore: update jsdoccomment, comment-parser, devDeps.9c9b723feat: ESM only; fixes #1626