perf: use metadata-only Find for single-fiber commands

Commands that locate a fiber by ID prefix (edit, tag, comment, unlink,
show) now use FindMetadata + targeted Read instead of full-parse Find,
avoiding unnecessary body parsing for the lookup step.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: cailmdaley

cmd/add.go

@@ -59,20 +59,21 @@ var addCmd = &cobra.Command{
 			}
 		}
 		if len(addDeps) > 0 {
+			felts, err := storage.ListMetadata()
+			if err != nil {
+				return err
+			}
+
 			// Resolve dependency IDs
 			for _, dep := range addDeps {
-				depFelt, err := storage.Find(dep)
+				depFelt, err := felt.FindByPrefix(felts, dep)
 				if err != nil {
 					return fmt.Errorf("dependency %q: %w", dep, err)
 				}
 				f.DependsOn = append(f.DependsOn, felt.Dependency{ID: depFelt.ID})
 			}
 
 			// Check for cycles
-			felts, err := storage.ListMetadata()
-			if err != nil {
-				return err
-			}
 			// Add the new felt temporarily for cycle check
 			felts = append(felts, f)
 			g := felt.BuildGraph(felts)

cmd/edit.go

@@ -38,7 +38,11 @@ Examples:
 		}
 
 		storage := felt.NewStorage(root)
-		f, err := storage.Find(args[0])
+		target, err := storage.FindMetadata(args[0])
+		if err != nil {
+			return err
+		}
+		f, err := storage.Read(target.ID)
 		if err != nil {
 			return err
 		}
@@ -108,9 +112,14 @@ Examples:
 			}
 		}
 		if cmd.Flags().Changed("depends-on") {
+			felts, err := storage.ListMetadata()
+			if err != nil {
+				return err
+			}
+
 			// Resolve and add dependencies
 			for _, dep := range editDeps {
-				depFelt, err := storage.Find(dep)
+				depFelt, err := felt.FindByPrefix(felts, dep)
 				if err != nil {
 					return fmt.Errorf("dependency %q: %w", dep, err)
 				}
@@ -125,10 +134,6 @@ Examples:
 			}
 
 			// Check for cycles
-			felts, err := storage.ListMetadata()
-			if err != nil {
-				return err
-			}
 			g := felt.BuildGraph(felts)
 			g.Nodes[f.ID] = f
 			g.Upstream[f.ID] = f.DependsOn
@@ -167,7 +172,11 @@ var commentCmd = &cobra.Command{
 		}
 
 		storage := felt.NewStorage(root)
-		f, err := storage.Find(args[0])
+		target, err := storage.FindMetadata(args[0])
+		if err != nil {
+			return err
+		}
+		f, err := storage.Read(target.ID)
 		if err != nil {
 			return err
 		}
@@ -256,12 +265,16 @@ var unlinkCmd = &cobra.Command{
 
 		storage := felt.NewStorage(root)
 
-		f, err := storage.Find(args[0])
+		source, err := storage.FindMetadata(args[0])
+		if err != nil {
+			return fmt.Errorf("source: %w", err)
+		}
+		f, err := storage.Read(source.ID)
 		if err != nil {
 			return fmt.Errorf("source: %w", err)
 		}
 
-		dep, err := storage.Find(args[1])
+		dep, err := storage.FindMetadata(args[1])
 		if err != nil {
 			return fmt.Errorf("dependency: %w", err)
 		}

cmd/show.go

@@ -52,17 +52,25 @@ Detail levels control progressive disclosure:
 			return outputJSON(f)
 		}
 
-		// Full render path: single List(), find target in list, build graph
-		felts, err := storage.List()
+		felts, err := storage.ListMetadata()
 		if err != nil {
 			return err
 		}
-		f, err := felt.FindByPrefix(felts, args[0])
+		target, err := felt.FindByPrefix(felts, args[0])
 		if err != nil {
 			return err
 		}
 		graph := felt.BuildGraph(felts)
 
+		f := target
+		if detail == DepthSummary || detail == DepthFull {
+			f, err = storage.Read(target.ID)
+			if err != nil {
+				return err
+			}
+			graph.Nodes[f.ID] = f
+		}
+
 		fmt.Print(renderFelt(f, graph, detail))
 		return nil
 	},

cmd/tag.go

@@ -20,7 +20,11 @@ var tagCmd = &cobra.Command{
 		}
 
 		storage := felt.NewStorage(root)
-		f, err := storage.Find(args[0])
+		target, err := storage.FindMetadata(args[0])
+		if err != nil {
+			return err
+		}
+		f, err := storage.Read(target.ID)
 		if err != nil {
 			return err
 		}
@@ -54,7 +58,11 @@ var untagCmd = &cobra.Command{
 		}
 
 		storage := felt.NewStorage(root)
-		f, err := storage.Find(args[0])
+		target, err := storage.FindMetadata(args[0])
+		if err != nil {
+			return err
+		}
+		f, err := storage.Read(target.ID)
 		if err != nil {
 			return err
 		}

internal/felt/storage.go

@@ -72,6 +72,12 @@ func (s *Storage) ReadMetadata(id string) (*Felt, error) {
 	return s.readWithMode(id, ParseMetadataOnly)
 }
 
+// FindMetadata returns the first felt matching the ID prefix or hex suffix,
+// reading only metadata from the matching file.
+func (s *Storage) FindMetadata(query string) (*Felt, error) {
+	return s.findWithMode(query, ParseMetadataOnly)
+}
+
 func (s *Storage) readWithMode(id string, mode ParseMode) (*Felt, error) {
 	path := s.Path(id)
 	if mode == ParseMetadataOnly {
@@ -150,6 +156,10 @@ func (s *Storage) listWithMode(mode ParseMode, includeModTime bool) ([]*Felt, er
 // Find returns the first felt matching the ID prefix or hex suffix.
 // Uses ReadDir + filename matching to avoid reading all files.
 func (s *Storage) Find(query string) (*Felt, error) {
+	return s.findWithMode(query, ParseFull)
+}
+
+func (s *Storage) findWithMode(query string, mode ParseMode) (*Felt, error) {
 	entries, err := os.ReadDir(s.root)
 	if err != nil {
 		return nil, fmt.Errorf("reading directory: %w", err)
@@ -176,7 +186,7 @@ func (s *Storage) Find(query string) (*Felt, error) {
 	case 0:
 		return nil, fmt.Errorf("no felt found matching %q", query)
 	case 1:
-		f, err := s.Read(matchIDs[0])
+		f, err := s.readWithMode(matchIDs[0], mode)
 		if err != nil {
 			return nil, err
 		}

internal/felt/storage_test.go

@@ -249,6 +249,41 @@ func TestStorageListMetadataWithModTimePopulatesModifiedAt(t *testing.T) {
 	}
 }
 
+func TestStorageFindMetadataSkipsBody(t *testing.T) {
+	dir := t.TempDir()
+	s := NewStorage(dir)
+	s.Init()
+
+	f := &Felt{
+		ID:        "test-task-12345678",
+		Title:     "Test Task",
+		Status:    StatusOpen,
+		CreatedAt: time.Now(),
+		Outcome:   "Metadata survives",
+		Body:      "Body should be skipped.",
+	}
+	if err := s.Write(f); err != nil {
+		t.Fatalf("Write() error: %v", err)
+	}
+
+	found, err := s.FindMetadata("1234")
+	if err != nil {
+		t.Fatalf("FindMetadata() error: %v", err)
+	}
+	if found.ID != f.ID {
+		t.Errorf("ID = %q, want %q", found.ID, f.ID)
+	}
+	if found.Outcome != f.Outcome {
+		t.Errorf("Outcome = %q, want %q", found.Outcome, f.Outcome)
+	}
+	if found.Body != "" {
+		t.Errorf("Body = %q, want empty", found.Body)
+	}
+	if found.ModifiedAt.IsZero() {
+		t.Fatal("ModifiedAt should be populated for FindMetadata")
+	}
+}
+
 func TestReadFrontmatter(t *testing.T) {
 	content := strings.NewReader(`---
 title: Test Task