feat(data): LP_INCIDENTS and LP_STORE_RISK demo data (#126)#150
Merged
Conversation
Adds 8 realistic LP incidents covering all 4 LP detection rules and their named signals (HighVolumeVoidRefund, HighValueVoidNoOverride, BulkGiftCardActivation, RapidCrossChannelRedemption, RepeatHigh- DiscountRelationship, HighDiscountConcentrationAtTerminal, Ghost- EmployeeAfterHours, ClosedStoreTerminalBurst). Adds LP_STORE_RISK with 10 store risk scores for the store risk scoring component. Also registers 4 LP rules in DETECTION_RULES and updates Impact tactic coverage from 2 to 4 in MITRE_COVERAGE. Closes #126
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
TFT444
commented
Jun 23, 2026
TFT444
left a comment
TFT444
left a comment
Owner
Author
There was a problem hiding this comment.
diff looks clean and met the requirement.ready to merge
TFT444
added a commit
that referenced
this pull request
Jun 23, 2026
…x Vercel build Vercel builds this branch in isolation — importing LP_INCIDENTS and LP_STORE_RISK from LossPrevention.jsx failed at bundle time because those exports were only present on the feature/lp-demo-data branch (PR #150), not here. Adding them directly to data.js on this branch so the preview deployment can build successfully.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
LP_INCIDENTS(8 incidents) covering all 4 LP rules and all 8 named detection signalsLP_STORE_RISK(10 stores) with risk scores, open incident counts, and trend direction — feeds the store risk scoring component (Loss Prevention — Store risk scoring dashboard component #125)DETECTION_RULES(LP-001 through LP-004)MITRE_COVERAGE— Impact tacticcoveredfrom 2 → 4LP_INCIDENTS coverage
How to test
Open
frontend/src/lib/data.jsand verify:LP_INCIDENTSis exported with 8 entries, all havingtechnique: 'T1657'LP_STORE_RISKis exported with 10 entries (8 stores with incidents, 2 clean)DETECTION_RULESincludes LP-001 through LP-004MITRE_COVERAGEImpact tactic showscovered: 4Closes #126