Update robot instructions to raise awareness of platform registrartion mode and clear instructions for generating translated strings

Author: rsmithlal

.github/copilot-instructions.md

@@ -12,6 +12,7 @@ This repository contains the **Better Together Community Engine** (an isolated R
 - **Use `ENV.fetch`** rather than `ENV[]`.
 - **Always add policy/authorization checks** on links/buttons to controller actions.
 - **i18n & Mobility**: every user-facing string must be translatable; include missing keys.
+- Provide translations for all available locales (e.g., en, es, fr) when adding new strings.
 
 ## Technology Stack
 
@@ -46,6 +47,11 @@ This repository contains the **Better Together Community Engine** (an isolated R
 
 ## Coding Guidelines
 
+## Test Environment Setup
+- Configure the host Platform in a before block for controller/request/feature tests.
+  - Create/set a Platform as host (with community) before requests.
+  - Toggle requires_invitation and provide invitation_code when needed.
+
 - **Ruby/Rails**
   - 2-space indent, snake_case methods, Rails conventions
   - Service objects in `app/services/`
@@ -57,13 +63,15 @@ This repository contains the **Better Together Community Engine** (an isolated R
   - Bootstrap utility classes; respect prefers-reduced-motion & other a11y prefs
   - Avoid inline JS; use Stimulus
   - External links in `.trix-content` get FA external-link icon unless internal/mailto/tel/pdf
+  - All user-facing copy must use t("...") and include keys across all locales (add to config/locales/en.yml, es.yml, fr.yml).
 - **Hotwire**
   - Use Turbo Streams for CRUD updates
   - Stimulus controllers in `app/javascript/controllers/`
   - No direct DOM manipulation without Stimulus targets/actions
 - **Background Jobs**
   - Sidekiq jobs under appropriate queues (`:default`, `:mailers`, `:metrics`, etc.)
   - Idempotent job design; handle retries
+  - When generating emails/notifications, localize both subject and body for all locales.
 - **Search**
   - Update `as_indexed_json` to include translated/plain-text fields as needed
 - **Encryption & Privacy**

AGENTS.md

@@ -39,3 +39,19 @@ Instructions for GitHub Copilot and other automated contributors working in this
   - Fallback: `npx -y @mermaid-js/mermaid-cli -i docs/your_diagram.mmd -o docs/your_diagram.png`.
 - PRs that add/modify models, associations, or flows must include corresponding docs and diagrams.
 - When notifications, policies, or routes change, ensure affected docs and diagrams are updated to match behavior.
+
+## Platform Registration Mode
+- Invitation-required: Platforms support `requires_invitation` (see `BetterTogether::Platform#settings`). When enabled, users must supply a valid invitation code to register. This is the default for hosted deployments.
+- Where to change: Host Dashboard → Platforms → Edit → “Requires Invitation”.
+- Effects:
+  - Devise registration page prompts for an invitation code when none is present.
+  - Accepted invitations prefill email, apply community/platform roles, and are marked accepted on successful sign‑up.
+
+## Translations & Locales
+- All user‑facing text must use I18n — do not hard‑code strings in views, controllers, models, or JS.
+- When adding new text, add translation keys for all available locales in this repo (e.g., `config/locales/en.yml`, `es.yml`, `fr.yml`).
+- Include translations for:
+  - Flash messages, validation errors, button/label text, email subjects/bodies, and Action Cable payloads.
+  - Any UI strings rendered from background jobs or notifiers.
+- Prefer existing keys where possible; group new keys under appropriate namespaces.
+- If a locale is missing a translation at review time, translate the English copy rather than leaving it undefined.