build(deps-dev): bump eslint-plugin-jsdoc and Node 22 floor#339
Conversation
There was a problem hiding this comment.
OpenCode Agent could not approve because GitHub Checks were still pending before approval.
- Result: REQUEST_CHANGES
- Reason: current-head GitHub Checks did not all complete before the bounded approval wait ended for
b31a52d8c85e9a2f36add04ff07d33fb971be2f8. - Head SHA:
b31a52d8c85e9a2f36add04ff07d33fb971be2f8 - Workflow run: 27635784166
- Workflow attempt: 1
Pending checks:
- build-baseline/build / windows / amd64: IN_PROGRESS (https://github.com/Seongho-Bae/bandscope/actions/runs/27635782835/job/81722481549)
- ci/gate / ci / rust-check: QUEUED (https://github.com/Seongho-Bae/bandscope/actions/runs/27635784933/job/81722485516)
- build-baseline/build / macos / amd64: IN_PROGRESS (https://github.com/Seongho-Bae/bandscope/actions/runs/27635782835/job/81722481548)
- build-baseline/build / macos / arm64: QUEUED (https://github.com/Seongho-Bae/bandscope/actions/runs/27635782835/job/81722481588)
The OpenCode approval gate must be rerun after these checks complete so failed Strix or other check logs can be mapped to exact source lines before approval.
OpenCode Review Overview
Pull request overviewRoutine updates to GitHub Actions workflows and package dependencies. Verified no breaking changes in dependent code paths through structural exploration. Security audit workflow lacks tests but this is pre-existing. FindingsNo blocking findings from OpenCode's independent review. Verification
Gate evidence
|
|
@coderabbitai review Scheduled PR review/merge pass found zero unresolved review threads, but this head is not approved yet (CHANGES_REQUESTED). Please review this current head so the normal merge gate can decide it. |
Bumps [eslint-plugin-jsdoc](https://github.com/gajus/eslint-plugin-jsdoc) from 62.9.0 to 63.0.5. - [Release notes](https://github.com/gajus/eslint-plugin-jsdoc/releases) - [Commits](gajus/eslint-plugin-jsdoc@v62.9.0...v63.0.5) --- updated-dependencies: - dependency-name: eslint-plugin-jsdoc dependency-version: 63.0.4 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
b31a52d to
1252cb2
Compare
…ugin-jsdoc-63.0.4
There was a problem hiding this comment.
Pull request overview
Updated eslint-plugin-jsdoc to v63.0.5. The changes are limited to the dependency version and the lockfile update. No CI failures were reported, indicating the update is safe.
Findings
No blocking findings from OpenCode's independent review.
Verification
- Review source: independent OpenCode review of the current checkout, focused changed hunks, and current-head GitHub Check evidence.
- Result: APPROVE
- Reason: Dependency update with no breaking changes detected and no CI failures
Gate evidence
- Head SHA:
85e13b536d8191083dce9f78239369e1c1895e5b - Workflow run: 27729286976
- Workflow attempt: 1
There was a problem hiding this comment.
Pull request overview
OpenCode found current-head GitHub Check failures and could not approve until they are mapped to source-backed fixes.
Findings
Line-specific fallback findings:
No deterministic missing-string markers or Strix report locations were recognized. Use the failed-check evidence below to map each failed check to exact local source lines before approving.
Verification
- Review source: independent OpenCode failed-check diagnosis using current-head check evidence.
- Result: REQUEST_CHANGES
- Reason: one or more GitHub Checks failed on current head
1f0be73b45525ff516b35dffbb8b0d6d03253017.
Gate evidence
- Head SHA:
1f0be73b45525ff516b35dffbb8b0d6d03253017 - Workflow run: 27730684901
- Workflow attempt: 1
Failed checks:
- build-baseline/build / macos / amd64: CANCELLED (https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82036879878)
- build-baseline/build / macos / arm64: CANCELLED (https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82036879909)
- build-baseline/gate / build / macos: CANCELLED (https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82038327484)
- build-baseline/release-artifact / publish: CANCELLED (https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82038327658)
Failed check evidence for line-specific fixes:
Failed GitHub Check Evidence
- PR: #339
- Head SHA:
1f0be73b45525ff516b35dffbb8b0d6d03253017 - Repository:
Seongho-Bae/bandscope
Line-specific repair contract
-
Treat the check logs and annotations below as diagnostic evidence, not as a complete review.
-
For each actionable failed check, inspect the local source or diff and identify the exact file line that must change.
-
OpenCode
REQUEST_CHANGESfindings must includepath,line,root_cause,fix_direction,regression_test_direction, andsuggested_diff. -
Do not request changes with only a GitHub Actions URL or a generic check name.
-
When Strix logs contain multiple
Vulnerability ReportorModel ... Vulnerabilities ...sections, include every model-reported vulnerability in the review evidence and findings, including model name, title, severity, endpoint, and Code Locations/path:line evidence when present. -
Create one OpenCode finding per Strix model vulnerability report; do not satisfy two model reports with one combined finding, even when titles or locations match.
Failed check: build-baseline/build / macos / amd64
- Type:
check_run - Conclusion:
CANCELLED - Details URL: https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82036879878
- Workflow run id:
27730684902 - Check run id:
82036879878
Failed job steps
- step 11: Build native shell (cancelled)
Check annotations
- .github:1-1 [failure] The run was canceled by @seonghobae.
- .github:209-209 [failure] The operation was canceled.
Failed check: build-baseline/build / macos / arm64
- Type:
check_run - Conclusion:
CANCELLED - Details URL: https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82036879909
- Workflow run id:
27730684902 - Check run id:
82036879909
Check annotations
- .github:1-1 [failure] The run was canceled by @seonghobae.
Failed check: build-baseline/gate / build / macos
- Type:
check_run - Conclusion:
CANCELLED - Details URL: https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82038327484
- Workflow run id:
27730684902 - Check run id:
82038327484
Failed check: build-baseline/release-artifact / publish
- Type:
check_run - Conclusion:
CANCELLED - Details URL: https://github.com/Seongho-Bae/bandscope/actions/runs/27730684902/job/82038327658
- Workflow run id:
27730684902 - Check run id:
82038327658
…ugin-jsdoc-63.0.4
There was a problem hiding this comment.
Pull request overview
Routine updates to GitHub Actions workflows and package dependencies. Verified no breaking changes in dependent code paths through structural exploration. Security audit workflow lacks tests but this is pre-existing.
Findings
No blocking findings from OpenCode's independent review.
Verification
- Review source: independent OpenCode review of the current checkout, focused changed hunks, and current-head GitHub Check evidence.
- Result: APPROVE
- Reason: Minor workflow and dependency updates with no structural risks
Gate evidence
- Head SHA:
e5c624f739a89d8cf32693ab09cdcc73eb6c3c57 - Workflow run: 27740367975
- Workflow attempt: 1
|
Temporary ruleset handling for current head |
Maintainer scope note: this PR also tightens the repository Node.js runtime floor to
>=22.13 <23and pins CIactions/setup-nodecalls to22.22.3, matchingeslint-plugin-jsdoc@63engine requirements while keeping the repo on Node 22.Bumps eslint-plugin-jsdoc from 62.9.0 to 63.0.5.
Release notes
Sourced from eslint-plugin-jsdoc's releases.
... (truncated)
Commits
a50f71ffix(no-undefined-types): check descendant scopes for variables; fixes #1704b993425fix: ensure tsModule check can catch multiple modulesd8f4738fix(no-undefined-types): treat TS module vars as defined; fixes #170150a7fbcchore: update semver and devDeps.6041995docs: fix in output319e84bfix: allow typedef returns that may be void; fixes #1390 (#1699)938a1f0fix(empty-tags): preserve start and ending delimiters to avoid erros with s...1f857a9chore: update jsdoccomment, comment-parser, object-deep-merge, semver, devDeps.3d53b88docs: fix AST and Selectors links; closes #1691a0b05f5chore(deps): bump minimatch from 3.0.5 to 10.2.5