Skip to content

[Bug]: doc-coverage fails on latest main because binance docs are missing #970

@Astro-Han

Description

@Astro-Han

Description

doc-coverage is failing on rebased PRs even when the PR does not touch adapter docs.

On the latest main, the coverage script treats clis/binance as a documented adapter requirement, but there is no matching adapter doc page for binance. This causes unrelated PRs to fail the Doc Check workflow.

Steps to Reproduce

  1. Rebase any unrelated PR onto the latest main.
  2. Let GitHub Actions run the Doc Check workflow.
  3. See doc-coverage fail.

Local reproduction:

  1. Check out the latest main.
  2. Run bash scripts/check-doc-coverage.sh --strict.
  3. See the missing-doc error for binance.

Expected Behavior

doc-coverage should pass for unrelated PRs on a healthy main branch.

Either:

  • binance should have a matching adapter doc page, or
  • the coverage rule should skip adapter directories that are not meant to be documented yet.

OpenCLI Version

1.7.1

Node.js Version

22.x

Operating System

Linux

Logs / Screenshots

📊 Doc Coverage: 84/85 adapters documented

⚠️  Missing docs for 1 adapter(s):
   - binance  →  create docs/adapters/browser/binance.md or docs/adapters/desktop/binance.md

❌ Doc check failed (--strict mode).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions