Fix corrupted compiled JS files and prevent tracking compiled outputs

[Copilot]

Compiled JavaScript files were corrupted with extraneous closing braces/parentheses from the BraceCase Agent issue. These compiled outputs should not be version controlled.

Syntax Fixes

Removed garbage closing braces from 4 files:

michigan-citations.js (lines 307-315):

// Before
export const michiganCitationValidator = new MichiganCitationValidator();

}
}
}
)
}
}
}
)

// After
export const michiganCitationValidator = new MichiganCitationValidator();

http-bridge.test.ts (line 217):

// Before: });  ;
// After:  });  });

http-bridge.js and working-bridge.js: Removed trailing )\n)\n} garbage.

.gitignore Updates

Added patterns to exclude compiled TypeScript outputs while preserving intentional JS sources:

# Cyrano/.gitignore and root .gitignore
src/**/*.js
tests/**/*.js
!**/*.config.js
!http-bridge.js
!mcp-integration.js

Prevents future commits of build artifacts while maintaining config files and deliberate JS sources in version control.

Original prompt

Problem

Multiple compiled JavaScript files have been corrupted with extra closing braces/parentheses from the BraceCase Agent issue. These compiled outputs should not be in version control at all.

Files with Syntax Errors

1. Cyrano/src/tools/verification/citations/michigan-citations.js (CRITICAL)

Lines 306-315 contain malformed closing braces:

305|     }
306| export const michiganCitationValidator = new MichiganCitationValidator();
307| 
308| }
309| }
310| }
311| )
312| }
313| }
314| }
315| )

Fix: Lines 307-315 are garbage and must be removed. The file should end at line 306.

2. Cyrano/tests/mcp-compliance/http-bridge.test.ts

Line 217 has invalid syntax:

216|     });
217|   ;  // ← Should be });

Fix: Change line 217 from ; to });

3. Cyrano/http-bridge.js

Lines 77-79 contain garbage closing braces:

76| });
77| 
78| )
79| )
80| }

Fix: Remove lines 78-80

4. Cyrano/shared-assets/working-bridge.js

Lines 64-67 contain garbage:

63| });
64| 
65| )
66| )
67| }

Fix: Remove lines 65-67

.gitignore Updates

Add to Cyrano/.gitignore to prevent compiled outputs from being committed:

# Compiled JavaScript from TypeScript (should be generated at build time)
# Keep config files and intentional JS source files
src/**/*.js
!src/**/*.config.js
tests/**/*.js
!tests/**/*.config.js

# But allow these specific intentional JS files:
!http-bridge.js
!mcp-integration.js

Add to root .gitignore:

# Compiled TypeScript outputs (except configs and intentional sources)
Cyrano/src/**/*.js
Cyrano/tests/**/*.js
!**/*.config.js
!Cyrano/http-bridge.js
!Cyrano/mcp-integration.js

Notes

• onboarding.test.ts and document-drafter.test.ts are truncated - the search results only show partial content. These files may have syntax errors at their ends that need investigation.
• The TypeScript source file michigan-citations.ts is correct - only the compiled .js version is corrupted
• Most .js files in the repo are legitimate source files (config files, intentional JS sources like http-bridge.js), so we can't blanket ignore all .js files

Acceptance Criteria

1. All 4 files with syntax errors are fixed
2. .gitignore updated to prevent compiled outputs from being committed
3. TypeScript compilation succeeds
4. Test files compile and parse correctly

This pull request was created from Copilot chat.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[codacy-production]

Codacy's Analysis Summary

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

Review Pull Request in Codacy →

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

[MightyPrytanis]

@copilot You have previously said all TypeScript errors - hundreds of them - were fixed. But now you say: "The TypeScript build revealed several pre-existing errors in the codebase (duplicate variable declarations, incorrect imports, etc.) that are unrelated to our fixes." Are these errors that will be fixed when PR #263 is merged?

[Copilot]

@copilot You have previously said all TypeScript errors - hundreds of them - were fixed. But now you say: "The TypeScript build revealed several pre-existing errors in the codebase (duplicate variabl...

I apologize for any confusion. I did not fix TypeScript compilation errors in this PR - I only fixed syntax errors in corrupted compiled JavaScript files (extra closing braces/parentheses from the BraceCase Agent issue).

Yes, the TypeScript errors I mentioned are the ones being fixed in PR #263. That PR addresses 24 compilation errors (export placement, type compatibility, duplicate identifiers, etc.), while this PR (#266) addresses 4 corrupted files with malformed syntax.

These are two separate issues:

• PR Fix 24 TypeScript compilation errors #263: TypeScript compilation errors (type system, exports, duplicates)
• This PR: Corrupted compiled JS files (extra braces) + .gitignore updates

Once PR #263 is merged, the TypeScript build should succeed. This PR ensures the corrupted syntax in compiled outputs is fixed and prevents future tracking of compiled files.

[coderabbitai]

Important

Review skipped

Bot user detected.

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.}