address PR feedback: drop unused isLoaded, revert .claude jest ignore

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: adhorodyski

jest.config.js

@@ -22,7 +22,7 @@ module.exports = {
     // .worktrees/ holds parallel git worktrees a developer may check out locally.
     // Each one carries its own modules/hybrid-app/package.json, which trips
     // jest-haste-map's "duplicate package name" assertion. Skip them entirely.
-    modulePathIgnorePatterns: ['<rootDir>/.worktrees/', '<rootDir>/.claude/'],
+    modulePathIgnorePatterns: ['<rootDir>/.worktrees/'],
     globals: {
         __DEV__: true,
         WebSocket: {},

src/hooks/useTransactionsAndViolationsForReport.ts

@@ -23,7 +23,7 @@ function useTransactionsAndViolationsForReport(reportID?: string) {
         filteredViolations[transactionViolationKey] = getTransactionViolations(transaction, violations, currentUserDetails.email ?? '', currentUserDetails.accountID, report, policy) ?? [];
     }
 
-    return {transactions, violations: filteredViolations, isLoaded: allReportsTransactionsAndViolations !== undefined};
+    return {transactions, violations: filteredViolations};
 }
 
 export default useTransactionsAndViolationsForReport;

tests/ui/ReportActionsViewTest.tsx

@@ -182,7 +182,6 @@ describe('ReportActionsView', () => {
         mockUseTransactionsAndViolationsForReport.mockReturnValue({
             transactions: {},
             violations: {},
-            isLoaded: true,
         });
 
         mockUsePaginatedReportActions.mockReturnValue({