Skip to content

Remove Gemini engine and all references from codebase#40

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/fix-c1b2497b-821c-483d-a8ab-a6b0e7861c08
Closed

Remove Gemini engine and all references from codebase#40
Copilot wants to merge 1 commit intomainfrom
copilot/fix-c1b2497b-821c-483d-a8ab-a6b0e7861c08

Conversation

Copy link
Contributor

Copilot AI commented Aug 13, 2025

This PR completely removes the Gemini agentic engine from the gh-aw codebase as requested in the issue. The Gemini engine was one of three supported AI engines (alongside Claude and Codex) but is no longer needed.

Changes Made

Core Engine Removal

  • Deleted pkg/workflow/gemini_engine.go - Complete Gemini engine implementation
  • Deleted pkg/workflow/gemini_engine_test.go - All Gemini engine tests
  • Updated pkg/workflow/agentic_engine.go - Removed Gemini engine registration from NewEngineRegistry()

Test Updates

  • Updated pkg/workflow/agentic_engine_test.go - Fixed tests to expect 2 engines instead of 3
  • Updated pkg/workflow/engine_config_test.go - Removed Gemini from test configurations
  • All existing tests now pass with only Claude and Codex engines

CLI and Validation

  • Updated cmd/gh-aw/main.go - Removed "gemini" from CLI validation and help text
  • Updated pkg/parser/schemas/main_workflow_schema.json - Removed "gemini" from allowed engine enum values
  • CLI now correctly rejects --engine gemini with error: "Must be 'claude' or 'codex'"

Documentation Cleanup

  • Updated README.md - Removed Gemini from engine examples
  • Updated docs/frontmatter.md - Removed Gemini engine documentation and examples
  • Updated docs/secrets.md - Removed GEMINI_API_KEY documentation section
  • Updated docs/tools.md - Removed Gemini engine tools section
  • Updated pkg/cli/templates/instructions.md - Removed Gemini references from templates

Test Workflows

  • Deleted .github/workflows/test-gemini.md - Gemini test workflow markdown
  • Deleted .github/workflows/test-gemini.lock.yml - Compiled Gemini test workflow

Validation

The removal has been thoroughly tested:

  • ✅ All existing tests pass
  • ✅ Build, lint, and compilation successful
  • ✅ CLI correctly rejects "gemini" as invalid engine
  • ✅ Schema validation prevents "gemini" in workflow frontmatter
  • ✅ No remaining "gemini" references found in codebase
  • ✅ Claude and Codex engines continue to work normally

The system now cleanly supports only Claude (default) and Codex (experimental) engines, with all Gemini functionality completely removed.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • api.contoso.com
    • Triggering command: /tmp/go-build791013428/b256/cli.test -test.testlogfile=/tmp/go-build791013428/b256/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true (dns block)
    • Triggering command: /tmp/go-build3749983207/b256/cli.test -test.testlogfile=/tmp/go-build3749983207/b256/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true (dns block)
    • Triggering command: /tmp/go-build1284847619/b256/cli.test -test.testlogfile=/tmp/go-build1284847619/b256/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@dsyme dsyme closed this Aug 13, 2025
Copilot AI changed the title [WIP] remove agentic engine gemini and all references of gemini in docs Remove Gemini engine and all references from codebase Aug 13, 2025
Copilot AI requested a review from pelikhan August 13, 2025 15:01
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.

3 participants