Fix checking state cache (fix #1136), also switch to octokit methods

[itchyny]

Description:
This PR fixes #1136, also fixes a TODO comment about using octokit methods.

Related issue:

• State restoration fails if a repo has many caches #1136

Check list:

• Mark if documentation changes are required.
• Mark if tests were added or updated to cover the changes.
  • These methods are not covered by existing tests, possibly because they interact with the GitHub APIs.

[Georgegriff]

I think i ran into this issue, do you know if its likely this pr might progress?

[itchyny]

I hope some maintainer will look into this PR, but maintenance activity of this action looks to be stale.

[echang49]

Would love for a maintainer to merge this fix in. Have a feature that's built on top of this that's needed internally and will definitely help others.

[Dreamsorcerer]

@Jcambass @HarithaVattikuti @priyagupta108 Can someone merge this PR? The action is completely broken for many repos without this. We have stale issues that should have been closed weeks ago, but will never close until this fix is released.

[thehowl]

@chiranjib-swain Friendly ping to take a look at this :)

[chiranjib-swain]

Hi @itchyny, thanks for the PR!

It looks like the formatting check failed during CI. The following file(s) need to be formatted with Prettier:

src/classes/state/state-cache-storage.ts

Could you please run Prettier locally and commit the updated formatting? You can do this with:

npm run format

This will ensure your changes pass the formatting check.

Additionally, the PR is based on an older commit, which is causing errors like:

Failed to save:

Our services aren't available right now

When you get a chance, could you please merge the latest updates from the main branch into your PR?
This should resolve the issue and ensure everything builds correctly with the latest changes.

Thanks again for your contribution!

[itchyny]

@chiranjib-swain
Rebase and format done!

[Copilot]

Pull request overview

This PR fixes issue #1136 related to checking the state cache and modernizes the codebase by switching from raw HTTP requests to typed Octokit REST API methods.

Key changes:

• Replaced client.request() calls with proper client.rest.actions.* methods for better type safety and maintainability
• Fixed cache checking logic by using caches.some() instead of Boolean(caches.find()) for more idiomatic JavaScript
• Improved variable naming from issueResult to cachesResult for better clarity

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/classes/state/state-cache-storage.ts	Updated cache checking and deletion functions to use Octokit REST API methods instead of raw HTTP requests, improving type safety and code maintainability
dist/index.js	Compiled/bundled output reflecting the source code changes

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Spelling error: "exist" should be "exists".

Suggested change

	core.debug(`Error checking if cache exist: ${error.message}`);
	core.debug(`Error checking if cache exists: ${error.message}`);