As a MacBook Pro die-hard who personally can't give up the display...
Having owned and still owning both m4 (Max) and m1 generation cores (both Pro and base) - m4 generation every time, even if you need to drop to base m4.
Why? Single threaded performance is really, really much faster and it makes everything more responsive.
M1 isn't bad, but m4 is insane.
The Pro screen is nicer, speakers are nicer, etc. but the performance of the m4 generation cores vs. m1 is just night and day. If it was M3 Pro vs. M4 or M2 Pro vs. M4 it would be closer, but m1 is getting old now and the single thread performance jump is HUGE, and it makes an instantly noticeable difference to the responsiveness of apps on the machine.
The gap is big enough now that M4 base probably outperforms M1 Pro in multi core as well, and GPU would be on par but with RT support, etc. Plus the support time-frame, warranty, etc.