Repository navigation
release: prepare minimal SkillSpector 2.13.1 parser patch - #878
Draft
mohgupta-ship-it wants to merge 8 commits into
Draft
mohgupta-ship-it wants to merge 8 commits into
mohgupta-ship-it wants to merge 8 commits into
Conversation
From Codex. Scope the backported marker tests to the selected 2.13 release: retain semicolon marker controls and the existing invalid-Python syntax-error ledger. The unselected section-sign marker hardening is not claimed by this patch. Signed-off-by: Mohit Gupta <mohgupta@nvidia.com>
From Codex. Retain the existing operand fallback around recognized execution and reflection. Follow bounded in-memory bundled helpers and import initialization before isolating a report string value; unsupported local evidence stays conservative. Preserve output-only helpers and default non-shell calls without lowering findings or incomplete-scan severity. Signed-off-by: Mohit Gupta <mohgupta@nvidia.com>
From Codex. Prepare the selected 2.13 maintenance release, keeping third-party lock entries, provider defaults and the publication workflow unchanged. Exclude the release uv cache from source distributions and exercise the actual sdist backend. This commit does not tag or publish a release. Signed-off-by: Mohit Gupta <mohgupta@nvidia.com>
From Codex. A valid small bundled Python module can exceed the optional AST unparser recursion limit. Preserve conservative shell inspection instead of failing analysis, with a portable synthetic regression that verifies caller partial inspection and completed helper analysis. Clarify the existing maintenance release publication trigger. Signed-off-by: Mohit Gupta <mohgupta@nvidia.com>
From Codex. Retain conservative shell fallback when the optional import proof encounters recursion exhaustion while filling initialization stub locations or reparsing a validated helper. Cover deep module expressions, default arguments and return annotations without catching runtime deadline exceptions. Signed-off-by: Mohit Gupta <mohgupta@nvidia.com>
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.
From Codex, preparing this maintenance patch on Mohit’s behalf.
Prepare SkillSpector 2.13.1 from published v2.13.0 (
091a509b3d6ba8ff5be528966e1f43c2e68cf214) throughyashraj/v2.13.0-base.Ordinary report strings, quoted Python/Perl text and possessive Markdown prose can exhaust shell-parser fallback and produce false incomplete-inspection results. This patch backports the bounded host-language prerequisites from #778 and #784 with the parser correction from #849. It retains conservative incomplete inspection when report-shaped values reach recognized shell execution or ambiguous bundled helpers. Deep valid Python expressions retain that fallback when optional AST processing exceeds its recursion limit, instead of failing the analyzer.
This is the separate minimal patch selected for the reported completeness issue. #852 remains the broader alternative and retains its additional #452, #790, #797 and #798 backports. Those additions are not included here. The third-party dependency lock, provider/model policy, TypeScript safeguards, atomic finding-baseline safeguards and publishing workflow remain those of v2.13.0. Release metadata is 2.13.1, and source distributions exclude the release build cache.
Validation is bound to the unchanged source commit
c18d97262f1503efb2c57632054a607298b56e24; existing matching evidence is reused:188dd49c3194c892372b8fa0500ca185013d2d6e490e7f349d2d48320808e85a.This PR remains a draft while the required full-suite timing failures are investigated and resolved. Merging to
yashraj/v2.13.0-basewithrelease:publishalready present triggers the existing GitHub v2.13.1 release with wheel and source-distribution assets. Without that label, merging does not publish a release. The workflow does not publish to PyPI. No publication label is requested here.