Github Extension #6

thanosan23 · 2025-03-11T14:46:07Z

Added connection to Mantis Extensions to allow for GitHub space creation

gemini-code-assist

Hello @thanosan23, 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!

Summary of Changes

This pull request adds a new connection to Mantis Extensions that allows for the creation of GitHub spaces. It introduces the octokit library to fetch commit data from GitHub repositories, including commit messages, authors, dates, links, and diffs. This data is then used to create a Mantis space, which is injected into the GitHub repository content area as an iframe. The PR also includes minor UI tweaks to improve the appearance of the connection dialog.

Highlights

GitHub Connection: Adds a new connection to Mantis Extensions for creating spaces from GitHub repositories.
Dependency: octokit: Introduces the octokit library to fetch commit data from GitHub.
Data Extraction: Extracts commit messages, authors, dates, links, and diffs from GitHub repositories.
UI Injection: Injects the created Mantis space into the GitHub repository content area as an iframe.
UI Tweaks: Includes minor UI tweaks to improve the appearance of the connection dialog.

Changelog

Click here to see the changelog

.env
- Added PLASMO_PUBLIC_GITHUB_REDIRECT_URI and PLASMO_PUBLIC_GITHUB_AUTH variables for GitHub authentication.
- Commented out PLASMO_PUBLIC_GITHUB_CLIENT_ID and PLASMO_PUBLIC_GITHUB_CLIENT_SECRET as their usage is still being determined.
package.json
- Added octokit as a dependency.
plasmo.config.ts
- No significant changes.
src/connections/github/connection.tsx
- Implemented the GitHubConnection component to fetch commit data and create Mantis spaces.
- Uses octokit to retrieve commit information, including messages, authors, dates, links, and diffs.
- Injects the created Mantis space into the GitHub repository content area as an iframe.
- Includes functions to trigger the connection based on the URL and to create the space with extracted data.
src/content.tsx
- Updated the text color of the connection name, counter, and note text in the connection dialog to black.
src/driver.tsx
- Added GitHubConnection to the list of available connections.
yarn.lock
- Updated dependencies, including adding octokit and its related dependencies.

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 is currently in preview and 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 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.

Did you know?

The name 'Octokit' is a play on words, combining 'Octo' (referring to GitHub's Octocat mascot) and 'Kit' (referring to a set of tools or library).

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 introduces a new connection to Mantis Extensions for GitHub space creation. The changes include adding necessary dependencies, configuring environment variables, and implementing the connection logic. Overall, the code seems well-structured, but there are a few areas that could be improved for clarity and security.

Summary of Findings

Environment Variable Security: Storing the GitHub authentication token directly in the .env file is not secure, especially for a client-side application. This exposes the token to potential vulnerabilities. Consider using a more secure method for handling authentication, such as OAuth flow or storing the token securely on the server-side.
Error Handling: The error handling in the createSpace function could be improved. Currently, it catches errors during the diff retrieval but simply assigns "Diff unavailable". Consider logging the error or providing more informative feedback to the user.
Magic Numbers: The scale variable in the injectUI function is a magic number. It would be better to define it as a constant with a descriptive name.

Merge Readiness

The pull request introduces a significant new feature. However, due to the high severity security concern regarding the storage of the GitHub authentication token, I recommend addressing this issue before merging. Additionally, addressing the error handling and magic number issues would improve the overall quality of the code. I am unable to approve this pull request, and users should have others review and approve this code before merging.

gemini-code-assist · 2025-03-11T14:46:36Z

.env

+#PLASMO_PUBLIC_GITHUB_CLIENT_SECRET =
+PLASMO_PUBLIC_GITHUB_REDIRECT_URI = http://localhost:3000/oauth/callback
+PLASMO_PUBLIC_GITHUB_AUTH=


Storing the GitHub authentication token directly in the .env file is not secure, especially for a client-side application. This exposes the token to potential vulnerabilities. Consider using a more secure method for handling authentication, such as OAuth flow or storing the token securely on the server-side.

#PLASMO_PUBLIC_GITHUB_AUTH= # Consider using a more secure method for handling authentication, such as OAuth flow

src/connections/github/connection.tsx

gemini-code-assist

Code Review

This pull request introduces a new GitHub connection to the Mantis extension, allowing users to create spaces based on GitHub repository commits. The changes include adding necessary dependencies, configuring environment variables, implementing the GitHub connection logic, and updating the UI to support the new connection. There are a few suggestions for code improvements, such as using import instead of require, adding descriptive comments, using object destructuring, and removing debugging statements.

gemini-code-assist · 2025-06-20T19:33:12Z

src/connections/github/connection.tsx

+
+import githubIcon from "data-base64:../../../assets/github.png";
+import { getSpacePortal, registerAuthCookies, reqSpaceCreation } from "../../driver";
+const { Octokit } = require("@octokit/core");


It's generally recommended to use import instead of require for better readability and consistency in modern JavaScript/TypeScript code.

Suggested change

const { Octokit } = require("@octokit/core");

import { Octokit } from "@octokit/core";

gemini-code-assist · 2025-06-20T19:33:12Z

src/connections/github/connection.tsx

+            });
+            diff = diffResponse.data as string;


It's good practice to explicitly define the type of diffResponse.data to ensure type safety and prevent unexpected behavior. Consider using a type assertion or defining an interface for the expected response data.

gemini-code-assist · 2025-06-20T19:33:12Z

src/content.tsx

@@ -413,7 +413,7 @@ const ConnectionDialog = ({ activeConnections, close }: { activeConnections: Man
                        <ArrowHead disabled={connectionIdx === 0} left />
                    </button>
                    <div className="flex-grow" />
-                    <span className="text-gray-600">
+                    <span className="text-black-600">


This class name text-black-600 does not exist in tailwind, did you mean text-gray-600 or text-black?

<span className="text-gray-600">

aryanahou and others added 9 commits February 24, 2025 20:23

github commits

da75382

quick fix

7311765

fixed speed issues

06953a7

pagination

1d6e5ef

updated connections

bea9e1c

updated with main

9907a62

fixed env

6765931

added github diff

df3a737

merged with fixes on main

ef451df

gemini-code-assist bot reviewed Mar 11, 2025

View reviewed changes

gemini-code-assist bot suggested changes Mar 11, 2025

View reviewed changes

thanosan23 requested a review from LucaVor March 12, 2025 19:33

thanosan23 added 5 commits April 5, 2025 08:47

updated with main

85ded3d

more fixes

6b4a707

updated legacy code

9cfdc2f

merged with main

d62c1aa

fixed suggestions, updated package.json

a14b538

gemini-code-assist bot reviewed Jun 20, 2025

View reviewed changes

added diff summaries + author info + general commit info

1f051b0

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Github Extension #6

Github Extension #6

Uh oh!

thanosan23 commented Mar 11, 2025

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

gemini-code-assist bot Mar 11, 2025

Uh oh!

Uh oh!

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

gemini-code-assist bot Jun 20, 2025

Uh oh!

gemini-code-assist bot Jun 20, 2025

Uh oh!

gemini-code-assist bot Jun 20, 2025

Uh oh!

Uh oh!

	const { Octokit } = require("@octokit/core");
	import { Octokit } from "@octokit/core";

Github Extension #6

Are you sure you want to change the base?

Github Extension #6

Uh oh!

Conversation

thanosan23 commented Mar 11, 2025

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Summary of Changes

Highlights

Changelog

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Summary of Findings

Merge Readiness

Uh oh!

gemini-code-assist bot Mar 11, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Jun 20, 2025

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Jun 20, 2025

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Jun 20, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!