implementing dependency management in the cli

internal/commands/issues/dependencies.go

@@ -0,0 +1,125 @@
+package issues
+
+import (
+	"fmt"
+
+	"github.com/LazyBachelor/LazyPM/internal/models"
+	"github.com/spf13/cobra"
+)
+
+const depCmdExample = `pm dep ISSUE-1
+pm dep view ISSUE-1
+pm dep add ISSUE-1 ISSUE-2
+pm dep remove ISSUE-1 ISSUE-2`
+
+// DepCmd manages dependencies for issues.
+// Default behavior: view dependencies of an issue.
+var DepCmd = &cobra.Command{
+	Use:     "dep [subcommand] [issue id]",
+	Short:   "Manage dependencies for an issue",
+	Long:    `Manage dependencies between issues.`,
+	Example: depCmdExample,
+
+	// This command runs only when no subcommand is provided.
+	Args:              cobra.ExactArgs(1),
+	ValidArgsFunction: completeIssues,
+	RunE:              runDepViewCmd,
+}
+
+var DepViewCmd = &cobra.Command{
+	Use:               "view [issue id]",
+	Short:             "View dependencies of an issue",
+	Aliases:          []string{"list", "show"},
+	Args:              cobra.ExactArgs(1),
+	ValidArgsFunction: completeIssues,
+	RunE:              runDepViewCmd,
+}
+
+var DepAddCmd = &cobra.Command{
+	Use:               "add <issue id> <depends-on id>",
+	Short:             "Add a dependency: issue depends on depends-on issue",
+	Args:              cobra.ExactArgs(2),
+	ValidArgsFunction: completeIssues,
+	RunE:              runDepAddCmd,
+}
+
+var DepRemoveCmd = &cobra.Command{
+	Use:               "remove <issue id> <depends-on id>",
+	Short:             "Remove an existing dependency",
+	Aliases:          []string{"rm"},
+	Args:              cobra.ExactArgs(2),
+	ValidArgsFunction: completeIssues,
+	RunE:              runDepRemoveCmd,
+}
+
+func init() {
+	DepCmd.Aliases = []string{"dependencies"}
+
+	DepCmd.AddCommand(DepViewCmd)
+	DepCmd.AddCommand(DepAddCmd)
+	DepCmd.AddCommand(DepRemoveCmd)
+}
+
+func runDepViewCmd(cmd *cobra.Command, args []string) error {
+	issueID := args[0]
+	app := AppFromContext(cmd.Context())
+
+	deps, err := app.Issues.GetDependencies(cmd.Context(), issueID)
+	if err != nil {
+		return fmt.Errorf("failed to load dependencies for %s: %w", issueID, err)
+	}
+
+	if len(deps) == 0 {
+		cmd.Printf("Issue %s has no dependencies.\n", issueID)
+		return nil
+	}
+
+	cmd.Printf("Dependencies for %s:\n", issueID)
+	for _, d := range deps {
+		if d == nil {
+			continue
+		}
+		if d.Title != "" {
+			cmd.Printf("- %s: %s\n", d.ID, d.Title)
+		} else {
+			cmd.Printf("- %s\n", d.ID)
+		}
+	}
+
+	return nil
+}
+
+func runDepAddCmd(cmd *cobra.Command, args []string) error {
+	issueID := args[0]
+	dependsOnID := args[1]
+
+	app := AppFromContext(cmd.Context())
+
+	dep := &models.Dependency{
+		IssueID:     issueID,
+		DependsOnID: dependsOnID,
+		Type:        models.DepBlocks,
+	}
+
+	if err := app.Issues.AddDependency(cmd.Context(), dep, "cli"); err != nil {
+		return fmt.Errorf("failed to add dependency: %s depends on %s: %w", issueID, dependsOnID, err)
+	}
+
+	cmd.Printf("Added dependency: %s depends on %s\n", issueID, dependsOnID)
+	return runDepViewCmd(cmd, []string{issueID})
+}
+
+func runDepRemoveCmd(cmd *cobra.Command, args []string) error {
+	issueID := args[0]
+	dependsOnID := args[1]
+
+	app := AppFromContext(cmd.Context())
+
+	if err := app.Issues.RemoveDependency(cmd.Context(), issueID, dependsOnID, "cli"); err != nil {
+		return fmt.Errorf("failed to remove dependency: %s depends on %s: %w", issueID, dependsOnID, err)
+	}
+
+	cmd.Printf("Removed dependency: %s no longer depends on %s\n", issueID, dependsOnID)
+	return runDepViewCmd(cmd, []string{issueID})
+}
+

pkg/repl/executor.go

@@ -28,6 +28,12 @@ func (r *REPL) execute(input string) (string, error) {
 	if after, ok := strings.CutPrefix(input, "pm"); ok {
 
 		if strings.Contains(strings.Split(after, " ")[1], "start") {
+		//trimmedAfter := strings.TrimSpace(after)
+		//parts := shellSplit(trimmedAfter)
+
+		// Prevent accidental attempts to run "pm start ..." inside the REPL.
+		// (The REPL uses `status/title/help` and shell execution instead.)
+		//if len(parts) > 1 && strings.Contains(parts[1], "start") {
 			return "Nice try👻", nil
 		}
 

pkg/repl/suggestions.go

@@ -31,6 +31,7 @@ var baseSuggestions = []prompt.Suggest{
 	{Text: "delete", Description: "Delete an issue by ID"},
 	{Text: "comment", Description: "Add a comment on an issue by ID"},
 	{Text: "comments", Description: "List comments on an issue by ID"},
+	{Text: "dep", Description: "Manage dependencies for an issue"},
 
 	{Text: "new", Description: "Alias for create command"},
 	{Text: "ls", Description: "Alias for list command"},
@@ -123,6 +124,15 @@ var sprintSubcommands = []prompt.Suggest{
 	{Text: "del", Description: "Alias for delete subcommand"},
 }
 
+var depSubcommands = []prompt.Suggest{
+	{Text: "view", Description: "View dependencies of an issue"},
+	{Text: "list", Description: "Alias for view"},
+	{Text: "show", Description: "Alias for view"},
+	{Text: "add", Description: "Add a dependency"},
+	{Text: "remove", Description: "Remove a dependency"},
+	{Text: "rm", Description: "Alias for remove"},
+}
+
 // statusValues is a list of prompt suggestions for status types
 var statusValues = []prompt.Suggest{
 	{Text: "open", Description: "Open status"},
@@ -195,6 +205,73 @@ func commandSuggestions(words []string) []prompt.Suggest {
 func flagSuggestions(cmd string, words []string, text string) []prompt.Suggest {
 	lastWord, prevWord := parseWords(words, text)
 
+	// Custom completion logic for `pm dep ...`
+	if cmd == "dep" {
+		switch len(words) {
+		case 1:
+			// `pm dep ` => next token could be a subcommand; default view is handled
+			// once the user types an issue id
+			if strings.HasSuffix(text, " ") {
+				return depSubcommands
+			}
+			return nil
+
+		case 2:
+			// either:
+			// - pm dep <issueID> (default view)
+			// - pm dep <subcommand>
+			if strings.HasSuffix(text, " ") {
+				sub := words[1]
+				switch sub {
+				case "view", "list", "show", "add", "remove", "rm":
+					// After a valid subcommand we expect the issue id next.
+					return issueIDSuggestions("", true)
+				default:
+					// Treat it as an issue id already; no more suggestions.
+					return nil
+				}
+			}
+
+			// Typing the 2nd token: suggest subcommands and issue IDs that match.
+			suggests := make([]prompt.Suggest, 0, len(depSubcommands))
+			suggests = append(suggests, filterByPrefix(depSubcommands, lastWord)...)
+			//suggests = append(suggests, issueIDSuggestions(lastWord, true)...)
+			return suggests
+
+		case 3:
+			// If subcommand is `add/remove`, third token is the issue id.
+			// If subcommand is show/list, third token is the issue id.
+			sub := words[1]
+			switch sub {
+			case "add", "remove", "rm":
+				if strings.HasSuffix(text, " ") {
+					// `pm dep add <issueID> ` => next token is dependsOnID.
+					return issueIDSuggestions("", true)
+				}
+				return issueIDSuggestions(lastWord, true)
+			default:
+				// show/list/show => completing the issue id.
+				if strings.HasSuffix(text, " ") {
+					return nil
+				}
+				return issueIDSuggestions(lastWord, true)
+			}
+
+		case 4:
+			// add/remove/rm have a 4th token: dependsOnID.
+			sub := words[1]
+			if sub == "add" || sub == "remove" || sub == "rm" {
+				if strings.HasSuffix(text, " ") {
+					return nil
+				}
+				return issueIDSuggestions(lastWord, true)
+			}
+			return nil
+		}
+
+		return nil
+	}
+
 	if values := getFlagValues(prevWord); values != nil {
 		return filterByPrefix(values, lastWord)
 	}