Refresh Claude with new framework#149
Merged
itomek-amd merged 11 commits intoamd:mainfrom Jan 9, 2026
Merged
Conversation
- Create new standalone CLI at src/gaia/agents/code/cli.py following EMR pattern - Add gaia-code entry point in setup.py - Remove code subcommand from main CLI (parser and handler) - Delete deprecated src/gaia/agents/code/app.py - Update documentation: replace 'gaia code' with 'gaia-code' in 8 files The new gaia-code CLI includes: - RoutingAgent integration for intelligent language detection - Lemonade initialization (32768 context) - init subcommand for model setup - All features from previous implementation - Rich console output following EMR agent pattern
Remove confusing `run` and `init` subcommands from gaia-code CLI. The `run` subcommand was unnecessary (it's the only operation), and `init` is now automatic on first use. Changes: - Remove all subparser logic (run/init subcommands) - Move all arguments to top-level parser - Remove manual init command (auto-initializes on first run) - Fix help display to show all available arguments - Update examples to match actual usage - Reduce code from 611 to 435 lines (-176 lines) Users can now simply run: gaia-code "Build me an app" gaia-code --interactive gaia-code --help Instead of requiring: gaia-code run "Build me an app" gaia-code init
## Agent Changes - **Pruned 4 redundant agents** (28 → 24 total): - Merged webapp-developer into frontend-developer - Merged gaia-agent-developer and backend-architect into ai-engineer - Removed search-specialist (redundant with research-enforcer skill) - **Renamed 5 agents for clarity**: - ai-engineer → gaia-agent-builder (clarifies GAIA agent creation) - amd-optimizer → hardware-optimizer (hardware-level tuning focus) - python-pro → python-developer (removes vague "pro" suffix) - typescript-pro → typescript-developer (consistent naming) - architect-review → architecture-reviewer (consistent with code-reviewer) - **Created sdk-architect agent**: Provides SDK API design guidance and pattern consistency ## Configuration Updates - **CLAUDE.md**: Updated documentation structure, added agent list with clear use cases - **Pre-commit linting hook**: Auto-fixes code before commits without slowing edits - **Frontend agent**: Simplified, removed deprecated RAUX references ## Benefits - Clearer agent names indicate when to use each one - "NOT for..." guidance prevents confusion between overlapping agents - Reduced redundancy while maintaining coverage - Better documentation structure reflecting current Mintlify setup
There was a problem hiding this comment.
Pull request overview
This PR refactors the Claude Code agent system, reducing redundancy from 28 to 24 agents through strategic pruning and merging, while improving naming clarity and adding automated linting infrastructure.
Key Changes:
- Extracted code agent CLI from monolithic
src/gaia/cli.pyto dedicatedsrc/gaia/agents/code/cli.pymodule with newgaia-codeentry point - Consolidated 4 redundant agents (webapp-developer → frontend-developer, gaia-agent-developer + backend-architect + ai-engineer → gaia-agent-builder, search-specialist removed)
- Renamed 5 agents for clarity (ai-engineer → gaia-agent-builder, amd-optimizer → hardware-optimizer, python-pro → python-developer, typescript-pro → typescript-developer, architect-review → architecture-reviewer)
- Added new sdk-architect agent for SDK API design guidance and introduced pre-commit linting hook infrastructure
Reviewed changes
Copilot reviewed 27 out of 27 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
util/pre-commit-lint.sh |
New bash script for pre-commit linting, triggered by git commit commands |
src/gaia/cli.py |
Removed 300+ lines of code agent handling (parser and handler function moved to dedicated module) |
src/gaia/agents/code/cli.py |
New 435-line module containing extracted code agent CLI logic with argparse setup and command handling |
src/gaia/agents/code/app.py |
Deleted 266-line legacy CLI implementation replaced by new cli.py module |
setup.py |
Added gaia-code console script entry point for new CLI module |
docs/reference/cli.mdx |
Updated 8 command examples from gaia code to gaia-code |
docs/playbooks/code-agent/*.mdx |
Updated 10 command examples across 3 playbook files to use gaia-code |
docs/guides/*.mdx |
Updated 18 command examples across routing.mdx and code.mdx to use gaia-code |
CLAUDE.md |
Updated documentation structure references, added agent list with new names, improved organization |
.claude/settings.local.json |
Added PreToolUse hook configuration for bash commands to run pre-commit linting |
.claude/agents/webapp-developer.md |
Deleted 64-line agent merged into frontend-developer |
.claude/agents/typescript-developer.md |
Renamed from typescript-pro, updated description for clarity |
.claude/agents/search-specialist.md |
Deleted 64-line agent (redundant with research-enforcer skill) |
.claude/agents/sdk-architect.md |
New 195-line agent for SDK API design, pattern consistency, and breaking change evaluation |
.claude/agents/python-developer.md |
Renamed from python-pro, updated description and scope clarification |
.claude/agents/hardware-optimizer.md |
Renamed from amd-optimizer with clearer hardware optimization focus |
.claude/agents/gaia-agent-developer.md |
Deleted 54-line agent merged into gaia-agent-builder |
.claude/agents/gaia-agent-builder.md |
New 78-line agent consolidating agent creation, LLM integration, and service architecture |
.claude/agents/frontend-developer.md |
Simplified, removed deprecated RAUX references, updated structure |
.claude/agents/cli-developer.md |
Updated command reference from gaia code to gaia-code |
.claude/agents/backend-architect.md |
Deleted 106-line agent merged into gaia-agent-builder |
.claude/agents/architecture-reviewer.md |
Renamed from architect-reviewer for consistency |
.claude/agents/api-documenter.md |
Updated command examples from gaia code to gaia-code |
.claude/agents/ai-engineer.md |
Deleted 57-line agent merged into gaia-agent-builder |
Comments suppressed due to low confidence (1)
.claude/agents/architecture-reviewer.md:3
- The description still references the old agent name "architect-reviewer" in the examples. This should be updated to use the new name "architecture-reviewer" for consistency.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
itomek-amd
added
documentation
Updated agents following Anthropic's best practices for specificity and detailed guidance. Replaced generic examples with actual GAIA patterns. Changes: - CLAUDE.md: Added development commands section (setup, linting, testing) - api-documenter.md: Real Mintlify MDX patterns from docs/spec/ and docs/guides/ - cli-developer.md: Actual argparse patterns from src/gaia/cli.py with line refs - python-developer.md: Real Agent base class, @tool decorator, and ChatSDK patterns - typescript-developer.md: Actual Electron app patterns from src/gaia/apps/ All agents now reference real source files instead of generic placeholders, following Anthropic's guidance: "The more guidance you provide, the better the subagent will perform."
itomek-amd
added
the
ready_for_ci
itomek
changed the title
itomek
force-pushed
the
itomek/claude-agents
branch
from
a7bc11b to
fceb6e5
Compare
- Remove unused CodeAgent import from cli.py - Extract duplicated interactive mode code into helper function - Replace bash pre-commit hook with cross-platform Python version - Update hook configuration to use Python instead of bash - Fix agent name references in architecture-reviewer.md This improves code quality and Windows compatibility as requested in PR review comments. Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
- Remove pre-commit hook configuration from settings.local.json - Delete util/pre-commit-lint.py (hook wasn't working reliably) - Update copyright year in cli.py from 2024-2025 to 2025-2026 - Rely on CI checks for linting validation Hook issues discovered during testing: - TOOL_INPUT env var was empty, preventing command detection - Hook output was suppressed/invisible - Settings required restart to take effect - No clear feedback when hook ran or failed Manual linting recommended: python util/lint.py --all --fix Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
- Revert all docs/ changes (gaia code → gaia-code) - Revert setup.py (remove gaia-code entry point) - Delete src/gaia/agents/code/cli.py (new file removed) - Restore src/gaia/agents/code/app.py (previous implementation) These changes are being handled in a different branch.
- Restore src/gaia/cli.py to main branch state - Keep 'gaia code' command in main CLI - Code agent CLI refactoring handled in separate branch
kovtcharov-amd
approved these changes
Jan 8, 2026
This was referenced Jan 9, 2026
- Change all 16 agents from sonnet to opus model - Delete hardware-optimizer agent (out of scope) - Add docs.json reference to api-documenter and sdk-architect - Add GAIA logger context to python-developer - Update testing guidance to prefer GGUF/llama.cpp - Add guides/playbooks references to sdk-architect Created issues: - #156: Document Electron patterns for frontend-developer - #157: Explore custom GAIA agent builder with Opus Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Resolved conflicts in agent files: - api-documenter.md: Kept PR #149 Mintlify MDX focus with opus model - cli-developer.md: Kept PR #149 structure with opus model Merged PR #147 changes: - GAIA Code CLI update (gaia-code command structure) - Documentation updates for new CLI interface - New cli.py implementation Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Agent Changes
Pruned Agents
webapp-developer→ Merged intofrontend-developergaia-agent-developer+backend-architect→ Merged intogaia-agent-buildersearch-specialist→ Removed (redundant with research-enforcer skill)Renamed for Clarity
ai-engineergaia-agent-builderamd-optimizerhardware-optimizerpython-propython-developertypescript-protypescript-developerarchitect-reviewarchitecture-reviewercode-reviewernamingNew Agent
Configuration Updates
util/pre-commit-lint.sh+ settings.local.json hook - auto-fixes before commitsBenefits
✅ Agent names now clearly indicate when to use them
✅ "NOT for..." descriptions prevent confusion
✅ Less redundancy while maintaining full coverage
✅ Pre-commit linting keeps code clean automatically
✅ Documentation reflects current structure
🤖 Generated with Claude Code