@programinglive/dev-workflow-mcp-server 1.13.2 → 1.13.4

package/exec.js

@@ -52,8 +52,16 @@ function samePath(a, b) {
  * Menentukan direktori repo proyek tempat git boleh dijalankan.
  *
  * Urutan: eksplisit -> WORKFLOW_PROJECT_DIR/PROJECT_DIR -> kandidat.
- * Repo paket MCP selalu dilewati; kalau tidak ada pilihan lain, lebih baik
- * gagal dengan pesan jelas daripada commit ke repo yang salah.
+ *
+ * Repo paket MCP sengaja tidak dipilih sebagai cadangan: dulu server pernah
+ * menjalankan git di repo-nya sendiri saat cwd proyek gagal diresolusi, lalu
+ * melaporkan "No changes detected" sementara perubahan proyek tidak pernah
+ * ter-commit.
+ *
+ * Tetapi kandidat yang MEMANG berada di dalam repo paket berarti pemanggil
+ * sengaja bekerja di sana - misalnya saat mengembangkan server MCP ini sendiri.
+ * Membuangnya tanpa syarat membuat `commit_and_push` mustahil dipakai di repo
+ * paket, jadi kandidat seperti itu diterima. Repo proyek lain tetap menang.
  */
 export function resolveWorkingDirectory({ explicit = "", env = process.env, candidates = [] } = {}) {
   if (explicit) {
@@ -74,14 +82,22 @@ export function resolveWorkingDirectory({ explicit = "", env = process.env, cand
   }
 
   const found = candidates.map((candidate) => findRepoRoot(candidate)).filter(Boolean);
-  const usable = found.find((dir) => !samePath(dir, PACKAGE_DIR));
-  if (usable) {
-    return usable;
+
+  const projectRepo = found.find((dir) => !samePath(dir, PACKAGE_DIR));
+  if (projectRepo) {
+    return projectRepo;
+  }
+
+  // Hanya repo paket yang ditemukan. Itu hanya mungkin bila salah satu kandidat
+  // benar-benar berada di dalamnya, jadi pemanggil memilih repo ini dengan sadar.
+  if (found.some((dir) => samePath(dir, PACKAGE_DIR))) {
+    return path.resolve(PACKAGE_DIR);
   }
 
   throw new Error(
-    "Tidak bisa menentukan repo proyek: satu-satunya kandidat adalah repo paket server MCP. " +
-    "Set WORKFLOW_PROJECT_DIR ke direktori proyek, atau jalankan server MCP dari direktori proyek.",
+    "Tidak bisa menentukan repo proyek: tidak ada repository git pada " +
+    `${candidates.filter(Boolean).map((candidate) => `"${candidate}"`).join(", ") || "direktori kerja"}. ` +
+    "Set WORKFLOW_PROJECT_DIR ke direktori proyek, atau jalankan server MCP dari dalam direktori proyek.",
   );
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@programinglive/dev-workflow-mcp-server",
-  "version": "1.13.2",
+  "version": "1.13.4",
   "description": "MCP server to enforce development workflow discipline",
   "homepage": "https://devworkflow.programinglive.com/",
   "mcpName": "io.github.programinglive/dev-workflow-mcp-server",

package/scripts/release.cjs

@@ -40,6 +40,40 @@ function getCliArguments(argv = process.argv) {
     return { releaseType: 'patch', extraArgs: [] };
   }
 
+  // `--release-as` selalu diambil sebagai tipe rilis dan DIBUANG dari extraArgs.
+  //
+  // Kalau dibiarkan, argumen itu sampai ke standard-version dua kali: sekali
+  // dari releaseType, sekali dari extraArgs. yargs lalu mengubahnya menjadi
+  // array, dan `semver.inc("1.13.2", ["patch","patch"])` mengembalikan null -
+  // rilis menulis `"version": null` dan tag `vnull`.
+  const releaseAsIndex = rawArgs.indexOf('--release-as');
+  if (releaseAsIndex !== -1) {
+    const value = rawArgs[releaseAsIndex + 1];
+    if (!value || value.startsWith('-')) {
+      throw new Error(
+        '--release-as memerlukan nilai tipe rilis (major, minor, patch, atau versi semver).'
+      );
+    }
+    return {
+      releaseType: value,
+      extraArgs: [...rawArgs.slice(0, releaseAsIndex), ...rawArgs.slice(releaseAsIndex + 2)]
+    };
+  }
+
+  const inline = rawArgs.find((arg) => arg.startsWith('--release-as='));
+  if (inline) {
+    const value = inline.slice('--release-as='.length);
+    if (!value) {
+      throw new Error(
+        '--release-as memerlukan nilai tipe rilis (major, minor, patch, atau versi semver).'
+      );
+    }
+    return {
+      releaseType: value,
+      extraArgs: rawArgs.filter((arg) => arg !== inline)
+    };
+  }
+
   if (rawArgs[0].startsWith('-')) {
     return { releaseType: 'patch', extraArgs: rawArgs };
   }
@@ -62,17 +96,43 @@ function getNpmRunArgument(env = process.env) {
   return undefined;
 }
 
+/**
+ * Membuang setiap `--release-as` dari argumen tambahan.
+ *
+ * Pertahanan lapis kedua: tipe rilis sudah ditentukan lewat `releaseType`, jadi
+ * salinan apa pun di `extraArgs` hanya akan menggandakan opsi dan membuat yargs
+ * memberinya array - yang berakhir pada `semver.inc(...) === null`.
+ */
+function stripReleaseAs(extraArgs) {
+  if (!Array.isArray(extraArgs)) {
+    return [];
+  }
+
+  const kept = [];
+  for (let index = 0; index < extraArgs.length; index += 1) {
+    const arg = extraArgs[index];
+    if (arg === '--release-as') {
+      index += 1; // lewati nilainya juga
+      continue;
+    }
+    if (typeof arg === 'string' && arg.startsWith('--release-as=')) {
+      continue;
+    }
+    kept.push(arg);
+  }
+  return kept;
+}
+
 function buildStandardVersionArgs({ releaseType, extraArgs }) {
   const args = [];
-  
+  const remainingArgs = stripReleaseAs(extraArgs);
+
   // Handle --first-release flag
-  const isFirstRelease = Array.isArray(extraArgs) && extraArgs.includes('--first-release');
+  const isFirstRelease = remainingArgs.includes('--first-release');
   if (isFirstRelease) {
     args.push('--release-as', '0.0.1');
-    // Remove --first-release from extraArgs to avoid passing it to standard-version
-    extraArgs = extraArgs.filter(arg => arg !== '--first-release');
   } else if (releaseType) {
-    const normalized = releaseType.trim();
+    const normalized = String(releaseType).trim();
     const isValid = VALID_RELEASE_TYPES.has(normalized) || SEMVER_REGEX.test(normalized);
     if (!isValid) {
       const allowed = Array.from(VALID_RELEASE_TYPES).join(', ');
@@ -81,8 +141,8 @@ function buildStandardVersionArgs({ releaseType, extraArgs }) {
     args.push('--release-as', normalized);
   }
 
-  if (Array.isArray(extraArgs) && extraArgs.length > 0) {
-    args.push(...extraArgs);
+  if (remainingArgs.length > 0) {
+    args.push(...remainingArgs.filter((arg) => arg !== '--first-release'));
   }
 
   return args;
@@ -98,6 +158,52 @@ function loadPackageJson(cwd = process.cwd()) {
   }
 }
 
+/**
+ * Memastikan package.json berisi versi semver yang sah SETELAH rilis berjalan.
+ *
+ * MENGAPA ADA
+ * -----------
+ * 30 Oktober 2026 `perform_release` menulis `"version": null` ke package.json
+ * dan membuat tag `vnull`, lalu mem-push keduanya. Penyebabnya argumen
+ * `--release-as` terkirim dua kali sehingga `semver.inc()` mengembalikan null.
+ *
+ * `SEMVER_REGEX` sudah ada saat itu, tetapi hanya dipakai memvalidasi tipe
+ * rilis MASUKAN. Hasil rilis tidak pernah diperiksa, jadi kerusakan baru
+ * terlihat setelah tag keluar dari mesin.
+ *
+ * Fungsi ini menutup celah itu: standard-version dianggap belum berhasil
+ * sebelum package.json terbukti berisi semver yang sah.
+ */
+function verifyReleaseVersion({ cwd = process.cwd(), readFileSync = fs.readFileSync } = {}) {
+  const packageJsonPath = path.join(cwd, 'package.json');
+
+  let contents;
+  try {
+    contents = readFileSync(packageJsonPath, 'utf8');
+  } catch (error) {
+    throw new Error(`Tidak bisa membaca ${packageJsonPath}; hasil rilis tidak dapat diverifikasi.`);
+  }
+
+  let parsed;
+  try {
+    parsed = JSON.parse(contents);
+  } catch (error) {
+    throw new Error(`${packageJsonPath} bukan JSON yang sah setelah rilis; hasilnya tidak dapat diverifikasi.`);
+  }
+
+  const version = parsed?.version;
+
+  if (typeof version !== 'string' || !SEMVER_REGEX.test(version.trim())) {
+    throw new Error(
+      `Rilis gagal: package.json berisi version ${JSON.stringify(version)}, bukan semver yang sah. ` +
+      'Tag tidak boleh di-push. Periksa argumen --release-as, lalu pulihkan dengan ' +
+      `"git tag -d v${String(version)}" dan "git reset --hard HEAD~1" bila perlu.`
+    );
+  }
+
+  return version.trim();
+}
+
 function detectPackageManager({ env = process.env, cwd = process.cwd() } = {}) {
   const userAgent = env.npm_config_user_agent || '';
   if (userAgent.startsWith('pnpm/')) return 'pnpm';
@@ -189,6 +295,18 @@ function runRelease({
     return { ...releaseResult, status: releaseResult?.status || 1 };
   }
 
+  // Rilis hanya dianggap berhasil setelah package.json terbukti berisi semver
+  // yang sah. Tanpa pemeriksaan ini, `version: null` dan tag `vnull` lolos
+  // sampai di-push (kejadian 30 Oktober 2026).
+  let releasedVersion;
+  try {
+    releasedVersion = verifyReleaseVersion({ cwd });
+  } catch (error) {
+    safeLog('error', error.message);
+    return { status: 1, error };
+  }
+  safeLog('info', `✅ Released ${releasedVersion}`);
+
   const releaseNotesPath = dependencies.releaseNotesPath || path.join('docs', 'release-notes', 'RELEASE_NOTES.md');
   const updateNotes = dependencies.updateReleaseNotes || ((options = {}) => updateReleaseNotes({ rootDir: cwd, ...options }));
 
@@ -224,6 +342,7 @@ module.exports = {
   getNpmRunArgument,
   buildStandardVersionArgs,
   loadPackageJson,
+  verifyReleaseVersion,
   detectPackageManager,
   runProjectTests,
   runRelease,