@enrichlayer/el-linear 1.40.0 → 1.41.1

package/README.md

@@ -275,7 +275,11 @@ itself: `el-linear teams list --raw | jq '.[] | {key, id}'`, etc.
 [SOP-label parent gate](./docs/configuration.md#sop-label-parent-gate-validationsoplabelparentgate),
 and an opt-in
 [goal-completion gate](./docs/configuration.md#goal-completion-gate-validationgoalcompletiongate)
-(requires a falsifiable "Done when" / acceptance-criteria section).
+(requires a falsifiable "Done when" / acceptance-criteria section), plus an
+opt-in
+[intake-decision gate](./docs/configuration.md#intake-decision-gate-validationintakedecisiongate)
+that requires need, value, ownership, and placement to be decided before issue
+creation.
 el-linear can record each gate's fire/override decision to a local JSONL file so
 you can measure its **override-rate** and tell whether it's too aggressive. It is
 **off by default** and writes nothing unless you opt in (e.g.

package/dist/commands/issues.js

@@ -2,13 +2,14 @@ import { execFileSync } from "node:child_process";
 import { loadConfig } from "../config/config.js";
 import { enrichProjectResolverError, enrichValidationErrors, } from "../config/error-enrichment.js";
 import { evaluateGoalCompletion, formatGoalCompletionBlock, getGoalCompletionGateConfig, } from "../config/goal-completion-validation.js";
+import { evaluateIntakeDecision, formatIntakeDecisionBlock, getIntakeDecisionGateConfig, } from "../config/intake-decision-validation.js";
 import { enforceValidation, validateIssueCreation, } from "../config/issue-validation.js";
 import { resolveAssignee, resolveLabels, resolveMemberWithRegistry, resolveTeam, } from "../config/resolver.js";
 import { formatSopParentBlock, getSopLabelGateConfig, hasSopLabel, isUnresolvableReferenceError, } from "../config/sop-label-validation.js";
 import { resolveDefaultStatus } from "../config/status-defaults.js";
 import { enforceTerms } from "../config/term-enforcer.js";
 import { GET_ISSUE_RELATIONS_QUERY, GET_ISSUE_STATE_HISTORY_QUERY, } from "../queries/issues.js";
-import { DEFAULT_DUPLICATE_THRESHOLD, DEFAULT_HARD_BLOCK_THRESHOLD, formatDuplicateBlock, scoreDuplicateCandidates, tokenizeTitle, } from "../utils/duplicate-detection.js";
+import { bypassesDuplicateHardBlock, DEFAULT_DUPLICATE_THRESHOLD, DEFAULT_HARD_BLOCK_THRESHOLD, formatDuplicateBlock, scoreDuplicateCandidates, tokenizeTitle, } from "../utils/duplicate-detection.js";
 import { createFileService } from "../utils/file-service.js";
 import { applyFooter } from "../utils/footer.js";
 import { emitGateEvent } from "../utils/gate-telemetry.js";
@@ -588,7 +589,12 @@ async function enforceNoDuplicateIssue(title, options, issuesService) {
     // The gate would hard-block. Record the decision so `el-telemetry gates`
     // can compute override-rate (DEV-4834): `overridden` when the user passed
     // --allow-duplicate and we proceed anyway, `blocked` when we stop creation.
-    if (options.allowDuplicate) {
+    //
+    // The escape set lives in `bypassesDuplicateHardBlock` rather than inline so
+    // the remedy copy in `formatDuplicateBlock` is derived from the same source
+    // this decision reads (DEV-6205 — the message previously named a flag the
+    // gate never consulted).
+    if (bypassesDuplicateHardBlock(options)) {
         await emitGateEvent("el-linear", "issues create", {
             ...gateEvent,
             outcome: "overridden",
@@ -762,6 +768,43 @@ async function enforceGoalCompletion(description, options) {
     });
     throw new Error(`Issue creation blocked: ${message}`);
 }
+/**
+ * DEV-6163: require the intake judgment before any create-time resolution or
+ * mutation. Unlike general field validation, this gate is not silently
+ * bypassed by `--skip-validation`; only its narrow, recorded override applies.
+ */
+async function enforceIntakeDecision(description, options) {
+    const { mode, headers } = getIntakeDecisionGateConfig();
+    if (mode === "off") {
+        return;
+    }
+    const evaluation = evaluateIntakeDecision(description, headers);
+    if (evaluation.ok) {
+        return;
+    }
+    const gateEvent = { gate: "issues-create-intake-decision" };
+    if (options.allowMissingIntakeDecision) {
+        await emitGateEvent("el-linear", "issues create", {
+            ...gateEvent,
+            outcome: "overridden",
+        });
+        return;
+    }
+    const message = formatIntakeDecisionBlock({ evaluation, headers });
+    if (mode === "warn") {
+        await emitGateEvent("el-linear", "issues create", {
+            ...gateEvent,
+            outcome: "advisory",
+        });
+        outputWarning(message);
+        return;
+    }
+    await emitGateEvent("el-linear", "issues create", {
+        ...gateEvent,
+        outcome: "blocked",
+    });
+    throw new Error(`Issue creation blocked: ${message}`);
+}
 async function handleCreateIssue(title, options, command) {
     const rootOpts = getRootOpts(command);
     // DEV-5920 (cycle-2): resolve the description exactly ONCE, before anything
@@ -779,6 +822,7 @@ async function handleCreateIssue(title, options, command) {
     // normalization) — re-resolving an inline copy would corrupt a file that
     // intentionally contains backslash sequences.
     const rawDescription = resolveDescription(options);
+    await enforceIntakeDecision(rawDescription ?? "", options);
     const { teamInput, teamId, assigneeId, delegateId, labelIds, status, subscriberIds, priority, } = await resolveCreateInputs(title ?? "", options, rootOpts, rawDescription);
     const uploadResults = await uploadAttachmentsIfNeeded(options, rootOpts);
     const descriptionWithAttachments = buildDescriptionWithAttachments(rawDescription ?? "", uploadResults);
@@ -1401,10 +1445,11 @@ export function setupIssuesCommands(program) {
         .option("--due-date <date>", "due date (YYYY-MM-DD)")
         .option("--checkout", "create and checkout a git branch named after the issue")
         .option("--no-claim", "with --checkout, skip assigning the issue to the current Linear user and moving it to the first started state")
-        .option("--skip-validation", "skip all validation (labels, description, assignee, project, duplicate detection, SOP-parent gate, goal-completion gate)")
+        .option("--skip-validation", "skip general validation (labels, description, assignee, project, duplicate detection, SOP-parent gate, goal-completion gate); the intake-decision gate still requires its narrow override")
         .option("--allow-duplicate", "silence the duplicate-detection hard block and create even if a near-identical issue already exists (DEV-5590: only matters at/above the hard-block threshold — below it the gate is advisory-only and never blocks)")
         .option("--allow-unparented-sop", "skip the SOP-label parent gate and create an SOP-labeled issue without a parent SOP")
         .option("--allow-vague-goal", 'skip the goal-completion gate and create even without a falsifiable "Done when" / acceptance-criteria section')
+        .option("--allow-missing-intake-decision", "create without a complete intake decision when an accountable human approved the exception (recorded as a gate override)")
         .option("--no-auto-link", "skip auto-linking issue references found in the description")
         .option("--footer <text>", "text appended to the description (overrides config.messageFooter)")
         .option("--no-footer", "skip the configured messageFooter for this issue")

package/dist/commands/read-shortcut.js

@@ -73,14 +73,21 @@ export async function readIssues(issueIds, options, command) {
     const rootOpts = getRootOpts(command);
     const { graphQLService, issuesService } = await createIssuesService(rootOpts);
     const fileService = await createFileService(rootOpts);
-    const fieldName = typeof options.field === "string" ? options.field : null;
-    const bodyOnly = options.body === true;
-    const sectionsRaw = typeof options.sections === "string" ? options.sections : null;
+    // `issues` owns these options for its no-subcommand shorthand, while
+    // `issues read` registers the same options on the child command. Commander
+    // stores a duplicated flag on the parent even when it appears after `read`,
+    // so the action callback's local options can be empty. Merge the command
+    // hierarchy here, with local values winning, so every read route shares the
+    // same deterministic option contract.
+    const readOptions = { ...command.optsWithGlobals(), ...options };
+    const fieldName = typeof readOptions.field === "string" ? readOptions.field : null;
+    const bodyOnly = readOptions.body === true;
+    const sectionsRaw = typeof readOptions.sections === "string" ? readOptions.sections : null;
     // DEV-4476: --with opt-in includes (currently `relations`). Throws on
     // unknown values via parseWithIncludes — fail fast in the CLI per the
     // deterministic-CLI doctrine.
-    const includes = typeof options.with === "string"
-        ? parseWithIncludes(options.with)
+    const includes = typeof readOptions.with === "string"
+        ? parseWithIncludes(readOptions.with)
         : { relations: false };
     if (fieldName && sectionsRaw) {
         throw new Error("--field and --sections are mutually exclusive. Use --field for a single section (plain-text output) or --sections for multiple (JSON map).");

package/dist/config/config.d.ts

@@ -115,6 +115,18 @@ export interface ElLinearConfig {
          * "Acceptance criteria", "Success criteria"]`.
          */
         goalSectionHeaders?: string[];
+        /**
+         * OPT-IN explicit intake gate (DEV-6163). When `"warn"` or `"block"`,
+         * `issues create` requires an ordered `Intake decision` section recording
+         * why the work is needed, why it is worth doing, the existing/duplicate
+         * work checked, its canonical owner, concrete placement, and a `PROCEED`
+         * decision. Defaults to off for the
+         * open-source package. A narrow, recorded override is available through
+         * `--allow-missing-intake-decision`.
+         */
+        intakeDecisionGate?: false | "warn" | "block";
+        /** Headers accepted for the intake section. Defaults to `["Intake decision"]`. */
+        intakeSectionHeaders?: string[];
     };
     /**
      * Optional override for the Linear workspace URL key (the part after

package/dist/config/intake-decision-validation.d.ts

@@ -0,0 +1,54 @@
+/**
+ * Explicit issue-intake validation — DEV-6163.
+ *
+ * This gate does not try to decide whether work is worthwhile or where it
+ * belongs. It requires the author to record those judgments, in order, before
+ * `issues create` can mutate Linear. The deterministic part is completeness:
+ * needed, worth doing, existing/duplicate work checked, canonical owner,
+ * concrete placement, then a proceed decision.
+ *
+ * OPT-IN by design. el-linear is open source, so the gate is dormant unless a
+ * workspace config sets `validation.intakeDecisionGate` to `"warn"` or
+ * `"block"`.
+ */
+export declare const DEFAULT_INTAKE_SECTION_HEADERS: string[];
+export type IntakeDecisionGateMode = "off" | "warn" | "block";
+export interface IntakeDecisionGateConfig {
+    mode: IntakeDecisionGateMode;
+    headers: string[];
+}
+export declare function getIntakeDecisionGateConfig(): IntakeDecisionGateConfig;
+export type IntakeDecisionEvaluation = {
+    ok: true;
+    header: string;
+} | {
+    ok: false;
+    reason: "no-section";
+} | {
+    ok: false;
+    reason: "missing-field";
+    field: string;
+} | {
+    ok: false;
+    reason: "duplicate-field";
+    field: string;
+} | {
+    ok: false;
+    reason: "out-of-order";
+    field: string;
+} | {
+    ok: false;
+    reason: "invalid-field";
+    field: string;
+} | {
+    ok: false;
+    reason: "not-proceeding";
+    decision: string;
+};
+export declare function evaluateIntakeDecision(description: string, headers?: string[]): IntakeDecisionEvaluation;
+export declare function formatIntakeDecisionBlock(opts: {
+    evaluation: Exclude<IntakeDecisionEvaluation, {
+        ok: true;
+    }>;
+    headers: string[];
+}): string;

package/dist/config/intake-decision-validation.js

@@ -0,0 +1,161 @@
+/**
+ * Explicit issue-intake validation — DEV-6163.
+ *
+ * This gate does not try to decide whether work is worthwhile or where it
+ * belongs. It requires the author to record those judgments, in order, before
+ * `issues create` can mutate Linear. The deterministic part is completeness:
+ * needed, worth doing, existing/duplicate work checked, canonical owner,
+ * concrete placement, then a proceed decision.
+ *
+ * OPT-IN by design. el-linear is open source, so the gate is dormant unless a
+ * workspace config sets `validation.intakeDecisionGate` to `"warn"` or
+ * `"block"`.
+ */
+import { extractField, stripFencedCodeBlocks } from "../utils/extract-field.js";
+import { loadConfig } from "./config.js";
+export const DEFAULT_INTAKE_SECTION_HEADERS = ["Intake decision"];
+export function getIntakeDecisionGateConfig() {
+    const validation = loadConfig().validation;
+    const raw = validation?.intakeDecisionGate;
+    const mode = validation?.enabled !== false && (raw === "warn" || raw === "block")
+        ? raw
+        : "off";
+    const headers = validation?.intakeSectionHeaders &&
+        validation.intakeSectionHeaders.length > 0
+        ? validation.intakeSectionHeaders
+        : DEFAULT_INTAKE_SECTION_HEADERS;
+    return { mode, headers };
+}
+const FIELD_DEFINITIONS = [
+    { key: "needed", label: "Needed" },
+    { key: "worth", label: "Worth doing" },
+    { key: "existing", label: "Existing work" },
+    { key: "owner", label: "Owner" },
+    { key: "placement", label: "Placement" },
+    { key: "decision", label: "Decision" },
+];
+const FIELD_LINE = /^\s*(?:[-*+]\s+|\d+[.)]\s+)?(?:\*\*)?(Needed|Worth doing|Existing work|Owner|Placement|Decision)(?:\*\*)?\s*:\s*(.*?)\s*$/gim;
+const PLACEHOLDER = /^(?:tbd|todo|unknown|n\/?a|none|unsure|not decided|-)\.?$/i;
+const NON_SPECIFIC = /^(?:yes|no)$/i;
+const AFFIRMATIVE_WITH_REASON = /^yes\s*(?:[-—:;,]|because)\s*(\S.{2,})$/i;
+function normalizedKey(label) {
+    switch (label.toLowerCase()) {
+        case "needed":
+            return "needed";
+        case "worth doing":
+            return "worth";
+        case "existing work":
+            return "existing";
+        case "owner":
+            return "owner";
+        case "placement":
+            return "placement";
+        default:
+            return "decision";
+    }
+}
+export function evaluateIntakeDecision(description, headers = DEFAULT_INTAKE_SECTION_HEADERS) {
+    let section = null;
+    let matchedHeader = "";
+    for (const header of headers) {
+        section = extractField(description, header);
+        if (section !== null) {
+            matchedHeader = header;
+            break;
+        }
+    }
+    if (section === null) {
+        return { ok: false, reason: "no-section" };
+    }
+    // A template/example fence is not an operative decision record. Remove all
+    // fenced examples before matching so copied guidance cannot satisfy intake.
+    const operativeSection = stripFencedCodeBlocks(section);
+    const values = new Map();
+    for (const match of operativeSection.matchAll(FIELD_LINE)) {
+        const key = normalizedKey(match[1] ?? "");
+        if (values.has(key)) {
+            return {
+                ok: false,
+                reason: "duplicate-field",
+                field: FIELD_DEFINITIONS.find((field) => field.key === key)?.label ?? key,
+            };
+        }
+        values.set(key, { value: match[2]?.trim() ?? "", index: match.index });
+    }
+    let previousIndex = -1;
+    for (const definition of FIELD_DEFINITIONS) {
+        const entry = values.get(definition.key);
+        if (!entry) {
+            return {
+                ok: false,
+                reason: "missing-field",
+                field: definition.label,
+            };
+        }
+        if (entry.index < previousIndex) {
+            return {
+                ok: false,
+                reason: "out-of-order",
+                field: definition.label,
+            };
+        }
+        previousIndex = entry.index;
+    }
+    for (const key of ["needed", "worth"]) {
+        const value = values.get(key)?.value ?? "";
+        const match = value.match(AFFIRMATIVE_WITH_REASON);
+        const reason = match?.[1]?.trim() ?? "";
+        if (!match || PLACEHOLDER.test(reason)) {
+            return {
+                ok: false,
+                reason: "invalid-field",
+                field: key === "needed" ? "Needed" : "Worth doing",
+            };
+        }
+    }
+    for (const key of ["existing", "owner", "placement"]) {
+        const value = values.get(key)?.value ?? "";
+        if (value.length < 3 ||
+            PLACEHOLDER.test(value) ||
+            NON_SPECIFIC.test(value)) {
+            return {
+                ok: false,
+                reason: "invalid-field",
+                field: key === "existing"
+                    ? "Existing work"
+                    : key === "owner"
+                        ? "Owner"
+                        : "Placement",
+            };
+        }
+    }
+    const decision = values.get("decision")?.value ?? "";
+    if (!/^proceed$/i.test(decision)) {
+        return { ok: false, reason: "not-proceeding", decision };
+    }
+    return { ok: true, header: matchedHeader };
+}
+export function formatIntakeDecisionBlock(opts) {
+    const { evaluation } = opts;
+    let reason;
+    switch (evaluation.reason) {
+        case "no-section":
+            reason = `Issue description has no intake section (looked for: ${opts.headers.join(", ")}).`;
+            break;
+        case "missing-field":
+            reason = `The intake decision is missing the "${evaluation.field}" field.`;
+            break;
+        case "duplicate-field":
+            reason = `The intake decision repeats the "${evaluation.field}" field; record one unambiguous value.`;
+            break;
+        case "out-of-order":
+            reason = `The intake fields are out of order at "${evaluation.field}".`;
+            break;
+        case "invalid-field":
+            reason = `The intake field "${evaluation.field}" is empty, a placeholder, or lacks an explicit yes-and-reason judgment.`;
+            break;
+        case "not-proceeding":
+            reason = `The intake decision is "${evaluation.decision || "empty"}", not PROCEED.`;
+    }
+    return `${reason}\n\nRecord the decision before creating the issue, in this exact order:\n\n## ${opts.headers[0]}\n- Needed: Yes — <why this is needed>\n- Worth doing: Yes — <why the value exceeds the cost>\n- Existing work: <duplicate/search result and evidence>\n- Owner: <canonical owner or source of truth>\n- Placement: <team/project/repository/document path>\n- Decision: PROCEED\n\nIf an accountable human has approved an exceptional create, re-run with --allow-missing-intake-decision; that override is recorded.`;
+}

package/dist/utils/duplicate-detection.d.ts

@@ -70,6 +70,30 @@ export declare const DEFAULT_DUPLICATE_THRESHOLD = 0.35;
  * Overridable via `config.validation.duplicateHardBlockThreshold`.
  */
 export declare const DEFAULT_HARD_BLOCK_THRESHOLD = 0.6;
+/**
+ * The one CLI flag that lets a HARD-BLOCKED create proceed.
+ *
+ * Exported so the remedy copy in {@link formatDuplicateBlock} is BUILT from the
+ * same constant the gate honors, rather than restating it. DEV-6205 shipped a
+ * remedy naming `--parent <id>` alone — a flag the gate never reads — so the
+ * message promised an outcome the command refused. Deriving the copy from this
+ * constant makes that class of drift a compile-time concern instead of a
+ * proofreading one.
+ */
+export declare const DUPLICATE_GATE_OVERRIDE_FLAG = "--allow-duplicate";
+/**
+ * Does this parsed option set clear the duplicate gate's HARD block?
+ *
+ * The single source of truth for the escape set, consumed by the gate itself
+ * (`enforceNoDuplicateIssue`) and asserted against the rendered remedy copy by
+ * the composition test. Note `--skip-validation` also bypasses, but it returns
+ * long before scoring (it skips ALL field validation), so it is not part of the
+ * hard-block decision this predicate models and is deliberately not offered as
+ * a remedy.
+ */
+export declare function bypassesDuplicateHardBlock(options: {
+    allowDuplicate?: unknown;
+}): boolean;
 /** A scored duplicate candidate, ready to print in the block. */
 export interface DuplicateCandidate {
     identifier: string;
@@ -116,5 +140,30 @@ export declare function scoreDuplicateCandidates(title: string, candidates: Line
  * "advisory"` is printed as a warning when the score is below the hard-block
  * threshold: creation already proceeded, so the trailing hint differs (no
  * "re-run" — there's nothing to re-run).
+ *
+ * Three remedies, not two (DEV-6205). "Same work → comment" and "distinct →
+ * --allow-duplicate" leave out the most common real case: the new work is a
+ * piece of the match — neither a duplicate of it nor unrelated to it. Offering
+ * only the two extremes pushes the operator toward reusing the matched issue,
+ * which is actively harmful when that issue is a multi-phase parent: the branch
+ * then carries the parent's id, and merging it auto-closes work that isn't done.
+ * That is not hypothetical — it is what the omission cost on MAR-744, where a
+ * findings pack was filed against a six-criterion parent because `--parent` was
+ * never mentioned.
+ *
+ * The sub-issue remedy is per-tier, NOT shared, because the two tiers are at
+ * opposite sides of the create:
+ *
+ * - **block** — creation was refused, so the remedy is a re-run. `--parent`
+ *   alone does NOT satisfy the gate ({@link bypassesDuplicateHardBlock} reads
+ *   only `allowDuplicate`), so the copy is built from
+ *   {@link DUPLICATE_GATE_OVERRIDE_FLAG} and names both flags. A message that
+ *   names a command the gate then refuses is the DEV-6205 bug one level down;
+ *   the composition test parses this copy through the real CLI and asserts it
+ *   actually clears the gate.
+ * - **advisory** — creation ALREADY proceeded, so there is nothing to re-run
+ *   and a re-run would file a second issue (which then scores 1.0 against its
+ *   own twin and hard-blocks). The remedy is to attach the issue that now
+ *   exists, via `issues update`.
  */
 export declare function formatDuplicateBlock(candidates: DuplicateCandidate[], mode?: "block" | "advisory"): string;

package/dist/utils/duplicate-detection.js

@@ -141,6 +141,30 @@ const BOILERPLATE_STOPWORDS = new Set([
     "create",
     "update",
 ]);
+/**
+ * The one CLI flag that lets a HARD-BLOCKED create proceed.
+ *
+ * Exported so the remedy copy in {@link formatDuplicateBlock} is BUILT from the
+ * same constant the gate honors, rather than restating it. DEV-6205 shipped a
+ * remedy naming `--parent <id>` alone — a flag the gate never reads — so the
+ * message promised an outcome the command refused. Deriving the copy from this
+ * constant makes that class of drift a compile-time concern instead of a
+ * proofreading one.
+ */
+export const DUPLICATE_GATE_OVERRIDE_FLAG = "--allow-duplicate";
+/**
+ * Does this parsed option set clear the duplicate gate's HARD block?
+ *
+ * The single source of truth for the escape set, consumed by the gate itself
+ * (`enforceNoDuplicateIssue`) and asserted against the rendered remedy copy by
+ * the composition test. Note `--skip-validation` also bypasses, but it returns
+ * long before scoring (it skips ALL field validation), so it is not part of the
+ * hard-block decision this predicate models and is deliberately not offered as
+ * a remedy.
+ */
+export function bypassesDuplicateHardBlock(options) {
+    return Boolean(options.allowDuplicate);
+}
 /**
  * Tokenize a title into a set of salient lowercase keywords.
  *
@@ -219,19 +243,52 @@ export function scoreDuplicateCandidates(title, candidates, threshold = DEFAULT_
  * "advisory"` is printed as a warning when the score is below the hard-block
  * threshold: creation already proceeded, so the trailing hint differs (no
  * "re-run" — there's nothing to re-run).
+ *
+ * Three remedies, not two (DEV-6205). "Same work → comment" and "distinct →
+ * --allow-duplicate" leave out the most common real case: the new work is a
+ * piece of the match — neither a duplicate of it nor unrelated to it. Offering
+ * only the two extremes pushes the operator toward reusing the matched issue,
+ * which is actively harmful when that issue is a multi-phase parent: the branch
+ * then carries the parent's id, and merging it auto-closes work that isn't done.
+ * That is not hypothetical — it is what the omission cost on MAR-744, where a
+ * findings pack was filed against a six-criterion parent because `--parent` was
+ * never mentioned.
+ *
+ * The sub-issue remedy is per-tier, NOT shared, because the two tiers are at
+ * opposite sides of the create:
+ *
+ * - **block** — creation was refused, so the remedy is a re-run. `--parent`
+ *   alone does NOT satisfy the gate ({@link bypassesDuplicateHardBlock} reads
+ *   only `allowDuplicate`), so the copy is built from
+ *   {@link DUPLICATE_GATE_OVERRIDE_FLAG} and names both flags. A message that
+ *   names a command the gate then refuses is the DEV-6205 bug one level down;
+ *   the composition test parses this copy through the real CLI and asserts it
+ *   actually clears the gate.
+ * - **advisory** — creation ALREADY proceeded, so there is nothing to re-run
+ *   and a re-run would file a second issue (which then scores 1.0 against its
+ *   own twin and hard-blocks). The remedy is to attach the issue that now
+ *   exists, via `issues update`.
  */
 export function formatDuplicateBlock(candidates, mode = "block") {
     const lines = candidates.map((c) => `    ${c.identifier} · ${c.title} · ${c.state} · ${c.assignee}  (similarity ${c.score})`);
+    // Name the parent when there is exactly one candidate; with several there is
+    // no single right parent, and silently picking the top score would invite a
+    // wrong one.
+    const parentRef = candidates.length === 1 ? candidates[0].identifier : "<id>";
     const header = `Possible duplicate issue${candidates.length > 1 ? "s" : ""} found ` +
         "(by title-keyword overlap):\n" +
         `${lines.join("\n")}\n\n` +
         "  If one of these is the same work, comment on it instead of creating a new issue.\n";
     if (mode === "advisory") {
         return (header +
+            "  If this is a piece of one of them rather than a duplicate, attach it with " +
+            `\`issues update <new-id> --parent ${parentRef}\`.\n` +
             "  This is advisory only (DEV-5590) — creation is proceeding. Pass " +
             "--allow-duplicate to silence this notice next time.");
     }
     return (header +
-        "  If this is genuinely distinct, re-run with --allow-duplicate to proceed " +
+        "  If this is a piece of one of them rather than a duplicate, re-run with " +
+        `--parent ${parentRef} ${DUPLICATE_GATE_OVERRIDE_FLAG} to file it as a sub-issue.\n` +
+        `  If this is genuinely distinct, re-run with ${DUPLICATE_GATE_OVERRIDE_FLAG} to proceed ` +
         "(and consider --related-to to link the related issue).");
 }

package/dist/utils/extract-field.d.ts

@@ -16,6 +16,8 @@
  * Returns `null` when the field isn't found, so callers can distinguish
  * "missing" from "empty body".
  */
+/** Remove operative text inside CommonMark fenced code blocks, including an unclosed block through EOF. */
+export declare function stripFencedCodeBlocks(body: string): string;
 /**
  * Multi-section variant of `extractField`. Extracts each requested section
  * by name in one call and returns a `{section -> text|null}` map preserving

package/dist/utils/extract-field.js

@@ -28,7 +28,50 @@ const BOLD_PSEUDO_HEADER_RE = /^\*\*([^*\n]+?)(?::\s*)?\*\*\s*$/;
 // permitted as fence indent per the CommonMark spec). We toggle an inFence
 // flag while scanning so section headers inside code samples don't
 // terminate the real section.
-const FENCE_RE = /^ {0,3}(`{3,}|~{3,})/;
+const FENCE_DELIMITER_RE = /^ {0,3}(`{3,}|~{3,})(.*)$/;
+function fenceDelimiter(line) {
+    const match = line.match(FENCE_DELIMITER_RE);
+    const run = match?.[1] ?? "";
+    if (!run || new Set(run).size !== 1)
+        return null;
+    return { run, rest: match?.[2] ?? "" };
+}
+function openFence(line) {
+    const delimiter = fenceDelimiter(line);
+    if (!delimiter)
+        return null;
+    const marker = delimiter.run[0];
+    // CommonMark forbids backticks in the info string of a backtick fence.
+    if (marker === "`" && delimiter.rest.includes("`"))
+        return null;
+    return { marker, length: delimiter.run.length };
+}
+function closesFence(line, state) {
+    const delimiter = fenceDelimiter(line);
+    return (delimiter !== null &&
+        delimiter.run[0] === state.marker &&
+        delimiter.run.length >= state.length &&
+        delimiter.rest.trim().length === 0);
+}
+/** Remove operative text inside CommonMark fenced code blocks, including an unclosed block through EOF. */
+export function stripFencedCodeBlocks(body) {
+    const out = [];
+    let fence = null;
+    for (const line of body.split("\n")) {
+        if (fence) {
+            if (closesFence(line, fence))
+                fence = null;
+            continue;
+        }
+        const opened = openFence(line);
+        if (opened) {
+            fence = opened;
+            continue;
+        }
+        out.push(line);
+    }
+    return out.join("\n");
+}
 function normalize(s) {
     return s.toLowerCase().trim().replace(/\s+/g, " ");
 }
@@ -86,15 +129,19 @@ export function extractField(body, fieldName) {
         return null;
     const target = normalize(fieldName);
     const lines = body.split("\n");
-    let inFence = false;
+    let fence = null;
     let startIdx = -1;
     for (let i = 0; i < lines.length; i++) {
-        if (FENCE_RE.test(lines[i])) {
-            inFence = !inFence;
+        if (fence) {
+            if (closesFence(lines[i], fence))
+                fence = null;
             continue;
         }
-        if (inFence)
+        const opened = openFence(lines[i]);
+        if (opened) {
+            fence = opened;
             continue;
+        }
         const match = matchHeader(lines[i]);
         if (!match)
             continue;
@@ -106,14 +153,21 @@ export function extractField(body, fieldName) {
     if (startIdx === -1)
         return null;
     const out = [];
-    let sectionInFence = false;
+    let sectionFence = null;
     for (let i = startIdx; i < lines.length; i++) {
-        if (FENCE_RE.test(lines[i])) {
-            sectionInFence = !sectionInFence;
+        if (sectionFence) {
+            if (closesFence(lines[i], sectionFence))
+                sectionFence = null;
+            out.push(lines[i]);
+            continue;
+        }
+        const opened = openFence(lines[i]);
+        if (opened) {
+            sectionFence = opened;
             out.push(lines[i]);
             continue;
         }
-        if (!sectionInFence && matchHeader(lines[i]) !== null)
+        if (matchHeader(lines[i]) !== null)
             break;
         out.push(lines[i]);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@enrichlayer/el-linear",
-  "version": "1.40.0",
+  "version": "1.41.1",
   "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",