Unified Agent + reusable Go agent core.
test: remove brittle skill status tests
-98
-58
cmd/mistermorph/consolecmd/local_runtime_test.go
-58
cmd/mistermorph/consolecmd/local_runtime_test.go
···
5
5
"encoding/json"
6
6
"errors"
7
7
"fmt"
8
-
"os"
9
8
"path/filepath"
10
9
"strings"
11
10
"testing"
···
730
729
}
731
730
}
732
731
}
733
-
734
-
func TestConsoleLocalRuntimeSubmitTaskHandlesSkillCommand(t *testing.T) {
735
-
prevStateDir := viper.GetString("file_state_dir")
736
-
prevSkillsDirName := viper.GetString("skills.dir_name")
737
-
defer func() {
738
-
viper.Set("file_state_dir", prevStateDir)
739
-
viper.Set("skills.dir_name", prevSkillsDirName)
740
-
}()
741
-
stateDir := t.TempDir()
742
-
viper.Set("file_state_dir", stateDir)
743
-
viper.Set("skills.dir_name", "skills")
744
-
skillDir := filepath.Join(stateDir, "skills", "alpha")
745
-
if err := os.MkdirAll(skillDir, 0o755); err != nil {
746
-
t.Fatalf("mkdir skill dir: %v", err)
747
-
}
748
-
skillMD := "---\nname: alpha\ndescription: Alpha skill.\n---\n# Alpha\n"
749
-
if err := os.WriteFile(filepath.Join(skillDir, "SKILL.md"), []byte(skillMD), 0o644); err != nil {
750
-
t.Fatalf("write SKILL.md: %v", err)
751
-
}
752
-
753
-
store, err := daemonruntime.NewConsoleFileStore(daemonruntime.ConsoleFileStoreOptions{
754
-
HeartbeatTopicID: "_heartbeat",
755
-
Persist: false,
756
-
})
757
-
if err != nil {
758
-
t.Fatalf("NewConsoleFileStore() error = %v", err)
759
-
}
760
-
reader := viper.New()
761
-
reader.Set("skills.enabled", true)
762
-
generation := &consoleLocalRuntimeGeneration{reader: reader}
763
-
rt := &consoleLocalRuntime{
764
-
store: store,
765
-
generation: generation,
766
-
}
767
-
768
-
resp, err := rt.submitTask(context.Background(), daemonruntime.SubmitTaskRequest{
769
-
Task: "/skill",
770
-
})
771
-
if err != nil {
772
-
t.Fatalf("submitTask() error = %v", err)
773
-
}
774
-
if resp.Status != daemonruntime.TaskDone {
775
-
t.Fatalf("resp.Status = %q, want %q", resp.Status, daemonruntime.TaskDone)
776
-
}
777
-
task, ok := store.Get(resp.ID)
778
-
if !ok || task == nil {
779
-
t.Fatalf("store.Get(%q) missing", resp.ID)
780
-
}
781
-
result, _ := task.Result.(map[string]any)
782
-
final, _ := result["final"].(map[string]any)
783
-
output := strings.TrimSpace(fmt.Sprint(final["output"]))
784
-
for _, want := range []string{"**Loaded Skills (1)**", "* `alpha`:", "Alpha skill."} {
785
-
if !strings.Contains(output, want) {
786
-
t.Fatalf("final.output missing %q: %q", want, output)
787
-
}
788
-
}
789
-
}
-40
internal/skillsutil/status_test.go
-40
internal/skillsutil/status_test.go
···
5
5
"testing"
6
6
)
7
7
8
-
func TestRenderSkillStatusShowsLoadedAndAvailable(t *testing.T) {
9
-
root := t.TempDir()
10
-
writeSkill(t, root, "alpha")
11
-
writeSkill(t, root, "beta")
12
-
13
-
out, err := RenderSkillStatus(SkillsConfig{
14
-
Roots: []string{root},
15
-
Enabled: true,
16
-
Requested: []string{"alpha"},
17
-
}, nil)
18
-
if err != nil {
19
-
t.Fatalf("RenderSkillStatus() error = %v", err)
20
-
}
21
-
for _, want := range []string{"**Loaded Skills (1)**", "* `alpha`:", "**Available Skills (1)**", "* `beta`:"} {
22
-
if !strings.Contains(out, want) {
23
-
t.Fatalf("status missing %q:\n%s", want, out)
24
-
}
25
-
}
26
-
}
27
-
28
-
func TestRenderSkillStatusUsesCurrentLoadedSkills(t *testing.T) {
29
-
root := t.TempDir()
30
-
writeSkill(t, root, "alpha")
31
-
writeSkill(t, root, "beta")
32
-
33
-
out, err := RenderSkillStatus(SkillsConfig{
34
-
Roots: []string{root},
35
-
Enabled: true,
36
-
}, []string{"beta"})
37
-
if err != nil {
38
-
t.Fatalf("RenderSkillStatus() error = %v", err)
39
-
}
40
-
if !strings.Contains(out, "**Loaded Skills (1)**") || !strings.Contains(out, "* `beta`:") {
41
-
t.Fatalf("status missing current loaded skill:\n%s", out)
42
-
}
43
-
if !strings.Contains(out, "**Available Skills (1)**") || !strings.Contains(out, "* `alpha`:") {
44
-
t.Fatalf("status missing available skill:\n%s", out)
45
-
}
46
-
}
47
-
48
8
func TestRenderSkillStatusDisabled(t *testing.T) {
49
9
root := t.TempDir()
50
10
writeSkill(t, root, "alpha")