@bastani/atomic 0.8.25-alpha.1 → 0.8.26-alpha.1

package/dist/modes/interactive/interactive-mode.js

@@ -31,6 +31,7 @@ import { parseGitUrl } from "../../utils/git.js";
 import { getCwdRelativePath } from "../../utils/paths.js";
 import { getPiUserAgent } from "../../utils/pi-user-agent.js";
 import { killTrackedDetachedChildren } from "../../utils/shell.js";
+import { recordTimeSinceReset } from "../../core/timings.js";
 import { ensureTool } from "../../utils/tools-manager.js";
 import { checkForNewPiVersion } from "../../utils/version-check.js";
 import { ArminComponent } from "./components/armin.js";
@@ -404,10 +405,6 @@ export class InteractiveMode {
         this.registerSignalHandlers();
         // Load changelog (only show new entries, skip for resumed sessions)
         this.changelogMarkdown = this.getChangelogForDisplay();
-        // Ensure fd and rg are available (downloads if missing, adds to PATH via getBinDir)
-        // Both are needed: fd for autocomplete, rg for grep tool and bash commands
-        const [fdPath] = await Promise.all([ensureTool("fd"), ensureTool("rg")]);
-        this.fdPath = fdPath;
         // Add header container as first child
         this.ui.addChild(this.headerContainer);
         // Add the quiet startup identity (unless silenced). Resource details are
@@ -444,9 +441,21 @@ export class InteractiveMode {
         this.ui.setFocus(this.editor);
         this.setupKeyHandlers();
         this.setupEditorSubmitHandler();
-        // Start the UI before initializing extensions so session_start handlers can use interactive dialogs
+        // Start the UI before initializing extensions so session_start handlers can use interactive dialogs.
+        // fd/rg readiness is intentionally checked after first paint because ensureTool may spawn
+        // or download tools on cold machines.
         this.ui.start();
+        recordTimeSinceReset("time-to-first-frame");
         this.isInitialized = true;
+        void Promise.all([ensureTool("fd"), ensureTool("rg")])
+            .then(([fdPath]) => {
+            this.fdPath = fdPath;
+            this.setupAutocompleteProvider();
+        })
+            .catch((error) => {
+            const message = error instanceof Error ? error.message : String(error);
+            console.error(`Tool readiness check failed: ${message}`);
+        });
         // Initialize extensions first so resources are shown before messages
         await this.rebindCurrentSession();
         // Render initial messages AFTER showing loaded resources
@@ -461,8 +470,11 @@ export class InteractiveMode {
         this.footerDataProvider.onBranchChange(() => {
             this.ui.requestRender();
         });
-        // Initialize available provider count for footer display
-        await this.updateAvailableProviderCount();
+        // Initialize available provider count for footer display without delaying first-frame startup.
+        void this.updateAvailableProviderCount().catch((error) => {
+            const message = error instanceof Error ? error.message : String(error);
+            console.error(`Failed to update provider count: ${message}`);
+        });
     }
     /**
      * Update terminal title with session name and cwd.