Skip to content

⚡ refactor: optimize vibe-check-runner for concurrent processing and CI safety #321

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
beginwebdev2002 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

⚡ refactor: optimize vibe-check-runner for concurrent processing and CI safety #321

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
84 changes: 61 additions & 23 deletions vibe-check-runner.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,13 @@ import fsPromises from 'node:fs/promises';
import path from 'node:path';
import { execFileSync } from 'node:child_process';
import { GoogleGenAI } from '@google/genai';
import { pathToFileURL } from 'node:url';

const ai = new GoogleGenAI({ apiKey: process.env.GOOGLE_AI_API_KEY });

// Constants for scoring

async function fileOrDirExists(filePath) {
export async function fileOrDirExists(filePath) {
try {
await fsPromises.stat(filePath);
return true;
Expand All @@ -25,7 +26,7 @@ const SCORES = {
EFFICIENCY: 10,
};

function getModifiedFiles() {
export function getModifiedFiles() {
try {
// In CI (daily run), check files modified in the last 24 hours.
// We filter for non-empty lines that end in .md and are in frontend/ or backend/
Expand All @@ -43,7 +44,7 @@ function getModifiedFiles() {
}
}

async function syncBenchmarks(tech, mdContent, retries = 5, delay = 10000) {
export async function syncBenchmarks(tech, mdContent, retries = 5, delay = 10000) {
try {
const prompt = `Based on the following documentation:\n\n${mdContent}\n\n1. Generate a "Golden Prompt" (a comprehensive instruction for generating a typical module using this technology) in JSON format: {"golden_prompt": "...", "tech": "${tech}"}\n2. Generate a JSON Schema for TS-Morph AST validation rules enforcing DDD/FSD layers and strict typing for this technology. The generated JSON schema must explicitly follow a nested structure compatible with \`analyzeAST\`. Format: {"$schema": "...", "type": "object", "properties": {"forbidden_types": {"contains": {"enum": ["any"]}}}}.\n\nRespond strictly with ONLY a JSON array containing these two objects in order. No markdown wrappers.`;
const response = await ai.models.generateContent({
Expand Down Expand Up @@ -85,7 +86,7 @@ async function syncBenchmarks(tech, mdContent, retries = 5, delay = 10000) {
}
}

async function simulateAIGeneration(goldenPrompt, tech, mdContent, retries = 5, delay = 10000) {
export async function simulateAIGeneration(goldenPrompt, tech, mdContent, retries = 5, delay = 10000) {
try {
const prompt = `${goldenPrompt}\n\nConstraints and instructions from the following documentation:\n\n${mdContent}\n\nGenerate ONLY raw code. No markdown formatting, no explanations.`;
const response = await ai.models.generateContent({
Expand All @@ -110,7 +111,7 @@ async function simulateAIGeneration(goldenPrompt, tech, mdContent, retries = 5,
}
}

function analyzeAST(sourceFile, tech) {
export function analyzeAST(sourceFile, tech) {
let score = {
arch: SCORES.ARCH,
type: SCORES.TYPE,
Expand Down Expand Up @@ -222,7 +223,7 @@ function analyzeAST(sourceFile, tech) {
return { total, breakdown: score };
}

async function runVibeCheck() {
export async function runVibeCheck() {
console.log('Running Vibe-Check Runner...');

const modifiedFiles = getModifiedFiles();
Expand All @@ -241,12 +242,15 @@ async function runVibeCheck() {
console.warn('Failed to configure git user. If running locally, this is expected.');
}

for (const file of modifiedFiles) {
const filesByTech = new Map();

// Read all files concurrently and group by tech
await Promise.all(modifiedFiles.map(async (file) => {
console.log(`Processing ${file}...`);

if (!fs.existsSync(file)) {
if (!await fileOrDirExists(file)) {
console.log(`File ${file} does not exist. Skipping.`);
continue;
return;
}

let tech = '';
Expand All @@ -261,39 +265,71 @@ async function runVibeCheck() {
if (parts.length > 1) {
tech = parts[1];
} else {
continue;
return;
}
}

const mdContent = await fsPromises.readFile(file, 'utf-8');

await syncBenchmarks(tech, mdContent);
if (!filesByTech.has(tech)) {
filesByTech.set(tech, []);
}
filesByTech.get(tech).push({ file, mdContent });
}));

// Sync benchmarks sequentially per tech to deduplicate
const techMdContents = new Map();
for (const [tech, files] of filesByTech.entries()) {
const combinedContent = files.map(f => f.mdContent).join('\n\n---\n\n');
techMdContents.set(tech, combinedContent);
await syncBenchmarks(tech, combinedContent);
}

const fileResults = [];

// Generate code and analyze AST concurrently
const processingPromises = [];
for (const [tech, files] of filesByTech.entries()) {
const suitePath = path.join('benchmarks', 'suites', `${tech}.json`);
if (!fs.existsSync(suitePath)) {
console.log(`No benchmark suite found for ${tech}. Skipping.`);

if (!await fileOrDirExists(suitePath)) {
console.log(`No benchmark suite found for ${tech}. Skipping generation/analysis.`);
continue;
}

const suiteConfig = JSON.parse(await fsPromises.readFile(suitePath, 'utf-8'));

const generatedCode = await simulateAIGeneration(suiteConfig.golden_prompt, tech, mdContent);
for (const { file, mdContent } of files) {
processingPromises.push((async () => {
const generatedCode = await simulateAIGeneration(suiteConfig.golden_prompt, tech, mdContent);

if (!generatedCode) {
console.error(`Failed to generate code for ${tech}.`);
continue;
if (!generatedCode) {
console.error(`Failed to generate code for ${tech} (File: ${file}).`);
return null;
}

const sourceFile = project.createSourceFile(`temp_${tech}_${Buffer.from(file).toString('base64').replace(/=/g, '')}.ts`, generatedCode, { overwrite: true });
const { total: score, breakdown } = analyzeAST(sourceFile, tech);

console.log(`Fidelity Score for ${file}: ${score}%`);
console.log(`Breakdown:`, breakdown);

return { file, score, breakdown, generatedCode, mdContent };
})());
}
}

const sourceFile = project.createSourceFile(`temp_${tech}.ts`, generatedCode, { overwrite: true });
const { total: score, breakdown } = analyzeAST(sourceFile, tech);
const results = await Promise.all(processingPromises);

console.log(`Fidelity Score for ${file}: ${score}%`);
console.log(`Breakdown:`, breakdown);
// Handle side-effects sequentially
for (const result of results) {
if (!result) continue;
const { file, score, breakdown, generatedCode, mdContent } = result;

if (score >= 95) {
console.log(`✅ Validation passed for ${file}. Updating badge and auto-committing.`);

let content = await fsPromises.readFile(file, 'utf-8');
let content = mdContent;
if (!content.includes('[![Vibe-Coding Verified]')) {
content = content.replace(/^# /, '[![Vibe-Coding Verified](https://img.shields.io/badge/Vibe--Coding-Verified-brightgreen?style=for-the-badge)](#)\n\n# ');
await fsPromises.writeFile(file, content);
Expand Down Expand Up @@ -338,4 +374,6 @@ async function runVibeCheck() {
}
}

runVibeCheck().catch(console.error);
if (import.meta.url === pathToFileURL(process.argv[1]).href) {
runVibeCheck().catch(console.error);
}
Loading