Bug: need to validate goals prior to network calls

[mtfoley]

Describe the bug

If goals repos do not conform to repo full name conventions, data updates to stars etc will fail.

Steps to reproduce

Create an issue in the goals repo without using the full name convention. Wait for scheduled actions to show failures/errors.

will also happen if a renovatebot is used on the goals repo, since it creates an issue called “Dependency Dashboard”

https://docs.renovatebot.com/key-concepts/dashboard/

Screenshot of error message

Cc @takanome-dev

Affected services

opensauced.pizza

Platforms

No response

Browsers

No response

Environment

No response

Additional context

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

Contributing Docs

• I agree to follow this project's Contribution Docs

[takanome-dev]

Hey @mtfoley I'm also using renovatebot. Do you think that's the issue??

[mtfoley]

I do think that's probably it, yes. However, the actions repo should take that possibility into account.

[bdougie]

This should catch or skip those issues, here

[mtfoley]

Assigned @takanome-dev per the discussion in the Discord.

[0-vortex]

This should catch or skip those issues, here

Was thinking about a .then(issues => {}) iterator and excluding issues not created by open sauced integration a little bit above that:

actions/goals-caching/update-open-sauced-goals-cache.js

Lines 72 to 75 in 9a5db95

	stagedIssues = await octokit.rest.issues.listForRepo({
	owner: login,
	repo: "open-sauced-goals"
	})