agent-rules-init 0.3.0 → 0.4.0

package/dist/core/templates.js

@@ -1,44 +1,82 @@
 import { UI } from "./i18n.js";
-function renderSection(entries, lang) {
+import { SOURCE_FILES } from "./canonical-commands.js";
+function renderSection(entries, lang, options = {}) {
     const ui = UI[lang];
+    const { summaries = true } = options;
     return entries
         .map(({ detection, ruleSet }) => {
-        const conventions = ruleSet.conventions.map((c) => `- ${c}`).join("\n");
+        const conventionItems = options.operationalConventions === false
+            ? ruleSet.conventions.filter((item) => !/^(?:Run the tests|Run the repository|Ejecuta los tests|Ejecuta la suite)/i.test(item))
+            : ruleSet.conventions;
+        const conventions = conventionItems.map((c) => `- ${c}`).join("\n");
         const architecture = ruleSet.architectureNotes.map((a) => `- ${a}`).join("\n");
-        return [
+        const parts = [
             `## ${detection.language} (${detection.packId})`,
-            "",
-            ruleSet.summary,
-            "",
-            `### ${ui.sections.conventions}`,
-            conventions,
-            "",
-            `### ${ui.sections.architecture}`,
-            architecture,
-        ].join("\n");
+        ];
+        if (summaries)
+            parts.push("", ruleSet.summary);
+        if (options.conventions !== false && conventions)
+            parts.push("", `### ${ui.sections.conventions}`, conventions);
+        if (options.architecture !== false && architecture)
+            parts.push("", `### ${ui.sections.architecture}`, architecture);
+        return parts.join("\n");
     })
         .join("\n\n");
 }
-const SOURCE_FILES = {
-    npm: "package.json",
-    composer: "composer.json",
-    make: "Makefile",
-    mix: "mix.exs",
-    tox: "tox.ini",
-};
+function evidenceLabel(lang) {
+    return lang === "es" ? "evidencia" : "evidence";
+}
+function renderEvidenceSection(title, facts, lang) {
+    const high = facts.filter((fact) => fact.confidence === "high");
+    if (high.length === 0)
+        return "";
+    const lines = high.map((fact) => `- ${fact.statement} (${evidenceLabel(lang)}: ${fact.evidence.map((item) => `\`${item}\``).join(", ")})`);
+    return [`## ${title}`, "", ...lines].join("\n");
+}
+function architectureFactsTitle(lang) {
+    return lang === "es" ? "Arquitectura observada" : "Observed architecture";
+}
+function localConventionsTitle(lang) {
+    return lang === "es" ? "Convenciones locales verificadas" : "Verified local conventions";
+}
+function renderCanonical(facts, lang) {
+    const commands = facts.canonical.filter((command) => command.confidence === "high");
+    if (commands.length === 0)
+        return "";
+    return [
+        `## ${UI[lang].sections.canonical}`,
+        "",
+        ...commands.map((command) => `- ${command.kind}: \`${command.command}\` (${command.source})`),
+    ].join("\n");
+}
 export function renderRepoFacts(facts, lang) {
     const ui = UI[lang];
     const sections = [];
+    const canonical = facts.canonical.filter((c) => c.confidence === "high");
+    if (canonical.length > 0) {
+        const lines = canonical.map((c) => `- ${c.kind}: \`${c.command}\` (${c.source})`);
+        sections.push([`## ${ui.sections.canonical}`, "", ...lines].join("\n"));
+    }
     if (facts.commands.length > 0) {
-        const lines = facts.commands.map((c) => c.detail && c.detail !== c.invocation
-            ? `- \`${c.invocation}\` → \`${c.detail}\` (${SOURCE_FILES[c.source]})`
-            : `- \`${c.invocation}\` (${SOURCE_FILES[c.source]})`);
+        const lines = facts.commands.map((c) => {
+            const sourceFile = c.manifestPath ?? SOURCE_FILES[c.source];
+            return c.detail && c.detail !== c.invocation
+                ? `- \`${c.invocation}\` → \`${c.detail}\` (${sourceFile})`
+                : `- \`${c.invocation}\` (${sourceFile})`;
+        });
         for (const o of facts.omittedCommands)
             lines.push(`- ${ui.andMore(o.count, SOURCE_FILES[o.source])}`);
         sections.push([`## ${ui.sections.commands}`, "", ...lines].join("\n"));
     }
-    if (facts.structure.length > 0) {
+    if (facts.structure.length > 0 || facts.testDirs.length > 0 || facts.entrypoints.length > 0) {
         const lines = facts.structure.map((d) => (d.note ? `- \`${d.dir}\` — ${d.note}` : `- \`${d.dir}\``));
+        for (const dir of facts.testDirs) {
+            if (!facts.structure.some((d) => d.dir === dir))
+                lines.push(`- \`${dir}\` — ${ui.testDirNote}`);
+        }
+        for (const entry of facts.entrypoints) {
+            lines.push(`- ${ui.entrypointNote}: \`${entry.target}\` (${entry.source} "${entry.label}")`);
+        }
         sections.push([`## ${ui.sections.structure}`, "", ...lines].join("\n"));
     }
     if (facts.ciCommands.length > 0) {
@@ -47,6 +85,12 @@ export function renderRepoFacts(facts, lang) {
             lines.push(`- ${ui.andMore(facts.omittedCiCount)}`);
         sections.push([`## ${ui.sections.ci}`, "", ...lines].join("\n"));
     }
+    const architecture = renderEvidenceSection(architectureFactsTitle(lang), facts.architectureFacts ?? [], lang);
+    if (architecture)
+        sections.push(architecture);
+    const conventions = renderEvidenceSection(localConventionsTitle(lang), facts.conventionFacts ?? [], lang);
+    if (conventions)
+        sections.push(conventions);
     return sections.join("\n\n");
 }
 function renderDocument(title, entries, facts, lang) {
@@ -56,7 +100,7 @@ function renderDocument(title, entries, facts, lang) {
         "",
         UI[lang].generatedHeader,
         "",
-        renderSection(entries, lang),
+        renderSection(entries, lang, { architecture: !(facts?.architectureFacts?.length) }),
         ...(factsBlock ? ["", factsBlock] : []),
     ].join("\n");
 }
@@ -64,10 +108,39 @@ export function renderClaudeMd(entries, facts, lang) {
     return renderDocument("# CLAUDE.md", entries, facts, lang);
 }
 export function renderAgentsMd(entries, facts, lang) {
-    return renderDocument("# AGENTS.md", entries, facts, lang);
+    const intro = lang === "es"
+        ? "Reglas operativas para modificar este repositorio. Respeta el alcance y valida los cambios con los comandos indicados."
+        : "Operational rules for modifying this repository. Respect scope and validate changes with the commands below.";
+    const blocks = [
+        "# AGENTS.md", "", UI[lang].generatedHeader, "", intro, "",
+        renderSection(entries, lang, { architecture: !(facts?.architectureFacts?.length) }),
+    ];
+    if (facts) {
+        const canonical = renderCanonical(facts, lang);
+        const architecture = renderEvidenceSection(architectureFactsTitle(lang), facts.architectureFacts ?? [], lang);
+        const conventions = renderEvidenceSection(localConventionsTitle(lang), facts.conventionFacts ?? [], lang);
+        for (const block of [canonical, architecture, conventions])
+            if (block)
+                blocks.push("", block);
+    }
+    return blocks.join("\n");
 }
 export function renderCopilotInstructions(entries, facts, lang) {
-    return renderDocument("# Copilot Instructions", entries, facts, lang);
+    const intro = lang === "es"
+        ? "Aplica estas convenciones al completar y modificar código. Omite tareas operativas de terminal salvo que sean necesarias para el cambio."
+        : "Apply these conventions when completing and modifying code. Omit terminal operations unless the change requires them.";
+    const blocks = [
+        "# Copilot Instructions", "", UI[lang].generatedHeader, "", intro, "",
+        renderSection(entries, lang, { architecture: false, operationalConventions: false }),
+    ];
+    if (facts) {
+        const conventions = renderEvidenceSection(localConventionsTitle(lang), facts.conventionFacts ?? [], lang);
+        const architecture = renderEvidenceSection(architectureFactsTitle(lang), facts.architectureFacts ?? [], lang);
+        for (const block of [conventions, architecture])
+            if (block)
+                blocks.push("", block);
+    }
+    return blocks.join("\n");
 }
 export function renderPromptFiles(packId, templates) {
     return templates.flatMap((template) => [

package/dist/core/types.d.ts

@@ -1,10 +1,17 @@
 import type { Lang } from "./i18n.js";
+export type JsPackageManager = "npm" | "pnpm" | "yarn" | "bun";
 export interface PackageJsonManifest {
     name?: string;
+    main?: string;
     dependencies: Record<string, string>;
     devDependencies: Record<string, string>;
     scripts: Record<string, string>;
     moduleType: "module" | "commonjs";
+    /** Gestor aplicable al manifiesto, declarado en packageManager o inferido por el lock más cercano. */
+    packageManager?: JsPackageManager;
+}
+export interface LocatedPackageJsonManifest extends PackageJsonManifest {
+    path: string;
 }
 export interface ComposerJsonManifest {
     require: Record<string, string>;
@@ -17,6 +24,7 @@ export interface RepoSignals {
     hasFile: (relativePath: string) => boolean;
     hasDir: (relativeDir: string) => boolean;
     packageJson?: PackageJsonManifest;
+    packageJsons?: LocatedPackageJsonManifest[];
     pyprojectToml?: string;
     requirementsTxt?: string;
     environmentYml?: string;
@@ -40,6 +48,11 @@ export interface RepoSignals {
         path: string;
         content: string;
     }[];
+    /** Archivos locales de guía/configuración seleccionados de forma conservadora. */
+    guidanceFiles?: {
+        path: string;
+        content: string;
+    }[];
 }
 export type Confidence = "high" | "low";
 export interface DetectionField<T> {
@@ -53,6 +66,8 @@ export interface DetectionResult {
     packageManager?: DetectionField<string>;
     testRunner?: DetectionField<string>;
     linter?: DetectionField<string>;
+    /** Framework implementado por el propio repositorio; no implica una dependencia. */
+    frameworkSource?: string;
     usesTypeScript?: boolean;
     moduleFormat?: "module" | "commonjs";
 }
@@ -66,11 +81,12 @@ export interface PromptTemplate {
     title: string;
     body: string;
 }
-export type CommandSource = "npm" | "composer" | "make" | "mix" | "tox";
+export type CommandSource = JsPackageManager | "composer" | "make" | "mix" | "tox";
 export interface CommandEntry {
     source: CommandSource;
     invocation: string;
     detail?: string;
+    manifestPath?: string;
 }
 export interface DirEntry {
     dir: string;
@@ -80,6 +96,27 @@ export interface CiCommand {
     command: string;
     workflow: string;
 }
+export interface CanonicalCommand {
+    kind: "test" | "lint" | "build" | "format" | "typecheck";
+    command: string;
+    /** Archivo o señal del que procede el comando, p. ej. `package.json`, `ci: ci.yml`, `mvnw`. */
+    source: string;
+    confidence: Confidence;
+    /** "." para la raíz del repo; ruta del workspace para unidades anidadas. */
+    scope: string;
+}
+export interface EvidenceFact {
+    statement: string;
+    evidence: string[];
+    scope: string;
+    confidence: Confidence;
+}
+export interface ArchitectureFact extends EvidenceFact {
+    kind: "entrypoint" | "tests" | "layers" | "workspace" | "source-layout";
+}
+export interface ConventionFact extends EvidenceFact {
+    kind: "formatting" | "typescript" | "python-style" | "contributing";
+}
 export interface RepoFacts {
     commands: CommandEntry[];
     omittedCommands: {
@@ -89,10 +126,24 @@ export interface RepoFacts {
     structure: DirEntry[];
     ciCommands: CiCommand[];
     omittedCiCount: number;
+    canonical: CanonicalCommand[];
+    testDirs: string[];
+    entrypoints: {
+        label: string;
+        target: string;
+        source: string;
+    }[];
+    architectureFacts: ArchitectureFact[];
+    conventionFacts: ConventionFact[];
+}
+export interface PackContext {
+    facts: RepoFacts;
+    /** Señales del mismo alcance que facts; permite evitar canónicos de otro stack. */
+    signals?: RepoSignals;
 }
 export interface Pack {
     id: string;
     detect(signals: RepoSignals): DetectionResult | null;
-    rules(detection: DetectionResult, lang: Lang): RuleSet;
-    promptTemplates(detection: DetectionResult, lang: Lang): PromptTemplate[];
+    rules(detection: DetectionResult, lang: Lang, ctx?: PackContext): RuleSet;
+    promptTemplates(detection: DetectionResult, lang: Lang, ctx?: PackContext): PromptTemplate[];
 }

package/dist/core/writer.js

@@ -4,16 +4,16 @@ export function writeGeneratedFiles(rootPath, files) {
     return files.map(({ path: relativePath, content }) => {
         const absolutePath = path.join(rootPath, relativePath);
         try {
-            // An already-existing file is the expected outcome on a re-run (the tool's
-            // no-overwrite guarantee), not a failure — report it as skipped, exit 0.
-            if (fs.existsSync(absolutePath)) {
-                return { path: relativePath, status: "skipped" };
-            }
             fs.mkdirSync(path.dirname(absolutePath), { recursive: true });
-            fs.writeFileSync(absolutePath, content);
+            // `wx` makes the no-overwrite guarantee atomic: even if another process creates
+            // the file between directory creation and this call, Node returns EEXIST.
+            fs.writeFileSync(absolutePath, content, { flag: "wx" });
             return { path: relativePath, status: "written" };
         }
         catch (err) {
+            if (err.code === "EEXIST") {
+                return { path: relativePath, status: "skipped" };
+            }
             return { path: relativePath, status: "error", error: err.message };
         }
     });

package/dist/packs/cpp.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 = [
     ["find_package(qt", "qt"],
     ["find_package(boost", "boost"],
@@ -79,8 +79,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 (C/C++)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (C/C++)", body: refactorBody(lang) },

package/dist/packs/csharp.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.csproj;
     if (!source)
@@ -53,8 +53,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 (C#/.NET)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (C#/.NET)", body: refactorBody(lang) },

package/dist/packs/dart.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.pubspecYaml;
     if (!source)
@@ -44,13 +44,13 @@ function rules(detection, lang) {
     const runner = detection.testRunner?.value !== "unknown" ? detection.testRunner?.value : undefined;
     return {
         summary: summarySentence(lang, "Dart", framework, "pub"),
-        conventions: [t.style, runTestsConvention(lang, runner ?? unknownRunnerLabel(lang)), t.deps],
+        conventions: [t.style, runTestsConvention(lang, runner), t.deps],
         architectureNotes: [framework === "flutter" ? t.archFlutter : t.archPlain, t.immutability],
     };
 }
 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 (Dart)", body: reviewBody(lang, "", framework) },
         { id: "refactor", title: "Refactor (Dart)", body: refactorBody(lang) },

package/dist/packs/elixir.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.mixExs;
     if (!source)
@@ -50,7 +50,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 (Elixir)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Elixir)", body: refactorBody(lang) },

package/dist/packs/go.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 = [
     ["gin-gonic/gin", "gin"],
     ["labstack/echo", "echo"],
@@ -68,7 +68,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 (Go)", body: reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Go)", body: refactorBody(lang) },

package/dist/packs/java.js

@@ -1,4 +1,9 @@
-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 SPRING_RISK = {
+    es: "Presta especial atención a los límites de transacción (@Transactional), la inyección de dependencias y la separación controller/service/repository.",
+    en: "Pay special attention to transaction boundaries (@Transactional), dependency injection and the controller/service/repository separation.",
+};
 function detect(signals) {
     const source = signals.pomXml ?? signals.buildGradle;
     if (!source)
@@ -9,14 +14,19 @@ function detect(signals) {
     // (`alias(libs.plugins.kotlin.android)`) rather than the literal `kotlin("jvm")` or
     // `org.jetbrains.kotlin` plugin id, so a plain word-boundary check on "kotlin" is
     // more robust than trying to match every DSL style.
-    if (/\bkotlin\b/i.test(source))
+    if ((signals.buildGradle && /\bkotlin\b/i.test(signals.buildGradle)) ||
+        (signals.pomXml && /\bkotlin\b/i.test(signals.pomXml)))
         return null;
     const framework = /spring/i.test(source)
         ? { value: "spring", confidence: "high" }
         : { value: "none", confidence: "low" };
     const packageManager = signals.pomXml
-        ? { value: "maven", confidence: "high" }
-        : { value: "gradle", confidence: "high" };
+        ? 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 testRunner = /junit/i.test(source)
         ? { value: "junit", confidence: "high" }
         : { value: "unknown", confidence: "low" };
@@ -42,24 +52,60 @@ const TEXTS = {
         reviewFocus: "null-safety",
     },
 };
-function rules(detection, lang) {
+function rules(detection, lang, ctx) {
     const t = TEXTS[lang];
     const framework = detection.framework?.value !== "none" ? detection.framework?.value : undefined;
-    const testCmd = detection.packageManager?.value === "maven" ? "mvn test" : "gradle test";
+    const wrapperCmd = detection.packageManager?.value === "maven wrapper"
+        ? "./mvnw test"
+        : detection.packageManager?.value === "maven"
+            ? "mvn test"
+            : detection.packageManager?.value === "gradle wrapper"
+                ? "./gradlew test"
+                : "gradle test";
+    const testCmd = canonicalOf(ctx, "test", "java")?.command ?? wrapperCmd;
     return {
         summary: summarySentence(lang, "Java", framework, detection.packageManager?.value),
         conventions: [t.naming, 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", "java");
+    const build = canonicalOf(ctx, "build", "java");
+    const testDirs = ctx?.facts.testDirs ?? [];
+    const es = lang === "es";
+    const reviewParts = [];
+    reviewParts.push(es
+        ? `Revisa el diff actual contra las convenciones Java de este repositorio${framework ? ` (${framework})` : ""}.`
+        : `Review the current diff against this repository's Java 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 (build && build.command !== test?.command) {
+        reviewParts.push(es ? `CI también ejecuta \`${build.command}\`.` : `CI also runs \`${build.command}\`.`);
+    }
+    if (framework === "spring")
+        reviewParts.push(SPRING_RISK[lang]);
+    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 (Java)", body: reviewBody(lang, t.reviewFocus, framework) },
+        { id: "review", title: "Code Review (Java)", body: ctx ? reviewParts.join(" ") : reviewBody(lang, t.reviewFocus, framework) },
         { id: "refactor", title: "Refactor (Java)", body: refactorBody(lang) },
-        { id: "testing", title: "Testing (Java)", body: testingBody(lang, runner) },
+        { id: "testing", title: "Testing (Java)", body: ctx ? testingParts.join(" ") : testingBody(lang, runner) },
     ];
 }
 export const javaPack = { id: "java", detect, rules, promptTemplates };