Corpus judging register, verdict merge, and 20 Dependabot patches - #517
Closed
abdushakoor12 wants to merge 12 commits into
Closed
abdushakoor12 wants to merge 12 commits into
abdushakoor12 wants to merge 12 commits into
Conversation
brace-expansion, js-yaml, linkify-it, serialize-javascript and undici in site/ and clients/vscode/, plus the time crate in Cargo.lock. serialize-javascript and diff had no patched release inside mocha 11's ranges, so mocha moves to ^12. @vscode/test-cli 0.0.15 still pins mocha ^11.7.6, which would nest a vulnerable copy, so an override dedupes both to mocha 12. All three npm workspaces report 0 vulnerabilities. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
| timeout-minutes: 15 | ||
| steps: | ||
| - uses: actions/checkout@v7 | ||
| - uses: dtolnay/rust-toolchain@stable |
Collaborator
Author
|
Closing: this PR carried the whole deps branch (11 pre-existing commits, 154 unrelated files) when only the 4-file dependency patch was intended. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two things land together here — the corpus judging work this branch was already carrying, and a dependency security pass on top.
Dependency patches
Clears all 20 open Dependabot alerts (12 high):
serialize-javascriptanddiffhave no patched release inside mocha 11's^6.0.2/^7.0.0ranges, so mocha moves to^12.@vscode/test-cli@0.0.15still hard-pinsmocha: ^11.7.6and would nest a vulnerable copy, so an override dedupes both to a single mocha 12. CI's node22.xsatisfies mocha 12's>=22.12.0engine.All three npm workspaces now report 0 vulnerabilities.
npm auditalso clearedqs→ 6.16.0 anddiff→ 9.0.0.Corpus judging register
The blinded judging workspace, verdict merge, score gate and version-comparison scripts, with their spec sections and contract tests.
Verification
cargo build --workspace --all-targets·make _ci-analyze(clippy-D warnings, taxonomy, env/PATH) ·make _vsix-test— 476 tests green under mocha 12 · vscode typecheck, eslint,test:scripts(50) ·npm ciin both packages · site build.Known red
score-gate-source.test.mjs—[CORPUS-REGISTER-QUEUE]fails becausecorpus/judging-queue.jsonis empty. Left red rather than weakened.make linthangs onkill-deslop-processes.test.mjswith Homebrew node. Its fixture copiesprocess.execPathto a temp dir, but Homebrew'snodeis a 68KB launcher whose@rpath/libnode.dylibno longer resolves once moved, so the child dies before printingready— andstartNamedhas no rejection path for child exit and no timeout. Pre-existing, not addressed here.🤖 Generated with Claude Code