有料導線とレビュー訴求が重なるのを、ダイアログではなくゲートで止めています

// MainActivity.kt（移す前）
rewardedAdLoader.load { ad ->
    if (ad != null && !billing.isAdFree()) {
        // ロードが返ってきた「この瞬間」の状態しか見ていない
        RewardedIntersDialog(this, ad).show()
    }
}

// ModalGate.kt
package net.dolice.wallpapers.modal
 
import androidx.lifecycle.Lifecycle
import androidx.lifecycle.LifecycleOwner
import java.util.concurrent.atomic.AtomicReference
 
enum class ModalKind(val perSessionLimit: Int) {
    PAYWALL(1),
    REWARDED_INTERSTITIAL(2),
    REVIEW_INDUCTION(1),
}
 
sealed interface GateResult {
    data object Granted : GateResult
    data class Denied(val reason: String) : GateResult
}
 
object ModalGate {
    private const val COOL_DOWN_MS = 1_500L
 
    private val current = AtomicReference<ModalKind?>(null)
    private val shownCount = mutableMapOf<ModalKind, Int>()
    private var lastDismissedAtMs = 0L
 
    /**
     * 表示の可否を、表示する直前に問い直す。
     * show は許可された場合にのみ呼ばれる。
     */
    @Synchronized
    fun request(kind: ModalKind, owner: LifecycleOwner, show: () -> Unit): GateResult {
        // 罠2 への対処: 非同期処理から戻ってきた時点で、画面がまだ生きているかを毎回確かめる
        if (!owner.lifecycle.currentState.isAtLeast(Lifecycle.State.STARTED)) {
            return GateResult.Denied("owner is not started")
        }
        val holder = current.get()
        if (holder != null) {
            // 優先度が高くても割り込ませない（理由は後述）
            return GateResult.Denied("occupied by $holder")
        }
        if (System.currentTimeMillis() - lastDismissedAtMs < COOL_DOWN_MS) {
            // 罠3 への対処: 閉じた直後に次が出ると、閉じる操作が次のボタンに吸われる
            return GateResult.Denied("cool down")
        }
        val shown = shownCount[kind] ?: 0
        if (shown >= kind.perSessionLimit) {
            return GateResult.Denied("per-session limit reached for $kind")
        }
        current.set(kind)
        shownCount[kind] = shown + 1
        show()
        return GateResult.Granted
    }
 
    /** ダイアログが閉じたら必ず呼ぶ。呼び忘れるとゲートが開かないので、テストで担保する。 */
    @Synchronized
    fun release(kind: ModalKind) {
        if (current.get() == kind) {
            current.set(null)
            lastDismissedAtMs = System.currentTimeMillis()
        }
    }
 
    /** プロセス再生成をまたいで数え続けないよう、セッション開始時に初期化する。 */
    @Synchronized
    fun resetSession() {
        current.set(null)
        shownCount.clear()
        lastDismissedAtMs = 0L
    }
}

// MainActivity.kt（移した後）
rewardedAdLoader.load { ad ->
    if (ad == null) return@load
    val result = ModalGate.request(ModalKind.REWARDED_INTERSTITIAL, this) {
        RewardedIntersDialog(this, ad) {
            ModalGate.release(ModalKind.REWARDED_INTERSTITIAL)
        }.show()
    }
    if (result is GateResult.Denied) {
        // 出さなかった理由が残るので、後から配信ログと突き合わせられる
        Log.i(TAG, "rewarded modal skipped: ${result.reason}")
    }
}

#!/usr/bin/env bash
# tools/check_modal_gate.sh
# ModalGate を経由しないダイアログ表示を検出する。CI から呼ぶ。
set -euo pipefail
 
SRC_DIR="${1:-app/src/main/java}"
VIOLATIONS=0
 
# 対象は「◯◯Dialog.show()」の形だけに絞る。汎用の show() まで拾うと誤検出が増える。
while IFS= read -r hit; do
  file="${hit%%:*}"
  [ "$(basename "$file")" = "ModalGate.kt" ] && continue
  # 同じファイル内で request を通していれば、ゲート経由とみなす
  grep -q 'ModalGate\.request' "$file" && continue
  echo "NG $hit"
  VIOLATIONS=$((VIOLATIONS + 1))
done < <(grep -rnE '\b[A-Za-z]+Dialog\([^)]*\)\.show\(\)|\b[A-Za-z]+Dialog\.show\(' \
           "$SRC_DIR" --include='*.kt' || true)
 
if [ "$VIOLATIONS" -gt 0 ]; then
  echo "ModalGate を経由しない表示が ${VIOLATIONS} 件あります"
  exit 1
fi
echo "OK: ModalGate を経由しない表示はありません"