#131 get and include filing agency in case summaries; capture but don't show addresses

Author: jayhill

frontend/src/components/app/SearchResult.tsx

@@ -83,6 +83,13 @@ const SearchResult: React.FC<SearchResultProps> = ({ searchResult: sr }) => {
                                             }
                                             return null;
                                         })()}
+
+                                    {/* Filing agency: shown at top-level if the case summary has a single filing agency for all charges */}
+                                    {summary.filingAgency && (
+                                        <p className="mt-1 text-sm text-gray-600">
+                                            <span className="font-medium">Filing Agency:</span> {summary.filingAgency}
+                                        </p>
+                                    )}
                                 </div>
 
                                 {summary.charges && summary.charges.length > 0 && (
@@ -118,6 +125,13 @@ const SearchResult: React.FC<SearchResultProps> = ({ searchResult: sr }) => {
                                                                 <span className="font-medium">Fine:</span> ${charge.fine.toFixed(2)}
                                                             </div>
                                                         )}
+
+                                                        {/* Per-charge filing agency: only shown when no top-level filing agency is present */}
+                                                        {!summary.filingAgency && charge.filingAgency && (
+                                                            <div>
+                                                                <span className="font-medium">Filing Agency:</span> {charge.filingAgency}
+                                                            </div>
+                                                        )}
                                                     </div>
                                                     {charge.dispositions && charge.dispositions.length > 0 && (
                                                         <div className="mt-2 text-xs text-gray-600">

frontend/src/components/app/__tests__/SearchResult.test.tsx

@@ -50,8 +50,11 @@ const createTestCase = (override = {}): SearchResultType => ({
                         description: 'Guilty',
                     },
                 ],
+                filingAgency: null,
+                filingAgencyAddress: [],
             },
         ],
+        filingAgency: null,
     },
     ...override,
 });
@@ -218,4 +221,77 @@ describe('SearchResult component', () => {
         // Should still render the charge description
         expect(screen.getByText('Weird Charge')).toBeInTheDocument();
     });
+
+    it('displays top-level filing agency when present', () => {
+        const testCase = createTestCase({
+            caseSummary: {
+                caseName: 'State vs. Doe',
+                court: 'Circuit Court',
+                filingAgency: 'Metro PD',
+                charges: [
+                    {
+                        offenseDate: '2022-01-01',
+                        filedDate: '2022-01-02',
+                        description: 'Theft',
+                        statute: '123.456',
+                        degree: { code: 'M', description: 'Misdemeanor' },
+                        fine: 0,
+                        dispositions: [],
+                        filingAgency: 'Metro PD',
+                    },
+                ],
+            },
+        });
+
+        render(<SearchResult searchResult={testCase} />);
+
+        // Top-level Filing Agency should be present
+        expect(screen.getByText(/Filing Agency:/)).toBeInTheDocument();
+        expect(screen.getByText('Metro PD')).toBeInTheDocument();
+
+        // Per-charge filing agency should not be duplicated when top-level present
+        const chargeAgency = screen.queryAllByText(/Filing Agency:/).length;
+        expect(chargeAgency).toBe(1); // only the top-level label
+    });
+
+    it('displays per-charge filing agencies when they differ and no top-level is set', () => {
+        const testCase = createTestCase({
+            caseSummary: {
+                caseName: 'State vs. Doe',
+                court: 'Circuit Court',
+                charges: [
+                    {
+                        offenseDate: '2022-01-01',
+                        filedDate: '2022-01-02',
+                        description: 'Charge A',
+                        statute: '111',
+                        degree: { code: 'M', description: 'M' },
+                        fine: 0,
+                        dispositions: [],
+                        filingAgency: 'Dept A',
+                    },
+                    {
+                        offenseDate: '2022-02-01',
+                        filedDate: '2022-02-02',
+                        description: 'Charge B',
+                        statute: '222',
+                        degree: { code: 'M', description: 'M' },
+                        fine: 0,
+                        dispositions: [],
+                        filingAgency: 'Dept B',
+                    },
+                ],
+            },
+        });
+
+        render(<SearchResult searchResult={testCase} />);
+
+        // No single top-level filing agency — expect per-charge Filing Agency labels for each charge
+        const filingLabels = screen.queryAllByText(/Filing Agency:/);
+        expect(filingLabels.length).toBeGreaterThanOrEqual(2);
+
+        // Both per-charge filing agencies should be present
+        expect(screen.getByText('Dept A')).toBeInTheDocument();
+        expect(screen.getByText('Dept B')).toBeInTheDocument();
+    });
 });

serverless/lib/CaseProcessor.ts

@@ -639,6 +639,7 @@ function buildCaseSummary(rawData: Record<string, PortalApiResponse>): CaseSumma
             caseName: rawData['summary']['CaseSummaryHeader']['Style'] || '',
             court: rawData['summary']['CaseSummaryHeader']['Heading'] || '',
             charges: [],
+            filingAgency: null,
         };
 
         const chargeMap = new Map<number, Charge>();
@@ -663,8 +664,21 @@ function buildCaseSummary(rawData: Record<string, PortalApiResponse>): CaseSumma
                 },
                 fine: typeof chargeOffense['FineAmount'] === 'number' ? chargeOffense['FineAmount'] : 0,
                 dispositions: [],
+                filingAgency: null,
+                filingAgencyAddress: [],
             };
 
+            const filingAgencyRaw = chargeData['FilingAgencyDescription'];
+            if (filingAgencyRaw) {
+                charge.filingAgency = String(filingAgencyRaw).trim();
+            }
+
+            // Extract filing agency address if present. It will be an array of strings.
+            const filingAgencyAddressRaw = chargeData['FilingAgencyAddress'];
+            if (filingAgencyAddressRaw) {
+                charge.filingAgencyAddress.push(...(filingAgencyAddressRaw as any));
+            }
+
             // Add to charges array
             caseSummary.charges.push(charge);
 
@@ -674,6 +688,22 @@ function buildCaseSummary(rawData: Record<string, PortalApiResponse>): CaseSumma
             }
         });
 
+        // After processing charges, derive top-level filing agency if appropriate
+        try {
+            const definedAgencies = caseSummary.charges.map(ch => ch.filingAgency).filter((a): a is string => a !== null && a.length > 0);
+
+            const uniqueAgencies = Array.from(new Set(definedAgencies));
+
+            // If there's at least one defined agency, and all defined agencies are identical,
+            // set it on the case summary. Charges that lack an agency (null) are ignored for this decision.
+            if (uniqueAgencies.length === 1 && uniqueAgencies[0]) {
+                caseSummary.filingAgency = uniqueAgencies[0];
+                console.log(`🔔 Set Filing Agency to ${caseSummary.filingAgency}`);
+            }
+        } catch (faErr) {
+            console.error('Error computing top-level filing agency:', faErr);
+        }
+
         // Process dispositions and link them to charges
         const dispositionEvents = rawData['dispositionEvents']['Events'] || [];
         console.log(`📋 Found ${dispositionEvents.length} disposition events`);

serverless/lib/__tests__/CaseSearchProcessor.test.ts

@@ -40,6 +40,7 @@ describe('CaseSearchProcessor', () => {
                 caseName: 'Test vs State',
                 court: 'Test Court',
                 charges: [],
+                filingAgency: null,
             };
 
             const completeCase: SearchResult = {
@@ -284,6 +285,7 @@ describe('CaseSearchProcessor', () => {
                 caseName: 'Test vs State',
                 court: 'Test Court',
                 charges: [],
+                filingAgency: null,
             };
 
             mockStorageClient.getSearchResults.mockResolvedValue({