Function TestHamrpassRejectsControlChars

internal/tui/tui_test.go:3049–3079

TestHamrpassRejectsControlChars: a key with an embedded escape / NUL / DEL must be rejected by hamrpassValidate, not persisted, else every dial-out errors with net/http's cryptic "invalid header field value for Authorization". Real hamrpass keys are ASCII-printable.

(t *testing.T)

func TestFailureStreakSurvivesBatchedSuccess(t *testing.T) {
	m := newTestModel(t, func(http.ResponseWriter, *http.Request) {})
	for range maxToolFailStreak {
		m.lastToolKey = "edit_file|src/app.js"
		m.recordToolOutcome(tools.EditFileName, "(not found: old_string does not appear in src/app.js)")
		m.lastToolKey = "read_file|src/app.js"
		m.recordToolOutcome(tools.ReadFileName, "some file content")
	}
	if m.failStreak < maxToolFailStreak {
		t.Fatalf("interleaved READ success reset the streak: got %d, want >= %d", m.failStreak, maxToolFailStreak)
	}
	m.maybeFailureNudge()
	if n := countSystem(m.history); n != 1 {
		t.Fatalf("a same target failure streak must nudge, got %d system notes", n)
	}
	// A success on the FAILING target still clears it: that is real recovery.
	m.lastToolKey = "edit_file|src/app.js"
	m.failKey, m.failStreak = "edit_file|src/app.js", 3
	m.recordToolOutcome(tools.EditFileName, "edited src/app.js")
	if m.failStreak != 0 {
		t.Fatalf("a success on the failing target must clear the streak, got %d", m.failStreak)
	}
}

// TestVerifyNudgeSkipsReadOnlyTurn: a turn that only read files produced no
// artifact that could be falsely called green, so re grounding it is a wasted
// round trip on every research question.
func TestVerifyNudgeSkipsReadOnlyTurn(t *testing.T) {
	m := newTestModel(t, func(http.ResponseWriter, *http.Request) {})
	m.toolRounds = verifyNudgeMinRounds + 20
	m.llmRounds = verifyNudgeMinRounds + 20
	m.turnActed = false
	m.history = []chmctx.Message{
		{Role: chmctx.RoleUser, Content: "how does packing work?"},
		{Role: chmctx.RoleAssistant, Content: "Pack walks newest-first into Budget()."},
	}