Drop unused Runtime::registry accessor
The accessor was added to support a "redact other DB columns" pass
that never materialized; current redaction happens inline in
Runtime::sh against the borrowed RefCell. The #[allow(dead_code)]
annotation papered over that, leaving a misleading comment about
run.rs uses that don't exist. Delete it. If a future column audit
needs registry access, add a method then.
Assisted-by: Claude Opus 4.7 (1M context)
diff --git a/src/ci/runtime.rs b/src/ci/runtime.rs
index 551d0e7..05bfdea 100644
--- a/src/ci/runtime.rs
+++ b/src/ci/runtime.rs
@@ -205,15 +205,6 @@ impl Runtime {
self.registry.borrow_mut().resolve(name)
}
- /// Borrow the secret registry for redaction.
- ///
- /// Used by run.rs to audit DB columns; kept even though
- /// current callers access the registry through the sh path.
- #[allow(dead_code)]
- pub(super) fn registry(&self) -> std::cell::Ref<'_, SecretRegistry> {
- self.registry.borrow()
- }
-
/// Run `cmd` with `opts` and record its output against the
/// current job (if one is active). Non-zero exits come back in
/// `:exit`, not as `Err`.