@enrichlayer/el-linear 1.7.0 → 1.10.0

package/dist/utils/table-formatter.js

@@ -129,39 +129,54 @@ function pad(str, width) {
     const truncated = truncate(str, width);
     return truncated + " ".repeat(Math.max(0, width - truncated.length));
 }
-export function formatTable(issues, fieldNames) {
-    const columnKeys = fieldNames ?? DEFAULT_COLUMNS;
-    const columns = columnKeys
-        .map((key) => ALL_COLUMNS[key])
-        .filter((col) => col !== undefined);
+/**
+ * Render `rows` as a fixed-width text table using `columns`. Generic
+ * over the row type `T` so issues, projects, and other resources
+ * share the same width-padding + colorize machinery.
+ */
+export function renderFixedWidthTable(rows, columns) {
     if (columns.length === 0) {
         return "No valid columns specified.";
     }
     const header = columns.map((col) => pad(col.header, col.width)).join("  ");
     const separator = columns.map((col) => "─".repeat(col.width)).join("──");
-    const rows = issues.map((issue) => columns
+    const body = rows.map((row) => columns
         .map((col) => {
-        const padded = pad(col.extract(issue), col.width);
-        return col.colorize ? col.colorize(padded, issue) : padded;
+        const padded = pad(col.extract(row), col.width);
+        return col.colorize ? col.colorize(padded, row) : padded;
     })
         .join("  "));
-    return [header, separator, ...rows].join("\n");
+    return [header, separator, ...body].join("\n");
 }
-export function formatCsv(issues, fieldNames) {
-    const columnKeys = fieldNames ?? DEFAULT_COLUMNS;
-    const columns = columnKeys
-        .map((key) => ALL_COLUMNS[key])
-        .filter((col) => col !== undefined);
+/**
+ * Render `rows` as CSV using `columns`. Quotes any value containing
+ * `,` or `"` (RFC 4180-style escaping). Generic over the row type.
+ */
+export function renderCsv(rows, columns) {
     const header = columns.map((col) => col.header).join(",");
-    const rows = issues.map((issue) => columns
+    const body = rows.map((row) => columns
         .map((col) => {
-        const value = col.extract(issue);
+        const value = col.extract(row);
         return value.includes(",") || value.includes('"')
             ? `"${value.replace(/"/g, '""')}"`
             : value;
     })
         .join(","));
-    return [header, ...rows].join("\n");
+    return [header, ...body].join("\n");
+}
+export function formatTable(issues, fieldNames) {
+    const columnKeys = fieldNames ?? DEFAULT_COLUMNS;
+    const columns = columnKeys
+        .map((key) => ALL_COLUMNS[key])
+        .filter((col) => col !== undefined);
+    return renderFixedWidthTable(issues, columns);
+}
+export function formatCsv(issues, fieldNames) {
+    const columnKeys = fieldNames ?? DEFAULT_COLUMNS;
+    const columns = columnKeys
+        .map((key) => ALL_COLUMNS[key])
+        .filter((col) => col !== undefined);
+    return renderCsv(issues, columns);
 }
 // ── Markdown formatting ────────────────────────────────────────
 const STATUS_MD = {
@@ -234,16 +249,23 @@ const MD_COLUMNS = {
 function escapeMarkdownCell(str) {
     return str.replace(/\|/g, "\\|").replace(/\n/g, " ");
 }
-export function formatMarkdown(issues, fieldNames) {
-    const columnKeys = fieldNames ?? DEFAULT_COLUMNS;
-    const columns = columnKeys
-        .map((key) => MD_COLUMNS[key])
-        .filter((col) => col !== undefined);
+/**
+ * Render `rows` as a markdown table using `columns`. Generic over
+ * the row type. Escapes `|` and newlines in cell values.
+ */
+export function renderMarkdownTable(rows, columns) {
     if (columns.length === 0) {
         return "No valid columns specified.";
     }
     const header = `| ${columns.map((col) => col.header).join(" | ")} |`;
     const divider = `| ${columns.map((col) => (col.align === "right" ? "---:" : "---")).join(" | ")} |`;
-    const rows = issues.map((issue) => `| ${columns.map((col) => escapeMarkdownCell(col.extract(issue))).join(" | ")} |`);
-    return [header, divider, ...rows].join("\n");
+    const body = rows.map((row) => `| ${columns.map((col) => escapeMarkdownCell(col.extract(row))).join(" | ")} |`);
+    return [header, divider, ...body].join("\n");
+}
+export function formatMarkdown(issues, fieldNames) {
+    const columnKeys = fieldNames ?? DEFAULT_COLUMNS;
+    const columns = columnKeys
+        .map((key) => MD_COLUMNS[key])
+        .filter((col) => col !== undefined);
+    return renderMarkdownTable(issues, columns);
 }

package/dist/utils/validators.d.ts

@@ -9,6 +9,14 @@ export declare function parsePositiveInt(value: string, flagName: string): numbe
 export declare function validatePriority(value: string): number;
 export declare function validateHexColor(value: string): string;
 export declare function validateIsoDate(value: string): string;
-export declare function splitList(value: string): string[];
+/**
+ * Split a comma-separated list into trimmed, non-empty items.
+ *
+ * Accepts `undefined` so call sites can pipe through commander's
+ * optional flag values without an explicit truthy guard. `false`
+ * (commander's representation of `--no-foo`) and empty string both
+ * resolve to `[]`.
+ */
+export declare function splitList(value: string | undefined | null | false): string[];
 export declare function parsePriorityFilter(value: string): number[];
 export declare const PRIORITY_LABELS: Record<number, string>;

package/dist/utils/validators.js

@@ -42,7 +42,17 @@ export function validateIsoDate(value) {
     }
     return value;
 }
+/**
+ * Split a comma-separated list into trimmed, non-empty items.
+ *
+ * Accepts `undefined` so call sites can pipe through commander's
+ * optional flag values without an explicit truthy guard. `false`
+ * (commander's representation of `--no-foo`) and empty string both
+ * resolve to `[]`.
+ */
 export function splitList(value) {
+    if (!value)
+        return [];
     return value
         .split(",")
         .map((s) => s.trim())

package/dist/utils/workspace-url.d.ts

@@ -1,4 +1,8 @@
 import type { GraphQLService } from "./graphql-service.js";
-export declare function getWorkspaceUrlKey(graphQLService: GraphQLService): Promise<string>;
+export interface GetWorkspaceUrlKeyOptions {
+    /** Highest-priority source. Typically the `--workspace-url-key` flag. */
+    override?: string;
+}
+export declare function getWorkspaceUrlKey(graphQLService?: GraphQLService, options?: GetWorkspaceUrlKeyOptions): Promise<string>;
 /** Test helper: clear the cached URL key. Not called from production code. */
 export declare function _resetWorkspaceUrlKeyCache(): void;

package/dist/utils/workspace-url.js

@@ -4,11 +4,20 @@ import { loadConfig } from "../config/config.js";
  * issue URLs). Used to build canonical markdown link URLs like
  * `https://linear.app/<urlKey>/issue/<identifier>/`.
  *
- * Resolution order:
- *   1. `config.workspaceUrlKey` if explicitly set in el-linear config
- *   2. `viewer.organization.urlKey` from the Linear API (fetched once, cached)
+ * Resolution order (highest priority first):
  *
- * Cached in-process for the lifetime of the CLI invocation.
+ *   1. `options.override` — typically wired from a `--workspace-url-key` CLI flag
+ *      (per-invocation, wins over everything).
+ *   2. `EL_LINEAR_WORKSPACE_URL_KEY` env var.
+ *   3. `config.workspaceUrlKey` from `~/.config/el-linear/config.json`.
+ *   4. Live `viewer.organization.urlKey` GraphQL query (cached in-process).
+ *
+ * Layers 1–3 short-circuit without touching the network — this makes
+ * `el-linear refs wrap --no-validate` work offline once any one of them
+ * is set. Layer 4 is the original behavior preserved as a fallback.
+ *
+ * The graphQLService is only needed for layer 4 — pass `undefined` (or omit it)
+ * when the caller knows the network will not be used.
  */
 const VIEWER_ORG_URL_KEY_QUERY = /* GraphQL */ `
   query ViewerOrgUrlKey {
@@ -19,16 +28,37 @@ const VIEWER_ORG_URL_KEY_QUERY = /* GraphQL */ `
     }
   }
 `;
+const WORKSPACE_URL_KEY_ENV = "EL_LINEAR_WORKSPACE_URL_KEY";
 let cachedUrlKey;
-export async function getWorkspaceUrlKey(graphQLService) {
+export async function getWorkspaceUrlKey(graphQLService, options = {}) {
+    // 1. Per-invocation override — highest priority.
+    if (options.override) {
+        return options.override;
+    }
+    // 2. Env var. Read on every call (cheap; lets tests/CI flip the value
+    //    mid-process without restarting). Not cached, so a test that sets
+    //    EL_LINEAR_WORKSPACE_URL_KEY then unsets it sees the unset state.
+    const envKey = process.env[WORKSPACE_URL_KEY_ENV];
+    if (envKey) {
+        return envKey;
+    }
+    // Layers 3-4 are cached in-process — config and the live lookup are
+    // both stable for the CLI invocation's lifetime.
     if (cachedUrlKey) {
         return cachedUrlKey;
     }
+    // 3. Config override.
     const { workspaceUrlKey } = loadConfig();
     if (workspaceUrlKey) {
         cachedUrlKey = workspaceUrlKey;
         return workspaceUrlKey;
     }
+    // 4. Live API lookup — requires a GraphQL service.
+    if (!graphQLService) {
+        throw new Error("Could not resolve Linear workspace URL key — no override, env, or config set, " +
+            "and no GraphQL service was provided for the live lookup. " +
+            `Set \`workspaceUrlKey\` in your el-linear config, or \`${WORKSPACE_URL_KEY_ENV}\` in the environment.`);
+    }
     const data = await graphQLService.rawRequest(VIEWER_ORG_URL_KEY_QUERY);
     const fetched = data?.viewer?.organization?.urlKey;
     if (!fetched) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@enrichlayer/el-linear",
-  "version": "1.7.0",
+  "version": "1.10.0",
   "description": "A pragmatic CLI for Linear.app — deterministic team/label/member resolution, structured issue validation, configurable term enforcement, and a GraphQL escape hatch.",
   "main": "dist/main.js",
   "types": "dist/main.d.ts",