fix

Author: aidenybai

packages/react-doctor/src/cli.ts

@@ -3,7 +3,7 @@ import { existsSync } from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { Command } from "commander";
-import { SEPARATOR_LENGTH_CHARS } from "./constants.js";
+import { OPEN_BASE_URL, SEPARATOR_LENGTH_CHARS } from "./constants.js";
 import { scan } from "./scan.js";
 import type { DiffInfo, ScanOptions } from "./types.js";
 import { copyToClipboard } from "./utils/copy-to-clipboard.js";
@@ -27,6 +27,7 @@ interface CliFlags {
   fix: boolean;
   prompt: boolean;
   yes: boolean;
+  offline: boolean;
   project?: string;
   diff?: boolean | string;
 }
@@ -77,6 +78,7 @@ const program = new Command()
   .option("-y, --yes", "skip prompts, scan all workspace projects")
   .option("--project <name>", "select workspace project (comma-separated for multiple)")
   .option("--diff [base]", "scan only files changed vs base branch")
+  .option("--offline", "skip telemetry (anonymous, not stored, only used to calculate score)")
   .option("--fix", "open Ami to auto-fix all issues")
   .option("--prompt", "copy latest scan output to clipboard")
   .action(async (directory: string, flags: CliFlags) => {
@@ -108,6 +110,7 @@ const program = new Command()
           flags.prompt ||
           (isCliOverride("verbose") ? Boolean(flags.verbose) : (userConfig?.verbose ?? false)),
         scoreOnly: isScoreOnly,
+        offline: flags.offline,
       };
 
       const isAutomatedEnvironment = [
@@ -254,15 +257,25 @@ const openUrl = (url: string): void => {
   execSync(openCommand, { stdio: "ignore" });
 };
 
-const buildDeeplink = (directory: string): string => {
-  const encodedDirectory = encodeURIComponent(path.resolve(directory));
-  const encodedPrompt = encodeURIComponent(DEEPLINK_FIX_PROMPT);
-  return `ami://open-project?cwd=${encodedDirectory}&prompt=${encodedPrompt}&mode=agent&autoSubmit=true`;
+const buildDeeplinkParams = (directory: string): URLSearchParams => {
+  const params = new URLSearchParams();
+  params.set("cwd", path.resolve(directory));
+  params.set("prompt", DEEPLINK_FIX_PROMPT);
+  params.set("mode", "agent");
+  params.set("autoSubmit", "true");
+  return params;
 };
 
+const buildDeeplink = (directory: string): string =>
+  `ami://open-project?${buildDeeplinkParams(directory).toString()}`;
+
+const buildWebDeeplink = (directory: string): string =>
+  `${OPEN_BASE_URL}?${buildDeeplinkParams(directory).toString()}`;
+
 const openAmiToFix = (directory: string): void => {
   const isInstalled = isAmiInstalled();
   const deeplink = buildDeeplink(directory);
+  const webDeeplink = buildWebDeeplink(directory);
 
   if (!isInstalled) {
     if (process.platform === "darwin") {
@@ -274,7 +287,7 @@ const openAmiToFix = (directory: string): void => {
     }
     logger.break();
     logger.dim("Once Ami is running, open this link to start fixing:");
-    logger.info(deeplink);
+    logger.info(webDeeplink);
     return;
   }
 
@@ -286,7 +299,7 @@ const openAmiToFix = (directory: string): void => {
   } catch {
     logger.break();
     logger.dim("Could not open Ami automatically. Open this URL manually:");
-    logger.info(deeplink);
+    logger.info(webDeeplink);
   }
 };
 

packages/react-doctor/src/constants.ts

@@ -24,9 +24,14 @@ export const SCORE_API_URL = "https://www.react.doctor/api/score";
 
 export const SHARE_BASE_URL = "https://www.react.doctor/share";
 
+export const OPEN_BASE_URL = "https://www.react.doctor/open";
+
 export const GIT_LS_FILES_MAX_BUFFER_BYTES = 50 * 1024 * 1024;
 
 export const OFFLINE_MESSAGE =
   "You are offline, could not calculate score. Reconnect to calculate.";
 
+export const OFFLINE_FLAG_MESSAGE =
+  "Score not calculated. Remove --offline to calculate score.";
+
 export const DEFAULT_BRANCH_CANDIDATES = ["main", "master"];

packages/react-doctor/src/scan.ts

@@ -6,6 +6,7 @@ import { performance } from "node:perf_hooks";
 import {
   JSX_FILE_PATTERN,
   MILLISECONDS_PER_SECOND,
+  OFFLINE_FLAG_MESSAGE,
   OFFLINE_MESSAGE,
   PERFECT_SCORE,
   SCORE_BAR_WIDTH_CHARS,
@@ -264,6 +265,7 @@ const printSummary = (
   scoreResult: ScoreResult | null,
   projectName: string,
   totalSourceFileCount: number,
+  noScoreMessage: string,
 ): void => {
   const errorCount = diagnostics.filter((diagnostic) => diagnostic.severity === "error").length;
   const warningCount = diagnostics.filter((diagnostic) => diagnostic.severity === "warning").length;
@@ -326,7 +328,7 @@ const printSummary = (
       ),
     );
     summaryFramedLines.push(createFramedLine(""));
-    summaryFramedLines.push(createFramedLine(OFFLINE_MESSAGE, highlighter.dim(OFFLINE_MESSAGE)));
+    summaryFramedLines.push(createFramedLine(noScoreMessage, highlighter.dim(noScoreMessage)));
     summaryFramedLines.push(createFramedLine(""));
   }
 
@@ -358,6 +360,7 @@ export const scan = async (directory: string, inputOptions: ScanOptions = {}): P
     deadCode: inputOptions.deadCode ?? userConfig?.deadCode ?? true,
     verbose: inputOptions.verbose ?? userConfig?.verbose ?? false,
     scoreOnly: inputOptions.scoreOnly ?? false,
+    offline: inputOptions.offline ?? false,
     includePaths: inputOptions.includePaths,
   };
 
@@ -460,13 +463,14 @@ export const scan = async (directory: string, inputOptions: ScanOptions = {}): P
 
   const elapsedMilliseconds = performance.now() - startTime;
 
-  const scoreResult = await calculateScore(diagnostics);
+  const scoreResult = options.offline ? null : await calculateScore(diagnostics);
+  const noScoreMessage = options.offline ? OFFLINE_FLAG_MESSAGE : OFFLINE_MESSAGE;
 
   if (options.scoreOnly) {
     if (scoreResult) {
       logger.log(`${scoreResult.score}`);
     } else {
-      logger.dim(OFFLINE_MESSAGE);
+      logger.dim(noScoreMessage);
     }
     return;
   }
@@ -478,7 +482,7 @@ export const scan = async (directory: string, inputOptions: ScanOptions = {}): P
       printBranding(scoreResult.score);
       printScoreGauge(scoreResult.score, scoreResult.label);
     } else {
-      logger.dim(`  ${OFFLINE_MESSAGE}`);
+      logger.dim(`  ${noScoreMessage}`);
     }
     return;
   }
@@ -493,5 +497,6 @@ export const scan = async (directory: string, inputOptions: ScanOptions = {}): P
     scoreResult,
     projectInfo.projectName,
     displayedSourceFileCount,
+    noScoreMessage,
   );
 };

packages/react-doctor/src/types.ts

@@ -88,6 +88,7 @@ export interface ScanOptions {
   deadCode?: boolean;
   verbose?: boolean;
   scoreOnly?: boolean;
+  offline?: boolean;
   includePaths?: string[];
 }
 

packages/website/src/app/open/page.tsx

@@ -0,0 +1,77 @@
+"use client";
+
+import { useSearchParams } from "next/navigation";
+import { Suspense, useEffect, useState } from "react";
+
+const AMI_DEEPLINK_PREFIX = "ami://open-project";
+const AMI_RELEASES_URL = "https://github.com/millionco/ami-releases/releases";
+const REDIRECT_DELAY_MS = 500;
+
+const OpenPageContent = () => {
+  const searchParams = useSearchParams();
+  const deeplink = `${AMI_DEEPLINK_PREFIX}?${searchParams.toString()}`;
+  const [didAttemptOpen, setDidAttemptOpen] = useState(false);
+
+  useEffect(() => {
+    const timeout = setTimeout(() => {
+      window.location.href = deeplink;
+      setDidAttemptOpen(true);
+    }, REDIRECT_DELAY_MS);
+
+    return () => clearTimeout(timeout);
+  }, [deeplink]);
+
+  return (
+    <div className="mx-auto flex min-h-screen w-full max-w-3xl flex-col items-center justify-center bg-[#0a0a0a] p-6 font-mono text-base leading-relaxed text-neutral-300 sm:p-8 sm:text-lg">
+      <pre className="mb-6 text-green-400 leading-tight">
+        {`  ┌─────┐\n  │ ◠ ◠ │\n  │  ▽  │\n  └─────┘`}
+      </pre>
+
+      <div className="mb-8 text-center">
+        <h1 className="mb-2 text-xl text-white">Opening Ami...</h1>
+        <p className="text-neutral-500">
+          {didAttemptOpen
+            ? "If Ami didn\u2019t open, it may not be installed."
+            : "Redirecting to Ami to fix react-doctor issues."}
+        </p>
+      </div>
+
+      {didAttemptOpen && (
+        <div className="flex flex-col items-center gap-4">
+          <a
+            href={AMI_RELEASES_URL}
+            target="_blank"
+            rel="noreferrer"
+            className="inline-flex items-center gap-2 border border-white/20 bg-white px-4 py-2 text-black transition-all hover:bg-white/90 active:scale-[0.98]"
+          >
+            Download Ami
+          </a>
+
+          <div className="mt-4 text-center">
+            <p className="mb-2 text-sm text-neutral-500">Or open manually:</p>
+            <a
+              href={deeplink}
+              className="break-all text-sm text-blue-400 underline underline-offset-2 hover:text-blue-300"
+            >
+              Open deeplink
+            </a>
+          </div>
+        </div>
+      )}
+    </div>
+  );
+};
+
+const OpenPage = () => (
+  <Suspense
+    fallback={
+      <div className="flex min-h-screen items-center justify-center bg-[#0a0a0a] font-mono text-neutral-500">
+        Loading...
+      </div>
+    }
+  >
+    <OpenPageContent />
+  </Suspense>
+);
+
+export default OpenPage;