Support ByteStream input for string manipulation commands
Open
Nobody has claimed this yet.
A:streaming
category:meta-issue
- Dominant language
- Rust
- Stars
- 40.5k
- Forks
- 2.3k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 79
Description
Some commands which operate on strings could also operate on ByteStream values which are ByteStreamType::String as well. We don't need to coerce into a string for all operations. Additionally, not all of these commands need to have streamed output; for example, str stats has collected output but also needlessly collects its input.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the string manipulation commands and their handling of ByteStreamType::String, then inspect str stats as the concrete example in the issue. Done means applicable commands accept ByteStream string values without unnecessary coercion, and commands such as str stats do not needlessly collect their input.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100