opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: replace Go-hardcoded commands in interview template with generic placeholders

Browse source 
The Target Format in GetInterviewContext() had Go-specific examples
(go build, go test, golangci-lint) which biased the AI to use them
even for non-Go projects (e.g. Python with uv run).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
dj-oyu 2026-02-22 04:19:55 +09:00
parent 82818b9fb2
commit 54616aaa52
2 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
pkg/agent/memory.go
View file

@ -380,9 +380,9 @@ func (ms *MemoryStore) GetInterviewContext() string {
sb.WriteString("## Phase 2: <title>\n") sb.WriteString("## Phase 2: <title>\n")
sb.WriteString("- [ ] Step\n") sb.WriteString("- [ ] Step\n")
sb.WriteString("## Commands\n") sb.WriteString("## Commands\n")
sb.WriteString("build: go build ./...\n") sb.WriteString("build: <project-specific build command>\n")
sb.WriteString("test: go test ./pkg/... -count=1\n") sb.WriteString("test: <project-specific test command>\n")
sb.WriteString("lint: golangci-lint run\n") sb.WriteString("lint: <project-specific lint command>\n")
sb.WriteString("## Context\n") sb.WriteString("## Context\n")
sb.WriteString("<collected requirements, decisions, environment>\n") sb.WriteString("<collected requirements, decisions, environment>\n")
sb.WriteString("```\n") sb.WriteString("```\n")

8
pkg/agent/memory_test.go
View file

@ -336,11 +336,11 @@ func TestGetInterviewContext(t *testing.T) {
if !strings.Contains(ctx, "## Commands") { if !strings.Contains(ctx, "## Commands") {
t.Error("expected target format to include ## Commands section") t.Error("expected target format to include ## Commands section")
} }
if !strings.Contains(ctx, "go test") { if !strings.Contains(ctx, "project-specific test command") {
t.Error("expected target format Commands to include test command example") t.Error("expected target format Commands to include test command placeholder")
} }
if !strings.Contains(ctx, "golangci-lint") { if !strings.Contains(ctx, "project-specific lint command") {
t.Error("expected target format Commands to include lint command example") t.Error("expected target format Commands to include lint command placeholder")
} }
} }