feat(logging): skill for instrumenting logging#222
Conversation
Skill that enables logging and suggests some initial log messages
b7f7bd3 to
ae27d7a
Compare
ae27d7a to
8808771
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 03142fe. Configure here.
| say so explicitly rather than implying it passed. | ||
|
|
||
| 4. **Apply a high-value log validation check.** Review every added or modified log line | ||
| and remove or revise any log that does not pass this check: |
There was a problem hiding this comment.
Validation after verified status
Medium Severity
Per-app step 3e marks apps verified after tests, but step 4 can remove or rewrite logs afterward with no second verification pass, so inventory status can claim verified while instrumentation no longer matches what was checked.
Reviewed by Cursor Bugbot for commit 03142fe. Configure here.
| # Instrument Sentry Logging | ||
|
|
There was a problem hiding this comment.
Bug: The new skill sentry-instrument-logging/SKILL.md is missing the required breadcrumb navigation link, which violates documented project conventions.
Severity: LOW
Suggested Fix
Add the required breadcrumb link as the first line in the body of skills/sentry-instrument-logging/SKILL.md, after the frontmatter. The line should be: > [All Skills](../../SKILL_TREE.md) > [Feature Setup](../sentry-feature-setup/SKILL.md) > Instrument Sentry Logging.
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-instrument-logging/SKILL.md#L10-L11
Potential issue: The new skill file `skills/sentry-instrument-logging/SKILL.md` is
missing a breadcrumb navigation link, which is a documented requirement in `AGENTS.md`.
This link is present in all other similar skill files and is used for navigation within
the skill hierarchy. While its absence does not cause a crash or functional failure of
the skill itself, it violates the project's structural conventions and may impair
navigation for tools or users relying on this metadata.
2 participants
A pass at a skill that provides recommendations for instrumenting sentry logging.
Tested against:
Implementation note: For repositories containing multiple languages or SDKs, the skill delegates language-specific instrumentation to the corresponding SDK skills.