@ipul-labs/forge 1.16.8 → 1.17.0

package/server/forge/core/prompts.test.js

@@ -1,219 +0,0 @@
-import { describe, it, expect } from "vitest";
-
-import {
-	classifyIssue,
-	buildIdentifyFilesPrompt,
-	buildProposeFixPrompt,
-	buildRefineOrAskPrompt,
-} from "./prompts.js";
-
-// ─── classifyIssue ───────────────────────────────────────────────────────────
-
-describe("classifyIssue", () => {
-	it("detects feat prefix in title", () => {
-		expect(classifyIssue({ title: "feat(mixes): Live Mixes section" })).toBe("feature");
-		expect(classifyIssue({ title: "feat: add new page" })).toBe("feature");
-	});
-
-	it("detects fix prefix in title", () => {
-		expect(classifyIssue({ title: "fix: broken auth flow" })).toBe("fix");
-		expect(classifyIssue({ title: "fix(auth): token refresh race" })).toBe("fix");
-	});
-
-	it("detects chore prefix", () => {
-		expect(classifyIssue({ title: "chore: bump dependencies" })).toBe("chore");
-	});
-
-	it("detects refactor prefix", () => {
-		expect(classifyIssue({ title: "refactor: extract auth service" })).toBe("refactor");
-	});
-
-	it("falls back to labels when no title prefix", () => {
-		expect(classifyIssue({ title: "Add mixes page", labels: ["feature"] })).toBe("feature");
-		expect(classifyIssue({ title: "Add mixes page", labels: ["enhancement"] })).toBe("feature");
-		expect(classifyIssue({ title: "Auth broken", labels: ["bug"] })).toBe("fix");
-	});
-
-	it("handles label objects with name field", () => {
-		expect(classifyIssue({ title: "Add page", labels: [{ name: "Feature" }] })).toBe("feature");
-	});
-
-	it("falls back to description heuristics", () => {
-		expect(classifyIssue({
-			title: "Live Mixes section",
-			description: "## Approach\nReuse existing infrastructure...",
-		})).toBe("feature");
-
-		expect(classifyIssue({
-			title: "Multi-track mashups",
-			description: "## Frontend changes\n- Generalize StemsEngine...",
-		})).toBe("feature");
-	});
-
-	it("defaults to fix when nothing matches", () => {
-		expect(classifyIssue({ title: "Something is wrong" })).toBe("fix");
-		expect(classifyIssue({ title: "Update thing", labels: [] })).toBe("fix");
-	});
-
-	it("handles missing/null fields gracefully", () => {
-		expect(classifyIssue({})).toBe("fix");
-		expect(classifyIssue({ title: null, labels: null })).toBe("fix");
-	});
-});
-
-// ─── buildIdentifyFilesPrompt ────────────────────────────────────────────────
-
-describe("buildIdentifyFilesPrompt", () => {
-	const repoTrees = [
-		{ repo: "group/web-app", files: ["src/App.tsx", "src/types.ts"] },
-	];
-
-	it("uses feature selection rules for feat issues", () => {
-		const issue = { iid: 25, title: "feat(mixes): Live Mixes section", labels: [] };
-		const prompt = buildIdentifyFilesPrompt(issue, repoTrees);
-
-		expect(prompt).toContain("feature implementation");
-		expect(prompt).toContain("PATTERNS");
-		expect(prompt).toContain("max 15");
-		expect(prompt).not.toContain("failing assertion");
-	});
-
-	it("uses bug-fix selection rules for fix issues", () => {
-		const issue = { iid: 10, title: "fix: broken auth", labels: [] };
-		const prompt = buildIdentifyFilesPrompt(issue, repoTrees);
-
-		expect(prompt).toContain("bug fix");
-		expect(prompt).toContain("failing assertion");
-		expect(prompt).toContain("max 10");
-		expect(prompt).not.toContain("PATTERNS");
-	});
-
-	it("says implementing for features, fixing for bugs", () => {
-		const feat = buildIdentifyFilesPrompt(
-			{ iid: 1, title: "feat: new page", labels: [] }, repoTrees,
-		);
-		const fix = buildIdentifyFilesPrompt(
-			{ iid: 2, title: "fix: crash", labels: [] }, repoTrees,
-		);
-
-		expect(feat).toContain("implementing");
-		expect(fix).toContain("fixing");
-	});
-
-	it("includes multi-repo guidance when multiple repos present", () => {
-		const multi = [
-			{ repo: "group/web-app", files: ["src/App.tsx"] },
-			{ repo: "group/api", files: ["internal/handler.go"] },
-		];
-		const prompt = buildIdentifyFilesPrompt(
-			{ iid: 1, title: "fix: thing", labels: [] }, multi,
-		);
-		expect(prompt).toContain("Multi-repo guidance");
-		expect(prompt).toContain("group/web-app, group/api");
-	});
-});
-
-// ─── buildProposeFixPrompt ───────────────────────────────────────────────────
-
-describe("buildProposeFixPrompt", () => {
-	const fileContents = [
-		{ repo: "group/web-app", path: "src/App.tsx", content: "export default function App() {}" },
-	];
-
-	it("uses feat: prefix for feature issues", () => {
-		const issue = { iid: 25, title: "feat: add mixes", labels: [] };
-		const prompt = buildProposeFixPrompt(issue, fileContents);
-
-		expect(prompt).toContain('"feat:');
-		expect(prompt).toContain("Implement this feature");
-		expect(prompt).not.toContain("Root-cause discipline");
-	});
-
-	it("uses fix: prefix for bug issues", () => {
-		const issue = { iid: 10, title: "fix: broken auth", labels: [] };
-		const prompt = buildProposeFixPrompt(issue, fileContents);
-
-		expect(prompt).toContain('"fix:');
-		expect(prompt).toContain("Propose a targeted fix");
-		expect(prompt).toContain("Root-cause discipline");
-	});
-
-	it("includes feature discipline for feature issues", () => {
-		const issue = { iid: 25, title: "feat: mixes", labels: [] };
-		const prompt = buildProposeFixPrompt(issue, fileContents);
-
-		expect(prompt).toContain("Feature implementation rules");
-		expect(prompt).toContain("Follow the issue specification");
-		expect(prompt).toContain("Do NOT invent changes");
-	});
-
-	it("injects promptRules when provided", () => {
-		const issue = { iid: 1, title: "fix: thing", labels: [] };
-		const prompt = buildProposeFixPrompt(issue, fileContents, {
-			promptRules: "Never restructure the NavBar",
-		});
-
-		expect(prompt).toContain("Custom rules");
-		expect(prompt).toContain("Never restructure the NavBar");
-	});
-
-	it("omits promptRules block when empty", () => {
-		const issue = { iid: 1, title: "fix: thing", labels: [] };
-		const prompt = buildProposeFixPrompt(issue, fileContents, { promptRules: "" });
-
-		expect(prompt).not.toContain("Custom rules");
-	});
-
-	it("includes CI context when provided", () => {
-		const issue = { iid: 1, title: "fix: thing", labels: [] };
-		const prompt = buildProposeFixPrompt(issue, fileContents, {
-			ciContext: "Error: Cannot find module './missing'",
-		});
-
-		expect(prompt).toContain("CI pipeline failure logs");
-		expect(prompt).toContain("Cannot find module");
-	});
-});
-
-// ─── buildRefineOrAskPrompt ─────────────────────────────────────────────────
-
-describe("buildRefineOrAskPrompt", () => {
-	const issue = { iid: 1, title: "fix: thing", labels: [] };
-	const fileContents = [
-		{ repo: "r", path: "src/a.ts", content: "const x = 1;" },
-	];
-
-	it("injects promptRules", () => {
-		const prompt = buildRefineOrAskPrompt(issue, fileContents, {
-			feedback: "fix the types",
-			promptRules: "Always use strict TypeScript",
-		});
-
-		expect(prompt).toContain("Custom rules");
-		expect(prompt).toContain("Always use strict TypeScript");
-	});
-
-	it("omits promptRules when empty", () => {
-		const prompt = buildRefineOrAskPrompt(issue, fileContents, {
-			feedback: "fix it",
-			promptRules: "",
-		});
-		expect(prompt).not.toContain("Custom rules");
-	});
-
-	it("includes repo access block for tools mode", () => {
-		const prompt = buildRefineOrAskPrompt(issue, fileContents, {
-			feedback: "check types",
-			repoAccessMode: "tools",
-		});
-		expect(prompt).toContain("current working directory is a checkout");
-	});
-
-	it("includes repo access block for tree mode", () => {
-		const prompt = buildRefineOrAskPrompt(issue, fileContents, {
-			feedback: "check types",
-			repoAccessMode: "tree",
-		});
-		expect(prompt).toContain("full file tree above");
-	});
-});

package/server/forge/core/providers/azuredevops.js

@@ -1,409 +0,0 @@
-/**
- * server/forge/core/providers/azuredevops.js
- *
- * Azure DevOps REST API implementation of ScmProvider.
- * Maps Work Items → issues, Pull Requests → MRs, Pipelines → CI.
- *
- * Required env vars:
- *   AZURE_DEVOPS_TOKEN   — Personal Access Token (Base64 or raw PAT)
- *   AZURE_DEVOPS_ORG     — organization name, e.g. "ipul-labs"
- *   AZURE_DEVOPS_PROJECT — team project name, e.g. "web-app"
- *   AZURE_DEVOPS_REPO    — git repo name (often same as project)
- *
- * Optional:
- *   AZURE_DEVOPS_BASE_BRANCH  — default target branch (default: "main")
- */
-
-import { getConnection } from "../configStore.js";
-import { ScmProvider } from "./base.js";
-
-export class AzureDevOpsProvider extends ScmProvider {
-	/**
-	 * @param {string} [repo]  "org/project/repo" — overrides env vars
-	 */
-	constructor(repo) {
-		super();
-		if (repo && repo.includes("/")) {
-			const parts = repo.split("/");
-			this._org = parts[0];
-			this._project = parts[1];
-			this._repo = parts[2] ?? parts[1];
-		} else {
-			this._org = process.env.AZURE_DEVOPS_ORG ?? "";
-			this._project = process.env.AZURE_DEVOPS_PROJECT ?? "";
-			this._repo = process.env.AZURE_DEVOPS_REPO ?? this._project;
-		}
-		this._defaultBranch = process.env.AZURE_DEVOPS_BASE_BRANCH ?? "main";
-		this._vsBase = `https://dev.azure.com/${this._org}`;
-		this._vssmBase = `https://vsrm.dev.azure.com/${this._org}`;
-	}
-
-	get name() { return "azuredevops"; }
-	get supportsRebase() { return false; }
-
-	// ─── Internal helpers ──────────────────────────────────────────────────────
-
-	_headers() {
-		const token = getConnection("azuredevops").token;
-		// ADO accepts Basic auth with a PAT as the password (username can be anything)
-		const encoded = Buffer.from(`:${token}`).toString("base64");
-		return {
-			Authorization: `Basic ${encoded}`,
-			"Content-Type": "application/json",
-		};
-	}
-
-	_gitBase() {
-		return `${this._vsBase}/${encodeURIComponent(this._project)}/_apis/git/repositories/${encodeURIComponent(this._repo)}`;
-	}
-
-	_witBase() {
-		return `${this._vsBase}/${encodeURIComponent(this._project)}/_apis/wit`;
-	}
-
-	_pipelineBase() {
-		return `${this._vsBase}/${encodeURIComponent(this._project)}/_apis/pipelines`;
-	}
-
-	_buildBase() {
-		return `${this._vsBase}/${encodeURIComponent(this._project)}/_apis/build`;
-	}
-
-	async _req(url, options = {}) {
-		const sep = url.includes("?") ? "&" : "?";
-		const versioned = url.includes("api-version") ? url : `${url}${sep}api-version=7.1`;
-		const res = await fetch(versioned, {
-			...options,
-			headers: { ...this._headers(), ...options.headers },
-		});
-		if (res.status === 204) return null;
-		if (!res.ok) {
-			const text = await res.text().catch(() => "");
-			throw new Error(`AzureDevOps ${res.status} on ${url}: ${text}`);
-		}
-		return res.json();
-	}
-
-	// ─── Normalizers ──────────────────────────────────────────────────────────
-
-	_issue(raw) {
-		const fields = raw.fields ?? {};
-		return {
-			iid: raw.id,
-			title: fields["System.Title"] ?? "",
-			description: fields["System.Description"] ?? null,
-			labels: (fields["System.Tags"] ?? "").split(";").map((t) => t.trim()).filter(Boolean),
-			author: { name: fields["System.CreatedBy"]?.displayName ?? "" },
-			created_at: fields["System.CreatedDate"] ?? "",
-			web_url: raw._links?.html?.href ?? "",
-		};
-	}
-
-	_mr(raw) {
-		const status = raw.status ?? "";
-		return {
-			iid: raw.pullRequestId,
-			title: raw.title,
-			web_url: `${this._vsBase}/${encodeURIComponent(this._project)}/_git/${encodeURIComponent(this._repo)}/pullrequest/${raw.pullRequestId}`,
-			source_branch: (raw.sourceRefName ?? "").replace("refs/heads/", ""),
-			target_branch: (raw.targetRefName ?? "").replace("refs/heads/", ""),
-			state: status === "completed" ? "merged" : status === "abandoned" ? "closed" : "opened",
-			has_conflicts: raw.mergeStatus === "conflicts",
-			rebase_in_progress: false,
-			merge_status: raw.mergeStatus ?? "",
-			detailed_merge_status: null,
-			draft: raw.isDraft ?? false,
-			head_pipeline: null,
-			author: { name: raw.createdBy?.displayName ?? "" },
-			created_at: raw.creationDate ?? "",
-			merged_at: status === "completed" ? (raw.closedDate ?? null) : null,
-		};
-	}
-
-	_pipeline(raw) {
-		return {
-			id: raw.id,
-			status: this._buildStatus(raw),
-			web_url: raw._links?.web?.href ?? "",
-			created_at: raw.queueTime ?? raw.startTime ?? "",
-		};
-	}
-
-	_buildStatus(raw) {
-		const result = raw.result;
-		if (result === "succeeded") return "success";
-		if (result === "failed" || result === "partiallySucceeded") return "failed";
-		if (result === "canceled") return "cancelled";
-		const status = raw.status;
-		if (status === "inProgress") return "running";
-		if (status === "notStarted") return "pending";
-		return status ?? "unknown";
-	}
-
-	_job(raw) {
-		return {
-			id: raw.id,
-			name: raw.name,
-			stage: raw.stageName ?? "",
-			status: this._taskStatus(raw),
-			web_url: raw._links?.web?.href ?? "",
-			duration: raw.finishTime && raw.startTime
-				? Math.round((new Date(raw.finishTime) - new Date(raw.startTime)) / 1000)
-				: null,
-		};
-	}
-
-	_taskStatus(raw) {
-		const result = raw.result;
-		if (result === "succeeded") return "success";
-		if (result === "failed") return "failed";
-		if (result === "skipped") return "skipped";
-		const state = raw.state;
-		if (state === "inProgress") return "running";
-		if (state === "pending") return "pending";
-		return state ?? "unknown";
-	}
-
-	_diff(raw) {
-		const changeType = raw.changeType ?? "";
-		return {
-			old_path: raw.item?.path ?? "",
-			new_path: raw.item?.path ?? "",
-			diff: "",  // ADO diff content requires a separate API call
-			new_file: changeType === "add",
-			renamed_file: changeType === "rename",
-			deleted_file: changeType === "delete",
-		};
-	}
-
-	// ─── Issues (Work Items) ──────────────────────────────────────────────────
-
-	async listOpenIssues() {
-		// WIQL query for open issues in the project
-		const wiql = {
-			query: `SELECT [System.Id] FROM WorkItems WHERE [System.TeamProject] = '${this._project}' AND [System.State] <> 'Done' AND [System.State] <> 'Closed' AND [System.WorkItemType] IN ('Bug','Issue','Task','User Story') ORDER BY [System.ChangedDate] DESC`,
-		};
-		const result = await this._req(
-			`${this._witBase()}/wiql?$top=25`,
-			{ method: "POST", body: JSON.stringify(wiql) },
-		);
-		const ids = (result?.workItems ?? []).map((w) => w.id);
-		if (ids.length === 0) return [];
-
-		const batch = await this._req(
-			`${this._witBase()}/workitems?ids=${ids.join(",")}&$expand=fields`,
-		);
-		return (batch?.value ?? []).map((wi) => this._issue(wi));
-	}
-
-	async getIssue(iid) {
-		const raw = await this._req(`${this._witBase()}/workitems/${iid}?$expand=fields`);
-		return this._issue(raw);
-	}
-
-	async createIssue(title, description = "") {
-		const patch = [
-			{ op: "add", path: "/fields/System.Title", value: title },
-			{ op: "add", path: "/fields/System.Description", value: description },
-		];
-		const raw = await this._req(
-			`${this._witBase()}/workitems/$Issue`,
-			{
-				method: "POST",
-				headers: { "Content-Type": "application/json-patch+json" },
-				body: JSON.stringify(patch),
-			},
-		);
-		return this._issue(raw);
-	}
-
-	async updateIssue(iid, { title, description }) {
-		const patch = [];
-		if (title !== undefined) patch.push({ op: "replace", path: "/fields/System.Title", value: title });
-		if (description !== undefined) patch.push({ op: "replace", path: "/fields/System.Description", value: description });
-		const raw = await this._req(
-			`${this._witBase()}/workitems/${iid}`,
-			{
-				method: "PATCH",
-				headers: { "Content-Type": "application/json-patch+json" },
-				body: JSON.stringify(patch),
-			},
-		);
-		return this._issue(raw);
-	}
-
-	async closeIssue(iid) {
-		const patch = [{ op: "replace", path: "/fields/System.State", value: "Closed" }];
-		const raw = await this._req(
-			`${this._witBase()}/workitems/${iid}`,
-			{
-				method: "PATCH",
-				headers: { "Content-Type": "application/json-patch+json" },
-				body: JSON.stringify(patch),
-			},
-		);
-		return this._issue(raw);
-	}
-
-	// ─── Repository ───────────────────────────────────────────────────────────
-
-	async getFileTree(srcPath = "", ref = this._defaultBranch) {
-		const scopeParam = srcPath ? `&$scopePath=/${srcPath}` : "";
-		const data = await this._req(
-			`${this._gitBase()}/trees/${ref}?recursive=true${scopeParam}`,
-		);
-		return (data?.treeEntries ?? [])
-			.filter((item) => item.gitObjectType === "blob")
-			.filter((item) => /\.(ts|tsx|js|jsx|json|css|yaml|yml|md|go|py|rs|java|rb|sh|toml)$/.test(item.relativePath))
-			.map((item) => item.relativePath);
-	}
-
-	async getFileContent(filePath, ref = this._defaultBranch) {
-		const encoded = encodeURIComponent(filePath);
-		const url = `${this._gitBase()}/items?path=${encoded}&versionDescriptor.version=${ref}&versionDescriptor.versionType=branch&$format=text&api-version=7.1`;
-		const res = await fetch(url, { headers: this._headers() });
-		if (!res.ok) {
-			const text = await res.text().catch(() => "");
-			throw new Error(`AzureDevOps ${res.status} on items/${filePath}: ${text}`);
-		}
-		return res.text();
-	}
-
-	// ─── Mutations ────────────────────────────────────────────────────────────
-
-	async createBranch(branchName, ref = this._defaultBranch) {
-		// Get current SHA for ref
-		const refs = await this._req(`${this._gitBase()}/refs?filter=heads/${ref}`);
-		const sha = refs?.value?.[0]?.objectId;
-		if (!sha) throw new Error(`AzureDevOps: could not resolve ref ${ref}`);
-
-		return this._req(`${this._gitBase()}/refs`, {
-			method: "POST",
-			body: JSON.stringify([
-				{
-					name: `refs/heads/${branchName}`,
-					newObjectId: sha,
-					oldObjectId: "0000000000000000000000000000000000000000",
-				},
-			]),
-		});
-	}
-
-	async commitFiles(branch, message, actions) {
-		const changes = actions.map((action) => ({
-			changeType: action.action === "create" ? "add" : action.action === "delete" ? "delete" : "edit",
-			item: { path: `/${action.file_path}` },
-			newContent: action.action !== "delete"
-				? { content: action.content, contentType: "rawtext" }
-				: undefined,
-		}));
-
-		// Get current commit SHA
-		const refs = await this._req(`${this._gitBase()}/refs?filter=heads/${branch}`);
-		const oldSha = refs?.value?.[0]?.objectId ?? "0000000000000000000000000000000000000000";
-
-		return this._req(`${this._gitBase()}/pushes`, {
-			method: "POST",
-			body: JSON.stringify({
-				refUpdates: [{ name: `refs/heads/${branch}`, oldObjectId: oldSha }],
-				commits: [{ comment: message, changes }],
-			}),
-		});
-	}
-
-	async createMR({ sourceBranch, targetBranch = this._defaultBranch, title, description = "" }) {
-		const raw = await this._req(`${this._gitBase()}/pullrequests`, {
-			method: "POST",
-			body: JSON.stringify({
-				sourceRefName: `refs/heads/${sourceBranch}`,
-				targetRefName: `refs/heads/${targetBranch}`,
-				title,
-				description,
-			}),
-		});
-		return this._mr(raw);
-	}
-
-	// ─── PR queries ───────────────────────────────────────────────────────────
-
-	async listAllForgeMRs(state = "open") {
-		const adoStatus =
-			state === "merged" ? "completed"
-				: state === "closed" ? "abandoned"
-					: "active";
-		const data = await this._req(
-			`${this._gitBase()}/pullrequests?searchCriteria.status=${adoStatus}&$top=50`,
-		);
-		return (data?.value ?? [])
-			.filter((pr) => pr.sourceRefName?.includes("refs/heads/forge/"))
-			.map((pr) => this._mr(pr));
-	}
-
-	async listForgeMRsForIssue(issueIid) {
-		const data = await this._req(
-			`${this._gitBase()}/pullrequests?searchCriteria.status=active&$top=50`,
-		);
-		return (data?.value ?? [])
-			.filter((pr) => pr.sourceRefName?.includes(`refs/heads/forge/issue-${issueIid}-`))
-			.map((pr) => this._mr(pr));
-	}
-
-	async listMergedForgeMRsForIssue(issueIid) {
-		const data = await this._req(
-			`${this._gitBase()}/pullrequests?searchCriteria.status=completed&$top=50`,
-		);
-		return (data?.value ?? [])
-			.filter((pr) => pr.sourceRefName?.includes(`refs/heads/forge/issue-${issueIid}-`))
-			.map((pr) => this._mr(pr));
-	}
-
-	async getMR(mrIid) {
-		const raw = await this._req(`${this._gitBase()}/pullrequests/${mrIid}`);
-		return this._mr(raw);
-	}
-
-	async getMRDiffs(mrIid) {
-		const data = await this._req(
-			`${this._gitBase()}/pullrequests/${mrIid}/iterations/1/changes`,
-		);
-		return (data?.changeEntries ?? []).map((c) => this._diff(c));
-	}
-
-	// ─── CI / Pipelines ───────────────────────────────────────────────────────
-
-	async getMRPipelines(mrIid) {
-		// Get PR statuses which link to builds
-		const data = await this._req(
-			`${this._gitBase()}/pullrequests/${mrIid}/statuses`,
-		);
-		const buildIds = (data?.value ?? [])
-			.map((s) => s.targetUrl?.match(/buildId=(\d+)/)?.[1])
-			.filter(Boolean)
-			.map(Number);
-
-		if (buildIds.length === 0) return [];
-		return Promise.all(buildIds.map((id) => this.getPipeline(id)));
-	}
-
-	async getPipeline(pipelineId) {
-		const raw = await this._req(`${this._buildBase()}/builds/${pipelineId}`);
-		return this._pipeline(raw);
-	}
-
-	async getPipelineJobs(pipelineId) {
-		const data = await this._req(`${this._buildBase()}/builds/${pipelineId}/timeline`);
-		return (data?.records ?? [])
-			.filter((r) => r.type === "Task")
-			.map((r) => this._job(r));
-	}
-
-	async getJobTrace(jobId) {
-		const url = `${this._buildBase()}/builds/${jobId}/logs/1?api-version=7.1`;
-		const res = await fetch(url, { headers: this._headers() });
-		if (!res.ok) {
-			const text = await res.text().catch(() => "");
-			throw new Error(`AzureDevOps ${res.status} on builds/${jobId}/logs: ${text}`);
-		}
-		return res.text();
-	}
-}