From 7ff3de98a1fb36f8af1baf790ab5c96e71c0fa8f Mon Sep 17 00:00:00 2001
From: Todd Baert <todd.baert@dynatrace.com>
Date: Tue, 14 Jul 2026 08:56:38 -0400
Subject: [PATCH] chore: add API compat checks

Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
---
 api-changes.json                              | 36 +++++++++++++++++++
 pom.xml                                       | 36 +++++++++++++++++++
 .../dev/openfeature/sdk/FlagValueType.java    |  4 +--
 3 files changed, 74 insertions(+), 2 deletions(-)
 create mode 100644 api-changes.json

diff --git a/api-changes.json b/api-changes.json
new file mode 100644
index 000000000..afbf1256f
--- /dev/null
+++ b/api-changes.json
@@ -0,0 +1,36 @@
+[
+  {
+    "extension": "revapi.differences",
+    "id": "long-evaluation-api",
+    "configuration": {
+      "ignore": true,
+      "justification": "Long evaluation is an intentional additive API change. Features is not intended for external implementation. See https://github.com/open-feature/java-sdk/issues/501 and https://github.com/open-feature/java-sdk/pull/1985.",
+      "differences": [
+        {
+          "code": "java.method.addedToInterface",
+          "new": "method dev.openfeature.sdk.FlagEvaluationDetails<java.lang.Long> dev.openfeature.sdk.Features::getLongDetails(java.lang.String, java.lang.Long)"
+        },
+        {
+          "code": "java.method.addedToInterface",
+          "new": "method dev.openfeature.sdk.FlagEvaluationDetails<java.lang.Long> dev.openfeature.sdk.Features::getLongDetails(java.lang.String, java.lang.Long, dev.openfeature.sdk.EvaluationContext)"
+        },
+        {
+          "code": "java.method.addedToInterface",
+          "new": "method dev.openfeature.sdk.FlagEvaluationDetails<java.lang.Long> dev.openfeature.sdk.Features::getLongDetails(java.lang.String, java.lang.Long, dev.openfeature.sdk.EvaluationContext, dev.openfeature.sdk.FlagEvaluationOptions)"
+        },
+        {
+          "code": "java.method.addedToInterface",
+          "new": "method java.lang.Long dev.openfeature.sdk.Features::getLongValue(java.lang.String, java.lang.Long)"
+        },
+        {
+          "code": "java.method.addedToInterface",
+          "new": "method java.lang.Long dev.openfeature.sdk.Features::getLongValue(java.lang.String, java.lang.Long, dev.openfeature.sdk.EvaluationContext)"
+        },
+        {
+          "code": "java.method.addedToInterface",
+          "new": "method java.lang.Long dev.openfeature.sdk.Features::getLongValue(java.lang.String, java.lang.Long, dev.openfeature.sdk.EvaluationContext, dev.openfeature.sdk.FlagEvaluationOptions)"
+        }
+      ]
+    }
+  }
+]
diff --git a/pom.xml b/pom.xml
index d3b9dc18a..4b14d3b01 100644
--- a/pom.xml
+++ b/pom.xml
@@ -463,6 +463,42 @@
                         </configuration>
                     </plugin>
 
+                    <plugin>
+                        <groupId>org.revapi</groupId>
+                        <artifactId>revapi-maven-plugin</artifactId>
+                        <version>0.15.1</version>
+                        <configuration>
+                            <oldVersion>LATEST</oldVersion>
+                            <versionFormat>\d+\.\d+\.\d+</versionFormat>
+                            <failOnUnresolvedArtifacts>true</failOnUnresolvedArtifacts>
+                            <failOnUnresolvedDependencies>true</failOnUnresolvedDependencies>
+                            <failCriticality>error</failCriticality>
+                            <outputNonIdentifyingDifferenceInfo>false</outputNonIdentifyingDifferenceInfo>
+                            <buildFailureMessage>
+                                Fix the compatibility issue or add an exact, justified exception to api-changes.json.
+                            </buildFailureMessage>
+                            <analysisConfigurationFiles>
+                                <file>${project.basedir}/api-changes.json</file>
+                            </analysisConfigurationFiles>
+                        </configuration>
+                        <dependencies>
+                            <dependency>
+                                <groupId>org.revapi</groupId>
+                                <artifactId>revapi-java</artifactId>
+                                <version>0.28.4</version>
+                            </dependency>
+                        </dependencies>
+                        <executions>
+                            <execution>
+                                <id>api-compatibility</id>
+                                <phase>verify</phase>
+                                <goals>
+                                    <goal>check</goal>
+                                </goals>
+                            </execution>
+                        </executions>
+                    </plugin>
+
                     <plugin>
                         <groupId>org.jacoco</groupId>
                         <artifactId>jacoco-maven-plugin</artifactId>
diff --git a/src/main/java/dev/openfeature/sdk/FlagValueType.java b/src/main/java/dev/openfeature/sdk/FlagValueType.java
index 03883e493..8999befe6 100644
--- a/src/main/java/dev/openfeature/sdk/FlagValueType.java
+++ b/src/main/java/dev/openfeature/sdk/FlagValueType.java
@@ -4,8 +4,8 @@
 public enum FlagValueType {
     STRING,
     INTEGER,
-    LONG,
     DOUBLE,
     OBJECT,
-    BOOLEAN;
+    BOOLEAN,
+    LONG;
 }