Close TD-24 AGENTS test command
This commit is contained in:
@@ -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
|
- 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
|
- 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
|
- If you cannot fix a test, explain why and propose a solution
|
||||||
|
|
||||||
> **Zero failing tests. No exceptions.**
|
> **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.**
|
**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
|
- Store UI copy in language resources and resolve text through i18n helpers/hooks
|
||||||
- UI language MUST come from the operating system locale
|
- UI language MUST come from the operating system locale
|
||||||
- Rendering/preview/generated-content language MUST come from project preferences (`mainLanguage`), not UI locale
|
- Rendering/preview/generated-content language MUST come from project preferences (`mainLanguage`), not UI locale
|
||||||
|
|||||||
@@ -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
|
**Acceptance.** No bare `rescue _ ->` without either a narrow exception match
|
||||||
or a logged justification; suite green.
|
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
|
**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
|
test suite (`npm test`)" — stale from the pre-Elixir rewrite. It's an
|
||||||
|
|||||||
Reference in New Issue
Block a user