feat: add Ollama, Enhanced Sandbox, and Sub-agents#31
Conversation
- Ollama provider for local LLM support (localhost:11434) - Enhanced sandbox with Docker/Firejail/ulimit backends - Sub-agent system for specialized tasks (research, code-review, test-writer, refactor, documentation) New files: - src/providers/ollama.ts - Ollama streaming provider - src/sandbox/index.ts - Multi-backend sandbox implementation - src/tools/sandbox.ts - sandbox and sandbox_info tools - src/agents/subagent.ts - Sub-agent types and runner - src/tools/spawn.ts - spawn_agent and list_agents tools Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughIntroduces a comprehensive Factory-compatible repository readiness scanner as a new module (agent-readiness) with nine pillars and five levels, plus Ollama provider support and sub-agent spawning capabilities with sandbox isolation to the main codebase. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~50 minutes Possibly related PRs
Poem
✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Add agent-readiness CLI tool for scanning repository maturity against Factory Agent Readiness model (10 pillars, 5 levels, 80% gating). Security improvements: - Path traversal protection in profile loading and config files - ReDoS-safe regex checker using linear-time string scanning - LRU cache with size limits to prevent unbounded memory growth - Workspace pattern injection prevention Features: - 33 checks across 10 pillars (docs, style, build, test, security, observability, env, ci, monorepo, task_discovery) - YAML-based extensible profile system - JSON and Markdown output formats - Monorepo detection and app-scope scanning - Init command with templates for missing files Testing: - 43 tests covering checks, engine, security, and LRU cache - Test fixtures for minimal, standard, and monorepo scenarios Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Move sub-agent types to spawn.ts to avoid circular import - Add parseMemoryLimit() for Firejail --rlimit-as option - Firejail now respects maxMemory parameter Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Summary
New Tools
sandboxsandbox_infospawn_agentlist_agentsSub-agent Types
research- Explore codebases (glob, grep, read_file, list_dir, web_search)code-review- Review code (glob, grep, read_file, diff, git_diff)test-writer- Write tests (glob, grep, read_file, write_file, bash)refactor- Refactor code (glob, grep, read_file, edit_file, bash)documentation- Generate docs (glob, grep, read_file, write_file, edit_file)Test plan
npm run buildnano --provider ollama --model llama3.2sandbox_infotoollist_agentsandspawn_agent🤖 Generated with Claude Code
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.