Skip to content

Prepare finding decision schema compatibility - #498

Merged
morgaesis merged 12 commits into
mainfrom
finding-decisions-schema
Aug 9, 2026
Merged

Prepare finding decision schema compatibility#498
morgaesis merged 12 commits into
mainfrom
finding-decisions-schema

Conversation

@morgaesis

Copy link
Copy Markdown
Contributor

Finding decisions need a durable audit shape and stable repository identity before interactive dismissal can roll out safely across mixed application versions. This adds the compatible schema, normalizes legacy review jobs under an atomic database lock, and makes queue deduplication use the immutable GitHub repository ID while preserving old-writer compatibility.

@postil-dev postil-dev Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

info 2 advisory findings open
2 findings posted inline

Review details

Comment thread drizzle/0048_woozy_tigra.sql
Comment thread src/lib/queue.ts

@postil-dev postil-dev Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

warn review incomplete · info 2 advisory findings open

Review details

@postil-dev postil-dev Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

info 3 advisory findings open
2 findings posted inline

Review details

Comment thread src/lib/queue.ts
Comment thread src/lib/queue.ts
@morgaesis
morgaesis merged commit 4a166c5 into main Aug 9, 2026
5 checks passed
@morgaesis
morgaesis deleted the finding-decisions-schema branch August 9, 2026 20:23
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.

1 participant