agent-rules-init 0.3.0 → 0.4.0

package/dist/packs/js-ts.js

@@ -1,4 +1,5 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
+import { canonicalOf } from "../core/canonical-commands.js";
 const FRAMEWORKS = {
     next: "next",
     "@nestjs/core": "nestjs",
@@ -26,34 +27,66 @@ function detectFromDeps(deps, table) {
     }
     return undefined;
 }
+function detectPackageManagerFromCi(signals) {
+    const managers = new Set();
+    for (const workflow of signals.githubWorkflows ?? []) {
+        for (const rawLine of workflow.content.split(/\r?\n/)) {
+            const command = rawLine.trim().replace(/^(?:-\s*)?run:\s*(?:[|>]\s*)?/, "").trim();
+            const manager = /^(npm|pnpm|yarn|bun)\b/.exec(command)?.[1];
+            if (manager)
+                managers.add(manager);
+        }
+    }
+    return managers.size === 1 ? { value: [...managers][0], confidence: "high" } : undefined;
+}
 function detect(signals) {
     if (!signals.packageJson)
         return null;
     const allDeps = { ...signals.packageJson.dependencies, ...signals.packageJson.devDependencies };
+    const hasAnyFileNamed = (name) => signals.files.some((file) => file.split(/[\\/]/).pop() === name);
+    const frameworkSource = signals.packageJson.name
+        ? FRAMEWORKS[signals.packageJson.name]
+        : undefined;
     const framework = detectFromDeps(allDeps, FRAMEWORKS) ?? {
         value: "none",
-        confidence: "low",
+        confidence: frameworkSource ? "high" : "low",
     };
     const testRunner = detectFromDeps(allDeps, TEST_RUNNERS) ?? {
         value: "unknown",
         confidence: "low",
     };
     const linter = detectFromDeps(allDeps, LINTERS) ?? { value: "unknown", confidence: "low" };
-    const packageManager = signals.hasFile("pnpm-lock.yaml")
-        ? { value: "pnpm", confidence: "high" }
-        : signals.hasFile("yarn.lock")
-            ? { value: "yarn", confidence: "high" }
-            : signals.hasFile("package-lock.json")
-                ? { value: "npm", confidence: "high" }
-                : { value: "npm", confidence: "low" };
-    const usesTypeScript = Boolean(allDeps.typescript) || signals.hasFile("tsconfig.json");
-    const moduleFormat = signals.packageJson.moduleType;
+    const packageManager = signals.packageJson.packageManager
+        ? { value: signals.packageJson.packageManager, confidence: "high" }
+        : signals.hasFile("bun.lock") || signals.hasFile("bun.lockb")
+            ? { value: "bun", confidence: "high" }
+            : signals.hasFile("pnpm-lock.yaml")
+                ? { value: "pnpm", confidence: "high" }
+                : signals.hasFile("yarn.lock")
+                    ? { value: "yarn", confidence: "high" }
+                    : signals.hasFile("package-lock.json") || signals.hasFile("npm-shrinkwrap.json")
+                        ? { value: "npm", confidence: "high" }
+                        : hasAnyFileNamed("bun.lock") || hasAnyFileNamed("bun.lockb")
+                            ? { value: "bun", confidence: "high" }
+                            : hasAnyFileNamed("pnpm-lock.yaml")
+                                ? { value: "pnpm", confidence: "high" }
+                                : hasAnyFileNamed("yarn.lock")
+                                    ? { value: "yarn", confidence: "high" }
+                                    : hasAnyFileNamed("package-lock.json") || hasAnyFileNamed("npm-shrinkwrap.json")
+                                        ? { value: "npm", confidence: "high" }
+                                        : detectPackageManagerFromCi(signals) ?? { value: "npm", confidence: "low" };
+    const usesTypeScript = Boolean(allDeps.typescript) || signals.hasFile("tsconfig.json") || hasAnyFileNamed("tsconfig.json");
+    const moduleTypes = new Set((signals.packageJsons?.length ? signals.packageJsons : [signals.packageJson]).map((p) => p.moduleType));
+    // Mixed ESM/CommonJS workspaces should not receive a repo-wide rule that is wrong
+    // for half of the packages.
+    const moduleFormat = moduleTypes.size === 1 ? [...moduleTypes][0] : undefined;
     return {
         packId: "js-ts",
         language: usesTypeScript ? "TypeScript" : "JavaScript",
         framework,
         testRunner,
         linter,
+        frameworkSource,
         packageManager,
         usesTypeScript,
         moduleFormat,
@@ -68,7 +101,8 @@ const TEXTS = {
             "Mantén los componentes/módulos pequeños y con una responsabilidad clara.",
             "Coloca los tests junto al código que prueban o en un directorio `test/` espejo, según lo que ya use el repo.",
         ],
-        reviewFocus: "errores de tipado, condiciones de carrera en async/await",
+        reviewFocusTs: "errores de tipado, condiciones de carrera en async/await",
+        reviewFocusJs: "condiciones de carrera en async/await",
         refactorExtra: "Respeta los tipos existentes.",
     },
     en: {
@@ -79,33 +113,102 @@ const TEXTS = {
             "Keep components/modules small and single-purpose.",
             "Place tests next to the code they cover or in a mirrored `test/` directory, following what the repo already uses.",
         ],
-        reviewFocus: "typing errors, async/await race conditions",
+        reviewFocusTs: "typing errors, async/await race conditions",
+        reviewFocusJs: "async/await race conditions",
         refactorExtra: "Respect the existing types.",
     },
 };
-function rules(detection, lang) {
+const FRAMEWORK_RISKS = {
+    express: {
+        es: "Presta especial atención al flujo de middleware, la propagación de errores con `next(err)` y el ciclo de vida de la respuesta.",
+        en: "Pay special attention to middleware control flow, error propagation through `next(err)` and response lifecycle handling.",
+    },
+    react: {
+        es: "Presta especial atención a las dependencias de hooks (`useEffect`), renders innecesarios y estado derivado.",
+        en: "Pay special attention to hook dependencies (`useEffect`), unnecessary re-renders and derived state.",
+    },
+    next: {
+        es: "Presta especial atención a la frontera servidor/cliente (`use client`), el data fetching y el caché de rutas.",
+        en: "Pay special attention to the server/client boundary (`use client`), data fetching and route caching.",
+    },
+    nestjs: {
+        es: "Presta especial atención a los scopes de providers, la inyección de dependencias y los pipes de validación.",
+        en: "Pay special attention to provider scopes, dependency injection and validation pipes.",
+    },
+    fastify: {
+        es: "Presta especial atención al ciclo de vida de plugins, la encapsulación y los schemas de validación.",
+        en: "Pay special attention to plugin lifecycle, encapsulation and validation schemas.",
+    },
+};
+function rules(detection, lang, ctx) {
     const t = TEXTS[lang];
     const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
     const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
+    const testCmd = canonicalOf(ctx, "test", "js-ts")?.command ?? runner;
     const conventions = [];
     if (detection.usesTypeScript)
         conventions.push(t.tsStrict);
-    conventions.push(runTestsConvention(lang, runner ?? unknownRunnerLabel(lang)));
-    conventions.push(detection.moduleFormat === "module" ? t.esModules : t.commonJs);
+    conventions.push(runTestsConvention(lang, testCmd));
+    if (detection.moduleFormat) {
+        conventions.push(detection.moduleFormat === "module" ? t.esModules : t.commonJs);
+    }
     return {
-        summary: summarySentence(lang, detection.usesTypeScript ? "TypeScript" : "JavaScript", framework),
+        summary: detection.frameworkSource
+            ? lang === "es"
+                ? `Repositorio ${detection.usesTypeScript ? "TypeScript" : "JavaScript"} que implementa ${detection.frameworkSource}.`
+                : `${detection.usesTypeScript ? "TypeScript" : "JavaScript"} repository implementing ${detection.frameworkSource}.`
+            : summarySentence(lang, detection.usesTypeScript ? "TypeScript" : "JavaScript", framework),
         conventions,
         architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection, lang) {
+function promptTemplates(detection, lang, ctx) {
     const t = TEXTS[lang];
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
-    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
+    const focus = detection.usesTypeScript ? t.reviewFocusTs : t.reviewFocusJs;
+    const test = canonicalOf(ctx, "test", "js-ts");
+    const lint = canonicalOf(ctx, "lint", "js-ts");
+    const testDirs = ctx?.facts.testDirs ?? [];
+    const es = lang === "es";
+    const reviewParts = [];
+    const moduleLabel = detection.moduleFormat === "commonjs" ? "CommonJS" : detection.moduleFormat === "module" ? "ESM" : undefined;
+    reviewParts.push(es
+        ? `Revisa el diff actual contra las convenciones${moduleLabel ? ` ${moduleLabel}` : ""} de este repositorio${framework ? ` (${framework})` : ""}.`
+        : `Review the current diff against this repository's${moduleLabel ? ` ${moduleLabel}` : ""} conventions${framework ? ` (${framework})` : ""}.`);
+    if (test || lint) {
+        const commands = [test, lint].filter((c) => c !== undefined);
+        const list = commands.map((c) => `\`${c.command}\``).join(es ? " y " : " and ");
+        reviewParts.push(es ? `Ejecuta ${list} antes de dar por buena la revisión.` : `Run ${list} before approving the review.`);
+    }
+    const risk = FRAMEWORK_RISKS[framework ?? detection.frameworkSource ?? ""]?.[lang];
+    if (risk)
+        reviewParts.push(risk);
+    reviewParts.push(es ? `Busca también bugs: ${focus}.` : `Also look for bugs: ${focus}.`);
+    if (testDirs.length > 0) {
+        const dirs = testDirs.map((d) => `\`${d}\``).join(", ");
+        reviewParts.push(es ? `Los tests viven en ${dirs}.` : `Tests live under ${dirs}.`);
+    }
+    reviewParts.push(es
+        ? "Señala solo hallazgos concretos con archivo y línea."
+        : "Report only concrete findings with file and line references.");
+    const testingParts = [];
+    testingParts.push(testingBody(lang, runner));
+    if (test) {
+        testingParts.push(es ? `Verifica la suite con \`${test.command}\` antes de terminar.` : `Verify the suite with \`${test.command}\` before finishing.`);
+    }
+    if (testDirs.length > 0) {
+        const dirs = testDirs.map((d) => `\`${d}\``).join(", ");
+        testingParts.push(es ? `Coloca los tests nuevos en ${dirs}.` : `Place new tests under ${dirs}.`);
+    }
     return [
-        { id: "review", title: "Code Review (JS/TS)", body: reviewBody(lang, t.reviewFocus, framework) },
-        { id: "refactor", title: "Refactor (JS/TS)", body: refactorBody(lang, t.refactorExtra) },
-        { id: "testing", title: "Testing (JS/TS)", body: testingBody(lang, runner) },
+        {
+            id: "review",
+            title: "Code Review (JS/TS)",
+            body: ctx ? reviewParts.join(" ") : reviewBody(lang, focus, framework),
+        },
+        { id: "refactor", title: "Refactor (JS/TS)", body: refactorBody(lang, detection.usesTypeScript ? t.refactorExtra : undefined) },
+        { id: "testing", title: "Testing (JS/TS)", body: ctx ? testingParts.join(" ") : testingBody(lang, runner) },
     ];
 }
 export const jsTsPack = { id: "js-ts", detect, rules, promptTemplates };

package/dist/packs/kotlin.js

@@ -1,13 +1,17 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
 function detect(signals) {
-    const source = signals.buildGradle;
+    const gradleSource = signals.buildGradle && /\bkotlin\b/i.test(signals.buildGradle)
+        ? signals.buildGradle
+        : undefined;
+    const mavenSource = signals.pomXml && /\bkotlin\b/i.test(signals.pomXml)
+        ? signals.pomXml
+        : undefined;
+    const source = gradleSource ?? mavenSource;
     if (!source)
         return null;
     // Modern Gradle projects often declare the Kotlin plugin via a version catalog alias
     // (`alias(libs.plugins.kotlin.android)`) rather than the literal `kotlin("jvm")` or
     // `org.jetbrains.kotlin` plugin id, so a plain word-boundary check is more robust.
-    if (!/\bkotlin\b/i.test(source))
-        return null;
     const framework = /ktor/i.test(source)
         ? { value: "ktor", confidence: "high" }
         : /com\.android\.application|com\.android\.library/i.test(source)
@@ -25,7 +29,13 @@ function detect(signals) {
         language: "Kotlin",
         framework,
         testRunner,
-        packageManager: { value: "gradle", confidence: "high" },
+        packageManager: mavenSource && !gradleSource
+            ? signals.hasFile("mvnw") || signals.hasFile("mvnw.cmd")
+                ? { value: "maven wrapper", confidence: "high" }
+                : { value: "maven", confidence: "high" }
+            : signals.hasFile("gradlew") || signals.hasFile("gradlew.bat")
+                ? { value: "gradle wrapper", confidence: "high" }
+                : { value: "gradle", confidence: "high" },
     };
 }
 const TEXTS = {
@@ -52,15 +62,25 @@ function rules(detection, lang) {
     const t = TEXTS[lang];
     const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
     return {
-        summary: summarySentence(lang, "Kotlin", framework, "gradle"),
-        conventions: [t.style, runTestsConvention(lang, "`gradle test`"), t.nullsafety],
+        summary: summarySentence(lang, "Kotlin", framework, detection.packageManager?.value),
+        conventions: [
+            t.style,
+            runTestsConvention(lang, detection.packageManager?.value === "maven wrapper"
+                ? "`./mvnw test`"
+                : detection.packageManager?.value === "maven"
+                    ? "`mvn test`"
+                    : detection.packageManager?.value === "gradle wrapper"
+                        ? "`./gradlew test`"
+                        : "`gradle test`"),
+            t.nullsafety,
+        ],
         architectureNotes: t.arch,
     };
 }
 function promptTemplates(detection, lang) {
     const t = TEXTS[lang];
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
-    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
     return [
         { id: "review", title: "Code Review (Kotlin)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Kotlin)", body: refactorBody(lang) },

package/dist/packs/php.js

@@ -1,4 +1,4 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
 const FRAMEWORKS = {
     "laravel/framework": "laravel",
     "symfony/symfony": "symfony",
@@ -47,7 +47,7 @@ const TEXTS = {
 function rules(detection, lang) {
     const t = TEXTS[lang];
     const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
-    const testCmd = detection.testRunner?.value === "phpunit" ? "vendor/bin/phpunit" : unknownRunnerLabel(lang);
+    const testCmd = detection.testRunner?.value === "phpunit" ? "vendor/bin/phpunit" : undefined;
     return {
         summary: summarySentence(lang, "PHP", framework, "composer"),
         conventions: [t.style, runTestsConvention(lang, testCmd), t.deps],
@@ -55,8 +55,8 @@ function rules(detection, lang) {
     };
 }
 function promptTemplates(detection, lang) {
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
-    const runner = detection.testRunner?.value === "phpunit" ? "PHPUnit" : unknownRunnerLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value === "phpunit" ? "PHPUnit" : undefined;
     return [
         { id: "review", title: "Code Review (PHP)", body: reviewBody(lang, "", framework) },
         { id: "refactor", title: "Refactor (PHP)", body: refactorBody(lang) },

package/dist/packs/python.js

@@ -1,4 +1,5 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
+import { canonicalOf } from "../core/canonical-commands.js";
 const FRAMEWORKS = [
     ["fastapi", "fastapi"],
     ["django", "django"],
@@ -35,21 +36,36 @@ function extractPyprojectDependencySections(pyproject) {
         sections.push(...poetryDepsBlocks);
     return sections.length > 0 ? sections.join("\n") : pyproject;
 }
+function frameworkSourceProject(pyproject) {
+    const projectBlock = pyproject.match(/(?:^|\n)\[project\]([\s\S]*?)(?:\n\[|$)/)?.[1];
+    const name = projectBlock?.match(/(?:^|\n)\s*name\s*=\s*["']([^"']+)["']/i)?.[1].toLowerCase();
+    return FRAMEWORKS.find(([framework]) => framework === name)?.[1];
+}
 function detect(signals) {
     const source = signals.pyprojectToml ?? signals.requirementsTxt ?? signals.environmentYml;
     if (!source)
         return null;
     const searchText = signals.pyprojectToml ? extractPyprojectDependencySections(signals.pyprojectToml) : source;
-    const framework = findIn(searchText, FRAMEWORKS) ?? { value: "none", confidence: "low" };
+    const frameworkSource = signals.pyprojectToml
+        ? frameworkSourceProject(signals.pyprojectToml)
+        : undefined;
+    const framework = findIn(searchText, FRAMEWORKS) ?? {
+        value: "none",
+        confidence: frameworkSource
+            ? "high"
+            : "low",
+    };
     const testRunner = findIn(searchText, TEST_RUNNERS) ?? { value: "unknown", confidence: "low" };
     const packageManager = signals.environmentYml
         ? { value: "conda", confidence: "high" }
-        : signals.hasFile("poetry.lock")
-            ? { value: "poetry", confidence: "high" }
-            : signals.pyprojectToml
-                ? { value: "pip (pyproject.toml)", confidence: "low" }
-                : { value: "pip", confidence: "low" };
-    return { packId: "python", language: "Python", framework, testRunner, packageManager };
+        : signals.hasFile("uv.lock")
+            ? { value: "uv", confidence: "high" }
+            : signals.hasFile("poetry.lock")
+                ? { value: "poetry", confidence: "high" }
+                : signals.pyprojectToml
+                    ? { value: "pip (pyproject.toml)", confidence: "low" }
+                    : { value: "pip", confidence: "low" };
+    return { packId: "python", language: "Python", framework, frameworkSource, testRunner, packageManager };
 }
 const TEXTS = {
     es: {
@@ -73,24 +89,73 @@ const TEXTS = {
         refactorExtra: "Respect the existing type hints.",
     },
 };
-function rules(detection, lang) {
+const FRAMEWORK_RISKS = {
+    flask: {
+        es: "Presta especial atención al contexto de aplicación/petición, los blueprints y el manejo de errores HTTP.",
+        en: "Pay special attention to application/request context, blueprints and HTTP error handling.",
+    },
+    django: {
+        es: "Presta especial atención a migraciones pendientes, consultas N+1 del ORM y validación en forms/serializers.",
+        en: "Pay special attention to pending migrations, ORM N+1 queries and validation in forms/serializers.",
+    },
+    fastapi: {
+        es: "Presta especial atención a los modelos Pydantic, las dependencias async y los códigos de respuesta declarados.",
+        en: "Pay special attention to Pydantic models, async dependencies and declared response codes.",
+    },
+};
+function rules(detection, lang, ctx) {
     const t = TEXTS[lang];
     const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
     const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
+    const testCmd = canonicalOf(ctx, "test", "python")?.command ?? runner;
     return {
-        summary: summarySentence(lang, "Python", framework),
-        conventions: [t.style, runTestsConvention(lang, runner ?? unknownRunnerLabel(lang)), t.deps],
+        summary: detection.frameworkSource
+            ? lang === "es"
+                ? `Repositorio Python que implementa ${detection.frameworkSource}.`
+                : `Python repository implementing ${detection.frameworkSource}.`
+            : summarySentence(lang, "Python", framework),
+        conventions: [t.style, runTestsConvention(lang, testCmd), t.deps],
         architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection, lang) {
+function promptTemplates(detection, lang, ctx) {
     const t = TEXTS[lang];
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
-    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
+    const test = canonicalOf(ctx, "test", "python");
+    const testDirs = ctx?.facts.testDirs ?? [];
+    const hasTox = ctx?.facts.commands.some((c) => c.source === "tox") ?? false;
+    const es = lang === "es";
+    const reviewParts = [];
+    reviewParts.push(es
+        ? `Revisa el diff actual contra las convenciones Python de este repositorio${framework ? ` (${framework})` : ""}.`
+        : `Review the current diff against this repository's Python conventions${framework ? ` (${framework})` : ""}.`);
+    if (test) {
+        reviewParts.push(es ? `Ejecuta \`${test.command}\` antes de dar por buena la revisión.` : `Run \`${test.command}\` before approving the review.`);
+    }
+    if (hasTox) {
+        reviewParts.push(es ? "La matriz completa de entornos se ejecuta con tox (`tox.ini`)." : "The full environment matrix runs through tox (`tox.ini`).");
+    }
+    const risk = FRAMEWORK_RISKS[framework ?? detection.frameworkSource ?? ""]?.[lang];
+    if (risk)
+        reviewParts.push(risk);
+    reviewParts.push(es ? `Busca también bugs: ${t.reviewFocus}.` : `Also look for bugs: ${t.reviewFocus}.`);
+    if (testDirs.length > 0) {
+        const dirs = testDirs.map((d) => `\`${d}\``).join(", ");
+        reviewParts.push(es ? `Los tests viven en ${dirs}.` : `Tests live under ${dirs}.`);
+    }
+    reviewParts.push(es ? "Señala solo hallazgos concretos con archivo y línea." : "Report only concrete findings with file and line references.");
+    const testingParts = [testingBody(lang, runner)];
+    if (test)
+        testingParts.push(es ? `Verifica la suite con \`${test.command}\` antes de terminar.` : `Verify the suite with \`${test.command}\` before finishing.`);
+    if (testDirs.length > 0) {
+        const dirs = testDirs.map((d) => `\`${d}\``).join(", ");
+        testingParts.push(es ? `Coloca los tests nuevos en ${dirs}.` : `Place new tests under ${dirs}.`);
+    }
     return [
-        { id: "review", title: "Code Review (Python)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "review", title: "Code Review (Python)", body: ctx ? reviewParts.join(" ") : reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Python)", body: refactorBody(lang, t.refactorExtra) },
-        { id: "testing", title: "Testing (Python)", body: testingBody(lang, runner) },
+        { id: "testing", title: "Testing (Python)", body: ctx ? testingParts.join(" ") : testingBody(lang, runner) },
     ];
 }
 export const pythonPack = { id: "python", detect, rules, promptTemplates };

package/dist/packs/r.js

@@ -1,4 +1,4 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["shiny", "shiny"],
     ["plumber", "plumber"],
@@ -44,7 +44,7 @@ const TEXTS = {
 function rules(detection, lang) {
     const t = TEXTS[lang];
     const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
-    const testCmd = detection.testRunner?.value === "testthat" ? 'testthat::test_dir("tests")' : unknownRunnerLabel(lang);
+    const testCmd = detection.testRunner?.value === "testthat" ? 'testthat::test_dir("tests")' : undefined;
     return {
         summary: summarySentence(lang, "R", framework, detection.packageManager?.value),
         conventions: [t.style, runTestsConvention(lang, testCmd), t.deps],
@@ -52,8 +52,8 @@ function rules(detection, lang) {
     };
 }
 function promptTemplates(detection, lang) {
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
-    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
     return [
         { id: "review", title: "Code Review (R)", body: reviewBody(lang, "", framework) },
         { id: "refactor", title: "Refactor (R)", body: refactorBody(lang) },

package/dist/packs/ruby.js

@@ -1,4 +1,4 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.gemfile;
     if (!source)
@@ -40,25 +40,25 @@ const TEXTS = {
         ],
     },
 };
-function testCommand(detection, lang) {
+function testCommand(detection) {
     if (detection.testRunner?.value === "rspec")
         return "bundle exec rspec";
     if (detection.testRunner?.value === "minitest")
         return "bundle exec rake test";
-    return unknownRunnerLabel(lang);
+    return undefined;
 }
 function rules(detection, lang) {
     const t = TEXTS[lang];
     const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
     return {
         summary: summarySentence(lang, "Ruby", framework, "bundler"),
-        conventions: [t.style, runTestsConvention(lang, testCommand(detection, lang)), t.deps],
+        conventions: [t.style, runTestsConvention(lang, testCommand(detection)), t.deps],
         architectureNotes: t.arch,
     };
 }
 function promptTemplates(detection, lang) {
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
-    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
     return [
         { id: "review", title: "Code Review (Ruby)", body: reviewBody(lang, "", framework) },
         { id: "refactor", title: "Refactor (Ruby)", body: refactorBody(lang) },

package/dist/packs/rust.js

@@ -1,4 +1,4 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["actix-web", "actix-web"],
     ["axum", "axum"],
@@ -57,7 +57,7 @@ function rules(detection, lang) {
 }
 function promptTemplates(detection, lang) {
     const t = TEXTS[lang];
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
     return [
         { id: "review", title: "Code Review (Rust)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Rust)", body: refactorBody(lang, t.refactorExtra) },

package/dist/packs/scala.js

@@ -1,4 +1,4 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["playframework", "play"],
     ["akka-http", "akka"],
@@ -68,8 +68,8 @@ function rules(detection, lang) {
 }
 function promptTemplates(detection, lang) {
     const t = TEXTS[lang];
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
-    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
     return [
         { id: "review", title: "Code Review (Scala)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Scala)", body: refactorBody(lang) },

package/dist/packs/swift.js

@@ -1,4 +1,4 @@
-import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, } from "../core/i18n.js";
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.packageSwift;
     if (!source)
@@ -52,7 +52,7 @@ function rules(detection, lang) {
 }
 function promptTemplates(detection, lang) {
     const t = TEXTS[lang];
-    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
     return [
         { id: "review", title: "Code Review (Swift)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Swift)", body: refactorBody(lang) },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-rules-init",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "Generates CLAUDE.md, AGENTS.md, copilot-instructions.md and prompt files from what your repo actually is.",
   "license": "MIT",
   "type": "module",
@@ -10,11 +10,13 @@
     "agent-rules-init": "dist/bin.js"
   },
   "files": [
-    "dist"
+    "dist",
+    "README.md",
+    "LICENSE"
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/racama29/agent-rules-init.git",
+    "url": "git+https://github.com/racama29/agent-rules-init.git",
     "directory": "packages/cli"
   },
   "homepage": "https://github.com/racama29/agent-rules-init#readme",