Skip to content

build: propose le plugin platform aux sessions Claude Code du repo - #11

Open
grams wants to merge 1 commit into
masterfrom
claude/repo-feedback-btqj75
Open

build: propose le plugin platform aux sessions Claude Code du repo#11
grams wants to merge 1 commit into
masterfrom
claude/repo-feedback-btqj75

Conversation

@grams

@grams grams commented Jul 28, 2026

Copy link
Copy Markdown

Description of your changes

Ajoute .claude/settings.json pour proposer le plugin platform du marketplace Evaneos aux sessions Claude Code ouvertes sur ce repo, via le mécanisme documenté dans le README du marketplace (section « Automatic Installation for Teams ») :

  • déclare le marketplace evaneos-claude-marketplace ;
  • active le plugin platform.

Pourquoi — les skills /gotcha et /feedback du plugin déposent les faits surprenants et frictions plateforme dans Evaneos/platform-feedback. Les 26 issues actuelles du sas ont toutes le même auteur : le plugin s'installe à la main, utilisateur par utilisateur, donc personne d'autre n'a le chemin de capture sous la main au moment où la friction se produit. Ce repo figure dans la whitelist des repos d'infra du plugin (PC_INFRA_REPOS, plugins/platform/hooks/lib-platform-context.sh).

À l'ouverture du repo dans Claude Code, l'utilisateur qui fait confiance au repo se voit proposer l'installation — rien ne s'installe sans son accord. Seul platform est activé, pas team-platform (réservé aux membres de l'équipe).

⚠️ Ce repo est un fork. Le fichier ajouté est propre à Evaneos et n'a pas vocation à remonter upstream ; il crée une divergence d'un fichier avec crossplane-contrib/provider-sql, sans conflit possible avec le code du provider.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

Non coché volontairement : le changement ne touche aucun code Go ni aucune configuration de build — c'est un fichier de configuration d'éditeur, hors du périmètre de make reviewable test. Le template est hérité de l'upstream Crossplane.

How has this code been tested

Pas de test automatisé applicable : le fichier n'est lu que par Claude Code, jamais par la CI, le build ou le runtime du provider. Validité JSON vérifiée ; conformité au schéma enabledPlugins / extraKnownMarketplaces alignée sur l'exemple du README du marketplace.


Generated by Claude Code

grams commented Jul 28, 2026

Copy link
Copy Markdown
Author

e2e-tests en échec — pas causé par cette PR, et je ne le corrige pas ici.

Le job échoue à l'installation du provider dans le cluster kind, avant tout test :

>>>>>>> waiting for provider to be installed
error: timed out waiting for the condition on providers/provider-sql
make[1]: *** [Makefile:88: test-integration] Error 1

Le build de l'xpkg, le push vers le registre in-cluster et la création de l'objet Provider passent ; c'est la réconciliation par Crossplane (chart 2.3.4, kind v1.23.4) qui n'aboutit pas dans les 3 minutes.

Pourquoi ce n'est pas le diff : la PR ajoute un unique fichier, .claude/settings.json, lu par Claude Code seul. Aucun code Go, aucun Makefile, aucun workflow, aucun manifeste ne le référence — il ne peut pas influer sur l'installation d'un provider dans un cluster kind.

Ce que je ne peux pas affirmer : que la base est rouge elle aussi. Le dernier run CI sur master date du 04/05/2026 (succès) — trois mois sans exécution. Je ne peux donc pas parler de « régression connue de la base », seulement constater que rien n'a validé ce chemin depuis, et que le symptôme (timeout d'install, versions de chart et de kind figées) pointe vers une dérive d'environnement sur ce fork, pas vers ce changement.

Remettre l'e2e Crossplane au vert sur un fork resté en arrière de l'upstream est un chantier propre, sans rapport avec l'ajout d'un fichier de configuration d'éditeur. Deux voies possibles, au choix des mainteneurs : traiter la dérive dans une PR dédiée, ou merger celle-ci en connaissance de cause si l'e2e-tests n'est pas un check requis.

La PR reste en draft.


Generated by Claude Code

@grams
grams marked this pull request as ready for review July 28, 2026 16:24
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