pdurlej/platform

Fork 0

docs(openclaw): rescue scheduler observability spec kit #631

Merged

pdurlej merged 1 commit from codex/m08-openclaw-scheduler-spec into main

2026-05-30 09:13:50 +02:00

codex commented

2026-05-30 08:57:53 +02:00

Collaborator

Summary

Replacement for stale WIP PR #568. This rescues only the intended OpenClaw Scheduler Observability Spec Kit and inert Python scaffold from ollama/dziadek-openclaw-scheduler-spec, rebased onto current main.

Why replacement instead of updating #568

PR #568's source branch was old and carried accidental reverse-diff churn against current main, including many unrelated deletions. This PR keeps the valuable content and drops the stale branch drift.

What changed

Adds docs/specs/openclaw-scheduler-observability-v0/ with constitution, spec, plan, tasks, implementation notes, and README.
Adds inert scripts/openclaw_scheduler/ scaffold for Slice (a) inventory work.
Documents that this PR does not close #135 by itself; it prepares the implementation path.

Safety

No runtime calls.
No VPS1000 mutation.
No scheduler/cron/systemd writes.
No upgrade.
Inert scaffold only; functions raise NotImplementedError until Slice (a) implements read-only inventory.

Validation

python3 -m py_compile scripts/openclaw_scheduler/__init__.py scripts/openclaw_scheduler/_types.py scripts/openclaw_scheduler/inventory.py passed.
PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json passed.

Follow-up

After merge, #135 should remain open and be decomposed/continued with a concrete read-only Slice (a): scheduler inventory on VPS1000, no repairs.

Refs #135
Supersedes #568

## Summary Replacement for stale WIP PR #568. This rescues only the intended OpenClaw Scheduler Observability Spec Kit and inert Python scaffold from `ollama/dziadek-openclaw-scheduler-spec`, rebased onto current `main`. ## Why replacement instead of updating #568 PR #568's source branch was old and carried accidental reverse-diff churn against current `main`, including many unrelated deletions. This PR keeps the valuable content and drops the stale branch drift. ## What changed - Adds `docs/specs/openclaw-scheduler-observability-v0/` with constitution, spec, plan, tasks, implementation notes, and README. - Adds inert `scripts/openclaw_scheduler/` scaffold for Slice (a) inventory work. - Documents that this PR does not close #135 by itself; it prepares the implementation path. ## Safety - No runtime calls. - No VPS1000 mutation. - No scheduler/cron/systemd writes. - No upgrade. - Inert scaffold only; functions raise `NotImplementedError` until Slice (a) implements read-only inventory. ## Validation - `python3 -m py_compile scripts/openclaw_scheduler/__init__.py scripts/openclaw_scheduler/_types.py scripts/openclaw_scheduler/inventory.py` passed. - `PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json` passed. ## Follow-up After merge, #135 should remain open and be decomposed/continued with a concrete read-only Slice (a): scheduler inventory on VPS1000, no repairs. Refs #135 Supersedes #568

codex added 1 commit

2026-05-30 08:57:53 +02:00

docs(openclaw): rescue scheduler observability spec kit

base-is-main / guard (pull_request) Successful in 1s

Details

canary-required / collect-diff (pull_request) Successful in 4s

Details

patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s

Details

patchwarden-pr-sanity / collect-diff (pull_request) Successful in 5s

Details

canary-required / canary (pull_request) Has been skipped

Details

patchwarden-client-dry-run / dry-run (pull_request) Successful in 23s

Details

patchwarden-pr-sanity / sanity (pull_request) Successful in 2m31s

Details

26eaf4f10a

codex referenced this pull request

2026-05-30 08:59:18 +02:00

WIP: docs(specs): OpenClaw Scheduler Observability — Spec Kit + inventory code #568

codex commented

2026-05-30 09:05:43 +02:00

Author

Collaborator

Patchwarden PR sanity

Status: advisory_findings
PR: 631
Commit: fca8c469a025d7141b72abc304db0703d7624129
Security-sensitive label: missing
Authority: advisory model review plus deterministic blockers only
3+3 canary: still alive; this does not replace it

Deterministic findings

No deterministic findings.

Model reviewers

`global-glm` / `glm-5.1:cloud`

Status: ok
Verdict: OK
Findings: none

`global-deepseek` / `deepseek-v4-pro:cloud`

Status: ok
Verdict: OK
Findings: none

`redteam` / `kimi-k2.6:cloud`

Status: ok
Verdict: NOT_OK
medium Hardcoded UID 1000 in active cron path
- Evidence: scripts/openclaw_scheduler/inventory.py line 17: ACTIVE_CRON_PATH = Path("/run/user/1000/openclaw/cron/jobs.json"). The spec itself acknowledges in 02-plan.md risks that '/run/user/1000/openclaw/ requires correct uid context to read', yet t
- Next: Resolve the openclaw user's uid dynamically (e.g., pwd.getpwnam('openclaw').pw_uid) or accept an explicit --runtime-dir flag so the inventory script remains correct regardless of which user executes it.

Policy notes

GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot.
Optional red-team model is enabled only when PLATFORMCTL_PR_SANITY_REDTEAM_MODEL is configured.
Auto-merge is not enabled here.

# Patchwarden PR sanity - Status: `advisory_findings` - PR: `631` - Commit: `fca8c469a025d7141b72abc304db0703d7624129` - Security-sensitive label: `missing` - Authority: advisory model review plus deterministic blockers only - 3+3 canary: still alive; this does not replace it ## Deterministic findings No deterministic findings. ## Model reviewers ### `global-glm` / `glm-5.1:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ### `global-deepseek` / `deepseek-v4-pro:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ### `redteam` / `kimi-k2.6:cloud` - Status: `ok` - Verdict: `NOT_OK` - **`medium`** Hardcoded UID 1000 in active cron path - Evidence: `scripts/openclaw_scheduler/inventory.py line 17: ACTIVE_CRON_PATH = Path("/run/user/1000/openclaw/cron/jobs.json"). The spec itself acknowledges in 02-plan.md risks that '/run/user/1000/openclaw/ requires correct uid context to read', yet t` - Next: Resolve the openclaw user's uid dynamically (e.g., pwd.getpwnam('openclaw').pw_uid) or accept an explicit --runtime-dir flag so the inventory script remains correct regardless of which user executes it. ## Policy notes - GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot. - Optional red-team model is enabled only when `PLATFORMCTL_PR_SANITY_REDTEAM_MODEL` is configured. - Auto-merge is not enabled here.