ci(spec): add repository validation checks#4
Conversation
|
Warning Review limit reached
More reviews will be available in 10 minutes and 41 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more credits in the billing tab to continue. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Ready to review this PR? Stage has broken it down into 3 individual chapters for you:
Chapters generated by Stage for commit 5121a20 on Jun 12, 2026 7:19pm UTC. |
There was a problem hiding this comment.
No issues found across 1 file
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Architecture diagram
sequenceDiagram
participant Dev as Developer
participant CI as GitHub Actions CI
participant Mise as mise (task runner)
participant Tools as CLI Tools (jq,ajv)
participant Schemas as Schema Files
participant Fixtures as Fixture Files
participant Manifest as Manifest JSON
Note over Dev,CI: CI Pipeline Validation Flow
Dev->>CI: Push code changes
CI->>Mise: mise run check
Mise->>Mise: Resolve tasks in dependency order
alt check:json phase
Mise->>Tools: jq empty schema/*.json
Tools-->>Mise: JSON syntax OK
Mise->>Tools: jq empty fixtures/validation/*.json
Tools-->>Mise: JSON syntax OK
Mise->>Tools: jq empty *.trail.jsonl (one per file)
Tools-->>Mise: JSONL syntax OK
end
alt check:schema phase
Mise->>Tools: ajv compile schema/draft.json
Schemas->>Tools: Draft schema
Tools-->>Mise: Schema compiles
Mise->>Tools: ajv compile schema/v0.1.0.json
Schemas->>Tools: Version schema
Tools-->>Mise: Schema compiles
Mise->>Tools: ajv compile manifest.schema.json
Schemas->>Tools: Manifest schema
Tools-->>Mise: Schema compiles
Mise->>Tools: ajv validate manifest.json against manifest.schema.json
Manifest->>Tools: Manifest content
Tools-->>Mise: Validation passes
end
alt check:fixtures phase
Mise->>Tools: jq extract fixture paths from manifest.json
Manifest-->>Tools: fixture list paths
Mise->>Mise: Sort manifest paths (in memory)
Mise->>Mise: Find all .trail.jsonl files on disk
Mise->>Mise: Sort disk paths
Mise->>Mise: Check for duplicate paths in manifest
Note over Mise: If duplicate found → fail with exit 1
Mise->>Mise: diff manifest paths vs disk paths
Note over Mise: If mismatch found → fail with exit 1
end
Mise-->>CI: All checks pass / fail exit code
CI-->>Dev: Build status result
Shadow auto-approve: would auto-approve. This is a CI/tooling-only change that adds validation tools (jq, ajv-cli) and new tasks for JSON syntax, schema compilation, and fixture manifest integrity checks. No business logic, public API, schema definitions, or production code are modified.
Re-trigger cubic
c38dc0f to
5121a20
Compare
Pull Request
Summary
Related Issue
Public Impact
Impact description:
Verification
MISE_DATA_DIR=/private/tmp/mise-data-agent-trail MISE_CACHE_DIR=/private/tmp/mise-cache-agent-trail npm_config_cache=/private/tmp/npm-cache-agent-trail mise run check- passedReviewer Notes
--validate-formats=falseto avoid addingajv-formatssolely for schema meta-checking date-time annotations.