fix app uri printing bug

sdk was ignoring the port for url printing, which broke links to
braintrust hosted on nonstandard ports

Author: realark

src/main/java/dev/braintrust/config/BraintrustConfig.java

@@ -96,14 +96,17 @@ URI fetchProjectURI(BraintrustApiClient client) {
         try {
             var orgAndProject = client.getProjectAndOrgInfo().orElseThrow();
             var baseURI = new URI(appUrl());
+            var path =
+                    "/app/%s/p/%s"
+                            .formatted(
+                                    orgAndProject.orgInfo().name(), orgAndProject.project().name());
             return new URI(
                     baseURI.getScheme(),
+                    baseURI.getUserInfo(),
                     baseURI.getHost(),
-                    baseURI.getPath()
-                            + "/app/"
-                            + orgAndProject.orgInfo().name()
-                            + "/p/"
-                            + orgAndProject.project().name(),
+                    baseURI.getPort(),
+                    path,
+                    null,
                     null);
         } catch (URISyntaxException e) {
             throw new RuntimeException(e);

src/test/java/dev/braintrust/config/BraintrustConfigTest.java

@@ -46,14 +46,27 @@ void projectUriFetching() {
                 new dev.braintrust.api.BraintrustApiClient.OrganizationAndProjectInfo(
                         orgInfo, project);
         var client = new dev.braintrust.api.BraintrustApiClient.InMemoryImpl(orgAndProjectInfo);
-        var config =
-                BraintrustConfig.of(
-                        "BRAINTRUST_API_KEY", "foobar",
-                        "BRAINTRUST_API_URL", "https://api.braintrust.dev",
-                        "BRAINTRUST_APP_URL", "https://braintrust.dev",
-                        "BRAINTRUST_DEFAULT_PROJECT_ID", projectId);
-        assertEquals(
-                "https://braintrust.dev/app/Test%20Org/p/some%20project",
-                config.fetchProjectURI(client).toASCIIString());
+        { // standard braintrust.dev
+            var config =
+                    BraintrustConfig.of(
+                            "BRAINTRUST_API_KEY", "foobar",
+                            "BRAINTRUST_API_URL", "https://api.braintrust.dev",
+                            "BRAINTRUST_APP_URL", "https://braintrust.dev",
+                            "BRAINTRUST_DEFAULT_PROJECT_ID", projectId);
+            assertEquals(
+                    "https://braintrust.dev/app/Test%20Org/p/some%20project",
+                    config.fetchProjectURI(client).toASCIIString());
+        }
+        { // self-hosted with a non-standard port
+            var config =
+                    BraintrustConfig.of(
+                            "BRAINTRUST_API_KEY", "foobar",
+                            "BRAINTRUST_API_URL", "http://someselfhost:8080",
+                            "BRAINTRUST_APP_URL", "http://someselfhost:3001",
+                            "BRAINTRUST_DEFAULT_PROJECT_ID", projectId);
+            assertEquals(
+                    "http://someselfhost:3001/app/Test%20Org/p/some%20project",
+                    config.fetchProjectURI(client).toASCIIString());
+        }
     }
 }