@unbrained/pm-cli 2026.5.14 → 2026.5.24

package/dist/cli/commands/create.js

@@ -1,3 +1,5 @@
+
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="64b2c911-6e92-5df3-a24a-40c811b5939e")}catch(e){}}();
 import { pathExists, removeFileIfExists, writeFileAtomic } from "../../core/fs/fs-utils.js";
 import { appendHistoryEntry, createHistoryEntry } from "../../core/history/history.js";
 import { generateItemId, normalizeItemId } from "../../core/item/id.js";
@@ -5,9 +7,11 @@ import { canonicalDocument, normalizeFrontMatter, serializeItemDocument } from "
 import { normalizeParentReferenceValue, validateMissingParentReference, } from "../../core/item/parent-reference-policy.js";
 import { validateSprintOrReleaseValue } from "../../core/item/sprint-release-format.js";
 import { createStdinTokenResolver, parseCsvKv, parseOptionalNumber, parseTags } from "../../core/item/parse.js";
+import { resolvePriority } from "../../core/item/priority.js";
 import { normalizeStatusInput } from "../../core/item/status.js";
 import { canonicalizeCommandOptionKey, commandOptionFlagLabel, resolveItemTypeRegistry, resolveCommandOptionPolicyState, resolveTypeDefinition, resolveTypeName, validateTypeOptions, } from "../../core/item/type-registry.js";
 import { acquireLock } from "../../core/lock/lock.js";
+import { buildInvalidTypeError } from "../../core/schema/item-types-file.js";
 import { collectRuntimeCreateFieldValues } from "../../core/schema/runtime-field-values.js";
 import { resolveRuntimeFieldRegistry, resolveRuntimeStatusRegistry, } from "../../core/schema/runtime-schema.js";
 import { EXIT_CODE, FRONT_MATTER_KEY_ORDER } from "../../core/shared/constants.js";
@@ -19,7 +23,9 @@ import { locateItem } from "../../core/store/item-store.js";
 import { getHistoryPath, getItemPath, getSettingsPath, resolvePmRoot } from "../../core/store/paths.js";
 import { readSettings } from "../../core/store/settings.js";
 import { normalizeRiskInput, normalizeSeverityInput, parseConfidenceInput, parseRegressionInput, } from "./metadata-normalizers.js";
-import { DEPENDENCY_KIND_VALUES, ISSUE_SEVERITY_VALUES, RECURRENCE_FREQUENCY_VALUES, RECURRENCE_WEEKDAY_VALUES, RISK_VALUES, SCOPE_VALUES, } from "../../types/index.js";
+import { resolveEventEndAt } from "./event-validation-messages.js";
+import { looksLikeStructuredLinkedTestEntry, normalizeStructuredLinkedTestEntry } from "./linked-test-entry.js";
+import { DEPENDENCY_KIND_VALUES, ISSUE_SEVERITY_VALUES, RECURRENCE_FREQUENCY_VALUES, RECURRENCE_WEEKDAY_VALUES, RISK_VALUES, SCOPE_VALUES, weekdayOrderIndex, } from "../../types/index.js";
 const CREATE_MODE_VALUES = ["strict", "progressive"];
 const SCHEDULE_CREATE_PRESET_VALUES = ["lightweight"];
 const SCHEDULE_CREATE_PRESET_TYPES = new Set(["Reminder", "Meeting", "Event"]);
@@ -314,9 +320,6 @@ function parseCreateUnsetTargets(raw, runtimeFieldRegistry) {
     }
     return { frontMatterKeys, optionKeys };
 }
-function weekdayOrderIndex(value) {
-    return RECURRENCE_WEEKDAY_VALUES.indexOf(value);
-}
 function parseDependencies(raw, nowValue, prefix) {
     if (!raw || raw.length === 0)
         return { values: undefined, explicitEmpty: false };
@@ -597,26 +600,8 @@ export function parseTests(raw) {
     assertNoLegacyNoneTokens(raw, "--test", "Use --clear-tests to clear linked tests.");
     const values = raw.map((entry) => {
         const trimmedEntry = entry.trim();
-        const kv = looksLikeStructuredEntry(trimmedEntry, [
-            "command",
-            "path",
-            "scope",
-            "timeout",
-            "timeout_seconds",
-            "pm_context_mode",
-            "env_set",
-            "env_clear",
-            "shared_host_safe",
-            "assert_stdout_contains",
-            "assert_stdout_regex",
-            "assert_stderr_contains",
-            "assert_stderr_regex",
-            "assert_stdout_min_lines",
-            "assert_json_field_equals",
-            "assert_json_field_gte",
-            "note",
-        ])
-            ? parseCsvKv(entry, "--test")
+        const kv = looksLikeStructuredLinkedTestEntry(trimmedEntry)
+            ? normalizeStructuredLinkedTestEntry(parseCsvKv(entry, "--test"), "--test")
             : { command: trimmedEntry };
         const command = parseOptionalString(kv.command);
         const filePath = parseOptionalString(kv.path);
@@ -676,10 +661,10 @@ function parseReminders(raw, nowValue) {
     assertNoLegacyNoneTokens(raw, "--reminder", "Use --clear-reminders to clear reminders.");
     const values = raw.map((entry) => {
         const kv = parseCsvKv(entry, "--reminder");
-        const atRaw = parseOptionalString(kv.at);
-        const textRaw = parseOptionalString(kv.text);
+        const atRaw = parseOptionalString(kv.at ?? kv.date);
+        const textRaw = parseOptionalString(kv.text ?? kv.title);
         if (!atRaw || !textRaw) {
-            throw new PmCliError("--reminder requires at=<iso|relative> and text=<value>", EXIT_CODE.USAGE);
+            throw new PmCliError("--reminder requires at=<iso|relative> or date=<iso|relative>, plus text=<value> or title=<value>", EXIT_CODE.USAGE);
         }
         const text = textRaw.trim();
         if (!text) {
@@ -772,10 +757,8 @@ function parseEvents(raw, nowValue) {
         }
         const startAt = resolveIsoOrRelative(startRaw, referenceDate, "event.start");
         const endRaw = parseOptionalString(kv.end)?.trim();
-        const endAt = endRaw ? resolveIsoOrRelative(endRaw, referenceDate, "event.end") : undefined;
-        if (endAt && endAt <= startAt) {
-            throw new PmCliError("--event end must be after start", EXIT_CODE.USAGE);
-        }
+        const durationRaw = parseOptionalString(kv.duration)?.trim();
+        const endAt = resolveEventEndAt(startAt, endRaw, durationRaw, referenceDate);
         const titleRaw = parseOptionalString(kv.title);
         const descriptionRaw = parseOptionalString(kv.description);
         const locationRaw = parseOptionalString(kv.location);
@@ -811,10 +794,10 @@ function parseEvents(raw, nowValue) {
     });
     return { values, explicitEmpty: false };
 }
-function buildChangedFields(frontMatter, explicitUnsets) {
+function buildChangedFields(frontMatter, body, explicitUnsets) {
     const changed = [
         ...FRONT_MATTER_KEY_ORDER.filter((key) => frontMatter[key] !== undefined),
-        "body",
+        ...(body.length > 0 ? ["body"] : []),
         ...explicitUnsets.map((key) => `unset:${key}`),
     ];
     return Array.from(new Set(changed));
@@ -1003,7 +986,7 @@ function requireCreateOptionByType(typeDefinition, options, createMode, clearOpt
         return false;
     };
     const hasOptionMutation = (optionKey) => hasOptionValue(optionKey) || clearOptionKeys.has(optionKey);
-    const baseRequiredOptions = new Set(["title", "description", "type"]);
+    const baseRequiredOptions = new Set(["title", "type"]);
     if (createMode === "strict") {
         for (const field of typeDefinition.required_create_fields) {
             baseRequiredOptions.add(normalizeCreatePolicyOptionKey(field, typeName, "required_create_fields"));
@@ -1083,11 +1066,11 @@ function createExampleTokensForFlag(flag, typeName, openStatus) {
         case "--learning":
             return ["--learning", "\"author=maintainer,created_at=now,text=Durable lesson\""];
         case "--file":
-            return ["--file", "\"path=src/example.ts,scope=project,note=implementation file\""];
+            return ["--file", "\"path=src/example.ts,note=implementation file\""];
         case "--test":
-            return ["--test", "\"command=node scripts/run-tests.mjs test,scope=project,timeout_seconds=240\""];
+            return ["--test", "\"command=node scripts/run-tests.mjs test,timeout_seconds=240\""];
         case "--doc":
-            return ["--doc", "\"path=README.md,scope=project,note=reference doc\""];
+            return ["--doc", "\"path=README.md,note=reference doc\""];
         default:
             return [flag, "\"<value>\""];
     }
@@ -1112,6 +1095,9 @@ function buildTypeSpecificCreateExample(typeDefinition, missingCreateFlags, miss
 }
 function requireStringOption(value, flag) {
     if (value === undefined) {
+        if (flag === "--title") {
+            throw new PmCliError('Missing required option --title. Why required: every item needs a human-readable title for lookup, search, and reporting. Retry: pass the title as the first positional argument (example: pm create "Fix login bug" --type Issue) or with --title.', EXIT_CODE.USAGE);
+        }
         throw new PmCliError(`Missing required option ${flag}`, EXIT_CODE.USAGE);
     }
     return value;
@@ -1122,11 +1108,7 @@ function selectAuthor(explicitAuthor, settingsAuthor) {
     return trimmed || "unknown";
 }
 function ensurePriority(rawPriority) {
-    const parsed = parseOptionalNumber(rawPriority, "priority (expected 0..4: 0=critical, 1=high, 2=medium, 3=low, 4=minimal)");
-    if (![0, 1, 2, 3, 4].includes(parsed)) {
-        throw new PmCliError("Priority must be 0..4 (0=critical, 1=high, 2=medium, 3=low, 4=minimal)", EXIT_CODE.USAGE);
-    }
-    return parsed;
+    return resolvePriority(rawPriority);
 }
 function mergeCreateOptionsWithTemplate(templateOptions, explicitOptions) {
     const merged = {};
@@ -1219,12 +1201,27 @@ export async function runCreate(options, global) {
         const templateOptions = await loadCreateTemplateOptionsFromRuntime(templateName, global, pmRoot);
         resolvedOptions = normalizeLegacyNoneCreateOptions(mergeCreateOptionsWithTemplate(templateOptions, resolvedOptions));
     }
+    if (resolvedOptions.type === undefined) {
+        // Default-type fallback is suppressed under explicit --create-mode strict, where the strict
+        // required-option contract takes precedence and surfaces the missing_required_option envelope.
+        const explicitStrictMode = typeof resolvedOptions.createMode === "string"
+            && resolvedOptions.createMode.trim().toLowerCase() === "strict";
+        if (!explicitStrictMode) {
+            const defaultType = settings.governance.create_default_type?.trim();
+            if (defaultType && defaultType.length > 0 && resolveTypeName(defaultType, typeRegistry)) {
+                resolvedOptions.type = defaultType;
+            }
+            else if (resolveTypeName("Task", typeRegistry)) {
+                resolvedOptions.type = "Task";
+            }
+        }
+    }
     if (resolvedOptions.type === undefined) {
         throw new PmCliError("Missing required option --type <value>", EXIT_CODE.USAGE);
     }
     const resolvedTypeName = resolveTypeName(resolvedOptions.type, typeRegistry);
     if (!resolvedTypeName) {
-        throw new PmCliError(`Invalid type value "${resolvedOptions.type}". Allowed: ${typeRegistry.types.join(", ")}`, EXIT_CODE.USAGE);
+        throw new PmCliError(buildInvalidTypeError(resolvedOptions.type, typeRegistry.types), EXIT_CODE.USAGE);
     }
     const typeDefinition = resolveTypeDefinition(resolvedTypeName, typeRegistry);
     if (!typeDefinition) {
@@ -1448,6 +1445,9 @@ export async function runCreate(options, global) {
     if (combinedMissingFlags.length > 0) {
         const nextValidExample = buildTypeSpecificCreateExample(typeDefinition, missingRequiredCreateFlags, missingRequiredTypeOptionKeys, statusRegistry.open_status);
         const nextSteps = [`Run "pm create --help --type ${type}" for type-aware required option guidance.`];
+        if (combinedMissingFlags.includes("--title")) {
+            nextSteps.push('Title can also be passed as the first positional argument (example: pm create "Your title" --type ' + type + ').');
+        }
         if (createMode === "strict") {
             nextSteps.push('For staged onboarding, retry with "--create-mode progressive".');
             if (SCHEDULE_CREATE_PRESET_TYPES.has(type)) {
@@ -1475,7 +1475,7 @@ export async function runCreate(options, global) {
         });
     }
     const id = await generateItemId(pmRoot, settings.id_prefix);
-    const status = resolvedOptions.status !== undefined ? parseStatusValue(resolvedOptions.status, statusRegistry) : statusRegistry.open_status;
+    let status = resolvedOptions.status !== undefined ? parseStatusValue(resolvedOptions.status, statusRegistry) : statusRegistry.open_status;
     const priority = resolvedOptions.priority !== undefined ? ensurePriority(resolvedOptions.priority) : 2;
     const tags = unsetTargets.frontMatterKeys.has("tags")
         ? []
@@ -1549,6 +1549,10 @@ export async function runCreate(options, global) {
     const parentReferencePolicy = settings.validation.parent_reference;
     const sprintReleasePolicy = settings.validation.sprint_release_format;
     const validationWarnings = [];
+    // Event-type items with no attached schedule never surface on the calendar; warn (never block).
+    if (type.toLowerCase() === "event" && (events.values === undefined || events.values.length === 0)) {
+        validationWarnings.push(`event_without_schedule:${id}:no_time_set`);
+    }
     if (parent !== undefined) {
         parent = normalizeParentReferenceValue(parent);
         const parentLocated = await locateItem(pmRoot, parent, settings.id_prefix, settings.item_format, typeRegistry.type_to_folder);
@@ -1576,6 +1580,30 @@ export async function runCreate(options, global) {
     const blockedBy = unsetTargets.frontMatterKeys.has("blocked_by") || resolvedOptions.blockedBy === undefined
         ? undefined
         : parseOptionalString(resolvedOptions.blockedBy);
+    let dependencyValues = dependencies.values;
+    if (blockedBy !== undefined) {
+        const normalizedBlockedBy = normalizeItemId(blockedBy, settings.id_prefix);
+        const blockedByLocated = await locateItem(pmRoot, normalizedBlockedBy, settings.id_prefix, settings.item_format, typeRegistry.type_to_folder);
+        if (blockedByLocated) {
+            const hasBlockedByDependency = (dependencyValues ?? []).some((dependency) => dependency.id === blockedByLocated.id && dependency.kind === "blocked_by");
+            if (!hasBlockedByDependency) {
+                dependencyValues = [
+                    ...(dependencyValues ?? []),
+                    {
+                        id: blockedByLocated.id,
+                        kind: "blocked_by",
+                        created_at: nowValue,
+                        author,
+                    },
+                ];
+            }
+            if (resolvedOptions.status === undefined) {
+                status = statusRegistry.blocked_statuses.has("blocked")
+                    ? "blocked"
+                    : [...statusRegistry.blocked_statuses].sort((left, right) => left.localeCompare(right))[0] ?? statusRegistry.open_status;
+            }
+        }
+    }
     const blockedReason = unsetTargets.frontMatterKeys.has("blocked_reason") || resolvedOptions.blockedReason === undefined
         ? undefined
         : parseOptionalString(resolvedOptions.blockedReason);
@@ -1621,7 +1649,7 @@ export async function runCreate(options, global) {
         ? undefined
         : parseOptionalString(resolvedOptions.customerImpact);
     const title = requireStringOption(resolvedOptions.title, "--title");
-    const description = requireStringOption(resolvedOptions.description, "--description");
+    const description = resolvedOptions.description ?? "";
     const body = resolvedOptions.body ?? "";
     const frontMatter = normalizeFrontMatter({
         id,
@@ -1668,7 +1696,7 @@ export async function runCreate(options, global) {
         component,
         regression,
         customer_impact: customerImpact,
-        dependencies: dependencies.values,
+        dependencies: dependencyValues,
         comments: comments.values,
         notes: notes.values,
         learnings: learnings.values,
@@ -1732,7 +1760,7 @@ export async function runCreate(options, global) {
     finally {
         await lockRelease();
     }
-    const changedFields = buildChangedFields(frontMatter, explicitUnsetKeys);
+    const changedFields = buildChangedFields(frontMatter, body, explicitUnsetKeys);
     const outputItem = structuredClone(frontMatter);
     return {
         item: outputItem,
@@ -1740,4 +1768,5 @@ export async function runCreate(options, global) {
         warnings: [...validationWarnings, ...hookWarnings],
     };
 }
-//# sourceMappingURL=create.js.map
+//# sourceMappingURL=create.js.map
+//# debugId=64b2c911-6e92-5df3-a24a-40c811b5939e