diff --git a/AGENTS.md b/AGENTS.md index 3abe7d8..35e5e62 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -55,7 +55,7 @@ This document provides context and best practices for GitHub Copilot when workin - Never leave tests failing, even if they appear unrelated to your changes - If a test failure is pre-existing, fix it as part of your current work -- Run the full test suite (`npm test`) before considering any task complete +- Run the full test suite (`mix test`) before considering any task complete - If you cannot fix a test, explain why and propose a solution > **Zero failing tests. No exceptions.** @@ -106,7 +106,7 @@ This document provides context and best practices for GitHub Copilot when workin **All user-facing text MUST follow proper i18n patterns.** -- Do not hardcode UI strings directly in React components, menu templates, dialogs, or toasts +- Do not hardcode UI strings directly in LiveView/HEEx components, menu templates, dialogs, or toasts - Store UI copy in language resources and resolve text through i18n helpers/hooks - UI language MUST come from the operating system locale - Rendering/preview/generated-content language MUST come from project preferences (`mainLanguage`), not UI locale diff --git a/TECHDEBTS.md b/TECHDEBTS.md index d17c289..5c8a96f 100644 --- a/TECHDEBTS.md +++ b/TECHDEBTS.md @@ -887,7 +887,11 @@ call, and delete rescues that guard nothing. Shutdown.ex's rescues stay as-is **Acceptance.** No bare `rescue _ ->` without either a narrow exception match or a logged justification; suite green. -### TD-24: Fix stale `npm test` reference in AGENTS.md +### TD-24: Fix stale `npm test` reference in AGENTS.md ✅ DONE (2026-06-12) + +**Status: implemented.** `AGENTS.md` now tells agents to run `mix test` +instead of the stale `npm test`, and the nearby UI i18n wording was updated to +refer to LiveView/HEEx components rather than leftover React-era terminology. **Context.** AGENTS.md "Fix All Test Failures" section says "Run the full test suite (`npm test`)" — stale from the pre-Elixir rewrite. It's an