Skip to content

Feat/gemini integration#1

Closed
atreyakamat wants to merge 6 commits into
sam-agents:mainfrom
atreyakamat:feat/gemini-integration
Closed

Feat/gemini integration#1
atreyakamat wants to merge 6 commits into
sam-agents:mainfrom
atreyakamat:feat/gemini-integration

Conversation

@atreyakamat
Copy link
Copy Markdown
Contributor

This Pull Request integrates the Gemini CLI as a first-class supported platform for the SAM (Smart Agent Manager)
autonomous TDD system. It provides the same robust agent orchestration available for Claude Code, Cursor, and
Antigravity, specifically optimized for the Gemini CLI environment.

🚀 Key Changes

  1. Core Platform Integration
  • bin/cli.js Update: Added native support for the gemini platform. The CLI now automatically scaffolds the
    .gemini/skills/ and .gemini/commands/ directories during installation.
  • Command Templates: Added templates/.gemini/commands/sam.toml, providing a standardized SAM prompt interface that
    allows users to invoke agents or start the TDD pipeline via natural language commands.
  1. Agent Skill Generation
  • 9 Specialized Skills: Automatically generates 9 Gemini-native skills (Orchestrator, Atlas, Dyna, Titan, Argus,
    Sage, Iris, Cosmo, and the full TDD Pipeline).
  • Rich Contextual References: Each skill includes a references/ folder containing the full agent definitions, design
    standards, and workflow documentation, ensuring the AI has the context needed to perform complex TDD tasks.
  1. Documentation & UX
  • User Guides: Added GEMINI.md, SAM_GEMINI_GUIDE.md, and SAM_GEMINI_USAGE.md to provide clear onboarding for Gemini
    CLI users.
  • Developer Guidelines: Updated AGENTS.md with technical specifications for the Gemini platform integration.
  1. Automated Verification
  • New Test Suite: Introduced scripts/verify-gemini.js to validate the installation logic.
  • package.json Integration: Added an npm test script to ensure Gemini integration remains stable in future releases.

✅ Verification Results
Verified the integration by running the new test suite:

1 npm test
2 # Result: SUCCESS: SAM Gemini integration verified and production-ready!
3 # ✓ _sam directory created
4 # ✓ .gemini/skills directory created
5 # ✓ All 9 skills verified successfully

@santoshrout
Copy link
Copy Markdown
Contributor

Hey Atreya (@atreyakamat) — thanks for this contribution! Great to see Gemini CLI support being added to SAM.

Reviewed the PR and found few issues I would like to add. For now, I am going ahead and fixing it. Later we discuss some code review processes that we can follow.


Isues that I found which are a Must-Fix
Default platform changed from claude to gemini — In promptPlatform(), the default for empty input and invalid input was changed to gemini. The CLI is a general-purpose tool and claude must remain the default.

showHelp() removes the Antigravity example — The Gemini example replaces the Antigravity line instead of being added alongside it. Antigravity and Gemini are separate platforms (different output paths: .agent/skills/ vs .gemini/skills/) and both should be listed.

Wrong script filenames in docs — CONTRIBUTING.md references verify_pr.js and SAM_DEPLOYMENT_GUIDE.md references verify_sam_gemini.js. The actual file added by this PR is scripts/verify-gemini.js.

_sam/docs/ not synced with templates/_sam/docs/ — New doc files were added to templates/_sam/docs/ but not to _sam/docs/. The convention is that _sam/ is the source of truth and templates/_sam/ mirrors it. Edits should go into _sam/ first, then sync to templates.

copyDocumentationToReferences() duplicates all docs into every agent — This function copies the entire docs folder, all agent definitions, design standards, and workflow files into every agent's references/ directory (9 copies of everything). Each agent should only have its own definition in references. Additionally, this function is retrofitted into the Antigravity generator, which changes existing behavior — that should be a separate PR if needed.

Suggestions
copyDocumentationToReferences() accepts targetDir and platform parameters that are never used inside the function — these can be removed.

generateGeminiSkills is a near-copy of generateAntigravitySkills — consider extracting a shared generator function that takes the output path as a parameter.

scripts/verify-gemini.js creates a test-output/ directory — add it to .gitignore so it doesn't get accidentally committed.

We appreciate the effort here and the Gemini support is a welcome addition. We'll get these sorted out and follow up with you directly.

@santoshrout santoshrout mentioned this pull request Mar 10, 2026
4 tasks
@santoshrout
Copy link
Copy Markdown
Contributor

Thank you for this contribution @atreyakamat! 🎉

We've incorporated your Gemini integration into PR #12 with some fixes applied (default platform restored to claude, removed file duplication, synced docs, fixed script references). Your commits are preserved in the history.

Thanks again for being one of our first contributors!

@santoshrout
Copy link
Copy Markdown
Contributor

Closing in favor of PR #12 which includes this work with fixes applied.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants