@savvy-web/silk-effects 3.2.1 → 3.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@savvy-web/silk-effects",
-	"version": "3.2.1",
+	"version": "3.2.2",
 	"private": false,
 	"description": "Shared Effect library for Silk Suite conventions",
 	"homepage": "https://github.com/savvy-web/systems/tree/main/packages/silk-effects",

package/schemas/WorkspaceAnalysisSchemas.js

@@ -1,4 +1,5 @@
 import { ChangesetConfigFile, SilkChangesetConfigFile, VersioningStrategyResult } from "./VersioningSchemas.js";
+import { trimTrailingSlashes } from "../utils/TrailingSlash.js";
 import { TagStrategyType } from "./TagStrategySchemas.js";
 import { Equal, Function, Hash, Option, Pretty, Schema } from "effect";
 import { PublishConfig, PublishTarget } from "workspaces-effect";
@@ -39,7 +40,7 @@ const KNOWN_REGISTRIES = {
 * both sides so `hasTarget`/`targetFor` match regardless of which form a target
 * carries (binding-driven, placeholder, or access-branch fallback).
 */
-const sameRegistry = (a, b) => a.replace(/\/+$/, "") === b.replace(/\/+$/, "");
+const sameRegistry = (a, b) => trimTrailingSlashes(a) === trimTrailingSlashes(b);
 const WorkspaceVersion = Schema.Struct({ current: Schema.String });
 /**
 * A fully analyzed workspace with publish targets, versioning status,

package/services/SilkPublishability.js

@@ -1,4 +1,5 @@
 import { PublishTargetBindingError } from "../errors/PublishTargetBindingError.js";
+import { trimTrailingSlashes } from "../utils/TrailingSlash.js";
 import { ChangesetConfig } from "./ChangesetConfig.js";
 import { Effect, Layer } from "effect";
 import { isAbsolute, join, relative } from "node:path";
@@ -186,18 +187,16 @@ var SilkPublishability = class {
 * normalizes to `.`, matching how a root-directory target is recorded.
 *
 * @remarks
-* Trailing slashes are trimmed with an index scan rather than `/\/+$/`. That
-* regex is unanchored at the start, so the engine retries the match from every
-* position and degrades to O(n²) on a path of many slashes (CodeQL
-* `js/polynomial-redos`). `dir` reaches here from `dist/prod/targets.json`, which
-* the bundler writes but a consumer repo could hand-edit.
+* Trailing-slash trimming is delegated to the shared `trimTrailingSlashes`
+* index-scan helper rather than `/\/+$/`: that regex is unanchored at the
+* start, so the engine retries the match from every position and degrades to
+* O(n²) on a path of many slashes (CodeQL `js/polynomial-redos`). `dir` reaches
+* here from `dist/prod/targets.json`, which the bundler writes but a consumer
+* repo could hand-edit.
 */
 const normalizeDir = (dir) => {
 	const slashed = dir.replaceAll("\\", "/");
-	const withoutPrefix = slashed.startsWith("./") ? slashed.slice(2) : slashed;
-	let end = withoutPrefix.length;
-	while (end > 0 && withoutPrefix[end - 1] === "/") end -= 1;
-	const normalized = withoutPrefix.slice(0, end);
+	const normalized = trimTrailingSlashes(slashed.startsWith("./") ? slashed.slice(2) : slashed);
 	return normalized === "" ? "." : normalized;
 };
 /** True when a built target directory's package.json is `private: true`. Missing/unreadable/malformed → false. */

package/utils/TrailingSlash.js

@@ -0,0 +1,18 @@
+//#region src/utils/TrailingSlash.ts
+/**
+* Trim trailing slashes from a string.
+*
+* @remarks
+* Trims trailing slashes with an index scan rather than `/\/+$/`. That regex is
+* unanchored at the start, so the engine retries the match from every position
+* and degrades to O(n²) on a string of many slashes (CodeQL `js/polynomial-redos`).
+* Only a trailing run of slashes is removed; interior slash runs are untouched.
+*/
+const trimTrailingSlashes = (s) => {
+	let end = s.length;
+	while (end > 0 && s[end - 1] === "/") end -= 1;
+	return s.slice(0, end);
+};
+
+//#endregion
+export { trimTrailingSlashes };