Repository navigation
Conversation
Add `tools/openapi-request-validator`, a request-only validator built on `express-openapi-validator`: an Express middleware the unit-test mock server mounts when `OPENAPI_SPEC_PATH` is set, a validating reverse proxy the integration tests run through, and a report script that aggregates the findings and emits GitHub Actions warning annotations. Both CI jobs fetch the published specification, run the tests with validation on, and print the report. Nothing is blocked or failed on a finding; the result is a warning while the specification and the client catch up with each other. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ocuALGHVYTScW9Ht9Ytjo
github-actions
Bot
added
the
t-tooling
The report script prints only the warning lines, as GitHub Actions annotations in CI, instead of a grouped text block followed by them. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ocuALGHVYTScW9Ht9Ytjo
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ocuALGHVYTScW9Ht9Ytjo
Contributor
|
✅ The link checker did not find any broken links. See more at https://github.com/apify/apify-client-js/actions/runs/40516734837#summary-122232402278 |
…kflow The Check workflow and local test runs behave as before. The new workflow_dispatch workflow runs the unit and integration tests with the validation option, on by default. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ocuALGHVYTScW9Ht9Ytjo
The workflow cannot be started by hand before it exists on master. The pull request trigger is a test aid and is removed before merging. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ocuALGHVYTScW9Ht9Ytjo
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017ocuALGHVYTScW9Ht9Ytjo
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
tools/openapi-request-validator: a request-only validator (no response checks) built onexpress-openapi-validator, with the same false-positive filters the API server uses.OPENAPI_SPEC_PATHis set; all 1175 tests still pass.pricingModel=FLAT_PRICE_PER_MONTHmissing from the spec enum, plus one validator quirk now filtered).Checkworkflow and local runs are unchanged, and a new manualRun testsworkflow runs unit + integration tests with the validation option on by default, printing findings as::warning::annotations.tools/openapi-request-validator/README.md.🤖 Generated with Claude Code
https://claude.ai/code/session_017ocuALGHVYTScW9Ht9Ytjo