chore: remove reporting of attachments to sentry [IDE-1046] (#675)

* chore: remove reporting of attachments to sentry

IntelliJ marked the API used as internal with 2025.1. This means we cannot report it anymore.

In the future, we would use analytics service anyways for error reporting, so they turn up both in datadog and snowflake

* fix: Add INTERNAL_API_USAGES to plugin verifier failure levels

The plugin verifier task was missing `INTERNAL_API_USAGES` in its failure level configuration. This change adds it to ensure that internal API usages are also considered as failures during verification.

This should lead to us catching compatibility issues with internal api changes in newer IDE versions.

Relates to: #None

Author: bastiandoetsch

CHANGELOG.md

@@ -1,8 +1,34 @@
 # Snyk Security Changelog
+## [2.12.1]
+### Changed
+- remove usage of internal IntelliJ API to allow plugin release
 
-## [2.11.2]
+## [2.12.0]
 ### Changed
- - Using Code Lens or Code Actions to view an AI Fix now shows the affected issue in the tree view and details pane of the Snyk window.
+- chore: bump ls protocol version to 18 by @ShawkyZ in https://github.com/snyk/snyk-intellij-plugin/pull/646
+- chore: update IntelliJ plugin to support 2025.1, fix workspaceFolder config by @bastiandoetsch in https://github.com/snyk/snyk-intellij-plugin/pull/647
+- fix: force-save settings on receiving auth info by @bastiandoetsch in https://github.com/snyk/snyk-intellij-plugin/pull/650
+- feat: scan summary panel [IDE-894] by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/648
+- feat: add job to compare hashes of static resource files by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/649
+- feat: add reference folder to branch chooser by @bastiandoetsch in https://github.com/snyk/snyk-intellij-plugin/pull/652
+- fix: always set reference and base branch folder by @ShawkyZ in https://github.com/snyk/snyk-intellij-plugin/pull/653
+- fix: folderConfig type [IDE-871] by @bastiandoetsch in https://github.com/snyk/snyk-intellij-plugin/pull/655
+- chore: remove unnecessary scripts which were migrated to language server [IDE-655] by @DariusZdroba in https://github.com/snyk/snyk-intellij-plugin/pull/651
+- fix: check for disposal before updating the issue cache by @bastiandoetsch in https://github.com/snyk/snyk-intellij-plugin/pull/658
+- feat/ai fix issue panel [IDE-956] [IDE-760] by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/656
+- fix: only include folders in their parent projects when calculating content roots [IDE-1012] by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/659
+- refactor: remove unused code now that the IDE no longer sends AI fix … by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/662
+- fix: only update tree view on UI thread by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/661
+- fix: add border to delta sumamry toggle to improve visibility by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/660
+- fix: use URI to determine file path by @bastiandoetsch in https://github.com/snyk/snyk-intellij-plugin/pull/664
+- fix: update cache on receive diags by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/663
+- fix: tree node race selection on windows by @ShawkyZ in https://github.com/snyk/snyk-intellij-plugin/pull/667
+- fix: scale web fonts using 10px as a reference size.  by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/666
+- fix: round severity icons updated [IDE-897] by @acke in https://github.com/snyk/snyk-intellij-plugin/pull/669
+- fix: update code and warning colours for ai fix and ignores by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/670
+- fix/icon updates [IDE-987] by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/671
+- fix: icon updates [IDE-897] by @andrewrobinsonhodges-snyk in https://github.com/snyk/snyk-intellij-plugin/pull/672
+- fix: create CLI download directory if it non existent by @DariusZdroba in https://github.com/snyk/snyk-intellij-plugin/pull/673
 
 ## [2.11.1]
 ### Changed

build.gradle.kts

@@ -189,7 +189,8 @@ tasks {
         failureLevel.set(
             listOf(
                 org.jetbrains.intellij.tasks.RunPluginVerifierTask.FailureLevel.COMPATIBILITY_PROBLEMS,
-                org.jetbrains.intellij.tasks.RunPluginVerifierTask.FailureLevel.INVALID_PLUGIN
+                org.jetbrains.intellij.tasks.RunPluginVerifierTask.FailureLevel.INVALID_PLUGIN,
+                org.jetbrains.intellij.tasks.RunPluginVerifierTask.FailureLevel.INTERNAL_API_USAGES,
             )
         )
     }

src/main/kotlin/snyk/errorHandler/SnykErrorReportSubmitter.kt

@@ -1,8 +1,6 @@
 package snyk.errorHandler
 
-import com.intellij.diagnostic.AbstractMessage
 import com.intellij.diagnostic.PluginException
-import com.intellij.openapi.diagnostic.Attachment
 import com.intellij.openapi.diagnostic.ErrorReportSubmitter
 import com.intellij.openapi.diagnostic.IdeaLoggingEvent
 import com.intellij.openapi.diagnostic.SubmittedReportInfo
@@ -23,21 +21,16 @@ class SnykErrorReportSubmitter : ErrorReportSubmitter() {
         for (event in events) {
             var throwable = event.throwable
             val description = additionalInfo ?: ""
-            var attachments = listOf<Attachment>()
 
-            if (event.data is AbstractMessage) {
-                throwable = (event.data as AbstractMessage).throwable
-                attachments = (event.data as AbstractMessage).includedAttachments
-            }
             if (throwable is PluginException && throwable.cause != null) {
                 // unwrap PluginManagerCore.createPluginException
                 throwable = throwable.cause
             }
+
             SentryErrorReporter.submitErrorReport(
                 throwable,
                 consumer = consumer,
                 description = description,
-                attachments = attachments
             )
         }
         return true