ci: recent failures resolved in PR #34 — action needed to merge and close stale PRs

[erudenko]

CI Maintenance Agent Report — 2026-04-13

Recently Failed Runs

Run ID	Workflow	Branch	Failure
24298790072	Test Plugins (test (20))	fix/ci-workflow-trigger-broadening	tsc --noEmit — 3 unused TS imports (TS6133)
24298790076	Test Stats Plugin	fix/ci-workflow-trigger-broadening	Stale hardcoded date 2026-03-26 outside 7-day query window
24298782547	Test Stats Plugin	fix/ci-workflow-trigger-broadening	Same stale date failure (earlier push on same branch)

All three failures are on the PR #32 (fix/ci-workflow-trigger-broadening) branch. That PR only modified the workflow YAML trigger rules and was stacked directly on fix/ci-type-errors-and-stale-test-date — a base that still carried the broken TS imports and stale test dates. Once CI started running (thanks to the trigger fix), the pre-existing code failures surfaced.

---

Root Causes

1. Unused TypeScript imports (TS6133) — tsc --noEmit fails under noUnusedLocals: true:

   • tools/claudeup-core/src/__tests__/integration/conventions-integration.test.ts — removeGitignoreEntries
   • tools/claudeup-core/src/__tests__/unit/doctor.test.ts — vi
   • tools/claudeup-core/src/services/conventions-manager.ts — existsSync

2. Stale hardcoded test dates — plugins/stats/tests/db.test.ts and integration.test.ts used "2026-03-26", which is now 18 days in the past and outside the 7-day/14-day rolling windows used by getSessionSummary, getTopTools, and getDurationTrend.

3. browser-use version mismatch — plugins/browser-use/plugin.json is at 1.1.2 but .claude-plugin/marketplace.json was left at 1.1.1, causing marketplace-sync integration test failures.

4. Hardcoded branch filters in workflow triggers — Both test-plugins.yml and test-stats.yml had branches: [main, fix/ci-type-errors-and-stale-test-date], making every other branch invisible to CI (PRs fix(ci): remove unused TS imports + fix stale stats test dates #22, fix(ci): resolve merge conflict — unused TS imports + stale stats test dates #23, fix(marketplace): bump browser-use version 1.1.1 → 1.1.2 in marketplace.json #28, fix(marketplace): bump browser-use version 1.1.1 → 1.1.2 to fix CI #31, fix(ci): remove hardcoded branch filters from workflow triggers #32 all had no test runs).

---

Current Fix PRs

PR	Branch	CI	Mergeable	Notes
#34	fix/ci-all-fixes-final	✅ 13/13 green	clean	Most recent; recommended merge target
#33	fix/ci-consolidated	✅ 7/7 green	❌ dirty (conflict)	Superseded by #34
#24	fix/ci-24065489776-ts-and-dates	✅ 7/7 green	—	Comprehensive code fix; #34 is based on it
#32	fix/ci-workflow-trigger-broadening	❌ 2 failing	—	Superseded by #33/#34
#31	fix/ci-main-browser-use-version	⚪ no test CI	—	Targets dist-only main; no workflow files
#23, #22, #28	various	⚪ no test CI	—	Superseded; branches invisible to old CI trigger

---

Recommended Actions

1. Merge PR fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 (fix/ci-all-fixes-final) into fix/ci-type-errors-and-stale-test-date — it is the most recent, fully green, and conflict-free fix covering all four root causes.

2. Close PRs fix(ci): remove hardcoded branch filters from workflow triggers #32, fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33, fix(marketplace): bump browser-use version 1.1.1 → 1.1.2 to fix CI #31, fix(marketplace): bump browser-use version 1.1.1 → 1.1.2 in marketplace.json #28, fix(ci): resolve merge conflict — unused TS imports + stale stats test dates #23, fix(ci): remove unused TS imports + fix stale stats test dates #22 as superseded by fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34. Each has either merge conflicts, no test CI coverage, or is a strict subset of what fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 provides.

3. Keep PR fix(ci): remove 3 unused TS imports + fix stale stats dates + re-enable CI triggers #24 open or merge it first — fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 is stacked on fix(ci): remove 3 unused TS imports + fix stale stats dates + re-enable CI triggers #24's head, so either merge fix(ci): remove 3 unused TS imports + fix stale stats dates + re-enable CI triggers #24 first and then fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34, or merge fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 directly (since fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 carries all of fix(ci): remove 3 unused TS imports + fix stale stats dates + re-enable CI triggers #24's changes).

---

No new fix branch was created by this agent — all necessary code changes already exist in the open PRs above.