screenci 0.0.72 → 0.0.73

package/README.md

@@ -41,7 +41,7 @@ already know Playwright locators, navigation, and waiting, you already know
 most of the automation layer.
 
 ```ts
-// videos/onboarding.screenci.ts
+// recordings/onboarding.screenci.ts
 import { hide, speed, time, video } from 'screenci'
 
 video('Onboarding flow', async ({ page }) => {

package/dist/cli.d.ts

@@ -45,6 +45,9 @@ export type StudioUploadNotice = {
 };
 export declare function formatStudioUrl(appUrl: string, projectId: string, videoId: string): string;
 type OrgPlan = 'free' | 'starter' | 'business';
+export declare class SignInTimeoutError extends Error {
+    constructor(message: string);
+}
 export declare function withUploadRetry<T>(fn: () => Promise<T>, signal: AbortSignal | undefined, maxAttempts?: number): Promise<T>;
 export declare function attachUploadAbortStdinListener(input: Pick<NodeJS.ReadStream, 'on' | 'off' | 'pause'>, onAbort: (signal: NodeJS.Signals) => void): () => void;
 export declare function collectUploadAssets(data: RecordingData, configDir: string): Promise<PreparedUploadAsset[]>;
@@ -53,6 +56,7 @@ export declare function annotateRecordingDataWithAssetHashes(data: RecordingData
 export declare function formatUploadStartFailureMessage(videoName: string, status: number, responseText: string, secret: string): string;
 export declare function formatFailedVideoMessage(videoName: string, message: string): string;
 export declare function printUploadStartFailureMessage(videoName: string, status: number, responseText: string, secret: string): void;
+export declare function displayAssetPath(assetPath: string): string;
 export declare function uploadRecordings(screenciDir: string, projectName: string, apiUrl: string, secret: string, specificEntry?: string, verbose?: boolean): Promise<{
     projectId: string | null;
     recordId: string | null;
@@ -73,12 +77,14 @@ export declare function extractGrep(args: string[]): string | undefined;
 export declare function ensureScreenciSecret(resolvedConfigPath?: string, opts?: {
     interactive?: boolean;
     pollAuth?: boolean;
+    noPollAuth?: boolean;
 }): Promise<string | undefined>;
 export declare function main(): Promise<void>;
 export declare function parseRecordCliArgs(args: string[]): {
     configPath: string | undefined;
     verbose: boolean;
     pollAuth: boolean;
+    noPollAuth: boolean;
     remote: boolean;
     languages: string | undefined;
     otherArgs: string[];

package/dist/cli.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../cli.ts"],"names":[],"mappings":"AAwBA,OAAO,KAAK,EACV,uBAAuB,EACvB,aAAa,EAEd,MAAM,iBAAiB,CAAA;AASxB,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,iBAAiB,CAAA;AAC/C,OAAO,KAAK,EAAE,kBAAkB,EAAkB,MAAM,gBAAgB,CAAA;AAKxE,OAAO,EAGL,uBAAuB,EACvB,gBAAgB,EAChB,iBAAiB,EAOlB,MAAM,sBAAsB,CAAA;AAQ7B,OAAO,EAAE,uBAAuB,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,CAAA;AAqBvE,KAAK,yBAAyB,GAAG;IAC/B,KAAK,CAAC,EAAE,MAAM,CAAA;IACd,KAAK,CAAC,EAAE,KAAK,CAAC;QAAE,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAA;IAChC,MAAM,CAAC,EAAE,yBAAyB,EAAE,CAAA;CACrC,CAAA;AAUD;;;;;;;;GAQG;AACH,wBAAgB,wBAAwB,CACtC,GAAG,GAAE,MAAM,CAAC,UAAwB,EACpC,MAAM,GAAE;IAAE,KAAK,CAAC,EAAE,OAAO,CAAA;CAAmB,EAC5C,KAAK,GAAE;IAAE,KAAK,CAAC,EAAE,OAAO,CAAA;CAAkB,GACzC,OAAO,CAIT;AAED,wBAAgB,2BAA2B,CACzC,MAAM,EAAE,SAAS,yBAAyB,EAAE,GAC3C,MAAM,EAAE,CAiBV;AAiND,KAAK,mBAAmB,GAAG;IACzB,QAAQ,EAAE,MAAM,CAAA;IAChB,IAAI,EAAE,MAAM,CAAA;IACZ,IAAI,EAAE,MAAM,CAAA;IACZ,IAAI,CAAC,EAAE,MAAM,CAAA;IACb,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,WAAW,CAAC,EAAE,MAAM,CAAA;CACrB,CAAA;AASD,MAAM,MAAM,gBAAgB,GAAG;IAAE,IAAI,EAAE,IAAI,CAAA;CAAE,GAAG;IAAE,OAAO,EAAE,IAAI,CAAA;CAAE,CAAA;AAEjE,MAAM,MAAM,kBAAkB,GAAG;IAC/B,SAAS,EAAE,MAAM,CAAA;IACjB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;IACtB,MAAM,EAAE,gBAAgB,CAAA;CACzB,CAAA;AAED,wBAAgB,eAAe,CAC7B,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,GACd,MAAM,CAER;AAED,KAAK,OAAO,GAAG,MAAM,GAAG,SAAS,GAAG,UAAU,CAAA;AAkE9C,wBAAsB,eAAe,CAAC,CAAC,EACrC,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,EACpB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,WAAW,SAAI,GACd,OAAO,CAAC,CAAC,CAAC,CAWZ;AAoTD,wBAAgB,8BAA8B,CAC5C,KAAK,EAAE,IAAI,CAAC,MAAM,CAAC,UAAU,EAAE,IAAI,GAAG,KAAK,GAAG,OAAO,CAAC,EACtD,OAAO,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,OAAO,KAAK,IAAI,GACxC,MAAM,IAAI,CAiBZ;AAkaD,wBAAsB,mBAAmB,CACvC,IAAI,EAAE,aAAa,EACnB,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,mBAAmB,EAAE,CAAC,CAiGhC;AAED,wBAAgB,cAAc,CAC5B,KAAK,EAAE,QAAQ,GAAG,uBAAuB,GACxC,QAAQ,GAAG,uBAAuB,CAKpC;AAED,wBAAgB,oCAAoC,CAClD,IAAI,EAAE,aAAa,EACnB,MAAM,EAAE,mBAAmB,EAAE,GAC5B,aAAa,CAqEf;AAQD,wBAAgB,+BAA+B,CAC7C,SAAS,EAAE,MAAM,EACjB,MAAM,EAAE,MAAM,EACd,YAAY,EAAE,MAAM,EACpB,MAAM,EAAE,MAAM,GACb,MAAM,CAeR;AAKD,wBAAgB,wBAAwB,CACtC,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,GACd,MAAM,CASR;AAED,wBAAgB,8BAA8B,CAC5C,SAAS,EAAE,MAAM,EACjB,MAAM,EAAE,MAAM,EACd,YAAY,EAAE,MAAM,EACpB,MAAM,EAAE,MAAM,GACb,IAAI,CAcN;AA8GD,wBAAsB,gBAAgB,CACpC,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,EACnB,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,EACd,aAAa,CAAC,EAAE,MAAM,EACtB,OAAO,UAAQ,GACd,OAAO,CAAC;IACT,SAAS,EAAE,MAAM,GAAG,IAAI,CAAA;IACxB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAA;IACvB,WAAW,EAAE,OAAO,CAAA;IACpB,gBAAgB,EAAE,MAAM,EAAE,CAAA;IAC1B,mBAAmB,EAAE,KAAK,CAAC;QAAE,SAAS,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC,CAAA;IAClE,aAAa,EAAE,kBAAkB,EAAE,CAAA;IACnC,IAAI,EAAE,OAAO,GAAG,IAAI,CAAA;CACrB,CAAC,CA8GD;AA8LD,wBAAgB,0BAA0B,CACxC,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,aAAa,GAAG,SAAS,GAClC,MAAM,GAAG,SAAS,CAepB;AAED,wBAAgB,gCAAgC,CAC9C,YAAY,EAAE,MAAM,GACnB,kBAAkB,GAAG,SAAS,CAmBhC;AAED,wBAAgB,wBAAwB,CACtC,YAAY,EAAE,MAAM,GACnB,OAAO,GAAG,SAAS,CAQrB;AAkCD,wBAAgB,wBAAwB,CAAC,kBAAkB,EAAE,MAAM,GAAG,MAAM,CAS3E;AAkGD,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,MAAM,GAAG,SAAS,CAY9D;AA2ND,wBAAsB,oBAAoB,CACxC,kBAAkB,CAAC,EAAE,MAAM,EAC3B,IAAI,GAAE;IAAE,WAAW,CAAC,EAAE,OAAO,CAAC;IAAC,QAAQ,CAAC,EAAE,OAAO,CAAA;CAAO,GACvD,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CA8I7B;AAED,wBAAsB,IAAI,kBA8XzB;AAQD,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG;IAClD,UAAU,EAAE,MAAM,GAAG,SAAS,CAAA;IAC9B,OAAO,EAAE,OAAO,CAAA;IAChB,QAAQ,EAAE,OAAO,CAAA;IACjB,MAAM,EAAE,OAAO,CAAA;IACf,SAAS,EAAE,MAAM,GAAG,SAAS,CAAA;IAC7B,SAAS,EAAE,MAAM,EAAE,CAAA;CACpB,CAoDA;AAgUD,wBAAgB,WAAW,CAAC,KAAK,EAAE,OAAO,GAAG,IAAI,CAchD"}
+{"version":3,"file":"cli.d.ts","sourceRoot":"","sources":["../cli.ts"],"names":[],"mappings":"AA8BA,OAAO,KAAK,EACV,uBAAuB,EACvB,aAAa,EAEd,MAAM,iBAAiB,CAAA;AASxB,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,iBAAiB,CAAA;AAC/C,OAAO,KAAK,EAAE,kBAAkB,EAAkB,MAAM,gBAAgB,CAAA;AAKxE,OAAO,EAGL,uBAAuB,EACvB,gBAAgB,EAChB,iBAAiB,EAOlB,MAAM,sBAAsB,CAAA;AAQ7B,OAAO,EAAE,uBAAuB,EAAE,gBAAgB,EAAE,iBAAiB,EAAE,CAAA;AAyBvE,KAAK,yBAAyB,GAAG;IAC/B,KAAK,CAAC,EAAE,MAAM,CAAA;IACd,KAAK,CAAC,EAAE,KAAK,CAAC;QAAE,KAAK,EAAE,MAAM,CAAA;KAAE,CAAC,CAAA;IAChC,MAAM,CAAC,EAAE,yBAAyB,EAAE,CAAA;CACrC,CAAA;AAUD;;;;;;;;GAQG;AACH,wBAAgB,wBAAwB,CACtC,GAAG,GAAE,MAAM,CAAC,UAAwB,EACpC,MAAM,GAAE;IAAE,KAAK,CAAC,EAAE,OAAO,CAAA;CAAmB,EAC5C,KAAK,GAAE;IAAE,KAAK,CAAC,EAAE,OAAO,CAAA;CAAkB,GACzC,OAAO,CAIT;AAED,wBAAgB,2BAA2B,CACzC,MAAM,EAAE,SAAS,yBAAyB,EAAE,GAC3C,MAAM,EAAE,CAiBV;AAiND,KAAK,mBAAmB,GAAG;IACzB,QAAQ,EAAE,MAAM,CAAA;IAChB,IAAI,EAAE,MAAM,CAAA;IACZ,IAAI,EAAE,MAAM,CAAA;IACZ,IAAI,CAAC,EAAE,MAAM,CAAA;IACb,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,WAAW,CAAC,EAAE,MAAM,CAAA;CACrB,CAAA;AASD,MAAM,MAAM,gBAAgB,GAAG;IAAE,IAAI,EAAE,IAAI,CAAA;CAAE,GAAG;IAAE,OAAO,EAAE,IAAI,CAAA;CAAE,CAAA;AAEjE,MAAM,MAAM,kBAAkB,GAAG;IAC/B,SAAS,EAAE,MAAM,CAAA;IACjB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;IACtB,MAAM,EAAE,gBAAgB,CAAA;CACzB,CAAA;AAED,wBAAgB,eAAe,CAC7B,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,GACd,MAAM,CAER;AAED,KAAK,OAAO,GAAG,MAAM,GAAG,SAAS,GAAG,UAAU,CAAA;AAmD9C,qBAAa,kBAAmB,SAAQ,KAAK;gBAC/B,OAAO,EAAE,MAAM;CAI5B;AAsBD,wBAAsB,eAAe,CAAC,CAAC,EACrC,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,EACpB,MAAM,EAAE,WAAW,GAAG,SAAS,EAC/B,WAAW,SAAI,GACd,OAAO,CAAC,CAAC,CAAC,CAWZ;AAoTD,wBAAgB,8BAA8B,CAC5C,KAAK,EAAE,IAAI,CAAC,MAAM,CAAC,UAAU,EAAE,IAAI,GAAG,KAAK,GAAG,OAAO,CAAC,EACtD,OAAO,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,OAAO,KAAK,IAAI,GACxC,MAAM,IAAI,CAiBZ;AAkaD,wBAAsB,mBAAmB,CACvC,IAAI,EAAE,aAAa,EACnB,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC,mBAAmB,EAAE,CAAC,CAiGhC;AAED,wBAAgB,cAAc,CAC5B,KAAK,EAAE,QAAQ,GAAG,uBAAuB,GACxC,QAAQ,GAAG,uBAAuB,CAKpC;AAED,wBAAgB,oCAAoC,CAClD,IAAI,EAAE,aAAa,EACnB,MAAM,EAAE,mBAAmB,EAAE,GAC5B,aAAa,CAqEf;AAQD,wBAAgB,+BAA+B,CAC7C,SAAS,EAAE,MAAM,EACjB,MAAM,EAAE,MAAM,EACd,YAAY,EAAE,MAAM,EACpB,MAAM,EAAE,MAAM,GACb,MAAM,CAeR;AAKD,wBAAgB,wBAAwB,CACtC,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,MAAM,GACd,MAAM,CASR;AAED,wBAAgB,8BAA8B,CAC5C,SAAS,EAAE,MAAM,EACjB,MAAM,EAAE,MAAM,EACd,YAAY,EAAE,MAAM,EACpB,MAAM,EAAE,MAAM,GACb,IAAI,CAcN;AAED,wBAAgB,gBAAgB,CAAC,SAAS,EAAE,MAAM,GAAG,MAAM,CAM1D;AAkHD,wBAAsB,gBAAgB,CACpC,WAAW,EAAE,MAAM,EACnB,WAAW,EAAE,MAAM,EACnB,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,EACd,aAAa,CAAC,EAAE,MAAM,EACtB,OAAO,UAAQ,GACd,OAAO,CAAC;IACT,SAAS,EAAE,MAAM,GAAG,IAAI,CAAA;IACxB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAA;IACvB,WAAW,EAAE,OAAO,CAAA;IACpB,gBAAgB,EAAE,MAAM,EAAE,CAAA;IAC1B,mBAAmB,EAAE,KAAK,CAAC;QAAE,SAAS,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC,CAAA;IAClE,aAAa,EAAE,kBAAkB,EAAE,CAAA;IACnC,IAAI,EAAE,OAAO,GAAG,IAAI,CAAA;CACrB,CAAC,CA8GD;AA8LD,wBAAgB,0BAA0B,CACxC,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,aAAa,GAAG,SAAS,GAClC,MAAM,GAAG,SAAS,CAepB;AAED,wBAAgB,gCAAgC,CAC9C,YAAY,EAAE,MAAM,GACnB,kBAAkB,GAAG,SAAS,CAmBhC;AAED,wBAAgB,wBAAwB,CACtC,YAAY,EAAE,MAAM,GACnB,OAAO,GAAG,SAAS,CAQrB;AAkCD,wBAAgB,wBAAwB,CAAC,kBAAkB,EAAE,MAAM,GAAG,MAAM,CAS3E;AA8GD,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,MAAM,GAAG,SAAS,CAY9D;AAyND,wBAAsB,oBAAoB,CACxC,kBAAkB,CAAC,EAAE,MAAM,EAC3B,IAAI,GAAE;IACJ,WAAW,CAAC,EAAE,OAAO,CAAA;IACrB,QAAQ,CAAC,EAAE,OAAO,CAAA;IAClB,UAAU,CAAC,EAAE,OAAO,CAAA;CAChB,GACL,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CAoJ7B;AAED,wBAAsB,IAAI,kBAmYzB;AAQD,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG;IAClD,UAAU,EAAE,MAAM,GAAG,SAAS,CAAA;IAC9B,OAAO,EAAE,OAAO,CAAA;IAChB,QAAQ,EAAE,OAAO,CAAA;IACjB,UAAU,EAAE,OAAO,CAAA;IACnB,MAAM,EAAE,OAAO,CAAA;IACf,SAAS,EAAE,MAAM,GAAG,SAAS,CAAA;IAC7B,SAAS,EAAE,MAAM,EAAE,CAAA;CACpB,CAwDA;AAkUD,wBAAgB,WAAW,CAAC,KAAK,EAAE,OAAO,GAAG,IAAI,CAchD"}

package/dist/cli.js

@@ -4,7 +4,7 @@ import { existsSync, mkdirSync, readFileSync, readdirSync, realpathSync, rmSync,
 import { createHash, randomUUID } from 'crypto';
 import { createRequire } from 'module';
 import { appendFile, readdir, readFile, stat, writeFile } from 'fs/promises';
-import { delimiter, dirname, relative as pathRelative, resolve } from 'path';
+import { delimiter, dirname, isAbsolute, relative as pathRelative, resolve, } from 'path';
 import { fileURLToPath, pathToFileURL } from 'url';
 import { Command, CommanderError } from 'commander';
 import pc from 'picocolors';
@@ -27,11 +27,15 @@ const SCREENCI_LINK_SESSION_POLL_INTERVAL_MS = 2_000;
 // sign-in. We poll on a slower cadence than the interactive loop so a long wait
 // for a human to click the link does not hammer the backend.
 const SCREENCI_LINK_SESSION_POLL_FLAG_INTERVAL_MS = 5_000;
-// `record --poll-auth` does not wait forever: after this long without a
-// completed sign-in we stop polling and exit cleanly so an agent or CI step does
-// not hang indefinitely. The link stays valid, so the command can be rerun. The
-// default can be overridden with SCREENCI_POLL_AUTH_TIMEOUT_MS (milliseconds).
+// A waiting `record` does not wait forever: after this long without a completed
+// sign-in we stop polling so a non-interactive agent run does not hang. The link
+// stays valid, so the command can be rerun. The default can be overridden with
+// SCREENCI_POLL_AUTH_TIMEOUT_MS (milliseconds).
 const SCREENCI_LINK_SESSION_POLL_FLAG_TIMEOUT_MS = 5 * 60 * 1_000;
+// An interactive `record` (a human at a terminal) waits longer before giving up,
+// since a person may take a while to open the link and finish signing in. Also
+// overridable with SCREENCI_POLL_AUTH_TIMEOUT_MS.
+const SCREENCI_LINK_SESSION_INTERACTIVE_TIMEOUT_MS = 15 * 60 * 1_000;
 const require = createRequire(import.meta.url);
 /**
  * Reports whether the current session can complete an interactive browser
@@ -184,7 +188,7 @@ function resolveRecordingFileCandidates(filePath, configDir, sourceFilePath) {
     return [
         filePath,
         ...(sourceFileCandidate ? [sourceFileCandidate] : []),
-        resolve(configDir, 'videos', filePath),
+        resolve(configDir, 'recordings', filePath),
         resolve(configDir, pathRelative('/app', filePath)),
     ];
 }
@@ -243,6 +247,17 @@ class RecordFailureHintError extends Error {
         this.cause = cause;
     }
 }
+// Thrown when we wait for a browser sign-in and the deadline passes before it
+// completes. The link stays valid, so the message tells the user to sign in and
+// rerun. A timed-out sign-in means recording did not happen, so the caller maps
+// this to a non-zero exit (it is a failure, not the clean print-and-exit handoff
+// used by CI).
+export class SignInTimeoutError extends Error {
+    constructor(message) {
+        super(message);
+        this.name = 'SignInTimeoutError';
+    }
+}
 function isUploadCancelledError(err) {
     return (err instanceof UploadCancelledError ||
         (err instanceof Error &&
@@ -1017,6 +1032,13 @@ export function printUploadStartFailureMessage(videoName, status, responseText,
     }
     logger.warn(message);
 }
+export function displayAssetPath(assetPath) {
+    if (!isAbsolute(assetPath)) {
+        return assetPath;
+    }
+    const rel = pathRelative(process.cwd(), assetPath);
+    return rel.length > 0 ? rel : assetPath;
+}
 async function uploadAssets(assets, apiUrl, secret, recordingId, signal, throwIfAborted, progressReporter) {
     const logInfo = (message) => {
         if (progressReporter) {
@@ -1050,7 +1072,7 @@ async function uploadAssets(assets, apiUrl, secret, recordingId, signal, throwIf
             }
             const checkBody = (await checkRes.json());
             if (checkBody.exists) {
-                logInfo(`${pc.green('✔')} Asset already exists: ${asset.path}`);
+                logInfo(`${pc.green('✔')} Asset already exists: ${displayAssetPath(asset.path)}`);
                 continue;
             }
             if (!asset.fileBuffer || !asset.contentType) {
@@ -1077,14 +1099,14 @@ async function uploadAssets(assets, apiUrl, secret, recordingId, signal, throwIf
             if (!res.ok) {
                 const text = await res.text();
                 if (res.status === 409 && text.includes('already exists')) {
-                    logInfo(`${pc.green('✔')} Asset already exists: ${asset.path}`);
+                    logInfo(`${pc.green('✔')} Asset already exists: ${displayAssetPath(asset.path)}`);
                 }
                 else {
                     throw new UploadAssetError(`Failed to upload asset ${asset.path}: ${res.status} ${text}${hint401(res.status, secret)}`);
                 }
             }
             else {
-                logInfo(`Asset uploaded: ${asset.path}`);
+                logInfo(`Asset uploaded: ${displayAssetPath(asset.path)}`);
             }
         }
         catch (err) {
@@ -1387,8 +1409,22 @@ async function loadRecordConfigWithoutPlaywrightCollision(resolvedConfigPath) {
 }
 async function requireScreenCISecret(configPath, opts = {}) {
     const { resolvedConfigPath, screenciConfig } = await loadScreenCIConfigAndEnv(configPath);
-    const secret = process.env.SCREENCI_SECRET ??
-        (await ensureScreenciSecret(resolvedConfigPath, opts));
+    let secret;
+    try {
+        secret =
+            process.env.SCREENCI_SECRET ??
+                (await ensureScreenciSecret(resolvedConfigPath, opts));
+    }
+    catch (err) {
+        // A waited-out sign-in means recording did not happen: print the (already
+        // formatted) message and exit non-zero so the failure is loud, instead of
+        // looking like a successful run.
+        if (err instanceof SignInTimeoutError) {
+            logger.error(err.message);
+            process.exit(1);
+        }
+        throw err;
+    }
     if (!secret) {
         // In a non-interactive session ensureScreenciSecret already printed the
         // sign-in link and the next step, so we exit without repeating guidance.
@@ -1567,14 +1603,12 @@ async function pollLinkSessionOnce(spec) {
     }
     return { status };
 }
-function getPollAuthTimeoutMs() {
+function getAuthPollTimeoutMs(defaultMs) {
     const raw = process.env.SCREENCI_POLL_AUTH_TIMEOUT_MS;
     if (raw === undefined)
-        return SCREENCI_LINK_SESSION_POLL_FLAG_TIMEOUT_MS;
+        return defaultMs;
     const parsed = Number(raw);
-    return Number.isFinite(parsed) && parsed >= 0
-        ? parsed
-        : SCREENCI_LINK_SESSION_POLL_FLAG_TIMEOUT_MS;
+    return Number.isFinite(parsed) && parsed >= 0 ? parsed : defaultMs;
 }
 async function pollLinkSession(spec, pollIntervalMs = SCREENCI_LINK_SESSION_POLL_INTERVAL_MS, deadlineEpochMs) {
     for (;;) {
@@ -1598,6 +1632,7 @@ async function pollLinkSession(spec, pollIntervalMs = SCREENCI_LINK_SESSION_POLL
 export async function ensureScreenciSecret(resolvedConfigPath, opts = {}) {
     const interactive = opts.interactive ?? true;
     const pollAuth = opts.pollAuth ?? false;
+    const noPollAuth = opts.noPollAuth ?? false;
     const existingSecret = process.env.SCREENCI_SECRET;
     if (existingSecret)
         return existingSecret;
@@ -1639,69 +1674,80 @@ export async function ensureScreenciSecret(resolvedConfigPath, opts = {}) {
             logger.info(`Successfully saved SCREENCI_SECRET to ${envFilePath}`);
             return secret;
         };
-        if (!interactive) {
-            // Non-interactive sessions cannot complete a browser sign-in here, so by
-            // default we never block. Reuse or create the persisted session and check
-            // its status once; if a stored session is stale, recreate it and check
-            // once more. When the session is already completed (the sign-in happened
-            // between runs) we pick up the secret; otherwise we print the link and
-            // return so the caller can surface it and rerun later. The exception is
-            // `pollAuth` (the `--poll-auth` flag), which opts in to waiting: it keeps
-            // polling on a slow cadence until sign-in completes.
-            let spec = await ensureSpec();
-            let result = await pollLinkSessionOnce(spec);
-            if (result.status === 'expired' ||
-                result.status === 'consumed' ||
-                result.status === 'invalid') {
-                deletePersistedLinkSessionSpec(specPath);
-                spec = await ensureSpec();
-                result = await pollLinkSessionOnce(spec);
-            }
-            if (result.status === 'completed' && result.secret) {
-                return await saveCompletedSecret(result.secret);
-            }
-            if (pollAuth) {
-                // The caller asked us to wait for sign-in. Print the link, then poll on
-                // a slow cadence until the human finishes signing in, and continue
-                // recording automatically once the secret lands. We re-print the link
-                // on every (re)created session so the latest valid link is always
-                // visible, including after a stale session is recreated. We do not wait
-                // forever: after a default timeout we stop polling and exit cleanly so
-                // an agent or CI step does not hang. The link stays valid for a rerun.
-                const timeoutMs = getPollAuthTimeoutMs();
-                const timeoutMinutes = Math.round(timeoutMs / 60_000);
-                const deadlineEpochMs = Date.now() + timeoutMs;
-                for (;;) {
-                    logger.info(`Sign-in required to record. Open this link to sign in:\n${pc.cyan(spec.appUrl)}\n` +
-                        `Waiting for sign-in (checking every 5 seconds, up to ${timeoutMinutes} minutes). Recording continues automatically once you finish.`);
-                    const polled = await pollLinkSession(spec, SCREENCI_LINK_SESSION_POLL_FLAG_INTERVAL_MS, deadlineEpochMs);
-                    if (polled.status === 'completed' && polled.secret) {
-                        return await saveCompletedSecret(polled.secret);
-                    }
-                    if (polled.status === 'timed-out' || Date.now() >= deadlineEpochMs) {
-                        logger.info(`Timed out after ${timeoutMinutes} minutes waiting for sign-in. The link is still valid:\n${pc.cyan(spec.appUrl)}\n` +
-                            `After signing in, rerun ${pc.cyan(getSuggestedScreenciCommand('record'))} to continue, or ${pc.cyan(getSuggestedScreenciCommand('record', '--poll-auth'))} to wait again.`);
-                        return undefined;
-                    }
-                    deletePersistedLinkSessionSpec(specPath);
-                    spec = await ensureSpec();
-                }
-            }
+        // Decide whether this run waits for a browser sign-in or just prints the
+        // link and hands off. A human at a terminal always waits. A plain
+        // non-interactive run (a coding agent: no terminal, no CI) also waits by
+        // default, so `npx screenci record` records without needing a flag. CI and
+        // an explicit SCREENCI_NONINTERACTIVE never wait (CI must not hang and is
+        // expected to provide SCREENCI_SECRET), and `--no-poll-auth` opts back into
+        // that print-and-exit handoff. `--poll-auth` forces waiting and is kept for
+        // backwards compatibility (it is the default for agents now).
+        const isCI = process.env.CI === 'true';
+        const nonInteractiveForced = process.env.SCREENCI_NONINTERACTIVE === '1';
+        const waitForSignIn = noPollAuth
+            ? false
+            : interactive || pollAuth || (!isCI && !nonInteractiveForced);
+        const pollIntervalMs = interactive
+            ? SCREENCI_LINK_SESSION_POLL_INTERVAL_MS
+            : SCREENCI_LINK_SESSION_POLL_FLAG_INTERVAL_MS;
+        const timeoutMs = getAuthPollTimeoutMs(interactive
+            ? SCREENCI_LINK_SESSION_INTERACTIVE_TIMEOUT_MS
+            : SCREENCI_LINK_SESSION_POLL_FLAG_TIMEOUT_MS);
+        // Check the current/stored session once. If sign-in already completed (for
+        // example between runs) we pick up the secret immediately. Recreate a stale
+        // session and check once more before deciding to wait or hand off.
+        let spec = await ensureSpec();
+        let result = await pollLinkSessionOnce(spec);
+        if (result.status === 'expired' ||
+            result.status === 'consumed' ||
+            result.status === 'invalid') {
+            deletePersistedLinkSessionSpec(specPath);
+            spec = await ensureSpec();
+            result = await pollLinkSessionOnce(spec);
+        }
+        if (result.status === 'completed' && result.secret) {
+            return await saveCompletedSecret(result.secret);
+        }
+        if (!waitForSignIn) {
+            // CI / SCREENCI_NONINTERACTIVE / --no-poll-auth: print the link and hand
+            // off without waiting. The caller exits cleanly (0); a pending sign-in
+            // here is an expected handoff, not a failure.
             logger.info(`Sign-in required to record. Open this link to sign in:\n${pc.cyan(spec.appUrl)}\n` +
-                `This session is non-interactive, so sign-in can't complete here. After signing in, rerun ${pc.cyan(getSuggestedScreenciCommand('record'))} to continue, or run ${pc.cyan(getSuggestedScreenciCommand('record', '--poll-auth'))} to print the link again and wait for sign-in, continuing automatically.`);
+                `This session won't wait for sign-in. After signing in, rerun ${pc.cyan(getSuggestedScreenciCommand('record'))} to record. In CI, set SCREENCI_SECRET (from ${getScreenCISecretsUrl()}) to record without an interactive sign-in.`);
             return undefined;
         }
+        // Wait for sign-in: print the link, then poll until the human finishes and
+        // continue recording automatically once the secret lands. We re-print the
+        // link on every (re)created session so the latest valid link is always
+        // visible. We do not wait forever: after the timeout we throw so the caller
+        // exits non-zero (a timed-out sign-in means nothing was recorded). The link
+        // stays valid for a rerun.
+        const timeoutMinutes = Math.max(1, Math.round(timeoutMs / 60_000));
+        const intervalSeconds = Math.max(1, Math.round(pollIntervalMs / 1_000));
+        const deadlineEpochMs = Date.now() + timeoutMs;
         for (;;) {
-            const spec = await ensureSpec();
-            logger.info(`Open this link to sign in and connect the CLI:\n${pc.cyan(spec.appUrl)}\n`);
-            const result = await pollLinkSession(spec);
-            if (result.status === 'completed' && result.secret) {
-                return await saveCompletedSecret(result.secret);
+            logger.info(`Sign-in required to record. Open this link to sign in:\n${pc.cyan(spec.appUrl)}\n` +
+                `Waiting for sign-in (checking every ${intervalSeconds} seconds, up to ${timeoutMinutes} minutes). Recording continues automatically once you finish.`);
+            const polled = await pollLinkSession(spec, pollIntervalMs, deadlineEpochMs);
+            if (polled.status === 'completed' && polled.secret) {
+                return await saveCompletedSecret(polled.secret);
             }
+            if (polled.status === 'timed-out' || Date.now() >= deadlineEpochMs) {
+                throw new SignInTimeoutError(`Timed out after ${timeoutMinutes} minutes waiting for sign-in. The link is still valid:\n${pc.cyan(spec.appUrl)}\n` +
+                    `After signing in, rerun ${pc.cyan(getSuggestedScreenciCommand('record'))} to record.`);
+            }
+            // Session went stale (expired/consumed/invalid) before sign-in: recreate
+            // and keep waiting until the deadline.
             deletePersistedLinkSessionSpec(specPath);
+            spec = await ensureSpec();
         }
     }
     catch (err) {
+        // A timed-out sign-in is a deliberate, already-formatted failure: let it
+        // propagate so the caller can exit non-zero rather than swallowing it as a
+        // generic auth error.
+        if (err instanceof SignInTimeoutError)
+            throw err;
         const msg = err instanceof Error ? err.message : String(err);
         logger.warn(`Authentication failed: ${msg}`);
         logger.info(`You can add SCREENCI_SECRET manually to ${resolvedConfigPath ? await resolveProjectEnvFilePath(resolvedConfigPath) : '.env'} later. Get it from ${getScreenCISecretsUrl()}.`);
@@ -1724,7 +1770,8 @@ export async function main() {
         .command('record [playwrightArgs...]')
         .description('Record videos using Playwright')
         .option('-v, --verbose', 'verbose output')
-        .option('--poll-auth', 'wait for sign-in to complete (polling every 5s, up to 5 minutes) instead of exiting, then continue recording')
+        .option('--poll-auth', 'wait for sign-in before recording (now the default off-CI; kept for backwards compatibility)')
+        .option('--no-poll-auth', 'do not wait for sign-in: print the link and exit cleanly if not signed in (the default under CI)')
         .option('--remote', 'trigger the GitHub Actions recording workflow for this project remotely instead of recording locally')
         .option('--languages <langs>', 'record/render only these languages (comma-separated, e.g. fi,en)')
         .allowUnknownOption(true)
@@ -1740,7 +1787,7 @@ export async function main() {
         }
         let playwrightFailure = null;
         try {
-            await run('record', parsed.otherArgs, parsed.configPath, parsed.verbose, false, parsed.pollAuth, parsed.languages);
+            await run('record', parsed.otherArgs, parsed.configPath, parsed.verbose, false, parsed.pollAuth, parsed.languages, parsed.noPollAuth);
         }
         catch (error) {
             if (!(error instanceof Error))
@@ -1977,6 +2024,7 @@ export function parseRecordCliArgs(args) {
     let configPath;
     let verbose = false;
     let pollAuth = false;
+    let noPollAuth = false;
     let remote = false;
     let languages;
     const otherArgs = [];
@@ -2013,6 +2061,9 @@ export function parseRecordCliArgs(args) {
         else if (arg === '--poll-auth') {
             pollAuth = true;
         }
+        else if (arg === '--no-poll-auth') {
+            noPollAuth = true;
+        }
         else if (arg === '--remote') {
             remote = true;
         }
@@ -2024,6 +2075,7 @@ export function parseRecordCliArgs(args) {
         configPath,
         verbose,
         pollAuth,
+        noPollAuth,
         remote,
         languages,
         otherArgs,
@@ -2154,7 +2206,7 @@ async function fetchRecordOptionsEnv(configPath, verbose) {
         return {};
     }
 }
-async function run(command, additionalArgs, customConfigPath, verbose = false, mockRecord = false, pollAuth = false, languages) {
+async function run(command, additionalArgs, customConfigPath, verbose = false, mockRecord = false, pollAuth = false, languages, noPollAuth = false) {
     const configPath = resolveScreenCIConfigPathOrExit(customConfigPath);
     if (command === 'test' || process.env.SCREENCI_RECORDING !== 'true') {
         await loadEnvFileFromConfigSource(configPath, false);
@@ -2165,6 +2217,7 @@ async function run(command, additionalArgs, customConfigPath, verbose = false, m
             await requireScreenCISecret(configPath, {
                 interactive: detectInteractiveSession(),
                 pollAuth,
+                noPollAuth,
             });
         }
         validateArgs(additionalArgs);