Skip to content

Fix/agent fixes#18

Merged
danielturull merged 3 commits into
mainfrom
fix/agent-fixes
Jun 10, 2026
Merged

Fix/agent fixes#18
danielturull merged 3 commits into
mainfrom
fix/agent-fixes

Conversation

@danielturull

Copy link
Copy Markdown
Contributor

No description provided.

Extract common code into shared/ modules to eliminate duplication:
- shared/url_parser.py: add deduce_repo_url() for git forge URL deduction
- shared/git_runner.py: consolidate subprocess/git execution from agent
  and corrector into a single module

Refactor cve_metadata_extractor:
- Remove duplicated find_hash(), CVE_ID_RE, find_cve_json_file() from
  utils.py (now imported from shared/url_parser)
- Add download_commit_patches() to debian.py
- Update imports across osv, ubuntu, cvelistv5, mirrors modules

Fix cve_corrector/workflow.py:
- Improve conflict detection to check for actual unmerged entries (U
  status) rather than any dirty state
- Preserve current_step if already past cherry_pick_to_devtool

Update AGENTS.md with CI lint instructions.
Update tests to match refactored import locations.

Signed-off-by: Daniel Turull <daniel.turull@ericsson.com>
After agent conflict resolution, the workspace source tree may be left
on the CVE branch. devtool finish requires the devtool branch to
generate patches correctly — otherwise it times out or produces
incorrect output.

Add _ensure_devtool_branch() helper that checks the current branch and
forces checkout to devtool if needed, called at the start of the finish
step.

Signed-off-by: Daniel Turull <daniel.turull@ericsson.com>
Comment thread shared/url_parser.py Dismissed
Comment thread shared/url_parser.py Dismissed
Comment thread shared/url_parser.py Dismissed
…ssing-series

Skip CVEs already present in the output file to avoid redundant API
calls on re-runs. Add --reprocess-missing-series flag to selectively
re-fetch data for CVEs that lack series information in existing output.

Signed-off-by: Daniel Turull <daniel.turull@ericsson.com>
@danielturull danielturull merged commit b45434d into main Jun 10, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants