Confused about using separated with map/take

[trickster]

Trying to create a email address parser for example, here local_part should be john.doe etc. I am trying to just recognize this (previously using recognize, now take), In nom it used to be like this for example

fn local_part(input: &str) -> IResult<&str, &str> {
    recognize(separated_list1(char('.'), alt((atom, quoted_string))))(input)
}

use winnow::prelude::*;
use winnow::stream::AsChar;
use winnow::{
    PResult, Parser,
    combinator::{alt, delimited, separated},
    token::{one_of, take_while},
};

const SPECIAL: &'static str = "!#$%&'*+/=?^_`{|}~-";

fn atom<'a>(input: &'a mut &str) -> PResult<&'a str> {
    take_while(0.., |c: char| {
        c.is_ascii_alphanumeric() || SPECIAL.contains(c)
    })
    .parse_next(input)
}

fn quoted_string<'a>(input: &mut &'a str) -> PResult<&'a str> {
    delimited(
        one_of('"'),
        take_while(0.., |c: char| c != '"' && c != '\\'),
        one_of('"'),
    )
    .take()
    .parse_next(input)
}

fn local_part<'i>(input: &mut &'i str) -> PResult<&'i str> {
    separated(1, one_of('.'), alt((atom, quoted_string)))
        .map(|()| ())
        .take()
        .parse_next(input)
}

fn main() {}

I get this error.

r[E0308]: mismatched types
  --> src/main.rs:29:5
   |
29 | /     separated(1, one_of('.'), alt((atom, quoted_string)))
30 | |         .map(|()| ())
31 | |         .take()
   | |_______________^ one type is more general than the other
   |
   = note: expected enum `Result<&_, _>`
              found enum `Result<&_, _>`
note: the lifetime requirement is introduced here
  --> /Users/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winnow-0.6.20/src/combinator/branch.rs:55:19
   |
55 |     Alternatives: Alt<Input, Output, Error>,
   |                   ^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0521]: borrowed data escapes outside of function
  --> src/main.rs:29:5
   |
28 |   fn local_part<'i>(input: &'i mut &str) -> PResult<&'i str> {
   |                     -----          - let's call the lifetime of this reference `'1`
   |                     |
   |                     `input` is a reference that is only valid in the function body
29 | /     separated(1, one_of('.'), alt((atom, quoted_string)))
30 | |         .map(|()| ())
31 | |         .take()
32 | |         .parse_next(input)
   | |                          ^
   | |                          |
   | |__________________________`input` escapes the function body here
   |                            argument requires that `'1` must outlive `'static`
   |
   = note: requirement occurs because of a mutable reference to `&str`
   = note: mutable references are invariant over their type parameter
   = help: see <https://doc.rust-lang.org/nomicon/subtyping.html> for more information about variance

error[E0308]: mismatched types
  --> src/main.rs:29:5
   |
29 | /     separated(1, one_of('.'), alt((atom, quoted_string)))
30 | |         .map(|()| ())
31 | |         .take()
32 | |         .parse_next(input)
   | |__________________________^ one type is more general than the other
   |
   = note: expected enum `Result<&_, _>`
              found enum `Result<&_, _>`
note: the lifetime requirement is introduced here
  --> /Users/user/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winnow-0.6.20/src/combinator/branch.rs:55:19
   |
55 |     Alternatives: Alt<Input, Output, Error>,
   |                   ^^^^^^^^^^^^^^^^^^^^^^^^^

Some errors have detailed explanations: E0308, E0521.
For more information about an error, try `rustc --explain E0308`.

[trickster]

Lifetime issues resolved with

fn atom<'a>(input: &mut &'a str) -> PResult<&'a str> {
fn quoted_string<'a>(input: &mut &'a str) -> PResult<&'a str> {

[epage]

Yes, needing lifetimes in these cases is my one disappointment with switching from Fn(i) -> i to Fn(&mut i).

[trickster]

When do I use parse_next vs parse_peek?

[epage]

#624 clarifies this but it looks like it isn't released

    /// <div class="warning">
    ///
    /// Generally, prefer [`Parser::parse_next`].
    /// This is primarily intended for:
    /// - Migrating from older versions / `nom`
    /// - Testing [`Parser`]s
    ///
    /// For look-ahead parsing, see instead [`peek`].
    ///
    /// </div>