Repository navigation
fix(release-management): match versioned dependency coordinates - #30
Draft
MakerFlyFly wants to merge 1 commit into
Draft
MakerFlyFly wants to merge 1 commit into
MakerFlyFly wants to merge 1 commit into
Conversation
A full deny coordinate followed by a colon was rejected by the suffix boundary, so versioned Gradle and Maven declarations returned no hit. Allow that delimiter only for coordinate terms while retaining the existing bare-artifact and token boundaries. Cover string declarations, lock entries, dependency trees, normalization, manifest controls, false positives and the CLI hard-stop output. Generated-by: Codex (GPT-6)
This was referenced Nov 1, 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
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
group:artifactfollowed by:versionor Maven type/version fields currently returns no Category-X hit. Permit that colon suffix for coordinate terms so release-prepare Step 2b emits the existing hard stop for these occurrences.Type of change
tools/*/withpyproject.toml)Test plan
release-managementsuite, 718 passed.ruff check src tests,ruff format --check src tests, andmypypass; mypy checked 64 source files.prek run --all-files --skip workspace-pytestpasses, including lychee, workspace lint/type checks, validators and spec checks. The first run normalized the three changed files to LF; the rerun passed.prek run workspace-pytest --from-ref 30ee18c2a7d46a42ce94278beb197fa106e94763 --to-ref HEADpasses. The actual commit also ran its normal hooks successfully.pre-first-pr-check: categories A-E pass, readinessready, zero blocking items.RFC-AI-0004 compliance
Linked issues
Closes #29.
Notes for reviewers
mainfor human audit before any upstream submission. The diff contains one commit and three files; no unrelated upstream commits are included.pre-first-pr-checkcompleted successfully.Gen-AI disclosure: Codex (GPT-6) assisted with reproduction, implementation, tests and this description. The commit carries a
Generated-by:trailer. Human contributor review is pending; this fork Draft PR and its linked issue are the audit copies.