From bfe70f559949f18473316d07e3252800b0dac886 Mon Sep 17 00:00:00 2001 From: dave Date: Sat, 28 Mar 2026 16:48:42 +0000 Subject: [PATCH] storkit: done 439_refactor_unify_story_stuck_states_into_a_single_status_field --- ...stuck_states_into_a_single_status_field.md | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+) create mode 100644 .storkit/work/5_done/439_refactor_unify_story_stuck_states_into_a_single_status_field.md diff --git a/.storkit/work/5_done/439_refactor_unify_story_stuck_states_into_a_single_status_field.md b/.storkit/work/5_done/439_refactor_unify_story_stuck_states_into_a_single_status_field.md new file mode 100644 index 00000000..e72b2b40 --- /dev/null +++ b/.storkit/work/5_done/439_refactor_unify_story_stuck_states_into_a_single_status_field.md @@ -0,0 +1,25 @@ +--- +name: "Deduplicate strip_prefix_ci / strip_bot_mention into chat::util" +--- + +# Refactor 439: Deduplicate strip_prefix_ci / strip_bot_mention into chat::util + +## Current State + +- TBD + +## Desired State + +Eight Matrix transport files (assign.rs, delete.rs, start.rs, rebuild.rs, reset.rs, rmtree.rs, htop.rs, timer.rs) each contain their own private copies of `strip_prefix_ci` and `strip_bot_mention`. The canonical versions already live in `chat::util` with the correct `is_char_boundary` guard. The duplicates should be removed and all call sites should use `util::strip_bot_mention` instead. + +## Acceptance Criteria + +- [ ] All 8 private copies of strip_prefix_ci are removed +- [ ] All 8 private copies of strip_bot_mention are removed +- [ ] All call sites use chat::util::strip_bot_mention instead +- [ ] Existing tests in util.rs continue to pass +- [ ] No new copies of strip_prefix_ci exist outside util.rs + +## Out of Scope + +- TBD