Stop suggesting start --dry-run when lint errors block import.

Dry runs go through the same lint gate as real imports, so the
lint-blocked remediation and next_steps pointed at a command that
could not succeed. Blocked lint now suggests re-running lint after
fixing errors instead.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: no-itsbackpack

internal/import/d1/errors.go

@@ -24,7 +24,7 @@ const (
 const (
 	wranglerMissingRemediation = "Install wrangler, use npx wrangler d1 export, or pass --input if you already have a dump."
 	pgloaderInstallRemediation = "Install pgloader (brew install pgloader on macOS; see https://pgloader.readthedocs.io/en/latest/install.html for other platforms)"
-	lintBlockedRemediation     = "Fix lint errors or run `pscale import d1 lint` for details; use `import d1 start --dry-run` for a read-only preview"
+	lintBlockedRemediation     = "Fix the reported lint errors, then re-run; `pscale import d1 lint` shows details. Import (including --dry-run) is blocked until lint passes"
 )
 
 type MigrationError struct {

internal/import/d1/lint.go

@@ -186,17 +186,21 @@ func isJSONText(col ColumnSchema) bool {
 
 // LintNextSteps returns agent next steps based on lint results.
 func LintNextSteps(result *LintResult) []NextStep {
-	steps := []NextStep{
+	input := filepath.Base(result.InputPath)
+	if result.ErrorCount > 0 {
+		return []NextStep{{
+			Command: "pscale import d1 lint --input " + input,
+			Reason:  "Re-run lint after fixing the reported errors; import (including --dry-run) is blocked until lint passes",
+		}}
+	}
+	return []NextStep{
 		{
-			Command: "pscale import d1 start <database> --input " + filepath.Base(result.InputPath) + " --dry-run",
+			Command: "pscale import d1 start <database> --input " + input + " --dry-run",
 			Reason:  "Preview import plan and get a migration ID",
 		},
-	}
-	if result.ErrorCount == 0 {
-		steps = append(steps, NextStep{
-			Command: "pscale import d1 start <database> --input " + filepath.Base(result.InputPath),
+		{
+			Command: "pscale import d1 start <database> --input " + input,
 			Reason:  "Run import after lint passes",
-		})
+		},
 	}
-	return steps
 }

internal/import/d1/lint_test.go

@@ -92,6 +92,28 @@ func TestLint(t *testing.T) {
 	}
 }
 
+func TestLintNextStepsBlockedOnErrorsDoesNotSuggestStart(t *testing.T) {
+	blocked := &LintResult{InputPath: "/tmp/export.sql", ErrorCount: 1}
+	steps := LintNextSteps(blocked)
+	if len(steps) != 1 {
+		t.Fatalf("steps = %d, want 1", len(steps))
+	}
+	if !strings.Contains(steps[0].Command, "import d1 lint") {
+		t.Fatalf("blocked next step should re-run lint, got %q", steps[0].Command)
+	}
+
+	clean := &LintResult{InputPath: "/tmp/export.sql"}
+	steps = LintNextSteps(clean)
+	if len(steps) != 2 {
+		t.Fatalf("steps = %d, want 2", len(steps))
+	}
+	for _, step := range steps {
+		if !strings.Contains(step.Command, "import d1 start") {
+			t.Fatalf("clean next steps should suggest start, got %q", step.Command)
+		}
+	}
+}
+
 func TestPlan(t *testing.T) {
 	plan, err := Plan(PlanOptions{
 		InputPath: testFixture(t),