From a9b83ce196004774aa44265c6d788f6f6ff5e137 Mon Sep 17 00:00:00 2001
From: skyflow-bharti <bharti.sagar@skyflow.com>
Date: Thu, 15 Oct 2026 19:41:10 +0530
Subject: [PATCH] SK-3222 fix internal release

---
 .github/workflows/internal-release.yml |  2 +-
 flowvault/pom.xml                      | 31 ++++++++++--------------
 skyvault/pom.xml                       | 33 ++++++++++----------------
 3 files changed, 26 insertions(+), 40 deletions(-)

diff --git a/.github/workflows/internal-release.yml b/.github/workflows/internal-release.yml
index 808cc7e4..5b13107b 100644
--- a/.github/workflows/internal-release.yml
+++ b/.github/workflows/internal-release.yml
@@ -46,7 +46,7 @@ jobs:
     uses: ./.github/workflows/shared-build-and-deploy.yml
     with:
       ref: ${{ github.ref_name }}
-      server-id: central
+      server-id: skyflow-jfrog
       profile: jfrog
       tag: 'internal'
       module: ${{ needs.resolve-module.outputs.module }}
diff --git a/flowvault/pom.xml b/flowvault/pom.xml
index 6ea99c9a..3389f94d 100644
--- a/flowvault/pom.xml
+++ b/flowvault/pom.xml
@@ -76,25 +76,18 @@
                             </artifactSet>
                         </configuration>
                     </execution>
-                </executions>
-            </plugin>
-            <plugin>
-                <!-- Comparison-only jar merging com.skyflow:common, mirroring skyvault. This
-                     exists so japicmp compares the whole surface a consumer actually gets, since
-                     classes a customer imports (Credentials, SkyflowException, BearerToken) live
-                     in common and would otherwise read as missing - japicmp only sees what is
-                     physically inside the jars it is given.
-                     Written with outputFile, NOT shadedArtifactAttached: attaching it made it a
-                     secondary artifact, and every attached artifact is deployed, so this
-                     build-internal jar was being published to Maven Central as a permanent
-                     -with-common classifier. outputFile writes the jar and neither replaces the
-                     main artifact nor attaches it. The path is unchanged, so the japicmp
-                     newVersion path below still resolves. -->
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-shade-plugin</artifactId>
-                <version>3.6.0</version>
-                <executions>
                     <execution>
+                    <!-- Comparison-only jar merging com.skyflow:common, mirroring skyvault. This
+                         exists so japicmp compares the whole surface a consumer actually gets, since
+                         classes a customer imports (Credentials, SkyflowException, BearerToken) live
+                         in common and would otherwise read as missing - japicmp only sees what is
+                         physically inside the jars it is given.
+                         Written with outputFile, NOT shadedArtifactAttached: attaching it made it a
+                         secondary artifact, and every attached artifact is deployed, so this
+                         build-internal jar was being published to Maven Central as a permanent
+                         -with-common classifier. outputFile writes the jar and neither replaces the
+                         main artifact nor attaches it. The path is unchanged, so the japicmp
+                         newVersion path below still resolves. -->
                         <id>shade-for-japicmp</id>
                         <phase>package</phase>
                         <goals>
@@ -180,7 +173,7 @@
             <id>jfrog</id>
             <distributionManagement>
                 <repository>
-                    <id>central</id>
+                    <id>skyflow-jfrog</id>
                     <name>prekarilabs.jfrog.io-releases</name>
                     <url>https://prekarilabs.jfrog.io/artifactory/skyflow-java</url>
                 </repository>
diff --git a/skyvault/pom.xml b/skyvault/pom.xml
index ce14cc82..47c04de5 100644
--- a/skyvault/pom.xml
+++ b/skyvault/pom.xml
@@ -57,26 +57,19 @@
                             </artifactSet>
                         </configuration>
                     </execution>
-                </executions>
-            </plugin>
-            <plugin>
-                <!-- Produces an additional, comparison-only jar that merges in
-                     com.skyflow:common, same pattern flowvault already uses. It exists only so
-                     japicmp can compare an apples-to-apples "everything a v2 consumer gets" surface
-                     against the old 2.1.0 baseline, which was a single self-contained jar
-                     (predating the common/v2 split). Without this, classes that moved into
-                     common (Credentials, ErrorCode, BearerToken, Token, etc.) would show up as
-                     false-positive "removed" classes, since japicmp only compares what's
-                     physically inside the two jars it's given.
-                     Written with outputFile rather than shadedArtifactAttached: attaching it made
-                     it a secondary artifact, and attached artifacts are deployed, so a jar
-                     documented here as comparison-only was being published. outputFile keeps it
-                     on disk without attaching. Path unchanged, so japicmp still resolves it. -->
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-shade-plugin</artifactId>
-                <version>3.6.0</version>
-                <executions>
                     <execution>
+                    <!-- Produces an additional, comparison-only jar that merges in
+                         com.skyflow:common, same pattern flowvault already uses. It exists only so
+                         japicmp can compare an apples-to-apples "everything a v2 consumer gets" surface
+                         against the old 2.1.0 baseline, which was a single self-contained jar
+                         (predating the common/v2 split). Without this, classes that moved into
+                         common (Credentials, ErrorCode, BearerToken, Token, etc.) would show up as
+                         false-positive "removed" classes, since japicmp only compares what's
+                         physically inside the two jars it's given.
+                         Written with outputFile rather than shadedArtifactAttached: attaching it made
+                         it a secondary artifact, and attached artifacts are deployed, so a jar
+                         documented here as comparison-only was being published. outputFile keeps it
+                         on disk without attaching. Path unchanged, so japicmp still resolves it. -->
                         <id>shade-for-japicmp</id>
                         <phase>package</phase>
                         <goals>
@@ -170,7 +163,7 @@
             <id>jfrog</id>
             <distributionManagement>
                 <repository>
-                    <id>central</id>
+                    <id>skyflow-jfrog</id>
                     <name>prekarilabs.jfrog.io-releases</name>
                     <url>https://prekarilabs.jfrog.io/artifactory/skyflow-java</url>
                 </repository>