Skip to content

ci: 7 open fix-PRs stalled with none merged — action needed to unblock CI on source branch #44

@erudenko

Description

@erudenko

Summary

Automated CI/CD maintenance scan (2026-04-14) found 3 actively failing workflow runs on PR #32 and 6 additional open fix-PRs that address the same failures but have not yet been merged. All of the underlying failures are already fixed in PRs that show fully-green CI — but the branch they target (fix/ci-type-errors-and-stale-test-date) remains broken because none have been merged.


Failing Runs

Run ID Workflow Branch Failing Job Error
24298790072 Test Plugins fix/ci-workflow-trigger-broadening (PR #32) test (20) tsc --noEmit — unused TS imports
24298790076 Test Stats Plugin fix/ci-workflow-trigger-broadening (PR #32) test Stale hardcoded date in db.test.ts
24298782547 Test Stats Plugin fix/ci-workflow-trigger-broadening (PR #32) test Same stale date

These runs are NOT transient — they are repeatable code failures.


Root Causes

1. Unused TypeScript imports (TS6133 — noUnusedLocals)

tsc --noEmit fails on three imports left behind after refactors:

File Unused import
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 outside the 7/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 still has 1.1.1. The marketplace-sync integration test catches this drift.

4. CI workflow triggers were too narrow (now fixed on source branch)

test-plugins.yml and test-stats.yml had hardcoded branches allowlists, making PRs on most branches invisible to CI. This was partially addressed on the source branch already.


Status of All Open Fix-PRs

PR Branch CI Scope Recommended action
#24 fix/ci-24065489776-ts-and-dates ✅ All green TS imports + stale dates + browser-use Candidate to merge
#33 fix/ci-consolidated ✅ All green All of #24 + trigger broadening Best candidate to merge (supersedes #32)
#34 fix/ci-all-fixes-final ✅ All green integration.test.ts dates + marketplace.json Candidate to merge
#32 fix/ci-workflow-trigger-broadening ❌ 3 failures Trigger only (no code fixes) Close as superseded by #33
#31 fix/ci-main-browser-use-version ⚠️ No test CI marketplace.json only, targets main (dist orphan) Close — CI can't run on main
#28 fix/ci-24276607217-browser-use-version ⚠️ No test CI marketplace.json only Close as superseded by #24/#33
#23 fix/ci-typescript-and-stats-dates-resolved ⚠️ No test CI TS imports + stale dates Close as superseded by #24/#33
#22 fix/ci-typescript-and-stats-dates ⚠️ No test CI TS imports + stale dates Close as superseded by #24/#33

PRs #22, #23, #28 showed only GitGuardian checks because the branch-filter issue prevented CI from running on them at the time.


Recommended Path Forward

  1. Merge PR fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33 (fix/ci-consolidated) — it is the most complete fix: TS imports, stale dates, browser-use version, and trigger broadening. All 7 CI checks are green. This resolves the underlying failures and supersedes all other fix-PRs.
  2. Close 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 as superseded with reference to fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33.
  3. Review PR fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 — if integration.test.ts date fixes aren't already in fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33, merge fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 as a follow-up; otherwise close it.
  4. Verify that after merging fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33, the base branch fix/ci-type-errors-and-stale-test-date goes fully green.

Filed by automated CI maintenance agent. No new fix branch was created because comprehensive fixes already exist in open PRs with green CI.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingci

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions