agent-rules-init 0.2.0 → 0.3.0

package/dist/packs/kotlin.js

@@ -1,8 +1,12 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.buildGradle;
     if (!source)
         return null;
-    if (!/kotlin\(|org\.jetbrains\.kotlin/i.test(source))
+    // 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" }
@@ -24,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"],
@@ -15,12 +16,32 @@ function findIn(haystack, table) {
     }
     return undefined;
 }
+// Searching the whole pyproject.toml text (project name, URLs, script entry points, etc.)
+// produces false positives — e.g. Flask's own pyproject.toml has `name = "flask"`, which
+// isn't a dependency at all. Scope the search to the actual dependency declarations.
+function extractPyprojectDependencySections(pyproject) {
+    const sections = [];
+    const mainDeps = pyproject.match(/(?:^|\n)dependencies\s*=\s*\[([\s\S]*?)\]/);
+    if (mainDeps)
+        sections.push(mainDeps[1]);
+    const optionalDeps = pyproject.match(/\[project\.optional-dependencies\]([\s\S]*?)(?:\n\[|$)/);
+    if (optionalDeps)
+        sections.push(optionalDeps[1]);
+    const dependencyGroups = pyproject.match(/\[dependency-groups\]([\s\S]*?)(?:\n\[|$)/);
+    if (dependencyGroups)
+        sections.push(dependencyGroups[1]);
+    const poetryDepsBlocks = pyproject.match(/\[tool\.poetry(?:\.group\.\w+)?\.dependencies\]([\s\S]*?)(?:\n\[|$)/g);
+    if (poetryDepsBlocks)
+        sections.push(...poetryDepsBlocks);
+    return sections.length > 0 ? sections.join("\n") : pyproject;
+}
 function detect(signals) {
     const source = signals.pyprojectToml ?? signals.requirementsTxt ?? signals.environmentYml;
     if (!source)
         return null;
-    const framework = findIn(source, FRAMEWORKS) ?? { value: "none", confidence: "low" };
-    const testRunner = findIn(source, TEST_RUNNERS) ?? { value: "unknown", confidence: "low" };
+    const searchText = signals.pyprojectToml ? extractPyprojectDependencySections(signals.pyprojectToml) : source;
+    const framework = findIn(searchText, FRAMEWORKS) ?? { value: "none", confidence: "low" };
+    const testRunner = findIn(searchText, TEST_RUNNERS) ?? { value: "unknown", confidence: "low" };
     const packageManager = signals.environmentYml
         ? { value: "conda", confidence: "high" }
         : signals.hasFile("poetry.lock")
@@ -30,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 };

package/dist/packs/r.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["shiny", "shiny"],
     ["plumber", "plumber"],
@@ -22,39 +23,41 @@ function detect(signals) {
         : { value: "CRAN", confidence: "low" };
     return { packId: "r", language: "R", framework, testRunner, packageManager };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto R${framework !== "none" ? ` con ${framework}` : ""} (${detection.packageManager?.value}).`,
-        conventions: [
-            "Sigue la guía de estilo tidyverse (style.tidyverse.org) salvo que el proyecto ya use otra.",
-            `Ejecuta los tests con ${detection.testRunner?.value === "testthat" ? "testthat::test_dir(\"tests\")" : "el test runner del proyecto"} antes de terminar una tarea.`,
-            "Declara toda dependencia nueva en DESCRIPTION y actualiza renv.lock si el proyecto usa renv.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue la guía de estilo tidyverse (style.tidyverse.org) salvo que el proyecto ya use otra.",
+        deps: "Declara toda dependencia nueva en DESCRIPTION y actualiza renv.lock si el proyecto usa renv.",
+        arch: [
             "Mantén las funciones puras y con una responsabilidad clara; evita modificar el entorno global.",
             "Separa el análisis/scripts de la lógica reutilizable empaquetada en funciones.",
         ],
+    },
+    en: {
+        style: "Follow the tidyverse style guide (style.tidyverse.org) unless the project already uses another one.",
+        deps: "Declare every new dependency in DESCRIPTION and update renv.lock if the project uses renv.",
+        arch: [
+            "Keep functions pure and single-purpose; avoid mutating the global environment.",
+            "Separate analysis/scripts from reusable logic packaged as functions.",
+        ],
+    },
+};
+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);
+    return {
+        summary: summarySentence(lang, "R", framework, detection.packageManager?.value),
+        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 !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
     return [
-        {
-            id: "review",
-            title: "Code Review (R)",
-            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 (R)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (R)",
-            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 (R)", body: reviewBody(lang, "", framework) },
+        { id: "refactor", title: "Refactor (R)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (R)", body: testingBody(lang, runner) },
     ];
 }
 export const rPack = { id: "r", detect, rules, promptTemplates };

package/dist/packs/ruby.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 function detect(signals) {
     const source = signals.gemfile;
     if (!source)
@@ -21,39 +22,47 @@ function detect(signals) {
         packageManager: { value: "bundler", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Ruby${framework !== "none" ? ` con ${framework}` : ""} (bundler).`,
-        conventions: [
-            "Sigue la guía de estilo Ruby estándar (snake_case para métodos/variables, CamelCase para clases).",
-            `Ejecuta los tests con ${detection.testRunner?.value === "rspec" ? "bundle exec rspec" : detection.testRunner?.value === "minitest" ? "bundle exec rake test" : "el test runner del proyecto"} antes de terminar una tarea.`,
-            "Declara toda dependencia nueva en el Gemfile, nunca instales una gema sin registrarla.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue la guía de estilo Ruby estándar (snake_case para métodos/variables, CamelCase para clases).",
+        deps: "Declara toda dependencia nueva en el Gemfile, nunca instales una gema 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 the standard Ruby style guide (snake_case for methods/variables, CamelCase for classes).",
+        deps: "Declare every new dependency in the Gemfile; never install a gem 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 testCommand(detection, lang) {
+    if (detection.testRunner?.value === "rspec")
+        return "bundle exec rspec";
+    if (detection.testRunner?.value === "minitest")
+        return "bundle exec rake test";
+    return unknownRunnerLabel(lang);
+}
+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],
+        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 !== "unknown" ? detection.testRunner.value : unknownRunnerLabel(lang);
     return [
-        {
-            id: "review",
-            title: "Code Review (Ruby)",
-            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 (Ruby)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Ruby)",
-            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 (Ruby)", body: reviewBody(lang, "", framework) },
+        { id: "refactor", title: "Refactor (Ruby)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (Ruby)", body: testingBody(lang, runner) },
     ];
 }
 export const rubyPack = { id: "ruby", detect, rules, promptTemplates };

package/dist/packs/rust.js

@@ -1,3 +1,4 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["actix-web", "actix-web"],
     ["axum", "axum"],
@@ -23,39 +24,44 @@ function detect(signals) {
         packageManager: { value: "cargo", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Rust${framework !== "none" ? ` con ${framework}` : ""} (cargo).`,
-        conventions: [
-            "Sigue el formato estándar de `rustfmt`; corre `cargo clippy` para detectar problemas idiomáticos.",
-            "Ejecuta los tests con `cargo test` antes de terminar una tarea.",
-            "Evita `unwrap()`/`expect()` en código de producción salvo que la invariante esté justificada.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue el formato estándar de `rustfmt`; corre `cargo clippy` para detectar problemas idiomáticos.",
+        unwrap: "Evita `unwrap()`/`expect()` en código de producción salvo que la invariante esté justificada.",
+        arch: [
             "Mantén los módulos con una responsabilidad clara; usa `mod.rs` o archivos de módulo con nombre explícito según lo que ya use el repo.",
             "Declara toda dependencia nueva en Cargo.toml y mantén Cargo.lock sincronizado.",
         ],
+        reviewFocus: "usos innecesarios de `unwrap()`/`clone()`",
+        refactorExtra: "Respeta el sistema de ownership existente.",
+    },
+    en: {
+        style: "Follow the standard `rustfmt` formatting; run `cargo clippy` to catch idiomatic issues.",
+        unwrap: "Avoid `unwrap()`/`expect()` in production code unless the invariant is justified.",
+        arch: [
+            "Keep modules single-purpose; use `mod.rs` or explicitly named module files following what the repo already uses.",
+            "Declare every new dependency in Cargo.toml and keep Cargo.lock in sync.",
+        ],
+        reviewFocus: "unnecessary uses of `unwrap()`/`clone()`",
+        refactorExtra: "Respect the existing ownership model.",
+    },
+};
+function rules(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    return {
+        summary: summarySentence(lang, "Rust", framework, "cargo"),
+        conventions: [t.style, runTestsConvention(lang, "`cargo test`"), t.unwrap],
+        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 (Rust)",
-            body: `Revisa el diff actual buscando bugs, usos innecesarios de \`unwrap()\`/\`clone()\` y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Rust)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable. Respeta el sistema de ownership existente.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Rust)",
-            body: `Escribe tests con \`cargo test\` para el código señalado. Cubre el camino feliz y al menos un caso límite.`,
-        },
+        { id: "review", title: "Code Review (Rust)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "refactor", title: "Refactor (Rust)", body: refactorBody(lang, t.refactorExtra) },
+        { id: "testing", title: "Testing (Rust)", body: testingBody(lang, "`cargo test`") },
     ];
 }
 export const rustPack = { id: "rust", detect, rules, promptTemplates };

package/dist/packs/scala.js

@@ -1,11 +1,14 @@
+import { refactorBody, reviewBody, runTestsConvention, summarySentence, testingBody, unknownFrameworkLabel, unknownRunnerLabel, } from "../core/i18n.js";
 const FRAMEWORKS = [
     ["playframework", "play"],
     ["akka-http", "akka"],
     ["akka.actor", "akka"],
+    ["scalatra", "scalatra"],
 ];
 const TEST_RUNNERS = [
     ["scalatest", "scalatest"],
     ["munit", "munit"],
+    ["specs2", "specs2"],
 ];
 function detect(signals) {
     const source = signals.buildSbt;
@@ -34,39 +37,43 @@ function detect(signals) {
         packageManager: { value: "sbt", confidence: "high" },
     };
 }
-function rules(detection) {
-    const framework = detection.framework?.value ?? "none";
-    return {
-        summary: `Proyecto Scala${framework !== "none" ? ` con ${framework}` : ""} (sbt).`,
-        conventions: [
-            "Sigue la guía de estilo oficial de Scala; ejecuta `scalafmt` si el proyecto ya lo usa.",
-            `Ejecuta los tests con \`sbt test\` antes de terminar una tarea.`,
-            "Prefiere estructuras inmutables y funciones puras cuando el proyecto ya siga ese estilo.",
-        ],
-        architectureNotes: [
+const TEXTS = {
+    es: {
+        style: "Sigue la guía de estilo oficial de Scala; ejecuta `scalafmt` si el proyecto ya lo usa.",
+        immutability: "Prefiere estructuras inmutables y funciones puras cuando el proyecto ya siga ese estilo.",
+        arch: [
             "Mantén los módulos con una responsabilidad clara; usa `case class` para modelos de datos.",
             "Declara toda dependencia nueva en `build.sbt`.",
         ],
+        reviewFocus: "usos innecesarios de mutabilidad",
+    },
+    en: {
+        style: "Follow the official Scala style guide; run `scalafmt` if the project already uses it.",
+        immutability: "Prefer immutable structures and pure functions when the project already follows that style.",
+        arch: [
+            "Keep modules single-purpose; use `case class` for data models.",
+            "Declare every new dependency in `build.sbt`.",
+        ],
+        reviewFocus: "unnecessary mutability",
+    },
+};
+function rules(detection, lang) {
+    const t = TEXTS[lang];
+    const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
+    return {
+        summary: summarySentence(lang, "Scala", framework, "sbt"),
+        conventions: [t.style, runTestsConvention(lang, "`sbt test`"), t.immutability],
+        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 (Scala)",
-            body: `Revisa el diff actual buscando bugs, usos innecesarios de mutabilidad y desviaciones de las convenciones de ${framework}. Señala solo problemas concretos con línea de archivo.`,
-        },
-        {
-            id: "refactor",
-            title: "Refactor (Scala)",
-            body: `Propón refactors que reduzcan duplicación y mejoren la legibilidad sin cambiar comportamiento observable.`,
-        },
-        {
-            id: "testing",
-            title: "Testing (Scala)",
-            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 (Scala)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "refactor", title: "Refactor (Scala)", body: refactorBody(lang) },
+        { id: "testing", title: "Testing (Scala)", body: testingBody(lang, runner) },
     ];
 }
 export const scalaPack = { id: "scala", detect, rules, promptTemplates };