@scrymore/scry-deployer 0.4.0 → 0.5.0

package/bin/cli.js

@@ -147,6 +147,30 @@ async function runDeployment(argv) {
                 '\n⚠️  Metadata was uploaded but not queued for processing.\n' +
                 '   The Storybook is hosted, but its components are NOT being indexed.'
             );
+        } else if (argv.withAnalysis) {
+            // The gap between the two branches above, and the most damaging
+            // state of the three: analysis was asked for, produced nothing, and
+            // this command used to say "Upload successful" and stop. Nothing is
+            // ever indexed, no error is printed, and CI stays green — so the
+            // first sign of trouble is a customer reporting that search is empty
+            // days later (ISSUES.md #24).
+            //
+            // Exit non-zero. The Storybook is hosted, so "failure" overstates it
+            // slightly, but the job asked for was to make components searchable
+            // and that did not happen. A red build is the only signal that gets
+            // acted on.
+            process.exitCode = 1;
+            logger.error(
+                '\n❌ Analysis produced no metadata, so NOTHING WILL BE INDEXED.\n' +
+                '   The Storybook is hosted and browsable, but no component will be\n' +
+                '   searchable from this build.\n\n' +
+                '   You asked for --with-analysis and it did not complete. The cause is\n' +
+                '   in the coverage output above — commonly a missing Playwright browser\n' +
+                '   (run: npx playwright install chromium-headless-shell) or a TypeScript\n' +
+                '   resolution error in the analyzer.\n\n' +
+                '   Exiting non-zero deliberately: a green build here would mean search\n' +
+                '   silently returns nothing.'
+            );
         }
 
     } finally {

package/lib/init.js

@@ -79,14 +79,18 @@ async function runInit(argv) {
         logger.success('✅ Created .github/workflows/deploy-pr-preview.yml\n');
 
         // Step 6: Setup GitHub variables (if gh CLI available)
+        // Tracked, not assumed — the closing summary reports what actually happened.
+        let ghConfigured = null;
         const step6Start = Date.now();
         if (!argv.skipGhSetup && isGhCliAvailable()) {
             logger.info('6/8: Setting up GitHub repository variables...');
             try {
                 await setupGitHubVariables(argv.project, argv.apiKey, argv.apiUrl, logger);
+                ghConfigured = true;
                 const step6Duration = Date.now() - step6Start;
                 logger.success(`✅ GitHub variables configured [${step6Duration}ms]\n`);
             } catch (error) {
+                ghConfigured = false;
                 const step6Duration = Date.now() - step6Start;
                 logger.error(`⚠️  GitHub setup failed: ${error.message} [${step6Duration}ms]`);
                 logger.info('You can set these up manually later.\n');
@@ -126,7 +130,10 @@ async function runInit(argv) {
         logger.info('━'.repeat(50));
         logger.info(`[TIMING] Total setup time: ${totalDuration}ms (${(totalDuration / 1000).toFixed(2)}s)\n`);
         
-        showSuccessMessage(argv.project, envInfo, argv.apiUrl, pushResult.success);
+        showSuccessMessage(argv.project, envInfo, argv.apiUrl, pushResult.success, {
+            ghConfigured,
+            committed: commitResult.success,
+        });
 
     } catch (error) {
         logger.error(`\n❌ Setup failed: ${error.message}`);
@@ -313,22 +320,63 @@ function isGhCliAvailable() {
 /**
  * Setup GitHub variables using gh CLI
  */
-async function setupGitHubVariables(projectId, apiKey, apiUrl, logger) {
+/**
+ * Does this `gh` know about `gh variable`? It arrived in gh 2.21 (December 2022),
+ * and Ubuntu 22.04 still ships 2.4.0 — so plenty of machines do not have it.
+ * Without this check the very first call throws, the secret after it is never
+ * reached, and CI ends up with no credentials at all.
+ */
+function ghSupportsVariables() {
     try {
-        // Set variables
-        execSync(`gh variable set SCRY_PROJECT_ID --body "${projectId}"`, { stdio: 'pipe' });
-        logger.debug('   ✓ Set SCRY_PROJECT_ID');
+        execSync('gh variable --help', { stdio: 'pipe' });
+        return true;
+    } catch {
+        return false;
+    }
+}
 
-        execSync(`gh variable set SCRY_API_URL --body "${apiUrl}"`, { stdio: 'pipe' });
-        logger.debug('   ✓ Set SCRY_API_URL');
+function setVariableViaApi(repo, name, value) {
+    // `gh api` predates `gh variable` by years, so it is the safe fallback.
+    const payload = JSON.stringify({ name, value });
+    try {
+        execSync(`gh api -X POST repos/${repo}/actions/variables --input -`, {
+            stdio: ['pipe', 'pipe', 'pipe'], input: payload,
+        });
+    } catch {
+        // Already present: update rather than create.
+        execSync(`gh api -X PATCH repos/${repo}/actions/variables/${name} --input -`, {
+            stdio: ['pipe', 'pipe', 'pipe'], input: payload,
+        });
+    }
+}
 
-        // Set secret (API key)
-        execSync(`gh secret set SCRY_API_KEY --body "${apiKey}"`, { stdio: 'pipe' });
-        logger.debug('   ✓ Set SCRY_API_KEY');
+async function setupGitHubVariables(projectId, apiKey, apiUrl, logger) {
+    const repo = execSync('gh repo view --json nameWithOwner -q .nameWithOwner', { stdio: 'pipe' })
+        .toString().trim();
 
-    } catch (error) {
-        throw new Error(`Failed to set GitHub variables: ${error.message}`);
+    const useCli = ghSupportsVariables();
+    if (!useCli) {
+        logger.debug('   gh is too old for `gh variable`; using `gh api` instead');
     }
+
+    // Each of these is reported separately. A failure part-way through used to
+    // abandon the rest silently, which is how a repository ended up with the
+    // variables missing *and* the secret missing while setup reported success.
+    const setVar = (name, value) => {
+        if (useCli) execSync(`gh variable set ${name} --body "${value}"`, { stdio: 'pipe' });
+        else setVariableViaApi(repo, name, value);
+    };
+
+    setVar('SCRY_PROJECT_ID', projectId);
+    logger.debug('   ✓ Set SCRY_PROJECT_ID');
+
+    setVar('SCRY_API_URL', apiUrl);
+    logger.debug('   ✓ Set SCRY_API_URL');
+
+    // `gh secret set` needs libsodium encryption, so there is no simple `gh api`
+    // fallback — but it has existed since gh 1.x, so it works where variables do not.
+    execSync(`gh secret set SCRY_API_KEY --body "${apiKey}"`, { stdio: 'pipe' });
+    logger.debug('   ✓ Set SCRY_API_KEY');
 }
 
 /**
@@ -377,12 +425,25 @@ function gitCommit(logger) {
             '.storybook-deployer.json'
         ];
 
+        // Each file is staged on its own. `git add` fails on a path that
+        // .gitignore excludes, and one throw used to abort the loop before the
+        // commit — so a leftover `.storybook-deployer.json` ignore rule from the
+        // pre-0.4.0 workaround silently prevented the *workflows* being committed,
+        // and CI was never set up at all.
+        let staged = 0;
         for (const file of filesToAdd) {
-            if (fs.existsSync(file)) {
+            if (!fs.existsSync(file)) continue;
+            try {
                 execSync(`git add "${file}"`, { stdio: 'pipe' });
+                staged++;
                 logger.debug(`   ✓ Added ${file}`);
+            } catch {
+                logger.debug(`   • Skipped ${file} (ignored by .gitignore)`);
             }
         }
+        if (staged === 0) {
+            return { success: false, message: 'Nothing could be staged' };
+        }
 
         // Commit
         const commitMessage = 'chore: add Scry Storybook deployment workflows';
@@ -433,7 +494,7 @@ function gitPush(logger) {
 /**
  * Show success message
  */
-function showSuccessMessage(projectId, envInfo, apiUrl, pushed) {
+function showSuccessMessage(projectId, envInfo, apiUrl, pushed, results = {}) {
     console.log(`
 🎉 ${pushed ? 'Setup Complete and Deployed!' : 'Setup Complete!'}
 
@@ -442,9 +503,14 @@ Your Storybook deployment is configured and ready to go.
 📦 What was set up:
    ✅ Configuration file (.storybook-deployer.json — no credentials, safe to commit)
    ✅ GitHub Actions workflows (.github/workflows/)
-   ✅ Repository variables (SCRY_PROJECT_ID, SCRY_API_URL)
-   ✅ Repository secret (SCRY_API_KEY)
-   ${pushed ? '✅ Changes committed and pushed' : '⚠️  Manual push required'}
+   ${results.ghConfigured === true
+        ? '✅ Repository variables and secret (SCRY_PROJECT_ID, SCRY_API_URL, SCRY_API_KEY)'
+        : results.ghConfigured === false
+            ? '❌ Repository variables and secret NOT set — CI cannot deploy until you fix this'
+            : '⚠️  Repository variables and secret not attempted — CI cannot deploy until you set them'}
+   ${results.committed === false
+        ? '❌ Changes NOT committed — the workflows are staged but uncommitted'
+        : (pushed ? '✅ Changes committed and pushed' : '⚠️  Committed; manual push required')}
 
 ${!pushed ? `
 📌 Next Step:
@@ -474,4 +540,4 @@ Happy deploying! ✨
 `);
 }
 
-module.exports = { runInit, createConfigFile };
+module.exports = { runInit, createConfigFile, gitCommit };

package/lib/templates.js

@@ -59,11 +59,27 @@ function getCacheValue(packageManager) {
 /**
  * Generate main deployment workflow
  */
+/**
+ * The command that fetches and runs a package that is not a project dependency.
+ *
+ * `npx` cannot be assumed. In a pnpm workflow it resolves against the
+ * pnpm-managed environment and reports "playwright: not found" — exit 127 —
+ * even with --yes. Each package manager has its own equivalent, so use it.
+ */
+function getDlxCommand(packageManager) {
+    switch (packageManager) {
+        case 'pnpm': return 'pnpm dlx';
+        case 'yarn': return 'yarn dlx';
+        default: return 'npx --yes';
+    }
+}
+
 function generateMainWorkflow(projectId, apiUrl, packageManager, buildCmd) {
     const pmSetup = getPackageManagerSetup(packageManager);
     const installCmd = getInstallCommand(packageManager);
     const cache = getCacheValue(packageManager);
     const runCmd = packageManager === 'npm' ? `npm run ${buildCmd}` : `${packageManager} run ${buildCmd}`;
+    const dlx = getDlxCommand(packageManager);
 
     return `# Auto-generated by Scry Storybook Deployer
 # Deploy Storybook to production on push to main branch
@@ -96,6 +112,16 @@ ${cache}
       - name: Build Storybook
         run: ${runCmd}
 
+      - name: Install Playwright browser
+        # Screenshot capture drives --with-analysis, and without a browser every
+        # story fails, no metadata archive is produced, and nothing is ever
+        # indexed — while the deploy still exits 0 and the workflow goes green.
+        # Chromium headless shell only: a few seconds, versus minutes for the
+        # full browser set.
+        # The version is left floating so it tracks sbcov's own playwright
+        # ^1.41.0 optional dependency and installs a matching browser build.
+        run: ${dlx} playwright install --with-deps chromium-headless-shell
+
       - name: Deploy to Scry
         run: |
           npx @scrymore/scry-deployer \\
@@ -121,6 +147,7 @@ function generatePRWorkflow(projectId, apiUrl, packageManager, buildCmd) {
     const installCmd = getInstallCommand(packageManager);
     const cache = getCacheValue(packageManager);
     const runCmd = packageManager === 'npm' ? `npm run ${buildCmd}` : `${packageManager} run ${buildCmd}`;
+    const dlx = getDlxCommand(packageManager);
 
     return `# Auto-generated by Scry Storybook Deployer
 # Deploy Storybook preview for pull requests
@@ -157,6 +184,16 @@ ${cache}
       - name: Build Storybook
         run: ${runCmd}
 
+      - name: Install Playwright browser
+        # Screenshot capture drives --with-analysis, and without a browser every
+        # story fails, no metadata archive is produced, and nothing is ever
+        # indexed — while the deploy still exits 0 and the workflow goes green.
+        # Chromium headless shell only: a few seconds, versus minutes for the
+        # full browser set.
+        # The version is left floating so it tracks sbcov's own playwright
+        # ^1.41.0 optional dependency and installs a matching browser build.
+        run: ${dlx} playwright install --with-deps chromium-headless-shell
+
       - name: Deploy Preview
         id: deploy
         run: |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scrymore/scry-deployer",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "A CLI to automate the deployment of Storybook static builds.",
   "main": "index.js",
   "bin": {