feat(asset-creator): inspect local ComfyUI workflows before adaptation - #59
Draft
zedarvates wants to merge 1 commit into
Draft
zedarvates wants to merge 1 commit into
zedarvates wants to merge 1 commit into
Conversation
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




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 :
inspect_workflow_fileet une CLI utilisant la même logique ;cnr_id/aux_idetver, sans les assimiler aux versions installées ;Le diagnostic est séparé de la soumission. Son code de sortie 0 signifie que l’inventaire a abouti ;
execution_verifiedreste toujoursfalse. 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.git diff --cached --checkvérifiés.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.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.