poe-code 3.0.275 → 3.0.276

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poe-code",
-  "version": "3.0.275",
+  "version": "3.0.276",
   "description": "CLI tool to configure Poe API for developer workflows.",
   "type": "module",
   "main": "./dist/index.js",

package/packages/package-lint/dist/model.js

@@ -14,6 +14,11 @@ function toStringRecord(value) {
     }
     return out;
 }
+function toBinRecord(value, packageName) {
+    if (typeof value === "string")
+        return { [packageName]: value };
+    return toStringRecord(value);
+}
 function toStringArray(value) {
     return Array.isArray(value) ? value.filter((v) => typeof v === "string") : [];
 }
@@ -46,7 +51,7 @@ async function loadPackage(fs, rootDir, relDir, isRoot) {
         repositoryDirectory: typeof repository?.directory === "string" ? repository.directory : undefined,
         ecosystem,
         exports: pkg.exports,
-        bin: toStringRecord(pkg.bin),
+        bin: toBinRecord(pkg.bin, typeof pkg.name === "string" ? pkg.name : relDir),
         files: Array.isArray(pkg.files)
             ? pkg.files.filter((f) => typeof f === "string")
             : [],
@@ -88,6 +93,29 @@ function normalizeWorkingDir(wd) {
         p = p.slice(2);
     return p.endsWith("/") ? p.slice(0, -1) : p;
 }
+function splitShellLine(line) {
+    return line
+        .trim()
+        .split(" ")
+        .map((part) => part.trim())
+        .filter(Boolean);
+}
+function hasPublishRun(run) {
+    for (const line of run.split("\n")) {
+        const trimmed = line.trim();
+        if (trimmed.length === 0 || trimmed.startsWith("#"))
+            continue;
+        const parts = splitShellLine(trimmed);
+        if (parts[0] === "npm" && parts[1] === "publish")
+            return true;
+        if (parts.includes("semantic-release"))
+            return true;
+    }
+    return false;
+}
+function isPypiPublishAction(uses) {
+    return uses === "pypa/gh-action-pypi-publish" || uses.startsWith("pypa/gh-action-pypi-publish@");
+}
 function parseLockstepGroup(value) {
     if (typeof value !== "string")
         return { dirs: [], valid: false };
@@ -115,17 +143,19 @@ function parseReleaseWorkflow(file, raw) {
     const lockstepGroups = [];
     const jobs = doc?.jobs && typeof doc.jobs === "object" ? doc.jobs : {};
     for (const job of Object.values(jobs)) {
+        const workflowJob = job && typeof job === "object" ? job : {};
         const activeLockstepGroups = [];
-        const steps = Array.isArray(job?.steps)
-            ? job.steps
-            : [];
+        const steps = Array.isArray(workflowJob.steps) ? workflowJob.steps : [];
+        const defaultWorkingDir = typeof workflowJob.defaults?.run?.["working-directory"] === "string"
+            ? normalizeWorkingDir(workflowJob.defaults.run["working-directory"])
+            : undefined;
         for (const step of steps) {
             const run = typeof step.run === "string" ? step.run : "";
             const uses = typeof step.uses === "string" ? step.uses : "";
             const wd = typeof step["working-directory"] === "string"
                 ? normalizeWorkingDir(step["working-directory"])
-                : undefined;
-            if (run.includes("npm publish") || run.includes("semantic-release")) {
+                : defaultWorkingDir;
+            if (hasPublishRun(run)) {
                 const targetDir = wd ?? ".";
                 targetDirs.add(targetDir);
                 for (const group of activeLockstepGroups) {
@@ -133,14 +163,14 @@ function parseReleaseWorkflow(file, raw) {
                         group.publishedDirs.push(targetDir);
                 }
             }
-            if (uses.startsWith("pypa/gh-action-pypi-publish")) {
+            if (isPypiPublishAction(uses)) {
                 const pd = step.with?.["packages-dir"];
                 if (typeof pd === "string")
                     targetDirs.add(dirFromArtifactPath(pd));
             }
             if (uses === "./.github/actions/prepare-lockstep-release") {
                 const parsed = parseLockstepGroup(step.with?.packages);
-                const validVersion = typeof step.with?.version === "string" && step.with.version.length > 0;
+                const validVersion = typeof step.with?.version === "string" && step.with.version.trim().length > 0;
                 const group = { ...parsed, valid: parsed.valid && validVersion, publishedDirs: [] };
                 lockstepGroups.push(group);
                 activeLockstepGroups.push(group);
@@ -159,7 +189,9 @@ async function loadReleaseWorkflows(fs, rootDir) {
         return [];
     }
     const files = entries
-        .filter((e) => !e.isDirectory() && e.name.startsWith("release") && e.name.endsWith(".yml"))
+        .filter((e) => !e.isDirectory() &&
+        e.name.startsWith("release") &&
+        (e.name.endsWith(".yml") || e.name.endsWith(".yaml")))
         .map((e) => e.name)
         .sort();
     const workflows = await Promise.all(files.map(async (file) => parseReleaseWorkflow(file, await fs.readFile(path.join(dir, file)))));

package/packages/package-lint/dist/report.js

@@ -55,7 +55,13 @@ export function formatReport(result, opts) {
     const errors = result.violations.filter((v) => v.severity === "error").length;
     const warnings = result.violations.length - errors;
     if (result.summary.ok) {
-        lines.push(color.green(`✓ all ${result.summary.rules} rules passed`));
+        if (result.skipped.length > 0) {
+            const passed = result.summary.rules - result.skipped.length;
+            lines.push(color.green(`✓ ${passed} rules passed · ${result.skipped.length} skipped`));
+        }
+        else {
+            lines.push(color.green(`✓ all ${result.summary.rules} rules passed`));
+        }
     }
     else {
         const summary = `${failedRules} rules failed · ${result.violations.length} violations`;

package/packages/package-lint/dist/rules/index.js

@@ -23,6 +23,15 @@ export const rules = [
     publishedBinMustBeExecutable
 ];
 export function runRules(model, build, only) {
+    if (only && only.length > 0) {
+        const known = new Set(rules.map((rule) => rule.id));
+        const unknown = only.filter((id) => !known.has(id));
+        if (unknown.length > 0) {
+            throw new Error(`Unknown package-lint rule: ${unknown.join(", ")}. Known rules: ${rules
+                .map((rule) => rule.id)
+                .join(", ")}`);
+        }
+    }
     const selected = only && only.length > 0 ? rules.filter((r) => only.includes(r.id)) : rules;
     const violations = [];
     const skipped = [];

package/packages/package-lint/dist/rules/published-bin-must-be-executable.js

@@ -1,5 +1,36 @@
 import { isGenuinelyPublished } from "../model.js";
 const id = "published-bin-must-be-executable";
+function splitShellLine(line) {
+    return line
+        .trim()
+        .split(" ")
+        .map((part) => part.trim())
+        .filter(Boolean);
+}
+function stripQuotes(value) {
+    let out = value;
+    if (((out.startsWith('"') && out.endsWith('"')) || (out.startsWith("'") && out.endsWith("'"))) &&
+        out.length >= 2) {
+        out = out.slice(1, -1);
+    }
+    return out.replaceAll("\\", "/");
+}
+function prepackRunsSetBinExecutable(prepack) {
+    for (const line of prepack.split("\n")) {
+        const trimmed = line.trim();
+        if (trimmed.length === 0 || trimmed.startsWith("#"))
+            continue;
+        for (const part of splitShellLine(trimmed)) {
+            const token = stripQuotes(part);
+            if (token === "scripts/set-bin-executable.mjs" ||
+                token === "./scripts/set-bin-executable.mjs" ||
+                token.endsWith("/scripts/set-bin-executable.mjs")) {
+                return true;
+            }
+        }
+    }
+    return false;
+}
 /**
  * `tsc` emits bin files with mode 0644, so a package that builds its bin with
  * the compiler ships a binary that loses its executable bit and fails with
@@ -19,7 +50,7 @@ export const publishedBinMustBeExecutable = {
             const bins = Object.values(pkg.bin);
             if (bins.length === 0)
                 continue;
-            if ((pkg.scripts.prepack ?? "").includes("set-bin-executable"))
+            if (prepackRunsSetBinExecutable(pkg.scripts.prepack ?? ""))
                 continue;
             violations.push({
                 rule: id,

package/packages/package-lint/dist/source-imports.js

@@ -36,7 +36,12 @@ function extractImportsFromAst(text, fileName) {
         }
         else if (ts.isExportDeclaration(statement) && statement.moduleSpecifier) {
             if (ts.isStringLiteralLike(statement.moduleSpecifier)) {
-                out.push({ specifier: statement.moduleSpecifier.text, typeOnly: statement.isTypeOnly });
+                let typeOnly = statement.isTypeOnly;
+                if (!typeOnly && statement.exportClause && ts.isNamedExports(statement.exportClause)) {
+                    const elements = statement.exportClause.elements;
+                    typeOnly = elements.length > 0 && elements.every((e) => e.isTypeOnly);
+                }
+                out.push({ specifier: statement.moduleSpecifier.text, typeOnly });
             }
         }
         else if (ts.isImportEqualsDeclaration(statement)) {
@@ -62,13 +67,12 @@ function extractImportsFromAst(text, fileName) {
     return out;
 }
 export function extractRelevantImports(text, fileName) {
-    if (text.includes("import type") || text.includes("export type") || text.includes("{ type ")) {
-        return extractImportsFromAst(text, fileName);
-    }
-    return ts.preProcessFile(text, true, true).importedFiles.map(({ fileName: specifier }) => ({
-        specifier,
-        typeOnly: false
-    }));
+    return extractImportsFromAst(text, fileName);
+}
+function isSourceFile(name) {
+    if (name.endsWith(".d.ts") || name.endsWith(".d.mts") || name.endsWith(".d.cts"))
+        return false;
+    return (name.endsWith(".ts") || name.endsWith(".tsx") || name.endsWith(".mts") || name.endsWith(".cts"));
 }
 async function listSourceFiles(fs, dir) {
     let entries;
@@ -84,8 +88,7 @@ async function listSourceFiles(fs, dir) {
         if (entry.isDirectory()) {
             files.push(...(await listSourceFiles(fs, full)));
         }
-        else if ((entry.name.endsWith(".ts") || entry.name.endsWith(".tsx")) &&
-            !entry.name.endsWith(".d.ts")) {
+        else if (isSourceFile(entry.name)) {
             files.push(full);
         }
     }