feat: finalize opencode integration unit#164
Merged
Conversation
[unit: opencode-integration] - Add per-unit short-term memory in .agents/memory/short-term/ - Split planning into bespoke agents (discovery, requirements) - Add all sub-agents with agency-agents references - Add QA agent for quality checks after each subagent - Update memory encoding (episodic/semantic) - Add unit reference guidance for GitHub PRs/issues
- Pre-commit now stages changes after go fmt and eslint --fix - All integration tests pass with Go 1.26 - Updated distrobox-setup.sh for proper Go 1.26, docker-compose - go fmt applied to backend files [unit: opencode-integration]
- Added get_go() helper to find Go 1.26 - All go commands now use - Removed -v from go test to reduce noise [unit: opencode-integration]
- Added 100ms sleep after subscriptions are created - Tests were failing due to race condition (subscriber not ready) - Added -p 1 -count=1 to run tests sequentially without caching [unit: opencode-integration]
[unit: opencode-integration]
- Discovery now runs before any new document (not just once) - No QA required for discovery agent - Dynamic questions loop until fully understood - Prior documents used as context to avoid repeat questions - Orchestrator updated to reflect special discovery handling [unit: opencode-integration]
- planning-discovery: Only asks questions in a loop, NO documents - planning-document: NEW agent that creates problem_space.md and bsd.md - Orchestrator updated: discovery -> document -> QA flow [unit: opencode-integration]
[unit: opencode-integration]
- Discovery: manual conversation until user says done - Other agents: task tool blocks until complete, full output returned - QA runs immediately after task tool completes - Use task_id to resume for fixes [unit: opencode-integration]
- Created .opencode/agents/planning-document.md (proper subagent) - Orchestrator: handle early subagent returns via task_id - Orchestrator: always resume rather than ask user - Orchestrator: NEVER use general agent type [unit: opencode-integration]
…test unit - Updated all document agents: planning-document, planning-requirements, research, architecture, implementation, testing - Each now has: ONE DOCUMENT PER PR guidance - Deleted fun unit test files - Updated memory to reflect fun unit removal [unit: opencode-integration]
- Rule now in AGENTS.md (applies to all agents) - Orchestrator updated to enforce one document per PR - Removed duplicates from individual agent files - Rule: spawn agent multiple times for multiple docs [unit: opencode-integration]
- Added: read existing documents in unit directory for context - Added: clarify one document per session (orchestrator spawns twice) - Added: guidance on handling existing documents [unit: opencode-integration]
- All agents: read existing documents in unit for context - All agents: clarify one document per session - All agents: guidance on handling existing documents - Applies to: research, architecture, implementation, testing [unit: opencode-integration]
- Discovery questions go to user verbatim - User answers go to discovery verbatim - Orchestrator never skips any discovery responses - Re-use same task_id for same discovery session [unit: opencode-integration]
- SKILL.md now defines explicit document sequence - Removed redundant sequence info from planning-document agent - Templates define structure and order [unit: opencode-integration]
…s from all sub agents - These are handled by orchestrator or skill - Source of truth is unit-planning skill [unit: opencode-integration]
- Added design.md agent for visual design and mockups - Updated reference agents: - backend.md: Backend Architect - frontend.md: Frontend Developer - architecture.md: Added Database Optimizer - implementation.md: Added SRE and Security Engineer - Updated tester.md to run ALL tests (backend, frontend, make) - Updated qa.md to remove subjective language - Updated orchestrator.md with: - Clear discovery flow (show FULL verbatim responses) - Always reuse task_id - QA after EVERY subagent (except discovery) - Files affected reporting - Template-to-agent mapping - general agent for small tasks - Updated AGENTS.md with General Principles - Added gh to distrobox-setup.sh
Owner
Author
|
/oc review this |
Owner
Author
|
@oc review this |
Owner
Author
|
/opencode review this |
Owner
Author
|
@OpenCode review this |
- Removed documentation section from AGENTS.md - Added to orchestrator with unit completion guidance - Added reminder to update design/README.md when unit is complete
|
I don't think oc nor OpenCode wants to review this - way too big a delta. ;) |
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
2 participants
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
This PR finalizes the OpenCode integration - a complete overhaul of how development agents work in the ACE Framework.
New Agent System
.opencode/agents/directory with 13 agent files:Reference Agent Updates
Orchestrator Features
Skill System
Memory System
.agents/memory/short-term/for unit-specific state.agents/memory/long-term.jsonfor persistent learningsDev Infrastructure
.dev/distrobox-setup.sh- development environment setup.dev/pre-commit.sh- pre-commit quality gates.github/workflows/opencode.yml- GitHub workflowBackend Fixes
Removed
.openhands/directory (replaced by opencode)Files Changed: 74 files
See git diff for full details
Changelog