feat: OpenClaw foundation — MCP-first universal runtime

.claude/skills/doctor.md

@@ -138,22 +138,22 @@ Look for:
 
 If logs show ONLY startup messages but no ongoing publishing, the node may be stalled.
 
-### Step 9: ros-z Topic Compatibility
-Check for hyphens in topic names AND machine IDs (ros-z rejects hyphens in path components):
+### Step 9: Topic Format Validation
+Check for hyphens in topic names AND machine IDs (use underscores for consistency):
 ```bash
 # Check config files for hyphenated topics
 for cfg in ~/.bubbaloop/nodes/*/config.yaml ~/.bubbaloop/nodes/*/*/config.yaml; do
   [ -f "$cfg" ] || continue
   if grep -qP 'publish_topic:.*-' "$cfg" 2>/dev/null; then
-    echo "WARNING: $cfg has hyphenated topic (ros-z will reject)"
+    echo "WARNING: $cfg has hyphenated topic (should use underscores)"
   fi
 done
 
 # Check service files for unhyphenated machine IDs
 for f in ~/.config/systemd/user/bubbaloop-*.service; do
   mid=$(grep 'BUBBALOOP_MACHINE_ID=' "$f" 2>/dev/null | grep -oP '=\K.*')
   if [ -n "$mid" ] && echo "$mid" | grep -q '-'; then
-    echo "WARNING: $f has hyphenated MACHINE_ID=$mid (ros-z will reject)"
+    echo "WARNING: $f has hyphenated MACHINE_ID=$mid (should use underscores)"
   fi
 done
 ```
@@ -181,7 +181,7 @@ for svc in $(systemctl --user list-units 'bubbaloop-*' --no-pager --plain | grep
 done
 ```
 
-Data topics use underscores (ros-z requirement) but schema queryables might use hyphens.
+Data topics use underscores (convention) but schema queryables might use hyphens.
 This mismatch prevents `discoverSchemaForTopic()` from mapping topics to schemas.
 If `--fix`: update node code to use consistent naming (prefer underscores for both).
 

.claude/skills/gemini-review.md

@@ -54,7 +54,7 @@ Focus on:
 1. Correctness bugs (logic errors, off-by-one, race conditions)
 2. API misuse (wrong function signatures, missing error handling)
 3. Security issues (injection, unsafe patterns)
-4. Zenoh/ros-z specific issues (topic naming, session lifecycle)
+4. Zenoh-specific issues (topic naming, session lifecycle, queryable patterns)
 
 For each finding, categorize as:
 - CRITICAL: Must fix before merge

.claude/skills/validate.md

@@ -1,55 +1,75 @@
 # Validate System
 
-Full system validation workflow for bubbaloop. Runs all test suites, linting, and optionally consults Gemini CLI for a second review.
+Full system validation workflow for bubbaloop. Runs all test suites, linting, and optionally MCP smoke tests.
 
 ## Usage
 ```
-/validate            # Full validation (Rust + dashboard + clippy)
-/validate --quick    # Rust only, skip dashboard
+/validate            # Full validation (Rust + MCP integration + clippy)
+/validate --quick    # Rust unit tests only
+/validate --smoke    # Full validation + MCP stdio smoke test
 /validate --gemini   # Full validation + Gemini CLI review
 ```
 
 ## Steps
 
 ### Step 1: Rust Compilation Check
-Run `cargo check --lib -p bubbaloop` to verify the library compiles. This is fast and catches most issues.
+Run `pixi run check` to verify the full crate compiles (includes MCP as core dependency).
 
-### Step 2: Rust Test Suite
-Run `cargo test --lib -p bubbaloop` and report the number of passing tests. All tests must pass.
+### Step 2: Clippy Lint
+Run `pixi run clippy` and verify zero warnings. The project enforces `-D warnings` with `--all-features`.
 
-### Step 3: Dashboard Tests (skip if --quick)
-Run `cd /home/nvidia/bubbaloop/dashboard && npm test` and report the number of passing tests across test files. All tests must pass.
+### Step 3: Rust Unit Tests
+Run `pixi run test` and report the number of passing tests. All tests must pass. Expected: ~325 tests.
+
+### Step 4: MCP Integration Tests (skip if --quick)
+Run `cargo test --features test-harness --test integration_mcp` and report results. These test the full MCP protocol stack (tool routing, RBAC, validation) against MockPlatform over duplex transport. Expected: ~35 tests.
+
+### Step 5: MCP Stdio Smoke Test (only if --smoke)
+Test the real binary with a live MCP request over stdio:
+
+```bash
+(printf '{"jsonrpc":"2.0","method":"initialize","id":1,"params":{"protocolVersion":"2024-11-05","capabilities":{},"clientInfo":{"name":"smoke-test","version":"1.0"}}}\n{"jsonrpc":"2.0","method":"notifications/initialized"}\n{"jsonrpc":"2.0","method":"tools/call","id":2,"params":{"name":"get_machine_info","arguments":{}}}\n'; sleep 10) | timeout 15 ./target/release/bubbaloop mcp --stdio 2>/dev/null
+```
 
-### Step 4: Clippy Lint
-Run `pixi run clippy` and verify zero warnings. The project enforces `-D warnings`.
+Verify:
+- Response id=1: initialize response with `protocolVersion` and `capabilities.tools`
+- Response id=2: `get_machine_info` returns JSON with `arch`, `hostname`, `os` fields
 
-### Step 5: Template Integrity Check
+If the binary doesn't exist, build it first with `cargo build --release` (takes ~9 minutes on ARM64).
+
+### Step 6: Dashboard Tests (skip if --quick)
+Run `cd /home/nvidia/bubbaloop/dashboard && npm test` and report the number of passing tests across test files. All tests must pass.
+
+### Step 7: Template Integrity Check
 Verify template files exist and have required content:
 - `templates/rust-node/node.yaml.template` must contain `{{node_name}}`
 - `templates/python-node/node.yaml.template` must contain `{{node_name}}`
 - `templates/rust-node/Cargo.toml.template` must NOT have ambiguous git+path for bubbaloop-schemas
 - Python template must NOT contain `complete=True` on queryables
 
-### Step 6: Gemini CLI Review (only if --gemini)
+### Step 8: Gemini CLI Review (only if --gemini)
 Use `NODE_OPTIONS="--max-old-space-size=4096" gemini` to review changed files:
 - Pipe each changed file via stdin with a focused review prompt
 - Categorize findings as: FIX (real bugs), DEFER (pre-existing/roadmap), or FALSE POSITIVE
 - Only fix real bugs; document deferred items
 
-### Step 7: Summary Report
+### Step 9: Summary Report
 Output a table with:
 
 | Check | Result | Details |
 |-------|--------|---------|
-| Rust compile | PASS/FAIL | |
-| Rust tests | PASS/FAIL | N tests |
-| Dashboard tests | PASS/FAIL/SKIP | N tests, M files |
+| Compile | PASS/FAIL | |
 | Clippy | PASS/FAIL | N warnings |
+| Unit tests | PASS/FAIL | N tests |
+| MCP integration | PASS/FAIL/SKIP | N tests |
+| MCP smoke test | PASS/FAIL/SKIP | stdio init + tool call |
+| Dashboard tests | PASS/FAIL/SKIP | N tests, M files |
 | Templates | PASS/FAIL | |
 | Gemini review | N findings | X fixed, Y deferred |
 
-### Important Constraints (Jetson)
+### Important Constraints (Jetson ARM64)
 - Do NOT run parallel cargo/pixi commands -- too slow on ARM64
 - Run steps sequentially
-- Use `cargo check --lib -p bubbaloop` (not full binary check, needs dashboard/dist/)
+- `pixi run check` (not `cargo check --lib`) to include all features
 - Use `NODE_OPTIONS="--max-old-space-size=4096"` for Gemini CLI to avoid OOM
+- Release builds take ~9 minutes -- only build for --smoke tests

.gitignore

@@ -73,5 +73,6 @@ Thumbs.db
 # Tool config
 .omc/
 
-# External repo assembly (not part of this repo)
+# External repos and stale build artifacts (node source lives in separate repos)
 bubbaloop-nodes-official/
+crates/bubbaloop-nodes/