make lookups faster

cmd/plcbundle/commands/did.go

···
 
 func newDIDResolveCommand() *cobra.Command {
 	var (
-		verbose    bool
-		showTiming bool
-		raw        bool
+		verbose bool
+		raw     bool
 	)
 
 	cmd := &cobra.Command{
···
 		Example: `  # Resolve DID
   plcbundle did resolve did:plc:524tuhdhh3m7li5gycdn6boe
 
-  # Show timing breakdown
-  plcbundle did resolve did:plc:524tuhdhh3m7li5gycdn6boe --timing
+  # Show timings and other details
+  plcbundle did resolve did:plc:524tuhdhh3m7li5gycdn6boe --verbose
 
   # Get raw PLC state (not W3C format)
   plcbundle did resolve did:plc:524tuhdhh3m7li5gycdn6boe --raw
···
 
 			// Show resolution timing if it was a handle
 			if input != did {
-				if showTiming {
+				if verbose {
 					fmt.Fprintf(os.Stderr, "Handle resolution: %s → %s (%s)\n",
 						input, did, handleResolveTime)
 				} else {
···
 				}
 			}
 
-			if showTiming {
+			if verbose {
 				fmt.Fprintf(os.Stderr, "Resolving DID: %s\n", did)
-			}
-
-			if verbose {
 				mgr.GetDIDIndex().SetVerbose(true)
 			}
 
···
 			}
 
 			// Display timing if requested
-			if showTiming {
+			if verbose {
 				if handleResolveTime > 0 {
 					fmt.Fprintf(os.Stderr, "Handle: %s | ", handleResolveTime)
 				}
···
 	}
 
 	cmd.Flags().BoolVarP(&verbose, "verbose", "v", false, "Verbose debug output")
-	cmd.Flags().BoolVar(&showTiming, "timing", false, "Show timing breakdown")
 	cmd.Flags().BoolVar(&raw, "raw", false, "Output raw PLC state (not W3C document)")
 
 	return cmd

internal/didindex/bundle.go

···
 type BundleProvider interface {
 	LoadBundleForDIDIndex(ctx context.Context, bundleNumber int) (*BundleData, error)
 	LoadOperation(ctx context.Context, bundleNumber int, position int) (*plcclient.PLCOperation, error)
+	LoadOperations(ctx context.Context, bundleNumber int, positions []int) (map[int]*plcclient.PLCOperation, error)
 	GetBundleIndex() BundleIndexProvider
 }
 

internal/didindex/lookup.go

···
 
 	var results []OpLocationWithOperation
 	for bundleNum, locs := range bundleMap {
-		bundle, err := provider.LoadBundleForDIDIndex(ctx, int(bundleNum))
+		positions := make([]int, len(locs))
+		for i, l := range locs {
+			positions[i] = l.PositionInt()
+		}
+		opsMap, err := provider.LoadOperations(ctx, int(bundleNum), positions)
 		if err != nil {
 			dim.logger.Printf("Warning: failed to load bundle %d: %v", bundleNum, err)
 			continue
 		}
-
-		for _, loc := range locs {
-			if loc.PositionInt() >= len(bundle.Operations) {
-				continue
+		for i, l := range locs {
+			if op, ok := opsMap[positions[i]]; ok {
+				results = append(results, OpLocationWithOperation{
+					Operation: *op,
+					Bundle:    l.BundleInt(),
+					Position:  l.PositionInt(),
+				})
 			}
-
-			op := bundle.Operations[loc.Position()]
-			results = append(results, OpLocationWithOperation{
-				Operation: op,
-				Bundle:    loc.BundleInt(),
-				Position:  loc.PositionInt(),
-			})
 		}
 	}