agent-rules-init 0.2.1 → 0.3.0

package/dist/packs/dart.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.pubspecYaml;
     if (!source)
@@ -21,41 +22,39 @@ function detect(signals) {
         packageManager: { value: "pub", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
+const TEXTS = {
+    es: {
+        style: "Sigue la guía de estilo oficial de Dart (dart.dev/effective-dart/style).",
+        deps: "Declara toda dependencia nueva en pubspec.yaml, nunca la instales sin registrarla.",
+        archFlutter: "Separa la lógica de negocio de los widgets cuando el proyecto ya siga ese patrón (p. ej. BLoC/Provider/Riverpod).",
+        archPlain: "Mantén los módulos con una responsabilidad clara.",
+        immutability: "Prefiere `final`/`const` sobre variables mutables cuando sea posible.",
+    },
+    en: {
+        style: "Follow the official Dart style guide (dart.dev/effective-dart/style).",
+        deps: "Declare every new dependency in pubspec.yaml; never install one without registering it.",
+        archFlutter: "Keep business logic out of widgets when the project already follows that pattern (e.g. BLoC/Provider/Riverpod).",
+        archPlain: "Keep modules single-purpose.",
+        immutability: "Prefer `final`/`const` over mutable variables when possible.",
+    },
+};
+function rules(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
     return {
-        summary: `Proyecto Dart${framework !== "none" ? ` con ${framework}` : ""} (pub).`,
-        conventions: [
-            "Sigue la guía de estilo oficial de Dart (dart.dev/effective-dart/style).",
-            `Ejecuta los tests con ${detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : "el test runner del proyecto"} antes de terminar una tarea.`,
-            "Declara toda dependencia nueva en pubspec.yaml, nunca la instales sin registrarla.",
-        ],
-        architectureNotes: [
-            framework === "flutter"
-                ? "Separa la lógica de negocio de los widgets cuando el proyecto ya siga ese patrón (p. ej. BLoC/Provider/Riverpod)."
-                : "Mantén los módulos con una responsabilidad clara.",
-            "Prefiere `final`/`const` sobre variables mutables cuando sea posible.",
-        ],
+        summary: summarySentence(lang, "Dart", framework, "pub"),
+        conventions: [t.style, runTestsConvention(lang, runner ?? unknownRunnerLabel(lang)), t.deps],
+        architectureNotes: [framework === "flutter" ? t.archFlutter : t.archPlain, t.immutability],
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+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);
     return [
-        {
-            id: "review",
-            title: "Code Review (Dart)",
-            body: `Revisa el diff actual buscando bugs y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Dart)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Dart)",
-            body: `Escribe tests con ${detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : "el test runner del proyecto"} para el código señalado. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (Dart)", body: reviewBody(lang, "", framework) },
+        { id: "refactor", title: "Refactor (Dart)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (Dart)", body: testingBody(lang, runner) },
     ];
 }
 export const dartPack = { id: "dart", detect, rules, promptTemplates };

package/dist/packs/elixir.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.mixExs;
     if (!source)
@@ -18,39 +19,42 @@ function detect(signals) {
         packageManager: { value: "mix", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Elixir${framework !== "none" ? ` con ${framework}` : ""} (mix).`,
-        conventions: [
-            "Sigue la guía de estilo oficial de Elixir; ejecuta `mix format` antes de terminar una tarea.",
-            "Ejecuta los tests con `mix test` antes de terminar una tarea.",
-            "Prefiere pattern matching y pipelines (`|>`) sobre anidar llamadas de función.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue la guía de estilo oficial de Elixir; ejecuta `mix format` antes de terminar una tarea.",
+        patterns: "Prefiere pattern matching y pipelines (`|>`) sobre anidar llamadas de función.",
+        arch: [
             "Respeta la separación entre contextos (lógica de negocio) y la capa web si el proyecto ya usa Phoenix.",
             "Declara toda dependencia nueva en `mix.exs` y mantén `mix.lock` sincronizado.",
         ],
+        reviewFocus: "procesos sin supervisar correctamente",
+    },
+    en: {
+        style: "Follow the official Elixir style guide; run `mix format` before finishing a task.",
+        patterns: "Prefer pattern matching and pipelines (`|>`) over nesting function calls.",
+        arch: [
+            "Respect the separation between contexts (business logic) and the web layer if the project already uses Phoenix.",
+            "Declare every new dependency in `mix.exs` and keep `mix.lock` in sync.",
+        ],
+        reviewFocus: "improperly supervised processes",
+    },
+};
+function rules(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    return {
+        summary: summarySentence(lang, "Elixir", framework, "mix"),
+        conventions: [t.style, runTestsConvention(lang, "`mix test`"), t.patterns],
+        architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+function promptTemplates(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
     return [
-        {
-            id: "review",
-            title: "Code Review (Elixir)",
-            body: `Revisa el diff actual buscando bugs, procesos sin supervisar correctamente y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Elixir)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Elixir)",
-            body: `Escribe tests con \`mix test\` (ExUnit) para el código señalado. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (Elixir)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "refactor", title: "Refactor (Elixir)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (Elixir)", body: testingBody(lang, "`mix test` (ExUnit)") },
     ];
 }
 export const elixirPack = { id: "elixir", detect, rules, promptTemplates };

package/dist/packs/go.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["gin-gonic/gin", "gin"],
     ["labstack/echo", "echo"],
@@ -36,39 +37,42 @@ function detect(signals) {
         packageManager: { value: "go modules", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Go${framework !== "none" ? ` con ${framework}` : ""} (go modules).`,
-        conventions: [
-            "Sigue el formato estándar de `gofmt`; no introduzcas estilos alternativos.",
-            "Ejecuta los tests con `go test ./...` antes de terminar una tarea.",
-            "Maneja los errores explícitamente (`if err != nil`); no los ignores en silencio.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue el formato estándar de `gofmt`; no introduzcas estilos alternativos.",
+        errors: "Maneja los errores explícitamente (`if err != nil`); no los ignores en silencio.",
+        arch: [
             "Mantén los paquetes con una responsabilidad clara; evita paquetes `util` genéricos.",
             "Declara toda dependencia nueva vía `go get` y mantén `go.mod`/`go.sum` sincronizados.",
         ],
+        reviewFocus: "manejo de errores omitido",
+    },
+    en: {
+        style: "Follow the standard `gofmt` formatting; do not introduce alternative styles.",
+        errors: "Handle errors explicitly (`if err != nil`); never ignore them silently.",
+        arch: [
+            "Keep packages single-purpose; avoid generic `util` packages.",
+            "Declare every new dependency via `go get` and keep `go.mod`/`go.sum` in sync.",
+        ],
+        reviewFocus: "missing error handling",
+    },
+};
+function rules(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    return {
+        summary: summarySentence(lang, "Go", framework, "go modules"),
+        conventions: [t.style, runTestsConvention(lang, "`go test ./...`"), t.errors],
+        architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+function promptTemplates(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
     return [
-        {
-            id: "review",
-            title: "Code Review (Go)",
-            body: `Revisa el diff actual buscando bugs, manejo de errores omitido y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Go)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Go)",
-            body: `Escribe tests con \`go test\` para el código señalado. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (Go)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "refactor", title: "Refactor (Go)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (Go)", body: testingBody(lang, "`go test`") },
     ];
 }
 export const goPack = { id: "go", detect, rules, promptTemplates };

package/dist/packs/java.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.pomXml ?? signals.buildGradle;
     if (!source)
@@ -21,39 +22,44 @@ function detect(signals) {
         : { value: "unknown", confidence: "low" };
     return { packId: "java", language: "Java", framework, packageManager, testRunner };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Java${framework !== "none" ? ` con ${framework}` : ""} (${detection.packageManager?.value}).`,
-        conventions: [
-            "Sigue las convenciones de nombrado estándar de Java (PascalCase para clases, camelCase para métodos).",
-            `Ejecuta los tests con ${detection.packageManager?.value === "maven" ? "mvn test" : "gradle test"} antes de terminar una tarea.`,
-            "No añadas dependencias nuevas sin declararlas en el gestor de build existente.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        naming: "Sigue las convenciones de nombrado estándar de Java (PascalCase para clases, camelCase para métodos).",
+        deps: "No añadas dependencias nuevas sin declararlas en el gestor de build existente.",
+        arch: [
             "Respeta la separación en capas (controller/service/repository) si el proyecto ya la usa.",
             "Prefiere inyección de dependencias sobre instanciación manual cuando el framework ya la ofrezca.",
         ],
+        reviewFocus: "null-safety",
+    },
+    en: {
+        naming: "Follow standard Java naming conventions (PascalCase for classes, camelCase for methods).",
+        deps: "Do not add new dependencies without declaring them in the existing build tool.",
+        arch: [
+            "Respect the layered separation (controller/service/repository) if the project already uses it.",
+            "Prefer dependency injection over manual instantiation when the framework already provides it.",
+        ],
+        reviewFocus: "null-safety",
+    },
+};
+function rules(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const testCmd = detection.packageManager?.value === "maven" ? "mvn test" : "gradle test";
+    return {
+        summary: summarySentence(lang, "Java", framework, detection.packageManager?.value),
+        conventions: [t.naming, runTestsConvention(lang, testCmd), t.deps],
+        architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+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);
     return [
-        {
-            id: "review",
-            title: "Code Review (Java)",
-            body: `Revisa el diff actual buscando bugs, null-safety y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Java)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Java)",
-            body: `Escribe tests con ${detection.testRunner?.value ?? "el test runner del proyecto"} para el código señalado. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (Java)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "refactor", title: "Refactor (Java)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (Java)", body: testingBody(lang, runner) },
     ];
 }
 export const javaPack = { id: "java", detect, rules, promptTemplates };

package/dist/packs/js-ts.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 const FRAMEWORKS = {
     next: "next",
     "@nestjs/core": "nestjs",
@@ -58,44 +59,53 @@ function detect(signals) {
         moduleFormat,
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    const testRunner = detection.testRunner?.value ?? "unknown";
-    const conventions = [];
-    if (detection.usesTypeScript) {
-        conventions.push("Usa TypeScript estricto; evita `any` salvo justificación explícita.");
-    }
-    conventions.push(`Ejecuta los tests con ${testRunner === "unknown" ? "el test runner del proyecto" : testRunner} antes de dar por terminada una tarea.`);
-    conventions.push(detection.moduleFormat === "module"
-        ? "Sigue el estilo de módulos ES existente (import/export), no mezcles con require()."
-        : "Sigue el estilo CommonJS existente (require()/module.exports), no mezcles con import/export.");
-    return {
-        summary: `Proyecto ${detection.usesTypeScript ? "TypeScript" : "JavaScript"}${framework !== "none" ? ` con ${framework}` : ""}.`,
-        conventions,
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        tsStrict: "Usa TypeScript estricto; evita `any` salvo justificación explícita.",
+        esModules: "Sigue el estilo de módulos ES existente (import/export), no mezcles con require().",
+        commonJs: "Sigue el estilo CommonJS existente (require()/module.exports), no mezcles con import/export.",
+        arch: [
             "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",
+        refactorExtra: "Respeta los tipos existentes.",
+    },
+    en: {
+        tsStrict: "Use strict TypeScript; avoid `any` unless explicitly justified.",
+        esModules: "Follow the existing ES modules style (import/export); do not mix in require().",
+        commonJs: "Follow the existing CommonJS style (require()/module.exports); do not mix in import/export.",
+        arch: [
+            "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",
+        refactorExtra: "Respect the existing types.",
+    },
+};
+function rules(detection, lang) {
+    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 conventions = [];
+    if (detection.usesTypeScript)
+        conventions.push(t.tsStrict);
+    conventions.push(runTestsConvention(lang, runner ?? unknownRunnerLabel(lang)));
+    conventions.push(detection.moduleFormat === "module" ? t.esModules : t.commonJs);
+    return {
+        summary: summarySentence(lang, detection.usesTypeScript ? "TypeScript" : "JavaScript", framework),
+        conventions,
+        architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+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);
     return [
-        {
-            id: "review",
-            title: "Code Review (JS/TS)",
-            body: `Revisa el diff actual buscando bugs de tipado, condiciones de carrera en async/await, y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (JS/TS)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable. Respeta los tipos existentes.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (JS/TS)",
-            body: `Escribe tests para el código señalado usando el test runner detectado (${detection.testRunner?.value ?? "el del proyecto"}). Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { 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) },
     ];
 }
 export const jsTsPack = { id: "js-ts", detect, rules, promptTemplates };

package/dist/packs/kotlin.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.buildGradle;
     if (!source)
@@ -27,39 +28,43 @@ function detect(signals) {
         packageManager: { value: "gradle", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Kotlin${framework !== "none" ? ` con ${framework}` : ""} (gradle).`,
-        conventions: [
-            "Sigue las convenciones de estilo oficiales de Kotlin (kotlinlang.org/docs/coding-conventions.html).",
-            "Ejecuta los tests con `gradle test` antes de terminar una tarea.",
-            "Prefiere tipos no-nulos y `val` sobre `var` salvo que la mutabilidad sea necesaria.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue las convenciones de estilo oficiales de Kotlin (kotlinlang.org/docs/coding-conventions.html).",
+        nullsafety: "Prefiere tipos no-nulos y `val` sobre `var` salvo que la mutabilidad sea necesaria.",
+        arch: [
             "Respeta la separación en capas (controller/service/repository) si el proyecto ya la usa.",
             "Prefiere inyección de dependencias sobre instanciación manual cuando el framework ya la ofrezca.",
         ],
+        reviewFocus: "uso innecesario de `!!`",
+    },
+    en: {
+        style: "Follow the official Kotlin style conventions (kotlinlang.org/docs/coding-conventions.html).",
+        nullsafety: "Prefer non-null types and `val` over `var` unless mutability is required.",
+        arch: [
+            "Respect the layered separation (controller/service/repository) if the project already uses it.",
+            "Prefer dependency injection over manual instantiation when the framework already provides it.",
+        ],
+        reviewFocus: "unnecessary `!!` usage",
+    },
+};
+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],
+        architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+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);
     return [
-        {
-            id: "review",
-            title: "Code Review (Kotlin)",
-            body: `Revisa el diff actual buscando bugs, uso innecesario de \`!!\` y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Kotlin)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Kotlin)",
-            body: `Escribe tests con ${detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : "el test runner del proyecto"} para el código señalado. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (Kotlin)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "refactor", title: "Refactor (Kotlin)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (Kotlin)", body: testingBody(lang, runner) },
     ];
 }
 export const kotlinPack = { id: "kotlin", detect, rules, promptTemplates };

package/dist/packs/php.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 const FRAMEWORKS = {
     "laravel/framework": "laravel",
     "symfony/symfony": "symfony",
@@ -25,39 +26,41 @@ function detect(signals) {
         packageManager: { value: "composer", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto PHP${framework !== "none" ? ` con ${framework}` : ""} (composer).`,
-        conventions: [
-            "Sigue PSR-12 para el estilo de código.",
-            `Ejecuta los tests con ${detection.testRunner?.value === "phpunit" ? "vendor/bin/phpunit" : "el test runner del proyecto"} antes de terminar una tarea.`,
-            "Declara toda dependencia nueva en composer.json, nunca la instales sin registrarla.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue PSR-12 para el estilo de código.",
+        deps: "Declara toda dependencia nueva en composer.json, nunca la instales sin registrarla.",
+        arch: [
             "Respeta la estructura MVC del framework si el proyecto ya la sigue.",
             "Evita lógica de negocio en los controladores cuando el proyecto ya use capas de servicio.",
         ],
+    },
+    en: {
+        style: "Follow PSR-12 for code style.",
+        deps: "Declare every new dependency in composer.json; never install one without registering it.",
+        arch: [
+            "Respect the framework's MVC structure if the project already follows it.",
+            "Avoid business logic in controllers when the project already uses service layers.",
+        ],
+    },
+};
+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);
+    return {
+        summary: summarySentence(lang, "PHP", framework, "composer"),
+        conventions: [t.style, runTestsConvention(lang, testCmd), t.deps],
+        architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+function promptTemplates(detection, lang) {
+    const framework = detection.framework?.value !== "none" ? detection.framework.value : unknownFrameworkLabel(lang);
+    const runner = detection.testRunner?.value === "phpunit" ? "PHPUnit" : unknownRunnerLabel(lang);
     return [
-        {
-            id: "review",
-            title: "Code Review (PHP)",
-            body: `Revisa el diff actual buscando bugs y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (PHP)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (PHP)",
-            body: `Escribe tests con ${detection.testRunner?.value === "phpunit" ? "PHPUnit" : "el test runner del proyecto"} para el código señalado. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (PHP)", body: reviewBody(lang, "", framework) },
+        { id: "refactor", title: "Refactor (PHP)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (PHP)", body: testingBody(lang, runner) },
     ];
 }
 export const phpPack = { id: "php", detect, rules, promptTemplates };

package/dist/packs/python.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["fastapi", "fastapi"],
     ["django", "django"],
@@ -50,39 +51,46 @@ function detect(signals) {
                 : { value: "pip", confidence: "low" };
     return { packId: "python", language: "Python", framework, testRunner, packageManager };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Python${framework !== "none" ? ` con ${framework}` : ""}.`,
-        conventions: [
-            "Sigue PEP 8; usa type hints en funciones públicas.",
-            `Ejecuta los tests con ${detection.testRunner?.value ?? "el test runner del proyecto"} antes de terminar una tarea.`,
-            "No introduzcas dependencias nuevas sin añadirlas al manifiesto de dependencias existente.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue PEP 8; usa type hints en funciones públicas.",
+        deps: "No introduzcas dependencias nuevas sin añadirlas al manifiesto de dependencias existente.",
+        arch: [
             "Mantén la lógica de negocio separada de la capa de framework cuando el proyecto ya siga ese patrón.",
             "Usa entornos virtuales; no instales paquetes globalmente.",
         ],
+        reviewFocus: "manejo de excepciones incorrecto",
+        refactorExtra: "Respeta los type hints existentes.",
+    },
+    en: {
+        style: "Follow PEP 8; use type hints on public functions.",
+        deps: "Do not introduce new dependencies without adding them to the existing dependency manifest.",
+        arch: [
+            "Keep business logic separate from the framework layer when the project already follows that pattern.",
+            "Use virtual environments; never install packages globally.",
+        ],
+        reviewFocus: "incorrect exception handling",
+        refactorExtra: "Respect the existing type hints.",
+    },
+};
+function rules(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
+    return {
+        summary: summarySentence(lang, "Python", framework),
+        conventions: [t.style, runTestsConvention(lang, runner ?? unknownRunnerLabel(lang)), t.deps],
+        architectureNotes: t.arch,
     };
 }
-function promptTemplates(detection) {
-    const framework = detection.framework?.value ?? "el framework del proyecto";
+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);
     return [
-        {
-            id: "review",
-            title: "Code Review (Python)",
-            body: `Revisa el diff actual buscando bugs, manejo de excepciones incorrecto y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Python)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable. Respeta los type hints existentes.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Python)",
-            body: `Escribe tests para el código señalado usando ${detection.testRunner?.value ?? "el test runner del proyecto"}. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (Python)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "refactor", title: "Refactor (Python)", body: refactorBody(lang, t.refactorExtra) },
+        { id: "testing", title: "Testing (Python)", body: testingBody(lang, runner) },
     ];
 }
 export const pythonPack = { id: "python", detect, rules, promptTemplates };