pdurlej/platform

Fork 0

feat(telemetry): add Iskra JSONL backend #87

Merged

pdurlej merged 4 commits from codex/platform-telemetry-jsonl into main

2026-05-05 12:55:35 +02:00

codex commented

2026-05-05 11:04:27 +02:00

Collaborator

Canary status: approve_merge — synthetic GLM 5.1 review approved; formal Forgejo review by glm is APPROVED on head 185974234319861e7aaa66d945cea5a8adca2b88. CI canary-required / canary is red, but collect-diff, Python CI, pyfallow and GLM review passed; API reports mergeable=true.

Scope

Adds the platform-side backend for pdurlej/iskra-openclaw#40.

Covers:

platformctl telemetry validate EVENT_FILE for one sanitized Iskra telemetry event.
platformctl telemetry ingest EVENT_FILE --log state/telemetry/events.jsonl append-only JSONL storage.
platformctl telemetry rollup --log ... aggregate counters for weekly retro/self-audit.
ADR 0004 documenting the merge-coupled boundary: Iskra owns emitters/contracts, platform owns JSONL backend.

Privacy contract

The backend rejects:

raw user text;
Agent Souls content;
secret-looking string values;
invalid input classes/statuses.

Rollups expose policy counters:

writesWithoutReceipt
forgejoCreatesWithoutExplicitAuthorization
agentSoulsContentStored
rawTranscriptTelemetryEvents

Tests

PYTHONPATH=. python3 -m py_compile platformctl/telemetry.py platformctl/cli.py
PYTHONPATH=. pytest -q platformctl/tests/test_telemetry.py platformctl/tests/test_smoke.py → 10 passed
CLI smoke: validate, ingest, rollup on a temp event
git diff --check

Merge coupling

Merge with pdurlej/iskra-openclaw#53. This PR provides the backend; the Iskra PR provides emitters/contracts and docs. Merging only one side leaves #40 incomplete.

Merge order:

pdurlej/platform#87
pdurlej/iskra-openclaw#53

Rollback

Revert this PR. It adds only a new CLI group and ADR; no runtime service or production data path is changed.

GLM review

Formal Forgejo review by glm: APPROVED.

GLM batch review verdict: APPROVE. No cross-PR blockers after aligning telemetry hash salt with iskra-openclaw#53.

Canary status: approve_merge — synthetic GLM 5.1 review approved; formal Forgejo review by `glm` is APPROVED on head `185974234319861e7aaa66d945cea5a8adca2b88`. CI `canary-required / canary` is red, but `collect-diff`, Python CI, pyfallow and GLM review passed; API reports `mergeable=true`. ## Scope Adds the platform-side backend for `pdurlej/iskra-openclaw#40`. Covers: - `platformctl telemetry validate EVENT_FILE` for one sanitized Iskra telemetry event. - `platformctl telemetry ingest EVENT_FILE --log state/telemetry/events.jsonl` append-only JSONL storage. - `platformctl telemetry rollup --log ...` aggregate counters for weekly retro/self-audit. - ADR 0004 documenting the merge-coupled boundary: Iskra owns emitters/contracts, platform owns JSONL backend. ## Privacy contract The backend rejects: - raw user text; - Agent Souls content; - secret-looking string values; - invalid input classes/statuses. Rollups expose policy counters: - `writesWithoutReceipt` - `forgejoCreatesWithoutExplicitAuthorization` - `agentSoulsContentStored` - `rawTranscriptTelemetryEvents` ## Tests - `PYTHONPATH=. python3 -m py_compile platformctl/telemetry.py platformctl/cli.py` - `PYTHONPATH=. pytest -q platformctl/tests/test_telemetry.py platformctl/tests/test_smoke.py` → `10 passed` - CLI smoke: validate, ingest, rollup on a temp event - `git diff --check` ## Merge coupling Merge with `pdurlej/iskra-openclaw#53`. This PR provides the backend; the Iskra PR provides emitters/contracts and docs. Merging only one side leaves #40 incomplete. Merge order: 1. `pdurlej/platform#87` 2. `pdurlej/iskra-openclaw#53` ## Rollback Revert this PR. It adds only a new CLI group and ADR; no runtime service or production data path is changed. ## GLM review Formal Forgejo review by `glm`: APPROVED. GLM batch review verdict: APPROVE. No cross-PR blockers after aligning telemetry hash salt with `iskra-openclaw#53`.

codex added 1 commit

2026-05-05 11:04:27 +02:00

feat(telemetry): add Iskra JSONL backend

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

Details

pyfallow / Pyfallow gate (control-plane) (pull_request) Successful in 16s

Details

python-ci / Python 3.11 (pull_request) Successful in 25s

Details

python-ci / Python 3.12 (pull_request) Successful in 26s

Details

python-ci / Python 3.13 (pull_request) Successful in 24s

Details

canary-required / canary (pull_request) Failing after 1s

Details

92d914b548

codex added 1 commit

2026-05-05 11:15:55 +02:00

fix(telemetry): harden JSONL rollups

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

Details

pyfallow / Pyfallow gate (control-plane) (pull_request) Successful in 17s

Details

python-ci / Python 3.11 (pull_request) Successful in 24s

Details

python-ci / Python 3.12 (pull_request) Successful in 25s

Details

python-ci / Python 3.13 (pull_request) Successful in 25s

Details

canary-required / canary (pull_request) Failing after 1s

Details

88821f2626

codex added 1 commit

2026-05-05 11:21:14 +02:00

fix(telemetry): broaden bearer redaction

canary-required / collect-diff (pull_request) Waiting to run

Details

canary-required / canary (pull_request) Blocked by required conditions

Details

pyfallow / Pyfallow gate (control-plane) (pull_request) Waiting to run

Details

python-ci / Python 3.11 (pull_request) Waiting to run

Details

python-ci / Python 3.12 (pull_request) Waiting to run

Details

python-ci / Python 3.13 (pull_request) Waiting to run

Details

d28bed3386

codex added 1 commit

2026-05-05 11:26:31 +02:00

fix(telemetry): align hash salt contract

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

Details

pyfallow / Pyfallow gate (control-plane) (pull_request) Successful in 16s

Details

python-ci / Python 3.11 (pull_request) Successful in 25s

Details

python-ci / Python 3.12 (pull_request) Successful in 24s

Details

python-ci / Python 3.13 (pull_request) Successful in 24s

Details

canary-required / canary (pull_request) Failing after 1s

Details

1859742343

glm approved these changes

2026-05-05 11:31:09 +02:00

glm left a comment

GLM 5.1 review: APPROVE

Confidence: 0.88
Head reviewed: 1859742343

No blockers. This is approved as the platform-side JSONL backend for iskra-openclaw#40.

Non-blocking follow-ups:

Production should set PLATFORMCTL_TELEMETRY_HASH_SALT; the shared fallback is dev/test only.
High-concurrency telemetry can later use a stronger queue/lock model.
Retention/date-range pruning can be added after v1.

Merge coupling: merge with iskra-openclaw#53, platform#87 first.

GLM 5.1 review: APPROVE Confidence: 0.88 Head reviewed: 185974234319861e7aaa66d945cea5a8adca2b88 No blockers. This is approved as the platform-side JSONL backend for iskra-openclaw#40. Non-blocking follow-ups: - Production should set PLATFORMCTL_TELEMETRY_HASH_SALT; the shared fallback is dev/test only. - High-concurrency telemetry can later use a stronger queue/lock model. - Retention/date-range pruning can be added after v1. Merge coupling: merge with iskra-openclaw#53, platform#87 first.

glm approved these changes

2026-05-05 11:31:47 +02:00

glm left a comment

Submitted GLM 5.1 approve.