Make mission log readable without --entry

• [MSN] No active missions on the board
• [EXC] Board idle, no stories queued or active
• [HLT] 2 warnings, no structural errors detected

Author: rupurt

crates/keel-cli/src/cli/commands/management/mission/log.rs

@@ -0,0 +1,78 @@
+//! Mission log command
+
+use std::fs;
+use std::path::Path;
+
+use anyhow::Result;
+
+use keel::domain::model::Board;
+use keel::infrastructure::loader::load_board;
+use keel::read_model::show_selector::{ShowEntityKind, resolve_show_selector};
+
+/// Show the mission log for a mission with an explicit board directory.
+pub fn run_with_dir(board_dir: &Path, id: &str) -> Result<()> {
+    let content = read_log_with_dir(board_dir, id)?;
+    print!("{content}");
+    if !content.ends_with('\n') {
+        println!();
+    }
+    Ok(())
+}
+
+/// Read the raw mission log content for a mission.
+pub fn read_log_with_dir(board_dir: &Path, id: &str) -> Result<String> {
+    let board = load_board(board_dir)?;
+    let resolved_id = resolve_mission_id_with_board(board_dir, &board, id)?;
+    let mission = board.require_mission(&resolved_id)?;
+    let log_path = mission.path.parent().unwrap().join("LOG.md");
+    Ok(fs::read_to_string(log_path)?)
+}
+
+/// Resolve a mission ID or HEAD selector for mission-log operations.
+pub fn resolve_mission_id(board_dir: &Path, id: &str) -> Result<String> {
+    let board = load_board(board_dir)?;
+    resolve_mission_id_with_board(board_dir, &board, id)
+}
+
+fn resolve_mission_id_with_board(board_dir: &Path, board: &Board, id: &str) -> Result<String> {
+    Ok(resolve_show_selector(
+        board_dir,
+        board,
+        ShowEntityKind::Mission,
+        id,
+    )?)
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use keel::application::mission_lifecycle::MissionLifecycleService;
+    use keel::test_helpers::{TestBoardBuilder, TestMission};
+
+    #[test]
+    fn read_log_with_dir_returns_full_log_contents() {
+        let temp = TestBoardBuilder::new()
+            .mission(TestMission::new("M1").title("Mission One"))
+            .build();
+
+        MissionLifecycleService::log(temp.path(), "M1", "First entry").unwrap();
+        MissionLifecycleService::log(temp.path(), "M1", "Second entry").unwrap();
+
+        let log = read_log_with_dir(temp.path(), "M1").unwrap();
+
+        assert!(log.contains("# Mission One - Decision Log"));
+        assert!(log.contains("First entry"));
+        assert!(log.contains("Second entry"));
+    }
+
+    #[test]
+    fn resolve_mission_id_accepts_head_selectors() {
+        let temp = TestBoardBuilder::new()
+            .mission(TestMission::new("M2").title("Mission Two"))
+            .mission(TestMission::new("M1").title("Mission One"))
+            .build();
+
+        assert_eq!(resolve_mission_id(temp.path(), "HEAD").unwrap(), "M1");
+        assert_eq!(resolve_mission_id(temp.path(), "HEAD~").unwrap(), "M2");
+    }
+}

crates/keel-cli/src/cli/commands/management/mission/mod.rs

@@ -7,6 +7,7 @@ use clap::{ArgGroup, Subcommand};
 
 pub mod attach;
 pub mod list;
+pub mod log;
 pub mod new;
 pub mod next;
 pub mod play;
@@ -115,13 +116,13 @@ pub enum MissionAction {
         /// Mission ID
         id: String,
     },
-    /// Add entry to mission log
+    /// Show mission log or add entry to mission log
     Log {
-        /// Mission ID
+        /// Mission ID or HEAD selector (HEAD, HEAD~, HEAD~~, HEAD^)
         id: String,
         /// Log entry text
         #[arg(long)]
-        entry: String,
+        entry: Option<String>,
     },
     /// Digest mission log entries
     Digest {
@@ -197,11 +198,17 @@ pub fn run(ctx: &spoke_auth::ExecutionContext, action: MissionAction) -> Result<
         MissionAction::Abandon { id } => {
             keel::application::mission_lifecycle::MissionLifecycleService::abandon(&board_dir, &id)
         }
-        MissionAction::Log { id, entry } => {
-            keel::application::mission_lifecycle::MissionLifecycleService::log(
-                &board_dir, &id, &entry,
-            )
-        }
+        MissionAction::Log { id, entry } => match entry {
+            Some(entry) => {
+                let resolved_id = log::resolve_mission_id(&board_dir, &id)?;
+                keel::application::mission_lifecycle::MissionLifecycleService::log(
+                    &board_dir,
+                    &resolved_id,
+                    &entry,
+                )
+            }
+            None => log::run_with_dir(&board_dir, &id),
+        },
         MissionAction::Digest { id } => {
             keel::application::mission_lifecycle::MissionLifecycleService::digest(&board_dir, &id)
         }

crates/keel-cli/src/cli_tests.rs

@@ -443,6 +443,32 @@ fn cli_parses_mission_resume() {
     ));
 }
 
+#[test]
+fn cli_parses_mission_log_without_entry() {
+    let cli = Cli::try_parse_from(["board", "mission", "log", "M1"]).unwrap();
+    assert!(matches!(
+        cli.command,
+        Commands::Management(ManagementCommands::Mission {
+            action: MissionAction::Log { id, entry: None }
+        }) if id == "M1"
+    ));
+}
+
+#[test]
+fn cli_parses_mission_log_with_entry() {
+    let cli =
+        Cli::try_parse_from(["board", "mission", "log", "M1", "--entry", "Did some work"]).unwrap();
+    assert!(matches!(
+        cli.command,
+        Commands::Management(ManagementCommands::Mission {
+            action: MissionAction::Log {
+                id,
+                entry: Some(entry)
+            }
+        }) if id == "M1" && entry == "Did some work"
+    ));
+}
+
 #[test]
 fn cli_parses_pulse_with_json() {
     let matches = crate::cli::build_cli()