Is there some general logic for what code should be @prefect.task'd and what code can be left in the @prefect.flow without tasks? (talking strictly prefect 2)

use tasks when you need more fine-granular visibility or task specific features such as running tasks in parallel/concurrently + caching + concurrency limit (tags)