relacher 0.0.7-rc.2 → 0.0.7-rc.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "relacher",
-  "version": "0.0.7-rc.2",
+  "version": "0.0.7-rc.5",
   "description": "Scriptable release orchestration library for monorepos",
   "type": "module",
   "private": false,

package/src/prepare.ts

@@ -14,7 +14,7 @@ import type {
   PrepareOptions,
 } from './types';
 import type { UpdateAction, UpdateActionResolved } from './updater';
-import { VcsProviderService, type VcsProvider } from './vcs';
+import { VcsProviderService } from './vcs';
 import { VersionManagerService, type VersionManager } from './versioning';
 import { isBumpEqual, isPreRelease } from './versioning/bump';
 import type { BumpSize } from './versioning/types';
@@ -177,33 +177,8 @@ export function prepare(
 
     const cwd = options.cwd || process.cwd();
 
-    let trackedPackages: PackageConfig[] = packages;
-    if (Option.isSome(vcsOption)) {
-      const vcs: VcsProvider = vcsOption.value;
-      if (typeof vcs.isTracked === 'function') {
-        const filtered: PackageConfig[] = [];
-        for (const pkg of packages) {
-          const targetPath = pkg.manifestPath
-            ? path.resolve(cwd, pkg.manifestPath)
-            : path.resolve(cwd, pkg.watch?.[0] || '.');
-
-          const isTracked = yield* vcs.isTracked(targetPath);
-          if (isTracked) {
-            filtered.push(pkg);
-          }
-        }
-        trackedPackages = filtered;
-
-        const activeNames = new Set(trackedPackages.map((p) => p.name));
-        for (const pkg of trackedPackages) {
-          if (pkg.depends) {
-            pkg.depends = pkg.depends.filter((d) => activeNames.has(d));
-          }
-        }
-      }
-    }
-
-    const items = yield* initReportItems(trackedPackages, cwd, options);
+    // Packages passed in were already validated and tracked by the builder.
+    const items = yield* initReportItems(packages, cwd, options);
     const sorted = topologicalSort(items);
     versionManager.propagateBumps(sorted);
     versionManager.propagateCoupledBumps(sorted);

package/src/vcs/common.ts

@@ -1,4 +1,5 @@
 import { execFileSync } from 'node:child_process';
+import path from 'node:path';
 
 export function isInsideGitWorkTree(cwd: string): boolean {
   try {
@@ -14,7 +15,11 @@ export function isInsideGitWorkTree(cwd: string): boolean {
 
 export function isGitTracked(filePath: string, cwd: string): boolean {
   try {
-    execFileSync('git', ['ls-files', '--error-unmatch', filePath], {
+    const relPath = path.isAbsolute(filePath)
+      ? path.relative(cwd, filePath).replace(/\\/g, '/')
+      : filePath.replace(/\\/g, '/');
+
+    execFileSync('git', ['ls-files', '--error-unmatch', relPath || '.'], {
       cwd,
       stdio: ['ignore', 'ignore', 'ignore'],
     });
@@ -38,7 +43,12 @@ export function isInsideJjWorkTree(cwd: string): boolean {
 
 export function isJjTracked(filePath: string, cwd: string): boolean {
   try {
-    const out = execFileSync('jj', ['--color=never', 'files', filePath], {
+    // Jujutsu requires relative paths from cwd:
+    const relPath = path.isAbsolute(filePath)
+      ? path.relative(cwd, filePath).replace(/\\/g, '/')
+      : filePath.replace(/\\/g, '/');
+
+    const out = execFileSync('jj', ['--color=never', 'files', relPath || '.'], {
       cwd,
       encoding: 'utf8',
       stdio: ['ignore', 'pipe', 'ignore'],