SIENTIAPDE-1972: F3 close-out - ledger de commits + carry-forward notes (Task 16.end)

This commit is contained in:
PedroHMCosme
2026-07-15 08:50:59 -03:00
parent 52223e6c09
commit e9ce9e50e2

View File

@@ -91,7 +91,14 @@ At the end of each phase Fx, before starting the next:
- **asyncua version hygiene:** requirements pin `asyncua==1.1.5` but the dev/test env runs 1.1.8. The DataValue/StatusCode_ behavior was verified identical on both, but align the pin (bump to 1.1.8) or the env during F4 (Task 21) so version-specific bugs can't slip through.
- **`_SESSION_BAD_NAMES` in write.py** duplicates a subset of Laborious's reconnectable-error list (cross-repo) — move to `sientia_do` if it drifts (ponytail note in code).
- **`_server_cfg` in write.py** is minimal (url/security off the cmd); full connector-DB lookup by `server_id` is Task 18 (F4 wiring) — ponytail note in code.
**F3 — Active reads + gRPC (C4):** _(fill at F3 close-out)_
**F3 — Active reads + gRPC (C4)** — base `4855205`. Status: ✅ complete, 206 unit tests green, ruff clean.
- `99a715c` feat(gateway): ActiveReadEngine (scheduled + read_now) (Task 14)
- `1798b69` feat(gateway): gRPC proto + generated stubs (C4) (Task 15)
- `52223e6` feat(gateway): gRPC server ReadTag (C4) (Task 16)
- Carry-forward to Task 18 (wiring):
- Wire the gRPC `slot_lookup` seam (address→names from the current slot) so on-demand reads persist; if the lookup becomes I/O-bound, move it inside the servicer's try/except.
- gRPC server is internal-only, no auth (no consumer yet — likely frontend later, per feature owner).
- Edge case (minor): `read_now` with `SourceTimestamp=None` persists `timestamp=None`; add a guard if it matters in practice.
**F4 — Wiring, health, deploy, hardening:** _(fill at F4 close-out)_
### Final: open per-phase PRs (only after F4 is complete, tested, working)
@@ -839,9 +846,9 @@ async def test_readtag_maps_response():
---
### Task 16.end: F3 close-out (see [Phase Delivery](#phase-delivery--commit-ledger))
- [ ] Full `pytest tests` green + `ruff` clean.
- [ ] Record F3 commit SHAs (base + range) in the [Commit Ledger](#commit-ledger).
- [ ] Do NOT open the PR yet.
- [x] Full `pytest tests` green (206) + `ruff` clean.
- [x] F3 commit SHAs recorded in the [Commit Ledger](#commit-ledger) (base `4855205`).
- [ ] PR intentionally NOT opened yet — opened only after F4 closes.
---