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

fix: CPX-1259 Strip Markdown code block from response #67

Merged
merged 1 commit into from
Jan 22, 2025
Merged

Conversation

BC-MLombardi
Copy link
Contributor

What/Why?

Since the AI model update, responses have been getting returned inside a html Markdown code block. This PR strips these from the response.

I also tried to use the json mode of the model, but the responses were shorter and lower quality so I went with this approach instead.

Rollout/Rollback

Revert.

Testing

Before:

before.mov

After:

after.mov

@bigcommerce/team-data

Copy link

vercel bot commented Jan 17, 2025

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

Name Status Preview Comments Updated (UTC)
ai-app-foundation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 17, 2025 6:43pm

@BC-MLombardi BC-MLombardi marked this pull request as ready for review January 17, 2025 18:46
@BC-MLombardi BC-MLombardi requested a review from a team as a code owner January 17, 2025 18:46
@BC-MLombardi BC-MLombardi requested a review from a team January 17, 2025 18:46
@BC-MLombardi BC-MLombardi merged commit 9b6805f into main Jan 22, 2025
3 checks passed
@BC-MLombardi BC-MLombardi deleted the CPX-1259 branch January 22, 2025 19:12
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.

2 participants