Handle missing GraphQL aliases as non-blocking repository skips (#68)

* Initial plan

* Handle missing GraphQL aliases as soft skips

Agent-Logs-Url: https://github.com/rpothin/PowerPlatform-OpenSource-Hub/sessions/2104b38a-97ba-4bfc-aa87-bc5bc7cc5b07

Co-authored-by: rpothin <23240245+rpothin@users.noreply.github.com>

* Remove accidental parity report artifact

Agent-Logs-Url: https://github.com/rpothin/PowerPlatform-OpenSource-Hub/sessions/2104b38a-97ba-4bfc-aa87-bc5bc7cc5b07

Co-authored-by: rpothin <23240245+rpothin@users.noreply.github.com>

* Trim missing-repo warning output

Agent-Logs-Url: https://github.com/rpothin/PowerPlatform-OpenSource-Hub/sessions/2104b38a-97ba-4bfc-aa87-bc5bc7cc5b07

Co-authored-by: rpothin <23240245+rpothin@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: rpothin <23240245+rpothin@users.noreply.github.com>

Author: Copilot

.github/workflows/update-github-repositories-details.yml

@@ -260,6 +260,17 @@ jobs:
           if (metricsPath && fs.existsSync(metricsPath)) {
             const metrics = JSON.parse(fs.readFileSync(metricsPath, 'utf8'));
             const failures = metrics.detailFailures ?? 0;
+            const missingRepoSkips = metrics.missingRepoSkips ?? 0;
+            if (missingRepoSkips > 0) {
+              const missingRepoSkipNames = metrics.missingRepoSkipNames ?? [];
+              const displayedNames = missingRepoSkipNames.slice(0, 5).join(', ');
+              const moreCount = Math.max(0, missingRepoSkipNames.length - 5);
+              const affectedRepos =
+                displayedNames.length > 0
+                  ? `${displayedNames}${moreCount > 0 ? `, and ${moreCount} more` : ''}`
+                  : 'names not recorded';
+              console.log(`::warning::TypeScript pipeline skipped ${missingRepoSkips} repo(s) whose GraphQL alias was absent (likely deleted/private). Affected: ${affectedRepos}`);
+            }
             if (failures > 0) {
               blockingFailures.push(`TypeScript pipeline had ${failures} repository detail hydration failure(s); all must succeed in production live mode`);
             }

Pipeline/src/generator.ts

@@ -6,6 +6,7 @@ import { loadSearchCriteria } from "./config.js";
 import { writeGeneratedRepositoryFilesAtomically } from "./generatedFiles.js";
 import { deduplicateByFullName, isRecentlyUpdated, normalizeRepositoryRecord, serializeRecords, sortByPopularity } from "./normalization.js";
 import { validateRecordsWithSchema } from "./schema.js";
+import { isMissingAliasError } from "./types.js";
 import type { CandidateProvider, GenerateResult, PipelineMetrics, RepositoryRecord } from "./types.js";
 
 export interface GenerateOptions {
@@ -99,6 +100,14 @@ export async function generateRepositoryDetails(options: GenerateOptions): Promi
         return null;
       }
       if (result instanceof Error) {
+        if (isMissingAliasError(result)) {
+          metrics.missingRepoSkips += 1;
+          metrics.missingRepoSkipNames.push(repository.fullName);
+          metrics.warnings.push(
+            `Skipping '${repository.fullName}' because it is no longer accessible (GraphQL alias absent): ${result.message}`
+          );
+          return null;
+        }
         if (isPatPolicyError(result)) {
           metrics.patPolicyFailures += 1;
           metrics.patPolicyFailureNames.push(repository.fullName);
@@ -172,6 +181,8 @@ function createMetrics(criteriaCount: number): PipelineMetrics {
     activeRepositories: 0,
     detailRequests: 0,
     detailFailures: 0,
+    missingRepoSkips: 0,
+    missingRepoSkipNames: [],
     patPolicyFailures: 0,
     patPolicyFailureNames: [],
     detailBatchCalls: 0,

Pipeline/src/providers.ts

@@ -303,7 +303,12 @@ export class OctokitRepositoryProvider implements CandidateProvider {
               result.set(repo.fullName, new Error(rootLevelErrors.map((e) => e.message).join("; ")));
             } else if (node === undefined) {
               // Alias absent with no error — unexpected, likely a malformed response.
-              result.set(repo.fullName, new Error(`GraphQL response missing alias '${alias}' for ${repo.fullName}`));
+              result.set(
+                repo.fullName,
+                Object.assign(new Error(`GraphQL response missing alias '${alias}' for ${repo.fullName}`), {
+                  isMissingAlias: true as const
+                })
+              );
             } else {
               // GraphQL returned null with no error. All repos in this batch were found by the
               // search API so they exist. GitHub can silently null-out repos for PAT-policy

Pipeline/src/types.ts

@@ -198,6 +198,14 @@ export interface CandidateProvider {
   batchGetRepositoryDetails?(repos: SearchRepository[]): Promise<Map<string, RepositoryDetails | Error>>;
 }
 
+export type MissingAliasError = Error & {
+  isMissingAlias: true;
+};
+
+export function isMissingAliasError(error: unknown): error is MissingAliasError {
+  return error instanceof Error && (error as Partial<MissingAliasError>).isMissingAlias === true;
+}
+
 export interface PipelineMetrics {
   criteriaCount: number;
   searchRequests: number;
@@ -206,6 +214,8 @@ export interface PipelineMetrics {
   activeRepositories: number;
   detailRequests: number;
   detailFailures: number;
+  missingRepoSkips: number;
+  missingRepoSkipNames: string[];
   patPolicyFailures: number;
   patPolicyFailureNames: string[];
   detailBatchCalls: number;

Pipeline/tests/generator.test.ts

@@ -166,6 +166,8 @@ describe("generateRepositoryDetails", () => {
       activeRepositories: 2,
       detailRequests: 2,
       detailFailures: 0,
+      missingRepoSkips: 0,
+      missingRepoSkipNames: [],
       generatedRecords: 2
     });
 
@@ -329,6 +331,34 @@ describe("generateRepositoryDetails", () => {
     expect(result.metrics.patPolicyFailures).toBe(0);
   });
 
+  it("routes missing GraphQL aliases in batch path to missingRepoSkips", async () => {
+    const configPath = path.join(outputRoot, "criteria-batch-missing-alias.json");
+    const outputPath = path.join(outputRoot, "details-batch-missing-alias.json");
+    await writeFile(configPath, JSON.stringify([{ topic: "powerplatform", searchLimit: 3 }]), "utf8");
+
+    const missingAliasError = Object.assign(new Error("GraphQL response missing alias 'repo1' for owner/missing"), {
+      isMissingAlias: true as const
+    });
+
+    const result = await generateRepositoryDetails({
+      configPath,
+      outputPath,
+      schemaPath,
+      provider: new BatchFakeProvider(
+        { powerplatform: [searchRepository("owner/ok"), searchRepository("owner/missing")] },
+        { "owner/ok": repositoryDetails(20), "owner/missing": missingAliasError }
+      ),
+      now: new Date("2026-01-01T00:00:00Z"),
+      workflowRunId: "test-run"
+    });
+
+    expect(result.records.map((r) => r.fullName)).toEqual(["owner/ok"]);
+    expect(result.metrics.missingRepoSkips).toBe(1);
+    expect(result.metrics.missingRepoSkipNames).toEqual(["owner/missing"]);
+    expect(result.metrics.detailFailures).toBe(0);
+    expect(result.metrics.warnings.join("\n")).toContain("owner/missing");
+  });
+
   it("routes PAT-policy errors in batch path to patPolicyFailures", async () => {
     const configPath = path.join(outputRoot, "criteria-batch-pat.json");
     const outputPath = path.join(outputRoot, "details-batch-pat.json");

Pipeline/tests/providers.test.ts

@@ -202,6 +202,26 @@ describe("OctokitRepositoryProvider.batchGetRepositoryDetails", () => {
     expect((bad as Error).message).toContain("Could not resolve to a Repository");
   });
 
+  it("tags a missing-alias error when GraphQL omits a repo alias with no errors", async () => {
+    const client = makeClient([
+      {
+        data: {
+          rateLimit: RATE_LIMIT_OK,
+          repo0: goodNode()
+        },
+        errors: []
+      }
+    ]);
+
+    const provider = new OctokitRepositoryProvider(client);
+    const results = await provider.batchGetRepositoryDetails([repo("owner/ok"), repo("owner/missing")]);
+
+    const r = results.get("owner/missing");
+    expect(r).toBeInstanceOf(Error);
+    expect((r as Error).message).toContain("GraphQL response missing alias 'repo1' for owner/missing");
+    expect((r as any).isMissingAlias).toBe(true);
+  });
+
   // ---------------------------------------------------------------------------
   // Root-level GraphQL errors (PAT-policy scenario)
   // ---------------------------------------------------------------------------