Skip to content

fix(auth): fix JS interop lints#17802

Merged
Lyokone merged 4 commits intomainfrom
fix_auth_js_interop
Oct 22, 2025
Merged

fix(auth): fix JS interop lints#17802
Lyokone merged 4 commits intomainfrom
fix_auth_js_interop

Conversation

@kevmoo
Copy link
Copy Markdown
Contributor

@kevmoo kevmoo commented Oct 20, 2025

There are a number of (likely) incorrect uses of JS interop types in this library. I just went through and fixed all of the latest lints with the latest SDK.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

@kevmoo kevmoo changed the title firebase_auth: fix JS interop lints fix: firebase_auth: fix JS interop lints Oct 20, 2025
@cynthiajoan cynthiajoan changed the title fix: firebase_auth: fix JS interop lints fix(auth): fix JS interop lints Oct 21, 2025
@cynthiajoan
Copy link
Copy Markdown
Collaborator

/gemini review

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request does a great job of modernizing the JavaScript interop layer for Firebase Auth on the web. By updating to use extension type and typed JSPromise, you've significantly improved type safety and code readability. The resulting removal of explicit casts and the ability to use tear-offs make the code much cleaner and more idiomatic. I have one minor suggestion regarding an inconsistency in the JS interop definitions, but overall this is an excellent refactoring.

@kevmoo
Copy link
Copy Markdown
Contributor Author

kevmoo commented Oct 21, 2025

All of the types across flutterfire should be review similarly. Happy to help!

Copy link
Copy Markdown
Contributor

@MichaelVerdon MichaelVerdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Lyokone Lyokone merged commit 0956646 into main Oct 22, 2025
29 of 31 checks passed
@Lyokone Lyokone deleted the fix_auth_js_interop branch October 22, 2025 08:46
@firebase firebase locked and limited conversation to collaborators Nov 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants