@unbrained/pm-cli 2026.6.16 → 2026.6.17

package/dist/cli/commands/create.d.ts

@@ -1,6 +1,7 @@
 import { type ResolvedItemTypeDefinition } from "../../core/item/type-registry.js";
 import { type RuntimeFieldRegistry } from "../../core/schema/runtime-schema.js";
 import type { GlobalOptions } from "../../core/shared/command-types.js";
+import { type LifecycleTransitionSuggestion } from "./lifecycle-transitions.js";
 import type { Comment, ItemMetadata, LinkedDoc, LinkedFile, LinkedTest, LogNote } from "../../types/index.js";
 export interface CreateCommandOptions {
     title?: string;
@@ -80,6 +81,7 @@ export interface CreateResult {
     changed_fields: string[];
     warnings: string[];
     parent_source?: "focus";
+    next_transition?: LifecycleTransitionSuggestion;
 }
 interface CreateUnsetFieldDefinition {
     optionKey: string;

package/dist/cli/commands/create.js

@@ -1,12 +1,12 @@
 
-!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]="76f041c5-ca2b-59ab-a1a7-b0e121eb6d1d")}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]="bf0be612-8a89-5572-97e8-6774b0a5e63d")}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, mergeAdditiveTags, parseCsvKv, parseOptionalNumber, parseTags } from "../../core/item/parse.js";
+import { assertNoUnknownCsvKeys, createStdinTokenResolver, looksLikeGenericKeyValueEntry, mergeAdditiveTags, parseCsvKv, parseOptionalNumber, parseTags, } from "../../core/item/parse.js";
 import { resolvePriority } from "../../core/item/priority.js";
 import { getFocusedItem } from "../../core/session/session-state.js";
 import { normalizeStatusInput } from "../../core/item/status.js";
@@ -28,6 +28,7 @@ import { getHistoryPath, getItemPath, getSettingsPath, resolvePmRoot } from "../
 import { readSettings } from "../../core/store/settings.js";
 import { normalizeRiskInput, normalizeSeverityInput, parseConfidenceInput, parseRegressionInput, } from "./metadata-normalizers.js";
 import { assertNoLegacyNoneToken, assertNoLegacyNoneTokens, isLegacyNoneToken } from "./legacy-none-tokens.js";
+import { suggestNextLifecycleTransition, } from "./lifecycle-transitions.js";
 import { parseLinkedTestAssertionEqualsMap, parseLinkedTestAssertionGteMap, parseLinkedTestBoolean, parseLinkedTestContextMode, parseLinkedTestEnvClear, parseLinkedTestEnvSet, parseLinkedTestMinLines, parseLinkedTestRegexList, parseLinkedTestStringList, } from "./linked-test-parsers.js";
 import { looksLikeStructuredLinkedTestEntry, normalizeStructuredLinkedTestEntry } from "./linked-test-entry.js";
 import { ensureEnumValue } from "./recurrence-parsers.js";
@@ -316,15 +317,21 @@ function parseCreateUnsetTargets(raw, runtimeFieldRegistry) {
     }
     return { frontMatterKeys, optionKeys };
 }
+/** Allowed CSV/markdown keys for the create `--dep` seed (GH-258). */
+const DEP_SEED_KEYS = ["id", "kind", "type", "author", "created_at"];
+/** Allowed CSV/markdown keys for create `--file`/`--doc` seeds (GH-258). */
+const LINKED_ARTIFACT_SEED_KEYS = ["path", "scope", "note"];
 function parseDependencies(raw, nowValue, prefix) {
     if (!raw || raw.length === 0)
         return { values: undefined, explicitEmpty: false };
     assertNoLegacyNoneTokens(raw, "--dep", "Use --clear-deps to clear dependencies.");
     const values = raw.map((entry) => {
         const trimmedEntry = entry.trim();
-        const kv = looksLikeStructuredEntry(trimmedEntry, ["id", "kind", "type", "author", "created_at"])
-            ? parseCsvKv(entry, "--dep")
-            : { id: trimmedEntry, kind: "related" };
+        const isStructured = looksLikeStructuredEntry(trimmedEntry, DEP_SEED_KEYS);
+        const kv = isStructured ? parseCsvKv(entry, "--dep") : { id: trimmedEntry, kind: "related" };
+        if (isStructured) {
+            assertNoUnknownCsvKeys(kv, "--dep", DEP_SEED_KEYS);
+        }
         const id = parseOptionalString(kv.id);
         const kind = normalizeDependencyKindInput(parseOptionalString(kv.kind ?? kv.type));
         if (!id || !kind) {
@@ -359,7 +366,12 @@ function looksLikeStructuredEntry(raw, keys) {
         return true;
     }
     const keyPattern = keys.map((key) => key.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")).join("|");
-    return new RegExp(`^(?:[-*+]\\s+)?(?:${keyPattern})\\s*[:=]`, "i").test(raw);
+    if (new RegExp(`^(?:[-*+]\\s+)?(?:${keyPattern})\\s*[:=]`, "i").test(raw)) {
+        return true;
+    }
+    // A first-key typo (e.g. `bogus=v,id=pm-2`) must still be parsed so the unknown
+    // key is rejected rather than swallowed as a bare id/path value (GH-258).
+    return looksLikeGenericKeyValueEntry(raw);
 }
 export function parseLogSeed(optionName, raw, nowValue, fallbackAuthor) {
     if (!raw || raw.length === 0)
@@ -422,9 +434,11 @@ export function parseFiles(raw) {
     assertNoLegacyNoneTokens(raw, "--file", "Use --clear-files to clear linked files.");
     const values = raw.map((entry) => {
         const trimmedEntry = entry.trim();
-        const kv = looksLikeStructuredEntry(trimmedEntry, ["path", "scope", "note"])
-            ? parseCsvKv(entry, "--file")
-            : { path: trimmedEntry };
+        const isStructured = looksLikeStructuredEntry(trimmedEntry, LINKED_ARTIFACT_SEED_KEYS);
+        const kv = isStructured ? parseCsvKv(entry, "--file") : { path: trimmedEntry };
+        if (isStructured) {
+            assertNoUnknownCsvKeys(kv, "--file", LINKED_ARTIFACT_SEED_KEYS);
+        }
         if (!kv.path) {
             throw new PmCliError("--file requires path=<value> or a bare file path", EXIT_CODE.USAGE);
         }
@@ -483,9 +497,11 @@ export function parseDocs(raw) {
     assertNoLegacyNoneTokens(raw, "--doc", "Use --clear-docs to clear linked docs.");
     const values = raw.map((entry) => {
         const trimmedEntry = entry.trim();
-        const kv = looksLikeStructuredEntry(trimmedEntry, ["path", "scope", "note"])
-            ? parseCsvKv(entry, "--doc")
-            : { path: trimmedEntry };
+        const isStructured = looksLikeStructuredEntry(trimmedEntry, LINKED_ARTIFACT_SEED_KEYS);
+        const kv = isStructured ? parseCsvKv(entry, "--doc") : { path: trimmedEntry };
+        if (isStructured) {
+            assertNoUnknownCsvKeys(kv, "--doc", LINKED_ARTIFACT_SEED_KEYS);
+        }
         if (!kv.path) {
             throw new PmCliError("--doc requires path=<value> or a bare doc path", EXIT_CODE.USAGE);
         }
@@ -1620,6 +1636,10 @@ export async function runCreate(options, global) {
         await lockRelease();
     }
     const outputItem = structuredClone(frontMatter);
+    // GH-216: nudge agents toward the underutilized in_progress state instead of
+    // jumping open -> closed. Only surfaces for workable types created in the open
+    // status when the workflow defines a distinct in_progress status to move to.
+    const nextTransition = suggestNextLifecycleTransition(id, type, status, statusRegistry);
     // 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
@@ -1633,6 +1653,7 @@ export async function runCreate(options, global) {
         changed_fields: changedFields,
         warnings: [...validationWarnings, ...hookWarnings],
         ...(parentSource !== undefined ? { parent_source: parentSource } : {}),
+        ...(nextTransition !== undefined ? { next_transition: nextTransition } : {}),
     };
 }
 export const _testOnlyCreateCommand = {
@@ -1656,4 +1677,4 @@ export const _testOnlyCreateCommand = {
     typeOptionExampleValue,
 };
 //# sourceMappingURL=create.js.map
-//# debugId=76f041c5-ca2b-59ab-a1a7-b0e121eb6d1d
+//# debugId=bf0be612-8a89-5572-97e8-6774b0a5e63d