fix(nodes): move test path setup to conftest and document rate-limit opt-out

Move sys.path.insert for tool_http_request into conftest.py so all
node unit tests share a single path setup entry point. Document the
"all-or-nothing" opt-out behavior in services.json rate-limit field
descriptions (all three must be 0 to disable).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: charliegillet

nodes/src/nodes/tool_http_request/services.json

@@ -114,19 +114,19 @@
 		"http_request.rateLimitPerSecond": {
 			"type": "number",
 			"title": "Max requests per second",
-			"description": "Maximum number of HTTP requests allowed per second. Uses a token-bucket algorithm for smooth enforcement.",
+			"description": "Maximum number of HTTP requests allowed per second. Uses a token-bucket algorithm for smooth enforcement. Set to 0 to disable (all three rate-limit fields must be 0 to fully opt out).",
 			"default": 10
 		},
 		"http_request.rateLimitPerMinute": {
 			"type": "number",
 			"title": "Max requests per minute",
-			"description": "Maximum number of HTTP requests allowed per minute. Provides a broader throttle beyond the per-second limit.",
+			"description": "Maximum number of HTTP requests allowed per minute. Provides a broader throttle beyond the per-second limit. Set to 0 to disable (all three rate-limit fields must be 0 to fully opt out).",
 			"default": 100
 		},
 		"http_request.maxConcurrentRequests": {
 			"type": "number",
 			"title": "Max concurrent requests",
-			"description": "Maximum number of HTTP requests that can be in-flight simultaneously.",
+			"description": "Maximum number of HTTP requests that can be in-flight simultaneously. Set to 0 to disable (all three rate-limit fields must be 0 to fully opt out).",
 			"default": 5
 		}
 	},

nodes/test/conftest.py

@@ -40,6 +40,12 @@
 PROJECT_ROOT = Path(__file__).parent.parent.parent
 sys.path.insert(0, str(PROJECT_ROOT / 'dist' / 'server'))
 
+# Add individual node source directories so unit tests can import modules
+# without triggering the top-level nodes/__init__.py (which requires the
+# engine runtime).
+NODES_SRC = Path(__file__).parent.parent / 'src' / 'nodes'
+sys.path.insert(0, str(NODES_SRC / 'tool_http_request'))
+
 # Load environment variables
 try:
     from dotenv import load_dotenv

nodes/test/test_rate_limiter.py

@@ -10,17 +10,9 @@
 import threading
 import time
 
-import sys
-from pathlib import Path
-
 import pytest
 
-# Add the node source directory to sys.path so we can import the module
-# without triggering the top-level nodes/__init__.py (which requires the
-# engine runtime).
-sys.path.insert(0, str(Path(__file__).resolve().parent.parent / 'src' / 'nodes' / 'tool_http_request'))
-
-from rate_limiter import RateLimiter, RateLimitError  # noqa: E402
+from rate_limiter import RateLimiter, RateLimitError
 
 
 class TestAcquireRelease: