flaker-management

flaker management skill

flaker-management is the operational companion to flaker-setup.

• flaker-setup Install, initialize, and wire the first advisory lane.
• flaker-management Run the lane over time, review health via drift, promote or demote checks, and keep flaky tests from eroding trust.

If the repository does not have flaker.toml and no CI lane yet, use flaker-setup first.

When this skill applies

• "flaker の運用方法を決めたい"
• "advisory から required にいつ上げるべき?"
• "E2E VRT を段階的に gate に入れたい"
• "nightly で flaky を triage したい"
• "quarantine をどう回す?"
• "週次レビューの playbook を作りたい"

Mental model: apply + drift

• flaker.toml is the desired state (gates, profiles, [promotion] thresholds, [quarantine].auto).
• flaker apply is the reconciler — idempotent; safe to run hourly/daily/on-demand. It auto-runs collect / calibrate / quarantine apply as needed based on current DB state.
• flaker status is the drift detector — reports which [promotion] thresholds are unmet, so promotion readiness is a boolean (ready / not ready), not a judgement call.

The canonical daily loop is:

flaker apply && flaker status

Read order

1. Read ../../docs/operations-guide.ja.md or ../../docs/operations-guide.md first, depending on the user's language.
2. Read ../../docs/flaker-management-quickstart.ja.md or ../../docs/flaker-management-quickstart.md for the first 10 minutes.
3. Read references/management-guide.ja.md for the full operating model.
4. If the user wants theory or justification, read references/theory.ja.md.
5. If the user wants copy-paste defaults, read references/presets.ja.md.
6. Reuse templates from assets/ instead of rewriting them.

What to inspect first

• flaker.toml — especially the [promotion] thresholds (defaults are documented; overriding signals intent)
• current GitHub Actions topology: pull_request, push, schedule
• latest flaker status output (drift + activity + health in one page)
• flaker status --gate merge --detail --json when you need exact promotion metrics
• flaker ops weekly for quarantine / flaky trend bundles
• whether @flaky tagging or quarantine manifest is already in use
• current PR runtime budget
• whether the focus is generic CI health, or specifically Playwright E2E / VRT

Required output shape

When applying this skill, return:

1. lane design: learning / verdict / rebalance
2. promotion criteria (align with [promotion] in flaker.toml; override only with justification)
3. demotion criteria
4. review cadence: per-PR / daily / weekly (daily is usually just flaker apply && flaker status)
5. exact flaker commands, config, and workflow snippets

Guardrails

• Do not move new E2E / VRT checks straight into required CI.
• Do not treat retries as proof of stability.
• Do not let quarantine become a graveyard; attach an owner and an exit rule.
• Keep a full scheduled lane even after PR gating starts.
• For AI-generated code, require a short per-test contract so visual checks encode intent, not just pixels.
• Do not promote --gate merge to required until flaker status drift reports ready.
• Do not use deprecated aliases in new scripts — analyze kpi, analyze eval, collect ci, debug doctor, quarantine suggest/apply, gate review/history/explain all print deprecation warnings in 0.7.0 and will be removed in 0.8.0. Use the primary commands instead.

flaker commands to prefer

# Daily
flaker apply
flaker status

# Weekly operator review
flaker status --markdown > .artifacts/status-weekly.md
flaker ops weekly --output .artifacts/flaker-weekly.md
flaker status --gate merge --detail --json > .artifacts/merge-gate.json

# Promotion snapshot (authoritative metrics)
flaker gate review merge --json > .artifacts/gate-review-merge.json  # DEPRECATED in 0.7.0; use status --gate merge --detail --json

# Incident
flaker debug retry
flaker debug confirm "<suite>:<test>" --repeat 10
flaker debug bisect --test "<name>"

Note: ops daily / weekly / incident are still first-class primary commands — apply does NOT emit the daily artifact yet. Use them directly.

Promotion / demotion decision rule

Promote --gate merge advisory → required iff flaker status drift reports ready (all 5 [promotion] thresholds met). Primary signal is flaker status — the drift section shows ready or lists unmet thresholds:

• matched_commits ≥ [promotion].matched_commits_min (default 20)
• false_negative_rate ≤ [promotion].false_negative_rate_max_percentage (default 5%)
• pass_correlation ≥ [promotion].pass_correlation_min_percentage (default 95%)
• holdout_fnr ≤ [promotion].holdout_fnr_max_percentage (default 10%)
• data_confidence ≥ [promotion].data_confidence_min (default moderate)

Demote back to advisory when ANY of the following holds for 1+ week:

• unexplained false failures continue
• flaky count trend rises and erodes trust
• owner becomes unavailable
• runtime budget is exceeded

Anti-patterns

• Using raw flaker collect ci / flaker collect calibrate (deprecated in 0.7.0) in daily cron when flaker apply already handles the ordering and idempotency.
• Using flaker analyze kpi (deprecated) instead of flaker status, or flaker analyze eval --markdown (deprecated) instead of flaker status --markdown.
• Basing promotion on flaker status numbers alone when they look close — flaker status --gate merge --detail --json is the authoritative source for exact values (the deprecated flaker gate review merge --json form also still works with a stderr warning).
• Ignoring flaker status drift holdout_fnr when holdout_ratio = 0; if holdout isn't configured, the threshold cannot be evaluated and drift treats it as unmet. Either configure [sampling].holdout_ratio or accept that holdout FNR will gate promotion.