Skip to content

Include first finding location in exit error output#288

Open
LarytheLord wants to merge 1 commit intoget-woke:mainfrom
LarytheLord:fix/issue-274-first-finding-location
Open

Include first finding location in exit error output#288
LarytheLord wants to merge 1 commit intoget-woke:mainfrom
LarytheLord:fix/issue-274-first-finding-location

Conversation

@LarytheLord
Copy link

@LarytheLord LarytheLord commented Feb 25, 2026

Summary

  • add ParsePathsWithReport to expose summary metadata (files with findings + first finding location)
  • keep existing ParsePaths behavior intact by delegating to the new report method
  • include first finding: file:line:column in the --exit-1-on-failure error message
  • add tests for parser report location and root command error formatting

Why

Closes #274 by making failure output immediately actionable even when users primarily see the terminal error line.

Validation

  • Added/updated unit tests in pkg/parser/parser_test.go and cmd/root_test.go
  • I could not run go test in this environment because the go binary is not installed.

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.

Pinpoint the location of 'findings' when you report there are 'findings'.

1 participant