fix: editor opens when passing through e

- fixes #32
- failure due to continuation of subsequent steps rather than early exit
- test suite updated

Author: jamesjohnsdev

cmd/create.go

@@ -104,29 +104,31 @@ var createCmd = &cobra.Command{
 				openEditor = false
 			}
 
-			labelsResp, err := prompt("Labels (comma-separated, Enter to skip): ")
-			if err != nil {
-				return err
-			}
-			for _, l := range strings.Split(labelsResp, ",") {
-				if l := strings.TrimSpace(l); l != "" {
-					iss.Labels = append(iss.Labels, l)
+			if !openEditor {
+				labelsResp, err := prompt("Labels (comma-separated, Enter to skip): ")
+				if err != nil {
+					return err
+				}
+				for _, l := range strings.Split(labelsResp, ",") {
+					if l := strings.TrimSpace(l); l != "" {
+						iss.Labels = append(iss.Labels, l)
+					}
 				}
-			}
 
-			assigneesResp, err := prompt("Assignees (comma-separated, Enter to skip): ")
-			if err != nil {
-				return err
-			}
-			for _, a := range strings.Split(assigneesResp, ",") {
-				if a := strings.TrimSpace(a); a != "" {
-					iss.Assignees = append(iss.Assignees, a)
+				assigneesResp, err := prompt("Assignees (comma-separated, Enter to skip): ")
+				if err != nil {
+					return err
+				}
+				for _, a := range strings.Split(assigneesResp, ",") {
+					if a := strings.TrimSpace(a); a != "" {
+						iss.Assignees = append(iss.Assignees, a)
+					}
 				}
-			}
 
-			iss.Milestone, err = prompt("Milestone (Enter to skip): ")
-			if err != nil {
-				return err
+				iss.Milestone, err = prompt("Milestone (Enter to skip): ")
+				if err != nil {
+					return err
+				}
 			}
 
 		default:
@@ -153,7 +155,9 @@ var createCmd = &cobra.Command{
 			c.Stdin = os.Stdin
 			c.Stdout = os.Stdout
 			c.Stderr = os.Stderr
-			_ = c.Run()
+			if err := c.Run(); err != nil {
+				return fmt.Errorf("editor exited with error: %w", err)
+			}
 		}
 
 		if createEditorFlag {

cmd/create_test.go

@@ -255,7 +255,8 @@ func TestCreateInteractive(t *testing.T) {
 		t.Setenv("VISUAL", "")
 		t.Setenv("EDITOR", script)
 
-		injectStdin(t, "My Bug\ne\n\n\n\n")
+		// Only title + body "e" — no labels/assignees/milestone prompts should appear.
+		injectStdin(t, "My Bug\ne\n")
 		_ = captureStdout(t, func() {
 			if err := createCmd.RunE(createCmd, nil); err != nil {
 				t.Fatalf("unexpected error: %v", err)
@@ -275,6 +276,35 @@ func TestCreateInteractive(t *testing.T) {
 		}
 	})
 
+	t.Run("body e with editor skips labels assignees milestone prompts", func(t *testing.T) {
+		parent := makeProjectDir(t, nil)
+		chdirTo(t, parent)
+		resetCreateFlag(t)
+
+		script := filepath.Join(t.TempDir(), "editor.sh")
+		if err := os.WriteFile(script, []byte(
+			"#!/bin/sh\ncat > \"$1\" <<'EOF'\n---\ntitle: My Bug\nstate: open\n---\nEOF\n",
+		), 0755); err != nil {
+			t.Fatal(err)
+		}
+		t.Setenv("VISUAL", "")
+		t.Setenv("EDITOR", script)
+
+		// If Labels/Assignees/Milestone prompts were shown, this input would
+		// block waiting for three more lines. Completing without hanging
+		// confirms they are skipped.
+		injectStdin(t, "My Bug\ne\n")
+		_ = captureStdout(t, func() {
+			if err := createCmd.RunE(createCmd, nil); err != nil {
+				t.Fatalf("unexpected error: %v", err)
+			}
+		})
+
+		if files := readMDFiles(t, openIssuesDir(t, parent)); len(files) != 1 {
+			t.Errorf("expected 1 file, got %d", len(files))
+		}
+	})
+
 	t.Run("T-number increments from existing issues", func(t *testing.T) {
 		parent := makeProjectDir(t, []issueFixture{
 			{"T1-existing.md", issue.Issue{Title: "Existing", State: "open"}},