deeplink-parity 0.6.0 → 0.6.2

package/dist/cli.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { writeFile } from 'node:fs/promises';
+import { stat, writeFile } from 'node:fs/promises';
 import { resolve } from 'node:path';
 import { localSource, networkSource } from './fetch/wellKnown.js';
 import { exitCodeFor, printReport } from './report/console.js';
@@ -86,12 +86,35 @@ function parseArgs(argv) {
         format,
     };
 }
+/**
+ * A mistyped path must fail loudly. Scanning a directory that does not exist would
+ * read exactly like an empty repository — "no candidates", "nothing declared" — and
+ * a wrong answer that looks like a clean one is the failure mode this tool exists
+ * to prevent.
+ */
+async function assertRootsExist(roots) {
+    for (const root of roots) {
+        let ok = false;
+        try {
+            ok = (await stat(root)).isDirectory();
+        }
+        catch {
+            // fall through to the error below
+        }
+        if (!ok) {
+            console.error(`Not a directory: ${root}`);
+            console.error('Pass the path to each checkout, e.g. deeplink-parity ./app-ios ./app-android');
+            process.exit(2);
+        }
+    }
+}
 async function main() {
     const opts = parseArgs(process.argv);
     if (opts.help) {
         console.log(USAGE);
         return;
     }
+    await assertRootsExist(opts.roots);
     if (opts.command === 'init') {
         process.exit(await runInit(opts.roots, opts.yes));
     }

package/dist/routes/candidates.js

@@ -53,8 +53,8 @@ const SHAPES = [
     { name: 'when/switch branch', pattern: String.raw `"(\/[^"\s]+)"\s*->` },
     { name: 'any path string', pattern: String.raw `"(\/[a-z0-9][a-z0-9\/_-]*)"` },
 ];
-export async function suggestRegex(file) {
-    const content = await readFile(file, 'utf8');
+export async function suggestRegex(files) {
+    const content = (await Promise.all(files.map((f) => readFile(f, 'utf8')))).join('\n');
     const suggestions = [];
     for (const shape of SHAPES) {
         const paths = [

package/dist/routes/init.js

@@ -13,32 +13,41 @@ function rootRelative(file, roots) {
     }
     return file;
 }
-async function pickFile(rl, platform, candidates, yes) {
+async function pickFiles(rl, platform, candidates, yes) {
     const top = candidates.filter((c) => c.platform === platform).slice(0, 5);
     if (top.length === 0) {
         if (yes)
-            return undefined;
+            return [];
         const manual = (await rl.question(`No ${platform} route-file candidates found. Enter a path, or leave blank to skip: `)).trim();
-        return manual || undefined;
+        return manual ? [manual] : [];
     }
     if (yes)
-        return top[0].file;
+        return [top[0].file];
     console.log(`\n${platform === 'ios' ? 'iOS' : 'Android'} route file candidates:`);
     top.forEach((c, i) => console.log(`  ${i + 1}. ${c.file}  (${c.pathStrings} path strings)`));
     console.log(`  m. enter a path manually`);
     console.log(`  s. skip ${platform}`);
-    const answer = (await rl.question('> ')).trim().toLowerCase();
-    if (answer === 's' || answer === '')
-        return undefined;
-    if (answer === 'm') {
-        const manual = (await rl.question('path: ')).trim();
-        return manual || undefined;
+    console.log(`  (several numbers select several files, e.g. "1 2")`);
+    // an answer that cannot be interpreted is asked again — never silently narrowed
+    for (;;) {
+        const answer = (await rl.question('> ')).trim().toLowerCase();
+        if (answer === 's' || answer === '')
+            return [];
+        if (answer === 'm') {
+            const manual = (await rl.question('path: ')).trim();
+            return manual ? [manual] : [];
+        }
+        const tokens = answer.split(/[,\s]+/);
+        const indexes = tokens.map((t) => Number.parseInt(t, 10));
+        const valid = indexes.length > 0 &&
+            indexes.every((n, i) => Number.isInteger(n) && String(n) === tokens[i] && n >= 1 && n <= top.length);
+        if (valid)
+            return [...new Set(indexes)].map((n) => top[n - 1].file);
+        console.log(`  Could not read "${answer}" — numbers 1-${top.length}, "m" or "s".`);
     }
-    const index = Number.parseInt(answer, 10);
-    return Number.isInteger(index) && index >= 1 && index <= top.length ? top[index - 1].file : undefined;
 }
-async function pickRegex(rl, file, yes) {
-    const suggestions = await suggestRegex(file);
+async function pickRegex(rl, files, yes) {
+    const suggestions = await suggestRegex(files);
     if (suggestions.length > 0) {
         const best = suggestions[0];
         const preview = best.paths.slice(0, PREVIEW).join(', ');
@@ -70,13 +79,15 @@ export async function runInit(roots, yes) {
     try {
         const picks = {};
         for (const platform of ['ios', 'android']) {
-            const file = await pickFile(rl, platform, candidates, yes);
-            if (!file)
+            const files = await pickFiles(rl, platform, candidates, yes);
+            if (files.length === 0)
                 continue;
-            const match = await pickRegex(rl, file, yes);
+            const match = await pickRegex(rl, files, yes);
             if (!match)
                 continue;
-            picks[platform] = { file: rootRelative(file, roots), match };
+            const relative = files.map((f) => rootRelative(f, roots));
+            picks[platform] =
+                relative.length === 1 ? { file: relative[0], match } : { files: relative, match };
         }
         if (!picks.ios && !picks.android) {
             console.log('\nNothing selected — no config written.');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "deeplink-parity",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "description": "Checks that what your mobile app declares about deep links matches what is actually hosted — across iOS and Android.",
   "type": "module",
   "bin": {