Fix 20 TypeScript syntax errors across 9 files

[Copilot]

Resolves structural syntax errors preventing TypeScript compilation: incomplete try/catch blocks, malformed IIFE closures, and missing braces.

Changes

Incomplete try/catch blocks (3 files)

• drafting-mode-executor.ts: Added missing try keyword before catch block
• potemkin-clio.ts: Added missing try keyword before catch block
• document-drafter.ts: Added missing closing brace for else block

Malformed IIFE closures (4 files)

• compliance-checker.ts: Fixed }(); → })();
• fact-checker.ts: Fixed (); → })();
• legal-email-drafter.ts: Fixed (); → })(); and added missing method closing brace
• legal-reviewer.ts: Fixed }(); → })();

Missing method closing braces (2 files)

• legal-research-service.ts: Added closing brace for searchGoogleScholar method
• michigan-citations.ts: Added closing brace for parseTraditional method

Extraneous braces removed (2 files)

• drafting-mode-executor.ts: Removed 2 extra closing braces
• potemkin-clio.ts: Removed 1 extra closing brace

Example Fix

Before:

// Missing try keyword
   catch (error) {
     return { success: false, error: error.message };
   }
}
}
}  // Extra braces

After:

  } catch (error) {
    return { success: false, error: error.message };
  }
}

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Manual Fix of Errors Identified in Previous Sweep</issue_title>
<issue_description>Remaining Work:
20 syntax errors remain in 10 files that require manual review:

6 incomplete try/catch blocks
8 "declaration or statement expected" errors
6 misplaced parentheses/unexpected tokens
These are complex structural issues that need careful inspection of the original code intent rather than automated fixes.</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Manual Fix of Errors Identified in Previous Sweep #260

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[coderabbitai]

Important

Review skipped

Ignore keyword(s) in the title.

⛔ Ignored keywords (2)

• WIP
• DO NOT REVIEW

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codacy-production]

Codacy's Analysis Summary

0 new issue (≤ 0 issue)
0 new security issue
0 complexity

Review Pull Request in Codacy →

✨ AI Reviewer available: add the codacy-review label to get contextual insights without leaving GitHub.

[MightyPrytanis]

@copilot Great. Can you tell me anything more about the remaining errors in the build output that are unrelated import/export and type compatibility issues that were not part of the original 20 syntax errors identified in the issue?

[Copilot]

@copilot Great. Can you tell me anything more about the remaining errors in the build output that are unrelated import/export and type compatibility issues that were not part of the original 20 syntax...

Yes! There are 24 remaining TypeScript errors in the build, categorized as follows:

Import/Export Issues (13 errors)

TS2724 - Missing exported member:

• 7 files trying to import lowercase instance names (e.g., costEstimationService) but only class names are exported (e.g., CostEstimationService)
• Files affected: chronometric modules/services, MAE engine, fact-checker tool

TS1031 - Invalid export modifier:

• 4 files have export const inside a class body (should be outside)
• Files: cost-estimation.ts, forensic-reconstruction.ts, multi-model-service.ts, chronometric.ts

Type Compatibility Issues (3 errors)

TS2554 & TS2345:

• http-bridge.ts (line 1024): z.record() needs 2 arguments, only 1 provided
• http-bridge.ts (lines 1071, 1112): Tax calculation year type mismatch - accepts number but expects specific years 2018 | 2019 | ... | 2025

Duplicate Identifier Issues (6 errors)

TS2300 & TS2451:

• pdf-extractor.ts: Duplicate createRequire imports and require variable declarations
• text-processor.ts: Variable normalized declared twice in same scope

Modifier Issues (2 errors)

TS1184:

• error-sanitizer.ts (lines 75, 95): export modifier appearing where not allowed - likely inside a class or other structure

These are all distinct from the 20 syntax errors you fixed (try/catch blocks, IIFE closures, missing braces). They're structural/design issues requiring different fixes:

• Refactoring exports to match imports
• Adding type assertions or updating type definitions
• Removing duplicate code
• Moving exports to proper scope