Skip to content

Clé Context7 au setup uniquement + pédagogie des options (T6.15/T6.16)#14

Merged
benoitvx merged 2 commits into
mainfrom
feat/context7-key-at-setup
Jul 21, 2026
Merged

Clé Context7 au setup uniquement + pédagogie des options (T6.15/T6.16)#14
benoitvx merged 2 commits into
mainfrom
feat/context7-key-at-setup

Conversation

@benoitvx

Copy link
Copy Markdown
Contributor

Contexte : retour onboarding alpha 21/07 (AC-R038, AC-R039). La clé Context7 était demandée dès albert-code install (phase A.4), avant toute explication et avant que l'utilisateur ait choisi de brancher ce MCP. De plus, les questions Y/n du setup n'expliquaient pas assez chaque option.

Résumé des changements :

lib/phases.sh | 108 insertions(+), 21 suppressions(-)
TESTS.md      | 5 scénarios de validation (S-ctx-1 à S-ctx-5)
README.md     | Nouvelle section skills/MCP + ordre install/setup/run
BACKLOG.md    | T6.15/T6.16 marqués implémentés
FEEDBACK.md   | AC-R038/AC-R039 passés à traités

T6.15 — Clé Context7 uniquement au setup :

  • Suppression du bloc A.4 dans phase_a() : plus aucun prompt Context7 pendant albert-code install
  • Le chemin existant du setup (scaffold_opencode_json) devient l'unique point de collecte : si Y à context7 et clé absente, prompt_secret + persist_zshenv
  • Piège corrigé : ensure_vm_runtime n'était appelé qu'en phase A. Une clé saisie au setup était bien persistée dans ~/.zshenv hôte, mais ~/.agent-vm/runtime.sh gardait CONTEXT7_API_KEY='' → la VM ne voyait jamais la clé. Ajout d'un appel ensure_vm_runtime en fin de phase_b() pour régénérer le bloc.

T6.16 — Pédagogie des options du setup :

  • Chaque MCP est précédé de 1-3 lignes d'explication (info) avant le confirm, avec les libellés exacts du ticket
  • Confirm raccourci en Installer le connecteur <nom> ?
  • Skills déjà harmonisées (description inline dans le confirm)

Validation : TESTS.md scénarios S-ctx-1 à S-ctx-5, dry-run install et setup, bash -n lib/phases.sh OK, garde-fou AC-R008 (chemins personnels) OK.

benoitvx added 2 commits July 21, 2026 16:35
…gogie MCP (T6.15/T6.16)

T6.15 (AC-R038) — Clé Context7 uniquement au setup :
- Suppression du bloc A.4 dans phase_a() : plus aucun prompt ni collecte
  de clé Context7 pendant albert-code install
- Ajout d'un appel ensure_vm_runtime en fin de phase_b() : la clé saisie
  au setup est maintenant propagée dans ~/.agent-vm/runtime.sh (avant elle
  restait à '' dans la VM car ensure_vm_runtime n'était appelé qu'en phase A)

T6.16 (AC-R039) — Pédagogie des options MCP :
- Chaque MCP est désormais précédé de 1-3 lignes d'explication (info)
  avant le confirm, avec les libellés exacts du ticket
- Confirm raccourci en « Installer le connecteur <nom> ? »

Validation : scénarios S-ctx-1 à S-ctx-5 dans TESTS.md.
…ack pour T6.15/T6.16

README : ajout d'une section expliquant ce que sont les skills et les MCP,
avec le tableau des 4 connecteurs et la mention que la clé Context7 n'est
demandée qu'au setup si le MCP est choisi. Ajout d'un avertissement sur
l'ordre install/setup/run (un run sans setup ouvre OpenCode non connecté).

BACKLOG.md : T6.15 et T6.16 marqués implémentés avec références aux
scénarios S-ctx-1 à S-ctx-5 dans TESTS.md.

FEEDBACK.md : AC-R038 et AC-R039 passés à « traité » avec renvois
respectifs vers BACKLOG et TESTS.
@benoitvx
benoitvx merged commit 0d1b500 into main Jul 21, 2026
2 checks passed
@benoitvx
benoitvx deleted the feat/context7-key-at-setup branch July 21, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant