[codex] test: enforce runtime dependency parity across manifests#685
Open
madara88645 wants to merge 1 commit into
Open
[codex] test: enforce runtime dependency parity across manifests#685madara88645 wants to merge 1 commit into
madara88645 wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
PR risk assessment
Risk level: Low
Code review: Not required (automation approval)
Evidence (from diff only)
| Area | Change | Risk note |
|---|---|---|
tests/test_snyk_workflow.py |
New regression tests (68 lines) | Locks pyproject.toml ↔ requirements.txt parity and Snyk workflow shape |
requirements.txt |
Adds httpx<0.29; normalizes jinja2 casing |
Brings deploy snapshot in line with existing pyproject.toml runtime deps (no new package introduced) |
.github/workflows/snyk.yml |
Drops pip install -e .; scans requirements.txt and pyproject.toml explicitly |
CI/security scanning only; tighter scope than prior editable install |
Why Low (not Medium+)
- No application/runtime logic, auth, API routes, or user-facing UI changed.
- Blast radius is limited to dependency manifests and the Snyk GitHub Action workflow.
httpx<0.29already appears inpyproject.toml; this PR fixes manifest drift rather than changing declared runtime behavior.- CI signal on the PR is green (Smoke, Analyze, snyk, CodeQL, GitGuardian, Vercel).
Actions taken
- Approved per Low-risk rules (no CODEOWNERS file; no reviewers requested; PR was not previously approved).
Slack summary
PR #685 — Low risk — APPROVED
[codex] test: enforce runtime dependency parity across manifests
https://github.com/madara88645/Compiler/pull/685
Changes: Snyk workflow scans explicit manifests; requirements.txt synced to pyproject.toml; new parity regression tests.
Product impact: Stronger CI/security alignment; no direct end-user behavior change.
Reviewers: None assigned (below Medium threshold).
Sent by Cursor Automation: Assign PR reviewers
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.


What changed
This PR tightens the Snyk workflow and Python dependency snapshots so they stay aligned.
requirements.txtandpyproject.tomlexplicitlyrequirements.txtby addinghttpx<0.29and normalizingjinja2casingWhy this changed
The repo had drift between the runtime dependency source of truth and the deploy/security snapshot. That makes it easy for security checks or deploy targets to validate a different package set than the app actually declares.
Product impact
This reduces the chance of silent package mismatch issues in CI, security scanning, and deployment packaging. It does not change end-user behavior directly.
Root cause
requirements.txthad fallen out of sync withpyproject.toml, and the Snyk workflow was scanning a broader editable install instead of the exact Python manifest files.Validation
uv run --extra dev pytest -q tests/test_snyk_workflow.py tests/test_ci_workflow.py