From 36be6f1081d634d7386ccdbfc3310eb3242f3385 Mon Sep 17 00:00:00 2001 From: Melvin Strobl Date: Sat, 15 Aug 2026 22:07:31 +0200 Subject: [PATCH] Drop the InfluxDB rotation item, note the orphaned flow state The token does not need rotating. Deleting a flow does leave its value, timestamp, version and history keys behind in Redis. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_01KkmeRiyeYmVZqJVwuyHq9o --- NOTEPAD.md | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/NOTEPAD.md b/NOTEPAD.md index 5a878eb..f3765f0 100644 --- a/NOTEPAD.md +++ b/NOTEPAD.md @@ -18,8 +18,10 @@ Deferring because out of scope is fine, but don't mention deferring than. - CHORE/UI: `make test-backend` cannot reach Postgres while the integrated stack is up — `compose.local.yml` does `db: ports: !reset []`. Run it against the container's address, or move the suite inside the compose network. +- BUG/FLOW: deleting a flow leaves its state in Redis — the value, timestamp, version and + history keys under `pipeline:{flow}.*` all survive. Clear the namespace on delete. - CHORE/UI: the Playwright specs run against the development stack and leave their users and - flows behind, which is why the flowbar fills with `test_flow_*`. Give them their own data + flows behind, which is why the flowbar filled with `test_flow_*`. Give them their own data or clean up after themselves, as `pytest` now does. - CHORE/UI: the standalone Vite origin (`make dev-frontend` on :5173) is still not in `BACKEND_CORS_ORIGINS`. `scripts/setup.sh` regenerates the list on every `make init` and @@ -28,9 +30,6 @@ Deferring because out of scope is fine, but don't mention deferring than. - CHORE/INFRA: `test-backend.yml` runs pytest on the runner host, so it keeps the fixed container name and port 5432 that `compose.ci.yml` resets everywhere else. Two concurrent runs of that one job collide. -- CHORE/FLOW: the InfluxDB token committed in the old `flow/nodes/influx_test.py` is in - pushed history (`d222ce5`). Rotate it in InfluxDB; removing it from history needs a - rewrite of published commits. - PERF/UI: (deferred for now) the Monaco chunk is 2.6 MB. It only loads when a node panel opens, but the editor could be trimmed further or swapped for CodeMirror if that becomes a problem. - CHORE/FLOW: `app/flow/nodes.py` is excluded from strict mypy (`[[tool.mypy.overrides]]` in