@straitlink/cli 0.3.1 → 0.3.3

package/README.md

@@ -34,7 +34,7 @@ npx @straitlink/cli events add purchase                     # print the code tha
 
 4. Prints the remaining steps for your app: the install command, the AndroidManifest intent filters (or the `app.json` block for Expo), the iPhone Associated Domains entry, what to fill in under Dashboard → Apps & stores, and the `curl` checks for `assetlinks.json` and `apple-app-site-association`. Steps the project already has, such as the intent filters, are marked `[already done]`.
 
-It never edits your existing files and never overwrites an earlier integration file unless you pass `--force`.
+It edits one existing file, and only when needed: in a Flutter or Android (Kotlin) app it adds `<uses-permission android:name="android.permission.INTERNET" />` to `AndroidManifest.xml` if it isn't there. Flutter's own template only has it in debug builds, so without it a release build can't reach Strait and no link or install is matched. Everything else it prints for you to add. It never overwrites an earlier integration file unless you pass `--force`, and `--dry-run` shows the manifest line without writing it.
 
 ## Use
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@straitlink/cli",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Strait deep links from the terminal: init sets them up in a React Native, Expo, Flutter or Android app, doctor checks your link host, links creates and lists links and saves QR codes.",
   "keywords": [
     "strait",

package/src/init.mjs

@@ -1,8 +1,8 @@
 // `strait init`: detect the app, collect key + host + scheme, write the
 // integration file(s), print the platform steps. Pure planning is separated from
 // writing so --dry-run and the tests see exactly what would happen.
-import { existsSync, mkdirSync, writeFileSync } from 'node:fs';
-import { dirname, join, resolve } from 'node:path';
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { dirname, join, relative, resolve } from 'node:path';
 import { FRAMEWORKS, LABELS, describeFolder, detectProject } from './detect.mjs';
 import { INSTALL } from './install.mjs';
 import { integrationFiles, nextSteps } from './templates.mjs';
@@ -11,7 +11,7 @@ import { UsageError, normalizeHost, normalizeKey, normalizeScheme, suggestScheme
 /**
  * Works out everything without touching the disk.
  * @param {object} o { dir, key, host, scheme, framework, force }
- * @returns {{ project, opts, files: Array<{path, abs, contents, why, exists}>, steps, conflicts: string[] }}
+ * @returns {{ project, opts, files: Array<{path, abs, contents, why, exists}>, edits: Array<{path, abs, contents, why, line}>, steps, conflicts: string[] }}
  */
 export function planInit({ dir, key, host, scheme, framework, force = false, installSource = INSTALL }) {
   const root = resolve(dir ?? '.');
@@ -39,7 +39,32 @@ export function planInit({ dir, key, host, scheme, framework, force = false, ins
     return { ...f, abs, exists: existsSync(abs) };
   });
   const conflicts = force ? [] : files.filter((f) => f.exists).map((f) => f.path);
-  return { project, opts, schemeGuessedFrom, files, steps: nextSteps(project, opts, installSource), conflicts };
+  return { project, opts, schemeGuessedFrom, files, edits: manifestEdits(project, root), steps: nextSteps(project, opts, installSource), conflicts };
+}
+
+// Flutter's template declares INTERNET only in the debug/profile manifests and an Android Studio
+// template not at all, so a release build can't reach Strait (QA 9 Nov 2026). React Native's and
+// Expo's templates already have it.
+const INTERNET = '<uses-permission android:name="android.permission.INTERNET" />';
+
+function manifestEdits(project, root) {
+  if (!['flutter', 'android'].includes(project.framework) || !project.manifestPath) return [];
+  const contents = addInternetPermission(readFileSync(project.manifestPath, 'utf8'));
+  if (!contents) return [];
+  const path = relative(root, project.manifestPath).split('\\').join('/');
+  return [{ path, abs: project.manifestPath, contents, line: INTERNET, why: 'adds the INTERNET permission release builds need' }];
+}
+
+/** The manifest with INTERNET added right after <manifest …>, or null when it already has it (or isn't a manifest). */
+export function addInternetPermission(text) {
+  const live = text.replace(/<!--[\s\S]*?-->/g, (c) => ' '.repeat(c.length)); // same length: indexes still match text
+  if (/<uses-permission\b[^>]*android:name\s*=\s*"android\.permission\.INTERNET"/.test(live)) return null;
+  const open = /<manifest\b[^>]*>/.exec(live);
+  if (!open) return null;
+  const eol = text.includes('\r\n') ? '\r\n' : '\n';
+  const at = open.index + open[0].length;
+  const pad = /\n([ \t]*)\S/.exec(live.slice(at))?.[1] || '    '; // the next element's indent
+  return `${text.slice(0, at)}${eol}${pad}${INTERNET}${text.slice(at)}`;
 }
 
 function emptyProject(dir) {
@@ -53,7 +78,8 @@ export function writeFiles(plan) {
     mkdirSync(dirname(f.abs), { recursive: true });
     writeFileSync(f.abs, f.contents);
   }
-  return plan.files.map((f) => f.path);
+  for (const e of plan.edits ?? []) writeFileSync(e.abs, e.contents);
+  return [...plan.files, ...(plan.edits ?? [])].map((f) => f.path);
 }
 
 // ---------- output ----------
@@ -86,12 +112,22 @@ export function formatPlan(plan, { dryRun = false } = {}) {
   // A conflict is a file the real run refuses to overwrite (no --force): never call it replaced.
   const label = (f) => (plan.conflicts.includes(f.path) ? ' (already exists: kept unless --force)' : f.exists ? ' (replaced)' : '');
   for (const f of files) out.push(`  ${f.path}${label(f)}  ${f.why}`);
+  if (plan.edits?.length) {
+    out.push('');
+    out.push(dryRun ? 'Would edit:' : 'Edited:');
+    for (const e of plan.edits) out.push(`  ${e.path}  ${e.why}`);
+  }
   if (dryRun) {
     for (const f of files) {
       out.push('');
       out.push(`--- ${f.path}`);
       out.push(f.contents.trimEnd());
     }
+    for (const e of plan.edits ?? []) {
+      out.push('');
+      out.push(`--- ${e.path}`);
+      out.push(`+    ${e.line}`);
+    }
   }
   out.push('');
   out.push('Next steps:');

package/src/install.mjs

@@ -22,10 +22,10 @@ export const sourceFor = (src, registry) => (typeof src.source === 'string' ? sr
  * `strait events add` reads them too: its iOS/Android eventId needs these SDKs.
  */
 export const VERSIONS = {
-  version: '0.11.0', // Swift (Unity is not written by this CLI)
+  version: '0.11.1', // Swift (Unity is not written by this CLI)
   rnVersion: '0.11.0',
-  flutterVersion: '0.10.1',
-  androidVersion: '0.10.0',
+  flutterVersion: '0.10.2',
+  androidVersion: '0.10.1',
 };
 
 export const PACKAGES = {

package/src/templates.mjs

@@ -353,6 +353,9 @@ export function nextSteps(project, opts, installSrc) {
     const needsFlutterMeta = fw === 'flutter' && !project.manifest.flutterDeepLinkingOff;
     const filtersDone = hasHost && hasScheme;
     const done = filtersDone && !needsFlutterMeta;
+    // init adds INTERNET to a manifest it finds; without one, print it (release builds need it).
+    const noManifest = (fw === 'flutter' || fw === 'android') && !project.manifestPath;
+    const internetXml = '<!-- Inside <manifest>, above <application> -->\n<uses-permission android:name="android.permission.INTERNET" />';
     steps.push({
       title: `Add the intent filters to ${where}`,
       done,
@@ -361,7 +364,7 @@ export function nextSteps(project, opts, installSrc) {
         : filtersDone
           ? `The intent filters for ${host} and ${scheme}:// are already there. Add this line inside your main activity:`
           : 'Inside your main activity. Keep the intent filters you already have.',
-      code: done ? undefined : [filtersDone ? null : manifestXml(opts), needsFlutterMeta ? flutterMeta : null].filter(Boolean).join('\n'),
+      code: done ? undefined : [noManifest ? internetXml : null, filtersDone ? null : manifestXml(opts), needsFlutterMeta ? flutterMeta : null].filter(Boolean).join('\n'),
       lang: 'xml',
     });
     if (fw !== 'android') {

package/src/version.mjs

@@ -1,2 +1,2 @@
 // Kept equal to package.json version (test/links.test.mjs checks it). Sent as the API User-Agent.
-export const CLI_VERSION = '0.3.1';
+export const CLI_VERSION = '0.3.3';