Skip to content

feat(asset-creator): inspect local ComfyUI workflows before adaptation - #59

Draft
zedarvates wants to merge 1 commit into
fix/asset-creator-preflight-before-uploadfrom
feat/asset-creator-workflow-inspection
Draft

zedarvates wants to merge 1 commit into
fix/asset-creator-preflight-before-uploadfrom
feat/asset-creator-workflow-inspection

Conversation

@zedarvates

@zedarvates zedarvates commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Les quatre recettes Trellis2 référencées par l’addon restent absentes du checkout. La lecture de deux exemples publics confirme aussi que le JSON d’éditeur (nodes/widgets_values) doit être distingué du prompt API (class_type/inputs). Un inventaire local est nécessaire avant de définir leurs adaptations pour Asset Factory.

Cette PR ajoute :

  • une fonction Python inspect_workflow_file et une CLI utilisant la même logique ;
  • la reconnaissance des deux structures, l’empreinte des octets, le nombre et les types de nœuds déclarés ;
  • les versions déclarées dans cnr_id / aux_id et ver, sans les assimiler aux versions installées ;
  • l’inventaire des quatre presets avec signalement des fichiers absents et sans remplacement implicite ;
  • un résultat JSON pour les diagnostics et erreurs d’arguments, ainsi qu’un guide d’interprétation.

Le diagnostic est séparé de la soumission. Son code de sortie 0 signifie que l’inventaire a abouti ; execution_verified reste toujours false. Les liens du graphe, entrées des nœuds, modèles installés et sorties ne sont pas validés par ce résultat. Aucun outil MCP n’est enregistré ; sa fonction Python constitue un point de réemploi futur.

Preuves

Head : 157fa7df966be5fdd7ea02a83f0169540b1430b1. Arbre local préparé et arbre distant identiques : a177ab6586181ae453e7f5e4a887fdee9dd83696.

  • Linux / Python 3.12.14 : 19 tests ciblés passent, dont les 9 de fix(asset-creator): validate Trellis templates before uploading images #58 et 10 nouveaux ; sockets interdits pendant les tests.
  • Ruff 0.16.8 : lint et format des deux nouveaux fichiers Python réussis ; liens documentaires locaux et git diff --cached --check vérifiés.
  • Deux JSON du dépôt officiel visualbruno/ComfyUI-Trellis2, figés à 14597418bbe33a440ead4667e2966408f0524a21, ont été inspectés hors réseau après vérification de leurs blobs Git : 24 et 27 nœuds, format éditeur. Empreintes et inventaires sont enregistrés ; ces fichiers publics servent de références et restent distincts des recettes locales à retrouver.
  • L’inventaire du checkout signale les quatre presets absents. Aucune machine personnelle n’a été inspectée.
  • Pas de résultat GitHub Actions revendiqué pour ce nouveau commit.

Guide, sources et relevés.

Dépendance et suite

Brouillon basé sur la branche de #58, au commit 552a39febeb8ec47e7cc28bf492cc4f39b008bae, afin de garder les deux tranches séparées. #58 reste inchangée ; au contrôle du 17 septembre, SonarCloud réussit sur ce SHA et aucun run Actions n’y est associé.

La suite nécessite les recettes réellement utilisées par Asset Factory et leurs exports API depuis la même installation. Les exemples publics contiennent des générateurs différents de ceux ciblés par les modifications de widgets existantes. Les correspondances de paramètres devront donc être explicites et versionnées. Conversion API, délais HTTP, reprise des tâches, génération GPU et validation des GLB restent à établir.

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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