Incorrect cache key in documentation

[ethan-mills-ukef]

Description:
In the advanced usage documentation, "Restore-Only Cache" specifies a cache key of ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}.

The actual cache key used is node-cache-${platform}-${arch}-${packageManager}-${fileHash}as defined at https://github.com/actions/setup-node/blob/670825a89dc0abd596e7a3abd0f5e3f6e5faf37c/src/cache-restore.ts#L43C21-L43C70

Action version:
Main

Platform:
N/A

Runner type:
N/A

Tools version:

Repro steps:
A description with steps to reproduce the issue. If you have a public example or repo to share, please provide the link.

Expected behavior:
Documentation reflects action behaviour

Actual behavior:
Documentation contains incorrect information

[ethan-mills-ukef]

I'm not sure if the instructions in the documentation will do what they claim at all - won't that save the cache on miss?

[v-mahabaleshwars]

Hi @ethan-mills-ukef,
Thank you for creating this issue. We will investigate it and provide feedback as soon as we have some updates.

[v-priya-kinthali]

Hello @ethan-mills-ukef👋,
We've raised a PR#1550 to update the documentation to reflect the actual cache key format used by the action.
Please take a look. Thanks!

[ethan-mills-ukef]

Looks great, thanks @priya-kinthali , @mahabaleshwars !

[v-priya-kinthali]

Hello @ethan-mills-ukef 👋,
Thanks for the confirmation. Since the documentation update PR#1550 has been merged, we are proceeding with closing the issue. Thanks again for bringing this to our attention.