Shorten repository project labels

packages/client-runtime/src/state/projectGrouping.test.ts

@@ -0,0 +1,38 @@
+import { describe, expect, it } from "@effect/vitest";
+
+import { deriveProjectGroupLabel } from "./projectGrouping.ts";
+
+const repositoryIdentity = (owner: string, name: string) => ({
+  canonicalKey: `github.com/${owner}/${name}`,
+  displayName: `${owner}/${name}`,
+  name,
+  owner,
+  locator: {
+    source: "git-remote" as const,
+    remoteName: "origin",
+    remoteUrl: `git@github.com:${owner}/${name}.git`,
+  },
+});
+
+describe("deriveProjectGroupLabel", () => {
+  it("prefers the short repository name over the owner-qualified display name", () => {
+    const project = {
+      title: "Custom title",
+      repositoryIdentity: repositoryIdentity("pingdotgg", "t3code"),
+    };
+
+    expect(deriveProjectGroupLabel({ representative: project, members: [project] })).toBe("t3code");
+  });
+
+  it("falls back to the display name when a repository name is unavailable", () => {
+    const { name: _name, ...identityWithoutName } = repositoryIdentity("macs-holding", "internal");
+    const project = {
+      title: "Custom title",
+      repositoryIdentity: identityWithoutName,
+    };
+
+    expect(deriveProjectGroupLabel({ representative: project, members: [project] })).toBe(
+      "macs-holding/internal",
+    );
+  });
+});

packages/client-runtime/src/state/projectGrouping.ts

@@ -165,19 +165,19 @@ export function deriveProjectGroupLabel(input: {
   readonly representative: Pick<EnvironmentProject, "title" | "repositoryIdentity">;
   readonly members: ReadonlyArray<Pick<EnvironmentProject, "title" | "repositoryIdentity">>;
 }): string {
-  const sharedDisplayNames = uniqueNonEmptyValues(
-    input.members.map((member) => member.repositoryIdentity?.displayName),
-  );
-  if (sharedDisplayNames.length === 1) {
-    return sharedDisplayNames[0]!;
-  }
-
   const sharedRepositoryNames = uniqueNonEmptyValues(
     input.members.map((member) => member.repositoryIdentity?.name),
   );
   if (sharedRepositoryNames.length === 1) {
     return sharedRepositoryNames[0]!;
   }
 
+  const sharedDisplayNames = uniqueNonEmptyValues(
+    input.members.map((member) => member.repositoryIdentity?.displayName),
+  );
+  if (sharedDisplayNames.length === 1) {
+    return sharedDisplayNames[0]!;
+  }
+
   return input.representative.title;
 }