chore(ci): fix load test scenario filter and add concurrency guard

[FSM1]

Summary

• Fix scenario filtering: Call pnpm exec vitest run --no-coverage <scenario> directly instead of going through the npm script with --. pnpm's -- separator causes Vitest's CAC CLI parser to put the filter into options['--'] instead of treating it as a positional file filter — so all 5 scenarios ran instead of just the selected one.
• Add concurrency group: load-test-<environment> ensures load tests queue instead of running simultaneously. Concurrent runs against the same staging API interfere with each other's latency measurements.
• Bump timeout: 30 → 60 minutes for high client count runs.

Test plan

• Trigger mixed-workload scenario and verify only mixed-workload.test.ts runs (not all 5)
• Trigger two runs simultaneously and verify the second queues behind the first

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Enhanced internal load testing infrastructure with improved job concurrency handling and extended timeout limits for more reliable test execution.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: daab76bb-1e37-49ed-98c7-d461b21cc91b

📥 Commits

Reviewing files that changed from the base of the PR and between c13c060 and 2293cb8.

📒 Files selected for processing (1)

• .github/workflows/load-test.yml

---

Walkthrough

The load-test workflow was modified to add concurrency controls preventing in-progress runs from cancellation, increase the job timeout from 30 to 60 minutes, and change the test execution command from a filtered pnpm script to a direct vitest invocation with a different working directory.

Changes

Cohort / File(s)	Summary
CI Workflow Configuration .github/workflows/load-test.yml	Added concurrency group with cancel-in-progress: false to prevent cancellation of in-progress load tests. Increased job timeout from 30 to 60 minutes. Modified test execution to use pnpm exec vitest run --no-coverage from tests/load directory instead of the filtered pnpm script approach.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 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 accurately and specifically describes the main changes: fixing the load test scenario filter and adding a concurrency guard to prevent concurrent runs.
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.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/load-test-scenario-filter

---

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.}