Remove unused --sort-order flag from codec and simplex

[nh13]

Summary

• Removes the --sort-order / -S CLI argument from codec and simplex consensus callers
• The flag was defined but never wired through — the output header is hardcoded to SO:unknown in create_unmapped_consensus_header(), so passing --sort-order Queryname silently did nothing
• duplex never had this flag; clip's --sort-order is functional and unaffected

Test plan

• cargo ci-test — 1998 tests pass
• cargo ci-fmt — clean
• cargo ci-lint — clean
• Verified fgumi codec --help and fgumi simplex --help no longer show --sort-order

[nh13]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.06%. Comparing base (91e5af6) to head (ce498f8).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #209   +/-   ##
=======================================
  Coverage   88.06%   88.06%           
=======================================
  Files         113      113           
  Lines       52795    52818   +23     
=======================================
+ Hits        46495    46516   +21     
- Misses       6300     6302    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 647fa629-74b2-4423-8ea5-7dde871472a5

📥 Commits

Reviewing files that changed from the base of the PR and between 90ec6fe and ce498f8.

📒 Files selected for processing (2)

• src/commands/codec.rs
• src/commands/simplex.rs

💤 Files with no reviewable changes (2)

• src/commands/codec.rs
• src/commands/simplex.rs

---

📝 Walkthrough

Walkthrough

The sort_order CLI option (-S/--sort-order) was removed from multiple command structures. This involved deleting the pub sort_order: Option<String> field from affected command structs and updating their corresponding test helper functions to no longer initialize this field. No other code paths or runtime behavior depend on this removed option.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	Title clearly describes the main change—removing the unused --sort-order flag from codec and simplex commands.
Description check	✅ Passed	Description is directly related to the changeset, explaining why the flag was removed, what issue it caused, and confirmation that tests pass.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch nh/fix_codec-consensus-calling

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}