Skip to content

track prompt#2609

Open
tusharmath wants to merge 2 commits intomainfrom
track-prompt
Open

track prompt#2609
tusharmath wants to merge 2 commits intomainfrom
track-prompt

Conversation

@tusharmath
Copy link
Collaborator

  • feat(ui): track prompt content before creating chat event
  • feat(ui): track prompt input before starting spinner

@augmentcode
Copy link

augmentcode bot commented Mar 20, 2026

🤖 Augment PR Summary

Summary: Adds prompt telemetry tracking for the initial CLI-provided prompt/piped input before message handling starts.
Why: Ensures prompt content is captured earlier (before chat event creation and before starting the spinner), matching the interactive prompt flow.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

// Handle direct prompt or piped input if provided (raw text messages)
let input = self.cli.prompt.clone().or(self.cli.piped_input.clone());
if let Some(input) = input {
tracker::prompt(input.clone());
Copy link

Choose a reason for hiding this comment

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

tracker::prompt forwards the full input string into telemetry (EventKind::Prompt); since this path can include cli.piped_input, it may unintentionally capture very large payloads or sensitive/secrets-bearing content.

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

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