Start free
Andrew Hanna

Andrew Hanna

La revue de code IA pour Salesforce a sa place dans toutes les offres, pas seulement la premium

La revue de code IA pour Salesforce a sa place dans toutes les offres, pas seulement la premium

Réponse courte : la revue automatisée lit l'Apex, les Flows et les métadonnées d'un changement avant qu'il n'atteigne une org, et signale ce qu'un relecteur humain survole : du SOQL dans une boucle, l'absence de contrôle de partage et de sécurité au niveau des champs, des ids codés en dur, des tests qui n'assertent rien, et des modifications de profil qui élargissent discrètement les accès. Cela ne vaut quelque chose que si la revue s'exécute sur chaque changement de chaque personne. C'est tout l'argument contre sa vente en option premium.

Que détecte réellement la revue de code IA avant un déploiement Salesforce ?

Trois couches. La plupart des équipes n'en inspectent qu'une.

Apex et tests

  • Du SOQL et du DML dans des boucles, ces dépassements de governor limits qui n'apparaissent qu'au volume de production.
  • Des classes sans with sharing, et des requêtes qui n'appliquent jamais le CRUD ni la sécurité au niveau des champs.
  • Des ids de record type, de profil ou propres à l'org codés en dur, qui cassent dès que le changement passe à l'org suivante.
  • Des blocs catch vides et des exceptions avalées.
  • Des classes de test qui exécutent le code sans rien asserter, ou qui s'appuient sur SeeAllData=true.
  • Une couverture d'org au-dessus de 75% alors que la nouvelle classe n'y contribue en rien.

Flows et automatisation

  • Des éléments Get Records ou DML placés dans une boucle.
  • Des chemins d'erreur absents, si bien que la panne arrive à l'utilisateur sous forme d'erreur brute.
  • Une nouvelle logique de Flow sur un objet où un trigger Apex fait déjà la même chose.

Métadonnées et permissions

C'est la couche que les outils de revue génériques ignorent, et c'est là que commencent la plupart des incidents en production.

  • Des modifications de profil ou de permission set qui accordent ModifyAllData ou ViewAllData.
  • Une sécurité au niveau des champs ouverte par effet de bord d'autre chose dans le même commit.
  • Des règles de validation ou des triggers désactivés dans le diff.
  • Un org-wide default qui passe de Private à Public Read/Write.
  • Des suppressions dans les destructive changes que personne n'a demandées.

Salesforce en fournit gratuitement une partie. Code Analyzer v5 réunit PMD, ESLint, RetireJS, un moteur regex et un scanner de Flow derrière une seule commande, et peut exécuter les règles de sécurité AppExchange via un rule selector (documentation Salesforce). C'est un bon plancher. C'est aussi une CLI, donc elle atteint surtout ceux qui vivent déjà dans un terminal.

Pourquoi un relecteur IA généraliste ne suffit-il pas ici ?

Un relecteur généraliste lit le diff comme du texte. Il ignore quels profils existent dans l'org cible, si le champ exposé contient des données réglementées, ou que le trigger qu'il lit se déclenche aussi depuis un Flow. Une revue Salesforce exige du contexte d'org : ce que le changement touche en aval, ce que l'org cible contient déjà, et ce que le déploiement s'apprête à supprimer. Sans ce contexte, vous obtenez des remarques de style soignées sur l'Apex et un silence complet sur le permission set.

Combien coûte la revue de code sur le marché du DevOps Salesforce ?

Le plus souvent, elle est facturée par personne. Gearset vend Code Reviews comme un produit distinct de ses offres DevOps, affiché à 160 $ par contributeur individuel et par mois au moment de la rédaction (tarifs Gearset). Salesforce Code Analyzer est gratuit mais taillé pour les développeurs. Entre les deux se trouve le groupe qui produit la majorité des changements dans une org type : les admins et les consultants qui travaillent dans Setup.

Pourquoi réserver la revue à une offre premium en annule-t-il l'intérêt ?

Une porte de revue vaut exactement sa couverture. Facturez-la par contributeur et trois choses suivent.

  • Le responsable du budget a une raison permanente de licencier moins de contributeurs.
  • Les contributeurs non licenciés sont en général ceux qui modifient profils, permission sets et Flows, la couche où la discipline de revue est déjà la plus faible.
  • Un contrôle qui inspecte la branche du développeur et laisse passer le changement de l'admin n'est pas une porte. C'est un rapport.

La même logique vaut à l'intérieur d'un produit. Si le contrôle qui aurait bloqué une escalade de droits se trouve dans l'offre au-dessus de la vôtre, l'escalade part quand même en production. Les contrôles qualité sont soit inclus par défaut, soit du théâtre.

C'est pourquoi Serpent met la revue de code IA sur toutes les offres, y compris le plan gratuit Essentials, avec un nombre d'utilisateurs illimité partout. Chaque changement est relu avant déploiement pour détecter les dérives de gouvernance, les problèmes de sécurité, les trous de couverture de tests et les écarts aux bonnes pratiques. La tarification est forfaitaire par entreprise plutôt que par siège : ajouter l'admin qui modifie les permission sets n'ajoute pas de ligne à la facture. Pour être précis sur la mesure : les actions IA consomment le quota mensuel de crédits, soit 30 crédits sur Essentials.

Comment placer la revue avant le déploiement ?

  1. Mettez chaque changement sous gestion de version, y compris ceux faits dans Setup par des gens qui n'ouvrent jamais Git.
  2. Lancez la revue sur le diff plutôt que sur l'org entière, pour que le résultat reste assez court pour être lu.
  3. Rendez les constats graves bloquants et le reste indicatif. Un mur d'avertissements est ignoré en une semaine.
  4. Traitez les constats sur les métadonnées et les permissions aussi sérieusement que ceux sur l'Apex.
  5. Gardez une approbation humaine sur le déploiement. La revue automatisée raccourcit la file, elle ne prend pas la décision.

FAQ

La revue de code IA remplace-t-elle la revue humaine ?

Non. Elle évacue les constats mécaniques pour que le relecteur humain garde son attention pour la logique métier et la conception.

La revue automatisée détecte-t-elle les problèmes de sécurité dans les métadonnées, et pas seulement dans l'Apex ?

Oui, si l'outil lit vraiment les métadonnées. Les changements de profils, de permission sets, de modèle de partage et de sécurité au niveau des champs sont ceux qu'il vaut le plus la peine de bloquer.

Salesforce Code Analyzer suffit-il à lui seul ?

C'est un bon plancher gratuit pour l'Apex, le JavaScript et les Flows. Il s'exécute en CLI, donc les équipes sans développeur ont généralement besoin d'un pipeline qui le lance à leur place.

La revue de code doit-elle bloquer un déploiement ou seulement avertir ?

Bloquez sur la sécurité et la perte de données, avertissez sur le style. Si tout bloque, les gens apprennent à contourner la porte.

La revue de code IA coûte-t-elle un supplément sur Serpent ?

Non. Elle est incluse dans toutes les offres, y compris Essentials gratuit, et les actions IA consomment le quota mensuel de crédits de l'offre.

Articles similaires

Curieux de livrer plus vite avant de vous lancer ? Parlons-en

Sans engagement.