Fill: refuse an empty rectangle; list plops whose plant is gone unnamed

A blank region name with a zero-area Region reached hexCenters, whose tiny-region rule plants one plop in the middle — a caller that said nothing about where got a plop at the centre. ListObjectPlantings also failed the whole listing if one plop's plant no longer existed; it now lists that plop unnamed. Co-Authored-By: Claude Fable 5 <[email protected]>
2026-08-23 00:16:54 -04:00
parent bc14bbed0d
commit a1baf4b871
2 changed files with 16 additions and 3 deletions
@@ -218,6 +218,10 @@ func (s *Service) Fill(ctx context.Context, actorID, objectID int64, spec FillSp
 		if region, err = NamedRegion(o, spec.RegionName); err != nil {
 			return nil, err
 		}
+	} else if !(region.MinX < region.MaxX && region.MinY < region.MaxY) {
+		// A zero or inverted rectangle is a caller that said nothing about where
+		// — not a request for the one plop hexCenters would put at its middle.
+		return nil, fmt.Errorf("%w: the fill rectangle is empty", domain.ErrInvalidInput)
 	}
 	return s.fillLoaded(ctx, actorID, o, region, spec)
 }
@@ -673,11 +677,13 @@ func (s *Service) ListObjectPlantings(ctx context.Context, actorID, objectID int
 		plant, ok := plants[pl.PlantID]
 		if !ok {
 			p, err := s.store.GetPlant(ctx, pl.PlantID)
-			if err != nil {
+			if err != nil && !errors.Is(err, domain.ErrNotFound) {
 				return nil, err
 			}
-			plant = *p
-			plants[pl.PlantID] = plant
+			if p != nil {
+				plant = *p
+			}
+			plants[pl.PlantID] = plant // a plant that no longer exists lists unnamed, not as an error
 		}
 		pl.DerivedCount = derivedCount(pl.RadiusCM, plant.SpacingCM)
 		out = append(out, describePlanting(pl, plant.Name))