repopo 0.5.0 → 0.7.0

package/CHANGELOG.md

@@ -1,5 +1,92 @@
 # repopo
 
+## 0.7.0
+
+### Minor Changes
+
+- Add configurable PackageScripts policy _[`#376`](https://github.com/tylerbutler/tools-monorepo/pull/376) [`c17462a`](https://github.com/tylerbutler/tools-monorepo/commit/c17462a8af35a8bfbb528f29825a0ef177923946) [@tylerbutler](https://github.com/tylerbutler)_
+
+  Enhances the `PackageScripts` policy with flexible configuration options to enforce package.json script requirements across your repository. The policy now supports two validation modes:
+
+  1.  **Required Scripts (`must`)**: Specify scripts that must be present in all package.json files
+  2.  **Mutually Exclusive Scripts (`mutuallyExclusive`)**: Define groups of scripts where exactly one from each group must exist
+
+  Example configuration:
+
+  ```typescript
+  makePolicy(PackageScripts, {
+    must: ["build", "clean"],
+    mutuallyExclusive: [
+      ["test", "test:unit"], // Either "test" or "test:unit", not both
+    ],
+  });
+  ```
+
+  The policy validates that:
+
+  - All scripts in the `must` array are present
+  - For each group in `mutuallyExclusive`, exactly one script from that group exists (not zero, not multiple)
+
+  This is useful for enforcing consistent npm script conventions across monorepo packages while allowing flexibility where needed.
+
+- New built-in policies _[`#308`](https://github.com/tylerbutler/tools-monorepo/pull/308) [`1856d58`](https://github.com/tylerbutler/tools-monorepo/commit/1856d582fb92692447b02c89cdb45dcd3c5f7370) [@tylerbutler](https://github.com/tylerbutler)_
+
+  Added three new policies for repository quality and safety:
+
+  - **LicenseFileExists**: Ensures a LICENSE file exists in the repository root (essential for open source projects)
+  - **NoLargeBinaryFiles**: Prevents large binary files from being committed (default 10MB max, suggests Git LFS for large assets)
+  - **RequiredGitignorePatterns**: Validates .gitignore contains required patterns to prevent committing sensitive files, dependencies, and build artifacts (auto-fixable)
+
+### Patch Changes
+
+<details><summary>Updated 2 dependencies</summary>
+
+<small>
+
+[`b0d8cb9`](https://github.com/tylerbutler/tools-monorepo/commit/b0d8cb9a9ee27a0b778ee58055bcbdd7d6d9b4eb) [`08e571f`](https://github.com/tylerbutler/tools-monorepo/commit/08e571f028e868d5db1c337e51804f5884cd2f4a) [`08e571f`](https://github.com/tylerbutler/tools-monorepo/commit/08e571f028e868d5db1c337e51804f5884cd2f4a)
+
+</small>
+
+- `@tylerbu/cli-api@0.8.0`
+- `@tylerbu/fundamentals@0.3.0`
+
+</details>
+
+## 0.6.0
+
+### Minor Changes
+
+- Configuration changes _[`#224`](https://github.com/tylerbutler/tools-monorepo/pull/224) [`2b7db34`](https://github.com/tylerbutler/tools-monorepo/commit/2b7db343d74ab2e518ab2f232f47ea365ca784b8) [@tylerbutler](https://github.com/tylerbutler)_
+
+  The configuration for repopo has changed a lot in this release. The goal is to improve the typing of config and make it
+  easier to configure things in one place.
+
+  Policies are now declared as `PolicyDefinition`, and are then combined with a configuration (if the policy has
+  configuration) using `makePolicy` to form a `PolicyInstance`, which is the primary type used internally.
+
+  Generators now generate `PolicyDefinition`s, and can be used directly inline with `makePolicy`.
+
+- Export policies from /policies entrypoint only _[`#224`](https://github.com/tylerbutler/tools-monorepo/pull/224) [`2b7db34`](https://github.com/tylerbutler/tools-monorepo/commit/2b7db343d74ab2e518ab2f232f47ea365ca784b8) [@tylerbutler](https://github.com/tylerbutler)_
+
+  Policies included with repopo are now exported from `repopo/policies`.
+
+### Patch Changes
+
+- Update package metadata _[`#221`](https://github.com/tylerbutler/tools-monorepo/pull/221) [`9e5225a`](https://github.com/tylerbutler/tools-monorepo/commit/9e5225abfb67af1575af13dff60830d8da28eafd) [@tylerbutler](https://github.com/tylerbutler)_
+
+<details><summary>Updated 2 dependencies</summary>
+
+<small>
+
+[`9e5225a`](https://github.com/tylerbutler/tools-monorepo/commit/9e5225abfb67af1575af13dff60830d8da28eafd)
+
+</small>
+
+- `@tylerbu/fundamentals@0.2.2`
+- `@tylerbu/cli-api@0.7.2`
+
+</details>
+
 ## 0.5.0
 
 ### Minor Changes

package/README.md

@@ -24,14 +24,12 @@ By default, all files in the repo are checked. You can exclude files completely
 d `excludeFiles` setting. It should be an array of strings/regular expressions. Paths that match any of these
 expressions will be completely excluded from all policies.
 
-You can exclude files from individual policies as well. To do this, set the `excludePoliciesForFiles` setting. It should
-be an object with keys that match policy names. The value is an array of strings/regular expressions. Paths that match
-any of these expressions will be excluded from that policy only.
+You can exclude files from individual policies as well by configuring the `excludeFiles` setting when calling
+`makePolicy`.
 
 ## Configuring individual policies
 
-Individual policies can be configured using the `policySettings` setting. Use the policy name as the key, with the
-policy settings as the value.
+Individual policies can be configured by passing configuration settings to a `PolicyDefinition` in `makePolicy`.
 
 # Included policies
 

package/THIRD-PARTY-LICENSES.txt

@@ -3,7 +3,7 @@ https://www.npmjs.com/package/generate-license-file
 
 The following npm package may be included in this product:
 
- - jsonfile@6.1.0
+ - jsonfile@6.2.0
 
 This package contains the following license:
 
@@ -27,7 +27,7 @@ OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHE
 
 The following npm package may be included in this product:
 
- - @rushstack/node-core-library@5.13.0
+ - @rushstack/node-core-library@5.18.0
 
 This package contains the following license:
 
@@ -60,7 +60,7 @@ WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 The following npm package may be included in this product:
 
- - @fluid-tools/build-infrastructure@0.56.0
+ - @fluid-tools/build-infrastructure@0.60.0
 
 This package contains the following license:
 
@@ -114,7 +114,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
 The following npm package may be included in this product:
 
- - simple-git@3.27.0
+ - simple-git@3.30.0
 
 This package contains the following license:
 
@@ -124,8 +124,8 @@ MIT
 
 The following npm packages may be included in this product:
 
- - @oclif/core@4.2.10
- - @oclif/plugin-help@6.2.27
+ - @oclif/core@4.8.0
+ - @oclif/plugin-help@6.2.35
 
 These packages each contain the following license:
 

package/esm/baseCommand.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"baseCommand.d.ts","sourceRoot":"","sources":["../src/baseCommand.ts"],"names":[],"mappings":"AAAA,OAAO,EACN,iBAAiB,EACjB,KAAK,kBAAkB,EACvB,KAAK,WAAW,EAChB,MAAM,kBAAkB,CAAC;AAE1B,OAAO,EAAE,KAAK,SAAS,EAAa,MAAM,YAAY,CAAC;AACvD,OAAO,EAAuB,KAAK,YAAY,EAAE,MAAM,aAAa,CAAC;AACrE,OAAO,KAAK,EAAyB,oBAAoB,EAAE,MAAM,cAAc,CAAC;AAIhF;;GAEG;AACH,8BAAsB,iBAAiB,CACrC,CAAC,SAAS,OAAO,iBAAiB,GAAG;IACpC,IAAI,EAAE,OAAO,iBAAiB,CAAC,IAAI,CAAC;IACpC,KAAK,EAAE,OAAO,iBAAiB,CAAC,KAAK,CAAC;CACtC,CAEF,SAAQ,iBAAiB,CAAC,CAAC,EAAE,YAAY,CACzC,YAAW,kBAAkB,CAAC,oBAAoB,CAAC,EAAE,WAAW;IAEhE,UAAmB,aAAa,eAAuB;IAEvD,gBAAyB,KAAK;gCAXK,wBACjB,wCACD,wBAChB;MAUU;IAEX,OAAO,CAAC,IAAI,CAAwB;IACpC,IAAW,GAAG,IAAI,SAAS,CAK1B;IAEqB,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;IA8B3C,OAAO,CAAC,QAAQ,CAAmC;IAGtC,UAAU,IAAI,OAAO,CAAC,oBAAoB,CAAC;CAMxD"}
+{"version":3,"file":"baseCommand.d.ts","sourceRoot":"","sources":["../src/baseCommand.ts"],"names":[],"mappings":"AAAA,OAAO,EACN,iBAAiB,EACjB,KAAK,kBAAkB,EACvB,KAAK,WAAW,EAChB,MAAM,kBAAkB,CAAC;AAE1B,OAAO,EAAE,KAAK,SAAS,EAAa,MAAM,YAAY,CAAC;AACvD,OAAO,EAAuB,KAAK,YAAY,EAAE,MAAM,aAAa,CAAC;AACrE,OAAO,KAAK,EAAyB,oBAAoB,EAAE,MAAM,cAAc,CAAC;AAKhF;;GAEG;AACH,8BAAsB,iBAAiB,CACrC,CAAC,SAAS,OAAO,iBAAiB,GAAG;IACpC,IAAI,EAAE,OAAO,iBAAiB,CAAC,IAAI,CAAC;IACpC,KAAK,EAAE,OAAO,iBAAiB,CAAC,KAAK,CAAC;CACtC,CAEF,SAAQ,iBAAiB,CAAC,CAAC,EAAE,YAAY,CACzC,YAAW,kBAAkB,CAAC,oBAAoB,CAAC,EAAE,WAAW;IAEhE,UAAmB,aAAa,eAAuB;IAEvD,gBAAgC,KAAK;;MAE1B;IAEX,OAAO,CAAC,IAAI,CAAwB;IACpC,IAAW,GAAG,IAAI,SAAS,CAK1B;IAEqB,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;IAyB3C,OAAO,CAAC,QAAQ,CAAmC;IAEtC,UAAU,IAAI,OAAO,CAAC,oBAAoB,CAAC;CAMxD"}

package/esm/baseCommand.js

@@ -2,6 +2,7 @@ import { CommandWithConfig, } from "@tylerbu/cli-api";
 import { findGitRootSync } from "@tylerbu/fundamentals/git";
 import { simpleGit } from "simple-git";
 import { DefaultPolicyConfig } from "./config.js";
+import { makePolicy } from "./makePolicy.js";
 import { newPerfStats } from "./perf.js";
 import { DefaultPolicies } from "./policy.js";
 /**
@@ -22,26 +23,23 @@ export class BaseRepopoCommand extends CommandWithConfig {
     async init() {
         await super.init();
         const excludeFiles = this.commandConfig?.excludeFiles?.map((e) => new RegExp(e, "i")) ?? [];
-        const excludePoliciesForFilesRaw = this.commandConfig?.excludePoliciesForFiles;
         const excludePoliciesForFiles = new Map();
-        if (excludePoliciesForFilesRaw) {
-            for (const [policyName, filters] of Object.entries(excludePoliciesForFilesRaw)) {
-                const regexes = filters.map((e) => new RegExp(e, "i"));
-                excludePoliciesForFiles.set(policyName, regexes);
-            }
+        for (const policy of this.commandConfig?.policies ?? []) {
+            const regexes = policy.excludeFiles?.map((e) => new RegExp(e, "i")) ?? [];
+            excludePoliciesForFiles.set(policy.name, regexes);
         }
         const gitRoot = findGitRootSync();
         this._git = simpleGit({ baseDir: gitRoot });
         this._context = {
-            excludeFiles,
-            policies: this.commandConfig?.policies ?? DefaultPolicies,
+            excludeFromAll: excludeFiles,
+            policies: this.commandConfig?.policies ??
+                DefaultPolicies.map((p) => makePolicy(p)),
             excludePoliciesForFiles,
             gitRoot,
             perfStats: newPerfStats(),
         };
     }
     _context;
-    // biome-ignore lint/suspicious/useAwait: interface defines this as async
     async getContext() {
         if (this._context === undefined) {
             throw new Error("Context not initialized.");

package/esm/baseCommand.js.map

@@ -1 +1 @@
-{"version":3,"file":"baseCommand.js","sourceRoot":"","sources":["../src/baseCommand.ts"],"names":[],"mappings":"AAAA,OAAO,EACN,iBAAiB,GAGjB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAC;AAC5D,OAAO,EAAkB,SAAS,EAAE,MAAM,YAAY,CAAC;AACvD,OAAO,EAAE,mBAAmB,EAAqB,MAAM,aAAa,CAAC;AAErE,OAAO,EAAE,YAAY,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAE9C;;GAEG;AACH,MAAM,OAAgB,iBAMrB,SAAQ,iBAAkC;IAGvB,aAAa,GAAG,mBAAmB,CAAC;IAEvD,MAAM,CAAmB,KAAK,GAAG;QAChC,GAAG,iBAAiB,CAAC,KAAK;KACjB,CAAC;IAEH,IAAI,CAAwB;IACpC,IAAW,GAAG;QACb,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC7B,MAAM,IAAI,KAAK,CAAC,2BAA2B,CAAC,CAAC;QAC9C,CAAC;QACD,OAAO,IAAI,CAAC,IAAI,CAAC;IAClB,CAAC;IAEe,KAAK,CAAC,IAAI;QACzB,MAAM,KAAK,CAAC,IAAI,EAAE,CAAC;QAEnB,MAAM,YAAY,GACjB,IAAI,CAAC,aAAa,EAAE,YAAY,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,MAAM,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;QAExE,MAAM,0BAA0B,GAC/B,IAAI,CAAC,aAAa,EAAE,uBAAuB,CAAC;QAE7C,MAAM,uBAAuB,GAA0B,IAAI,GAAG,EAAE,CAAC;QACjE,IAAI,0BAA0B,EAAE,CAAC;YAChC,KAAK,MAAM,CAAC,UAAU,EAAE,OAAO,CAAC,IAAI,MAAM,CAAC,OAAO,CACjD,0BAA0B,CAC1B,EAAE,CAAC;gBACH,MAAM,OAAO,GAAG,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,MAAM,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC;gBACvD,uBAAuB,CAAC,GAAG,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;YAClD,CAAC;QACF,CAAC;QAED,MAAM,OAAO,GAAG,eAAe,EAAE,CAAC;QAClC,IAAI,CAAC,IAAI,GAAG,SAAS,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC;QAC5C,IAAI,CAAC,QAAQ,GAAG;YACf,YAAY;YACZ,QAAQ,EAAE,IAAI,CAAC,aAAa,EAAE,QAAQ,IAAI,eAAe;YACzD,uBAAuB;YACvB,OAAO;YACP,SAAS,EAAE,YAAY,EAAE;SACzB,CAAC;IACH,CAAC;IAEO,QAAQ,CAAmC;IAEnD,yEAAyE;IAClE,KAAK,CAAC,UAAU;QACtB,IAAI,IAAI,CAAC,QAAQ,KAAK,SAAS,EAAE,CAAC;YACjC,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;QAC7C,CAAC;QACD,OAAO,IAAI,CAAC,QAAQ,CAAC;IACtB,CAAC","sourcesContent":["import {\n\tCommandWithConfig,\n\ttype CommandWithContext,\n\ttype RequiresGit,\n} from \"@tylerbu/cli-api\";\nimport { findGitRootSync } from \"@tylerbu/fundamentals/git\";\nimport { type SimpleGit, simpleGit } from \"simple-git\";\nimport { DefaultPolicyConfig, type RepopoConfig } from \"./config.js\";\nimport type { ExcludedPolicyFileMap, RepopoCommandContext } from \"./context.js\";\nimport { newPerfStats } from \"./perf.js\";\nimport { DefaultPolicies } from \"./policy.js\";\n\n/**\n * This class is the base for all repopo commands. It contains common flags and config loading.\n */\nexport abstract class BaseRepopoCommand<\n\t\tT extends typeof BaseRepopoCommand & {\n\t\t\targs: typeof CommandWithConfig.args;\n\t\t\tflags: typeof CommandWithConfig.flags;\n\t\t},\n\t>\n\textends CommandWithConfig<T, RepopoConfig>\n\timplements CommandWithContext<RepopoCommandContext>, RequiresGit\n{\n\tprotected override defaultConfig = DefaultPolicyConfig;\n\n\tstatic override readonly flags = {\n\t\t...CommandWithConfig.flags,\n\t} as const;\n\n\tprivate _git: SimpleGit | undefined;\n\tpublic get git(): SimpleGit {\n\t\tif (this._git === undefined) {\n\t\t\tthrow new Error(\"git property is undefined\");\n\t\t}\n\t\treturn this._git;\n\t}\n\n\tpublic override async init(): Promise<void> {\n\t\tawait super.init();\n\n\t\tconst excludeFiles: RegExp[] =\n\t\t\tthis.commandConfig?.excludeFiles?.map((e) => new RegExp(e, \"i\")) ?? [];\n\n\t\tconst excludePoliciesForFilesRaw =\n\t\t\tthis.commandConfig?.excludePoliciesForFiles;\n\n\t\tconst excludePoliciesForFiles: ExcludedPolicyFileMap = new Map();\n\t\tif (excludePoliciesForFilesRaw) {\n\t\t\tfor (const [policyName, filters] of Object.entries(\n\t\t\t\texcludePoliciesForFilesRaw,\n\t\t\t)) {\n\t\t\t\tconst regexes = filters.map((e) => new RegExp(e, \"i\"));\n\t\t\t\texcludePoliciesForFiles.set(policyName, regexes);\n\t\t\t}\n\t\t}\n\n\t\tconst gitRoot = findGitRootSync();\n\t\tthis._git = simpleGit({ baseDir: gitRoot });\n\t\tthis._context = {\n\t\t\texcludeFiles,\n\t\t\tpolicies: this.commandConfig?.policies ?? DefaultPolicies,\n\t\t\texcludePoliciesForFiles,\n\t\t\tgitRoot,\n\t\t\tperfStats: newPerfStats(),\n\t\t};\n\t}\n\n\tprivate _context: RepopoCommandContext | undefined;\n\n\t// biome-ignore lint/suspicious/useAwait: interface defines this as async\n\tpublic async getContext(): Promise<RepopoCommandContext> {\n\t\tif (this._context === undefined) {\n\t\t\tthrow new Error(\"Context not initialized.\");\n\t\t}\n\t\treturn this._context;\n\t}\n}\n"]}
+{"version":3,"file":"baseCommand.js","sourceRoot":"","sources":["../src/baseCommand.ts"],"names":[],"mappings":"AAAA,OAAO,EACN,iBAAiB,GAGjB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAC;AAC5D,OAAO,EAAkB,SAAS,EAAE,MAAM,YAAY,CAAC;AACvD,OAAO,EAAE,mBAAmB,EAAqB,MAAM,aAAa,CAAC;AAErE,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,YAAY,EAAE,MAAM,WAAW,CAAC;AACzC,OAAO,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAE9C;;GAEG;AACH,MAAM,OAAgB,iBAMrB,SAAQ,iBAAkC;IAGvB,aAAa,GAAG,mBAAmB,CAAC;IAEhD,MAAM,CAAmB,KAAK,GAAG;QACvC,GAAG,iBAAiB,CAAC,KAAK;KACjB,CAAC;IAEH,IAAI,CAAwB;IACpC,IAAW,GAAG;QACb,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YAC7B,MAAM,IAAI,KAAK,CAAC,2BAA2B,CAAC,CAAC;QAC9C,CAAC;QACD,OAAO,IAAI,CAAC,IAAI,CAAC;IAClB,CAAC;IAEe,KAAK,CAAC,IAAI;QACzB,MAAM,KAAK,CAAC,IAAI,EAAE,CAAC;QAEnB,MAAM,YAAY,GACjB,IAAI,CAAC,aAAa,EAAE,YAAY,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,MAAM,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;QAExE,MAAM,uBAAuB,GAA0B,IAAI,GAAG,EAAE,CAAC;QACjE,KAAK,MAAM,MAAM,IAAI,IAAI,CAAC,aAAa,EAAE,QAAQ,IAAI,EAAE,EAAE,CAAC;YACzD,MAAM,OAAO,GAAG,MAAM,CAAC,YAAY,EAAE,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,MAAM,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;YAC1E,uBAAuB,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;QACnD,CAAC;QAED,MAAM,OAAO,GAAG,eAAe,EAAE,CAAC;QAClC,IAAI,CAAC,IAAI,GAAG,SAAS,CAAC,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC,CAAC;QAC5C,IAAI,CAAC,QAAQ,GAAG;YACf,cAAc,EAAE,YAAY;YAC5B,QAAQ,EACP,IAAI,CAAC,aAAa,EAAE,QAAQ;gBAC5B,eAAe,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC;YAC1C,uBAAuB;YACvB,OAAO;YACP,SAAS,EAAE,YAAY,EAAE;SACzB,CAAC;IACH,CAAC;IAEO,QAAQ,CAAmC;IAE5C,KAAK,CAAC,UAAU;QACtB,IAAI,IAAI,CAAC,QAAQ,KAAK,SAAS,EAAE,CAAC;YACjC,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;QAC7C,CAAC;QACD,OAAO,IAAI,CAAC,QAAQ,CAAC;IACtB,CAAC","sourcesContent":["import {\n\tCommandWithConfig,\n\ttype CommandWithContext,\n\ttype RequiresGit,\n} from \"@tylerbu/cli-api\";\nimport { findGitRootSync } from \"@tylerbu/fundamentals/git\";\nimport { type SimpleGit, simpleGit } from \"simple-git\";\nimport { DefaultPolicyConfig, type RepopoConfig } from \"./config.js\";\nimport type { ExcludedPolicyFileMap, RepopoCommandContext } from \"./context.js\";\nimport { makePolicy } from \"./makePolicy.js\";\nimport { newPerfStats } from \"./perf.js\";\nimport { DefaultPolicies } from \"./policy.js\";\n\n/**\n * This class is the base for all repopo commands. It contains common flags and config loading.\n */\nexport abstract class BaseRepopoCommand<\n\t\tT extends typeof BaseRepopoCommand & {\n\t\t\targs: typeof CommandWithConfig.args;\n\t\t\tflags: typeof CommandWithConfig.flags;\n\t\t},\n\t>\n\textends CommandWithConfig<T, RepopoConfig>\n\timplements CommandWithContext<RepopoCommandContext>, RequiresGit\n{\n\tprotected override defaultConfig = DefaultPolicyConfig;\n\n\tpublic static override readonly flags = {\n\t\t...CommandWithConfig.flags,\n\t} as const;\n\n\tprivate _git: SimpleGit | undefined;\n\tpublic get git(): SimpleGit {\n\t\tif (this._git === undefined) {\n\t\t\tthrow new Error(\"git property is undefined\");\n\t\t}\n\t\treturn this._git;\n\t}\n\n\tpublic override async init(): Promise<void> {\n\t\tawait super.init();\n\n\t\tconst excludeFiles: RegExp[] =\n\t\t\tthis.commandConfig?.excludeFiles?.map((e) => new RegExp(e, \"i\")) ?? [];\n\n\t\tconst excludePoliciesForFiles: ExcludedPolicyFileMap = new Map();\n\t\tfor (const policy of this.commandConfig?.policies ?? []) {\n\t\t\tconst regexes = policy.excludeFiles?.map((e) => new RegExp(e, \"i\")) ?? [];\n\t\t\texcludePoliciesForFiles.set(policy.name, regexes);\n\t\t}\n\n\t\tconst gitRoot = findGitRootSync();\n\t\tthis._git = simpleGit({ baseDir: gitRoot });\n\t\tthis._context = {\n\t\t\texcludeFromAll: excludeFiles,\n\t\t\tpolicies:\n\t\t\t\tthis.commandConfig?.policies ??\n\t\t\t\tDefaultPolicies.map((p) => makePolicy(p)),\n\t\t\texcludePoliciesForFiles,\n\t\t\tgitRoot,\n\t\t\tperfStats: newPerfStats(),\n\t\t};\n\t}\n\n\tprivate _context: RepopoCommandContext | undefined;\n\n\tpublic async getContext(): Promise<RepopoCommandContext> {\n\t\tif (this._context === undefined) {\n\t\t\tthrow new Error(\"Context not initialized.\");\n\t\t}\n\t\treturn this._context;\n\t}\n}\n"]}

package/esm/commands/check.d.ts

@@ -19,18 +19,29 @@ export declare class CheckPolicy<T extends typeof BaseRepopoCommand & {
         readonly stdin: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
-    private executeAllPolicies;
     /**
-     * Routes files to their handlers and resolvers by regex testing their full paths. If a file fails a policy that has a
-     * resolver, the resolver will be invoked as well. Synchronizes the output, exit codes, and resolve
-     * decision for all handlers.
+     * Executes all policies against the provided paths.
+     *
+     * @param pathsToCheck - All paths that should be checked. Paths should be relative to the repository root.
+     * @param context - The context.
      */
-    private routeToHandlers;
-    private runPolicyOnFile;
+    private checkAllFiles;
     /**
      * Given a string that represents a path to a file in the repo, determines if the file should be checked, and if so,
      * routes the file to the appropriate handlers.
+     *
+     * @param relPath - A git repo-relative path to a file.
      */
     private checkOrExcludeFile;
+    private routeToPolicies;
+    private runPolicyOnFile;
+    private isPolicyExcluded;
+    private executePolicyHandler;
+    private handlePolicyResult;
+    private handleFixResult;
+    private handleFailureResult;
+    private attemptResolution;
+    private logPolicyFailure;
+    private logMessages;
 }
 //# sourceMappingURL=check.d.ts.map

package/esm/commands/check.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"check.d.ts","sourceRoot":"","sources":["../../src/commands/check.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,iBAAiB,EAAE,MAAM,mBAAmB,CAAC;AAKtD;;;;;;;;GAQG;AACH,qBAAa,WAAW,CACvB,CAAC,SAAS,OAAO,iBAAiB,GAAG;IACpC,IAAI,EAAE,OAAO,WAAW,CAAC,IAAI,CAAC;IAC9B,KAAK,EAAE,OAAO,WAAW,CAAC,KAAK,CAAC;CAChC,CACA,SAAQ,iBAAiB,CAAC,CAAC,CAAC;IAC7B,gBAAyB,OAAO,iEAC+B;IAE/D,gBAAyB,KAAK;gCAhBkB,wBAAuB,wCAAwC,wBAE/F;;;MA0BL;IAEW,GAAG,IAAI,OAAO,CAAC,IAAI,CAAC;YAmC5B,kBAAkB;IAehC;;;;OAIG;YACW,eAAe;YAwBf,eAAe;IAkG7B;;;OAGG;YACW,kBAAkB;CAwBhC"}
+{"version":3,"file":"check.d.ts","sourceRoot":"","sources":["../../src/commands/check.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,iBAAiB,EAAE,MAAM,mBAAmB,CAAC;AAgCtD;;;;;;;;GAQG;AACH,qBAAa,WAAW,CACvB,CAAC,SAAS,OAAO,iBAAiB,GAAG;IACpC,IAAI,EAAE,OAAO,WAAW,CAAC,IAAI,CAAC;IAC9B,KAAK,EAAE,OAAO,WAAW,CAAC,KAAK,CAAC;CAChC,CACA,SAAQ,iBAAiB,CAAC,CAAC,CAAC;IAC7B,gBAAgC,OAAO,iEACwB;IAE/D,gBAAgC,KAAK;;;;MAY1B;IAEW,GAAG,IAAI,OAAO,CAAC,IAAI,CAAC;IAwD1C;;;;;OAKG;YACW,aAAa;IAe3B;;;;;OAKG;YACW,kBAAkB;YAmBlB,eAAe;YAsCf,eAAe;IAwC7B,OAAO,CAAC,gBAAgB;YAYV,oBAAoB;YAsBpB,kBAAkB;IAwBhC,OAAO,CAAC,eAAe;YAoBT,mBAAmB;YAyBnB,iBAAiB;IAyB/B,OAAO,CAAC,gBAAgB;IAcxB,OAAO,CAAC,WAAW;CAOnB"}

package/esm/commands/check.js

@@ -1,11 +1,27 @@
 import { EOL as newline } from "node:os";
+import process from "node:process";
 import { Flags } from "@oclif/core";
 import { StringBuilder } from "@rushstack/node-core-library";
-import path from "pathe";
 import chalk from "picocolors";
 import { BaseRepopoCommand } from "../baseCommand.js";
 import { logStats, runWithPerf } from "../perf.js";
-import { isPolicyFixResult } from "../policy.js";
+import { isPolicyFixResult, } from "../policy.js";
+async function readStdin() {
+    return new Promise((resolve) => {
+        const stdin = process.stdin;
+        stdin.setEncoding("utf8");
+        let data = "";
+        stdin.on("data", (chunk) => {
+            data += chunk;
+        });
+        stdin.on("end", () => {
+            resolve(data);
+        });
+        if (stdin.isTTY) {
+            resolve("");
+        }
+    });
+}
 /**
  * This tool enforces policies across the code base via a series of handler functions. The handler functions are
  * associated with a regular expression, and all files matching that expression are passed to the handler function.
@@ -44,149 +60,185 @@ export class CheckPolicy extends BaseRepopoCommand {
         if (this.flags.stdin) {
             const stdInput = await readStdin();
             if (stdInput !== undefined) {
-                filePathsToCheck.push(...stdInput.split("\n"));
+                filePathsToCheck.push(...stdInput
+                    .replace(
+                // normalize slashes in case they're windows paths
+                /\\/g, "/")
+                    .split("\n"));
             }
         }
         else {
-            const gitFiles = (await this.git.raw("ls-files", "-co", "--exclude-standard", "--full-name")) ?? "";
-            filePathsToCheck.push(...gitFiles.split("\n"));
+            const gitFiles = (await this.git.raw("ls-files", 
+            // include staged files and untracked files
+            "-co", 
+            // exclude gitignored files and other standard ignore rules
+            "--exclude-standard", 
+            // Outputs paths relative to the root of the repository, regardless of the current working directory.
+            "--full-name")) ?? "";
+            filePathsToCheck.push(...gitFiles
+                .replace(
+            // normalize slashes in case they're windows paths
+            /\\/g, "/")
+                .split("\n"));
         }
-        await this.executeAllPolicies(filePathsToCheck);
+        const context = await this.getContext();
+        await this.checkAllFiles(filePathsToCheck, context);
     }
-    async executeAllPolicies(pathsToCheck) {
-        const commandContext = await this.getContext();
+    /**
+     * Executes all policies against the provided paths.
+     *
+     * @param pathsToCheck - All paths that should be checked. Paths should be relative to the repository root.
+     * @param context - The context.
+     */
+    async checkAllFiles(pathsToCheck, context) {
         try {
             for (const pathToCheck of pathsToCheck) {
-                // eslint-disable-next-line no-await-in-loop
-                await this.checkOrExcludeFile(pathToCheck, commandContext);
+                await this.checkOrExcludeFile(pathToCheck, context);
             }
         }
         finally {
             if (!this.flags.quiet) {
-                logStats(commandContext.perfStats, this);
+                logStats(context.perfStats, this);
             }
         }
     }
     /**
-     * Routes files to their handlers and resolvers by regex testing their full paths. If a file fails a policy that has a
-     * resolver, the resolver will be invoked as well. Synchronizes the output, exit codes, and resolve
-     * decision for all handlers.
+     * Given a string that represents a path to a file in the repo, determines if the file should be checked, and if so,
+     * routes the file to the appropriate handlers.
+     *
+     * @param relPath - A git repo-relative path to a file.
      */
-    async routeToHandlers(file, commandContext) {
-        const { policies, gitRoot } = commandContext;
-        // Use the repo-relative path so that regexes that specify string start (^) will match repo paths.
-        // Replace \ in result with / in case OS is Windows.
-        const relPath = path.relative(gitRoot, file).replace(/\\/g, "/");
-        const filteredPolicies = policies.filter((handler) => handler.match.test(relPath));
-        const resultsP = [];
-        for (const policy of filteredPolicies) {
-            resultsP.push(this.runPolicyOnFile(relPath, policy));
-        }
-        await Promise.all(resultsP);
+    async checkOrExcludeFile(relPath, context) {
+        const { perfStats } = context;
+        perfStats.count++;
+        try {
+            const wasProcessed = await this.routeToPolicies(relPath, context);
+            if (wasProcessed) {
+                perfStats.processed++;
+            }
+        }
+        catch (error) {
+            throw new Error(`Error routing ${relPath} to handler: ${error}\nStack:\n${error.stack}`);
+        }
     }
-    // biome-ignore lint/complexity/noExcessiveCognitiveComplexity: FIXME
-    async runPolicyOnFile(relPath, 
-    // policies: RepoPolicy[],
-    policy) {
-        const context = await this.getContext();
+    async routeToPolicies(relPath, context) {
+        const { policies, excludeFromAll } = context;
+        // Check exclusions
+        if (excludeFromAll.some((regex) => regex.test(relPath))) {
+            this.verbose(`Excluded all handlers: ${relPath}`);
+            return false; // File was excluded from all policies
+        }
+        // Run all matching policies
+        const matchingPolicies = policies.filter((policy) => policy.match.test(relPath));
+        // for(const policy of matchingPolicies) {
+        // 	bars.addFile(policy.name,
+        // }
+        // If no policies match this file, it's still considered processed
+        // (we examined it and determined no policies apply)
+        if (matchingPolicies.length === 0) {
+            return true;
+        }
+        const results = await Promise.all(matchingPolicies.map(async (policy) => {
+            return await this.runPolicyOnFile(relPath, policy, context);
+        }));
+        // File is considered processed if at least one policy ran on it
+        // If all matching policies excluded it, count as excluded
+        return results.some((ran) => ran);
+    }
+    async runPolicyOnFile(relPath, policy, context) {
         const { excludePoliciesForFiles, perfStats, gitRoot } = context;
-        // doing exclusion per handler
-        const exclusions = excludePoliciesForFiles.get(policy.name);
-        if (exclusions !== undefined &&
-            !exclusions.every((regex) => !regex.test(relPath))) {
+        // Check if the policy is excluded for the file
+        if (this.isPolicyExcluded(relPath, policy, excludePoliciesForFiles)) {
             this.verbose(`Excluded from '${policy.name}' policy: ${relPath}`);
-            return;
+            return false; // Policy didn't run on this file
         }
-        const result = await runWithPerf(policy.name, "handle", perfStats, async () => policy.handler({
-            file: relPath,
-            root: gitRoot,
-            resolve: this.flags.fix,
-            config: this.commandConfig?.perPolicyConfig?.[policy.name
-            // biome-ignore lint/suspicious/noExplicitAny: FIXME
-            ],
-        }));
+        try {
+            // Execute the policy handler
+            const result = await this.executePolicyHandler(relPath, policy, perfStats, gitRoot);
+            // Handle the result of the policy execution
+            await this.handlePolicyResult(result, relPath, policy, perfStats, gitRoot);
+        }
+        catch (error) {
+            // Log and rethrow the error for higher-level handling
+            this.error(`Error executing policy '${policy.name}' for file '${relPath}': ${error}`);
+        }
+        return true; // Policy ran (successfully or with error)
+    }
+    isPolicyExcluded(relPath, policy, excludePoliciesForFiles) {
+        return (excludePoliciesForFiles
+            .get(policy.name)
+            ?.some((regex) => regex.test(relPath)) ?? false);
+    }
+    async executePolicyHandler(relPath, policy, perfStats, gitRoot) {
+        try {
+            return await runWithPerf(policy.name, "handle", perfStats, () => policy.handler({
+                file: relPath,
+                root: gitRoot,
+                resolve: this.flags.fix,
+                config: policy.config,
+            }));
+        }
+        catch (error) {
+            this.error(`Error in policy handler '${policy.name}' for file '${relPath}': ${error}`);
+        }
+    }
+    async handlePolicyResult(result, relPath, policy, perfStats, gitRoot) {
         if (result === true) {
             return;
         }
-        const messages = new StringBuilder();
         if (isPolicyFixResult(result)) {
-            if (result.resolved) {
-                messages.append(`Resolved ${policy.name} policy failure for file: ${result.file}`);
-            }
-            else {
-                messages.append(`Error when trying to fix ${policy.name} policy failure in ${result.file}`);
-                process.exitCode = 1;
-            }
+            this.handleFixResult(result, policy);
         }
         else {
-            // result must be a PolicyFailureResult; check if there is a standalone resolver.
-            const { resolver } = policy;
-            if (this.flags.fix && resolver !== undefined) {
-                // Resolve the failure
-                messages.append(`${newline}attempting to resolve: ${relPath}`);
-                const resolveResult = await runWithPerf(policy.name, "resolve", perfStats, async () => resolver({ file: relPath, root: gitRoot }));
-                if (resolveResult.errorMessage !== undefined &&
-                    resolveResult.errorMessage !== "") {
-                    messages.append(newline + resolveResult.errorMessage);
-                }
-                if (!resolveResult.resolved) {
-                    process.exitCode = 1;
-                }
-            }
-            else {
-                // No resolver, or fix is false, so we're in the full failure case.
-                const autoFixable = result.autoFixable
-                    ? chalk.green(" (autofixable)")
-                    : "";
-                messages.append(`'${chalk.bold(policy.name)}' policy failure${autoFixable}: ${result.file}`);
-                messages.append(result.errorMessage === undefined
-                    ? ""
-                    : `${newline}\t${result.errorMessage}`);
-                process.exitCode = 1;
-            }
+            await this.handleFailureResult(result, relPath, policy, perfStats, gitRoot);
         }
-        if ((process.exitCode ?? 0) === 0) {
-            this.info(messages.toString());
+    }
+    handleFixResult(result, policy) {
+        const messages = new StringBuilder();
+        if (result.resolved) {
+            messages.append(`Resolved ${policy.name} policy failure for file: ${result.file}`);
         }
         else {
-            this.warning(messages.toString());
+            messages.append(`Error fixing ${policy.name} policy failure in ${result.file}`);
+            process.exitCode = 1;
         }
+        this.logMessages(messages);
     }
-    /**
-     * Given a string that represents a path to a file in the repo, determines if the file should be checked, and if so,
-     * routes the file to the appropriate handlers.
-     */
-    async checkOrExcludeFile(inputPath, commandContext) {
-        const { excludeFiles: exclusions, gitRoot, perfStats } = commandContext;
-        const filePath = path.join(gitRoot, inputPath).trim().replace(/\\/g, "/");
-        perfStats.count++;
-        if (!exclusions.every((value) => !value.test(inputPath))) {
-            this.verbose(`Excluded all handlers: ${inputPath}`);
-            return;
+    async handleFailureResult(result, relPath, policy, perfStats, gitRoot) {
+        const messages = new StringBuilder();
+        if (this.flags.fix && policy.resolver) {
+            await this.attemptResolution(relPath, policy, policy.resolver, perfStats, gitRoot, messages);
         }
-        try {
-            await this.routeToHandlers(filePath, commandContext);
+        else {
+            this.logPolicyFailure(result, policy, messages);
         }
-        catch (error) {
-            throw new Error(`Error routing ${filePath} to handler: ${error}\nStack:\n${error.stack}`);
+        this.logMessages(messages);
+    }
+    async attemptResolution(relPath, policy, resolver, perfStats, gitRoot, messages) {
+        messages.append(`${newline}Attempting to resolve: ${relPath}`);
+        const resolveResult = await runWithPerf(policy.name, "resolve", perfStats, () => resolver({ file: relPath, root: gitRoot }));
+        if (!resolveResult.resolved) {
+            process.exitCode = 1;
+        }
+        if (resolveResult.errorMessage) {
+            messages.append(newline + resolveResult.errorMessage);
         }
-        perfStats.processed++;
     }
-}
-async function readStdin() {
-    return new Promise((resolve) => {
-        const stdin = process.stdin;
-        stdin.setEncoding("utf8");
-        let data = "";
-        stdin.on("data", (chunk) => {
-            data += chunk;
-        });
-        stdin.on("end", () => {
-            resolve(data);
-        });
-        if (stdin.isTTY) {
-            resolve("");
+    logPolicyFailure(result, policy, messages) {
+        const autoFixable = result.autoFixable ? chalk.green(" (autofixable)") : "";
+        messages.append(`'${chalk.bold(policy.name)}' policy failure${autoFixable}: ${result.file}`);
+        if (result.errorMessage) {
+            messages.append(`${newline}\t${result.errorMessage}`);
         }
-    });
+        process.exitCode = 1;
+    }
+    logMessages(messages) {
+        if ((process.exitCode ?? 0) === 0) {
+            this.info(messages.toString());
+        }
+        else {
+            this.warning(messages.toString());
+        }
+    }
 }
 //# sourceMappingURL=check.js.map