4.4 KiB
4.4 KiB
Improvements Backlog
Purpose:
- Capture functional and architectural improvements discovered during analysis runs where functionality is not changed.
- Keep an implementation queue outside active feature work.
Lifecycle:
- Add item when discovered.
- When implemented, remove the item from this file and record the fix in session notes/PR.
Open Items (Beyond Current Top 10)
| ID | Date | Area | Summary | Evidence | Status |
|---|---|---|---|---|---|
| IMP-20260219-004 | 2026-02-19 | measurement editor | Label expression precedence can hide fallback label text. | nodes/measurement/measurement.html:63 |
open |
| IMP-20260219-005 | 2026-02-19 | measurement editor | success assigned without declaration in editor save path. |
nodes/measurement/measurement.html:131 |
open |
| IMP-20260219-016 | 2026-02-19 | generalFunctions/state | movementManager constructor writes startup console.log on runtime path, adding noisy non-structured logs. |
nodes/generalFunctions/src/state/movementManager.js:16 |
open |
| IMP-20260219-018 | 2026-02-19 | generalFunctions/helper | Legacy menu endpoint still uses generated class source (menuUtils.legacy.js) as compatibility fallback; plan removal after UI validation of stable bootstrap/data path. |
nodes/generalFunctions/src/helper/endpointUtils.js:113, nodes/generalFunctions/src/helper/menuUtils.js:575 |
open |
| IMP-20260219-019 | 2026-02-19 | generalFunctions/menu | menuUtils.js embeds extensive runtime console.* and inline API/debug logging in generated client scripts, increasing noise and making prod diagnostics harder. |
nodes/generalFunctions/src/helper/menuUtils.js:35 |
open |
| IMP-20260219-020 | 2026-02-19 | generalFunctions/predict | interpolation.js contains raw runtime debug log (console.log(this.interpolationtype)), leaking internal state in production paths. |
nodes/generalFunctions/src/predict/interpolation.js:127 |
open |
| IMP-20260219-021 | 2026-02-19 | generalFunctions/menu | Two active menu utility implementations (menuUtils.js and menuUtils_DEPRECATED.js) still coexist, increasing drift risk and maintenance overhead. |
nodes/generalFunctions/src/helper/menuUtils.js:1, nodes/generalFunctions/src/helper/menuUtils_DEPRECATED.js:1 |
open |
| IMP-20260219-022 | 2026-02-19 | generalFunctions/outliers | DynamicClusterDeviation.update() emits verbose console.log traces on each call with no log-level guard, unsafe for production telemetry volume. |
nodes/generalFunctions/src/outliers/outlierDetection.js:7 |
open |
| IMP-20260224-006 | 2026-02-24 | rotatingMachine prediction fallback | When only one pressure side is available, predictor uses absolute pressure as surrogate differential, which can materially bias flow prediction under varying suction/discharge conditions. | nodes/rotatingMachine/src/specificClass.js:573, nodes/rotatingMachine/src/specificClass.js:588 |
open |
| IMP-20260224-012 | 2026-02-24 | cross-node unit architecture | Canonical unit-anchor strategy is implemented in rotatingMachine plus phase-1 controllers (machineGroupControl, pumpingStation, valve, valveGroupControl); continue rollout to remaining nodes so all runtime paths use canonical storage + explicit ingress/egress units. |
nodes/machineGroupControl/src/specificClass.js:42, nodes/pumpingStation/src/specificClass.js:48, nodes/valve/src/specificClass.js:87, nodes/valveGroupControl/src/specificClass.js:78 |
open |
| IMP-20260323-001 | 2026-03-23 | architecture/security | temp/cloud.yml stores environment credentials directly in a repository-tracked target-state stack example; replace with env placeholders/secret injection and split illustrative architecture from deployable manifests. |
temp/cloud.yml:1 |
open |
| IMP-20260323-002 | 2026-03-23 | architecture/configuration | Intended database-backed configuration authority (tagcodering) is not yet visibly integrated as the primary runtime config backbone in this repository; define access pattern, schema ownership, and rollout path for edge/site/central consumers. |
architecture/stack-architecture-review.md:1 |
open |
| IMP-20260323-003 | 2026-03-23 | architecture/telemetry | Multi-level smart-storage strategy is a stated architecture goal, but signal classes, reconstruction guarantees, and authoritative-layer rules are not yet formalized; define telemetry policy before broad deployment. | architecture/stack-architecture-review.md:1 |
open |