Merge upstream NVIDIA/SkillSpector v2.3.11 (up to c2d09df)#9
Merged
Conversation
Signed-off-by: Hiten <erhiten6@gmail.com>
Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
…tion audit Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
…mentation Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
batch_scan.py main(): reconfigure stdout to UTF-8 on win32 so Rich terminal output with CJK characters renders correctly. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
…on criteria Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
…l test files - Add SPDX license header to 8 test files - Add from __future__ import annotations to 8 test files - Fix Unicode stdout crash in test_pool_wiring.py on Windows - Add conftest.py with pytest markers registration - 120 tests passing Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
set_api_pool previously only patched llm_utils.get_chat_model, but llm_analyzer_base uses a module-level from-import that created a local reference bypassing the pool. Graph analyzers (95% of LLM calls) were not using PooledChatModel. Now patches both llm_utils and llm_analyzer_base, plus adds LLMAnalyzerBase._llm verification to test_pool_wiring.py. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
Documentation (12 md, zero stale refs, cross-linked footers): - README: TOC, badges, all commands, reviewer index - REVIEW_RESPONSE: full 3-issue response, before/after tables - DESIGN: dual-patch mechanism, updated file layout - New CONTRIBUTING.md at module root (GitHub standard) - Archive 7->5: merged COMMAND_REFERENCE->README, RISK_TABLE->PITFALLS New thematic tests (44 tests, answering review concerns): - test_monkeypatch_invasiveness.py: 14 tests (thread isolation, import safety) - test_monkeypatch_fragility.py: 26 tests (per-patch guard, deep deps, atomicity) 164 tests total, all passing. Production code unchanged (runner.py fix 08f624c). Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
Signed-off-by: WhereIs38 <CinderellaDoyle@icloud.com>
Refresh the public SkillSpector tree from internal OSS branch release/oss-2026-06-30-2 at 3c2309dfde0a09f2768f60ef7e93b089d42d5fcb. Changes: - bump the package and lockfile snapshot from 2.3.7 to 2.3.9 - publish Bedrock provider dependencies in the resolved lockfile - carry the MCP least-privilege formatting fix - refresh public documentation formatting and ignored review artifacts Verification: - create-oss-release smoke gate: 1256 passed, 12 skipped, 6 xfailed - uv lock --check - pytest -q tests/test_mcp_least_privilege.py (14 passed) - Ruff lint and format checks on changed Python files - uv run reports SkillSpector 2.3.9 Signed-off-by: Keshav Pradeep <32313895+keshprad@users.noreply.github.com>
…-30-2 Sync OSS release snapshot
…ch-scanner feat: add multilingual batch scanner with parallel execution and LLM gap-fill
refactor: centralize cleanup and risk threshold
Refresh the public SkillSpector snapshot from internal branch release/oss-2026-07-06 at b86ae09dac661abef3a0c87258e7ec7f669f5f52. The merged public feature content already matches the internal OSS tree, so this release advances only the package and lockfile versions from 2.3.9 to 2.3.11. Verification: - create-oss-release.sh via uv run --all-extras (1,256 passed; 12 skipped; 34 deselected; 6 xfailed) - uv lock --check - git diff --check origin/main Signed-off-by: Keshav Pradeep <32313895+keshprad@users.noreply.github.com>
Sync OSS release snapshot (v2.3.11)
smoy
approved these changes
Jul 13, 2026
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
7 participants
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.
Summary
4cf0cb9 chore(release): bump version to 2.3.11), stopping at merge commitc2d09dfas requested.--no-ffmerge (fa7d15d), matching the prior sync pattern (d3fac00).Details
pyproject.tomlversion now reads2.3.11.contrib/multilingual/batch scanner (PR feat: add multilingual batch scanner with parallel execution and LLM gap-fill NVIDIA/SkillSpector#100), plus doc reorganization, SPDX headers, and Windows/CJK stdout fixes.c2d09df; the 6 upstream commits after it (PRs docs: correct MCP fixture expectations NVIDIA/SkillSpector#262, fix(mcp): prove stdio initialize compatibility (#199) NVIDIA/SkillSpector#248, Improve batch scan visibility: add README usage guide and rename contrib/multilingual → contrib/batch_scan NVIDIA/SkillSpector#258, …) are not included.benchmark/WIP (report module + generated artifacts) is stashed locally and intentionally excluded from this PR. Untracked.claude/anddocs/superpowers/artifacts are likewise excluded, consistent with5c17e2a.Benchmark
Report of performance when compared to Exaforce/SkillSpector main (Represented as Prune request token bloat)
report.md
Warning
Verification gap: the ExaForce schema-pruning monkeypatch (
e6ac14c,a2f6d97) targets upstream LLM/prompt code that this sync touched. The merge was textually clean but the patch's compatibility against the new upstream was not run locally. Please rely on CI + review, or run the exaforce pruning tests (cb631d8) before merging.