feat(nx-dev): get the correct query for ai feedback

[mandarini]

Current Behavior

We have a list of messages, and each one has an index, which is generated by the loop that renders the messages. The messages are Queries (by the user) and Responses (by the assistant).

The 👍 / 👎 feedback buttons are "attached" to the Response message. So, whenever a user clicks on the feedback button, we know that they clicked on the message with index X. Using that index X we assume that the Query that resulted in that Response is the message with index X-1. Since this logic is based on an assumption that the messages list will always be consistent and "correct" and sorted, we think that it is brittle.

Expected Behavior

Now that we use the useChat from ai/react package, we get back a list of messages, and each message has its own id. So, when we get a Response, we tie the id of the Response with the Query that gave that Response. That way, we can be sure that when a user clicks the feedback button, their feedback will be for the correct Query.

@bcabanes

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 1, 2023 1:59pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3f496a3. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx affected --target=e2e --base=1931390bd07e4a8cc6992c1fcad0bcb346cef48e --head=3f496a3537b309ad60470f178f36a78b1acff735 --parallel=1
• nx affected --target=lint --base=1931390bd07e4a8cc6992c1fcad0bcb346cef48e --head=3f496a3537b309ad60470f178f36a78b1acff735 --parallel=3
• nx affected --target=build --base=1931390bd07e4a8cc6992c1fcad0bcb346cef48e --head=3f496a3537b309ad60470f178f36a78b1acff735 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx affected --target=test --base=1931390bd07e4a8cc6992c1fcad0bcb346cef48e --head=3f496a3537b309ad60470f178f36a78b1acff735 --parallel=1
• nx-cloud record -- nx format:check --base=1931390bd07e4a8cc6992c1fcad0bcb346cef48e --head=3f496a3537b309ad60470f178f36a78b1acff735
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=1931390bd07e4a8cc6992c1fcad0bcb346cef48e --head=3f496a3537b309ad60470f178f36a78b1acff735

---

Sent with 💌 from NxCloud.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.