@unbrained/pm-cli 2026.5.18 → 2026.5.24

package/dist/cli/commands/create.js

@@ -1,5 +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]="f593bc8f-51a3-5004-8373-9eb6a695b553")}catch(e){}}();
+!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";
@@ -7,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";
@@ -21,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"]);
@@ -316,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 };
@@ -599,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);
@@ -678,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) {
@@ -774,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);
@@ -813,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));
@@ -1127,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 = {};
@@ -1244,7 +1221,7 @@ export async function runCreate(options, global) {
     }
     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) {
@@ -1498,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")
         ? []
@@ -1572,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);
@@ -1599,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);
@@ -1691,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,
@@ -1755,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,
@@ -1764,4 +1769,4 @@ export async function runCreate(options, global) {
     };
 }
 //# sourceMappingURL=create.js.map
-//# debugId=f593bc8f-51a3-5004-8373-9eb6a695b553
+//# debugId=64b2c911-6e92-5df3-a24a-40c811b5939e