Skip to content

id: validation gap — P4.11 DataScheme update-cost measurement at 100/1,000/5,000 members was never recorded #1396

Closed

Description

@dezren39

Gap found by the #1075 validation audit (Phase 4, P4.11) — 2026-10-27

The plan of record (doc/2026-10-11T11-08-42Z_design_remaining_work/plan-phases.md, P4.11) requires: "Measure DataScheme update cost at 100, 1,000 and 5,000 members with our adapter, for the BeeKEM comparison — recorded in the phase document; no pass or fail."

A fresh search on main 452d69e6 (2026-10-27) finds no such measurement recorded: no update-cost figures at 100/1,000/5,000 members in the phase documents under doc/2026-10-11T11-08-42Z_design_remaining_work/, in pkgs/id/src/encryption/tests.rs, or elsewhere in the repo (the BeeKEM assessment records why BeeKEM was not adopted, but not our adapter's measured update cost at those scales). The related size evidence that does exist is welcome-size only (test_each_epoch_adds_about_forty_bytes_to_the_welcome, ~40.0 B/epoch, P4.4).

Acceptance criteria

  • DataScheme update cost (time and/or message bytes) measured with the real adapter at 100, 1,000 and 5,000 members and recorded in the Phase 4 phase document, in a form the BeeKEM comparison can cite.
  • The bench/test that produces the numbers is committed and re-runnable (name it in the document).

Found by: queue task 1256 (validation audit of #1075), slice 3 (Phases 2–4 walk). Parent audit: #1075. Path-to-Phase-12 queue: #1235.

Activity

  1. added
    kind/testTest coverage, test infrastructure, or test-only fixes
    on Oct 27, 2026
  2. developing-today commented on Oct 27, 2026

    @developing-today
    Contributor
  3. added
    priority/highImportant and time-sensitive
    app/idPackage/app: id (pkgs/ or apps/ in this repo)
    nowOrg Project gate: active priority — orchestrator adds to the board
    on Oct 27, 2026
  4. dezren39 commented on Nov 2, 2026

    @dezren39
    MemberAuthor

    🤖 Agent note · state: picked up

  5. dezren39 commented on Nov 2, 2026

    @dezren39
    MemberAuthor

    🤖 Agent note · state: done

    Triage verdict — #1396: ALREADY DONE (audit miss); close, no follow-up packet

    The P4.11 measurement was recorded on main two weeks before this issue was filed — verified fresh this run against origin/main (0e26fa4b3), and back-checked at the issue's own search base (452d69e6, 2026-10-27):

    1. Runnable benchmark in tree: pkgs/id/src/encryption/tests.rs — measure_update_cost() plus test_the_update_cost_measurement_grows_with_the_members plus the ignored release benchmark bench_update_cost() iterating [100, 1_000, 5_000] members (run: cargo test --release --lib encryption::tests::bench_update_cost -- --ignored --nocapture). Present verbatim at 452d69e6 (lines ~2388–2397), so it predates this issue. Added by commit 2a3de9a9 "test(rust): DataScheme update cost measurement [P4.11]" (2026-10-12).
    2. Numbers recorded in the phase document: doc/2026-10-12T14-33-36Z_phase_4_groups_encryption/2026-10-12T14-33-36Z_phase_4_groups_encryption.md, section "P4.11: DataScheme update cost (measured)" — full table at 100 / 1,000 / 5,000 members (update bytes 30.7 KB / 310 KB / 1.54 MB; ~309 B and ~100 µs per member), side-by-side BeeKEM comparison table, fan-out break-even, and reading. That doc's own status table marks "P4.11 DataScheme cost measurement … done … numbers below". Recorded by commit f5dc8041 "docs(id): record P4.8 to P4.11 and the DataScheme update cost [phase 4]" (2026-10-12).
    3. Vehicle: both commits reached main via PR Phase 4: groups, encryption, and history #191 (adc1cad0, merged 2026-10-12) — 15 days before this issue's 2026-10-27 filing.

    Why the audit missed it: the audit searched doc/2026-10-11T11-08-42Z_design_remaining_work/ (the plan directory, which holds the P4.11 requirement, not results) and did not surface the results doc (doc/2026-10-12T14-33-36Z_phase_4_groups_encryption/) or the tests.rs benchmark, both of which already existed at its search base.

    Why no follow-up packet: there is no remaining work — the instrument exists and the numbers are recorded with the BeeKEM comparison the plan asked for. (One optional nit for the owner, not queue work: the phase-4 doc notes the BeeKEM assessment itself was deliberately not edited with the measured break-even; owner call.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

app/idPackage/app: id (pkgs/ or apps/ in this repo)kind/testTest coverage, test infrastructure, or test-only fixesnowOrg Project gate: active priority — orchestrator adds to the boardpriority/highImportant and time-sensitive

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions