Conversation
chore: lint hardening, prettier and CI quality gates
There was a problem hiding this comment.
🤖 Codesema Code Review — 💬 Commentaire
Release 0.11.0 : très large diff dont l'essentiel (~90%) est du reformatage Prettier + passage à oxlint/lefthook (accolades obligatoires, retours à la ligne, | undefined pour exactOptionalPropertyTypes, retrait des assertions non-null) — mécanique et sans risque. Deux apports de fond côté code visible : (1) git.ts isole l'environnement des sous-processus (subprocessEnv retire GIT_DIR/GIT_WORK_TREE/…), bien testé et opportun au moment où le repo installe ses propres hooks lefthook ; (2) le câblage du nouveau server_context dans l'entrée de revue et le prompt de l'agent. ATTENTION : le diff fourni est tronqué à 200 000 caractères et s'arrête au milieu de review.ts — le module cœur de la fonctionnalité, packages/cli/src/server-context.ts (fetch GET /api/cli/context, paramètre remote_url, validation de la réponse, timeout, avertissement de fraîcheur) ainsi que tout fichier situé après review.ts n'ont PAS pu être relus. Aucun défaut dans la partie visible ; verdict 'comment' car un relecteur humain doit encore valider server-context.ts et les points « à revoir » ci-dessous.
2 points relevés
- INFO
packages/cli/src/git.ts:21— subprocessEnv retire GIT_DIR/GIT_WORK_TREE/GIT_INDEX_FILE/… avant chaque appel git, si bien que git vise toujours le cwd et jamais le dépôt d'un hook englobant qui aurait posé ces variables. Protection très bienvenue au moment précis où ce MR installe ses propres hooks lefthook (pre-commit/pre-push), et le périmètre est bien maîtrisé (GIT_SSH_COMMAND, GIT_AUTHOR_/GIT_COMMITTER_, GIT_CONFIG_GLOBAL restent transmis). Comportement couvert par git.test.ts. - INFO
packages/cli/src/record.ts:69— buildRecord lève désormais explicitement record.invalidJson quand sanitizeRecord renvoie null, au lieu du!précédent : un review.json invalide produit un message clair plutôt qu'un plantage opaque en aval. Bon durcissement dans la lignée de la règle no-non-null-assertion introduite par ce MR.
🤝 Reviewé par @getCodesema[bot].
👉 Voir la review complète (diff annoté + explications)
Review générée automatiquement par l'agent Codesema.
0.11.0: server context channel (download conventions/learned rules/facts from codesema.com with staleness detection, remote_url query), multi-MR review, local branches in the sidebar, agentless diff preview. See CHANGELOG.md.
🤖 Generated with Claude Code
https://claude.ai/code/session_0148vx6QMTr5dmEgartvvzRh