Merge pull request #2 from Martian-Engineering/feat/pb-update-parent

feat(pb): support --parent in update

Author: jalehman

.pebbles/events.jsonl

@@ -292,3 +292,6 @@
 {"type":"create","timestamp":"2026-01-21T17:11:10.851111Z","issue_id":"pb-f42","payload":{"description":"Update CHANGELOG for 0.4.0, tag release, and run scripts/release.sh v0.4.0.","priority":"1","title":"Cut v0.4.0 release","type":"task"}}
 {"type":"status_update","timestamp":"2026-01-21T17:11:15.116199Z","issue_id":"pb-f42","payload":{"status":"in_progress"}}
 {"type":"close","timestamp":"2026-01-21T17:12:16.63905Z","issue_id":"pb-f42","payload":{}}
+{"type":"create","timestamp":"2026-01-23T22:04:09.803613Z","issue_id":"pb-533","payload":{"description":"Support --parent on pb update (set/clear parent), validate parent exists and not self, record event, reflect in model and show/list output, update help/docs.","priority":"2","title":"Add parent flag to pb update","type":"task"}}
+{"type":"status_update","timestamp":"2026-01-23T22:04:13.49696Z","issue_id":"pb-533","payload":{"status":"in_progress"}}
+{"type":"close","timestamp":"2026-01-24T01:44:50.295883Z","issue_id":"pb-533","payload":{}}

README.md

@@ -74,6 +74,9 @@ pb version
 
 # Update status and fields
 pb update pb-abc --status in_progress --type bug --priority P1 --description "Investigate regressions"
+# Set or clear a parent
+pb update pb-abc --parent pb-epic
+pb update pb-abc --parent none
 
 # Close an issue
 pb close pb-abc

cmd/pb/help_text.go

@@ -150,20 +150,24 @@ Usage:
   pb update <id> --status in_progress
   pb update <id> --type bug --priority P1
   pb update --description "Updated scope" <id>
+  pb update <id> --parent pb-epic
 
 Flags:
   --status <status>      New status (open, in_progress, closed). Example: --status in_progress
   --type <type>          Replace issue type (free-form). Example: --type chore
   --description <text>   Replace description (Markdown ok). Example: --description "New details"
   --priority <P0-P4>     Replace priority (P0-P4 or 0-4). Example: --priority P0
+  --parent <id|none>     Replace parent issue. Example: --parent pb-epic
 
 Details:
   - You can update multiple fields in one command.
   - Setting status to closed sets closed_at; other statuses clear closed_at.
+  - Clear the parent with --parent none (or --parent "").
 
 Workflows:
   - Start work: pb update <id> --status in_progress
   - Raise priority: pb update <id> --priority P1
+  - Set parent: pb update <id> --parent pb-epic
 `
 
 const closeHelp = `Close an issue.

cmd/pb/main.go

@@ -328,9 +328,11 @@ func runUpdate(root string, args []string) {
 	var issueType optionalString
 	var description optionalString
 	var priority optionalString
+	var parent optionalString
 	fs.Var(&issueType, "type", "New issue type")
 	fs.Var(&description, "description", "New description")
 	fs.Var(&priority, "priority", "New priority (P0-P4)")
+	fs.Var(&parent, "parent", "Replace parent issue (use \"none\" to clear)")
 	// Support `pb update <id> --status ...` by moving the id to the end.
 	if len(args) > 0 && !strings.HasPrefix(args[0], "-") {
 		args = append(args[1:], args[0])
@@ -343,7 +345,7 @@ func runUpdate(root string, args []string) {
 	if fs.NArg() != 1 {
 		exitError(fmt.Errorf("update requires issue id"))
 	}
-	if strings.TrimSpace(*status) == "" && !issueType.set && !description.set && !priority.set {
+	if strings.TrimSpace(*status) == "" && !issueType.set && !description.set && !priority.set && !parent.set {
 		exitError(fmt.Errorf("at least one field is required"))
 	}
 	if issueType.set && strings.TrimSpace(issueType.value) == "" {
@@ -354,16 +356,15 @@ func runUpdate(root string, args []string) {
 	}
 	id := fs.Arg(0)
 	// Confirm the issue exists in the cache.
-	if _, _, err := pebbles.GetIssue(root, id); err != nil {
+	issue, _, err := pebbles.GetIssue(root, id)
+	if err != nil {
 		exitError(err)
 	}
+	id = issue.ID
 	timestamp := pebbles.NowTimestamp()
+	var events []pebbles.Event
 	if strings.TrimSpace(*status) != "" {
-		event := pebbles.NewStatusEvent(id, *status, timestamp)
-		// Append the event and rebuild the cache for consistency.
-		if err := pebbles.AppendEvent(root, event); err != nil {
-			exitError(err)
-		}
+		events = append(events, pebbles.NewStatusEvent(id, *status, timestamp))
 	}
 	updatePayload := make(map[string]string)
 	if issueType.set {
@@ -380,7 +381,41 @@ func runUpdate(root string, args []string) {
 		updatePayload["priority"] = fmt.Sprintf("%d", parsed)
 	}
 	if len(updatePayload) > 0 {
-		event := pebbles.NewUpdateEvent(id, timestamp, updatePayload)
+		events = append(events, pebbles.NewUpdateEvent(id, timestamp, updatePayload))
+	}
+	if parent.set {
+		trimmedParent := strings.TrimSpace(parent.value)
+		clearParent := trimmedParent == "" || strings.EqualFold(trimmedParent, "none")
+		var parentIssue pebbles.Issue
+		if !clearParent {
+			parentIssue, _, err = pebbles.GetIssue(root, trimmedParent)
+			if err != nil {
+				exitError(err)
+			}
+			if parentIssue.ID == issue.ID {
+				exitError(fmt.Errorf("parent must be different from issue %s", issue.ID))
+			}
+		}
+		hierarchy, err := pebbles.GetIssueHierarchy(root, issue.ID)
+		if err != nil {
+			exitError(err)
+		}
+		for _, existingParent := range issueIDsFromIssues(hierarchy.Parents) {
+			events = append(events, pebbles.NewDepRemoveEvent(issue.ID, existingParent, pebbles.DepTypeParentChild, timestamp))
+		}
+		if !clearParent {
+			childID := issue.ID
+			if !pebbles.HasParentChildSuffix(parentIssue.ID, childID) {
+				childID, err = pebbles.NextChildIssueID(root, parentIssue.ID)
+				if err != nil {
+					exitError(err)
+				}
+				events = append(events, pebbles.NewRenameEvent(issue.ID, childID, timestamp))
+			}
+			events = append(events, pebbles.NewDepAddEvent(childID, parentIssue.ID, pebbles.DepTypeParentChild, timestamp))
+		}
+	}
+	for _, event := range events {
 		if err := pebbles.AppendEvent(root, event); err != nil {
 			exitError(err)
 		}
@@ -959,6 +994,13 @@ func printIssue(root string, issue pebbles.Issue, hierarchy pebbles.IssueHierarc
 	fmt.Println(header)
 	// Core metadata block.
 	fmt.Printf("Type: %s\n", renderIssueType(issue.IssueType))
+	if len(hierarchy.Parents) > 0 {
+		label := "Parent"
+		if len(hierarchy.Parents) > 1 {
+			label = "Parents"
+		}
+		fmt.Printf("%s: %s\n", label, strings.Join(issueIDsFromIssues(hierarchy.Parents), ", "))
+	}
 	fmt.Printf(
 		"Created: %s · Updated: %s\n\n",
 		formatDate(issue.CreatedAt),