AGENTS/Frontend.md

Rama main · réplica de lectura del repositorio original.

# Frontend Guidance

This file documents frontend dependencies, UI behaviour, styling and API contract conventions.

Read [Pinia Store Guidance](frontend-pinia.md) before adding or changing stores, their callers, or realtime integration. Store architecture and lifecycle guidance live there.

## Dependency Installation Policy

- Run `npm ci` from `BoardOil.Web` to restore the committed lockfile. The project `.npmrc` sets `ignore-scripts=true` for local installs, CI and demo builds; Docker copies it before installing dependencies.
- Keep dependency lifecycle scripts disabled. There are currently no approved exceptions. If an essential dependency needs an install script, review its exact package/version and introduce a narrow allowlist with strict enforcement using a supported npm version; do not enable all scripts as a workaround.
- Explicit commands such as `npm run build`, `npm test` and `npx playwright install chromium` still execute. Automatic pre/post hooks are disabled, so required preparation belongs in the explicit command.
- This policy limits install-time execution. Build tools, tests and bundled browser code still execute dependencies and require dependency review and appropriate credential isolation.
- The current locked install scripts belong only to optional macOS `fsevents` packages. Keep scripts disabled on macOS too; validate file watching if those packages or the watcher tooling change.

### Dependency Updates and Release Age

- Use Node 24 and the exact npm version in `BoardOil.Web/package.json` under `engines` (currently 11.19.0). `engine-strict=true` rejects incompatible versions during installation. CI and Docker install npm from this same pin before `npm ci`; changing the pin requires checking the release-age behaviour again.
- To set up npm locally, run the following from `BoardOil.Web`, then check `npm --version`:

  ```sh
  BOARDOIL_NPM_VERSION="$(node -p "require('./package.json').engines.npm")"
  npm install --global --ignore-scripts "npm@$BOARDOIL_NPM_VERSION"
  ```

- The project `.npmrc` sets `min-release-age=7` in days. When resolving registry dependencies, `npm install` and `npm update` select eligible direct and transitive releases; a newly requested exact version inside the window fails. Use targeted updates such as `npm update <package>` and review the resulting manifest and lockfile diff.
- `npm ci` restores the reviewed lockfile without rechecking publication ages. An existing lockfile can therefore contain a newer release admitted under an exception. Lockfile review is part of the policy; regenerating or accepting a lockfile produced with different settings needs the same review.
- For an urgent security fix, record the advisory, exact package/version and reason in the story and obtain review of the exception. Use a one-command package-specific exception, for example `npm install <package>@<version> --min-release-age-exclude=<package>`. The package's dependencies retain the cooldown unless separately reviewed. Do not persist broad exclusions, disable the cooldown globally or use `--force` to bypass policy.
- Review newly introduced transitive packages, registry/source changes, integrity changes and install scripts alongside version changes. Direct Git, URL and local-file dependencies do not have the same npm-registry publication-age guarantee and require explicit source review.
- A cooldown delays admission; it does not prove a package is safe. Keep dependency scripts disabled and follow the existing review, validation and main-only source-control workflow.
- `scripts/npm-release-age-policy.test.mjs` exercises the installed npm CLI against a temporary local registry with controlled release dates. It covers fresh resolution, updates, exact-version rejection, a named exception, locked CI resolution and npm-version enforcement without downloading packages from the public registry.

### Docker Publication Gate

- The nightly and tagged-release Docker workflows call `.github/workflows/npm-publication-gate.yml` before either architecture builds or uploads an image.
- The gate audits the checked-out npm lockfile, including development, optional and peer dependencies, using the pinned npm version. High/critical findings and audit errors block publication; lower-severity findings remain visible in the job log.
- Run `npm audit --package-lock-only --include=dev --include=optional --include=peer --audit-level=high` from `BoardOil.Web` to reproduce the check. This command does not install application packages or apply fixes.
- Fix a blocking dependency or explicitly agree a narrowly scoped exception before changing the gate. An audit-service failure calls for a retry once the service recovers, not a silent bypass. The gate has no automatic override.
- Dependabot owns ongoing alerts. This gate does not run in ordinary CI, local builds or static demo publication, and does not scan NuGet or container OS packages.

## Behaviour Conventions

- Route guards and auth checks should remain centralised through store/router integration.
- Prefer canonical route paths in the router. Do not add legacy/back-compat or convenience redirect/alias routes; 
- Avoid introductory “wall of text” at the top of pages; interfaces should generally stand on their own unless short context is essential.
- Describe settings and features by their stable product meaning. Do not narrow their labels or helper text to the code paths that happen to use them today; those uses can grow, and implementation-specific caveats create unnecessary copy maintenance.
- Add explanatory text only when it helps users make a decision or understand a meaningful consequence. Keep incidental implementation details in technical guidance, and let a clear label stand on its own.
- Use the shared button styles in `BoardOil.Web/src/styles/buttons.css` (`.btn`, `.btn--secondary`, `.btn--danger`, etc.) instead of creating one-off button variants per view.
- Prefer `.btn.btn--tab` for tab toggles.
- Prefer `.btn.btn--toolbar` for markdown toolbar actions/mode toggles.
- Prefer `.btn.btn--menu-item` for menu-panel button actions.
- For management/list pages with repeated row patterns (for example Boards/Tags/Users/Columns manager views), use the shared entity-row styles in `BoardOil.Web/src/styles/entity-rows.css`:
  - page shell: `.entity-rows-page` (or `.entity-rows-page--compact` for narrower pages)
  - list container: `.entity-rows-list`
  - row container: `.entity-row`
  - row content/action slots: `.entity-row-main`, `.entity-row-actions`
  - row title/badge helpers: `.entity-row-title`, `.entity-row-badges`, `.entity-row-action-icon`
- Keep this entity-row pattern as the default for new management-style rows; only add view-specific row classes when behaviour or visuals are genuinely unique.
- Keep shared/global classes in shared stylesheets (`BoardOil.Web/src/style.css`, `BoardOil.Web/src/styles/*.css`) only when they are reused across views/components or define app-wide layout/theme behavior.
- Keep page-specific/component-specific classes in the relevant Vue file (`<style scoped>`), not in global stylesheets.
- Keep non-`.btn` controls limited to intentional interaction widgets:
  - chip/suggestion controls inside tag editors (`.tag-pill-remove`, `.card-tag-editor-suggestion`)
  - inline title edit trigger (`.card-title-button`)
- Do not use `$event` in Vue templates. Bind named handlers (for example `@change="onRoleChange"`) and parse event details in script.
- Do not use nested ternary expressions in frontend code. Prefer explicit `if`/`switch` branches or helper functions.

## Style Architecture Conventions

- Treat style payloads as discriminated style models as early as possible in frontend flows.
- Keep style responsibilities split by module boundary:
  - persistence modules parse/serialize style JSON and handle fallback policy.
  - draft adapter modules handle UI draft defaults and mode-switch defaults.
  - renderer modules consume typed style models only and decide classes/inline style output.
- Keep semantic styles (`auto`, `presets`) class-driven in UI rendering. Avoid introducing inline color authority for those modes.
- Keep manual styles (`solid`, `gradient`) as the only modes that emit inline color presentation from user-authored values.
- If parsing fails, fall back to `auto` style with no properties.

## API Trust and Defensive Coding

- Prefer backend-owned validation for new or migrated forms. The UI should display structured field/general errors and preserve rejected drafts. Include missing backend validation/error contracts in the form's work.
- Frontend validation requires a good, explicitly agreed case; do not duplicate backend rules for convenience. Agreed UI-only examples are password confirmation and non-expiring-token acknowledgement. Browser constraints (`required`, length/range limits, native email validation) also count when they prevent requests reaching backend validation; keyboard/autocomplete hints and value-editing controls are separate concerns.

- Treat backend API contracts as authoritative for frontend read/write flows.
- Do not add client-side fallback/normalization code that re-derives API fields “just in case” without a concrete, current failure mode.
- If a guard is required, document the exact reason in code (what can fail, where it was observed) and keep the guard narrowly scoped.
- Prefer removing speculative defensive code when it only adds complexity and duplicates backend guarantees.