Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(nx-dev): improve initial prompt of the AI Chat to remove bad responses #19244

Merged
merged 1 commit into from Sep 20, 2023

Conversation

jaysoo
Copy link
Member

@jaysoo jaysoo commented Sep 19, 2023

This PR addresses a few issues:

  • AI making up bad links, which was due to us instructive it to always provide a link <-- This is a huge problem, and I think I've fixed all the occurrences of bad links
  • Contradictory information, such as claiming Nx<->Node compat versions, showing a table, then claiming different compat versions. This is fixed by asking it not to contradict itself.
  • AI answers with a few paragraphs, followed by Sorry, I don't know how to help you with that. This is mitigated by putting the ask first.
  • Image links are broken since they are copied from docs to /public/documentation of the Next.js app, updated logic to fix this.

Preview: https://nx-dev-git-fork-jaysoo-ai-chat-prompt-tweaks-nrwl.vercel.app/ai-chat

@vercel
Copy link

vercel bot commented Sep 19, 2023

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 20, 2023 5:16pm

@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants