Skip to content

feat: adopt The Professor persona with Groq streaming and Radio Comms UI#178

Merged
GauravKarakoti merged 1 commit into
GauravKarakoti:mainfrom
Yuva-Deekshitha-N:feat/professor-persona-groq-streaming
Jul 10, 2026
Merged

feat: adopt The Professor persona with Groq streaming and Radio Comms UI#178
GauravKarakoti merged 1 commit into
GauravKarakoti:mainfrom
Yuva-Deekshitha-N:feat/professor-persona-groq-streaming

Conversation

@Yuva-Deekshitha-N

Copy link
Copy Markdown
Contributor

Description:

Adopts "The Professor" persona for the AI inference layer. Updates system and user prompts to use cold, precise radio-comm style framing where every fix is an "adjustment to the plan." Implements Groq streaming (stream: true) for ultra-fast, low-latency responses. Renames the UI chat interface from "AI Security Explanation" to "Radio Comms." Also adds prompt injection sanitization on user-controlled inputs before LLM interpolation.

Type of Change:

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Performance improvement

Screenshots (if applicable):

radio comms

Testing:

  • Opened a test PR on a linked repository to trigger the webhook scan
  • Verified Radio Comms label appears in the findings dashboard
  • Confirmed The Professor's tone in AI-generated explanations and remediation steps
  • Verified Groq streaming assembles responses chunk-by-chunk via for await
  • Confirmed prompt injection sanitization strips malicious patterns from code snippets

Related Issues:

Closes #138

Checklist:

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have added tests that prove my fix/feature works
  • My commits follow the Conventional Commits format

Additional Notes:

  • sanitizeForPrompt() neutralizes code fences, strips injection phrases, and enforces a 2000 char limit on all user-controlled fields (description, fileLocation, codeSnippet) before LLM interpolation
  • Groq model used: llama-3.1-8b-instant

@vercel

vercel Bot commented Jul 10, 2026

Copy link
Copy Markdown

@Yuva-Deekshitha-N is attempting to deploy a commit to the Gaurav's projects Team on Vercel.

A member of the Team first needs to authorize it.

@secure-flow

secure-flow Bot commented Jul 10, 2026

Copy link
Copy Markdown

🛡️ SecureFlow AI Security Report

⚠️ Detected 1 potential issues matching your code policies. Please review them before merging.

🔴 CRITICAL | Vulnerability in src/ai/flows/developer-receives-ai-security-explanations.ts

The attacker logs environment variables containing sensitive information through the 'console.log' function in src/ai/flows/developer-receives-ai-security-explanations.ts, compromising confidentiality.

🛠️ View Remediation Suggestions

["Remove 'console.log(process.env.GROQ_API_KEY || 'dummy-key-for-build');' entirely to prevent leakage.","Replace process.env.GROQ_API_KEY with a secure alternative, like a config file or environment variable encryption."]


@GauravKarakoti GauravKarakoti merged commit 2f14389 into GauravKarakoti:main Jul 10, 2026
5 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rebrand AI Security Explanations to "The Professor" Persona

2 participants