Skip to content

refactor(diff): simplify security diff scans - #351

Merged
ianw-oai merged 1 commit into
mainfrom
dev/ianw/compact-security-diff-scans
Aug 11, 2026
Merged

refactor(diff): simplify security diff scans#351
ianw-oai merged 1 commit into
mainfrom
dev/ianw/compact-security-diff-scans

Conversation

@ianw-oai

@ianw-oai ianw-oai commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Build diff inventories from the selected revisions or working tree.
  • Reuse compact scan operations and remove duplicate ranking and review steps.
  • Keep deleted files, finding identities, scan limits, and existing path protections.

Tests

  • 1,011 SDK tests passed with 11 expected skips.
  • TypeScript, formatting, build, and installed-package checks passed.

faizan-oai
faizan-oai approved these changes Aug 11, 2026
View reviewed changes

@faizan-oai faizan-oai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have thoroughly reviewed this and looks great

@ianw-oai
ianw-oai merged commit f996db1 into main Aug 11, 2026
18 checks passed
@ianw-oai
ianw-oai deleted the dev/ianw/compact-security-diff-scans branch August 11, 2026 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zcrab-oai zcrab-oai zcrab-oai approved these changes
@faizan-oai faizan-oai faizan-oai approved these changes
@kmbroai kmbroai kmbroai approved these changes

Assignees

@kmbroai kmbroai

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ianw-oai @zcrab-oai @faizan-oai @kmbroai