Skip to content

[GB-MONITOR] Fix Mac Build After #347 (Sentry Exception Type Is Optional) - #348

Merged
jaywedgeworth22 merged 2 commits into
mainfrom
gb-monitor/codecaps-sentry-keypath-fix
Nov 2, 2026
Merged

jaywedgeworth22 merged 2 commits into
mainfrom
gb-monitor/codecaps-sentry-keypath-fix

Conversation

@jaywedgeworth22

@jaywedgeworth22 jaywedgeworth22 commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

Fixes Mac build broken by #347.

SentryTelemetry.beforeSend filters third-party failed requests using Sentry exception types, but exception.type is optional. The previous map(\.type) produces optional values and fails the Mac build.

Updated to compactMap exception types so nil types are ignored, restoring a successful build with no change in filtering behavior for valid types.

@jaywedgeworth22

Copy link
Copy Markdown
Member Author

@kody start-review

@kody-ai

kody-ai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

⚠️ 1 Kody Rule(s) were not evaluated

These rules declare that they need repository context beyond this diff, and the review could not retrieve it, so they were not judged on this pull request:

  • Never log tokens or credentials; only safeIssues may reach the handoff file
Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug ✅
Performance ✅
Security ✅
Business Logic ✅

Access your configuration settings here.

​

Comment thread Sources/CodeCaps/SentryTelemetry.swift Resolved
@jaywedgeworth22
jaywedgeworth22 enabled auto-merge (squash) November 2, 2026 03:20
@jaywedgeworth22
jaywedgeworth22 merged commit 2141f76 into main Nov 2, 2026
4 checks passed
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