Move the node generator out of scripts into node_gen #147

Merged
Claude merged 1 commit from chore/node-gen into feat/yaml-nodes 2026-10-18 06:35:58 +00:00
Collaborator

YN-31. scripts/yaml_node.py, gen_node.py, node_schema.py, build_yaml_nodes.sh and scripts/tests/ (with its fixtures) move to node_gen/ under the same names, with git mv so history follows. scripts/stamp.py and the other build and release helpers stay.

References updated: platformio.ini (the pre: script and its comment), gen_node.py's sys.path insert, the fixture glob and generator call in build_yaml_nodes.sh, the three YAML-node steps in .forgejo/workflows/build.yml (unittest discover -s node_gen/tests -t node_gen, the schema regenerate command, the build script), the usage line in node_schema.py, the fixture glob in test_node_schema.py, and the "generated by" comment the generator writes into main.cpp. A search of the whole repo for the old paths, including README.md, CONTRIBUTING.md, CLAUDE.md, docs/ and nodes/README.md, found no other mention of the generator's files, so no doc changes were needed.

No behaviour change. The generated code is the same apart from that one comment line. Tests are untouched apart from the path.

Tested: Python 163, none skipped locally, with the new discover command. The committed schema regenerates unchanged, all fixtures build through the pre: script, and pio run still builds every node.

YN-31. `scripts/yaml_node.py`, `gen_node.py`, `node_schema.py`, `build_yaml_nodes.sh` and `scripts/tests/` (with its fixtures) move to `node_gen/` under the same names, with `git mv` so history follows. `scripts/stamp.py` and the other build and release helpers stay. References updated: `platformio.ini` (the `pre:` script and its comment), `gen_node.py`'s `sys.path` insert, the fixture glob and generator call in `build_yaml_nodes.sh`, the three YAML-node steps in `.forgejo/workflows/build.yml` (unittest discover `-s node_gen/tests -t node_gen`, the schema regenerate command, the build script), the usage line in `node_schema.py`, the fixture glob in `test_node_schema.py`, and the "generated by" comment the generator writes into `main.cpp`. A search of the whole repo for the old paths, including `README.md`, `CONTRIBUTING.md`, `CLAUDE.md`, `docs/` and `nodes/README.md`, found no other mention of the generator's files, so no doc changes were needed. No behaviour change. The generated code is the same apart from that one comment line. Tests are untouched apart from the path. Tested: Python 163, none skipped locally, with the new discover command. The committed schema regenerates unchanged, all fixtures build through the `pre:` script, and `pio run` still builds every node.
Move the node generator out of scripts into node_gen
All checks were successful
Build and test / build (pull_request) Successful in 5m23s
Build and test / release (pull_request) Has been skipped
f94e2f9f4f
Claude merged commit d18c46acfb into feat/yaml-nodes 2026-10-18 06:35:58 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
CamperCan/CampervanControlSystems!147
No description provided.