Remove app server started completion time

This commit is contained in:
Ahmed Ibrahim
2026-05-28 11:16:30 -07:00
parent f71b306e3f
commit ba193ee95b
5 changed files with 2 additions and 11 deletions
@@ -1390,7 +1390,6 @@ fn app_server_started_event_serializes_expected_shape() {
runtime: sample_runtime_metadata(),
remote_control_enabled: true,
duration_ms: 987,
completed_at: 12,
},
});
@@ -1408,8 +1407,7 @@ fn app_server_started_event_serializes_expected_shape() {
"runtime_arch": "aarch64"
},
"remote_control_enabled": true,
"duration_ms": 987,
"completed_at": 12
"duration_ms": 987
}
})
);
@@ -1715,7 +1713,6 @@ async fn app_server_started_fact_emits_event() {
runtime: sample_runtime_metadata(),
remote_control_enabled: true,
duration_ms: 456,
completed_at: 12,
},
)),
&mut events,
@@ -1735,8 +1732,7 @@ async fn app_server_started_fact_emits_event() {
"runtime_arch": "aarch64"
},
"remote_control_enabled": true,
"duration_ms": 456,
"completed_at": 12
"duration_ms": 456
}
}])
);
-2
View File
@@ -22,7 +22,6 @@ use crate::facts::ThreadStartTimingFact;
use crate::facts::TrackEventsContext;
use crate::facts::TurnResolvedConfigFact;
use crate::facts::TurnTokenUsageFact;
use crate::now_unix_seconds;
use crate::reducer::AnalyticsReducer;
use codex_app_server_protocol::ClientRequest;
use codex_app_server_protocol::ClientResponsePayload;
@@ -172,7 +171,6 @@ impl AnalyticsEventsClient {
runtime: current_runtime_metadata(),
remote_control_enabled,
duration_ms,
completed_at: now_unix_seconds(),
}),
));
}
-1
View File
@@ -150,7 +150,6 @@ pub(crate) struct AppServerStartedEventParams {
pub(crate) runtime: CodexRuntimeMetadata,
pub(crate) remote_control_enabled: bool,
pub(crate) duration_ms: u64,
pub(crate) completed_at: u64,
}
#[derive(Serialize)]
-1
View File
@@ -279,7 +279,6 @@ pub(crate) struct AppServerStartedInput {
pub runtime: CodexRuntimeMetadata,
pub remote_control_enabled: bool,
pub duration_ms: u64,
pub completed_at: u64,
}
pub struct ThreadStartTimingFact {
-1
View File
@@ -543,7 +543,6 @@ impl AnalyticsReducer {
runtime: input.runtime,
remote_control_enabled: input.remote_control_enabled,
duration_ms: input.duration_ms,
completed_at: input.completed_at,
},
},
));