Uh oh!
There was an error while loading. Please reload this page.
ci: add mock test workflow for PRs - #977
Merged
Merged
Conversation
AhmedTMMforce-pushed
the
pr/add-test-ci-workflow
branch
from
February 13, 2026 19:07
4abc870 to
cf2f996CompareRuns `bash test/mock.sh` on every pull request targeting main. Includes concurrency grouping to cancel stale runs and a 10-minute timeout. Results are posted to the GitHub Actions step summary. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
AhmedTMMforce-pushed
the
pr/add-test-ci-workflow
branch
from
February 13, 2026 19:08
cf2f996 to
bde2266Comparelouisgv
approved these changes
Feb 13, 2026
louisgv
left a comment
Collaborator
There was a problem hiding this comment.
Security Review
Verdict: APPROVED
Findings
- None
Analysis
- Workflow uses
pull_requesttrigger (read-only context, no secrets exposed) — safe actions/checkout@v4— standard, acceptable version pinning- No
${{ }}expression injection inrun:blocks — all expressions only inconcurrency.group(safe context) - No secrets referenced — appropriate for test-only workflow
- Test output piped to
GITHUB_STEP_SUMMARYis filtered via grep + head -100 — low injection risk - 10-minute timeout prevents hung jobs
- Concurrency grouping with cancel-in-progress prevents stale CI runs
Tests
- YAML validation: PASS
- bash -n: N/A (no .sh files changed)
- bun test: N/A (no .ts files changed)
- curl|bash pattern: N/A
- macOS compat: N/A
-- security/pr-reviewer
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 freeto 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.
Summary
bash test/mock.shon every PR targetingmainContext
Extracted from #833 as a smaller, focused PR per reviewer feedback.
Test plan
bash test/mock.shruns successfully in CI🤖 Generated with Claude Code