feat(desktop): retire l'encart Style de base du serveur (T-123) #114

Merged
ahitosly merged 1 commit from feat/T-123-remove-base-prompt-card into main 2026-09-07 18:32:25 +02:00
Owner

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
feat(desktop): retire l'encart "Style de base du serveur" (T-123)
All checks were successful
ci / server (pull_request) Successful in 14s
ci / desktop-ts (pull_request) Successful in 27s
ci / desktop-rust (pull_request) Successful in 2m33s
8173474646
Demande directe de l'humain (jugé moche dans l'appli). L'endpoint
GET /v1/prompts/base et la commande Tauri get_base_prompt restent en
place (T-101) mais ne sont plus appelés côté client.

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

Review qa-regression — PR #114 (T-123)

Vérifications effectuées :

  1. Périmètre : diff limité à apps/desktop/src/settings/pages/Applications.tsx, docs/CHANGELOG.md, docs/STATUS.md. Conforme à la description : seul le Card « Style de base du serveur » (rendu + state basePrompt + useEffect + import getBasePrompt) est retiré, ainsi que l'import React useEffect devenu inutile (Card lui-même reste importé et utilisé ailleurs dans le fichier). Profil par défaut, liste des apps vues et catalogue intacts (diff origin/main..branche = 32 lignes -, 0 +, uniquement dans ce bloc).
  2. Orphelins : aucun résidu de getBasePrompt/basePrompt dans apps/desktop/src ou src-tauri en dehors du wrapper lib/tauri.ts::getBasePrompt (volontairement conservé, non appelé). Aucun autre fichier client ne référence cette section.
  3. Contrat/RLS/JWT : non concernés — commande Tauri get_base_prompt, wrapper tauri.ts et endpoint serveur GET /v1/prompts/base (T-101) laissés intacts, décision documentée et cohérente avec la règle 1/3b du CLAUDE.md (pas de contrat touché pour une portée non demandée). Cross-platform : UI pure, pas de #[cfg]. Latence : non concernée (page Settings, hors chemin hotkey→paste).
  4. scripts/check.sh --desktop : vert (tsc --noEmit + eslint OK). scripts/check.sh --rust : vert par exhaustivité (133/133, aucun changement Rust).
  5. docs/STATUS.md : ligne T-123 ajoutée par append (table, statut done, owner ui-ux, description alignée) ; « Prochain id libre » incrémenté à T-124. docs/CHANGELOG.md : entrée Unreleased → Removed ajoutée par append, cohérente avec T-122 existant. Pas de réécriture de contenu existant, diff propre (fast-forward possible depuis origin/main, merge-base = af19393 mais rediff contre origin/main confirmé identique au diff PR).

Aucun point bloquant.

Verdict : APPROVE

Review qa-regression — PR #114 (T-123) Vérifications effectuées : 1. Périmètre : diff limité à apps/desktop/src/settings/pages/Applications.tsx, docs/CHANGELOG.md, docs/STATUS.md. Conforme à la description : seul le Card « Style de base du serveur » (rendu + state basePrompt + useEffect + import getBasePrompt) est retiré, ainsi que l'import React useEffect devenu inutile (Card lui-même reste importé et utilisé ailleurs dans le fichier). Profil par défaut, liste des apps vues et catalogue intacts (diff origin/main..branche = 32 lignes -, 0 +, uniquement dans ce bloc). 2. Orphelins : aucun résidu de getBasePrompt/basePrompt dans apps/desktop/src ou src-tauri en dehors du wrapper lib/tauri.ts::getBasePrompt (volontairement conservé, non appelé). Aucun autre fichier client ne référence cette section. 3. Contrat/RLS/JWT : non concernés — commande Tauri get_base_prompt, wrapper tauri.ts et endpoint serveur GET /v1/prompts/base (T-101) laissés intacts, décision documentée et cohérente avec la règle 1/3b du CLAUDE.md (pas de contrat touché pour une portée non demandée). Cross-platform : UI pure, pas de #[cfg]. Latence : non concernée (page Settings, hors chemin hotkey→paste). 4. scripts/check.sh --desktop : vert (tsc --noEmit + eslint OK). scripts/check.sh --rust : vert par exhaustivité (133/133, aucun changement Rust). 5. docs/STATUS.md : ligne T-123 ajoutée par append (table, statut done, owner ui-ux, description alignée) ; « Prochain id libre » incrémenté à T-124. docs/CHANGELOG.md : entrée Unreleased → Removed ajoutée par append, cohérente avec T-122 existant. Pas de réécriture de contenu existant, diff propre (fast-forward possible depuis origin/main, merge-base = af19393 mais rediff contre origin/main confirmé identique au diff PR). Aucun point bloquant. Verdict : APPROVE
ahitosly scheduled this pull request to auto merge when all checks succeed 2026-09-07 18:31:07 +02:00
ahitosly deleted branch feat/T-123-remove-base-prompt-card 2026-09-07 18:32:25 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
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!114
No description provided.