refactor(daemon-service): own source access instead of injecting it - #1186
Conversation
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Performance Comparison
|
…plan-reopened' into HEAD # Conflicts: # crates/tracedecay/src/daemon/project_composition.rs
…plan-reopened' into HEAD
…plan-reopened' into HEAD
590615e
into
codex/tracedecay-total-redesign-plan-reopened
Fixes three defects introduced by #1178 and #1175, found in independent review.
refactor(daemon-service): own source access instead of injecting it— refactor(daemon-service): move query authz and remote protocol #1178 added a single-implementation injected closureimpl Fn(&ResolvedScope, &Path, &PinnedRuntimeConfiguration, UtcMicros) -> Result<ProjectSourceAccessSnapshot, _>toDaemonCallableCodeAuthorizationSource::productionandDaemonCodeGraphReadAdmission::productionpurely so relocated code could call back into root (the class refactor(workspace): drop one-impl pass-through ports outside root #1174 was deleting).daemon_owned_project_source_access_at,project_open_source_access_authority,DAEMON_REQUESTERandGRANT_HORIZONnow live intracedecay-daemon-service::callable_code_authorization— one definition each, root copies deleted — and the parameter is gone from both constructors. Semantics byte-identical (actoractor.tracedecay-daemon.project-open,project_owner_capabilities(), 24h horizon, sameInconsistent { field: "project-open requester" }mapping). Eight callers import from daemon-service directly; no root re-export.test(daemon-service): move remote observation coverage to owner—remote_query_result_observationhad been widened to unconditionalpub(pluspub mod remote_protocol) so two root tests could reach it. The two tests move into daemon-service (same names, same assertions); the function ispub(super)again and the module is private.refactor(graph-query): keep the admitted query port test-only—admitted_verified_graph_query_portwas#[cfg(test)]in root before refactor(root): contract the TraceDecay facade onto owner crates #1175 and became unconditionalpubintracedecay-graph-query; it is now#[cfg(any(test, feature = "test-helpers"))](its only caller is root'scfg(test)dispatch test support). Plaincargo check -p tracedecay-graph-queryno longer exports it.Verification: clippy
-D warnings --testson daemon-service/graph-query/root; daemon-service 258 (incl. the two moved observation tests); rootdaemon::project_open_owners13 incl.git_catalog_tests;invocation_tests::types_tests15; graph-query 55; fmt; commitlint. Two independent Opus passes; second pass's items (duplicated constants, test-only forwarding wrapper) fixed and folded.