Skip to content

FEAT create feedback dialogue#1956

Draft
hannahwestra25 wants to merge 6 commits into
microsoft:mainfrom
hannahwestra25:hawestra/gui-feedback-link
Draft

FEAT create feedback dialogue#1956
hannahwestra25 wants to merge 6 commits into
microsoft:mainfrom
hannahwestra25:hawestra/gui-feedback-link

Conversation

@hannahwestra25

Copy link
Copy Markdown
Contributor

Description

Add in-app feedback dialog to the Co-PyRIT GUI

Adds a "Send feedback" dialog so users can file feedback without leaving the
app. The dialog routes them to a pre-filled GitHub issue under their own
account — no backend, no telemetry, no credentials shipped.

What it does

  • New <FeedbackDialog> with a category dropdown: Bug / Feature / Doc /
    Praise / Other. Picking a category swaps the form fields to mirror the
    matching .github/ISSUE_TEMPLATE/*.md.
  • On submit, opens a new tab at github.com/microsoft/PyRIT/issues/new?...
    with title, body, and labels pre-filled. The user reviews and submits the
    issue themselves on github.com.
  • All issues are tagged GUI plus the category's existing label
    (bug, enhancement, documentation, praise).
  • Live client-side secret detection (18 regex rules covering OpenAI / Azure /
    AWS / GCP / HuggingFace / DB connection strings / private keys) warns the
    user before they submit and adds a confirm gate if anything matches. Match
    substrings are never logged — only the rule name.
  • New praise.md issue template + triage-feedback.yml workflow that
    auto-acknowledges every GUI feedback issue and, for praise, asks GitHub
    Models to verify it's actually praise (with a non-LLM keyword veto on
    problem words) before auto-closing.

Tests and Documentation

Added tests for corresponding components

api.key.mp4
feature.request.mp4
praise.mp4

@hannahwestra25 hannahwestra25 force-pushed the hawestra/gui-feedback-link branch from 3e9045f to 2775191 Compare June 8, 2026 20:45
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