summaryrefslogtreecommitdiff
path: root/internal/executor
diff options
context:
space:
mode:
authorPeter Stone <thepeterstone@gmail.com>2026-03-13 03:17:04 +0000
committerPeter Stone <thepeterstone@gmail.com>2026-03-13 03:17:04 +0000
commitfe414fac958330c2302d9175d66e1b338e5b1864 (patch)
treeef4941f5d01e84e7868e6b92bd0e6cecdcc2a64f /internal/executor
parentd5f83f8662c9f9c0fb52b206b06d4dd54a7788b4 (diff)
parent55c20922cc7a671787fe94fdd53a7eb72ebd2596 (diff)
merge: resolve conflicts with local/master (stats tab + summary styles)
Keep file-based summary approach (CLAUDOMATOR_SUMMARY_FILE) from HEAD. Combine Q&A History and Stats tab CSS from both branches. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/executor')
-rw-r--r--internal/executor/preamble.go1
-rw-r--r--internal/executor/preamble_test.go26
2 files changed, 26 insertions, 1 deletions
diff --git a/internal/executor/preamble.go b/internal/executor/preamble.go
index bc5c32c..5e57852 100644
--- a/internal/executor/preamble.go
+++ b/internal/executor/preamble.go
@@ -56,7 +56,6 @@ and the outcome. Write it to the path in $CLAUDOMATOR_SUMMARY_FILE:
This summary is displayed in the task UI so the user knows what happened.
---
-
`
func withPlanningPreamble(instructions string) string {
diff --git a/internal/executor/preamble_test.go b/internal/executor/preamble_test.go
new file mode 100644
index 0000000..448ad3a
--- /dev/null
+++ b/internal/executor/preamble_test.go
@@ -0,0 +1,26 @@
+package executor
+
+import (
+ "strings"
+ "testing"
+)
+
+func TestPlanningPreamble_ContainsFinalSummarySection(t *testing.T) {
+ if !strings.Contains(planningPreamble, "## Final Summary (mandatory)") {
+ t.Error("planningPreamble missing '## Final Summary (mandatory)' heading")
+ }
+}
+
+func TestPlanningPreamble_SummaryRequiresMarkdownHeader(t *testing.T) {
+ if !strings.Contains(planningPreamble, `Start it with "## Summary"`) {
+ t.Error("planningPreamble does not instruct agent to start summary with '## Summary'")
+ }
+}
+
+func TestPlanningPreamble_SummaryDescribesRequiredContent(t *testing.T) {
+ for _, phrase := range []string{"What was accomplished", "Key decisions made", "Any issues or follow-ups"} {
+ if !strings.Contains(planningPreamble, phrase) {
+ t.Errorf("planningPreamble missing required summary content description: %q", phrase)
+ }
+ }
+}