Skip to content

Conversation

@ronens88
Copy link
Contributor

  • Store branch name in context when provided via --branch flag
  • Include branch in scan_parameters dict sent to backend API
  • Fixes issue where branch_name and branch_id were null in JSON export

This ensures that when scanning a specific branch with 'cycode scan repository --branch ', the branch information is properly sent to the backend API so it can populate branch_name and branch_id fields in the detection details.

@gotbadger
Copy link
Collaborator

@ronens88 can you sign the commit and add some basic test coverage

@ronens88 ronens88 force-pushed the fix/include-branch-in-scan-parameters branch from e446850 to 692c2b8 Compare December 18, 2025 12:42
- Store branch name in context when provided via --branch flag
- Include branch in scan_parameters dict sent to backend API
- Add test coverage for branch parameter in scan_parameters
- Fixes issue where branch_name and branch_id were null in JSON export

This ensures that when scanning a specific branch with 'cycode scan repository --branch <branch>',
the branch information is properly sent to the backend API so it can populate branch_name and
branch_id fields in the detection details.
@ronens88 ronens88 force-pushed the fix/include-branch-in-scan-parameters branch from 692c2b8 to e997197 Compare December 18, 2025 12:48
@ronens88
Copy link
Contributor Author

@ronens88 can you sign the commit and add some basic test coverage

Done

@gotbadger gotbadger merged commit 7749b00 into cycodehq:main Dec 19, 2025
26 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.

3 participants