Skip to content

ci: PR #32 has failing checks — already fixed in PR #33 (needs merge to resolve backlog) #115

@erudenko

Description

@erudenko

CI Failure Summary

Automated CI/CD maintenance scan — 2026-05-02

Note: gh CLI was not available in this environment; analysis used GitHub MCP API to inspect check runs on all 8 open PRs.


Currently Failing CI

PR #32fix/ci-workflow-trigger-broadening

Job Result
test (Test Stats Plugin) ❌ failure
test (20) (Test Plugins, Node 20) ❌ failure
test (22) (Test Plugins, Node 22) ⚠️ cancelled
Validate marketplace.json format ✅ success
Validate shared dependencies ✅ success
GitGuardian ✅ success

Root cause: PR #32 is stacked on fix/ci-type-errors-and-stale-test-date which still carries two unfixed problems:

  1. Three unused TypeScript imports (TS6133) that cause tsc --noEmit to fail
  2. Hardcoded test date "2026-03-26" in plugins/stats/tests/ that is now outside the 7/14-day rolling query windows

PR #32 only changed workflow YAML (removed hardcoded branch filters) — it did not carry the code-level fixes — so CI ran for the first time on that branch but immediately hit the inherited failures.


Fix Already Exists

PR #33 (fix/ci-consolidated) addresses all of these failures in one branch and has all 7 checks passing since 2026-04-12:

Job Result
test (Stats Plugin) ✅ success
test (20) ✅ success
test (22) ✅ success
marketplace-sync ✅ success
Validate marketplace.json format ✅ success
Validate shared dependencies ✅ success
GitGuardian ✅ success

PR #33 includes:

  • Removed 3 unused TS imports (TS6133)
  • Dynamic TODAY/YESTERDAY/TWO_DAYS_AGO constants replacing stale "2026-03-26" in stats tests
  • browser-use version bump 1.1.1 → 1.1.2 in marketplace.json
  • Workflow trigger broadening (removes hardcoded branches allowlist from both test-plugins.yml and test-stats.yml)

PR #34 (fix/ci-all-fixes-final) is stacked on #33 and is also all-green (13/13 checks passing), adding the same date fixes to integration.test.ts which #33 base missed.


Full PR CI Audit

PR Branch CI Notes
#22 fix/ci-typescript-and-stats-dates GitGuardian only Superseded by #24/#33; branch was invisible to test CI due to trigger bug
#23 fix/ci-typescript-and-stats-dates-resolved GitGuardian only Superseded by #24/#33; same trigger issue
#24 fix/ci-24065489776-ts-and-dates ✅ all 7 green Base fix; #33 is built on this
#28 fix/ci-24276607217-browser-use-version GitGuardian only Superseded by #24/#33; branch invisible to test CI
#31 fix/ci-main-browser-use-version GitGuardian only Targets dist-only main orphan — test workflows don't exist there
#32 fix/ci-workflow-trigger-broadening ❌ 2 failing Superseded by #33
#33 fix/ci-consolidated ✅ all 7 green Comprehensive fix; ready to merge
#34 fix/ci-all-fixes-final ✅ all 13 green Stacked on #33; adds integration.test.ts date fix

Recommended Actions

  1. Merge PR fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33 — it is all-green and consolidates all outstanding CI fixes. If integration.test.ts date fix from fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 is also desired, merge fix(ci): complete all-fixes branch — rolling stats dates in integration.test.ts + browser-use 1.1.2 #34 first (then fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33 becomes a subset) or cherry-pick its commit.

  2. Close stale/superseded 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 are all superseded by fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33. Closing them will reduce noise.

  3. Close PR fix(ci): remove hardcoded branch filters from workflow triggers #32 — it is a strict subset of fix(ci): consolidated — TS imports, stale dates, browser-use version, unrestricted triggers #33 (same workflow-trigger fix, minus the code fixes). Its CI will never be green without also applying the code fixes.

  4. Review PR fix(marketplace): bump browser-use version 1.1.1 → 1.1.2 to fix CI #31 — targets main (dist-only orphan branch) where test workflows don't exist. If the browser-use version fix is needed on main, it should either be part of the next dist publish or redirected to the source branch.


This issue was created by the automated CI/CD maintenance agent.

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