Fetch descriptoin

Author: spotta85

domain/audit.ts

@@ -9,6 +9,7 @@ export interface RequirementRule {
   progressUnit: RequirementProgressUnit;
   status: Status;
   courses: CourseId[];
+  summary?: string;
 }
 
 export interface AuditRequirement {
@@ -56,11 +57,6 @@ export interface AuditHistoryData {
   error?: string;
 }
 
-/**
- * The display name for an audit history entry, title-first: the audit's own
- * title, else its majors joined, else null. Callers supply their own final
- * fallback (an id, "Degree Requirements", etc.).
- */
 export function getAuditDisplayName(
   entry: AuditHistoryEntry | undefined,
 ): string | null {

features/audit-scraping/audit-page-parser.ts

@@ -147,6 +147,21 @@ export function scrapeRequirementSections(
         (cells[5] as HTMLElement).innerText,
       );
 
+      // After the progress columns, GPA rules carry a summary sentence
+      // ("X hours for a total of Y points were used to calculate the GPA.")
+      // while course-based rules carry a course-preview cell. Capture the
+      // former: the first trailing cell that is neither a course preview nor
+      // the show/hide-details link.
+      const summaryCell = Array.from(cells)
+        .slice(6)
+        .find(
+          (cell) =>
+            !cell.classList.contains("course_preview") &&
+            !cell.querySelector("a.details"),
+        );
+      const summary =
+        (summaryCell as HTMLElement | undefined)?.innerText.trim() || undefined;
+
       const rule: RequirementRule = {
         text: (cells[2] as HTMLElement).innerText.trim(),
         requiredHours: requiredProgress.value,
@@ -155,6 +170,7 @@ export function scrapeRequirementSections(
         progressUnit: requiredProgress.unit,
         status: getRuleStatus(row.classList),
         courses: [],
+        ...(summary ? { summary } : {}),
       };
 
       // Parse details row (courses)

features/dashboard/degree-audit-page.tsx

@@ -19,15 +19,12 @@ const SidePanel = () => {
 
   const gpaSection = sections.find((section) => isGpaSection(section.title));
   const gpaRule = gpaSection?.rules[0];
-  const gpaSummary = parseGpaSummary(gpaRule?.text);
+  const gpaSummary = parseGpaSummary(gpaRule?.summary);
 
-  // TODO(debug): remove once real GPA is confirmed. If `counted` logs as a whole
-  // number (e.g. 3) the stored audit was scraped before the parseFloat fix and
-  // needs a re-scrape; the raw scraped text is in gpaRule.text.
   console.log("[GPA card]", {
     required: gpaRule?.requiredHours,
     counted: gpaRule?.appliedHours,
-    ruleText: gpaRule?.text,
+    summaryText: gpaRule?.summary,
     parsedSummary: gpaSummary,
   });
 

tests/scraping/__snapshots__/audit-scraper.test.ts.snap

@@ -674,6 +674,7 @@ OR
           "remainingHours": 0,
           "requiredHours": 2,
           "status": "Completed",
+          "summary": "42 hours for a total of 158.62 points were used to calculate the GPA.",
           "text": "A University GPA of 2.00 is required on all courses undertaken (including credit by examination, correspondence, and extension) for which a grade or symbol other than Q, W, X, or CR is recorded.",
         },
         {
@@ -693,6 +694,7 @@ OR
           "remainingHours": 0,
           "requiredHours": 2,
           "status": "Completed",
+          "summary": "24 hours for a total of 86.62 points were used to calculate the GPA.",
           "text": "Students must earn a grade point average of 2.0 in all mathematics and science courses required by the degree.",
         },
       ],

tests/scraping/audit-scraper.test.ts

@@ -71,4 +71,30 @@ describe("degree audit scraper", () => {
       log.mockRestore();
     }
   });
+
+  test("captures the GPA summary sentence, not course-rule previews", async () => {
+    const document = await loadAuditDocument("audit-results-real.html");
+    const log = spyOn(console, "log").mockImplementation(() => {});
+
+    try {
+      const { requirements } = parseAuditPage(document);
+      const rules = requirements.flatMap((requirement) => requirement.rules);
+
+      const summaries = rules
+        .map((rule) => rule.summary)
+        .filter((summary): summary is string => summary !== undefined);
+
+      // The GPA rules carry their calculation sentence...
+      expect(summaries.length).toBeGreaterThan(0);
+      // ...and every captured summary is that sentence — never a course preview
+      // or other trailing-cell content.
+      for (const summary of summaries) {
+        expect(summary).toMatch(
+          /^\d+(?:\.\d+)? hours for a total of \d+(?:\.\d+)? points/,
+        );
+      }
+    } finally {
+      log.mockRestore();
+    }
+  });
 });