From 4458369d6470a68980aa92b7b30d19889cd439a0 Mon Sep 17 00:00:00 2001 From: "K. Isom" Date: Wed, 23 Sep 2026 07:16:56 -0700 Subject: [PATCH] M3b plan: task 11 gives Podman::egress_dir(), so the field is read Without it the field was never read in task 11, clippy's dead_code failed the gate, and #[allow] is forbidden; the implementer stopped on the dilemma. Co-Authored-By: Claude Opus 5.5 (1M context) --- docs/plans/M3b/11-brokerd-container.md | 3 +++ docs/plans/M3b/README.md | 4 ++++ 2 files changed, 7 insertions(+) diff --git a/docs/plans/M3b/11-brokerd-container.md b/docs/plans/M3b/11-brokerd-container.md index 675c09b..8a45093 100644 --- a/docs/plans/M3b/11-brokerd-container.md +++ b/docs/plans/M3b/11-brokerd-container.md @@ -35,6 +35,9 @@ pub struct Podman { /* runner: Runner, egress_dir: PathBuf, log: Log, next: Atom impl Podman { /// `egress_dir` is `Config::egress_dir()`; task 12 uses it. pub fn new(runner: Runner, egress_dir: PathBuf, log: Log) -> Podman; + /// Where `http_fetch` calls get their directories. Nothing in this task calls it; it is public, + /// so the field counts as read and clippy's `dead_code` does not fire. Do not add `#[allow]`. + pub fn egress_dir(&self) -> &Path; } impl Runtime for Podman { fn run(&self, spec: &RunSpec) -> Result; diff --git a/docs/plans/M3b/README.md b/docs/plans/M3b/README.md index 0a0ebcd..ff18927 100644 --- a/docs/plans/M3b/README.md +++ b/docs/plans/M3b/README.md @@ -65,6 +65,10 @@ At the end: `make gate` prints `gate: ok` with about 638 tests. `egress-proxy` never matched; and `serve` joined each handler thread, serving one connection at a time. The attempt is saved in `.state/runs/M3b/08-first-attempt.diff`; the tree was reset, and the task now says both things explicitly and to debug inside the repository. Resume from task 08. +- 2026-09-23, task 11: the session ended in deliberation, with nothing written. The task stored + `egress_dir` for task 12 but gave nothing that read it, so clippy's `dead_code` would fail the + gate and `#[allow]` is forbidden: a task that could not be written as described (tip T16). The + reference had a public `egress_dir()` getter; the task now gives it. Resume from task 11. ## Running it