getTasks(forceUpdate: Boolean) redundant?

`suspend fun getTasks(forceUpdate: Boolean): Result>`

This is implemented by DefaultTasksRepository. Is it really necessary if the tasks are refreshed by ViewModel, which in turn calls updateTasksFromRemoteDataSource() ?

```
override suspend fun getTasks(forceUpdate: Boolean): Result> {
if (forceUpdate) {
try {
updateTasksFromRemoteDataSource()
} catch (ex: Exception) {
return Result.Error(ex)
}
}
return tasksLocalDataSource.getTasks()
}
```

Read DefaultTasksRepository's getTasks(forceUpdate) implementation alongside the ViewModel call to updateTasksFromRemoteDataSource() and tasksLocalDataSource.getTasks(). Trace when each entry point refreshes data, then document and implement the decided repository API behavior, with tests covering refresh and local-read paths.

Written by the indexing model from the issue text.