@team-agent/installer 0.5.8 → 0.5.9

package/crates/team-agent/src/messaging/results.rs

@@ -263,11 +263,21 @@ fn ensure_coordinator_after_collect(
 }
 
 /// `_coordinator_should_run`(`results.py:187-188`)。
+///
+/// The top-level `state.leader_receiver` read below is a coordinator-scope
+/// legacy compat bridge (pre-teams-map single-team layout) — deliberately
+/// NOT team-scoped because this predicate answers "is a coordinator worth
+/// starting for this workspace at all?" and does not itself route any
+/// message. The `ALLOWED-LEGACY-READ` marker documents the exception and
+/// keeps grep sweeps honest; delete once B1 canonical layout lands and
+/// every state carries a `teams` map
+/// (`.team/artifacts/next-version-staged-plan.md §5 Phase-Foundation-1`).
 fn coordinator_should_run(state: &serde_json::Value) -> bool {
     let has_session = state
         .get("session_name")
         .and_then(serde_json::Value::as_str)
         .is_some_and(|s| !s.is_empty());
+    // ALLOWED-LEGACY-READ: pre-teams-map coordinator-scope existence probe; never used for routing.
     has_session || leader_receiver_is_direct(state.get("leader_receiver"))
 }
 
@@ -942,6 +952,12 @@ fn report_result_for_owner_team_inner(
         );
     }
     out.insert("notification_event_id".to_string(), serde_json::Value::Null);
+    if let Some(warnings) = report_result_array(envelope, "warnings") {
+        out.insert(
+            "warnings".to_string(),
+            serde_json::Value::Array(warnings.clone()),
+        );
+    }
     Ok(serde_json::Value::Object(out))
 }
 
@@ -1036,6 +1052,9 @@ fn format_report_result_notification(
     if let Some(tests) = format_report_result_tests(envelope) {
         lines.push(tests);
     }
+    if let Some(warnings) = format_report_result_warnings(envelope) {
+        lines.push(warnings);
+    }
     if let Some(changes) = format_report_result_changes(envelope) {
         lines.push(changes);
     }
@@ -1075,6 +1094,29 @@ fn format_report_result_tests(envelope: &serde_json::Value) -> Option<String> {
     }
 }
 
+fn format_report_result_warnings(envelope: &serde_json::Value) -> Option<String> {
+    let parts = report_result_array(envelope, "warnings")?
+        .iter()
+        .filter_map(|warning| {
+            let code = report_field(warning, "code")?;
+            let field = report_field(warning, "field").unwrap_or("result");
+            let severity = report_field(warning, "severity").unwrap_or("warning");
+            let advisory =
+                if warning.get("advisory").and_then(serde_json::Value::as_bool) == Some(true) {
+                    " advisory"
+                } else {
+                    ""
+                };
+            Some(format!("{severity}{advisory} {field}: {code}"))
+        })
+        .collect::<Vec<_>>();
+    if parts.is_empty() {
+        None
+    } else {
+        Some(format!("Verification warnings: {}", parts.join(", ")))
+    }
+}
+
 fn report_result_array<'a>(
     envelope: &'a serde_json::Value,
     key: &str,
@@ -1203,6 +1245,14 @@ mod tests {
             "tests": [
                 {"command": "cargo test", "status": "passed"}
             ],
+            "warnings": [
+                {
+                    "code": "result_success_without_executed_tests",
+                    "field": "tests",
+                    "severity": "warning",
+                    "advisory": true
+                }
+            ],
             "risks": [
                 {"severity": "low", "description": "none known"}
             ],
@@ -1217,6 +1267,9 @@ mod tests {
             format_report_result_notification("res_1", "task-1", "worker", "success", &envelope);
         assert!(notification.contains("Task task-1 reported success from worker: done"));
         assert!(notification.contains("Tests: cargo test=passed"));
+        assert!(notification.contains(
+            "Verification warnings: warning advisory tests: result_success_without_executed_tests"
+        ));
         assert!(notification.contains("Changes: modified src/a.rs: patched delivery"));
         assert!(notification.contains("Risks: low: none known"));
         assert!(notification.contains("Artifacts: .team/artifacts/evidence.md: evidence"));

package/crates/team-agent/src/messaging/watchers.rs

@@ -459,15 +459,25 @@ pub(crate) fn requeue_blocked_leader_messages(
     owner_team_id: &TeamKey,
     claimed_pane_id: &PaneId,
 ) -> Result<usize, MessagingError> {
+    // E6 (0.5.9 offline-mailbox §6.5): also requeue rows that a third-party
+    // sender left in `queued_until_leader_attach` via the leader mailbox.
+    // Same idempotent requeue funnel (row/message_id/leader_notification_log
+    // PK are all preserved) — after attach/claim the coordinator's normal
+    // `deliver_pending_messages` picks them up as `accepted` and injects
+    // exactly once. status `queued_until_leader_attach` is deliberately NOT
+    // in the `claim_for_delivery` eligible set (see message_store.rs) so it
+    // could not have churned while the leader was unattached.
     let requeued = conn.execute(
         "update messages
          set status = 'accepted',
              error = null,
              updated_at = ?2
          where recipient = 'leader'
-           and status = 'failed'
-           and error = 'leader_not_attached'
-           and owner_team_id = ?1",
+           and owner_team_id = ?1
+           and (
+             (status = 'failed' and error = 'leader_not_attached')
+             or status = 'queued_until_leader_attach'
+           )",
         params![owner_team_id.as_str(), chrono::Utc::now().to_rfc3339()],
     )?;
     if requeued > 0 {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@team-agent/installer",
-  "version": "0.5.8",
+  "version": "0.5.9",
   "description": "npx installer for Team Agent",
   "keywords": [
     "codex",
@@ -20,9 +20,9 @@
     "team-agent-installer": "npm/install.mjs"
   },
   "optionalDependencies": {
-    "@team-agent/cli-darwin-arm64": "0.5.8",
-    "@team-agent/cli-darwin-x64": "0.5.8",
-    "@team-agent/cli-linux-x64": "0.5.8"
+    "@team-agent/cli-darwin-arm64": "0.5.9",
+    "@team-agent/cli-darwin-x64": "0.5.9",
+    "@team-agent/cli-linux-x64": "0.5.9"
   },
   "scripts": {
     "postinstall": "node npm/bincheck.mjs",