core-lead/molecule-core
1
0
Fork 0
forked from molecule-ai/molecule-core
Code Pull Requests Activity

fix(scheduler): strip CRLF from cron prompts on insert/update (closes #958)

Browse Source 
Windows CRLF in org-template prompt text caused empty agent responses
and phantom-producing detection. Strips \r at the handler level before
DB persist, plus a one-time migration to clean existing rows.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Hongming Wang 2026-04-18 07:45:14 -07:00
parent 0d538ab27a
commit 1dcdd01378
2 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
workspace-server/internal/handlers/schedules.go
View File

@ -5,6 +5,7 @@ import (
"encoding/json"
"log"
"net/http"
"strings"
"time"
"github.com/gin-gonic/gin"
@ -96,6 +97,10 @@ func (h *ScheduleHandler) Create(c *gin.Context) {
return
}
// Strip CRLF from prompts — org-template files committed on Windows
// inject \r\n, causing empty agent responses (issue #958).
body.Prompt = strings.ReplaceAll(body.Prompt, "\r", "")
if body.Timezone == "" {
body.Timezone = "UTC"
}
@ -161,6 +166,12 @@ func (h *ScheduleHandler) Update(c *gin.Context) {
return
}
// Strip CRLF from prompt if provided (issue #958).
if body.Prompt != nil {
clean := strings.ReplaceAll(*body.Prompt, "\r", "")
body.Prompt = &clean
}
// If cron_expr or timezone changed, revalidate and recompute next_run
var nextRunAt *time.Time
if body.CronExpr != nil || body.Timezone != nil {

3
workspace-server/migrations/033_strip_crlf_cron_prompts.up.sql Normal file
View File

@ -0,0 +1,3 @@
-- Issue #958: Strip CRLF from cron prompts inserted from Windows org-template files.
-- Carriage returns cause empty agent responses and phantom-producing detection.
UPDATE workspace_schedules SET prompt = REPLACE(prompt, E'\r', '') WHERE prompt LIKE E'%\r%';