@noice-tech/pi-changelog 1.0.0 → 1.0.2

package/README.md

@@ -32,6 +32,8 @@ Pi packages run with the permissions granted to Pi. **Only use this package when
 
 `auto` is accepted only as `/commit` inference input. Changelog and PR classifications are `feat`, `fix`, `improve`, or `internal`.
 
+`/commit` runs its worker on a side branch of the current session at low thinking, then restores your previous thinking level when it finishes. If you invoke it during an active agent turn, the change-type selector opens immediately; after your selection, the command waits for that turn to settle before starting the worker.
+
 The canonical shared classification rules live in `extensions/changelog/rules.md`.
 
 ## Repository-specific release-note style

package/extensions/changelog/index.ts

@@ -30,7 +30,7 @@ const CHANGE_TYPE_OPTIONS: Array<{ type: ChangeType; label: string }> = [
 
 const MESSAGE_TYPE = 'noice-changelog-commit-result'
 const PROMPT_MESSAGE_TYPE = 'noice-changelog-commit-worker-prompt'
-const STATUS_KEY = 'noice-changelog'
+const COMMIT_WORKER_WIDGET_KEY = 'noice-changelog-commit-worker'
 
 type CommitDisplayStatus = 'ok' | 'cancelled' | 'failed'
 
@@ -41,6 +41,7 @@ interface CommitResultDetails {
   status?: CommitDisplayStatus
 }
 
+let commitCommandPending = false
 let commitWorkerRunning = false
 let agentEndWaiter: ((messages: unknown[]) => void) | undefined
 let latestCommitWorkerMessages: unknown[] | undefined
@@ -154,26 +155,40 @@ export default function noiceChangelogExtension(pi: ExtensionAPI) {
       'Commit changes and create/update PR. Usage: /commit <changeType> <what was done>',
     getArgumentCompletions: getCommitArgumentCompletions,
     handler: async (args, ctx) => {
-      if (commitWorkerRunning) {
-        ctx.ui.notify('Commit worker is already running', 'warning')
+      if (commitCommandPending || commitWorkerRunning) {
+        ctx.ui.notify('Commit command is already active', 'warning')
         return
       }
 
-      await ctx.waitForIdle()
+      commitCommandPending = true
+      let prepared: Awaited<ReturnType<typeof prepareCommit>>
+      try {
+        prepared = await prepareCommit(args, ctx)
+      } catch (error) {
+        commitCommandPending = false
+        throw error
+      }
 
-      const parsed = await resolveChangeTypeAndContext(args, ctx)
-      if (!parsed) {
+      if (!prepared) {
+        commitCommandPending = false
         return
       }
 
+      const { parsed, prompt } = prepared
       const startLeafId = ctx.sessionManager.getLeafId()
-      const prompt = await buildWorkerPrompt(parsed.changeType, parsed.context)
+      const previousThinkingLevel = pi.getThinkingLevel()
 
+      // Establish the running guard before releasing the pending guard. Keeping
+      // this transition synchronous prevents a re-entrant command from starting
+      // a second worker and overwriting the singleton agent-end waiter.
       commitWorkerRunning = true
-      ctx.ui.setStatus(STATUS_KEY, 'running in session branch...')
-      ctx.ui.notify(`Starting commit worker (${parsed.changeType})`, 'info')
+      commitCommandPending = false
 
       try {
+        showCommitWorkerBanner(ctx)
+        ctx.ui.notify(`Starting commit worker (${parsed.changeType})`, 'info')
+        pi.setThinkingLevel('low')
+
         const agentEnd = waitForNextAgentEndAfterIdle(ctx)
         latestCommitWorkerMessages = undefined
         pi.sendMessage(
@@ -302,7 +317,8 @@ export default function noiceChangelogExtension(pi: ExtensionAPI) {
         })
         ctx.ui.notify(`Commit worker failed:\n${message}`, 'error')
       } finally {
-        ctx.ui.setStatus(STATUS_KEY, undefined)
+        pi.setThinkingLevel(previousThinkingLevel)
+        ctx.ui.setWidget(COMMIT_WORKER_WIDGET_KEY, undefined)
         commitWorkerRunning = false
         latestCommitWorkerMessages = undefined
       }
@@ -310,6 +326,20 @@ export default function noiceChangelogExtension(pi: ExtensionAPI) {
   })
 }
 
+function showCommitWorkerBanner(ctx: ExtensionCommandContext) {
+  const message = 'Commit worker running on a side branch of this session…'
+
+  if (ctx.mode !== 'tui') {
+    ctx.ui.setWidget(COMMIT_WORKER_WIDGET_KEY, [message])
+    return
+  }
+
+  ctx.ui.setWidget(
+    COMMIT_WORKER_WIDGET_KEY,
+    (_tui, theme) => new Text(theme.fg('warning', message), 1, 0)
+  )
+}
+
 function getCommitArgumentCompletions(prefix: string) {
   const trimmedStart = prefix.trimStart()
   const leadingWhitespace = prefix.slice(0, prefix.length - trimmedStart.length)
@@ -341,6 +371,29 @@ function getCommitArgumentCompletions(prefix: string) {
   ]
 }
 
+async function prepareCommit(
+  args: string | undefined,
+  ctx: ExtensionCommandContext
+) {
+  const parsed = await resolveChangeTypeAndContext(args, ctx)
+  if (!parsed) return null
+
+  if (!ctx.isIdle()) {
+    ctx.ui.notify(
+      'Commit queued; waiting for the current agent turn to finish',
+      'info'
+    )
+  }
+
+  await ctx.waitForIdle()
+  const prompt = await buildWorkerPrompt(parsed.changeType, parsed.context)
+  // Prompt loading is asynchronous. Re-check idle so another user turn cannot
+  // slip in between the original wait and worker startup.
+  await ctx.waitForIdle()
+
+  return { parsed, prompt }
+}
+
 async function resolveChangeTypeAndContext(
   args: string | undefined,
   ctx: ExtensionCommandContext

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noice-tech/pi-changelog",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "Pi changelog workflow for commits, pull requests, and public release notes.",
   "type": "module",
   "license": "MIT",
@@ -48,6 +48,6 @@
   },
   "scripts": {
     "typecheck": "tsc --noEmit --allowImportingTsExtensions --moduleResolution bundler --module ESNext --target ES2022 --skipLibCheck extensions/changelog/index.ts",
-    "test": "node ../../scripts/smoke-pack.mjs @noice-tech/pi-changelog packages/changelog"
+    "test": "node --test test/*.test.mjs && node ../../scripts/smoke-pack.mjs @noice-tech/pi-changelog packages/changelog"
   }
 }