Remove mentor (#335)

* update mentor structure

* testdao

* add vertx codegen

* remove jvmMain

* './gradlew :protocol:build && ./gradlew :protocol:makeExternalJar' working

* should be val

* './gradlew :protocol:build && ./gradlew :protocol:makeExternalJar && rm -rf protocol/src/jvmMain && git checkout -- protocol/src' is better

* './gradlew :protocol:build && ./gradlew :protocol:makeExternalJar && rm -rf protocol/src/jvmMain && git checkout -- protocol/src' no changes

* temp hack

* switch to jsonMappers

* working on discovery

* include in maven

* service discovery working

* services

* local tracing test

* local tracing test

* uniform

* remove mentor

* local tracing wip

* instant marshalling

* local tracing wip

* local tracing wip

* local tracing wip

* add log count summary

* finish log count indicators
added synchronous event listener

* todos

* merge updates

* moved library

* fix path

* include step

* default false

* more structured services

* remove comments

* auto discover services

Author: BFergerson

README.md

@@ -19,8 +19,6 @@ SourceMarker (alpha) is an APM-powered IDE-based plugin. Inspired by the Feedbac
 - Database access metrics
 - Integrated distributed trace mapping
 - Active exception correlation/tracking
-- Operational Advice
-  - [The Ramp](mentor/src/main/kotlin/com/sourceplusplus/mentor/impl/job/RampDetectionMentor.kt)
 
 ## Demonstration
 
@@ -75,7 +73,6 @@ SourceMarker (alpha) is an APM-powered IDE-based plugin. Inspired by the Feedbac
 | ----------------------------- | -------------------------------------------------------------------- | ------- |
 | :mapper                       | Tracks source code artifact refactoring                              | 0.2.0   |
 | :marker                       | Used to tie visual marks & popups to source code artifacts           | 0.2.0   |
-| :mentor                       | Produces source code artifact informative/cautionary advice          | 0.2.0   |
 | :portal                       | Used to visually display contextualized artifact data/advice         | 0.2.0   |
 | :protocol                     | Common communication data models                                     | 0.2.0   |
 

config/detekt/detekt.yml

@@ -23,3 +23,7 @@ style:
     active: false
   WildcardImport:
     active: false
+
+exceptions:
+  TooGenericExceptionCaught:
+    active: false

gradle.properties

@@ -24,7 +24,6 @@ kotlin.stdlib.default.dependency = true
 
 mapperVersion = 0.2.0
 markerVersion = 0.2.0
-mentorVersion = 0.2.0
 portalVersion = 0.2.0
 protocolVersion = 0.2.0
 

marker/src/main/kotlin/com/sourceplusplus/marker/source/mark/SourceMarkPopupAction.kt

@@ -10,6 +10,8 @@ import com.sourceplusplus.marker.source.SourceFileMarker
 import com.sourceplusplus.marker.source.mark.api.ClassSourceMark
 import com.sourceplusplus.marker.source.mark.api.MethodSourceMark
 import com.sourceplusplus.marker.source.mark.api.SourceMark
+import com.sourceplusplus.marker.source.mark.api.event.SourceMarkEvent
+import com.sourceplusplus.marker.source.mark.api.event.SourceMarkEventCode
 import org.jetbrains.annotations.NotNull
 
 /**
@@ -64,6 +66,8 @@ open class SourceMarkPopupAction : AnAction() {
     }
 
     open fun performPopupAction(sourceMark: SourceMark, editor: Editor) {
-        sourceMark.displayPopup(editor)
+        sourceMark.triggerEvent(SourceMarkEvent(sourceMark, SourceMarkEventCode.PORTAL_OPENING)) {
+            sourceMark.displayPopup(editor)
+        }
     }
 }

marker/src/main/kotlin/com/sourceplusplus/marker/source/mark/api/SourceMark.kt

@@ -26,6 +26,7 @@ import com.sourceplusplus.marker.source.mark.api.config.SourceMarkConfiguration
 import com.sourceplusplus.marker.source.mark.api.event.SourceMarkEvent
 import com.sourceplusplus.marker.source.mark.api.event.SourceMarkEventCode
 import com.sourceplusplus.marker.source.mark.api.event.SourceMarkEventListener
+import com.sourceplusplus.marker.source.mark.api.event.SynchronousSourceMarkEventListener
 import com.sourceplusplus.marker.source.mark.api.key.SourceKey
 import com.sourceplusplus.marker.source.mark.gutter.GutterMark
 import com.sourceplusplus.marker.source.mark.inlay.InlayMark
@@ -35,10 +36,8 @@ import org.slf4j.LoggerFactory
 import java.awt.event.MouseEvent
 import java.awt.event.MouseMotionListener
 import java.util.*
-import java.util.concurrent.Future
 import java.util.concurrent.atomic.AtomicBoolean
 import javax.swing.SwingUtilities
-import kotlin.collections.ArrayList
 import kotlin.concurrent.schedule
 
 /**
@@ -133,8 +132,18 @@ interface SourceMark : JBPopupListener, MouseMotionListener, VisibleAreaListener
     fun getEventListeners(): List<SourceMarkEventListener>
     fun addEventListener(listener: SourceMarkEventListener)
     fun triggerEvent(event: SourceMarkEvent, listen: (() -> Unit)? = null) {
+        //sync listeners
+        getEventListeners()
+            .filterIsInstance<SynchronousSourceMarkEventListener>()
+            .forEach { it.handleEvent(event) }
+
+        //async listeners
         GlobalScope.launch {
-            getEventListeners().forEach { it.handleEvent(event) }
+            getEventListeners().forEach {
+                if (it !is SynchronousSourceMarkEventListener) {
+                    it.handleEvent(event)
+                }
+            }
             listen?.invoke()
         }
     }

marker/src/main/kotlin/com/sourceplusplus/marker/source/mark/api/event/SourceMarkEventCode.kt

@@ -13,8 +13,9 @@ enum class SourceMarkEventCode(private val code: Int) : IEventCode {
     MARK_ADDED(1000),
     MARK_REMOVED(1001),
     NAME_CHANGED(1002),
-    PORTAL_OPENED(1003),
-    PORTAL_CLOSED(1004);
+    PORTAL_OPENING(1003),
+    PORTAL_OPENED(1004),
+    PORTAL_CLOSED(1005);
 
     override fun code(): Int {
         return this.code

marker/src/main/kotlin/com/sourceplusplus/marker/source/mark/api/event/SynchronousSourceMarkEventListener.kt

@@ -0,0 +1,14 @@
+package com.sourceplusplus.marker.source.mark.api.event
+
+import com.sourceplusplus.marker.source.mark.api.SourceMark
+
+/**
+ * Used to listen for events produced by [SourceMark]s.
+ * Calls handleEvent synchronously.
+ *
+ * @since 0.2.1
+ * @author [Brandon Fergerson](mailto:[email protected])
+ */
+fun interface SynchronousSourceMarkEventListener : SourceMarkEventListener {
+    override fun handleEvent(event: SourceMarkEvent)
+}