@andrii_kremlovskyi/playwright-failure-analyzer 0.1.0 → 0.1.2

package/README.md

@@ -109,6 +109,10 @@ CI disables stored-login fallback by default. Local usage supports existing CLI
 login; the package never starts an interactive login or stores a supplied token.
 Explicit `allowStoredLogin: false` is available for headless library consumers.
 
+The SDK reads local CLI credentials independently of VS Code's sign-in.
+On Windows, the isolated runtime preserves both `PATH` and `PATHEXT` so it can
+discover `gh.exe` and reuse GitHub CLI authentication.
+
 In GitHub Actions, pass `${{ github.token }}` through `COPILOT_GITHUB_TOKEN` and
 grant the job `copilot-requests: write`. Organization-owned repositories require
 the **Allow use of Copilot CLI billed to the organization** policy. This does not
@@ -297,6 +301,10 @@ configuration discovery, and run in an isolated working directory. This limits
 agent actions; it does not make sensitive evidence safe to disclose. BYOK/provider
 environment overrides are not forwarded to the owned runtime.
 
+Session state stays in an owned temporary configuration directory, which is
+removed after the sessions disconnect and the runtime stops. Local login
+configuration is kept separate and is not deleted during analysis cleanup.
+
 Screenshots can be extracted for human inspection, but are not automatically
 submitted for visual analysis. Global setup/infrastructure errors and failures
 without trace evidence are not diagnosed by a report-only model fallback.
@@ -310,9 +318,11 @@ npm pack --dry-run
 npm pack
 ```
 
-Tests use generated reports and fake model clients. The runtime smoke test starts
-the bundled runtime and requests local status only, using a synthetic credential;
-it performs no model inference. No real reports or credentials belong in fixtures.
+Tests use generated reports and fake model clients. Real-runtime checks cover
+startup and concurrent session creation/disconnection with both explicit-token
+and separate local-login storage. They use synthetic credentials and an inert
+loopback provider, verify temporary-state cleanup, and make no model inference
+requests. No real reports or credentials belong in fixtures.
 
 Release checks cover Windows (Node 22) and Linux (Node 24), including independent
 tarball installation, CommonJS/ESM imports, declarations, the executable, and

package/dist/copilot-client.d.ts

@@ -23,6 +23,7 @@ export declare function resolveAuthentication(options?: AuthenticationOptions, e
     gitHubToken?: string;
     useLoggedInUser: boolean;
 };
+export declare function runtimeEnvironment(env?: NodeJS.ProcessEnv): NodeJS.ProcessEnv;
 export declare function usingClient<T>(authentication: AuthenticationOptions, callback: (client: AnalysisClient, sdk: {
     getAuthStatus(): Promise<{
         isAuthenticated: boolean;

package/dist/copilot-client.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.evidenceSessionConfig = void 0;
 exports.resolveAuthentication = resolveAuthentication;
+exports.runtimeEnvironment = runtimeEnvironment;
 exports.usingClient = usingClient;
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_os_1 = __importDefault(require("node:os"));
@@ -46,13 +47,16 @@ function resolveAuthentication(options = {}, env = process.env) {
         useLoggedInUser: !token && allowStoredLogin,
     };
 }
+function runtimeEnvironment(env = process.env) {
+    // Keep OS/login/proxy support, not application secrets, NODE_OPTIONS or provider overrides.
+    // PATHEXT is needed to discover gh.exe for stored GitHub CLI authentication on Windows.
+    return Object.fromEntries(Object.entries(env).filter(([name]) => /^(PATH|PATHEXT|SYSTEMROOT|WINDIR|COMSPEC|HOME|USERPROFILE|HOMEDRIVE|HOMEPATH|TMP|TEMP|TMPDIR|APPDATA|LOCALAPPDATA|XDG_CONFIG_HOME|XDG_DATA_HOME|XDG_CACHE_HOME|HTTP_PROXY|HTTPS_PROXY|NO_PROXY|SSL_CERT_FILE|SSL_CERT_DIR|NODE_EXTRA_CA_CERTS|GH_HOST|GH_CONFIG_DIR)$/i.test(name)));
+}
 async function usingClient(authentication, callback, signal) {
     const auth = resolveAuthentication(authentication);
     (0, shared_1.assertNotAborted)(signal);
     const { CopilotClient, RuntimeConnection } = await import("@github/copilot-sdk");
     const root = node_fs_1.default.mkdtempSync(node_path_1.default.join(node_os_1.default.tmpdir(), "playwright-failure-analyzer-"));
-    // Keep OS/login/proxy support, not application secrets, NODE_OPTIONS or provider overrides.
-    const environment = Object.fromEntries(Object.entries(process.env).filter(([name]) => /^(PATH|SYSTEMROOT|WINDIR|COMSPEC|HOME|USERPROFILE|HOMEDRIVE|HOMEPATH|TMP|TEMP|TMPDIR|APPDATA|LOCALAPPDATA|XDG_CONFIG_HOME|XDG_DATA_HOME|XDG_CACHE_HOME|HTTP_PROXY|HTTPS_PROXY|NO_PROXY|SSL_CERT_FILE|SSL_CERT_DIR|NODE_EXTRA_CA_CERTS|GH_HOST|GH_CONFIG_DIR)$/i.test(name)));
     const options = {
         ...auth,
         connection: RuntimeConnection.forStdio(),
@@ -61,7 +65,7 @@ async function usingClient(authentication, callback, signal) {
         baseDirectory: auth.useLoggedInUser
             ? process.env.COPILOT_HOME || node_path_1.default.join(node_os_1.default.homedir(), ".copilot")
             : root,
-        env: environment,
+        env: runtimeEnvironment(),
         logLevel: "none",
     };
     const sdk = new CopilotClient(options);
@@ -93,7 +97,9 @@ async function usingClient(authentication, callback, signal) {
                     disconnect: async () => {
                         if (signal?.aborted)
                             return; // The owned runtime is being forcibly stopped.
-                        await (0, shared_1.withCleanup)(() => session.disconnect(), () => sdk.deleteSession(session.sessionId));
+                        // deleteSession searches baseDirectory, not the isolated session configDirectory.
+                        // The owned session files are removed below, after the runtime stops.
+                        await session.disconnect();
                     },
                 };
             },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andrii_kremlovskyi/playwright-failure-analyzer",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Evidence-based Playwright failure analysis with GitHub Copilot, for local tools and CI",
   "license": "MIT",
   "type": "commonjs",