pkg-pr-new 0.0.7 → 0.0.9

package/index.ts

@@ -6,21 +6,27 @@ import { createHash } from "node:crypto";
 import { hash } from "ohash";
 import fsSync from "fs";
 import fs from "fs/promises";
-import { Octokit } from "@octokit/action";
-import { getPackageManifest } from "query-registry";
-import { extractOwnerAndRepo, extractRepository } from "@pkg-pr-new/utils";
+import { getPackageManifest, type PackageManifest } from "query-registry";
+import type { Comment } from "@pkg-pr-new/utils";
+import {
+  abbreviateCommitHash,
+  extractOwnerAndRepo,
+  extractRepository,
+} from "@pkg-pr-new/utils";
 import fg from "fast-glob";
 import ignore from "ignore";
 import "./environments";
 import pkg from "./package.json" with { type: "json" };
 import { isBinaryFile } from "isbinaryfile";
 import { readPackageJSON, writePackageJSON } from "pkg-types";
+import { createDefaultTemplate } from "./template";
 
 declare global {
   var API_URL: string;
 }
 
-const publishUrl = new URL("/publish", API_URL);
+const apiUrl = process.env.API_URL ?? API_URL;
+const publishUrl = new URL("/publish", apiUrl);
 
 const main = defineCommand({
   meta: {
@@ -46,6 +52,11 @@ const main = defineCommand({
             description:
               "generate stackblitz templates out of directories in the current repo with the new built packages",
           },
+          comment: {
+            type: "string", // "off", "create", "update" (default)
+            description: `"off" for no comments (silent mode). "create" for comment on each publish. "update" for one comment across the pull request with edits on each publish (default)`,
+            default: "update",
+          },
         },
         run: async ({ args }) => {
           const paths = (args._.length ? args._ : ["."])
@@ -55,7 +66,7 @@ const main = defineCommand({
           const templates = (
             typeof args.template === "string"
               ? [args.template]
-              : ([...(args.template ?? [])] as string[])
+              : ([...(args.template || [])] as string[])
           )
             .flatMap((p) => (fg.isDynamicPattern(p) ? fg.sync(p) : p))
             .map((p) => path.resolve(p));
@@ -65,6 +76,8 @@ const main = defineCommand({
           const isCompact = !!args.compact;
           const isPnpm = !!args.pnpm;
 
+          const comment: Comment = args.comment as Comment;
+
           if (!process.env.TEST && process.env.GITHUB_ACTIONS !== "true") {
             console.error(
               "Continuous Releases are only available in Github Actions.",
@@ -72,14 +85,11 @@ const main = defineCommand({
             process.exit(1);
           }
 
-          new Octokit(); // gh authentication
-
           const {
             GITHUB_REPOSITORY,
             GITHUB_RUN_ID,
             GITHUB_RUN_ATTEMPT,
             GITHUB_ACTOR_ID,
-            GITHUB_JOB
           } = process.env;
 
           const [owner, repo] = GITHUB_REPOSITORY.split("/");
@@ -87,15 +97,14 @@ const main = defineCommand({
           const metadata = {
             owner,
             repo,
-            job: GITHUB_JOB,
-            runId: Number(GITHUB_RUN_ID),
+            run: Number(GITHUB_RUN_ID),
             attempt: Number(GITHUB_RUN_ATTEMPT),
             actor: Number(GITHUB_ACTOR_ID),
           };
 
           const key = hash(metadata);
 
-          const checkResponse = await fetch(new URL("/check", API_URL), {
+          const checkResponse = await fetch(new URL("/check", apiUrl), {
             method: "POST",
             body: JSON.stringify({
               owner,
@@ -110,6 +119,7 @@ const main = defineCommand({
           }
 
           const { sha } = await checkResponse.json();
+          const abbreviatedSha = abbreviateCommitHash(sha);
 
           const deps: Map<string, string> = new Map();
 
@@ -128,8 +138,8 @@ const main = defineCommand({
             deps.set(
               pJson.name,
               new URL(
-                `/${owner}/${repo}/${pJson.name}@${sha}`,
-                API_URL,
+                `/${owner}/${repo}/${pJson.name}@${abbreviatedSha}`,
+                apiUrl,
               ).href,
             );
           }
@@ -177,6 +187,21 @@ const main = defineCommand({
             await restore();
           }
 
+          const noDefaultTemplate = args.template === false;
+
+          if (!templates.length && !noDefaultTemplate) {
+            const project = createDefaultTemplate(
+              Object.fromEntries(deps.entries()),
+            );
+
+            for (const filePath of Object.keys(project)) {
+              formData.append(
+                `template:default:${encodeURIComponent(filePath)}`,
+                project[filePath],
+              );
+            }
+          }
+
           const restoreMap = new Map<
             string,
             Awaited<ReturnType<typeof writeDeps>>
@@ -219,6 +244,7 @@ const main = defineCommand({
           const res = await fetch(publishUrl, {
             method: "POST",
             headers: {
+              "sb-comment": comment,
               "sb-compact": `${isCompact}`,
               "sb-key": key,
               "sb-shasums": JSON.stringify(shasums),
@@ -259,30 +285,18 @@ runMain(main);
 
 // TODO: we'll add support for yarn if users hit issues with npm
 async function resolveTarball(pm: "npm" | "pnpm", p: string) {
-  if (pm === "npm") {
-    const { stdout } = await ezSpawn.async("npm pack --json", {
-      stdio: "overlapped",
-      cwd: p,
-    });
-
-    const { filename, shasum }: { filename: string; shasum: string } =
-      JSON.parse(stdout)[0];
-
-    return { filename, shasum };
-  } else if (pm === "pnpm") {
-    const { stdout } = await ezSpawn.async("pnpm pack", {
-      stdio: "overlapped",
-      cwd: p,
-    });
-    const filename = stdout.trim();
-
-    const shasum = createHash("sha1")
-      .update(await fs.readFile(path.resolve(p, filename)))
-      .digest("hex");
-
-    return { filename, shasum };
-  }
-  throw new Error("Could not resolve package manager");
+  const { stdout } = await ezSpawn.async(`${pm} pack`, {
+    stdio: "overlapped",
+    cwd: p,
+  });
+  const lines = stdout.split("\n").filter(Boolean);
+  const filename = lines[lines.length - 1].trim();
+
+  const shasum = createHash("sha1")
+    .update(await fs.readFile(path.resolve(p, filename)))
+    .digest("hex");
+
+  return { filename, shasum };
 }
 
 async function writeDeps(p: string, deps: Map<string, string>) {
@@ -314,23 +328,33 @@ function hijackDeps(
 }
 
 async function verifyCompactMode(packageName: string) {
-  const error = new Error(
-    `pkg-pr-new cannot resolve ${packageName} from npm. --compact flag depends on the package being available in npm.
-Make sure to have your package on npm first or configure the 'repository' field in your package.json properly.`,
-  );
+  let manifest: PackageManifest;
+
   try {
-    const manifest = await getPackageManifest(packageName);
+    manifest = await getPackageManifest(packageName);
+  } catch {
+    throw new Error(
+      `pkg-pr-new cannot resolve ${packageName} from npm. --compact flag depends on the package being available in npm.
+Make sure to have your package on npm first.`,
+    );
+  }
 
-    const repository = extractRepository(manifest);
-    if (!repository) {
-      throw error;
-    }
+  const instruction = `Make sure to configure the 'repository' / 'repository.url' field in its package.json properly.
+See https://docs.npmjs.com/cli/v10/configuring-npm/package-json#repository for details.`;
 
-    const match = extractOwnerAndRepo(repository);
-    if (!match) {
-      throw error;
-    }
-  } catch {
-    throw error;
+  const repository = extractRepository(manifest);
+  if (!repository) {
+    throw new Error(
+      `pkg-pr-new cannot extract the repository link from the ${packageName} manifest. --compact flag requires the link to be present.
+${instruction}`,
+    );
+  }
+
+  const match = extractOwnerAndRepo(repository);
+  if (!match) {
+    throw new Error(
+      `pkg-pr-new cannot extract the owner and repo names from the ${packageName} repository link: ${repository}. --compact flag requires these names.
+${instruction}`,
+    );
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pkg-pr-new",
-  "version": "0.0.7",
+  "version": "0.0.9",
   "description": "",
   "main": "index.js",
   "type": "module",
@@ -22,7 +22,7 @@
     "ignore": "^5.3.1",
     "isbinaryfile": "^5.0.2",
     "pkg-types": "^1.1.1",
-    "query-registry": "^3.0.0"
+    "query-registry": "^3.0.1"
   },
   "devDependencies": {
     "@pkg-pr-new/utils": "workspace:^",

package/template.ts

@@ -0,0 +1,54 @@
+export const createDefaultTemplate = (
+  dependencies: Record<string, string>,
+) => ({
+  "index.js": "",
+  "README.md": `
+# Default Template
+
+This is a template that leverages the experimental templates feature in the \`pkg.pr.new\` tool.
+
+## Overview
+
+Templates are particularly useful for creating live, interactive examples of your packages, which can be very beneficial for both development and documentation purposes.
+
+As a user, you can check the package.json file and see the new generated packages! You can just copy those and put them in your package.json or install them with your favorite package manager.
+
+${Object.values(dependencies)
+  .map(
+    (url) => `
+\`\`\`sh
+npm i ${url}
+\`\`\`
+`,
+  )
+  .join("")}
+
+## Usage
+
+To use this feature as a maintainer, you can run the following command:
+
+\`\`\`sh
+npx pkg-pr-new publish ./packages/A --template ./examples/*
+\`\`\`
+
+## Benefits
+
+- Interactive Demos: Automatically create live demos that users can interact with directly in their browser.
+- Enhanced Testing: Quickly spin up environments to test your package in different scenarios.
+- Improved Sharing: Easily share working examples of your package with collaborators or users without needing them to set up their environment.
+`,
+  "package.json": JSON.stringify(
+    {
+      name: "default",
+      version: "1.0.0",
+      description: "generated by pkg.pr.new",
+      main: "index.js",
+      dependencies,
+      keywords: [],
+      author: "pkg.pr.new",
+      license: "ISC",
+    },
+    null,
+    2,
+  ),
+});