@mjasnikovs/pi-task 0.13.38 → 0.14.1

package/README.md

@@ -9,7 +9,7 @@
 [![npm](https://img.shields.io/npm/v/@mjasnikovs/pi-task?color=cb3837&logo=npm)](https://www.npmjs.com/package/@mjasnikovs/pi-task)
 [![license](https://img.shields.io/badge/license-MIT-blue.svg)](./LICENSE)
 [![pi extension](https://img.shields.io/badge/pi-extension-7c3aed)](https://www.npmjs.com/package/@earendil-works/pi-coding-agent)
-[![tests](https://img.shields.io/badge/tests-731%20passing-3fb950)](#development)
+[![tests](https://img.shields.io/badge/tests-788%20passing-3fb950)](#development)
 [![types](https://img.shields.io/badge/TypeScript-strict-3178c6?logo=typescript&logoColor=white)](./tsconfig.json)
 
 </div>
@@ -106,7 +106,7 @@ Delivery is **server → push service → device** over the [Web Push](https://d
 2. **Share → Add to Home Screen**, then open the app from that icon. iOS only permits notifications for installed PWAs.
 3. Launch the app, **tap the bell**, and **Allow** when prompted.
 
-The subscription is kept in memory, so after restarting `pi` just reload the app (it re-subscribes automatically) or tap the bell again. Notifications are suppressed while the app is focused in the foreground — the in-page UI already shows the prompt there.
+Subscriptions are mirrored to `${XDG_DATA_HOME:-~/.local/share}/pi-task/subscriptions.json` and reloaded on startup, so they survive a full `pi` restart — the server keeps reaching a backgrounded device without waiting for it to re-register. Notifications are suppressed while the app is focused in the foreground — the in-page UI already shows the prompt there.
 
 VAPID keys are generated once and persisted to `${XDG_DATA_HOME:-~/.local/share}/pi-task/vapid.json` (deleting them invalidates existing subscriptions). The JWT contact (`sub`) defaults to the project URL; override it with `PI_REMOTE_PUSH_SUBJECT` (e.g. your own `mailto:you@domain.com`). To debug delivery, set `PI_REMOTE_PUSH_DEBUG=1` and tail `/tmp/pi-task-push.log` — it records each push and the **push-service HTTP status** (`201` delivered, `403`/`400` token/key problem, `410` stale subscription).
 
@@ -167,7 +167,7 @@ Tasks are persisted to `<cwd>/.pi-tasks/TASK_NNNN.md`. Add `.pi-tasks/` to your
 
 ```sh
 bun install
-bun test src/      # 731 tests across 57 files
+bun test src/      # 788 tests across 60 files
 bun run lint       # prettier + eslint + tsc --noEmit
 bun run build      # tsc → dist/
 ```

package/dist/task/enforce-guidelines.d.ts

@@ -74,6 +74,13 @@ export declare function classifyEnforceChildFailure(r: EnforceChildResult): stri
  * names of any new (untracked) files. Non-destructive — it does not touch the
  * index, so the later `git add -A` in gitCommitAll still stages everything
  * (including fixes the enforcement child makes).
+ *
+ * The `.pi-tasks/` directory is excluded from both git commands. Those task
+ * files are committable (tracked) by design, so they show up in `git diff HEAD`
+ * and `git ls-files --others` — git does not honor the fd/ripgrep `.ignore` that
+ * keeps them out of the worker's find/grep discovery. Without this pathspec the
+ * enforce child is handed its own TASK_*.md / TASK_AUTO_*.md bookkeeping as
+ * "changes to verify" and edits them, corrupting the front matter mid-run.
  */
 export declare function captureDiff(cwd: string, signal?: AbortSignal, spawnFn?: SpawnFn): Promise<string>;
 export interface EnforcementDeps {

package/dist/task/enforce-guidelines.js

@@ -20,6 +20,7 @@ import * as fsp from 'node:fs/promises';
 import * as path from 'node:path';
 import { runChildDefault } from '../shared/child-process.js';
 import { USER_CANCELLED } from './child-runner.js';
+import { TASKS_DIR_NAME } from './task-types.js';
 /** Filenames discovered in the working directory (cwd only — no tree walk). */
 export const GUIDELINE_FILENAMES = ['AGENTS.md', 'CLAUDE.md'];
 /** Tools the fix pass is allowed: read/search to inspect, edit/write to fix. */
@@ -131,11 +132,28 @@ export function classifyEnforceChildFailure(r) {
  * names of any new (untracked) files. Non-destructive — it does not touch the
  * index, so the later `git add -A` in gitCommitAll still stages everything
  * (including fixes the enforcement child makes).
+ *
+ * The `.pi-tasks/` directory is excluded from both git commands. Those task
+ * files are committable (tracked) by design, so they show up in `git diff HEAD`
+ * and `git ls-files --others` — git does not honor the fd/ripgrep `.ignore` that
+ * keeps them out of the worker's find/grep discovery. Without this pathspec the
+ * enforce child is handed its own TASK_*.md / TASK_AUTO_*.md bookkeeping as
+ * "changes to verify" and edits them, corrupting the front matter mid-run.
  */
 export async function captureDiff(cwd, signal, spawnFn) {
+    // `:(exclude)<dir>` is a git pathspec that drops everything under the tasks
+    // directory from the result, leaving only real source changes to verify.
+    const excludeTasks = `:(exclude)${TASKS_DIR_NAME}`;
     const run = (args) => runChildDefault({ command: 'git', args }, cwd, signal, { mode: 'text' }, spawnFn);
-    const tracked = await run(['diff', 'HEAD']);
-    const untracked = await run(['ls-files', '--others', '--exclude-standard']);
+    const tracked = await run(['diff', 'HEAD', '--', '.', excludeTasks]);
+    const untracked = await run([
+        'ls-files',
+        '--others',
+        '--exclude-standard',
+        '--',
+        '.',
+        excludeTasks
+    ]);
     const parts = [];
     if (tracked.exitCode === 0 && tracked.stdout.trim().length > 0)
         parts.push(tracked.stdout.trim());

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.13.38",
+    "version": "0.14.1",
     "description": "Deterministic spec-orchestration for local models, with a bundled real-time remote web view and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",