Skip to content

ci: 65 failed workflow runs — all fixed in open PRs, needs merge decision #134

@erudenko

Description

@erudenko

CI Maintenance Audit — 2026-05-06

Automated CI maintenance scan found 65 total failed workflow runs (10 most recent analyzed). All failures have already been fixed in open PRs. No new fix branches are needed. The only remaining action is choosing which PR(s) to merge.


Root Causes (all resolved in open PRs)

1. Stale hardcoded test dates — plugins/stats/tests/

db.test.ts and integration.test.ts used the hardcoded date "2026-03-26". As time passed, this fell outside the 7-day and 14-day rolling windows used by getSessionSummary, getTopTools, and getDurationTrend, so those DB queries returned zero rows and assertions failed.

Fixed by: replacing with dynamic TODAY/YESTERDAY/TWO_DAYS_AGO constants.

2. Unused TypeScript imports (TS6133) — tools/claudeup-core/

tsc --noEmit fails under noUnusedLocals for three imports left behind after refactors:

  • conventions-integration.test.ts: removeGitignoreEntries
  • doctor.test.ts: vi
  • conventions-manager.ts: existsSync

Fixed by: removing the three unused imports.

3. browser-use version mismatch — marketplace.json

plugins/browser-use/plugin.json was bumped to v1.1.2 but .claude-plugin/marketplace.json was left at v1.1.1. The marketplace-sync integration test catches this drift:

AssertionError: Plugin browser-use: version mismatch - plugin.json has 1.1.2, marketplace.json has 1.1.1

Fixed by: bumping marketplace.json to 1.1.2.

4. Workflow triggers had hardcoded branch filters

Both test-plugins.yml and test-stats.yml had a branches allowlist ([main, fix/ci-type-errors-and-stale-test-date]), making CI invisible on all other branches. This caused fixes on fix/* branches to go untested, leading to the proliferation of fix PRs.

Fixed by: removing the branches key entirely (path filters still limit CI scope).


Status of the 10 Most Recent Failed Runs

Run ID Workflow Branch Commit Root Cause Current branch HEAD
24307096245 Test Stats Plugin fix/ci-all-fixes-final 84411b6 Stale dates in integration.test (not yet fixed in that commit) ✅ GREEN (3c64990)
24307096246 Test Plugins fix/ci-all-fixes-final 84411b6 Same ✅ GREEN (3c64990)
24307087724 Test Plugins fix/ci-all-fixes-final c12a54c marketplace-sync version mismatch (partial fix) ✅ GREEN
24306967746 Test Plugins fix/ci-all-fixes-final ea72ec0 marketplace-sync version mismatch (partial fix) ✅ GREEN
24306950584 Test Plugins fix/ci-all-fixes-final e3d5e8d marketplace-sync version mismatch (partial fix) ✅ GREEN
24306895086 Test Stats Plugin fix/ci-all-fixes-final b6de79a Stale dates not yet fixed in integration.test ✅ GREEN
24306895101 Test Plugins fix/ci-all-fixes-final b6de79a marketplace-sync version mismatch ✅ GREEN
24298790076 Test Stats Plugin fix/ci-workflow-trigger-broadening 6f38db6 Base branch still had broken TS + stale dates ⚠️ Branch itself still fails, superseded by PR #33
24298790072 Test Plugins fix/ci-workflow-trigger-broadening 6f38db6 Same ⚠️ Same
24298782547 Test Stats Plugin fix/ci-workflow-trigger-broadening 6f38db6 Same ⚠️ Same

All failures are from intermediate commits on fix branches, not from main.


Open PRs — Current CI State

PR Branch Base All checks green? Notes
#24 fix/ci-24065489776-ts-and-dates fix/ci-type-errors-and-stale-test-date ✅ Yes Comprehensive fix; all others stack on it
#28 fix/ci-24276607217-browser-use-version fix/ci-24065489776-ts-and-dates ✅ Yes Subset of #24; superseded
#31 fix/ci-main-browser-use-version main ⚠️ No CI (main has no workflows) browser-use version fix only
#32 fix/ci-workflow-trigger-broadening fix/ci-type-errors-and-stale-test-date ❌ Fails Trigger fix only; base still has broken TS+dates; superseded by #33
#33 fix/ci-consolidated fix/ci-type-errors-and-stale-test-date All 7 checks green Consolidates everything: TS imports + stale dates + browser-use version + trigger broadening
#34 fix/ci-all-fixes-final fix/ci-type-errors-and-stale-test-date All 13 checks green Same fixes as #33 via different commit history

Recommended Action

PR #33 (fix/ci-consolidated) is the cleanest consolidation of all fixes with 7/7 checks green. Merging it (and then #24 if needed for the stack) will resolve all CI failures.

PRs that can be closed as superseded once #33 or #24 merges:

PR #31 targets main (the dist-only orphan branch) where test workflows don't exist — CI can never run there. That PR should be closed or redirected to the correct source branch.


Filed by automated CI maintenance agent — no new fix branches created since all fixes are already in open PRs.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions