refactor(geom): rename sector() => asSector(), minor refactor

Author: postspectacular

packages/geom/package.json

@@ -235,6 +235,9 @@
 		"./as-polyline": {
 			"default": "./as-polyline.js"
 		},
+		"./as-sector": {
+			"default": "./as-sector.js"
+		},
 		"./as-svg": {
 			"default": "./as-svg.js"
 		},
@@ -406,9 +409,6 @@
 		"./scatter": {
 			"default": "./scatter.js"
 		},
-		"./sector": {
-			"default": "./sector.js"
-		},
 		"./simplify": {
 			"default": "./simplify.js"
 		},

packages/geom/src/sector.ts renamed to packages/geom/src/as-sector.ts

@@ -13,15 +13,15 @@ import { __copyAttribsRaw } from "./internal/copy.js";
  * @param arc
  * @param attribs
  */
-export const sector = (arc: Arc, attribs?: Attribs) => {
-	const path = new Path();
-	path.addSegments(
-		{ type: "m", point: copy(arc.pos) },
-		{ type: "l", geo: new Line([copy(arc.pos), arc.pointAt(0)]) },
-		{ type: "a", geo: arc },
-		{ type: "l", geo: new Line([arc.pointAt(1), copy(arc.pos)]) },
-		{ type: "z" }
+export const asSector = (arc: Arc, attribs?: Attribs) =>
+	new Path(
+		[
+			{ type: "m", point: copy(arc.pos) },
+			{ type: "l", geo: new Line([copy(arc.pos), arc.pointAt(0)]) },
+			{ type: "a", geo: arc },
+			{ type: "l", geo: new Line([arc.pointAt(1), copy(arc.pos)]) },
+			{ type: "z" },
+		],
+		[],
+		attribs || __copyAttribsRaw(arc.attribs)
 	);
-	path.attribs = attribs || __copyAttribsRaw(arc.attribs);
-	return path;
-};

packages/geom/src/index.ts

@@ -74,6 +74,7 @@ export * from "./as-cubic.js";
 export * from "./as-path.js";
 export * from "./as-polygon.js";
 export * from "./as-polyline.js";
+export * from "./as-sector.js";
 export * from "./as-svg.js";
 export * from "./bounds.js";
 export * from "./center.js";
@@ -97,7 +98,6 @@ export * from "./rotate-around-axis.js";
 export * from "./scale.js";
 export * from "./scale-with-center.js";
 export * from "./scatter.js";
-export * from "./sector.js";
 export * from "./simplify.js";
 export * from "./split-arclength.js";
 export * from "./split-at.js";