@pasosdejesus/mpdj 0.1.0 → 0.1.1

package/dist/index.js

@@ -80,7 +80,7 @@ const commands = {
         handler: (n) => {
             if (!n)
                 throw new Error('req:sync-to-repo <N> [N2 ...]');
-            return runReqSyncToRepo([n, ...process.argv.slice(3)].join(' '));
+            return runReqSyncToRepo([n, ...process.argv.slice(4)].join(' '));
         },
         description: 'Updates remote issue from local REQ',
         argRequired: true,
@@ -101,10 +101,9 @@ const commands = {
     },
     'req:close': {
         handler: (n) => {
-            const num = parseInt(n || '', 10);
-            if (isNaN(num))
-                throw new Error('req:close <N>');
-            return runReqClose(num);
+            if (!n)
+                throw new Error('req:close <N> [N2 N3 ...]');
+            return runReqClose([n, ...process.argv.slice(4)].join(' '));
         },
         description: 'Closes a requirement',
         argRequired: true,

package/dist/tasks/req.d.ts

@@ -6,4 +6,4 @@ export declare function runReqCompare(number?: number): Promise<void>;
 export declare function runReqSyncFromRepo(arg?: string): Promise<void>;
 export declare function runReqSyncToRepo(arg?: string): Promise<void>;
 export declare function runReqCreate(title?: string, typeArg?: string): Promise<void>;
-export declare function runReqClose(number: number): Promise<void>;
+export declare function runReqClose(arg?: string): Promise<void>;

package/dist/tasks/req.js

@@ -67,10 +67,12 @@ function parseReqFile(content, number) {
     let title = '';
     let issueType = 'R';
     const firstLine = lines[0] || '';
-    const headerMatch = firstLine.match(/^#\s*([RFIP])-#(\d+):?\s*(.*)/);
+    const headerMatch = firstLine.match(/^#\s*([RFIPE])-#(\d+):?\s*(.*)/);
     if (headerMatch) {
         issueType = headerMatch[1];
         title = headerMatch[3]?.trim().replace(/`/g, '') || '';
+        // Strip any embedded old prefix (e.g., "I-#1 Titulo" → "Titulo")
+        title = title.replace(/^[RFIPE]-#\d+:\s*/, '');
     }
     // ── Fallback: old YAML frontmatter ───────────────
     if (!title && lines[0]?.trim() === '---') {
@@ -80,7 +82,7 @@ function parseReqFile(content, number) {
                 const titleMatch = lines[i].match(/^title:\s*"?(.+?)"?\s*$/);
                 if (titleMatch) {
                     title = titleMatch[1]
-                        .replace(/^[RFIP]-#\d+:\s*/, '')
+                        .replace(/^[RFIPE]-#\d+:\s*/, '')
                         .replace(/`/g, '')
                         .trim();
                 }
@@ -89,7 +91,7 @@ function parseReqFile(content, number) {
     }
     // ── Fallback: old format first-line ──────────────
     if (!title) {
-        const oldMatch = firstLine.match(/^(?:#\s*)?[RFIP]-#\d+\s*(.*)/);
+        const oldMatch = firstLine.match(/^(?:#\s*)?[RFIPE]-#\d+\s*(.*)/);
         if (oldMatch) {
             title = (oldMatch[1] || '').trim().replace(/`/g, '');
         }
@@ -99,7 +101,7 @@ function parseReqFile(content, number) {
         for (const line of lines) {
             const m = line.match(/^title:\s*"?(.+?)"?\s*$/);
             if (m) {
-                title = m[1].replace(/^[RFIP]-#\d+:\s*/, '').replace(/`/g, '').trim();
+                title = m[1].replace(/^[RFIPE]-#\d+:\s*/, '').replace(/`/g, '').trim();
                 break;
             }
         }
@@ -526,9 +528,10 @@ async function runCompareAll(config, token) {
         const reqDir = await findReqDir();
         const localReqs = await loadAllReqs(reqDir);
         const localNums = new Set(localReqs.map(r => r.number));
-        // Pre-load local mtimes and content fingerprints
+        // Pre-load local mtimes, content fingerprints, and title format
         const localMtimeMap = new Map();
         const localFingerprintMap = new Map();
+        const localHasPrefix = new Map();
         for (const r of localReqs) {
             try {
                 const s = await stat(join(reqDir, `${r.number}.md`));
@@ -536,6 +539,8 @@ async function runCompareAll(config, token) {
                 const content = await readFile(join(reqDir, `${r.number}.md`), 'utf-8');
                 const normalized = content.trim().replace(/\r/g, '').replace(/\n{3,}/g, '\n\n');
                 localFingerprintMap.set(r.number, `${normalized.length}:${normalized.slice(0, 200)}`);
+                // Check if local REQ has proper R-#N: / F-#N: / I-#N: / P-#N: title format
+                localHasPrefix.set(r.number, /^#\s*[RFIPE]-#\d+:/.test(content));
             }
             catch {
                 // file inaccessible
@@ -570,8 +575,9 @@ async function runCompareAll(config, token) {
         const shown = new Set();
         for (const issue of issues) {
             const hasLocal = localNums.has(issue.iid);
-            const typeMatch = issue.title.match(/^([RFIP])-#[?\d]/);
-            const issueType = typeMatch ? typeMatch[1] : 'R';
+            const localReq = localReqs.find(r => r.number === issue.iid);
+            const typeMatch = issue.title.match(/^([RFIPE])-#[?\d]/);
+            const issueType = (hasLocal && localReq) ? localReq.type : (typeMatch ? typeMatch[1] : 'R');
             shown.add(issue.iid);
             let sync = '—';
             if (hasLocal && issue.updated_at) {
@@ -597,10 +603,16 @@ async function runCompareAll(config, token) {
                         else {
                             sync = 'loc+';
                         }
+                        // Title format check: if local has R-#N: prefix and remote doesn't, local wins
+                        if (sync === 'sync' && localHasPrefix.get(issue.iid)) {
+                            const remoteHasPrefix = /^[RFIPE]-#[?\d]+:/.test(issue.title);
+                            if (!remoteHasPrefix)
+                                sync = 'loc+';
+                        }
                     }
                 }
             }
-            const displayTitle = issue.title.replace(/^[RFIP]-#[?\d]+:?\s*/, '').replace(/`/g, '').trim();
+            const displayTitle = issue.title.replace(/^[RFIPE]-#[?\d]+:?\s*/, '').replace(/`/g, '').trim();
             console.log(`  ${String(issue.iid).padEnd(4)} │ ${issueType.padEnd(4)} │ ${displayTitle.slice(0, titleW).padEnd(titleW)} │ ${hasLocal ? '  ✓' : '  ✗'}   │ ${shortState(issue.state).padEnd(stateW)} │ ${sync}`);
         }
         // Show local-only REQs (not in remote open issues)
@@ -614,7 +626,7 @@ async function runCompareAll(config, token) {
                 try {
                     const issue = await fetchRemoteIssue(config, r.number, token);
                     remoteState = issue.state || 'unknown';
-                    remoteTitle = (issue.title || '').replace(/^[RFIP]-#[?\d]+:?\s*/, '').trim();
+                    remoteTitle = (issue.title || '').replace(/^[RFIPE]-#[?\d]+:?\s*/, '').trim();
                     remoteUpdatedAt = issue.updated_at || issue.updatedAt || null;
                     shown.add(r.number);
                     if (remoteUpdatedAt) {
@@ -646,6 +658,12 @@ async function runCompareAll(config, token) {
                 catch {
                     // Issue doesn't exist remotely
                 }
+                // Title format check: if local has prefix and remote doesn't, local wins
+                if (sync === 'sync' && localHasPrefix.get(r.number)) {
+                    const remoteHasPrefix = /^[RFIPE]-#[?\d]+:/.test(remoteTitle || '');
+                    if (!remoteHasPrefix && remoteState !== '—')
+                        sync = 'loc+';
+                }
                 const title = remoteTitle || r.title;
                 console.log(`  ${String(r.number).padEnd(4)} │ ${r.type.padEnd(4)} │ ${title.slice(0, titleW).padEnd(titleW)} │   ✓   │ ${shortState(remoteState).padEnd(stateW)} │ ${sync}`);
             }
@@ -693,7 +711,7 @@ export async function runReqCompare(number) {
     try {
         const issue = await fetchRemoteIssue(config, number, token);
         const remoteDesc = issue.description || issue.body || '';
-        const remoteTitle = (issue.title || '').replace(/^[RFIP]-#\d+:\s*/, '').trim();
+        const remoteTitle = (issue.title || '').replace(/^[RFIPE]-#\d+:\s*/, '').trim();
         const remoteState = issue.state || 'unknown';
         const remoteUpdatedAt = issue.updated_at || issue.updatedAt || null;
         // ── Header info ───────────────────────────────
@@ -788,14 +806,14 @@ async function syncFromRepoOne(number) {
     try {
         const issue = await fetchRemoteIssue(config, number, token);
         const rawBody = issue.description || issue.body || '';
-        const issueTitle = (issue.title || '').replace(/^[RFIP]-#\d+:\s*/, '').replace(/`/g, '').trim();
-        const issueTypeMatch = (issue.title || '').match(/^([RFIP])-#\d+:/);
+        const issueTitle = (issue.title || '').replace(/^[RFIPE]-#\d+:\s*/, '').replace(/`/g, '').trim();
+        const issueTypeMatch = (issue.title || '').match(/^([RFIPE])-#\d+:/);
         const issueType = issueTypeMatch ? issueTypeMatch[1] : 'R';
         const status = issue.state === 'closed' ? 'Cerrado' : 'Pendiente';
         const createdAt = (issue.created_at || issue.createdAt || new Date().toISOString()).slice(0, 10);
         // Build content: use raw body if it already has our header, otherwise wrap
         let content;
-        if (rawBody.trimStart().match(/^(?:---|#\s*[RFIP]-#)/)) {
+        if (rawBody.trimStart().match(/^(?:---|#\s*[RFIPE]-#)/)) {
             content = rawBody;
         }
         else {
@@ -940,21 +958,22 @@ async function syncToRepoOne(number) {
         // Try to update existing issue
         try {
             const meta = parseReqFile(localContent, number);
-            await updateRemoteIssue(config, number, meta.title, localContent, token);
+            const fullTitle = `${meta.type}-#${number}: ${meta.title}`;
+            await updateRemoteIssue(config, number, fullTitle, localContent, token);
             success(`✅ Issue remoto #${number} actualizado desde REQ/${number}.md.`);
         }
         catch (updateErr) {
             if (updateErr.message?.includes('404')) {
                 // Issue doesn't exist — create it
                 const meta = parseReqFile(localContent, number);
-                const newIssue = await createRemoteIssue(config, meta.title, localContent, token);
+                const newIssue = await createRemoteIssue(config, `${meta.type}-#?: ${meta.title}`, localContent, token);
                 const remoteIid = newIssue.iid || newIssue.number;
                 info(`✅ Issue creado en remoto como #${remoteIid}.`);
                 if (remoteIid && remoteIid !== number) {
                     // Rename local file to match remote IID
                     const newPath = join(reqDir, `${remoteIid}.md`);
                     const { rename } = await import('node:fs/promises');
-                    let newContent = localContent.replace(new RegExp(`([RFIP])-#${number}\\b`, 'g'), `$1-#${remoteIid}`);
+                    let newContent = localContent.replace(new RegExp(`([RFIPE])-#${number}\\b`, 'g'), `$1-#${remoteIid}`);
                     await writeFile(newPath, newContent);
                     const { unlink } = await import('node:fs/promises');
                     await unlink(filePath);
@@ -1127,7 +1146,19 @@ async function closeRemoteIssue(config, iid, title, token) {
     }
     throw new Error('Configuración de repositorio incompleta');
 }
-export async function runReqClose(number) {
+export async function runReqClose(arg) {
+    const numbers = parseNumberList(arg);
+    if (numbers.length === 0) {
+        problem('Uso: req:close <N> [N2 N3 ...]');
+        return;
+    }
+    for (const number of numbers) {
+        info(`Cerrando #${number}...`);
+        await closeOne(number);
+    }
+    success(`✅ ${numbers.length} requerimiento(s) cerrado(s).`);
+}
+async function closeOne(number) {
     const reqDir = await findReqDir();
     const filePath = join(reqDir, `${number}.md`);
     if (!existsSync(filePath)) {
@@ -1148,10 +1179,11 @@ export async function runReqClose(number) {
         // 1. Sync local → remote
         const localContent = await readFile(filePath, 'utf-8');
         const meta = parseReqFile(localContent, number);
-        await updateRemoteIssue(config, number, meta.title, localContent, token);
+        const fullTitle = `${meta.type}-#${number}: ${meta.title}`;
+        await updateRemoteIssue(config, number, fullTitle, localContent, token);
         info('✓ Sincronizado local → remoto.');
         // 2. Close remote issue
-        await closeRemoteIssue(config, number, meta.title, token);
+        await closeRemoteIssue(config, number, fullTitle, token);
         info('✓ Issue cerrado en el repositorio remoto.');
         // 3. Delete local REQ
         const { unlink } = await import('node:fs/promises');
@@ -1159,6 +1191,6 @@ export async function runReqClose(number) {
         success(`✅ REQ/${number}.md cerrado y eliminado.`);
     }
     catch (err) {
-        problem(`Error al cerrar: ${err.message}`);
+        problem(`Error al cerrar #${number}: ${err.message}`);
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pasosdejesus/mpdj",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Extensión Pasos de Jesús para m CLI — marco cristiano, credenciales, requerimientos",
   "author": "",
   "license": "ISC",
@@ -55,7 +55,7 @@
     "REQ"
   ],
   "dependencies": {
-    "@pasosdejesus/m": "0.15.0"
+    "@pasosdejesus/m": "0.16.1"
   },
   "peerDependencies": {
     "vitest": "^2.1.9"