Add userinfo.email scope when in studio

[joehan]

Description

Seems like we need this to access the monospace-pa API

[gemini-code-assist]

Summary of Changes

Hello @joehan, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

I've implemented a change to ensure that the necessary userinfo.email OAuth scope is requested when our application is operating within a Firebase Studio environment. This adjustment is crucial for enabling proper access to the monospace-pa API, addressing a dependency identified for its functionality.

Highlights

• OAuth Scope Addition: I've added the userinfo.email OAuth scope to the list of required scopes when the application is running within a Firebase Studio environment.
• Conditional Scope Requirement: The userinfo.email scope is now conditionally pushed to the requiredScopes array in src/requireAuth.ts only if isFirebaseStudio() returns true.
• New Scope Constant: A new constant, USERINFO_EMAIL, has been defined in src/scopes.ts to represent the https://www.googleapis.com/auth/userinfo.email OAuth scope.

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 in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in issue comments and review comments.
Help	/gemini help	Displays a list of available commands.

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 or fill out our survey to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

This pull request adds the userinfo.email scope when running in Firebase Studio, which is required for accessing the monospace-pa API. The changes are logical and correctly implement the required functionality. I've identified a couple of minor areas for improvement regarding code consistency and redundancy, and have provided suggestions to address them.

[samedson]

🚢

[codecov-commenter]

Codecov Report

❌ Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.76%. Comparing base (f1c2d21) to head (539eb57).
⚠️ Report is 16 commits behind head on master.

Files with missing lines	Patch %	Lines
src/requireAuth.ts	50.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #8935      +/-   ##
==========================================
- Coverage   52.18%   50.76%   -1.43%     
==========================================
  Files         418      497      +79     
  Lines       30694    32952    +2258     
  Branches     6348     6845     +497     
==========================================
+ Hits        16018    16728     +710     
- Misses      13234    14751    +1517     
- Partials     1442     1473      +31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.