semantic-release-linear-app 0.2.0 → 0.3.0-next.2

package/dist/lib/success.d.ts

@@ -2,14 +2,9 @@ import { SuccessContext } from "semantic-release";
 import { PluginConfig, LinearContext } from "../types";
 interface ExtendedContext extends SuccessContext {
     linear?: LinearContext;
-    branch: {
-        name: string;
-        [key: string]: unknown;
-    };
 }
 /**
  * Update Linear issues after a successful release
- * Only uses branch name for issue detection - single source of truth
  */
 export declare function success(pluginConfig: PluginConfig, context: ExtendedContext): Promise<void>;
 export {};

package/dist/lib/success.js

@@ -1,61 +1,94 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.success = success;
+const execa_1 = require("execa");
 const linear_client_1 = require("./linear-client");
 const parse_issues_1 = require("./parse-issues");
+/**
+ * Find source branches that contain the given commits
+ */
+async function findSourceBranches(commits, logger) {
+    const branches = new Set();
+    if (commits.length === 0)
+        return branches;
+    try {
+        // Get all branches that contain these commits
+        const { stdout } = await (0, execa_1.execa)("git", [
+            "branch",
+            "-r",
+            "--contains",
+            commits[0].hash,
+            "--merged",
+        ]);
+        // Parse remote branch names
+        const branchLines = stdout.split("\n").map((b) => b.trim());
+        for (const line of branchLines) {
+            // Extract branch name from origin/feature/ENG-123-description
+            const match = line.match(/origin\/(.+)/);
+            if (match &&
+                !["main", "master", "develop", "stable", "HEAD"].includes(match[1])) {
+                branches.add(match[1]);
+                logger.log(`Found source branch: ${match[1]}`);
+            }
+        }
+    }
+    catch (error) {
+        logger.debug(`Git branch lookup failed: ${error.message}`);
+    }
+    return branches;
+}
 /**
  * Update Linear issues after a successful release
- * Only uses branch name for issue detection - single source of truth
  */
 async function success(pluginConfig, context) {
-    const { logger, nextRelease, linear, branch } = context;
+    const { logger, nextRelease, linear, commits } = context;
     if (!linear) {
         logger.log("Linear context not found, skipping issue updates");
         return;
     }
-    if (!branch.name) {
-        logger.log("No branch name available, skipping Linear updates");
+    if (!commits || commits.length === 0) {
+        logger.log("No commits found in release, skipping Linear updates");
         return;
     }
-    const { removeOldLabels = true, addComment = false, dryRun = false, skipBranches = ["main", "master", "develop", "staging", "production"], requireIssueInBranch = true, } = pluginConfig;
-    // Check if this branch should be processed
-    if (requireIssueInBranch && !(0, parse_issues_1.shouldProcessBranch)(branch.name, skipBranches)) {
-        logger.log(`Branch "${branch.name}" doesn't contain Linear issues or is in skip list, skipping updates`);
+    const { removeOldLabels = true, addComment = false, dryRun = false, } = pluginConfig;
+    // Find all branches that contributed to this release
+    const sourceBranches = await findSourceBranches(commits, logger);
+    if (sourceBranches.size === 0) {
+        logger.log("No source branches found, skipping Linear updates");
         return;
     }
-    const client = new linear_client_1.LinearClient(linear.apiKey);
-    const version = nextRelease.version;
-    const channel = nextRelease.channel || "latest";
-    // Format the label based on configuration
-    const labelName = `${linear.labelPrefix}${version}`;
-    const labelColor = getLabelColor(nextRelease.type);
-    logger.log(`Updating Linear issues for release ${version} (${channel}) from branch "${branch.name}"`);
-    // Extract issue IDs from branch name ONLY
-    const issueIds = (0, parse_issues_1.parseIssuesFromBranch)(branch.name, linear.teamKeys);
-    if (issueIds.length === 0) {
-        logger.log(`No Linear issues found in branch name "${branch.name}"`);
-        if (requireIssueInBranch) {
-            logger.warn("⚠️  Consider using branch names like: feature/ENG-123-description");
-        }
+    // Extract Linear issue IDs from all found branches
+    const issueIds = new Set();
+    for (const branchName of Array.from(sourceBranches)) {
+        const branchIssues = (0, parse_issues_1.parseIssuesFromBranch)(branchName, linear.teamKeys);
+        branchIssues.forEach((id) => issueIds.add(id));
+    }
+    if (issueIds.size === 0) {
+        logger.log(`No Linear issues found in branches: ${Array.from(sourceBranches).join(", ")}`);
         return;
     }
-    logger.log(`Found ${issueIds.length} Linear issue(s) in branch "${branch.name}": ${issueIds.join(", ")}`);
+    logger.log(`Found ${issueIds.size} Linear issue(s): ${Array.from(issueIds).join(", ")} ` +
+        `from ${sourceBranches.size} branch(es)`);
     if (dryRun) {
-        logger.log("[Dry run] Would update issues:", issueIds);
-        logger.log(`[Dry run] Would apply label: ${labelName}`);
-        logger.log(`[Dry run] Branch: ${branch.name}`);
+        logger.log("[Dry run] Would update issues:", Array.from(issueIds));
+        logger.log(`[Dry run] Would apply label: ${linear.labelPrefix}${nextRelease.version}`);
         return;
     }
+    // Initialize Linear client and prepare label
+    const client = new linear_client_1.LinearClient(linear.apiKey);
+    const version = nextRelease.version;
+    const channel = nextRelease.channel || "latest";
+    const labelName = `${linear.labelPrefix}${version}`;
+    const labelColor = getLabelColor(nextRelease.type);
     // Ensure the version label exists
     const label = await client.ensureLabel(labelName, labelColor);
     logger.log(`✓ Ensured label exists: ${labelName}`);
-    // Update each issue
-    const results = await Promise.allSettled(issueIds.map(async (issueId) => {
+    // Update each Linear issue
+    const results = await Promise.allSettled(Array.from(issueIds).map(async (issueId) => {
         try {
-            // Get the issue first
             const issue = await client.getIssue(issueId);
             if (!issue) {
-                logger.warn(`Issue ${issueId} not found in Linear, skipping`);
+                logger.warn(`Issue ${issueId} not found in Linear`);
                 return { issueId, status: "not_found" };
             }
             // Remove old version labels if configured
@@ -66,7 +99,9 @@ async function success(pluginConfig, context) {
             await client.addLabelToIssue(issue.id, label.id);
             // Add comment if configured
             if (addComment) {
-                const comment = formatComment(version, channel, nextRelease, branch.name);
+                const emoji = channel === "latest" ? "🚀" : "🔬";
+                const channelText = channel === "latest" ? "" : ` (${channel} channel)`;
+                const comment = `${emoji} Released in version ${version}${channelText}`;
                 await client.addComment(issue.id, comment);
             }
             logger.log(`✓ Updated issue ${issueId}`);
@@ -79,10 +114,9 @@ async function success(pluginConfig, context) {
         }
     }));
     // Log summary
-    const updated = results.filter((r) => r.status === "fulfilled" && r.value.status === "updated").length;
-    const failed = results.filter((r) => r.status === "rejected" ||
-        (r.status === "fulfilled" && r.value.status === "failed")).length;
-    const notFound = results.filter((r) => r.status === "fulfilled" && r.value.status === "not_found").length;
+    const updated = results.filter((r) => r.status === "fulfilled" && r.value?.status === "updated").length;
+    const failed = results.filter((r) => r.status === "rejected" || r.value?.status === "failed").length;
+    const notFound = results.filter((r) => r.status === "fulfilled" && r.value?.status === "not_found").length;
     logger.log(`Linear update complete: ${updated} updated, ${failed} failed, ${notFound} not found`);
 }
 /**
@@ -100,17 +134,3 @@ function getLabelColor(releaseType) {
     };
     return colors[releaseType] || "#4752C4"; // Default blue
 }
-/**
- * Format comment for Linear issue
- */
-function formatComment(version, channel, release, branchName) {
-    const emoji = channel === "latest" ? "🚀" : "🔬";
-    const channelText = channel === "latest" ? "stable" : channel;
-    let comment = `${emoji} **Released in \`v${version}\`** (${channelText})\n\n`;
-    comment += `📌 Released from branch: \`${branchName}\`\n\n`;
-    const githubRepo = process.env.GITHUB_REPOSITORY;
-    if (release.gitTag && githubRepo) {
-        comment += `[View release →](https://github.com/${githubRepo}/releases/tag/${release.gitTag})`;
-    }
-    return comment;
-}

package/dist/lib/verify.js

@@ -18,8 +18,12 @@ async function verifyConditions(pluginConfig, context) {
         throw new error_1.default("No Linear API key found", "ENOLINEARTOKEN", "Please provide a Linear API key via plugin config or LINEAR_API_KEY environment variable.");
     }
     // Validate team keys format if provided
-    if (teamKeys.length > 0 && !teamKeys.every((key) => /^[A-Z]+$/.test(key))) {
-        throw new error_1.default("Invalid team key format", "EINVALIDTEAMKEY", `Team keys must be uppercase letters only. Got: ${teamKeys.join(", ")}`);
+    const teamKeyPattern = /^[A-Z]+$/;
+    const branchPattern = /^[A-Za-z0-9._-]+\/[A-Za-z0-9][A-Za-z0-9._-]*$/;
+    const invalidTeamKeys = teamKeys.filter((key) => !teamKeyPattern.test(key) && !branchPattern.test(key));
+    if (invalidTeamKeys.length > 0) {
+        throw new error_1.default("Invalid team key format", "EINVALIDTEAMKEY", "Team keys must be uppercase letters (e.g. SD) or branch names (e.g. caio/tk-519-title). " +
+            `Invalid: ${invalidTeamKeys.join(", ")}`);
     }
     // Test API connection
     const client = new linear_client_1.LinearClient(linearApiKey);

package/dist/lib/verify.test.js

@@ -11,6 +11,11 @@ jest.mock("@semantic-release/error", () => {
     }
     return { __esModule: true, default: SemanticReleaseError };
 });
+jest.mock("./linear-client", () => ({
+    LinearClient: jest.fn().mockImplementation(() => ({
+        testConnection: jest.fn().mockResolvedValue({}),
+    })),
+}));
 jest.mock("node-fetch", () => ({ __esModule: true, default: jest.fn() }));
 const verify_1 = require("./verify");
 describe("verify", () => {
@@ -27,4 +32,11 @@ describe("verify", () => {
         // Validation happens BEFORE the API call, so it fails fast
         await expect((0, verify_1.verifyConditions)({ apiKey: "test", teamKeys: ["eng-123"] }, mockContext)).rejects.toThrow("Invalid team key format");
     });
+    test("accepts valid branch like team key", async () => {
+        // Accepts branch patterns without hitting API
+        await expect((0, verify_1.verifyConditions)({
+            apiKey: "test",
+            teamKeys: ["caio/tk-519-title"],
+        }, mockContext)).resolves.toBeUndefined();
+    });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "semantic-release-linear-app",
-    "version": "0.2.0",
+    "version": "0.3.0-next.2",
     "description": "Semantic-release plugin to update Linear issues with version labels",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",
@@ -46,6 +46,7 @@
     },
     "dependencies": {
         "@semantic-release/error": "^4.0.0",
+        "execa": "^9.6.0",
         "node-fetch": "^3.3.2"
     },
     "devDependencies": {

package/src/lib/success.ts

@@ -1,38 +1,70 @@
 import { SuccessContext } from "semantic-release";
+import { execa } from "execa";
 import { LinearClient } from "./linear-client";
-import { parseIssuesFromBranch, shouldProcessBranch } from "./parse-issues";
-import {
-  PluginConfig,
-  LinearContext,
-  IssueUpdateResult,
-  ReleaseType,
-} from "../types";
+import { parseIssuesFromBranch } from "./parse-issues";
+import { PluginConfig, LinearContext, ReleaseType } from "../types";
 
 interface ExtendedContext extends SuccessContext {
   linear?: LinearContext;
-  branch: {
-    name: string;
-    [key: string]: unknown;
-  };
+}
+
+/**
+ * Find source branches that contain the given commits
+ */
+async function findSourceBranches(
+  commits: readonly { hash: string }[],
+  logger: SuccessContext["logger"],
+): Promise<Set<string>> {
+  const branches = new Set<string>();
+
+  if (commits.length === 0) return branches;
+
+  try {
+    // Get all branches that contain these commits
+    const { stdout } = await execa("git", [
+      "branch",
+      "-r",
+      "--contains",
+      commits[0].hash,
+      "--merged",
+    ]);
+
+    // Parse remote branch names
+    const branchLines = stdout.split("\n").map((b: string) => b.trim());
+    for (const line of branchLines) {
+      // Extract branch name from origin/feature/ENG-123-description
+      const match = line.match(/origin\/(.+)/);
+      if (
+        match &&
+        !["main", "master", "develop", "stable", "HEAD"].includes(match[1])
+      ) {
+        branches.add(match[1]);
+        logger.log(`Found source branch: ${match[1]}`);
+      }
+    }
+  } catch (error) {
+    logger.debug(`Git branch lookup failed: ${(error as Error).message}`);
+  }
+
+  return branches;
 }
 
 /**
  * Update Linear issues after a successful release
- * Only uses branch name for issue detection - single source of truth
  */
 export async function success(
   pluginConfig: PluginConfig,
   context: ExtendedContext,
 ): Promise<void> {
-  const { logger, nextRelease, linear, branch } = context;
+  const { logger, nextRelease, linear, commits } = context;
 
   if (!linear) {
     logger.log("Linear context not found, skipping issue updates");
     return;
   }
 
-  if (!branch.name) {
-    logger.log("No branch name available, skipping Linear updates");
+  if (!commits || commits.length === 0) {
+    logger.log("No commits found in release, skipping Linear updates");
     return;
   }
 
@@ -40,67 +72,61 @@ export async function success(
     removeOldLabels = true,
     addComment = false,
     dryRun = false,
-    skipBranches = ["main", "master", "develop", "staging", "production"],
-    requireIssueInBranch = true,
   } = pluginConfig;
 
-  // Check if this branch should be processed
-  if (requireIssueInBranch && !shouldProcessBranch(branch.name, skipBranches)) {
-    logger.log(
-      `Branch "${branch.name}" doesn't contain Linear issues or is in skip list, skipping updates`,
-    );
+  // Find all branches that contributed to this release
+  const sourceBranches = await findSourceBranches(commits, logger);
+
+  if (sourceBranches.size === 0) {
+    logger.log("No source branches found, skipping Linear updates");
     return;
   }
 
-  const client = new LinearClient(linear.apiKey);
-  const version = nextRelease.version;
-  const channel = nextRelease.channel || "latest";
-
-  // Format the label based on configuration
-  const labelName = `${linear.labelPrefix}${version}`;
-  const labelColor = getLabelColor(nextRelease.type as ReleaseType);
-
-  logger.log(
-    `Updating Linear issues for release ${version} (${channel}) from branch "${branch.name}"`,
-  );
-
-  // Extract issue IDs from branch name ONLY
-  const issueIds = parseIssuesFromBranch(branch.name, linear.teamKeys);
+  // Extract Linear issue IDs from all found branches
+  const issueIds = new Set<string>();
+  for (const branchName of Array.from(sourceBranches)) {
+    const branchIssues = parseIssuesFromBranch(branchName, linear.teamKeys);
+    branchIssues.forEach((id) => issueIds.add(id));
+  }
 
-  if (issueIds.length === 0) {
-    logger.log(`No Linear issues found in branch name "${branch.name}"`);
-    if (requireIssueInBranch) {
-      logger.warn(
-        "⚠️  Consider using branch names like: feature/ENG-123-description",
-      );
-    }
+  if (issueIds.size === 0) {
+    logger.log(
+      `No Linear issues found in branches: ${Array.from(sourceBranches).join(", ")}`,
+    );
     return;
   }
 
   logger.log(
-    `Found ${issueIds.length} Linear issue(s) in branch "${branch.name}": ${issueIds.join(", ")}`,
+    `Found ${issueIds.size} Linear issue(s): ${Array.from(issueIds).join(", ")} ` +
+      `from ${sourceBranches.size} branch(es)`,
   );
 
   if (dryRun) {
-    logger.log("[Dry run] Would update issues:", issueIds);
-    logger.log(`[Dry run] Would apply label: ${labelName}`);
-    logger.log(`[Dry run] Branch: ${branch.name}`);
+    logger.log("[Dry run] Would update issues:", Array.from(issueIds));
+    logger.log(
+      `[Dry run] Would apply label: ${linear.labelPrefix}${nextRelease.version}`,
+    );
     return;
   }
 
+  // Initialize Linear client and prepare label
+  const client = new LinearClient(linear.apiKey);
+  const version = nextRelease.version;
+  const channel = nextRelease.channel || "latest";
+  const labelName = `${linear.labelPrefix}${version}`;
+  const labelColor = getLabelColor(nextRelease.type as ReleaseType);
+
   // Ensure the version label exists
   const label = await client.ensureLabel(labelName, labelColor);
   logger.log(`✓ Ensured label exists: ${labelName}`);
 
-  // Update each issue
+  // Update each Linear issue
   const results = await Promise.allSettled(
-    issueIds.map(async (issueId): Promise<IssueUpdateResult> => {
+    Array.from(issueIds).map(async (issueId) => {
       try {
-        // Get the issue first
         const issue = await client.getIssue(issueId);
-
         if (!issue) {
-          logger.warn(`Issue ${issueId} not found in Linear, skipping`);
+          logger.warn(`Issue ${issueId} not found in Linear`);
           return { issueId, status: "not_found" };
         }
 
@@ -114,12 +140,10 @@ export async function success(
 
         // Add comment if configured
         if (addComment) {
-          const comment = formatComment(
-            version,
-            channel,
-            nextRelease,
-            branch.name,
-          );
+          const emoji = channel === "latest" ? "🚀" : "🔬";
+          const channelText =
+            channel === "latest" ? "" : ` (${channel} channel)`;
+          const comment = `${emoji} Released in version ${version}${channelText}`;
           await client.addComment(issue.id, comment);
         }
 
@@ -135,17 +159,13 @@ export async function success(
 
   // Log summary
   const updated = results.filter(
-    (r) => r.status === "fulfilled" && r.value.status === "updated",
+    (r) => r.status === "fulfilled" && r.value?.status === "updated",
   ).length;
-
   const failed = results.filter(
-    (r) =>
-      r.status === "rejected" ||
-      (r.status === "fulfilled" && r.value.status === "failed"),
+    (r) => r.status === "rejected" || r.value?.status === "failed",
   ).length;
-
   const notFound = results.filter(
-    (r) => r.status === "fulfilled" && r.value.status === "not_found",
+    (r) => r.status === "fulfilled" && r.value?.status === "not_found",
   ).length;
 
   logger.log(
@@ -169,26 +189,3 @@ function getLabelColor(releaseType: ReleaseType): string {
 
   return colors[releaseType] || "#4752C4"; // Default blue
 }
-
-/**
- * Format comment for Linear issue
- */
-function formatComment(
-  version: string,
-  channel: string,
-  release: SuccessContext["nextRelease"],
-  branchName: string,
-): string {
-  const emoji = channel === "latest" ? "🚀" : "🔬";
-  const channelText = channel === "latest" ? "stable" : channel;
-
-  let comment = `${emoji} **Released in \`v${version}\`** (${channelText})\n\n`;
-  comment += `📌 Released from branch: \`${branchName}\`\n\n`;
-
-  const githubRepo = process.env.GITHUB_REPOSITORY;
-  if (release.gitTag && githubRepo) {
-    comment += `[View release →](https://github.com/${githubRepo}/releases/tag/${release.gitTag})`;
-  }
-
-  return comment;
-}

package/src/lib/verify.test.ts

@@ -10,6 +10,12 @@ jest.mock("@semantic-release/error", () => {
   return { __esModule: true, default: SemanticReleaseError };
 });
 
+jest.mock("./linear-client", () => ({
+  LinearClient: jest.fn().mockImplementation(() => ({
+    testConnection: jest.fn().mockResolvedValue({}),
+  })),
+}));
+
 jest.mock("node-fetch", () => ({ __esModule: true, default: jest.fn() }));
 
 import { verifyConditions } from "./verify";
@@ -36,4 +42,17 @@ describe("verify", () => {
       verifyConditions({ apiKey: "test", teamKeys: ["eng-123"] }, mockContext),
     ).rejects.toThrow("Invalid team key format");
   });
+
+  test("accepts valid branch like team key", async () => {
+    // Accepts branch patterns without hitting API
+    await expect(
+      verifyConditions(
+        {
+          apiKey: "test",
+          teamKeys: ["caio/tk-519-title"],
+        },
+        mockContext,
+      ),
+    ).resolves.toBeUndefined();
+  });
 });

package/src/lib/verify.ts

@@ -25,11 +25,18 @@ export async function verifyConditions(
   }
 
   // Validate team keys format if provided
-  if (teamKeys.length > 0 && !teamKeys.every((key) => /^[A-Z]+$/.test(key))) {
+  const teamKeyPattern = /^[A-Z]+$/;
+  const branchPattern = /^[A-Za-z0-9._-]+\/[A-Za-z0-9][A-Za-z0-9._-]*$/;
+  const invalidTeamKeys = teamKeys.filter(
+    (key) => !teamKeyPattern.test(key) && !branchPattern.test(key),
+  );
+
+  if (invalidTeamKeys.length > 0) {
     throw new SemanticReleaseError(
       "Invalid team key format",
       "EINVALIDTEAMKEY",
-      `Team keys must be uppercase letters only. Got: ${teamKeys.join(", ")}`,
+      "Team keys must be uppercase letters (e.g. SD) or branch names (e.g. caio/tk-519-title). " +
+        `Invalid: ${invalidTeamKeys.join(", ")}`,
     );
   }