imdone-core 2.1.4 → 2.1.6

package/.github/workflows/release.yml

@@ -0,0 +1,70 @@
+name: Release
+
+on:
+  push:
+    branches:
+      - master
+
+permissions:
+  contents: write
+
+concurrency:
+  group: release-${{ github.ref }}
+  cancel-in-progress: false
+
+jobs:
+  release:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Use Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 22.x
+          cache: npm
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Test
+        run: npm test
+
+      - name: Read package version
+        id: package
+        run: |
+          VERSION="$(node -p "require('./package.json').version")"
+          echo "version=$VERSION" >> "$GITHUB_OUTPUT"
+          echo "tag=v$VERSION" >> "$GITHUB_OUTPUT"
+
+      - name: Check for existing release
+        id: release
+        env:
+          GH_TOKEN: ${{ github.token }}
+          TAG: ${{ steps.package.outputs.tag }}
+        run: |
+          if gh release view "$TAG" >/dev/null 2>&1; then
+            echo "exists=true" >> "$GITHUB_OUTPUT"
+          else
+            echo "exists=false" >> "$GITHUB_OUTPUT"
+          fi
+
+      - name: Create GitHub release
+        if: steps.release.outputs.exists == 'false'
+        env:
+          GH_TOKEN: ${{ github.token }}
+          TAG: ${{ steps.package.outputs.tag }}
+          VERSION: ${{ steps.package.outputs.version }}
+        run: |
+          gh release create "$TAG" \
+            --target "$GITHUB_SHA" \
+            --title "$VERSION" \
+            --generate-notes
+
+      - name: Skip existing release
+        if: steps.release.outputs.exists == 'true'
+        run: echo "Release ${{ steps.package.outputs.tag }} already exists; skipping."

package/.imdone-cli.yml

@@ -1,2 +1,3 @@
 backlogDir: backlog
-provider: jira
+providers:
+  - github

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## 2.1.5
+
+- Create GitHub releases automatically from the npm package version when changes merge to `master`.
+- Improve test coverage for environment detection, board actions, views, and view retrieval.
+- Add enforced coverage thresholds to prevent coverage regressions.
+- Update `monquery` to `0.2.2`.
+- Update the imdone CLI project config to use the GitHub provider format.
+
+## 2.1.4
+
+- Remove the deprecated `node-fetch` dependency from plugin downloads so installs no longer pull in `node-domexception`.
+
 ## 2.1.3
 
 - Update imdone-api dependency

package/lib/file.js

@@ -535,16 +535,38 @@ export class File extends Emitter {
     }
   }
   extractTasks (config) {
+    // INSTRUMENTATION: Track timing for hang investigation
+    const startTime = Date.now()
+    log(`[INSTRUMENT] File.extractTasks START for ${this.path}`)
+
     this.tasks = []
     if (this.isMarkDownFile()) {
       const data = this.parseFrontMatter(config)
-      if (data.imdone_ignore || data['kanban-plugin']) return this
+      if (data.imdone_ignore || data['kanban-plugin']) {
+        log(`[INSTRUMENT] File.extractTasks END (ignored) - ${Date.now() - startTime}ms`)
+        return this
+      }
     }
+
+    const frontMatterTime = Date.now()
+    log(`[INSTRUMENT] Front matter check took ${frontMatterTime - startTime}ms`)
+
     if (this.isCodeFile()) {
+      log(`[INSTRUMENT] About to extractTasksInCodeFile`)
       this.extractTasksInCodeFile(config)
+      log(`[INSTRUMENT] extractTasksInCodeFile took ${Date.now() - frontMatterTime}ms`)
     } else {
+      log(`[INSTRUMENT] About to extractTasksInNonCodeFile`)
       this.extractTasksInNonCodeFile(config)
+      log(`[INSTRUMENT] extractTasksInNonCodeFile took ${Date.now() - frontMatterTime}ms`)
+    }
+
+    const totalTime = Date.now() - startTime
+    if (totalTime > 1000) {
+      log(`[INSTRUMENT] ⚠️  SLOW File.extractTasks: ${this.path} took ${totalTime}ms`)
     }
+    log(`[INSTRUMENT] File.extractTasks END - total ${totalTime}ms`)
+
     return this
   }
 
@@ -560,20 +582,62 @@ export class File extends Emitter {
   // order:40
   // -->
   extractTasksInCodeFile (config) {
+    // INSTRUMENTATION: Track regex iterations
+    const startTime = Date.now()
     var self = this
     var commentRegex = this.getCodeCommentRegex()
+    log(`[INSTRUMENT] getCodeCommentRegex took ${Date.now() - startTime}ms`)
+
     var result
+    var iterationCount = 0
+    const contentLength = self.getContent()?.length || 0
+    log(`[INSTRUMENT] Starting regex loop, content length: ${contentLength}`)
+
     while ((result = commentRegex.exec(self.getContent())) !== null) {
+      iterationCount++
+      if (iterationCount % 100 === 0) {
+        log(`[INSTRUMENT] Regex iteration ${iterationCount}, elapsed: ${Date.now() - startTime}ms`)
+      }
+
       var comment = result[0]
       var commentStart = result.index
       self.extractCodeStyleTasks(config, commentStart, comment)
+
+      // Safety check for infinite loop
+      if (iterationCount > 10000) {
+        log(`[INSTRUMENT] ⚠️  WARNING: Exceeded 10000 regex iterations, breaking loop`)
+        break
+      }
     }
+
+    const totalTime = Date.now() - startTime
+    log(`[INSTRUMENT] extractTasksInCodeFile END - ${iterationCount} iterations, ${totalTime}ms`)
   }
 
   extractTasksInNonCodeFile (config) {
-    this.extractHashStyleTasks(config, 0, this.getContent())
-    this.extractLinkStyleTasks(config, 0, this.getContent())
-    this.extractCheckboxTasks(config, 0, this.getContent())
+    // INSTRUMENTATION: Track each extraction method
+    const startTime = Date.now()
+    const content = this.getContent()
+    const contentLength = content?.length || 0
+    log(`[INSTRUMENT] extractTasksInNonCodeFile START, content length: ${contentLength}`)
+
+    log(`[INSTRUMENT] About to extractHashStyleTasks`)
+    this.extractHashStyleTasks(config, 0, content)
+    const hashTime = Date.now()
+    log(`[INSTRUMENT] extractHashStyleTasks took ${hashTime - startTime}ms`)
+
+    log(`[INSTRUMENT] About to extractLinkStyleTasks`)
+    this.extractLinkStyleTasks(config, 0, content)
+    const linkTime = Date.now()
+    log(`[INSTRUMENT] extractLinkStyleTasks took ${linkTime - hashTime}ms`)
+
+    log(`[INSTRUMENT] About to extractCheckboxTasks`)
+    this.extractCheckboxTasks(config, 0, content)
+    const checkboxTime = Date.now()
+    log(`[INSTRUMENT] extractCheckboxTasks took ${checkboxTime - linkTime}ms`)
+
+    const totalTime = Date.now() - startTime
+    log(`[INSTRUMENT] extractTasksInNonCodeFile END - total ${totalTime}ms`)
   }
 
   deleteBlankLine (lineNo) {

package/lib/repository.js

@@ -755,20 +755,54 @@ export class Repository extends Emitter {
   // order:-100
   // -->
   async extractTasks (file) {
+    // INSTRUMENTATION: Track timing for hang investigation
+    const extractStartTime = Date.now()
+    logger.debug(`[INSTRUMENT] extractTasks START for ${file.path}`)
+
     if (file.content === null) {
       await this.readFileContent(file)
     }
+
+    const readTime = Date.now()
+    logger.debug(`[INSTRUMENT] readFileContent took ${readTime - extractStartTime}ms`)
+
     file.on('task.found', this.taskFoundListener)
     file.on('task.modified', this.taskModifiedListener)
     const fileContent = file.content
+
+    logger.debug(`[INSTRUMENT] About to call extractAndTransformTasks, content length: ${fileContent?.length || 0}`)
     file.extractAndTransformTasks(this.getConfig())
+
+    const transformTime = Date.now()
+    logger.debug(`[INSTRUMENT] extractAndTransformTasks took ${transformTime - readTime}ms`)
+
     if (!file.isModified() || fileContent === file.content) {
       this.resetFile(file)
+      const totalTime = Date.now() - extractStartTime
+      logger.debug(`[INSTRUMENT] extractTasks END (no changes) - total ${totalTime}ms`)
       return file
     }
+
+    logger.debug(`[INSTRUMENT] About to call file.extractTasks`)
     file.extractTasks(this.getConfig())
-    if (file.modified) await this.writeFile(file)
+
+    const extractTime = Date.now()
+    logger.debug(`[INSTRUMENT] file.extractTasks took ${extractTime - transformTime}ms`)
+
+    if (file.modified) {
+      logger.debug(`[INSTRUMENT] About to writeFile`)
+      await this.writeFile(file)
+      logger.debug(`[INSTRUMENT] writeFile took ${Date.now() - extractTime}ms`)
+    }
+
     this.resetFile(file)
+
+    const totalTime = Date.now() - extractStartTime
+    if (totalTime > 1000) {
+      logger.warn(`[INSTRUMENT] ⚠️  SLOW extractTasks: ${file.path} took ${totalTime}ms`)
+    }
+    logger.debug(`[INSTRUMENT] extractTasks END - total ${totalTime}ms`)
+
     return file
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "imdone-core",
-  "version": "2.1.4",
+  "version": "2.1.6",
   "description": "imdone-core",
   "repository": {
     "type": "git",
@@ -84,7 +84,7 @@
     "markdown-it-emoji": "^2.0.0",
     "markdown-it-mark": "^3.0.1",
     "moment": "^2.29.1",
-    "monquery": "github:imdone/node-monquery#0.2.1",
+    "monquery": "github:imdone/node-monquery#0.2.2",
     "multilang-extract-comments": "^0.4.0",
     "open": "^9.1.0",
     "ora": "^8.2.0",

package/vitest.config.js

@@ -2,14 +2,25 @@ import { defineConfig } from 'vitest/config';
 
 export default defineConfig({
   test: {
-    include: ['lib/**/*.spec.js'], // Only look for test files inside src/
+    include: ['lib/**/*.spec.js'],
     coverage: {
-      provider: "v8", // or "istanbul" if using @vitest/coverage-istanbul
-      reporter: ["text"], // Ensure text output is included
+      provider: "v8",
+      reporter: ["text"],
       clean: false,
       reportOnFailure: true,
-      include: ['lib'], // Only include files from src/
-      exclude: ['_lib', 'lib/**/index.*']
+      include: ['lib/**/*.js'],
+      exclude: [
+        '_lib',
+        'lib/**/index.*',
+        'lib/**/__tests__/**',
+        'lib/**/*.spec.js',
+      ],
+      thresholds: {
+        statements: 87,
+        branches: 83,
+        functions: 80,
+        lines: 87,
+      },
     },
   },
 });