Skip to content

[0.1.7] Throttle gem-not-installed balloon to once per 15 min - #61

Merged
unurgunite merged 1 commit into
v0.1.7from
fix/balloon-throttle-gem-missing
Sep 10, 2026
Merged

unurgunite merged 1 commit into
v0.1.7from
fix/balloon-throttle-gem-missing

Conversation

@unurgunite

@unurgunite unurgunite commented Sep 10, 2026

Copy link
Copy Markdown
Member

Description

Throttles the "Docscribe gem not found" balloon so it no longer spams on every opened file.

showGemNotInstalledBalloon fired from doAnnotate on each .rb file when the gem is missing. Now the first show goes through immediately and repeats are suppressed per project directory until 15 minutes elapse (BALLOON_THROTTLE_MS). Other failures keep flowing to idea.log unchanged.

Changes

  • annotator/DocscribeAnnotator.ktlastGemBalloonShown map (keyed by projectDir) + pure shouldShowGemBalloon(lastShownMs, nowMs) gate checked before notify.
  • annotator/DocscribeAnnotatorBalloonThrottleTest.kt — 5 tests: interval constant, first show, suppressed within window, allowed at boundary and after window.

Checklist

  • Tests pass (./gradlew test)
  • Formatting clean (./gradlew spotlessCheck)
  • Static analysis clean (./gradlew detekt)
  • Plugin verified (./gradlew verifyPlugin)
  • CHANGELOG updated (if applicable)

@unurgunite unurgunite self-assigned this Sep 10, 2026
@unurgunite unurgunite added the enhancement New feature or request label Sep 10, 2026
@unurgunite
unurgunite force-pushed the fix/balloon-throttle-gem-missing branch from 4b2a939 to be9c012 Compare September 10, 2026 07:26
@unurgunite
unurgunite force-pushed the fix/balloon-throttle-gem-missing branch from be9c012 to b1484f6 Compare September 10, 2026 07:28
@unurgunite
unurgunite merged commit a931276 into v0.1.7 Sep 10, 2026
1 check passed
@unurgunite
unurgunite deleted the fix/balloon-throttle-gem-missing branch September 10, 2026 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant