@enrichlayer/el-linear 1.7.0 → 1.10.0

package/dist/commands/labels.js

@@ -6,7 +6,7 @@ import { createGraphQLService } from "../utils/graphql-service.js";
 import { createLinearService } from "../utils/linear-service.js";
 import { handleAsyncCommand, outputSuccess } from "../utils/output.js";
 import { getRootOpts } from "../utils/root-opts.js";
-import { validateHexColor } from "../utils/validators.js";
+import { parsePositiveInt, validateHexColor } from "../utils/validators.js";
 async function handleCreateLabel(name, options, command) {
     const rootOpts = getRootOpts(command);
     const teamId = resolveTeam(options.team);
@@ -23,45 +23,35 @@ async function handleCreateLabel(name, options, command) {
             name: options.parent,
             teamId,
         });
-        const issueLabels = parentResult.issueLabels;
-        const nodes = issueLabels?.nodes;
-        const parentLabel = nodes?.[0];
+        const parentLabel = parentResult.issueLabels.nodes[0];
         if (parentLabel) {
             input.parentId = parentLabel.id;
         }
     }
-    const result = await graphQLService.rawRequest(CREATE_LABEL_MUTATION, {
-        input,
-    });
-    const createResult = result.issueLabelCreate;
-    if (!createResult?.success) {
+    const result = await graphQLService.rawRequest(CREATE_LABEL_MUTATION, { input });
+    if (!result.issueLabelCreate.success || !result.issueLabelCreate.issueLabel) {
         throw new Error(`Failed to create label "${name}"`);
     }
-    outputSuccess(createResult.issueLabel);
+    outputSuccess(result.issueLabelCreate.issueLabel);
 }
 async function handleRetireLabel(labelId, _options, command) {
     const rootOpts = getRootOpts(command);
     const graphQLService = await createGraphQLService(rootOpts);
-    const result = await graphQLService.rawRequest(RETIRE_LABEL_MUTATION, {
-        id: labelId,
-    });
-    const retireResult = result.issueLabelRetire;
-    if (!retireResult?.success) {
+    const result = await graphQLService.rawRequest(RETIRE_LABEL_MUTATION, { id: labelId });
+    if (!result.issueLabelRetire.success || !result.issueLabelRetire.issueLabel) {
         throw new Error(`Failed to retire label "${labelId}"`);
     }
-    outputSuccess(retireResult.issueLabel);
+    outputSuccess(result.issueLabelRetire.issueLabel);
 }
 async function handleRestoreLabel(labelId, _options, command) {
     const rootOpts = getRootOpts(command);
     const graphQLService = await createGraphQLService(rootOpts);
-    const result = await graphQLService.rawRequest(RESTORE_LABEL_MUTATION, {
-        id: labelId,
-    });
-    const restoreResult = result.issueLabelRestore;
-    if (!restoreResult?.success) {
+    const result = await graphQLService.rawRequest(RESTORE_LABEL_MUTATION, { id: labelId });
+    if (!result.issueLabelRestore.success ||
+        !result.issueLabelRestore.issueLabel) {
         throw new Error(`Failed to restore label "${labelId}"`);
     }
-    outputSuccess(restoreResult.issueLabel);
+    outputSuccess(result.issueLabelRestore.issueLabel);
 }
 export function setupLabelsCommands(program) {
     const labels = program
@@ -73,22 +63,24 @@ export function setupLabelsCommands(program) {
         .command("list")
         .description("List all available labels")
         .option("--team <team>", "filter by team key, name, or ID")
+        .option("--name <substring>", "filter by case-insensitive substring on label name")
         .option("-l, --limit <number>", "limit results", "100")
         .action(handleAsyncCommand(async (options, command) => {
         const rootOpts = getRootOpts(command);
         const teamFilter = options.team ? resolveTeam(options.team) : undefined;
-        const limit = Number.parseInt(options.limit, 10);
+        const nameFilter = options.name;
+        const limit = parsePositiveInt(options.limit, "--limit");
         const ttl = resolveCacheTTL({
             configTTL: loadConfig().cacheTTLSeconds,
             noCacheFlag: rootOpts.cache === false,
         });
-        // Cache key includes the team filter so list-with-team and list-
-        // without-team don't collide. Same `limit` participates because
-        // a smaller list isn't a valid cached answer for a larger ask.
-        const cacheKey = `labels-list-team:${teamFilter ?? "_all"}-limit:${limit}`;
+        // Cache key includes the team filter and name filter so different
+        // filter combinations don't collide. Same `limit` participates
+        // because a smaller list isn't a valid cached answer for a larger ask.
+        const cacheKey = `labels-list-team:${teamFilter ?? "_all"}-name:${nameFilter ?? "_all"}-limit:${limit}`;
         const result = await cached(cacheKey, ttl, async () => {
             const service = await createLinearService(rootOpts);
-            return service.getLabels(teamFilter, limit);
+            return service.getLabels(teamFilter, limit, nameFilter);
         });
         outputSuccess({
             data: result.labels,

package/dist/commands/profile/migrate-legacy.js

@@ -38,7 +38,8 @@
 import { promises as fsp } from "node:fs";
 import path from "node:path";
 import { confirm, input, password } from "@inquirer/prompts";
-import { ACTIVE_PROFILE_FILE, CONFIG_DIR, CONFIG_PATH, PROFILES_DIR, profilePaths, TOKEN_PATH, } from "../../config/paths.js";
+import { atomicWrite } from "../../auth/oauth-fs.js";
+import { ACTIVE_PROFILE_FILE, CONFIG_DIR, CONFIG_PATH, isSafeProfileName as isSafeName, PROFILES_DIR, profilePaths, TOKEN_PATH, } from "../../config/paths.js";
 import { sanitizeForLog, validateToken } from "../init/token.js";
 const DEFAULT_PROMPTS = { input, password, confirm };
 /** Register `el-linear profile migrate-legacy` under the parent `profile` command. */
@@ -165,10 +166,7 @@ async function resolveAndValidateToken(options, prompts, validate) {
 async function copyConfigIntoProfile(destConfigPath, options, prompts) {
     const sourceContent = await fsp.readFile(CONFIG_PATH, "utf8");
     if (!(await pathExists(destConfigPath))) {
-        await fsp.writeFile(destConfigPath, sourceContent, {
-            mode: 0o644,
-            encoding: "utf8",
-        });
+        await atomicWrite(destConfigPath, sourceContent, 0o644);
         return;
     }
     const destContent = await fsp.readFile(destConfigPath, "utf8");
@@ -192,21 +190,15 @@ async function copyConfigIntoProfile(destConfigPath, options, prompts) {
             throw new Error("Aborted by user.");
         }
     }
-    await fsp.writeFile(destConfigPath, sourceContent, {
-        mode: 0o644,
-        encoding: "utf8",
-    });
+    await atomicWrite(destConfigPath, sourceContent, 0o644);
 }
 async function writeProfileToken(destTokenPath, token, options, prompts) {
     const newline = `${token}\n`;
     if (!(await pathExists(destTokenPath))) {
-        await fsp.writeFile(destTokenPath, newline, {
-            mode: 0o600,
-            encoding: "utf8",
-        });
-        // fs.writeFile mode is only honored on file creation; chmod defensively
-        // in case a parent process pre-created the file with permissive perms.
-        await fsp.chmod(destTokenPath, 0o600);
+        // atomicWrite creates the tmp file with the requested mode and
+        // renames into place, so there is no window where the token is
+        // readable at a looser mode than 0o600.
+        await atomicWrite(destTokenPath, newline, 0o600);
         return;
     }
     const existing = (await fsp.readFile(destTokenPath, "utf8")).trim();
@@ -230,11 +222,7 @@ async function writeProfileToken(destTokenPath, token, options, prompts) {
             throw new Error("Aborted by user.");
         }
     }
-    await fsp.writeFile(destTokenPath, newline, {
-        mode: 0o600,
-        encoding: "utf8",
-    });
-    await fsp.chmod(destTokenPath, 0o600);
+    await atomicWrite(destTokenPath, newline, 0o600);
 }
 async function ensureActiveProfile(name) {
     await fsp.mkdir(CONFIG_DIR, { recursive: true, mode: 0o700 });
@@ -244,10 +232,7 @@ async function ensureActiveProfile(name) {
     }
     if (current === name)
         return;
-    await fsp.writeFile(ACTIVE_PROFILE_FILE, `${name}\n`, {
-        mode: 0o644,
-        encoding: "utf8",
-    });
+    await atomicWrite(ACTIVE_PROFILE_FILE, `${name}\n`, 0o644);
 }
 async function pathExists(p) {
     try {
@@ -258,14 +243,6 @@ async function pathExists(p) {
         return false;
     }
 }
-/**
- * Profile names land in filesystem paths and the active-profile marker.
- * Same conservative charset as `profile add` to avoid `..` traversal,
- * shell metacharacters, and Unicode lookalikes.
- */
-function isSafeName(name) {
-    return /^[A-Za-z0-9][A-Za-z0-9_.-]{0,63}$/.test(name);
-}
 // Re-export internals so the integration test (and follow-up commands)
 // can compose them without re-implementing the idempotency rules.
 export { CONFIG_PATH, PROFILES_DIR, TOKEN_PATH };

package/dist/commands/profile.js

@@ -23,7 +23,7 @@
 import { promises as fsp } from "node:fs";
 import path from "node:path";
 import { confirm } from "@inquirer/prompts";
-import { ACTIVE_PROFILE_FILE, CONFIG_DIR, CONFIG_PATH, PROFILES_DIR, profilePaths, resolveActiveProfile, setActiveProfileForSession, TOKEN_PATH, } from "../config/paths.js";
+import { ACTIVE_PROFILE_FILE, CONFIG_DIR, CONFIG_PATH, isSafeProfileName as isSafeName, PROFILES_DIR, profilePaths, resolveActiveProfile, setActiveProfileForSession, TOKEN_PATH, } from "../config/paths.js";
 import { outputSuccess, outputWarning } from "../utils/output.js";
 import { runFullWizard } from "./init/index.js";
 import { registerMigrateLegacy } from "./profile/migrate-legacy.js";
@@ -180,12 +180,3 @@ async function pathExists(p) {
         return false;
     }
 }
-/**
- * Profile names land in filesystem paths AND get written to a
- * single-line marker. Restrict to a conservative charset so we can't
- * accidentally pick up `..` traversal, command-line metacharacters, or
- * Unicode lookalikes that would confuse `profile use`.
- */
-function isSafeName(name) {
-    return /^[A-Za-z0-9][A-Za-z0-9_.-]{0,63}$/.test(name);
-}

package/dist/commands/project-milestones.js

@@ -5,7 +5,7 @@ import { createLinearService, } from "../utils/linear-service.js";
 import { handleAsyncCommand, outputSuccess } from "../utils/output.js";
 import { getRootOpts } from "../utils/root-opts.js";
 import { isUuid } from "../utils/uuid.js";
-import { validateIsoDate } from "../utils/validators.js";
+import { parsePositiveInt, validateIsoDate } from "../utils/validators.js";
 async function resolveMilestoneId(milestoneNameOrId, graphQLService, linearService, projectNameOrId) {
     if (isUuid(milestoneNameOrId)) {
         return milestoneNameOrId;
@@ -17,22 +17,17 @@ async function resolveMilestoneId(milestoneNameOrId, graphQLService, linearServi
             name: milestoneNameOrId,
             projectId,
         });
-        const projectData = findRes.project;
-        const milestonesData = projectData?.projectMilestones;
-        nodes = milestonesData?.nodes || [];
+        nodes = findRes.project?.projectMilestones.nodes ?? [];
     }
     if (nodes.length === 0) {
-        const globalRes = await graphQLService.rawRequest(FIND_PROJECT_MILESTONE_BY_NAME_GLOBAL, {
-            name: milestoneNameOrId,
-        });
-        const globalMilestones = globalRes.projectMilestones;
-        nodes = globalMilestones?.nodes || [];
+        const globalRes = await graphQLService.rawRequest(FIND_PROJECT_MILESTONE_BY_NAME_GLOBAL, { name: milestoneNameOrId });
+        nodes = globalRes.projectMilestones.nodes;
     }
     if (nodes.length === 0) {
         throw notFoundError("Milestone", milestoneNameOrId);
     }
     if (nodes.length > 1) {
-        const matches = nodes.map((m) => `"${m.name}" in project "${m.project?.name}"`);
+        const matches = nodes.map((m) => `"${m.name}" in project "${m.project.name}"`);
         throw multipleMatchesError("milestone", milestoneNameOrId, matches, "specify --project or use the milestone ID");
     }
     return nodes[0].id;
@@ -44,11 +39,9 @@ async function handleListMilestones(options, command) {
     const projectId = await linearService.resolveProjectId(options.project);
     const result = await graphQLService.rawRequest(LIST_PROJECT_MILESTONES_QUERY, {
         projectId,
-        first: Number.parseInt(options.limit || "50", 10),
+        first: parsePositiveInt(options.limit || "50", "--limit"),
     });
-    const projectData = result.project;
-    const milestonesData = projectData?.projectMilestones;
-    const nodes = milestonesData?.nodes || [];
+    const nodes = result.project?.projectMilestones.nodes ?? [];
     outputSuccess({ data: nodes, meta: { count: nodes.length } });
 }
 async function handleReadMilestone(milestoneIdOrName, options, command) {
@@ -76,11 +69,11 @@ async function handleCreateMilestone(name, options, command) {
         description: options.description,
         targetDate: options.targetDate,
     });
-    const createData = result.projectMilestoneCreate;
-    if (!createData?.success) {
+    if (!result.projectMilestoneCreate.success ||
+        !result.projectMilestoneCreate.projectMilestone) {
         throw new Error("Failed to create project milestone");
     }
-    outputSuccess(createData.projectMilestone);
+    outputSuccess(result.projectMilestoneCreate.projectMilestone);
 }
 async function handleUpdateMilestone(milestoneIdOrName, options, command) {
     const rootOpts = getRootOpts(command);
@@ -102,11 +95,11 @@ async function handleUpdateMilestone(milestoneIdOrName, options, command) {
         updateVars.sortOrder = Number.parseFloat(options.sortOrder);
     }
     const result = await graphQLService.rawRequest(UPDATE_PROJECT_MILESTONE_MUTATION, updateVars);
-    const updateData = result.projectMilestoneUpdate;
-    if (!updateData?.success) {
+    if (!result.projectMilestoneUpdate.success ||
+        !result.projectMilestoneUpdate.projectMilestone) {
         throw new Error("Failed to update project milestone");
     }
-    outputSuccess(updateData.projectMilestone);
+    outputSuccess(result.projectMilestoneUpdate.projectMilestone);
 }
 export function setupProjectMilestonesCommands(program) {
     const projectMilestones = program

package/dist/commands/projects.d.ts

@@ -1,2 +1,6 @@
-import type { Command } from "commander";
+import type { Command, OptionValues } from "commander";
+export declare function resolveProjectStateFilter(options: OptionValues): {
+    states?: string[];
+    excludeStates?: string[];
+};
 export declare function setupProjectsCommands(program: Command): void;