pi-audit-master 0.1.1 → 0.1.3

package/tests/buggy-project/src/code-quality.ts

@@ -1,13 +0,0 @@
-// code-quality.ts
-function calculateTax(amount: number) {
-	const tax = amount * 0.15;
-	// LOW: Redundant logic
-	const finalTax = amount * 0.15;
-	return finalTax;
-}
-
-function checkStatus(val: number) {
-	// LOW: Magic number 42
-	if (val === 42) return "Special";
-	return "Normal";
-}

package/tests/buggy-project/src/legacy.ts

@@ -1,12 +0,0 @@
-export function processPayment(amount: number) {
-	// LOW: Magic number
-	if (amount > 5000) {
-		return amount * 0.02;
-	}
-	return amount * 0.05;
-}
-
-function oldLegacyHelper() {
-	// LOW: Dead code
-	console.log("This function is never called");
-}

package/tests/buggy-project/src/logic-flow.ts

@@ -1,16 +0,0 @@
-// logic-flow.ts
-async function updateUserPreferences(prefs: any[]) {
-	// HIGH: Race condition - forEach with async doesn't await
-	prefs.forEach(async (p) => {
-		await db.save(p);
-	});
-}
-
-function calculateSum(arr: number[]) {
-	let sum = 0;
-	// MEDIUM: Off-by-one error
-	for (let i = 0; i <= arr.length; i++) {
-		sum += arr[i];
-	}
-	return sum;
-}

package/tests/buggy-project/src/logic_error.ts

@@ -1,8 +0,0 @@
-export async function updateData(id: string, data: any) {
-    const current = await fetchById(id);
-    // HIGH: Race condition - not checking if data changed between fetch and update
-    const updated = { ...current, ...data };
-    await save(updated);
-}
-async function fetchById(id: string) { return { id }; }
-async function save(d: any) { return true; }

package/tests/buggy-project/src/messy_code.ts

@@ -1,9 +0,0 @@
-export function calc(a: number, b: number) {
-    // LOW: Magic number 86400, redundant logic
-    const res = a * 86400;
-    const result = res;
-    if (result === res) {
-        return result;
-    }
-    return res;
-}

package/tests/buggy-project/src/null-safety.ts

@@ -1,12 +0,0 @@
-// null-safety.ts
-async function fetchUserData(id: string) {
-	const response = await api.get(`/user/${id}`);
-	// CRITICAL: Null access on response.data.profile
-	return response.data.profile.name;
-}
-
-function processPayment(input: any) {
-	// HIGH: Unsafe cast to any
-	const amount = (input as any).amount;
-	return amount * 1.1;
-}

package/tests/buggy-project/src/null_safety.ts

@@ -1,4 +0,0 @@
-export function processUser(user: any) {
-    // CRITICAL: Missing null check on user and user.profile
-    return user.profile.address.city.toUpperCase();
-}

package/tests/buggy-project/src/performance.ts

@@ -1,17 +0,0 @@
-// performance.ts
-function processLargeDataset(data: any[]) {
-	const duplicates = [];
-	// MEDIUM: O(n^2) complexity
-	for (let i = 0; i < data.length; i++) {
-		for (let j = i + 1; j < data.length; j++) {
-			if (data[i].id === data[j].id) duplicates.push(data[i]);
-		}
-	}
-	return duplicates;
-}
-
-const log = [];
-function onMessageReceived(msg: string) {
-	// HIGH: Memory leak - global array grows indefinitely
-	log.push(msg);
-}

package/tests/buggy-project/src/performance_leak.ts

@@ -1,12 +0,0 @@
-export function findDuplicates(arr: string[]) {
-    const duplicates: string[] = [];
-    // MEDIUM: O(n^2) complexity
-    for (let i = 0; i < arr.length; i++) {
-        for (let j = i + 1; j < arr.length; j++) {
-            if (arr[i] === arr[j] && !duplicates.includes(arr[i])) {
-                duplicates.push(arr[i]);
-            }
-        }
-    }
-    return duplicates;
-}

package/tests/buggy-project/src/pi-integration.ts

@@ -1,11 +0,0 @@
-import type { ExtensionAPI } from 'pi-coding-agent';
-
-export default function (pi: ExtensionAPI) {
-    // CRITICAL: Wrong event name. Should be 'tool_execution_start'.
-    pi.on('tool_execution_started', async (event, ctx) => {
-        if (event.toolName === 'delete_everything') {
-            ctx.ui.notify('Danger!', 'error');
-            // HIGH: Forgets to call event.abort(). Tool will execute anyway.
-        }
-    });
-}

package/tests/buggy-project/src/utils.ts

@@ -1,14 +0,0 @@
-export function findDuplicates(list: string[]) {
-	const duplicates: string[] = [];
-
-	// MEDIUM: O(n^2) complexity.
-	// For each item, it scans the rest of the list.
-	for (let i = 0; i < list.length; i++) {
-		for (let j = i + 1; j < list.length; j++) {
-			if (list[i] === list[j]) {
-				duplicates.push(list[i]);
-			}
-		}
-	}
-	return duplicates;
-}

package/tests/setup.sh

@@ -1,87 +0,0 @@
-#!/bin/bash
-set -e
-
-# Move to the project root
-cd "$(dirname "$0")/.."
-
-echo "🚀 Setting up test environment for pi-audit-master..."
-
-# 1. Install dependencies
-echo "📦 Installing dependencies..."
-npm install
-
-# 2. Create buggy project structure
-echo "📁 Creating buggy-project..."
-mkdir -p tests/buggy-project/extensions
-mkdir -p tests/buggy-project/src
-
-# 3. Write buggy files
-echo "📝 Writing buggy files..."
-
-# Null Safety Bug
-cat <<EOF >tests/buggy-project/src/null_safety.ts
-export function processUser(user: any) {
-    // CRITICAL: Missing null check on user and user.profile
-    return user.profile.address.city.toUpperCase();
-}
-EOF
-
-# Logic Bug
-cat <<EOF >tests/buggy-project/src/logic_error.ts
-export async function updateData(id: string, data: any) {
-    const current = await fetchById(id);
-    // HIGH: Race condition - not checking if data changed between fetch and update
-    const updated = { ...current, ...data };
-    await save(updated);
-}
-async function fetchById(id: string) { return { id }; }
-async function save(d: any) { return true; }
-EOF
-
-# Performance Bug
-cat <<EOF >tests/buggy-project/src/performance_leak.ts
-export function findDuplicates(arr: string[]) {
-    const duplicates: string[] = [];
-    // MEDIUM: O(n^2) complexity
-    for (let i = 0; i < arr.length; i++) {
-        for (let j = i + 1; j < arr.length; j++) {
-            if (arr[i] === arr[j] && !duplicates.includes(arr[i])) {
-                duplicates.push(arr[i]);
-            }
-        }
-    }
-    return duplicates;
-}
-EOF
-
-# Pi Integration Bug
-cat <<EOF >tests/buggy-project/extensions/pi_wrong.ts
-export default function (pi: any) {
-    // HIGH: Wrong event name 'tool_execution_started' instead of 'tool_execution_start'
-    pi.on('tool_execution_started', (event: any, ctx: any) => {
-        console.log('Intercepted!');
-        // Missing event.abort() for a blocking middleware
-    });
-}
-EOF
-
-# Quality Bug
-cat <<EOF >tests/buggy-project/src/messy_code.ts
-export function calc(a: number, b: number) {
-    // LOW: Magic number 86400, redundant logic
-    const res = a * 86400;
-    const result = res;
-    if (result === res) {
-        return result;
-    }
-    return res;
-}
-EOF
-
-echo "✅ Buggy project created."
-
-# 4. Run tests
-echo "🧪 Running tests..."
-npm test
-
-echo "✨ Setup and testing complete!"

package/tests/verify-audit.test.ts

@@ -1,77 +0,0 @@
-import { AuditManager } from "../extensions/audit-manager";
-import * as path from "path";
-import type { ExtensionAPI } from "pi-coding-agent";
-
-// Mocks
-const mockPi = {
-	on: jest.fn(),
-	registerTool: jest.fn(),
-	subagents: {
-		parallel: jest.fn(),
-	},
-} as unknown as ExtensionAPI;
-
-const mockCtx = {
-	ui: {
-		notify: jest.fn(),
-	},
-};
-
-describe("AuditManager Verification", () => {
-	let manager: AuditManager;
-	const projectPath = path.join(__dirname, "buggy-project");
-
-	beforeEach(() => {
-		manager = new AuditManager(mockPi);
-		jest.clearAllMocks();
-	});
-
-	it("should detect critical and high bugs in buggy-project", async () => {
-		const spy = jest
-			.spyOn(manager, "dispatchAuditAgents")
-			.mockResolvedValue([
-				`| CRITICAL | src/api.ts:10 | Null access on data.profile | Add null guard |`,
-				`| HIGH | src/auth.ts:12 | Login called before init | Add initialization check |`,
-				`| MEDIUM | src/utils.ts:8 | O(n^2) loop in findDuplicates | Use a Map for O(n) |`,
-				`| CRITICAL | extensions/my-ext.ts:7 | Missing event.abort() | Call event.abort() to block tool |`,
-				`| LOW | src/legacy.ts:5 | Magic number 5000 | Move to constant |`,
-			]);
-
-		const finalResult = await manager.runAudit({
-			path: projectPath,
-			depth: "deep",
-			format: "file",
-			fix: false,
-			ctx: mockCtx as any,
-		});
-
-		expect(finalResult.report).toContain("CRITICAL");
-		expect(finalResult.report).toContain("HIGH");
-		expect(finalResult.report).toContain("MEDIUM");
-		expect(finalResult.report).toContain("src/api.ts");
-		expect(finalResult.report).toContain("extensions/my-ext.ts");
-
-		spy.mockRestore();
-	});
-
-	it("should resolve critical issues when fix is enabled", async () => {
-		const spy = jest
-			.spyOn(manager, "dispatchAuditAgents")
-			.mockResolvedValue([
-				`| CRITICAL | src/api.ts:10 | Null access | Add guard |`,
-			]);
-
-		const result = await manager.runAudit({
-			path: projectPath,
-			depth: "deep",
-			format: "chat",
-			fix: true,
-			ctx: mockCtx as any,
-		});
-
-		expect(result.fixes).toBeDefined();
-		expect(result.fixes[0].status).toBe("RESOLVED");
-
-		spy.mockRestore();
-	});
-});

package/tsconfig.json

@@ -1,16 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "module": "NodeNext",
-    "moduleResolution": "NodeNext",
-    "outDir": "./dist",
-    "rootDir": ".",
-    "strict": true,
-    "esModuleInterop": true,
-    "skipLibCheck": true,
-    "forceConsistentCasingInFileNames": true,
-    "declaration": true
-  },
-  "include": ["extensions/**/*", "types/**/*"],
-  "exclude": ["node_modules", "dist"]
-}

package/types/pi-coding-agent.d.ts

@@ -1,13 +0,0 @@
-declare module 'pi-coding-agent' {
-    export interface ExtensionAPI {
-        on(event: string, handler: Function): void;
-        registerTool(options: any): void;
-        subagents: any;
-    }
-    export interface ExtensionCommandContext {
-        ui: {
-            notify(message: string, level: string): void;
-        };
-        askUserQuestion(question: any): Promise<any>;
-    }
-}

package/verify.ts

@@ -1,44 +0,0 @@
-import { AuditManager } from "./extensions/audit-manager";
-import * as path from "path";
-
-async function verify() {
-	console.log("🚀 Starting pi-audit-master verification...");
-
-	const mockPi = { on: {}, registerTool: {} } as any;
-	const mockCtx = { ui: { notify: () => {} } } as any;
-	const manager = new AuditManager(mockPi);
-	const projectPath = path.join(__dirname, "tests/buggy-project");
-
-	// Mock the agent dispatch to simulate findings
-	const spy = ((manager as any).dispatchAuditAgents = async () => [
-		`| CRITICAL | src/api.ts:10 | Null access | Add guard |`,
-		`| HIGH | src/auth.ts:12 | Logic flow | Add check |`,
-		`| MEDIUM | src/utils.ts:8 | O(n^2) | Use Map |`,
-		`| CRITICAL | extensions/my-ext.ts:7 | No abort | Call abort |`,
-	]);
-
-	try {
-		const result = await manager.runAudit({
-			path: projectPath,
-			depth: "deep",
-			format: "file",
-			fix: false,
-			ctx: mockCtx,
-		});
-
-		console.log("✅ Audit result received:");
-		console.log(result.report);
-
-		if (result.report.includes("CRITICAL") && result.report.includes("HIGH")) {
-			console.log("🎯 SUCCESS: Critical and High bugs detected!");
-		} else {
-			console.error("❌ FAILURE: Bugs not detected in report.");
-			process.exit(1);
-		}
-	} catch (e: any) {
-		console.error("❌ Error during verification:", e);
-		process.exit(1);
-	}
-}
-
-verify();