fix(fedora): expose recorded reasons and reverse requirements - #224
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
This PR is already merged (dedf54b of ee517f2 into PyRo1121/fix-fedora-size-cli at 2026-09-05T21:01:15Z). I am not reopening, reverting, rebasing, approving, or stacking further work onto this closed PR.
No leftover merge conflict remains. I did not find a leftover non-breaking hole that I can land without changing author-scoped product limits or widening package-name validation.
I will not force these through:
- Historical dependency-chain reconstruction, an installer identity, or removal-safety claims (explicitly out of scope in the PR)
- Allowing
:in package selectors so full NEVRA epochs can be typed (that would widenvalidate_package_name) - Exhaustive CLI coverage, live multilib, or published-release repair
- Sibling #219's explicit breaking
sha2/p256/p384/toml_editpin bump
A later author-owned change can take those; this run will not.
Sent by Cursor Automation: Untitled


Changes
whynow displays the recorded DNF5 installation reason.why --reverselists current direct requiring packages with native reasons and full identities. Neither mode claims historical causality, an installer identity, or removal safety.Reuse the bounded native query adapter. Keep reason labels intact, reject malformed rows and unsafe operands, and reject missing or ambiguous selectors. Installed-fact queries ignore exclusion filters; transaction policy is unchanged. Reason-only mode does not fetch reverse requirements.
Verification
Verified debug CLI SHA-256:
1995cd1970ff941f518d953acd86d47de4b6ba89188d96d744aae84c0f7f6c17.Limits
No historical dependency-chain reconstruction, exhaustive CLI coverage, benchmark, or published-release repair is claimed. Other distro feature combinations and live multilib remain to be verified.
Manual review only. No subagents were used. The automated review tool abstained on this external checkout. Draft pending independent review. No host package changes, merges, releases, or deployments.