Skip to content

feat: gemini cli module #246

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open

feat: gemini cli module #246

wants to merge 27 commits into from

Conversation

35C4n0r
Copy link
Contributor

@35C4n0r 35C4n0r commented Jul 21, 2025

Closes #237
/claim #237

Description

https://www.loom.com/share/5b099c73935f4f87b8fdafe1509bb79d?sid=7fea43d6-86e9-45ae-9892-efeb3c820b82
Updated: https://www.loom.com/share/62e907eae8544d8cbbe560d7f63bd02d?sid=201212fa-eb58-44b5-8706-8bf9c2c37433

Type of Change

  • New module
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder-labs/modules/gemini
New version: v1.0.0
Breaking change: [ ] Yes [ ] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun run fmt)
  • Changes tested locally

Related Issues

@algora-pbc algora-pbc bot mentioned this pull request Jul 21, 2025
4 tasks
@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 22, 2025

@matifali PR ready for review

@bpmct
Copy link
Member

bpmct commented Jul 22, 2025

Hey @35C4n0r! This is awesome!! Can you also get it working with tasks and provide a basic example snippet in the README?

See https://registry.coder.com/templates/coder-labs/tasks-docker for an example template that uses the Tasks view

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 22, 2025

On it 👍

@DevelopmentCats
Copy link
Contributor

DevelopmentCats commented Jul 22, 2025

Since this is a bounty claim, could you please provide the screen recording of this module being loaded into a workspace, and display the modules functionality as outlined in: #213

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 23, 2025

@DevelopmentCats it's already there in the description of the PR (It's a loom link).

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 23, 2025

loom updated in PR description

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 23, 2025

Both #229 and this PR are ready for review :)

Copy link
Contributor

@DevelopmentCats DevelopmentCats left a comment

Choose a reason for hiding this comment

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

This all looks good to me, but the only thing you are missing is an avatar and README for your namespace.

You can look at the coder registry namespace for reference.

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 23, 2025

missing is an avatar and README for your namespace.
@DevelopmentCats that is in #229.

@35C4n0r 35C4n0r requested a review from DevelopmentCats July 23, 2025 19:39
@DevelopmentCats
Copy link
Contributor

The job is failing because of the missing README for the namespace, so we might need to wait on the tmux module commit before pushing this one but ultimately it looks good!

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 24, 2025

@DevelopmentCats @hugodutka tests updated. Ready for review.

@35C4n0r 35C4n0r requested a review from hugodutka July 24, 2025 19:20
@bpmct
Copy link
Member

bpmct commented Jul 25, 2025

@35C4n0r can we put this under the coder-labs contributor instead of yours?

We hope to support co-authoring so it can show up as both, but I want to associate the agent modules with the Coder name as we will ship updates/patches to them and there is no expectation for you to always do that :)

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 25, 2025

@bpmct sure, refactoring it 👍🏻

@35C4n0r
Copy link
Contributor Author

35C4n0r commented Jul 25, 2025

@bpmct Done ! Please lmk if this needs any more changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gemini CLI Module
5 participants