Mitch
f52bf9d8ea
Pytest treats the rootdir as a Package because the ComfyUI entry point __init__.py lives there, and it inferred the rootdir's package name from the dir name. In local checkouts the dir is `hold_counter`, which clashed with our subpackage of the same name and caused `import hold_counter.node` to resolve to the rootdir's __init__.py (no `node` attribute). The CI runner sees a different dir name (`ComfyUI-HoldCounter`), where the same machinery instead tried to import the rootdir __init__.py standalone with no parent package and exploded on the relative import. Renaming the subpackage to `_impl/` (signalling it is internal) breaks the name clash and makes pytest collection work in both environments. Also adds `pythonpath = ["."]` and `--import-mode=importlib` to pyproject's pytest config so the rootdir is importable without sys.path hacks and pytest does not synthesise a Package collector. Other CI hygiene (also folded in here): * setup-uv@v3 -> @v6, actions/checkout@v4 -> @v5 to clear the GitHub Node.js 20 deprecation warnings. * setup-uv `cache-dependency-glob: pyproject.toml` (uv.lock is gitignored, so the default **/uv.lock glob fails the job). Public-facing surface unchanged (NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS, WEB_DIRECTORY remain at the rootdir __init__.py, which is the only thing ComfyUI ever imports).
26 lines
427 B
Python
26 lines
427 B
Python
"""Implementation package for the HoldCounter ComfyUI custom node."""
|
|
|
|
from .node import (
|
|
DEFAULT_MODE,
|
|
MODES,
|
|
HoldCounter,
|
|
NodeState,
|
|
)
|
|
|
|
NODE_CLASS_MAPPINGS = {
|
|
"HoldCounter": HoldCounter,
|
|
}
|
|
|
|
NODE_DISPLAY_NAME_MAPPINGS = {
|
|
"HoldCounter": "Hold Counter",
|
|
}
|
|
|
|
__all__ = [
|
|
"DEFAULT_MODE",
|
|
"MODES",
|
|
"NODE_CLASS_MAPPINGS",
|
|
"NODE_DISPLAY_NAME_MAPPINGS",
|
|
"HoldCounter",
|
|
"NodeState",
|
|
]
|