mirror of
https://github.com/affaan-m/ECC.git
synced 2026-08-26 01:12:24 +02:00
Gaurav Dubey
9d1ecb0754
The `.opencode/commands/*.md` frontmatter referenced agents with the Claude Code plugin namespace (`agent: everything-claude-code:<name>`), but ECC's opencode integration registers its agents unscoped in `opencode.json`'s `agent` map (`code-reviewer`, `planner`, ...), and that file's own `command` section already references them unscoped. The `everything-claude-code:` scope resolves under no opencode config (the opencode plugin package is `ecc-universal`, and inline-config agents are bare), so subtask commands like `/code-review` hard-fail with `Agent not found: everything-claude-code:code-reviewer`. Non-subtask commands fall back to the default agent and appear to work — which is why only some commands failed. Strip the `everything-claude-code:` prefix from all 30 command frontmatter agent ids so they match the registered agents, fix the MIGRATION.md example, and replace the test that enforced the broken scoped invariant with one that asserts each command agent id is a registered opencode agent (fails on the old scoped ids, passes on the fix). Fixes #2477
88 lines
1.5 KiB
Markdown
88 lines
1.5 KiB
Markdown
---
|
|
description: Fix Go build and vet errors
|
|
agent: go-build-resolver
|
|
subtask: true
|
|
---
|
|
|
|
# Go Build Command
|
|
|
|
Fix Go build, vet, and compilation errors: $ARGUMENTS
|
|
|
|
## Your Task
|
|
|
|
1. **Run go build**: `go build ./...`
|
|
2. **Run go vet**: `go vet ./...`
|
|
3. **Fix errors** one by one
|
|
4. **Verify fixes** don't introduce new errors
|
|
|
|
## Common Go Errors
|
|
|
|
### Import Errors
|
|
```
|
|
imported and not used: "package"
|
|
```
|
|
**Fix**: Remove unused import or use `_` prefix
|
|
|
|
### Type Errors
|
|
```
|
|
cannot use x (type T) as type U
|
|
```
|
|
**Fix**: Add type conversion or fix type definition
|
|
|
|
### Undefined Errors
|
|
```
|
|
undefined: identifier
|
|
```
|
|
**Fix**: Import package, define variable, or fix typo
|
|
|
|
### Vet Errors
|
|
```
|
|
printf: call has arguments but no formatting directives
|
|
```
|
|
**Fix**: Add format directive or remove arguments
|
|
|
|
## Fix Order
|
|
|
|
1. **Import errors** - Fix or remove imports
|
|
2. **Type definitions** - Ensure types exist
|
|
3. **Function signatures** - Match parameters
|
|
4. **Vet warnings** - Address static analysis
|
|
|
|
## Build Commands
|
|
|
|
```bash
|
|
# Build all packages
|
|
go build ./...
|
|
|
|
# Build with race detector
|
|
go build -race ./...
|
|
|
|
# Build for specific OS/arch
|
|
GOOS=linux GOARCH=amd64 go build ./...
|
|
|
|
# Run go vet
|
|
go vet ./...
|
|
|
|
# Run staticcheck
|
|
staticcheck ./...
|
|
|
|
# Format code
|
|
gofmt -w .
|
|
|
|
# Tidy dependencies
|
|
go mod tidy
|
|
```
|
|
|
|
## Verification
|
|
|
|
After fixes:
|
|
```bash
|
|
go build ./... # Should succeed
|
|
go vet ./... # Should have no warnings
|
|
go test ./... # Tests should pass
|
|
```
|
|
|
|
---
|
|
|
|
**IMPORTANT**: Fix errors only. No refactoring, no improvements. Get the build green with minimal changes.
|