Skip to content

[skill-drift] fix(multi-sdk): update 4 skills for recent SDK changes#95

Open
github-actions[bot] wants to merge 1 commit intomainfrom
fix/skill-drift-2026-04-09-c96972975b23b23d
Open

[skill-drift] fix(multi-sdk): update 4 skills for recent SDK changes#95
github-actions[bot] wants to merge 1 commit intomainfrom
fix/skill-drift-2026-04-09-c96972975b23b23d

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Apr 9, 2026

SDK Changes

The following PRs were merged in the last 7 days that affect skill files:

Changes Made

skills/sentry-cocoa-sdk/references/tracing.md

  • Added strictTraceContinuation and orgId to the Configuration table
  • Added a "Strict Trace Continuation" subsection under Distributed Tracing with a Swift code example

skills/sentry-php-sdk/references/metrics.md

  • Added a new "Threshold-Based Auto-Flushing" section documenting metric_flush_threshold for PHP/Laravel
  • Added metric_flush_threshold to the Symfony YAML configuration example

skills/sentry-react-native-sdk/SKILL.md

  • Added attachAllThreads (iOS only) to the Native/Mobile Options table
  • Added a new "App Start Accuracy — Sentry.appLoaded()" section explaining when and how to call it

skills/sentry-react-sdk/references/tracing.md

  • Added Sentry.sentryOnError as a simpler alternative to SentryRouteErrorBoundary in the React Router v7 error handling section

Verified Against

  • getsentry/sentry-cocoa@main (PR #7705)
  • getsentry/sentry-php@master (PR #2059)
  • getsentry/sentry-react-native@main (PRs #5960, #5940)
  • getsentry/sentry-javascript@develop (PR #20120)

Note

🔒 Integrity filter blocked 21 items

The following items were blocked because they don't meet the GitHub integrity level.

  • #282 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #20123 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5067 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5951 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5914 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5913 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5902 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5900 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5901 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5897 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5893 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5892 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5891 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5863 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5864 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • #5865 list_pull_requests: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".
  • ... and 5 more items

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

Generated by SDK Skill Drift Detector · ● 1.1M ·

  • expires on Apr 23, 2026, 8:22 AM UTC

…or recent SDK changes

- sentry-cocoa-sdk: add strictTraceContinuation and orgId config options (getsentry/sentry-cocoa#7705)
- sentry-php-sdk: add metric_flush_threshold config option for auto-flushing metrics (getsentry/sentry-php#2059)
- sentry-react-native-sdk: add attachAllThreads iOS option and Sentry.appLoaded() API (getsentry/sentry-react-native#5960, #5940)
- sentry-react-sdk: export sentryOnError for react-router v7 onError handler (getsentry/sentry-javascript#20120)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Comment on lines +459 to +466

const router = Sentry.wrapCreateBrowserRouterV7(createBrowserRouter)([
{
path: "/",
element: <RootLayout />,
onError: Sentry.sentryOnError, // automatically captures route errors
children: [...],
},
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: The documentation example for React Router v7 incorrectly uses the onError property on a route object. The correct property is errorElement, causing silent failure of error reporting.
Severity: HIGH

Suggested Fix

Replace the incorrect onError: Sentry.sentryOnError example with the correct pattern using the errorElement property, such as errorElement: <SentryRouteErrorBoundary />. This will align the new example with the established, correct pattern already present in the documentation and ensure routing errors are captured by Sentry.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: skills/sentry-react-sdk/references/tracing.md#L459-L466

Potential issue: The documentation example for React Router v7 error handling suggests
using `onError: Sentry.sentryOnError` on individual route objects. However, `onError` is
not a valid property for route objects in React Router v7; it is only valid on the
`<RouterProvider>` component. The correct property is `errorElement`. This is
contradicted by another example in the same file which correctly uses `errorElement`.
Code following the new, incorrect example will silently fail to capture routing errors,
leading to a complete loss of error visibility for those routes.

Did we get this right? 👍 / 👎 to inform future reviews.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants