pi-harness-runtime 1.1.445 → 1.1.447

package/scripts/test-r2.ts

@@ -1,63 +0,0 @@
-/**
- * Test R2 Connection
- * 
- * Run: bun run scripts/test-r2.ts
- */
-
-import { loadR2Config, getR2Client, uploadLogsToR2, isR2Configured } from "./r2-storage.js";
-
-async function main() {
-  console.log("=".repeat(60));
-  console.log("R2 Connection Test");
-  console.log("=".repeat(60));
-
-  // Check if configured
-  console.log("\n1. Checking configuration...");
-  if (!isR2Configured()) {
-    console.log("❌ R2 not configured - missing key files");
-    process.exit(1);
-  }
-  console.log("✓ R2 credentials found");
-
-  const config = loadR2Config();
-  console.log(`   Account ID: ${config!.accountId.slice(0, 8)}...`);
-  console.log(`   Bucket: ${config!.bucket}`);
-  console.log(`   Endpoint: ${config!.endpoint || "(auto)"}`);
-
-  // Test connection
-  console.log("\n2. Testing connection...");
-  const client = getR2Client();
-  if (!client) {
-    console.log("❌ Failed to create R2 client");
-    process.exit(1);
-  }
-  console.log("✓ R2 client created");
-
-  // Upload test
-  console.log("\n3. Testing upload...");
-  const testContent = `R2 Test Log
-================
-Timestamp: ${new Date().toISOString()}
-Test message for R2 upload verification
-`;
-
-  const result = await uploadLogsToR2(testContent, "test");
-  if (!result) {
-    console.log("❌ Upload failed");
-    process.exit(1);
-  }
-
-  console.log("✓ Upload successful!");
-  console.log(`   Key: ${result.key}`);
-  console.log(`   Presigned URL: ${result.url.slice(0, 80)}...`);
-  console.log(`   Expires in: ${result.expiresIn / 3600} hours`);
-
-  console.log("\n" + "=".repeat(60));
-  console.log("✅ R2 is working correctly!");
-  console.log("=".repeat(60));
-}
-
-main().catch(err => {
-  console.error("❌ Test failed:", err);
-  process.exit(1);
-});

package/scripts/test-telegram-approval.ts

@@ -1,68 +0,0 @@
-/**
- * Test Telegram Approval Flow
- * 
- * Sends a test message with Yes/No buttons to verify the full flow:
- * 1. Message sent to Telegram with inline keyboard
- * 2. User clicks button
- * 3. Webhook receives callback
- * 4. Callback processor handles approval/rejection
- * 5. Job command written for pi-harness to resume
- */
-
-import { homedir as osHomedir } from "node:os";
-import * as fs from "node:fs";
-import * as path from "node:path";
-
-// Load config
-const keysDir = path.join(process.env.HOME || osHomedir(), ".pi-harness-runtime", "keys");
-
-const botToken = fs.readFileSync(path.join(keysDir, "telegram-bot-token.txt"), "utf8").trim();
-const chatId = fs.readFileSync(path.join(keysDir, "telegram-chat-id.txt"), "utf8").trim();
-const webhookUrl = fs.existsSync(path.join(keysDir, "telegram-webhook-url.txt"))
-  ? fs.readFileSync(path.join(keysDir, "telegram-webhook-url.txt"), "utf8").trim()
-  : "https://yourdomain.com/api/telegram/webhook";
-
-async function sendTestApproval() {
-  const message = `*Test Approval Request*\n\n` +
-    `A job is waiting for your approval:\n\n` +
-    `*Job:* Code Review for PR #123\n` +
-    `*Status:* Waiting for human review\n\n` +
-    `Do you want to approve this job to continue?`;
-
-  const inlineKeyboard = {
-    inline_keyboard: [
-      [
-        { text: "✅ Yes, Approve", callback_data: "approve:job-123" },
-        { text: "❌ No, Reject", callback_data: "reject:job-123" }
-      ],
-      [
-        { text: "🔄 Retry Later", callback_data: "retry:job-123" }
-      ]
-    ]
-  };
-
-  const response = await fetch(`https://api.telegram.org/bot${botToken}/sendMessage`, {
-    method: "POST",
-    headers: { "Content-Type": "application/json" },
-    body: JSON.stringify({
-      chat_id: chatId,
-      text: message,
-      parse_mode: "Markdown",
-      reply_markup: inlineKeyboard
-    })
-  });
-
-  const data = await response.json() as { ok: boolean; result?: { message_id: number } };
-  
-  if (data.ok) {
-    console.log(`✅ Test message sent! Message ID: ${data.result?.message_id}`);
-    console.log(`\n📱 Check your Telegram for the approval message.`);
-    console.log(`\nWhen you click a button, the webhook at:`);
-    console.log(`  ${webhookUrl}`);
-    console.log(`\nwill receive the callback and process it.`);
-  } else {
-    console.error(`❌ Failed to send message:`, data);
-  }
-}
-
-sendTestApproval().catch(console.error);

package/scripts/test-telegram.ts

@@ -1,74 +0,0 @@
-#!/usr/bin/env bun
-/**
- * Test Telegram notification
- *
- * Reads credentials from ~/.pi-harness-runtime/keys/
- * - telegram-bot-token.txt
- * - telegram-chat-id.txt
- *
- * DO NOT hardcode credentials here.
- * Rotate your bot token at https://t.me/BotFather if leaked.
- */
-import { NotificationCenter } from '../packages/notification/dist/notification-center.js';
-import type { NotificationContext } from '../packages/notification/dist/types.js';
-import { readFileSync, existsSync } from 'node:fs';
-import { join } from 'node:path';
-import { homedir } from 'node:os';
-
-const home = homedir();
-const keysDir = join(home, '.pi-harness-runtime', 'keys');
-
-const botTokenPath = join(keysDir, 'telegram-bot-token.txt');
-const chatIdPath = join(keysDir, 'telegram-chat-id.txt');
-
-// Check credentials exist
-if (!existsSync(botTokenPath)) {
-  console.error("ERROR: telegram-bot-token.txt not found");
-  console.error(`  Create: echo "{bot-token}" > ${botTokenPath}`);
-  process.exit(1);
-}
-
-if (!existsSync(chatIdPath)) {
-  console.error("ERROR: telegram-chat-id.txt not found");
-  console.error(`  Create: echo "{chat-id}" > ${chatIdPath}`);
-  process.exit(1);
-}
-
-const botToken = readFileSync(botTokenPath, 'utf8').trim();
-const chatId = readFileSync(chatIdPath, 'utf8').trim();
-
-if (!botToken || !chatId) {
-  console.error("ERROR: Bot token or chat ID is empty");
-  process.exit(1);
-}
-
-console.log("Sending test Telegram notification...");
-
-const nc = new NotificationCenter({
-  channels: [
-    {
-      id: "telegram",
-      type: "telegram",
-      enabled: true,
-      config: {
-        botToken,
-        chatId,
-        parseMode: "HTML" as const // Use HTML mode instead of MarkdownV2
-      }
-    }
-  ]
-});
-
-const context: NotificationContext = {
-  jobId: "test-123",
-  requirement: "Testing Telegram notification",
-  taskTitle: "Test"
-};
-
-nc.notify("HumanReviewNeeded", context).then((results) => {
-  console.log("Results:", JSON.stringify(results, null, 2));
-  process.exit(0);
-}).catch((e: unknown) => {
-  console.error("Error:", e);
-  process.exit(1);
-});

package/scripts/tsconfig.json

@@ -1,12 +0,0 @@
-{
-  "extends": "../tsconfig.base.json",
-  "compilerOptions": {
-    "module": "ESNext",
-    "moduleResolution": "bundler",
-    "types": ["node", "bun-types"],
-    "noEmit": true,
-    "allowImportingTsExtensions": true
-  },
-  "include": ["**/*.ts", "**/*.d.mts"],
-  "exclude": ["release/**"]
-}

package/scripts/typecheck-packages.mjs

@@ -1,235 +0,0 @@
-#!/usr/bin/env node
-/**
- * Type check every TypeScript project that ships a tsconfig.json.
- *
- * `npm run build` compiles the packages but swallows every error (`|| true`),
- * and the CI "Type check" step used to be a no-op (`npm run typecheck ||
- * echo "No typecheck script"` - there was no typecheck script and no root
- * tsconfig.json), so nothing was ever type checked.
- *
- * This script is the real check: it runs `tsc --noEmit` per project and
- * fails when a project reports MORE errors than its recorded baseline.
- *
- * A baseline is the number of type errors a project already has on HEAD.
- * It exists because eight packages, the harness and the scripts directory
- * have pre-existing type errors that are tracked separately; without a
- * baseline this script could never go green. A baseline is a CEILING, not
- * a waiver: one additional error in a baselined project fails the build
- * too, so the blind spot cannot grow.
- */
-import { execFileSync } from "node:child_process";
-import { existsSync, mkdtempSync, readdirSync, readFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join, relative } from "node:path";
-
-/**
- * Maximum tolerated number of type errors per project. A project that is
- * not listed must type check cleanly.
- */
-// Every project type checks cleanly. A non-zero entry is a debt that hides
-// a real error, so the baseline stays at zero until a project genuinely
-// compiles.
-const BASELINES = new Map([
-	["a2a-adapter", 0],
-	["clipboard-plugin", 0],
-	["provider-router", 0],
-	["providers", 0],
-	["quota-manager", 0],
-	["runtime", 0],
-	["voice-api", 0],
-	["worktree", 0],
-	["harness", 0],
-	// The scripts of the repository type check cleanly. The project
-	// lives in `scripts/tsconfig.json`, so the language server reads the
-	// same configuration as the compiler and no longer answers with a
-	// project that has no node types.
-	["scripts", 0],
-	["scripts/release", 0],
-	["root", 0],
-	["new-modules", 0],
-]);
-
-const TSC_TIMEOUT_MS = 300_000;
-
-const root = join(import.meta.dirname, "..");
-const packagesDir = join(root, "packages");
-const tsc = join(root, "node_modules", ".bin", "tsc");
-
-if (!existsSync(tsc)) {
-	console.error(`typecheck: ${tsc} not found - run "bun install" first`);
-	process.exit(1);
-}
-
-/** All projects to check, relative to the repo root. */
-const projects = [];
-/**
- * Projects that were expected but are missing.
- *
- * A missing tsconfig means the code behind it is NOT type checked - a
- * silent loss of coverage. The file may be absent on purpose (an installed
- * copy of the package does not ship the tsconfigs), so this is reported as
- * a note, not as a failure.
- */
-const missingProjects = [];
-for (const name of readdirSync(packagesDir).sort()) {
-	const project = join(packagesDir, name, "tsconfig.json");
-	if (existsSync(project)) projects.push(relative(root, project));
-}
-for (const extra of ["harness", "scripts", "scripts/release"]) {
-	if (existsSync(join(root, extra, "tsconfig.json"))) {
-		projects.push(join(extra, "tsconfig.json"));
-	}
-}
-// The root extension (index.ts) has no project of its own - it is the entry
-// point of the extension and was never type checked.
-if (existsSync(join(root, "tsconfig.root.json"))) {
-	projects.push("tsconfig.root.json");
-} else {
-	missingProjects.push("tsconfig.root.json");
-}
-// The new startup modules are checked under `strict` - the root project
-// itself cannot be (it carries thousands of pre-existing loose typings).
-if (existsSync(join(root, "tsconfig.new-modules.json"))) {
-	projects.push("tsconfig.new-modules.json");
-} else {
-	missingProjects.push("tsconfig.new-modules.json");
-}
-
-/** Baseline owner for a project path ("packages/runtime" -> that package). */
-function ownerOf(projectPath) {
-	const normalized = projectPath.split("\\").join("/");
-	if (normalized === "tsconfig.root.json") return "root";
-	if (normalized === "tsconfig.new-modules.json") return "new-modules";
-	if (normalized.startsWith("packages/")) {
-		return normalized.split("/")[1];
-	}
-	// A project in a folder of its own (the release sources in
-	// scripts/release) is a project of its own, and not the project of
-	// the folder that holds it.
-	if (normalized.endsWith("/tsconfig.json")) {
-		return normalized.slice(0, -"/tsconfig.json".length);
-	}
-	return normalized.split("/")[0];
-}
-
-const failures = [];
-const baselined = [];
-const improved = [];
-const skipped = [];
-let checked = 0;
-
-// `tsc --noEmit` still writes the .tsbuildinfo of an incremental or
-// composite project - into the repo, where those files are tracked. The
-// build info is redirected to a temp directory so a type check never
-// modifies the working tree.
-const buildInfoDir = mkdtempSync(join(tmpdir(), "typecheck-packages-"));
-
-for (const project of projects) {
-	const owner = ownerOf(project);
-	const baseline = BASELINES.get(owner) ?? 0;
-	checked++;
-	let output = "";
-	let timedOut = false;
-	const args = ["-p", join(root, project), "--noEmit", "--skipLibCheck"];
-	const projectConfig = readFileSync(join(root, project), "utf8");
-	if (/"(incremental|composite)"\s*:\s*true/.test(projectConfig)) {
-		args.push("--tsBuildInfoFile", join(buildInfoDir, `${owner}.tsbuildinfo`));
-	}
-	try {
-		output = execFileSync(tsc, args, {
-			cwd: root,
-			encoding: "utf8",
-			stdio: ["ignore", "pipe", "pipe"],
-			timeout: TSC_TIMEOUT_MS,
-			maxBuffer: 64 * 1024 * 1024,
-		});
-	} catch (error) {
-		output = `${error.stdout ?? ""}${error.stderr ?? ""}`;
-		timedOut = Boolean(error.killed) || error.signal === "SIGTERM";
-	}
-		const errorLines = output.split("\n").filter((line) => /error TS/.test(line));
-		// The root program pulls the harness files into its program, so their
-		// pre-existing errors would be counted a second time (the harness
-		// project has its own baseline). Only the errors of the root file
-		// count for the root project.
-		const projectErrors =
-			owner === "root"
-				? errorLines.filter((line) => /^index\.ts\(/.test(line))
-				: errorLines;
-		// TS18003: "No inputs were found in config file". A package whose
-		// sources are compiled output only (packages/*/src/*.js and *.d.ts
-		// are gitignored) has no TypeScript input in a fresh checkout, so
-		// there is nothing to check - that is not a type error.
-		const noInputs = projectErrors.some((line) => /error TS18003/.test(line));
-		const count = projectErrors.length;
-
-		if (noInputs) {
-			skipped.push(`${owner} (no TypeScript sources in this checkout)`);
-			continue;
-		}
-		if (timedOut) {
-		// A timeout is not a pass and not an error count - fail closed.
-		failures.push({
-			owner,
-			project,
-			count: -1,
-			output: `type check timed out after ${TSC_TIMEOUT_MS} ms`,
-		});
-		continue;
-	}
-	if (count > baseline) {
-		failures.push({ owner, project, count, output });
-		continue;
-	}
-	if (baseline > 0) {
-		if (count < baseline) {
-			improved.push(`${owner} (${count} errors, baseline ${baseline})`);
-		} else {
-			baselined.push(`${owner} (${count} known errors)`);
-		}
-	}
-}
-
-// A baseline for a project that does not exist (renamed or removed) is stale.
-for (const owner of BASELINES.keys()) {
-	const stillExists = projects.some((project) => ownerOf(project) === owner);
-	if (!stillExists) {
-		failures.push({
-			owner,
-			project: owner,
-			count: -1,
-			output: `stale baseline: no project found for "${owner}"`,
-		});
-	}
-}
-
-console.log(`typecheck: ${checked} projects checked`);
-if (skipped.length > 0) {
-	console.log(`typecheck: ${skipped.length} project(s) skipped: ${skipped.join(", ")}`);
-}
-if (baselined.length > 0) {
-	console.log(
-		`typecheck: ${baselined.length} project(s) with known pre-existing errors: ${baselined.join(", ")}`,
-	);
-}
-if (improved.length > 0) {
-	console.warn(
-		`typecheck: baseline can be lowered for: ${improved.join(", ")} (update BASELINES in ${relative(root, import.meta.filename)})`,
-	);
-}
-
-if (failures.length > 0) {
-	console.error(`typecheck: ${failures.length} project(s) with unexpected type errors:`);
-	for (const failure of failures) {
-		console.error(`--- ${failure.owner} (${failure.count < 0 ? failure.output : `${failure.count} errors`}) ---`);
-		if (failure.count >= 0) console.error(failure.output.trim());
-	}
-	process.exit(1);
-}
-
-console.log("typecheck: no unexpected type errors");
-if (missingProjects.length > 0) {
-	console.log(
-		`typecheck: note - ${missingProjects.length} project file(s) are missing, so their code is not type checked: ${missingProjects.join(", ")}`,
-	);
-}