Stabilize plugin-manager install-bundle tests under CI load #279

New issue

Closed

opened 2026-06-04 21:47:01 +00:00 by steve · 0 comments

steve commented

2026-06-04 21:47:01 +00:00

Owner

Problem

Forgejo checks fail on pnpm test after #278 added unit tests to CI.
plugin-manager.test.ts install-bundle cases (especially lapis-notebook) can
exceed Vitest's default 5s timeout under CI CPU load and default Turbo
parallelism.

Goal

Keep pnpm test green in Forgejo checks without weakening coverage.

Acceptance Criteria

External install-bundle cases in plugin-manager.test.ts have explicit timeouts
checks.yml sets TURBO_CONCURRENCY=1 and NODE_OPTIONS like docker:ci-check
pnpm test and pnpm check:all pass locally

Validation Commands

pnpm exec turbo run test --filter=@lapis-notes/api --force
pnpm test
pnpm check:all

Implementation Summary

Raised plugin-manager external install-bundle Vitest timeout to 30s (lapis-notebook exceeds 5s under load). Set TURBO_CONCURRENCY=1 and NODE_OPTIONS in checks.yml to match docker:ci-check. Validated: pnpm test, api test --force, docker:ci-check --skip-smoke --pull.

## Problem Forgejo checks fail on `pnpm test` after #278 added unit tests to CI. `plugin-manager.test.ts` install-bundle cases (especially `lapis-notebook`) can exceed Vitest's default 5s timeout under CI CPU load and default Turbo parallelism. ## Goal Keep `pnpm test` green in Forgejo checks without weakening coverage. ## Acceptance Criteria - [ ] External install-bundle cases in `plugin-manager.test.ts` have explicit timeouts - [ ] `checks.yml` sets `TURBO_CONCURRENCY=1` and `NODE_OPTIONS` like `docker:ci-check` - [ ] `pnpm test` and `pnpm check:all` pass locally ## Validation Commands - `pnpm exec turbo run test --filter=@lapis-notes/api --force` - `pnpm test` - `pnpm check:all` ## Implementation Summary Raised plugin-manager external install-bundle Vitest timeout to 30s (lapis-notebook exceeds 5s under load). Set TURBO_CONCURRENCY=1 and NODE_OPTIONS in checks.yml to match docker:ci-check. Validated: pnpm test, api test --force, docker:ci-check --skip-smoke --pull.

steve

2026-06-04 21:47:01 +00:00