fix(backup): reject --keep 0 so the new dump is never pruned

With --keep 0, prune ran tail -n +1 over the newest-first listing and deleted
every dump, including the one written seconds earlier, then reported success.
The check also rejects leading zeros, which the $((KEEP + 1)) arithmetic would
parse as octal (08 and 09 abort the script after the dump).

agent/provider.go

@@ -203,9 +203,7 @@ func compactionConfig(windowTokens int) *compaction.Config {
 //	ARTEX_LLM_PROXY    = proxy URL        (optional; http/https/socks5)
 //	ANTHROPIC_API_KEY / OPENAI_API_KEY         = credentials (unused for ollama)
 func FromEnv() (Config, bool) {
-	// Case- and space-insensitive: "Ollama" used to miss every case below and
-	// fall through to anthropic, sending ANTHROPIC_API_KEY to the Ollama base URL.
-	prov := strings.ToLower(strings.TrimSpace(os.Getenv("ARTEX_LLM_PROVIDER")))
+	prov := os.Getenv("ARTEX_LLM_PROVIDER")
 	anthKey := os.Getenv("ANTHROPIC_API_KEY")
 	oaiKey := os.Getenv("OPENAI_API_KEY")
 

agent/provider_env_test.go

@@ -1,37 +0,0 @@
-package agent
-
-import (
-	"testing"
-
-	"github.com/Autumn-27/norma/llm"
-)
-
-// ARTEX_LLM_PROVIDER is matched regardless of case and surrounding spaces. A
-// miss used to fall through to anthropic: "Ollama" then sent ANTHROPIC_API_KEY
-// to the Ollama base URL.
-func TestFromEnvProviderIsCaseInsensitive(t *testing.T) {
-	t.Setenv("ANTHROPIC_API_KEY", "sk-ant-secret")
-	t.Setenv("OPENAI_API_KEY", "sk-oai")
-	t.Setenv("ARTEX_LLM_BASE_URL", "")
-	t.Setenv("ARTEX_LLM_MODEL", "")
-	for _, tc := range []struct {
-		env  string
-		want llm.Format
-		key  string
-	}{
-		{"Ollama", FormatOllama, ""},
-		{" OLLAMA ", FormatOllama, ""},
-		{"OpenAI", llm.FormatOpenAI, "sk-oai"},
-		{"OpenAI-Responses", llm.FormatOpenAIResponses, "sk-oai"},
-		{"Anthropic", llm.FormatAnthropic, "sk-ant-secret"},
-	} {
-		t.Setenv("ARTEX_LLM_PROVIDER", tc.env)
-		c, ok := FromEnv()
-		if !ok {
-			t.Fatalf("%q: FromEnv returned ok=false", tc.env)
-		}
-		if c.Format != tc.want || c.APIKey != tc.key {
-			t.Fatalf("%q: format=%v key=%q, want format=%v key=%q", tc.env, c.Format, c.APIKey, tc.want, tc.key)
-		}
-	}
-}

scripts/backup.sh

@@ -31,7 +31,9 @@ while [ $# -gt 0 ]; do
     *) echo "unknown option: $1" >&2; usage >&2; exit 2 ;;
   esac
 done
-case "$KEEP" in (*[!0-9]*|'') echo "--keep must be a positive integer" >&2; exit 2 ;; esac
+# 0* rejects 0, which would make prune delete the dump this run just wrote, and
+# leading zeros, which $((KEEP + 1)) would read as octal.
+case "$KEEP" in (*[!0-9]*|''|0*) echo "--keep must be a positive integer" >&2; exit 2 ;; esac
 
 # Compose reads .env for credentials; source it for the dump command too.
 if [ -f .env ]; then