code hygiene improvements

Author: barrymichaeldoyle

.claude/settings.json

@@ -0,0 +1,16 @@
+{
+  "hooks": {
+    "PostToolUse": [
+      {
+        "matcher": "Write|Edit|MultiEdit",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "pnpm format:changed",
+            "timeout": 120
+          }
+        ]
+      }
+    ]
+  }
+}

.cursor/rules/format-after-edits.mdc

@@ -0,0 +1,11 @@
+---
+description: Run repo formatting after code edits
+globs:
+alwaysApply: true
+---
+
+After making code edits, run `pnpm format` before handing work back when the change spans multiple files or packages.
+
+For small, localized changes, run `pnpm format:changed` or `pnpm format:staged`.
+
+Do not skip formatting. This repository uses `oxfmt`, CI checks `pnpm format:check`, shared Claude settings run `pnpm format:changed` after edit tools, and a pre-commit hook runs `pnpm format:staged` on staged files.

.githooks/pre-commit

@@ -0,0 +1,3 @@
+#!/bin/sh
+
+pnpm format:staged

AGENTS.md

@@ -7,3 +7,7 @@ When working on Convex code, **always read `convex/_generated/ai/guidelines.md`
 Convex agent skills for common tasks can be installed by running `npx convex ai-files install`.
 
 <!-- convex-ai-end -->
+
+After making code edits, run `pnpm format` before handing work back when the change spans multiple files or packages.
+
+For small, localized changes, `pnpm format:changed` or `pnpm format:staged` is acceptable. A pre-commit hook runs `pnpm format:staged` automatically on staged files, and shared Claude settings run `pnpm format:changed` after edit tools.

CLAUDE.md

@@ -7,3 +7,7 @@ When working on Convex code, **always read `convex/_generated/ai/guidelines.md`
 Convex agent skills for common tasks can be installed by running `npx convex ai-files install`.
 
 <!-- convex-ai-end -->
+
+After making code edits, run `pnpm format` before handing work back when the change spans multiple files or packages.
+
+For small, localized changes, `pnpm format:changed` or `pnpm format:staged` is acceptable. A pre-commit hook runs `pnpm format:staged` automatically on staged files, and shared Claude settings run `pnpm format:changed` after edit tools.

CONTRIBUTING.md

@@ -42,7 +42,9 @@ pnpm test                # Run all tests
 pnpm test:cli            # CLI tests only
 pnpm test:vscode-extension  # VS Code extension tests only
 pnpm lint                # Lint
-pnpm format              # Format
+pnpm format              # Format the entire repo
+pnpm format:changed      # Format modified and untracked files
+pnpm format:staged       # Format only staged files
 ```
 
 ## Submitting a PR

README.md

@@ -2,6 +2,11 @@
 
 ![PatchPulse Banner](./assets/banner.png)
 
+[![CI](https://github.com/barrymichaeldoyle/patch-pulse/actions/workflows/ci.yml/badge.svg)](https://github.com/barrymichaeldoyle/patch-pulse/actions/workflows/ci.yml)
+[![npm version](https://img.shields.io/npm/v/patch-pulse)](https://www.npmjs.com/package/patch-pulse)
+[![npm downloads](https://img.shields.io/npm/dm/patch-pulse)](https://www.npmjs.com/package/patch-pulse)
+[![GitHub stars](https://img.shields.io/github/stars/barrymichaeldoyle/patch-pulse)](https://github.com/barrymichaeldoyle/patch-pulse/stargazers)
+
 PatchPulse helps you stay on top of npm dependency updates across your projects.
 
 Live docs: https://barrymichaeldoyle.github.io/patch-pulse/

knip.json

@@ -14,9 +14,8 @@
       "project": ["convex/**/*.ts", "!convex/_generated/**"]
     },
     "packages/vscode-extension": {
-      "entry": ["src/extension.ts", "src/test/**/*.ts", ".vscode-test.mjs"],
-      "project": ["src/**/*.ts"],
-      "ignoreDependencies": ["@vscode/test-electron"]
+      "entry": ["src/extension.ts"],
+      "project": ["src/**/*.ts"]
     }
   }
 }

package.json

@@ -34,18 +34,20 @@
     "dev:vscode-extension": "pnpm --filter patch-pulse-vscode-extension watch",
     "format": "pnpm exec oxfmt --write . '!packages/docs/.astro/**' '!packages/*/convex/_generated/**'",
     "format:check": "pnpm exec oxfmt --check . '!packages/docs/.astro/**' '!packages/*/convex/_generated/**'",
+    "format:changed": "node scripts/format-changed.mjs",
+    "format:staged": "node scripts/format-staged.mjs",
     "ci:check": "pnpm lint && pnpm format:check && pnpm knip && pnpm typecheck && pnpm test:shared && pnpm test:cli && pnpm test:notifier && pnpm build:cli && pnpm build:vscode-extension && pnpm build:docs",
     "knip": "pnpm exec knip",
     "lint": "pnpm exec oxlint .",
     "lint:fix": "pnpm exec oxlint --fix .",
     "test:cli": "pnpm --filter patch-pulse test:run",
     "test:notifier": "pnpm --filter @patch-pulse/notifier-bot test",
     "test:shared": "pnpm --filter @patch-pulse/shared test",
-    "test:vscode-extension": "pnpm --filter patch-pulse-vscode-extension test",
     "test": "pnpm run test:shared && pnpm run test:cli && pnpm run test:notifier",
     "typecheck": "pnpm --filter patch-pulse typecheck && pnpm --filter @patch-pulse/notifier-bot typecheck",
     "changeset": "changeset",
     "changeset:version": "changeset version && pnpm --filter patch-pulse exec node scripts/generate-version.cjs",
-    "changeset:publish": "pnpm build:cli && changeset publish"
+    "changeset:publish": "pnpm build:cli && changeset publish",
+    "prepare": "node scripts/setup-git-hooks.mjs"
   }
 }

packages/notifier-bot/convex/aiSummary.ts

@@ -45,9 +45,9 @@ function buildSummaryPrompt(args: {
 async function callOpenAiSummary(
   model: string,
   prompt: string,
-): Promise<string> {
-  const apiKey = process.env.OPENAI_API_KEY;
-  if (!apiKey) return '';
+): Promise<string | null> {
+  // Caller (summarizeReleaseEvidence) guards that the key exists before calling.
+  const apiKey = process.env.OPENAI_API_KEY!;
 
   const response = await fetch(OPENAI_API_URL, {
     method: 'POST',
@@ -86,7 +86,7 @@ async function callOpenAiSummary(
     output_text?: string;
   };
 
-  return data.output_text?.trim() ?? '';
+  return data.output_text?.trim() || null;
 }
 
 export async function summarizeReleaseEvidence(args: {
@@ -98,6 +98,8 @@ export async function summarizeReleaseEvidence(args: {
 }): Promise<{ model: string; summary: string } | null> {
   const prompt = buildSummaryPrompt(args);
 
+  if (!process.env.OPENAI_API_KEY) return null;
+
   for (const model of [
     process.env.OPENAI_SUMMARY_NANO_MODEL ?? DEFAULT_NANO_MODEL,
     process.env.OPENAI_SUMMARY_MINI_MODEL ?? DEFAULT_MINI_MODEL,