Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ hot cases; the one-time cases are justified on clarity grounds alone:
per pushed message;
- the client-pool scheduler couples an `Arc<PoolScheduler>` root, a
`Mutex<SchedulerState>`, an atomic single-flight flag, and spawn-on-demand
service tasks (`src/client/pool/scheduler.rs`) — once per acquisition or
service tasks (`src/client/pool/scheduler/mod.rs`) — once per acquisition or
lease drop;
- `ProtocolHooks::from_protocol` clones the same `Arc` of the protocol
object once per callback adapter — six clones for one protocol
Expand Down
2 changes: 1 addition & 1 deletion docs/adr-013-client-pool-scheduler-and-slot-ownership.md
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ single-owner coordination rule.
Primary code surfaces:

- `src/client/pool/client_pool.rs`;
- `src/client/pool/scheduler.rs`;
- `src/client/pool/scheduler/mod.rs`;
- `src/client/pool/handle.rs`;
- `src/client/pool/lease.rs`;
- `src/client/pool/slot.rs`;
Expand Down
4 changes: 2 additions & 2 deletions docs/adr-014-server-shutdown-ownership.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,11 @@ This ADR governs the server-runtime half of Epic

Primary code surfaces:

- `src/server/runtime/mod.rs` and `src/server/runtime/server.rs`;
- `src/server/runtime/mod.rs` and `src/server/runtime/server/mod.rs`;
- `src/server/runtime/supervisor.rs`;
- `src/server/shutdown.rs`;
- `src/server/error.rs`;
- `src/metrics.rs`.
- `src/metrics/mod.rs`.

Related issues and decisions:

Expand Down
2 changes: 1 addition & 1 deletion docs/developers-guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -1092,7 +1092,7 @@ build standard. The exception stands. Three test binaries fail:
`panic_in_server_is_mapped_to_io_error_other`, a handler panic that is not
caught.

The `src/metrics.rs` doctest still fails to link, with undefined AWS-LC
The `src/metrics/mod.rs` doctest still fails to link, with undefined AWS-LC
symbols. Re-measure on the next toolchain pin.

## Cargo workspace semantics
Expand Down
2 changes: 1 addition & 1 deletion docs/runtime-ownership-baselines.md
Original file line number Diff line number Diff line change
Expand Up @@ -776,7 +776,7 @@ The mechanism the rework removed is verifiable in the source, and it is worth
stating because it is what makes the result explicable rather than merely
observed. `stop_servicing` — the only writer that clears `is_servicing` to
false — has exactly one caller, `take_next_waiter_or_stop`
(`src/client/pool/scheduler.rs`). The immediate-lease arm instead resolves
(`src/client/pool/scheduler/mod.rs`). The immediate-lease arm instead resolves
through `next_waiter`, which selects a queued request and never clears the
flag, so a servicing worker can no longer be orphaned by a concurrent resolver.
The pre-#535 code cleared the guard on that path, leaving waiters queued with
Expand Down
2 changes: 1 addition & 1 deletion docs/users-guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -2906,7 +2906,7 @@ directories (one per shard on full dispatch runs) for offline triage.
[^29]: Implemented in `src/fairness.rs` (lines 1-79).
[^30]: Implemented in `src/connection.rs` (lines 240-441).
[^31]: Implemented in `src/connection.rs` (lines 22-46).
[^33]: Implemented in `src/metrics.rs` (lines 21-111).
[^33]: Implemented in `src/metrics/mod.rs` (lines 21-111).
[^34]: Implemented in `src/response.rs` (lines 46-151).
[^35]: Implemented in `src/response.rs` (lines 156-209).
[^36]: Demonstrated in `examples/async_stream.rs` (lines 1-23).
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -392,7 +392,7 @@ where
}

#[cfg(test)]
#[path = "scheduler_tests.rs"]
#[path = "../scheduler_tests.rs"]

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Update source-of-truth docs for the moved modules

This move leaves current traceability documentation pointing to files that no longer exist: docs/adr-013-client-pool-scheduler-and-slot-ownership.md:110 names src/client/pool/scheduler.rs, docs/adr-014-server-shutdown-ownership.md:41-45 names both src/server/runtime/server.rs and src/metrics.rs, and docs/users-guide.md:2857 also names src/metrics.rs. Update these active references to their new mod.rs paths so the repository's documented source surfaces remain navigable; historical execution-plan references can remain unchanged.

AGENTS.md reference: AGENTS.md:L44-L47

Useful? React with 👍 / 👎.

mod poison_tests;
mod service;
#[cfg(test)]
Expand Down
File renamed without changes.
Original file line number Diff line number Diff line change
Expand Up @@ -395,5 +395,5 @@ where
}

#[cfg(test)]
#[path = "server/observation_tests.rs"]
#[path = "observation_tests.rs"]
mod observation_tests;
4 changes: 2 additions & 2 deletions tests/workflow_contracts/dependency_scoping_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,8 @@ def test_prometheus_exporter_remains_available_to_non_loom_doctests(manifest: di
"metrics-exporter-prometheus must be available to non-Loom doctests"
)
assert "use metrics_exporter_prometheus::PrometheusBuilder;" in (
ROOT / "src" / "metrics.rs"
).read_text(), "src/metrics.rs must retain the documented Prometheus example"
ROOT / "src" / "metrics" / "mod.rs"
).read_text(), "src/metrics/mod.rs must retain the documented Prometheus example"


def test_metrics_feature_does_not_enable_exporter(manifest: dict[str, typ.Any]) -> None:
Expand Down
Loading