@ontrails/regrade 0.2.3 → 0.2.4

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @ontrails/regrade
 
+## 0.2.4
+
+### Patch Changes
+
+- [`b155fde`](https://github.com/outfitter-dev/trails/commit/b155fde9aa271ead2e76d675f2d281e28c9289f4): Report a directory that disappears between listing and reading as `vanished-directory` instead of `unreadable-directory` in `collectSourceTree`. Permission and other read failures still report `unreadable-directory`. Static project identity discovery now rescans, up to three attempts, when a directory vanishes mid-walk. A removed directory, such as a parallel test's temp dir, drops out of the next listing, and a renamed one, including one holding a `trails.config.*` marker, is found under its new path. Churn that outlasts the retries still fails closed. Regrade also fails closed on both reasons, because a vanished directory may have been renamed and its sources would otherwise escape a prepared run or apply.
+
 ## 0.2.3
 
 No changes in this release.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ontrails/regrade",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "Downstream migration reporting and safe rewrite helpers for Trails.",
   "repository": {
     "type": "git",
@@ -27,14 +27,14 @@
     "clean": "rm -rf dist *.tsbuildinfo"
   },
   "dependencies": {
-    "@ontrails/core": "^0.2.3",
-    "@ontrails/source": "^0.2.3",
-    "@ontrails/warden": "^0.2.3",
+    "@ontrails/core": "^0.2.4",
+    "@ontrails/source": "^0.2.4",
+    "@ontrails/warden": "^0.2.4",
     "tar": "7.5.22",
     "zod": "^4.3.5"
   },
   "devDependencies": {
-    "@ontrails/testing": "^0.2.3",
-    "@ontrails/topography": "^0.2.3"
+    "@ontrails/testing": "^0.2.4",
+    "@ontrails/topography": "^0.2.4"
   }
 }

package/src/downstream/collect.ts

@@ -54,6 +54,17 @@ export interface SkippedSource {
   readonly reason: string;
 }
 
+/**
+ * Whether a skipped directory leaves the collection incomplete.
+ *
+ * A directory that vanished mid-walk may have been renamed rather than
+ * removed, so its sources can still exist under a path the walk never saw.
+ * Regrade fails closed on both reasons.
+ */
+export const isIncompleteDirectorySkip = (entry: SkippedSource): boolean =>
+  entry.reason === 'unreadable-directory' ||
+  entry.reason === 'vanished-directory';
+
 /** Deterministic result of collecting downstream sources from a root. */
 export interface DownstreamSourceCollection {
   /** The root the collection ran against. */

package/src/downstream/file-renames.ts

@@ -22,6 +22,7 @@ import { createAstStringLiteralRenameClass } from './ast-rewrite.js';
 import {
   DEFAULT_IGNORED_DIRECTORIES,
   collectDownstreamSources,
+  isIncompleteDirectorySkip,
 } from './collect.js';
 import type {
   RegradeClassResult,
@@ -175,8 +176,7 @@ const fileRenameSourceStateHash = (params: {
   const unreadable = params.collected.skipped
     .filter(
       (entry) =>
-        entry.reason === 'unreadable-file' ||
-        entry.reason === 'unreadable-directory'
+        entry.reason === 'unreadable-file' || isIncompleteDirectorySkip(entry)
     )
     .map((entry) => entry.path);
   if (unreadable.length > 0) {

package/src/downstream/report.ts

@@ -29,6 +29,7 @@ import {
   DEFAULT_IGNORED_DIRECTORIES,
   DEFAULT_SOURCE_EXTENSIONS,
   collectDownstreamSources,
+  isIncompleteDirectorySkip,
 } from './collect.js';
 import type { DownstreamCollectionOptions, SkippedSource } from './collect.js';
 import {
@@ -1488,7 +1489,7 @@ const collectRegradeEvaluation = (
   const sourceBytes: { readonly bytes: string; readonly path: string }[] = [];
   const unreadable = new Set(
     collected.skipped
-      .filter((entry) => entry.reason === 'unreadable-directory')
+      .filter(isIncompleteDirectorySkip)
       .map((entry) => entry.path)
   );
   const manifestPaths = new Set<string>();
@@ -1589,7 +1590,7 @@ const collectRegradeSourceState = (
   const sourceBytes: { readonly bytes: string; readonly path: string }[] = [];
   const unreadable = new Set(
     collected.skipped
-      .filter((entry) => entry.reason === 'unreadable-directory')
+      .filter(isIncompleteDirectorySkip)
       .map((entry) => entry.path)
   );
   const manifestPaths = new Set<string>();

package/src/downstream/vocabulary.ts

@@ -23,6 +23,7 @@ import { z } from 'zod';
 import {
   DEFAULT_IGNORED_DIRECTORIES,
   collectDownstreamSources,
+  isIncompleteDirectorySkip,
 } from './collect.js';
 import type { DownstreamCollectionOptions, SkippedSource } from './collect.js';
 import type {
@@ -2232,8 +2233,7 @@ const unreadableVocabularyPaths = (
   skipped
     .filter(
       (entry) =>
-        entry.reason === 'unreadable-file' ||
-        entry.reason === 'unreadable-directory'
+        entry.reason === 'unreadable-file' || isIncompleteDirectorySkip(entry)
     )
     .map((entry) => entry.path);