From 5dbf4e94856e63d47accda85fbf3113b68f2bbd1 Mon Sep 17 00:00:00 2001 From: dreevelle <123705989+dreevelle@users.noreply.github.com> Date: Mon, 10 Aug 2026 12:21:36 +0100 Subject: [PATCH] Correct the local environment section The dev symlink this described is gone. All four dreevelle packages are now ComfyUI-Manager installs under their lowercase registry ids, and the working tree is not wired into ComfyUI at all, so the old wording would send someone editing files that are not live. Replaced with the accurate description, matching what ComfyUI-ResolutionTools already documented, including the reason not to leave a dev symlink in place: Manager's remove/replace path rmtree's the *resolved* path, so it deletes a symlink's target contents rather than unlinking. That destroyed a working tree on 2026-08-09. Keeping the warning in every package means it is present wherever the symlink recipe is. Docs only; CLAUDE.md is excluded from the published archive by .comfyignore, so nothing here reaches the registry. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_015p88qHFcfungDkCbG1n7z2 --- CLAUDE.md | 23 +++++++++++++++++++---- 1 file changed, 19 insertions(+), 4 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 0ebd797..b8fb0b9 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -19,10 +19,25 @@ The node is registered through ComfyUI's **V3 schema API**, not the legacy - ComfyUI checkout: `/home/dreevelle/comfy/ComfyUI` - Python venv (has `av`, `torch`, `numpy`): `/home/dreevelle/comfy-env/bin/python` -- The installed copy at `/home/dreevelle/comfy/ComfyUI/custom_nodes/ComfyUI-FastImageSequence` - is an **independent clone of the same GitHub remote**, not a symlink to this - working tree. Changes made here are not live in ComfyUI until they're pushed - and pulled there (or copied over). +- The package is published, and `ComfyUI/custom_nodes/comfyui-fast-image-sequence` + (lowercase, the registry id) is a **ComfyUI-Manager install** that tracks + registry releases. This working tree is not wired into ComfyUI. + +To test a change before releasing it, symlink this tree in under its CamelCase +name so it doesn't collide with the Manager install: + +```bash +ln -sfn /home/dreevelle/Projects/ComfyUI-FastImageSequence \ + /home/dreevelle/comfy/ComfyUI/custom_nodes/ComfyUI-FastImageSequence +``` + +**Remove that symlink before doing anything in ComfyUI-Manager for this +package.** Manager's remove/replace path does `shutil.rmtree` on the *resolved* +path, so it deletes the symlink target's contents instead of unlinking — on +2026-08-09 that wiped a sibling package's working tree, `.git` included, and it +had to be restored by re-cloning. Commit and push before any Manager operation, +and never leave a symlinked dev copy in place while installing or updating the +published version. Run ComfyUI: