fix: commit minor fmt residue blocking mergemaster cherry-picks
Master had 8 uncommitted single-line whitespace changes (blank-line trimming
in test mod headers, etc.) left over from a previous mergemaster cargo-fmt
run that didn't get committed. Each subsequent merge attempt hit:
cherry-pick failed: 'Your local changes to the following files would be
overwritten by merge. Please commit your changes or stash them.'
So merges had been silently un-mergeable for the last several rounds —
mergemaster correctly reported the issue but had no way to fix master's
own state from inside the merge_workspace.
Files affected (all whitespace-only):
- chat/transport/matrix/bot/messages/{handle_message,on_room_message}.rs
- chat/transport/slack/commands/{llm,mod}.rs
- http/mcp/agent_tools/worktree.rs
- http/workflow/story_ops/{create,criterion,update}.rs
cargo clippy --all-targets -- -D warnings: clean
cargo fmt --all --check: clean
2636 tests pass.
This commit is contained in:
@@ -265,4 +265,3 @@ pub(in crate::chat::transport::matrix::bot) async fn handle_message(
|
|||||||
// ---------------------------------------------------------------------------
|
// ---------------------------------------------------------------------------
|
||||||
// Tests
|
// Tests
|
||||||
// ---------------------------------------------------------------------------
|
// ---------------------------------------------------------------------------
|
||||||
|
|
||||||
|
|||||||
@@ -8,15 +8,13 @@ use matrix_sdk::{
|
|||||||
Client,
|
Client,
|
||||||
event_handler::Ctx,
|
event_handler::Ctx,
|
||||||
room::Room,
|
room::Room,
|
||||||
ruma::{
|
ruma::events::room::message::{MessageType, OriginalSyncRoomMessageEvent},
|
||||||
events::room::message::{MessageType, OriginalSyncRoomMessageEvent},
|
|
||||||
},
|
|
||||||
};
|
};
|
||||||
use std::sync::Arc;
|
use std::sync::Arc;
|
||||||
|
|
||||||
use super::super::context::BotContext;
|
use super::super::context::BotContext;
|
||||||
use super::super::mentions::{is_addressed_to_other, is_reply_to_bot, mentions_bot};
|
|
||||||
use super::super::format::markdown_to_html;
|
use super::super::format::markdown_to_html;
|
||||||
|
use super::super::mentions::{is_addressed_to_other, is_reply_to_bot, mentions_bot};
|
||||||
use super::super::verification::check_sender_verified;
|
use super::super::verification::check_sender_verified;
|
||||||
|
|
||||||
use super::handle_message;
|
use super::handle_message;
|
||||||
@@ -598,4 +596,3 @@ pub(in crate::chat::transport::matrix::bot) async fn on_room_message(
|
|||||||
handle_message(room_id_str, incoming_room_id, ctx, sender, user_message).await;
|
handle_message(room_id_str, incoming_room_id, ctx, sender, user_message).await;
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -193,4 +193,3 @@ pub(super) async fn handle_llm_message(
|
|||||||
}
|
}
|
||||||
|
|
||||||
// ── Tests ───────────────────────────────────────────────────────────────
|
// ── Tests ───────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
|
|||||||
@@ -353,7 +353,6 @@ mod tests {
|
|||||||
use super::*;
|
use super::*;
|
||||||
use std::collections::HashMap;
|
use std::collections::HashMap;
|
||||||
|
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn parse_slash_command_payload() {
|
fn parse_slash_command_payload() {
|
||||||
let body = "command=%2Fhuskies-status&text=&user_id=U123&channel_id=C456";
|
let body = "command=%2Fhuskies-status&text=&user_id=U123&channel_id=C456";
|
||||||
|
|||||||
@@ -2,9 +2,9 @@
|
|||||||
|
|
||||||
use serde_json::{Value, json};
|
use serde_json::{Value, json};
|
||||||
|
|
||||||
|
use crate::config::ProjectConfig;
|
||||||
use crate::http::context::AppContext;
|
use crate::http::context::AppContext;
|
||||||
use crate::service::settings::get_editor_command;
|
use crate::service::settings::get_editor_command;
|
||||||
use crate::config::ProjectConfig;
|
|
||||||
use crate::worktree;
|
use crate::worktree;
|
||||||
|
|
||||||
pub(crate) async fn tool_create_worktree(args: &Value, ctx: &AppContext) -> Result<String, String> {
|
pub(crate) async fn tool_create_worktree(args: &Value, ctx: &AppContext) -> Result<String, String> {
|
||||||
|
|||||||
@@ -1,8 +1,10 @@
|
|||||||
//! create_story_file: write new story to CRDT/content store.
|
//! create_story_file: write new story to CRDT/content store.
|
||||||
|
|
||||||
|
|
||||||
#[allow(unused_imports)]
|
#[allow(unused_imports)]
|
||||||
use super::super::{create_section_content, next_item_number, read_story_content, replace_section_content, slugify_name, story_stage, write_story_content};
|
use super::super::{
|
||||||
|
create_section_content, next_item_number, read_story_content, replace_section_content,
|
||||||
|
slugify_name, story_stage, write_story_content,
|
||||||
|
};
|
||||||
|
|
||||||
pub fn create_story_file(
|
pub fn create_story_file(
|
||||||
root: &std::path::Path,
|
root: &std::path::Path,
|
||||||
|
|||||||
@@ -3,8 +3,10 @@
|
|||||||
use std::path::Path;
|
use std::path::Path;
|
||||||
|
|
||||||
#[allow(unused_imports)]
|
#[allow(unused_imports)]
|
||||||
use super::super::{create_section_content, next_item_number, read_story_content, replace_section_content, slugify_name, story_stage, write_story_content};
|
use super::super::{
|
||||||
|
create_section_content, next_item_number, read_story_content, replace_section_content,
|
||||||
|
slugify_name, story_stage, write_story_content,
|
||||||
|
};
|
||||||
|
|
||||||
pub fn check_criterion_in_file(
|
pub fn check_criterion_in_file(
|
||||||
project_root: &Path,
|
project_root: &Path,
|
||||||
@@ -229,10 +231,10 @@ pub fn add_criterion_to_file(
|
|||||||
/// - String → quoted unless it looks like a bool, integer, or inline sequence
|
/// - String → quoted unless it looks like a bool, integer, or inline sequence
|
||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
|
use super::super::update_story_in_file;
|
||||||
use super::*;
|
use super::*;
|
||||||
use serde_json::Value;
|
use serde_json::Value;
|
||||||
use std::collections::HashMap;
|
use std::collections::HashMap;
|
||||||
use super::super::update_story_in_file;
|
|
||||||
use std::fs;
|
use std::fs;
|
||||||
|
|
||||||
#[allow(dead_code)]
|
#[allow(dead_code)]
|
||||||
|
|||||||
@@ -5,11 +5,13 @@ use std::collections::HashMap;
|
|||||||
use std::path::Path;
|
use std::path::Path;
|
||||||
|
|
||||||
#[allow(unused_imports)]
|
#[allow(unused_imports)]
|
||||||
use super::super::{create_section_content, next_item_number, read_story_content, replace_section_content, slugify_name, story_stage, write_story_content};
|
use super::super::{
|
||||||
|
create_section_content, next_item_number, read_story_content, replace_section_content,
|
||||||
|
slugify_name, story_stage, write_story_content,
|
||||||
|
};
|
||||||
|
|
||||||
use crate::io::story_metadata::set_front_matter_field;
|
use crate::io::story_metadata::set_front_matter_field;
|
||||||
|
|
||||||
|
|
||||||
fn json_value_to_yaml_scalar(value: &Value) -> String {
|
fn json_value_to_yaml_scalar(value: &Value) -> String {
|
||||||
match value {
|
match value {
|
||||||
Value::Bool(b) => b.to_string(),
|
Value::Bool(b) => b.to_string(),
|
||||||
|
|||||||
Reference in New Issue
Block a user