Skip to content

feat(sdk): persist security finding publication associations - #452

Open
kmbroai wants to merge 4 commits into
dev/kyleb/linear-publish-enginefrom
dev/kyleb/linear-publish-persistence
Open

feat(sdk): persist security finding publication associations#452
kmbroai wants to merge 4 commits into
dev/kyleb/linear-publish-enginefrom
dev/kyleb/linear-publish-persistence

Conversation

@kmbroai

@kmbroai kmbroai commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

Persist security-finding publication associations in existing local scan history without adding issue-creation or CLI behavior.

Changes

  • Add an append-only workbench migration for finding-to-external-issue associations.
  • Validate the completed scan, original scan directory, finding IDs, occurrence IDs, and destination before accepting publication records.
  • Add file-backed Python workbench preflight and transactional import commands with current-run database readback.
  • Add the TypeScript publication-store adapter and package allowlist entry.
  • Preserve existing scan history, support idempotent record replay, and keep independently created issues distinct.

Testing

  • Real SQLite publication-store, existing CLI workbench, and saved-scan-history tests: 27 passed with 147 assertions.
  • Validated missing databases, mismatched findings, cross-scan occurrences, invalid destinations, transaction rollback, and repeated publication.

Risk and rollout

  • The migration is additive and retains existing scan and finding records.
  • This layer does not contact Linear, start Codex, or expose publication commands.
  • Stacked on the connected Linear publication engine.

Public disclosure review

  • No customer, partner, prospect, or user identities, data, or identifying details are included.
  • No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
  • I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

@kmbroai
kmbroai marked this pull request as ready for review August 15, 2026 19:30

@soyeon-oai soyeon-oai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed the exact PR delta. I found one sealed-scan integrity issue in the publication preflight.

Comment thread sdk/typescript/_bundled_plugin/scripts/workbench_db.py
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