fix: swap action used for validating swagger file

[jpadilla]

Description

What problem is being solved?

Failures validating openapi swagger file https://github.com/openfga/api/actions/runs/21225752933/job/61082841107?pr=241

How is it being solved?

Switched github action used for that step.

What changes are made to solve it?

References

Review Checklist

• I have clicked on "allow edits by maintainers".
• I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
• The correct base branch is being used, if not main
• I have added tests to validate that the change in functionality is working as expected

Summary by CodeRabbit

• Chores
  • Enhanced API validation process in development workflows to improve overall quality assurance.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The PR updates the GitHub Actions workflow to replace the swagger-editor-validate action with apidom-validate. The change includes updating the action reference, modifying the definition-file parameter format to use single quotes, and adding a fails-on threshold parameter set to 1 to trigger failure on any validation error.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflow .github/workflows/review.yaml	Replaced swagger-editor-validate action with apidom-validate in the validate-openapi job; updated definition-file parameter to single-quoted string format; added fails-on: 1 parameter to fail on validation errors

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: swapping the GitHub Action used for validating OpenAPI/Swagger files from swagger-editor-validate to apidom-validate.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[dosubot]

Related Documentation

Checked 5 published document(s) in 0 knowledge base(s). No updates required.

^{How did I do? Any feedback?}  

[Copilot]

Pull request overview

This PR fixes a failing OpenAPI/Swagger validation step in the GitHub Actions workflow by switching to a different validation action.

Changes:

• Replaced swaggerexpert/swagger-editor-validate with swaggerexpert/apidom-validate action
• Added fails-on: 1 parameter to configure failure behavior
• Updated definition file path to use single quotes

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.github/workflows/review.yaml:
- Around line 44-47: Replace the non-existent action reference and invalid
input: change the action name
"swaggerexpert/apidom-validate@e5f5f23a9258709b041cf2b8d7040858057fface" to the
correct "swaggerexpert/swagger-editor-validate" (or, if you need ApiDOM
specifically, use "char0n/apidom-validate") and replace the unsupported
"fails-on" parameter with the correct input name "ignore-error" for
swagger-editor-validate; update the action invocation that uses definition-file
and its inputs accordingly so the workflow calls a valid action and uses valid
input keys.