Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

Conversation

@aaronherman
Copy link
Contributor

This pull request includes adding functionality to prompt assignees to update security events and adds a "visibility" field for case updates in Slack.

Workflow Enhancements:

  • Added send_event_update_prompt_reminder to case_new_create_flow in src/dispatch/case/flows.py and in src/dispatch/case/messaging.py to remind assignees to update security event details during triage, which sends an ephemeral Slack message to the assignee with a button to update case details.

Slack Integration Updates:

  • Added a new escalate_case command to ConversationCommands in src/dispatch/conversation/enums.py and mapped it to Slack configurations. [1] [2]
  • Added update_case_button_click handler in src/dispatch/plugins/dispatch_slack/case/interactive.py to process the "Update Case" button click and open the update modal.

Case Visibility Field:

  • Added case_visibility_select functionality in src/dispatch/plugins/dispatch_slack/fields.py to allow users to choose "Restricted" or "Open" visibility for cases.
  • Updated handle_edit_submission_event in src/dispatch/plugins/dispatch_slack/case/interactive.py to process the visibility field and save changes.
image

@aaronherman aaronherman requested review from Copilot and whitdog47 June 3, 2025 20:22
@aaronherman aaronherman added the enhancement New feature or request label Jun 3, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a reminder prompt for case assignees to complete security event triage in Slack, incorporates a new “visibility” field into case update modals, and wires up an escalate-case slash command.

  • Map escalate_case command to Slack slash commands
  • Introduce case_visibility_select field and handle it in the update modal/submission
  • Add send_event_update_prompt_reminder and invoke it in the new case flow

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/dispatch/plugins/dispatch_slack/plugin.py Register escalate_case in Slack command mappings
src/dispatch/plugins/dispatch_slack/fields.py Define case_visibility_select enum and builder function
src/dispatch/plugins/dispatch_slack/case/interactive.py Add “Update Case” button handler, inject visibility into modal, and process it on submit
src/dispatch/plugins/dispatch_slack/case/enums.py Add new CaseNotificationActions.update
src/dispatch/conversation/enums.py Add new ConversationCommands.escalate_case
src/dispatch/case/messaging.py Implement send_event_update_prompt_reminder
src/dispatch/case/flows.py Invoke the new reminder step in case_new_create_flow
Comments suppressed due to low confidence (1)

src/dispatch/case/messaging.py:381

  • Consider adding unit tests or an integration test for send_event_update_prompt_reminder to ensure ephemeral messages are sent under expected conditions.
def send_event_update_prompt_reminder(case: Case, db_session: Session) -> None:

aaronherman and others added 5 commits June 3, 2025 15:25
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Aaron Herman <aaron@aaronherman.co>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Aaron Herman <aaron@aaronherman.co>
@aaronherman aaronherman merged commit bfc3892 into main Jun 4, 2025
9 checks passed
@aaronherman aaronherman deleted the feat/prompt-for-triage branch June 4, 2025 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants