Batch Cache Operations can't return an empty list
Nobody has claimed this yet.
- Dominant language
- Bikeshed
- Stars
- 3.6k
- Forks
- 324
- Avg merge
- 14d 22h
- Merged PRs (30d)
- 1
Description
For batch cache operations, a request/response is appended to the list for every operation, so the list always has a size of number of operations.
Then the cache.delete steps are
- Let requestResponses be the result of running Batch Cache Operations with operations. If this throws an exception, set errorData to the exception.
- If requestResponses is not empty, resolve cacheJobPromise with true.
But this also means that when no entries are matched, it also returns a length of 1, meaning it always should be returning true?
I'm sorry if I'm misunderstanding anything but I'm stuck here.
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 reading the linked Batch Cache Operations algorithm and the cache.delete steps in the Service Workers specification. Trace how an operation with no matching cache entry contributes to requestResponses. Done means the specification clearly defines the empty-match case and the resulting cache.delete value, with any needed wording or algorithm correction reflected in both steps.
Written by the indexing model from the issue text.
Assessment
- Domain
- documentation
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100