feat(desktop): retire le champ Modèle du profil (T-122) #113

Merged

ahitosly merged 1 commit from feat/T-122-remove-model-field into main 2026-09-07 18:18:15 +02:00

Conversation 1 Commits 1 Files changed 3 +5 -10

ahitosly commented 2026-09-07 18:15:22 +02:00

Owner

Copy link

Tâche

T-xxx — (titre dans docs/STATUS.md)

Changements

Contrat modifié ?

• Non
• Oui : packages/contract / supabase/migrations — docs/API.md / docs/DATA.md mis à jour, label contract, approbation humaine requise

Tests

• scripts/check.sh vert
• Test manuel (OS, scénario) :

Impacts cross-domaine

Fichiers hors périmètre de l'agent (et pourquoi) :

Suivi

• docs/STATUS.md mis à jour (ligne de la tâche)
• docs/CHANGELOG.md : entrée ajoutée sous Unreleased
• docs/DECISIONS.md / docs/ECARTS.md mis à jour si décision ou écart

## Tâche T-xxx — (titre dans docs/STATUS.md) ## Changements - ## Contrat modifié ? - [ ] Non - [ ] Oui : `packages/contract` / `supabase/migrations` — `docs/API.md` / `docs/DATA.md` mis à jour, label `contract`, approbation humaine requise ## Tests - [ ] `scripts/check.sh` vert - [ ] Test manuel (OS, scénario) : ## Impacts cross-domaine Fichiers hors périmètre de l'agent (et pourquoi) : ## Suivi - [ ] `docs/STATUS.md` mis à jour (ligne de la tâche) - [ ] `docs/CHANGELOG.md` : entrée ajoutée sous Unreleased - [ ] `docs/DECISIONS.md` / `docs/ECARTS.md` mis à jour si décision ou écart

ahitosly added 1 commit 2026-09-07 18:15:22 +02:00

feat(desktop): retire le champ Modèle du formulaire de profil (T-122) ... 4a69da29f9

L'utilisateur ne doit plus pouvoir choisir le modèle de réécriture depuis
le client (demande directe). Le champ llm_model reste en base et dans
ProfileFormValues (préserve les profils existants, defaults à null pour
les nouveaux) mais n'est plus rendu ni modifiable dans ProfileForm.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VPfxLocis8QeuyZh4B6QLo

ahitosly commented 2026-09-07 18:16:25 +02:00

Author

Owner

Copy link

Review qa-regression — PR #113 (T-122)

Verdict : APPROVE

1. Périmètre : diff limité à apps/desktop/src/components/profile-form.tsx, docs/CHANGELOG.md, docs/STATUS.md — conforme à la description de la tâche, rien hors périmètre.
2. Contrats : packages/contract et supabase/migrations non touchés — pas de changement d'API ni de schéma. llm_model reste bien une colonne/un champ de type (lib/profiles.ts), confirmé volontairement non modifié. RLS non concernée.
3. Sécurité : n/a (retrait pur d'UI, pas d'auth/JWT/secret impliqué).
4. Robustesse : values.llm_model reste porté par le state (initial → jamais réassigné sans l'input retiré), donc un profil existant garde sa valeur en mémoire et à la sauvegarde (onSave(values) envoie toujours llm_model inchangé) ; les nouveaux profils restent créés à null via Applications.tsx::valuesFromProfile — vérifié, cohérent avec ce qu'annonce le CHANGELOG/STATUS. Label/Input restent utilisés ailleurs dans le fichier (nom, prompt) donc pas d'import mort.
5. Cross-platform : UI pure, sans objet.
6. Tests : scripts/check.sh --desktop vert (tsc + eslint), scripts/check.sh --rust vert aussi (133/133, aucun changement Rust attendu — confirmé). Pas de test dédié ajouté, cohérent avec la nature du changement (suppression pure de rendu, aucune nouvelle logique à couvrir).
7. Suivi : docs/STATUS.md (ligne T-122, done, description fidèle au diff) et docs/CHANGELOG.md (section Unreleased → Removed) mis à jour dans cette même PR.
8. Latence : non concerné (chemin hotkey-up → paste inchangé).

Aucun point bloquant.

Review qa-regression — PR #113 (T-122) **Verdict : APPROVE** 1. Périmètre : diff limité à `apps/desktop/src/components/profile-form.tsx`, `docs/CHANGELOG.md`, `docs/STATUS.md` — conforme à la description de la tâche, rien hors périmètre. 2. Contrats : `packages/contract` et `supabase/migrations` non touchés — pas de changement d'API ni de schéma. `llm_model` reste bien une colonne/un champ de type (`lib/profiles.ts`), confirmé volontairement non modifié. RLS non concernée. 3. Sécurité : n/a (retrait pur d'UI, pas d'auth/JWT/secret impliqué). 4. Robustesse : `values.llm_model` reste porté par le state (`initial` → jamais réassigné sans l'input retiré), donc un profil existant garde sa valeur en mémoire et à la sauvegarde (`onSave(values)` envoie toujours `llm_model` inchangé) ; les nouveaux profils restent créés à `null` via `Applications.tsx::valuesFromProfile` — vérifié, cohérent avec ce qu'annonce le CHANGELOG/STATUS. `Label`/`Input` restent utilisés ailleurs dans le fichier (nom, prompt) donc pas d'import mort. 5. Cross-platform : UI pure, sans objet. 6. Tests : `scripts/check.sh --desktop` vert (tsc + eslint), `scripts/check.sh --rust` vert aussi (133/133, aucun changement Rust attendu — confirmé). Pas de test dédié ajouté, cohérent avec la nature du changement (suppression pure de rendu, aucune nouvelle logique à couvrir). 7. Suivi : `docs/STATUS.md` (ligne T-122, `done`, description fidèle au diff) et `docs/CHANGELOG.md` (section Unreleased → Removed) mis à jour dans cette même PR. 8. Latence : non concerné (chemin hotkey-up → paste inchangé). Aucun point bloquant.

ahitosly scheduled this pull request to auto merge when all checks succeed 2026-09-07 18:16:37 +02:00

ahitosly merged commit 68e97ae07b into main 2026-09-07 18:18:15 +02:00

ahitosly deleted branch feat/T-122-remove-model-field 2026-09-07 18:18:16 +02:00

ahitosly referenced this pull request from a commit 2026-09-07 18:18:18 +02:00

feat(desktop): retire le champ Modèle du profil (T-122) (#113)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

cloud-rennais/aria!113

No description provided.