Change Micronaut context attribute key to avoid collision with prefab-sdk (#16)

* Change Micronaut context attribute key to avoid collision with prefab-sdk

Update the HTTP request attribute name from "prefab-contexts" to "reforge-sdk-contexts"
to prevent conflicts when using this SDK concurrently with the previous prefab-sdk.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Bumps version to 1.0.2

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: jkebinger

micronaut/pom.xml

@@ -4,7 +4,7 @@
   <parent>
     <groupId>com.reforge</groupId>
     <artifactId>sdk-parent</artifactId>
-    <version>1.0.1</version>
+    <version>1.0.2</version>
   </parent>
 
   <artifactId>sdk-micronaut-extension</artifactId>

micronaut/src/main/java/com/reforge/sdk/extensions/micronaut/ServerRequestContextStore.java

@@ -15,11 +15,11 @@
  */
 public class ServerRequestContextStore implements ContextStore {
 
-  public static final String ATTRIBUTE_NAME = "prefab-contexts";
+  public static final String ATTRIBUTE_NAME = "reforge-sdk-contexts";
 
   @Override
   public void addContext(Context context) {
-    getPrefabContextSet()
+    getReforgeContextSet()
       .ifPresentOrElse(
         prefabContextSet -> prefabContextSet.addContext(context),
         () -> setContext(context)
@@ -50,10 +50,10 @@ public Optional<ContextSetReadable> clearContext() {
 
   @Override
   public Optional<ContextSetReadable> getContext() {
-    return getPrefabContextSet().map(ContextSetReadable::readOnlyContextSetView);
+    return getReforgeContextSet().map(ContextSetReadable::readOnlyContextSetView);
   }
 
-  private Optional<ContextSet> getPrefabContextSet() {
+  private Optional<ContextSet> getReforgeContextSet() {
     return ServerRequestContext
       .currentRequest()
       .flatMap(objectHttpRequest ->

micronaut/src/test/java/com/reforge/sdk/extensions/micronaut/ServerRequestContextStoreTest.java

@@ -13,7 +13,7 @@
 
 class ServerRequestContextStoreTest {
 
-  ServerRequestContextStore prefabStateStore = new ServerRequestContextStore();
+  ServerRequestContextStore reforgeSdkContextStore = new ServerRequestContextStore();
   Context userContext = Context.newBuilder("user").put("country", "us").build();
   Context serverContext = Context.newBuilder("server").put("az", "2").build();
 
@@ -29,25 +29,25 @@ void beforeEach() {
 
     @Test
     void getContextReturnsEmpty() {
-      assertThat(prefabStateStore.getContext()).isEmpty();
+      assertThat(reforgeSdkContextStore.getContext()).isEmpty();
     }
 
     @Test
     void setContextQuietlyDoesNothing() {
-      assertThat(prefabStateStore.setContext(userAndServerContextSet)).isEmpty();
-      assertThat(prefabStateStore.getContext()).isEmpty();
+      assertThat(reforgeSdkContextStore.setContext(userAndServerContextSet)).isEmpty();
+      assertThat(reforgeSdkContextStore.getContext()).isEmpty();
     }
 
     @Test
     void addContextQuietlyDoesNothing() {
-      prefabStateStore.addContext(userContext);
-      assertThat(prefabStateStore.getContext()).isEmpty();
+      reforgeSdkContextStore.addContext(userContext);
+      assertThat(reforgeSdkContextStore.getContext()).isEmpty();
     }
 
     @Test
     void clearContextQuietlyDoesNothing() {
-      assertThat(prefabStateStore.clearContext()).isEmpty();
-      assertThat(prefabStateStore.getContext()).isEmpty();
+      assertThat(reforgeSdkContextStore.clearContext()).isEmpty();
+      assertThat(reforgeSdkContextStore.getContext()).isEmpty();
     }
   }
 
@@ -63,18 +63,18 @@ void beforeEach() {
 
     @Test
     void getContextReturnsEmptyWhenNoContextSet() {
-      assertThat(prefabStateStore.getContext()).isEmpty();
+      assertThat(reforgeSdkContextStore.getContext()).isEmpty();
     }
 
     @Test
     void setContextReturnsEmptyWhenNoContextSet() {
-      assertThat(prefabStateStore.setContext(userAndServerContextSet)).isEmpty();
+      assertThat(reforgeSdkContextStore.setContext(userAndServerContextSet)).isEmpty();
     }
 
     @Test
     void addContextWhenEmptyUpdatesTheContext() {
-      prefabStateStore.addContext(userContext);
-      assertThat(prefabStateStore.getContext())
+      reforgeSdkContextStore.addContext(userContext);
+      assertThat(reforgeSdkContextStore.getContext())
         .isPresent()
         .get()
         .usingRecursiveComparison()
@@ -83,8 +83,8 @@ void addContextWhenEmptyUpdatesTheContext() {
 
     @Test
     void clearReturnsEmpty() {
-      assertThat(prefabStateStore.clearContext()).isEmpty();
-      assertThat(prefabStateStore.getContext()).isEmpty();
+      assertThat(reforgeSdkContextStore.clearContext()).isEmpty();
+      assertThat(reforgeSdkContextStore.getContext()).isEmpty();
     }
 
     @Nested
@@ -98,12 +98,12 @@ class WithPreExistingContext {
 
       @BeforeEach
       void beforeEach() {
-        prefabStateStore.setContext(userAndServerContextSet);
+        reforgeSdkContextStore.setContext(userAndServerContextSet);
       }
 
       @Test
       void getReturnsExpectedSet() {
-        assertThat(prefabStateStore.getContext())
+        assertThat(reforgeSdkContextStore.getContext())
           .isPresent()
           .get()
           .usingRecursiveComparison()
@@ -112,22 +112,22 @@ void getReturnsExpectedSet() {
 
       @Test
       void clearWorksAsExpected() {
-        assertThat(prefabStateStore.clearContext())
+        assertThat(reforgeSdkContextStore.clearContext())
           .isPresent()
           .get()
           .usingRecursiveComparison()
           .isEqualTo(userAndServerContextSet);
-        assertThat(prefabStateStore.getContext()).isEmpty();
+        assertThat(reforgeSdkContextStore.getContext()).isEmpty();
       }
 
       @Test
       void setWorksAsExpected() {
-        assertThat(prefabStateStore.setContext(userContext))
+        assertThat(reforgeSdkContextStore.setContext(userContext))
           .isPresent()
           .get()
           .usingRecursiveComparison()
           .isEqualTo(userAndServerContextSet);
-        assertThat(prefabStateStore.getContext())
+        assertThat(reforgeSdkContextStore.getContext())
           .isPresent()
           .get()
           .usingRecursiveComparison()
@@ -136,8 +136,8 @@ void setWorksAsExpected() {
 
       @Test
       void addWorksAsExpected() {
-        prefabStateStore.addContext(newUserContext);
-        assertThat(prefabStateStore.getContext())
+        reforgeSdkContextStore.addContext(newUserContext);
+        assertThat(reforgeSdkContextStore.getContext())
           .isPresent()
           .get()
           .usingRecursiveComparison()

pom.xml

@@ -11,7 +11,7 @@
   <groupId>com.reforge</groupId>
   <artifactId>sdk-parent</artifactId>
 
-  <version>1.0.1</version>
+  <version>1.0.2</version>
   <packaging>pom</packaging>
   <name>Reforge SDK Parent POM</name>
   <description>Parent POM for Reforge SDK modules providing feature flags, configuration management, and A/B testing capabilities</description>

sdk/pom.xml

@@ -4,7 +4,7 @@
   <parent>
     <groupId>com.reforge</groupId>
     <artifactId>sdk-parent</artifactId>
-    <version>1.0.1</version>
+    <version>1.0.2</version>
   </parent>
 
   <artifactId>sdk</artifactId>