Skip to content

Record hydro security sweep inspection state#1217

Merged
brendancol merged 2 commits intomasterfrom
security-sweep-hydro-state
Apr 20, 2026
Merged

Record hydro security sweep inspection state#1217
brendancol merged 2 commits intomasterfrom
security-sweep-hydro-state

Conversation

@brendancol
Copy link
Copy Markdown
Contributor

Summary

Records the security sweep inspection result for the hydro subpackage in .claude/sweep-security-state.json.

No CRITICAL or HIGH severity issues were identified. MEDIUM and LOW findings are documented in the audit report but deferred per the sweep policy (only CRITICAL/HIGH trigger fixes).

Test plan

  • State file parses as valid JSON
  • Entry for hydro includes last_inspected, severity_max, categories_found, issue fields

No CRITICAL or HIGH severity issues found in hydro module.
MEDIUM findings: unbounded queue allocation (h*w*8 bytes in BFS
kernels), division by cellsize without zero-check, some secondary
inputs not validated with _validate_raster.
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label Apr 19, 2026
@brendancol
Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Co-authored-by: brendancol <433221+brendancol@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

@copilot resolve the merge conflicts in this pull request

Resolved in commit 3e85cf0. The conflict in .claude/sweep-security-state.json was an add/add conflict — both branches added the file independently. The merged version now includes all three entries: reproject and geotiff from master, plus hydro from this branch.

@brendancol brendancol merged commit fc44984 into master Apr 20, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance PR touches performance-sensitive code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants