Skip to content

Conversation

@laileni-aws
Copy link
Contributor

Problem

cwsprChatUserIntent is returning undefined for amazonq_interactWithMessage event. This event is emitted for insert_code_at_cursor_position, code_was_copied_to_clipboard etc.

Solution

  • Storing the mapping of messageId: UserIntent and sending this if user clicked on insert at cursor or copy to clipboard events.

License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions
Copy link

github-actions bot commented Oct 8, 2024

This pull request modifies code in src/ but no tests were added/updated. Confirm whether tests should be added or ensure the PR description explains why tests are not required.

@aws aws deleted a comment from github-actions bot Oct 8, 2024
@laileni-aws laileni-aws marked this pull request as ready for review October 8, 2024 20:52
@laileni-aws laileni-aws requested a review from a team as a code owner October 8, 2024 20:52
@laileni-aws laileni-aws changed the title feat(amazonq): Adding userIntent for insertAtCursor and copyToClipboard telemetry events. feat(amazonq): Adding userIntent for insertAtCursor and copyToClipboard events for telemetry. Oct 8, 2024
@github-actions
Copy link

github-actions bot commented Oct 8, 2024

This pull request implements a feature or fix, so it must include a changelog entry. See CONTRIBUTING.md#changelog for instructions.

eventId,
codeBlockIndex,
totalCodeBlocks,
messageUserIntentMap.get(messageId) ?? undefined
Copy link
Contributor

Choose a reason for hiding this comment

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

nice

@laileni-aws laileni-aws merged commit 2172bf9 into aws:master Oct 9, 2024
23 of 24 checks passed
@justinmk3
Copy link
Contributor

If this is a telemetry-only change, use the telemetry type. The types are listed here. Please avoid naming non-feature PRs feat and then ignoring the comment that requests a changelog...

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.

3 participants