docs: add Kan mcporter registration smoke #42

Merged

pdurlej merged 1 commit from codex/openclaw-mcporter-registration into codex/mcp-ops-smoke-health

2026-05-10 13:47:13 +02:00

codex commented

2026-05-10 01:27:48 +02:00

Collaborator

Canary status: missing — fire 3+3 review before merge

Summary

Adds the safe Kan-ductor side of the OpenClaw/mcporter registration work: a named kan config template, a local config validator, and documentation for owner-gated schema probing on VPS1000. This avoids touching the dirty local iskra-openclaw worktree or production OpenClaw config.

Closes #18.

Scope

Adds scripts/kan-mcporter-config-smoke.mjs.
Supports --print-template for the canonical mcpServers.kan.url entry.
Validates existing config via MCPORTER_CONFIG / OPENCLAW_KAN_MCP_URL.
Optionally probes mcporter list kan --schema via --probe-cli for owner-approved VPS1000 smoke.
Documents tailnet-only/named-server assumptions and server-side token boundary.

Non-goals

No production OpenClaw config mutation.
No secret/token changes.
No Matrix integration.
No edits to the currently dirty iskra-openclaw repo.

Spec sources read

Issue #18 acceptance criteria
docs/openclaw-kan-mcp.md
docs/agent-mcp-contract.md
Local iskra-openclaw repo status and mcporter/OpenClaw references

Tests / smoke

node --check scripts/kan-mcporter-config-smoke.mjs
node scripts/kan-mcporter-config-smoke.mjs --print-template
MCPORTER_CONFIG=<tmp-valid-config> node scripts/kan-mcporter-config-smoke.mjs
missing-server negative check exits 4
pnpm exec prettier --check scripts/kan-mcporter-config-smoke.mjs docs/openclaw-kan-mcp.md docs/agent-mcp-contract.md
git diff --check

Rollback

Remove the validator script and revert the docs to ad hoc URL instructions. No runtime state is changed.

Owner gates

Owner approval is still required before changing production OpenClaw/mcporter config or running --probe-cli on VPS1000.

Canary status: missing — fire 3+3 review before merge ## Summary Adds the safe Kan-ductor side of the OpenClaw/mcporter registration work: a named `kan` config template, a local config validator, and documentation for owner-gated schema probing on VPS1000. This avoids touching the dirty local `iskra-openclaw` worktree or production OpenClaw config. Closes #18. ## Scope - Adds `scripts/kan-mcporter-config-smoke.mjs`. - Supports `--print-template` for the canonical `mcpServers.kan.url` entry. - Validates existing config via `MCPORTER_CONFIG` / `OPENCLAW_KAN_MCP_URL`. - Optionally probes `mcporter list kan --schema` via `--probe-cli` for owner-approved VPS1000 smoke. - Documents tailnet-only/named-server assumptions and server-side token boundary. ## Non-goals - No production OpenClaw config mutation. - No secret/token changes. - No Matrix integration. - No edits to the currently dirty `iskra-openclaw` repo. ## Spec sources read - Issue #18 acceptance criteria - `docs/openclaw-kan-mcp.md` - `docs/agent-mcp-contract.md` - Local `iskra-openclaw` repo status and mcporter/OpenClaw references ## Tests / smoke - `node --check scripts/kan-mcporter-config-smoke.mjs` - `node scripts/kan-mcporter-config-smoke.mjs --print-template` - `MCPORTER_CONFIG=<tmp-valid-config> node scripts/kan-mcporter-config-smoke.mjs` - missing-server negative check exits `4` - `pnpm exec prettier --check scripts/kan-mcporter-config-smoke.mjs docs/openclaw-kan-mcp.md docs/agent-mcp-contract.md` - `git diff --check` ## Rollback Remove the validator script and revert the docs to ad hoc URL instructions. No runtime state is changed. ## Owner gates Owner approval is still required before changing production OpenClaw/mcporter config or running `--probe-cli` on VPS1000.

codex added 1 commit

2026-05-10 01:27:49 +02:00

docs(ops): add kan mcporter registration smoke 3dd440f457

codex commented

2026-05-10 08:48:14 +02:00

Author

Collaborator

3+3 Review Result

Status: OK with residual risk

Review lanes completed: tech-ollama-1, tech-ollama-2, tech-codex, product-ollama-1, product-ollama-2, product-claude-opus.

Deterministic checks on current stack tip:

PASS: @kan/db typecheck/build
PASS: @kan/shared typecheck/build
PASS: @kan/mcp typecheck/build
PASS: @kan/api typecheck/build and focused agent tests: 28/28
FAIL: targeted web ESLint currently reports apps/web/src/views/card/components/ActivityList.tsx:374 unused isAdmin
FAIL, known broader baseline: @kan/web typecheck still has existing bootstrap/workspace typing errors

Canary gate: current automated canary is not green for the draft stack; treat this as a review/checkpoint result, not merge approval.

Review notes:

No 3+3 blocker found. Residual risk: mcporter probe depends on external CLI/config shape; keep it smoke-only and non-mutating.

## 3+3 Review Result Status: **OK with residual risk** Review lanes completed: tech-ollama-1, tech-ollama-2, tech-codex, product-ollama-1, product-ollama-2, product-claude-opus. Deterministic checks on current stack tip: - PASS: `@kan/db` typecheck/build - PASS: `@kan/shared` typecheck/build - PASS: `@kan/mcp` typecheck/build - PASS: `@kan/api` typecheck/build and focused agent tests: 28/28 - FAIL: targeted web ESLint currently reports `apps/web/src/views/card/components/ActivityList.tsx:374` unused `isAdmin` - FAIL, known broader baseline: `@kan/web typecheck` still has existing bootstrap/workspace typing errors Canary gate: current automated canary is **not green** for the draft stack; treat this as a review/checkpoint result, not merge approval. Review notes: - No 3+3 blocker found. Residual risk: mcporter probe depends on external CLI/config shape; keep it smoke-only and non-mutating.