Skip to content

fix: repair baseline main ci checks#901

Merged
jackwener merged 1 commit intojackwener:mainfrom
Astro-Han:fix/main-ci-baseline
Apr 9, 2026
Merged

fix: repair baseline main ci checks#901
jackwener merged 1 commit intojackwener:mainfrom
Astro-Han:fix/main-ci-baseline

Conversation

@Astro-Han
Copy link
Copy Markdown
Contributor

Description

Fix baseline CI failures that reproduce on current main:

  • replace forbidden relative ../../src/types.js imports in jd and taobao adapter tests with the public @jackwener/opencli/types export
  • add the missing browser adapter docs for cnki and taobao so doc coverage passes
  • keep the change scoped to the baseline CI fix, with no unified-capture files touched

Related issue:
N/A

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🌐 New site adapter
  • 📝 Documentation
  • ♻️ Refactor
  • 🔧 CI / build / tooling

Checklist

  • I ran the checks relevant to this PR
  • I updated tests or docs if needed
  • I included output or screenshots when useful

Documentation (if adding/modifying an adapter)

  • Added doc page under docs/adapters/ (if new adapter)
  • Updated docs/adapters/index.md table (if new adapter)
  • Updated sidebar in docs/.vitepress/config.mts (if new adapter)
  • Updated README.md / README.zh-CN.md when command discoverability changed
  • Used positional args for the command's primary subject unless a named flag is clearly better
  • Normalized expected adapter failures to CliError subclasses instead of raw Error

Screenshots / Output

  • npm test
  • npm test -- --reporter=verbose src/package-exports.test.ts
  • bash scripts/check-doc-coverage.sh --strict
  • npm test -- --reporter=verbose --shard=2/2
  • bun vitest run --project unit --reporter=verbose

@jackwener jackwener merged commit caebede into jackwener:main Apr 9, 2026
11 checks passed
@Astro-Han Astro-Han deleted the fix/main-ci-baseline branch April 9, 2026 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants