Adding Copyable AI Rule Snippets to Docs #13752
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Bundle ReportChanges will increase total bundle size by 4.36kB (0.02%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
Files in
App Routes Affected:
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
Files in
|
…ry/sentry-docs into ai-rule-snippets-js-sdks * 'ai-rule-snippets-js-sdks' of https://github.com/getsentry/sentry-docs: Adding AI snippet to logs for nextjs, node, react [getsentry/action-github-commit] Auto commit running eslint fix Fixing import order Fixing ordering of imports Autofix fixing sort Adding typical initialization location for node Adding llm rules snippet for react, node, nextjs Adding support in expandable modal for copy button
nikolovlazar
left a comment
nikolovlazar
left a comment
There was a problem hiding this comment.
LGTM! I think this is a good balance of depth and file simplicity.
|
Hey guys, this broke the styling of our expandable component 😬 Also assuming parts of this was AI generated, we need to make sure to better review these changes – there were styling vars referenced that did not exist as well as some react code that did not make sense. Sorry I didn't get to review this earlier 🙏 EDIT: the expandable was updated two days ago in #13736 and since this PR was not rebased on the latest master, the UI issue was not visible in the deployment preview! 🤯 
|
fixes UI regression introduced in #13752
This fixes some content that was added in #13752: 1. Moves logs down below tracing, as that is IMHO more important (and stable - logs is still experimental 😬 ) (should not affect LLMs too much, but for consistency sake with other things...) 2. Remove the line about adding the `logger` integration, which does not exist 3. Fix the tracing snippets to be better - what we recommended there was actually not what we would want users to do.
Adding snippet copy functionality that users can copy to use in tools like Cursor, Windsurf, or any LLM.
Modified the Expandable component to support a copy property that can used to make snippets copyable in a user friendly way. Screenshot below. Typically this would be a separate PR - but unfortunately the functionality needed to be in place in order for this to work.
Content can be copied and added to a local rules file in the editor and referenced in configuration options - for example...
Per internal discussion, starting with high usage frameworks to start basic and evaluate
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.