8000 T67 — Inject store values in bep serve · Issue #2586 · gominimal/minimal · GitHub
[go: up one dir, main page]

Skip to content
8000

T67 — Inject store values in bep serve #2586

Closed

Description

@norrietaylor

Task

slice size difficulty model who estimated lines
Round-two conditions closed M needs care strongest an agent 280 (120 code, 160 tests)

A box's request to a store rule's registered upstream leaves bep serve with the stored key in the registered form, and the box never holds the key.

Covers:

  • BEP-032

Layers:

  • redeem:
    • crates/bep/src/redeemer.rs: BoxRedeemer overrides Redeemer::redeem_reference and returns Redeemed::Store for an admitted store member (Gatehouse §6.10 line 849, redemption step 4), with the header check still before redeem runs
    • crates/bep/src/proxy.rs: the bep serve path calls Flow::with_store(Arc::new(HostStore)) where it builds each Flow
    • crates/bep/src/forward.rs: the two interim notes on redeem_reference and with_store (T67 — Inject store values in bep serve #2586) are removed
    • cargo nextest run -p bep serve_injects_a_stored_key_for_an_admitted_store_member

Waits on:

Plan block
### T67 — Inject store values in bep serve
  slice: S10
  kind: task
  covers: BEP-032
  size: M
  est: 120 code, 160 tests
  executor: agent
  difficulty: needs care
  model: strongest
  diagnostics: an admitted store reference that the serve path cannot read is refused as `store_item_unreadable` in `min box audit`, never forwarded without the key
  observability: one debug line per store injection naming the store and identifier, never the value
  layers:
    - redeem:
      - crates/bep/src/redeemer.rs: `BoxRedeemer` overrides `Redeemer::redeem_reference` and returns `Redeemed::Store` for an admitted store member (Gatehouse §6.10 line 849, redemption step 4), with the `header` check still before `redeem` runs
      - crates/bep/src/proxy.rs: the `bep serve` path calls `Flow::with_store(Arc::new(HostStore))` where it builds each `Flow`
      - crates/bep/src/forward.rs: the two interim notes on `redeem_reference` and `with_store` (#2586) are removed
      test: cargo nextest run -p bep serve_injects_a_stored_key_for_an_admitted_store_member
  produces: none
  consumes: none
  proof: A box's request to a store rule's registered upstream leaves `bep serve` with the stored key in the registered form, and the box never holds the key.
  depends on:
    - blocked by #2257
    - blocked by #2292

Generated by Claude Code

Activity

  1. added
    kind:featureA new feature or capability. Applied by the inbox-route classifier.
    on Oct 25, 2026
  2. norrietaylor commented on Oct 26, 2026

    @norrietaylor
    MemberAuthor

    The epic owner added this residual to the plan on 2026-10-26 as T67 (slice S10). Its body is now the rendered task. The residual text it replaces is kept below.

    Residual text

    Residual from T28 #2574, arch C1 (ruling), epic #1501. The interim notes in crates/bep/src/forward.rs point to #2280, which is closed, so this issue now owns the work.

    #2574 makes the forwarder inject a stored key in the registered form. bep serve does not inject a store value yet, so the T28 proof holds in the forwarder only. BEP-032 stays open until redemption step 4 runs in bep serve.

    What closes this

    • crates/bep/src/redeemer.rs: BoxRedeemer overrides Redeemer::redeem_reference and returns Redeemed::Store for an admitted store member (Gatehouse §6.10 line 849, redemption step 4). The header check stays before redeem runs.
    • crates/bep/src/proxy.rs: the bep serve path calls Flow::with_store(Arc::new(HostStore)).
    • The two interim notes in crates/bep/src/forward.rs (on redeem_reference and with_store) are deleted.
    • A bep serve test injects a stored key for an admitted store member.

    Generated by Claude Code

  3. changed the title [-]Inject store values in bep serve: BoxRedeemer redeem_reference and Flow::with_store (BEP-032)[/-] [+]T67 — Inject store values in bep serve[/+] on Oct 26, 2026
  4. added
    kind:taskA task the plan skill rendered; the body carries ## Task
    size:M200 to 500 net lines; one PR
    model:strongestThe strongest model available: unknown ground, or large work that needs care
    and removed
    kind:featureA new feature or capability. Applied by the inbox-route classifier.
    on Oct 26, 2026
  5. minimal-foundry commented on Oct 26, 2026

    @minimal-foundry
    Contributor

    Done. #2597 merged.

    branch agent/2586-inject-store-values-in-bep-serve · epic #1501 · PR #2597

    Updated 2026-10-26T09:51:25.849Z.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    kind:taskA task the plan skill rendered; the body carries ## Taskmodel:strongestThe strongest model available: unknown ground, or large work that needs caresize:M200 to 500 net lines; one PR

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      0