@unbrained/pm-cli 2026.5.28 → 2026.5.29

package/dist/cli/commands/create.js

@@ -1,16 +1,17 @@
 
-!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]="10f962f6-810b-5b40-a073-b763d0b0d8b0")}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]="8e1bfe68-8472-50a9-949d-d6429ca8ff20")}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";
 import { canonicalDocument, normalizeFrontMatter, serializeItemDocument } from "../../core/item/item-format.js";
 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 { createStdinTokenResolver, mergeAdditiveTags, 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 { printError } from "../../core/output/output.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";
@@ -711,6 +712,14 @@ function requireCreateOptionByType(typeDefinition, options, createMode, clearOpt
         typeOption: options.typeOption,
     };
     const hasOptionValue = (optionKey) => {
+        // `--add-tags` mutates the same `tags` field as `--tags`, so it must count
+        // toward the `tags` command_option_policy (both the disabled guard and the
+        // required check) — otherwise `--add-tags` would bypass a rule disabling
+        // tags, or fail to satisfy a rule requiring them even though the created
+        // item ends up tagged.
+        if (optionKey === "tags") {
+            return scalarValues.tags !== undefined || (Array.isArray(options.addTags) && options.addTags.length > 0);
+        }
         if (optionKey in scalarValues) {
             return scalarValues[optionKey] !== undefined;
         }
@@ -1212,11 +1221,20 @@ export async function runCreate(options, global) {
     const id = await generateItemId(pmRoot, settings.id_prefix);
     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")
+    // `--unset tags` clears the field; combining it with `--add-tags` is the same
+    // contradiction that `--unset tags --tags ...` already rejects, so reject it
+    // here rather than silently letting the additions win over the clear.
+    if (unsetTargets.frontMatterKeys.has("tags") &&
+        Array.isArray(resolvedOptions.addTags) &&
+        resolvedOptions.addTags.length > 0) {
+        throw new PmCliError("Cannot combine --unset tags with --add-tags", EXIT_CODE.USAGE);
+    }
+    const baseTags = unsetTargets.frontMatterKeys.has("tags")
         ? []
         : resolvedOptions.tags !== undefined
             ? parseTags(resolvedOptions.tags)
             : [];
+    const tags = mergeAdditiveTags(baseTags, resolvedOptions.addTags);
     const deadline = unsetTargets.frontMatterKeys.has("deadline")
         ? undefined
         : resolvedOptions.deadline === undefined
@@ -1288,6 +1306,16 @@ export async function runCreate(options, global) {
     if (type.toLowerCase() === "event" && (events.values === undefined || events.values.length === 0)) {
         validationWarnings.push(`event_without_schedule:${id}:no_time_set`);
     }
+    // Calendar-relevant types (Milestone, Meeting, Reminder, Event) with NO deadline AND no
+    // reminders AND no events are invisible on `pm calendar`. Warn (never block) so the agent
+    // can attach a schedule via `pm update`.
+    const calendarRelevantTypes = new Set(["milestone", "meeting", "reminder", "event"]);
+    const hasDeadline = deadline !== undefined;
+    const hasReminders = reminders.values !== undefined && reminders.values.length > 0;
+    const hasEvents = events.values !== undefined && events.values.length > 0;
+    if (calendarRelevantTypes.has(type.toLowerCase()) && !hasDeadline && !hasReminders && !hasEvents) {
+        validationWarnings.push(`calendar_item_without_schedule:${id}:no_deadline_or_reminder_or_event`);
+    }
     if (parent !== undefined) {
         parent = normalizeParentReferenceValue(parent);
         const parentLocated = await locateItem(pmRoot, parent, settings.id_prefix, settings.item_format, typeRegistry.type_to_folder);
@@ -1497,6 +1525,14 @@ export async function runCreate(options, global) {
     }
     const changedFields = buildChangedFields(frontMatter, body, explicitUnsetKeys);
     const outputItem = structuredClone(frontMatter);
+    // After the create has committed (so the ID is real and shows up in the suggestion),
+    // emit a single non-blocking stderr hint when the new item would be invisible on `pm
+    // calendar`. The structured `calendar_item_without_schedule:*` warning above is what
+    // automation reads; this stderr line is the human/agent-facing version with a
+    // copy-pasteable `pm update` recipe.
+    if (calendarRelevantTypes.has(type.toLowerCase()) && !hasDeadline && !hasReminders && !hasEvents) {
+        printError(`[pm] warning: ${type} '${id}' has no deadline/reminder/event — it will not appear on the calendar. Add one via 'pm update ${id} --deadline <ISO>' or 'pm update ${id} --event "title|start|end"'.`);
+    }
     return {
         item: outputItem,
         changed_fields: changedFields,
@@ -1504,4 +1540,4 @@ export async function runCreate(options, global) {
     };
 }
 //# sourceMappingURL=create.js.map
-//# debugId=10f962f6-810b-5b40-a073-b763d0b0d8b0
+//# debugId=8e1bfe68-8472-50a9-949d-d6429ca8ff20