Skip to content

feat(skills-next): add android SDK references#236

Open
evanpurkhiser wants to merge 1 commit into
mainfrom
evanpurkhiser/feat-skills-next-add-android-sdk-references
Open

feat(skills-next): add android SDK references#236
evanpurkhiser wants to merge 1 commit into
mainfrom
evanpurkhiser/feat-skills-next-add-android-sdk-references

Conversation

@evanpurkhiser

Copy link
Copy Markdown
Member

Direct LLM port of the existing android SDK skill into the skills-next per-SDK reference layout under skills-next/references/sdks/android/. This content has NOT been reviewed at all — it is a machine-generated port of the existing SDK skill, and every file should be treated as unverified.

@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-skills-next-add-android-sdk-references branch from 2d4d8a0 to c2d4734 Compare July 2, 2026 21:14

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c2d4734. Configure here.

| Events not appearing in Sentry | DSN incorrect or SDK not initialized | Verify DSN; set `options.isDebug = true`; check Logcat for `[Sentry]` output |
| Crash events missing | UncaughtExceptionHandler disabled | Ensure `isEnableUncaughtExceptionHandler = true`; check for competing crash handlers (Firebase, Crashlytics) |
| NDK crashes not reported | NDK module not included or `isEnableNdk = false` | Add `sentry-android-ndk` dependency; confirm `isEnableNdk = true` |
| ANR events missing in release builds | `isAnrReportInDebug = false` suppresses debug builds | Expected; for release builds, verify `isAnrEnabled = true` and that `anrTimeoutIntervalMillis` is reached |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ANR release troubleshooting mismatch

Low Severity

The troubleshooting row titled “ANR events missing in release builds” lists a cause about isAnrReportInDebug suppressing debug builds. That setting does not explain missing ANRs in release; the cause and issue title contradict each other.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit c2d4734. Configure here.

Comment on lines +109 to +111
> ```kotlin
> setDuration(SystemClock.elapsedRealtime() - startMs) / 1000.0) // CORRECT
> setDuration(SystemClock.elapsedRealtime() - startMs) // WRONG — 1000× too large

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 "CORRECT" code example for setDuration() in the documentation has a syntax error due to mismatched parentheses, which will not compile.
Severity: LOW

Suggested Fix

Correct the syntax in the example by moving the division inside the function call and balancing the parentheses. Change setDuration(SystemClock.elapsedRealtime() - startMs) / 1000.0) to setDuration((SystemClock.elapsedRealtime() - startMs) / 1000.0).

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-next/references/sdks/android/crons.md#L109-L111

Potential issue: In the `crons.md` documentation file, a code example for
`setDuration()` is labeled as "CORRECT" but contains a syntax error. The line
`setDuration(SystemClock.elapsedRealtime() - startMs) / 1000.0)` has mismatched
parentheses, which would cause a compilation failure if used. The division by `1000.0`
and an extra closing parenthesis occur outside the `setDuration()` function call. This
is particularly misleading as it's part of a warning block intended to educate the user
(or an AI agent) on the correct usage, but it demonstrates invalid syntax. Other
examples in the same file use the correct syntax, such as
`setDuration((SystemClock.elapsedRealtime() - t0) / 1000.0)`.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant