fix(lint): resolve wsl lint findings (openclaw#35)

Thanks @salmonumbrella.

Co-authored-by: salmonumbrella <salmonumbrella@users.noreply.github.com>

Author: Peter Steinberger

internal/input/prompt_test.go

@@ -23,6 +23,7 @@ func TestPromptLineFrom(t *testing.T) {
 	if err != nil {
 		t.Fatalf("PromptLineFrom: %v", err)
 	}
+
 	if line != "hello" {
 		t.Fatalf("unexpected line: %q", line)
 	}

internal/secrets/store_integration_test.go

@@ -25,6 +25,7 @@ func TestSetAndGetSecret_FileBackend(t *testing.T) {
 	if err := SetSecret("test/key", []byte("value")); err != nil {
 		t.Fatalf("SetSecret: %v", err)
 	}
+
 	if val, err := GetSecret("test/key"); err != nil {
 		t.Fatalf("GetSecret: %v", err)
 	} else if string(val) != "value" {
@@ -40,6 +41,7 @@ func TestKeyringStore_TokenRoundTrip(t *testing.T) {
 	if err := store.SetToken("a@b.com", tok); err != nil {
 		t.Fatalf("SetToken: %v", err)
 	}
+
 	if got, err := store.GetToken("a@b.com"); err != nil {
 		t.Fatalf("GetToken: %v", err)
 	} else if got.RefreshToken != "rt" {
@@ -50,6 +52,7 @@ func TestKeyringStore_TokenRoundTrip(t *testing.T) {
 	if err != nil {
 		t.Fatalf("Keys: %v", err)
 	}
+
 	if len(keys) == 0 {
 		t.Fatalf("expected keys")
 	}

internal/secrets/store_more_test.go

@@ -30,6 +30,7 @@ func TestKeyringStore_ListDeleteDefault(t *testing.T) {
 	if err != nil {
 		t.Fatalf("ListTokens: %v", err)
 	}
+
 	if len(tokens) != 2 {
 		t.Fatalf("expected 2 tokens, got %d", len(tokens))
 	}
@@ -38,20 +39,19 @@ func TestKeyringStore_ListDeleteDefault(t *testing.T) {
 	if err != nil {
 		t.Fatalf("DeleteToken: %v", err)
 	}
-	_, err = store.GetToken(tok1.Email)
-	if err == nil {
+
+	if _, getErr := store.GetToken(tok1.Email); getErr == nil {
 		t.Fatalf("expected error for deleted token")
 	}
 
 	err = store.SetDefaultAccount("a@b.com")
 	if err != nil {
 		t.Fatalf("SetDefaultAccount: %v", err)
 	}
-	def, err := store.GetDefaultAccount()
-	if err != nil {
+
+	if def, err := store.GetDefaultAccount(); err != nil {
 		t.Fatalf("GetDefaultAccount: %v", err)
-	}
-	if def != "a@b.com" {
+	} else if def != "a@b.com" {
 		t.Fatalf("unexpected default account: %q", def)
 	}
 
@@ -65,6 +65,7 @@ func TestParseTokenKey(t *testing.T) {
 	if email, ok := ParseTokenKey("token:a@b.com"); !ok || email != "a@b.com" {
 		t.Fatalf("unexpected parse: %q ok=%v", email, ok)
 	}
+
 	if _, ok := ParseTokenKey("nope"); ok {
 		t.Fatalf("expected invalid token key")
 	}
@@ -74,6 +75,7 @@ func TestAllowedBackends(t *testing.T) {
 	if _, err := allowedBackends(KeyringBackendInfo{Value: "keychain"}); err != nil {
 		t.Fatalf("keychain allowed: %v", err)
 	}
+
 	if _, err := allowedBackends(KeyringBackendInfo{Value: "file"}); err != nil {
 		t.Fatalf("file allowed: %v", err)
 	}
@@ -88,6 +90,7 @@ func TestWrapKeychainError(t *testing.T) {
 
 		return
 	}
+
 	if !errors.Is(wrapped, errTestKeychain) || wrapped.Error() != errTestKeychain.Error() {
 		t.Fatalf("expected passthrough error, got: %v", wrapped)
 	}

internal/tracking/deploy_test.go

@@ -119,6 +119,7 @@ func TestDeployWorker_Success(t *testing.T) {
 	if err != nil {
 		t.Fatalf("DeployWorker: %v", err)
 	}
+
 	if dbID != "db-create" {
 		t.Fatalf("unexpected db id: %q", dbID)
 	}
@@ -139,6 +140,7 @@ func TestEnsureD1Database_InfoFallback(t *testing.T) {
 	if err != nil {
 		t.Fatalf("ensureD1Database: %v", err)
 	}
+
 	if dbID != "db-info" {
 		t.Fatalf("unexpected db id: %q", dbID)
 	}
@@ -157,32 +159,43 @@ func TestWriteWranglerConfig(t *testing.T) {
 	if err != nil {
 		t.Fatalf("read config: %v", err)
 	}
+
 	content := string(data)
 	if !strings.Contains(content, "worker-name") || strings.Contains(content, "old") {
 		t.Fatalf("missing name replacement: %q", content)
 	}
+
 	if !strings.Contains(content, "db-name") {
 		t.Fatalf("missing database_name replacement: %q", content)
 	}
+
 	if !strings.Contains(content, "db-id") {
 		t.Fatalf("missing database_id replacement: %q", content)
 	}
 }
 
 func writeWranglerFiles(t *testing.T, dir string) {
 	t.Helper()
-	if err := os.WriteFile(filepath.Join(dir, "wrangler.toml"), []byte("name = \"old\"\ndatabase_name = \"old\"\ndatabase_id = \"old\"\n"), 0o600); err != nil {
+	wranglerPath := filepath.Join(dir, "wrangler.toml")
+
+	err := os.WriteFile(wranglerPath, []byte("name = \"old\"\ndatabase_name = \"old\"\ndatabase_id = \"old\"\n"), 0o600)
+	if err != nil {
 		t.Fatalf("write wrangler.toml: %v", err)
 	}
-	if err := os.WriteFile(filepath.Join(dir, "schema.sql"), []byte(""), 0o600); err != nil {
+
+	schemaPath := filepath.Join(dir, "schema.sql")
+
+	err = os.WriteFile(schemaPath, []byte(""), 0o600)
+	if err != nil {
 		t.Fatalf("write schema.sql: %v", err)
 	}
 }
 
 func writeWranglerStub(t *testing.T, dir string) string {
 	t.Helper()
 	path := filepath.Join(dir, "wrangler")
-	script := `#!/bin/sh
+
+	err := os.WriteFile(path, []byte(`#!/bin/sh
 set -e
 cmd="$1"
 shift
@@ -222,8 +235,8 @@ case "$cmd" in
 esac
 echo "unexpected args" >&2
 exit 2
-`
-	if err := os.WriteFile(path, []byte(script), 0o700); err != nil {
+`), 0o700)
+	if err != nil {
 		t.Fatalf("write wrangler stub: %v", err)
 	}
 

internal/tracking/secrets_test.go

@@ -22,10 +22,12 @@ func TestSaveAndLoadSecrets(t *testing.T) {
 	if err := SaveSecrets("a@b.com", "track", "admin"); err != nil {
 		t.Fatalf("SaveSecrets: %v", err)
 	}
+
 	track, admin, err := LoadSecrets("a@b.com")
 	if err != nil {
 		t.Fatalf("LoadSecrets: %v", err)
 	}
+
 	if track != "track" || admin != "admin" {
 		t.Fatalf("unexpected secrets: %q %q", track, admin)
 	}
@@ -37,6 +39,7 @@ func TestLoadSecrets_LegacyFallback(t *testing.T) {
 	if err := secrets.SetSecret(legacyTrackingKeySecretKey, []byte("legacy-track")); err != nil {
 		t.Fatalf("SetSecret legacy: %v", err)
 	}
+
 	if err := secrets.SetSecret(legacyAdminKeySecretKey, []byte("legacy-admin")); err != nil {
 		t.Fatalf("SetSecret legacy admin: %v", err)
 	}
@@ -45,6 +48,7 @@ func TestLoadSecrets_LegacyFallback(t *testing.T) {
 	if err != nil {
 		t.Fatalf("LoadSecrets: %v", err)
 	}
+
 	if track != "legacy-track" || admin != "legacy-admin" {
 		t.Fatalf("unexpected legacy secrets: %q %q", track, admin)
 	}

scripts/gen-auth-services-md_test.go

@@ -8,17 +8,17 @@ import (
 )
 
 func TestMainUpdatesReadme(t *testing.T) {
-	dir := t.TempDir()
 	orig, _ := os.Getwd()
+
+	dir := t.TempDir()
 	if err := os.Chdir(dir); err != nil {
 		t.Fatalf("chdir: %v", err)
 	}
 
 	t.Cleanup(func() { _ = os.Chdir(orig) })
 
 	readme := filepath.Join(dir, "README.md")
-	content := "# Test\n" + startMarker + "\n" + endMarker + "\n"
-	if err := os.WriteFile(readme, []byte(content), 0o600); err != nil {
+	if err := os.WriteFile(readme, []byte("# Test\n"+startMarker+"\n"+endMarker+"\n"), 0o600); err != nil {
 		t.Fatalf("write README: %v", err)
 	}
 
@@ -28,10 +28,12 @@ func TestMainUpdatesReadme(t *testing.T) {
 	if err != nil {
 		t.Fatalf("read README: %v", err)
 	}
+
 	text := string(updated)
 	if !strings.Contains(text, startMarker) || !strings.Contains(text, endMarker) {
 		t.Fatalf("missing markers: %q", text)
 	}
+
 	if !strings.Contains(text, "|") {
 		t.Fatalf("expected markdown table: %q", text)
 	}