@askjo/pi-reflect 1.0.0 → 1.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@askjo/pi-reflect",
-  "version": "1.0.0",
+  "version": "1.2.0",
   "description": "Self-improving behavioral files for pi coding agents. Analyzes session transcripts for correction patterns and makes surgical edits to prevent recurrence.",
   "keywords": [
     "pi-package",

package/release.sh

@@ -0,0 +1,39 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+BUMP="${1:-patch}"
+if [[ "$BUMP" != "patch" && "$BUMP" != "minor" && "$BUMP" != "major" ]]; then
+  echo "Usage: ./release.sh [patch|minor|major]"
+  exit 1
+fi
+
+cd "$(dirname "$0")"
+
+echo "🔍 Pre-flight checks..."
+if [[ -n "$(git status --porcelain)" ]]; then
+  echo "❌ Working tree is dirty. Commit changes first."
+  exit 1
+fi
+if [[ "$(git branch --show-current)" != "main" ]]; then
+  echo "❌ Not on main."
+  exit 1
+fi
+git fetch origin main --quiet
+if [[ "$(git rev-parse HEAD)" != "$(git rev-parse origin/main)" ]]; then
+  echo "❌ Local main differs from origin/main."
+  exit 1
+fi
+
+echo "🧪 Running tests..."
+npm test
+
+echo "📦 Bumping version: $BUMP"
+npm version "$BUMP" --message "v%s"
+NEW_VERSION=$(node -p "require('./package.json').version")
+
+echo "📤 Pushing main and v${NEW_VERSION} tag..."
+git push origin main --follow-tags
+
+echo "✅ Release v${NEW_VERSION} triggered"
+echo "   Actions: https://github.com/jo-inc/pi-reflect/actions"
+echo "   Package: https://www.npmjs.com/package/@askjo/pi-reflect"

package/tests/apply-edits.test.ts

@@ -345,4 +345,55 @@ describe("applyEdits", () => {
 		assert.ok(result.includes("Line one, improved."));
 		assert.ok(result.includes("Continuation line, also improved."));
 	});
+
+	it("removes a rule with type=remove", () => {
+		const content = "- Rule A\n- Rule B (redundant)\n- Rule C\n";
+		const { result, applied } = applyEdits(content, [
+			{ type: "remove", old_text: "- Rule B (redundant)", new_text: "" },
+		]);
+		assert.equal(applied, 1);
+		assert.ok(!result.includes("Rule B"));
+		assert.ok(result.includes("Rule A"));
+		assert.ok(result.includes("Rule C"));
+	});
+
+	it("skips remove when text not found", () => {
+		const content = "- Rule A\n- Rule B\n";
+		const { applied, skipped } = applyEdits(content, [
+			{ type: "remove", old_text: "- Rule X", new_text: "" },
+		]);
+		assert.equal(applied, 0);
+		assert.equal(skipped.length, 1);
+	});
+
+	it("merges multiple rules into one", () => {
+		const content = "- Always check schema before SQL.\n- Verify column names before queries.\n- Other rule.\n";
+		const { result, applied } = applyEdits(content, [
+			{
+				type: "merge",
+				merge_sources: [
+					"- Always check schema before SQL.",
+					"- Verify column names before queries.",
+				],
+				new_text: "- Always check DB schema before writing any SQL query.",
+			},
+		]);
+		assert.equal(applied, 1);
+		assert.ok(result.includes("- Always check DB schema before writing any SQL query."));
+		assert.ok(!result.includes("Verify column names"));
+		assert.ok(result.includes("Other rule"));
+	});
+
+	it("skips merge when a source is not found", () => {
+		const content = "- Rule A\n- Rule B\n";
+		const { applied, skipped } = applyEdits(content, [
+			{
+				type: "merge",
+				merge_sources: ["- Rule A", "- Rule X"],
+				new_text: "- Merged rule.",
+			},
+		]);
+		assert.equal(applied, 0);
+		assert.equal(skipped.length, 1);
+	});
 });