bmad-dashboard 1.0.10 → 1.0.12

package/README.md

@@ -159,9 +159,9 @@ On every **push to `main`**, GitHub Actions:
 1. Runs tests and build in the `dashboard/` directory.
 2. If tests pass, bumps the **patch** version (e.g. `1.0.9` → `1.0.10`), publishes to npm via **Trusted Publishers (OIDC)**, and commits the version bump back to `main` (with message `chore(release): 1.0.10 [skip ci]` so the release commit does not trigger another publish).
 
-**Setup (one-time):** Use [npm Trusted Publishers](https://docs.npmjs.com/trusted-publishers). Your **npm account must be linked to your GitHub account** (npm → profile/Account → link GitHub). Then on [npmjs.com](https://www.npmjs.com/) go to **Packages** → **bmad-dashboard** → **Settings** → **Trusted publishing**, choose **GitHub Actions**, set **Workflow filename** to `release.yml`, and save. The workflow file is `.github/workflows/release.yml`.
+**Setup (one-time), either:** **Option A — Granular token (recommended):** Create a [Granular access token](https://docs.npmjs.com/about-access-tokens) on npm with **read and write** access for this package and **Bypass 2FA** enabled. Add it as repo secret **`NPM_TOKEN`** under **Settings** → **Secrets and variables** → **Actions**. **Option B — Trusted Publishers (OIDC):** Link npm to GitHub and set Trusted publishing for this package (workflow `release.yml`) on npmjs.com; leave `NPM_TOKEN` unset. The workflow file is `.github/workflows/release.yml`.
 
-**If you get 404 on publish:** The package must exist on npm before you can add a Trusted Publisher (you configure it in the package’s Settings). If [npmjs.com/package/bmad-dashboard](https://www.npmjs.com/package/bmad-dashboard) returns 404, do **one manual publish** from your machine: `cd dashboard && npm login && npm publish --access public` (enter OTP when prompted). After that, add the Trusted Publisher for `release.yml` in the package settings; then CI can publish future versions.
+**If OIDC gives 404 on publish:** The package must exist on npm before you can add a Trusted Publisher (you configure it in the package’s Settings). If [npmjs.com/package/bmad-dashboard](https://www.npmjs.com/package/bmad-dashboard) returns 404, do **one manual publish** from your machine: `cd dashboard && npm login && npm publish --access public` (enter OTP when prompted). After that, add the Trusted Publisher for `release.yml` in the package settings; then CI can publish future versions.
 
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bmad-dashboard",
-  "version": "1.0.10",
+  "version": "1.0.12",
   "description": "Local dashboard to view BMAD project state (phase, artifacts, stories, sprint) in real time",
   "type": "module",
   "bin": {

package/server/artifactDetails.js

@@ -11,20 +11,23 @@ import {
 
 const ARCH_STEP_IDS = [1, 2, 3, 4, 5, 6, 7, 8]
 
-/** Match a single AC line: bold type (Given/When/Then/And) or plain type, then text */
-const AC_LINE_BOLD = /^-\s*\*\*(Given|When|Then|And)\s*:?\s*(?:\*\*\s*)?(.+)$/
-const AC_LINE_PLAIN = /^-\s*(Given|When|Then|And)\s*:?\s*(.+)$/
+/** Match a single AC line: bold type (Given/When/Then/And) or plain type, then text. Support both - and * bullets. */
+const AC_LINE_BOLD = /^[-*]\s*\*\*(Given|When|Then|And)\s*:?\s*(?:\*\*\s*)?(.+)$/
+const AC_LINE_PLAIN = /^[-*]\s*(Given|When|Then|And)\s*:?\s*(.+)$/
 
 function parseStoryBody(body, num, title, statusByKey) {
   const id = num.replace('.', '-')
-  const rest = (body || '').trim()
+  const rest = (body || '').replace(/\r\n/g, '\n').replace(/\r/g, '\n').trim()
   let userStory = ''
   const acceptanceCriteria = []
-  // Multiple possible markers for start of acceptance criteria block
+  // Multiple possible markers for start of acceptance criteria block (case-insensitive)
   const acMarkers = [
     /\*\*Acceptance Criteria\s*:?\s*\*\*/i,
+    /\*\*Acceptance criteria\s*:?\s*\*\*/i,
     /^Acceptance Criteria\s*:?\s*$/im,
+    /^Acceptance criteria\s*:?\s*$/im,
     /^#+\s*Acceptance Criteria\s*:?\s*$/im,
+    /^#+\s*Acceptance criteria\s*:?\s*$/im,
   ]
   let acIndex = -1
   let acBlockStart = 0
@@ -44,10 +47,21 @@ function parseStoryBody(body, num, title, statusByKey) {
     const acBlock = rest.slice(acBlockStart).trim()
     const acLines = acBlock.split(/\n/).map((line) => line.trim()).filter(Boolean)
     for (const line of acLines) {
+      if (line.startsWith('###')) break // next story or heading
       let m = line.match(AC_LINE_BOLD)
       if (!m) m = line.match(AC_LINE_PLAIN)
       if (m) acceptanceCriteria.push({ type: m[1], text: m[2].trim() })
-      else if (line.startsWith('- ') && line.length > 2) acceptanceCriteria.push({ type: null, text: line.slice(2).trim() })
+      else if ((line.startsWith('- ') || line.startsWith('* ')) && line.length > 2)
+        acceptanceCriteria.push({ type: null, text: line.slice(2).trim() })
+    }
+    // If we found AC header but no criteria matched, take any list items in the block
+    if (acceptanceCriteria.length === 0 && acBlock.length > 0) {
+      for (const line of acLines) {
+        if (line.startsWith('###')) break
+        const bullet = line.match(/^[-*]\s+(.+)$/) || line.match(/^\d+\.\s+(.+)$/)
+        if (bullet && bullet[1].trim().length > 0)
+          acceptanceCriteria.push({ type: null, text: bullet[1].trim() })
+      }
     }
   }
   // Fallback: no AC header but body has bullet list that looks like criteria (Given/When/Then/And)
@@ -58,7 +72,8 @@ function parseStoryBody(body, num, title, statusByKey) {
       let m = t.match(AC_LINE_BOLD)
       if (!m) m = t.match(AC_LINE_PLAIN)
       if (m) acceptanceCriteria.push({ type: m[1], text: m[2].trim() })
-      else if (t.startsWith('- ') && t.length > 2) acceptanceCriteria.push({ type: null, text: t.slice(2).trim() })
+      else if ((t.startsWith('- ') || t.startsWith('* ')) && t.length > 2)
+        acceptanceCriteria.push({ type: null, text: t.slice(2).trim() })
     }
   }
   if (!userStory && rest) {