Skip to content

docs(skills): fix agent routing for doubao.com URLs#1082

Merged
fangshuyu-768 merged 1 commit into
mainfrom
feat/skills-add-doubao-domain
May 26, 2026
Merged

docs(skills): fix agent routing for doubao.com URLs#1082
fangshuyu-768 merged 1 commit into
mainfrom
feat/skills-add-doubao-domain

Conversation

@fangshuyu-768
Copy link
Copy Markdown
Collaborator

@fangshuyu-768 fangshuyu-768 commented May 25, 2026

Problem

When users provide doubao.com URLs such as https://www.doubao.com/docx/TOKEN, AI agents often skip lark-cli and fall back to WebFetch.

This is not a product capability gap. lark-cli already supports these resources because Doubao uses the same path patterns and token model as Feishu. The real issue is routing: the affected skill descriptions read like Feishu-only capabilities, so agents do not get a strong enough signal to choose the correct lark-cli skill.

Changes

Update the frontmatter descriptions of the affected skills so they act as explicit routing instructions for agents, not just compatibility notes.

  • tell the agent to directly use the matching skill when it sees a doubao.com URL or token for that resource type
  • tell the agent not to fall back just because the host is not a Feishu domain
  • make the routing rule explicit: route by URL path pattern and token, not by domain name

Affected skills:

Skill Doubao URL signal
lark-doc /docx/, /wiki/
lark-drive Doubao cloud-space resource URL/token
lark-sheets /sheets/
lark-slides /slides/
lark-wiki /wiki/

Why this helps

Before this change, the descriptions mostly said that these skills work for Feishu resources, with Doubao support implied or missing. After this change, the descriptions give agents a concrete decision rule: if the URL/token matches the resource pattern, use the corresponding lark-cli skill instead of WebFetch.

This makes the PR specifically about improving agent routing behavior, not about adding new end-user functionality.

Test Plan

  • Documentation-only change to skill frontmatter descriptions
  • Run node scripts/skill-format-check/index.js
  • Manual routing check: https://www.doubao.com/docx/... prefers lark-doc over WebFetch
  • Manual routing check: https://www.doubao.com/sheets/... prefers lark-sheets over WebFetch
  • Manual routing check: https://www.doubao.com/slides/... prefers lark-slides over WebFetch
  • Manual routing check: https://www.doubao.com/wiki/... prefers lark-wiki over WebFetch
  • Regression check: equivalent feishu.cn URLs continue routing to the same lark-cli skills

Summary by CodeRabbit

  • Documentation
    • Updated skill descriptions across multiple modules to clarify supported URL patterns and routing behavior based on path patterns and tokens rather than domain names.

Review Change Stack

@github-actions github-actions Bot added domain/ccm PR touches the ccm domain size/L Large or sensitive change across domains or core paths labels May 25, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b19cf6d6-19fe-4b82-ac2d-6ba418b58527

📥 Commits

Reviewing files that changed from the base of the PR and between 5bc44a3 and 2a1ccd1.

📒 Files selected for processing (5)
  • skills/lark-doc/SKILL.md
  • skills/lark-drive/SKILL.md
  • skills/lark-sheets/SKILL.md
  • skills/lark-slides/SKILL.md
  • skills/lark-wiki/SKILL.md
✅ Files skipped from review due to trivial changes (3)
  • skills/lark-sheets/SKILL.md
  • skills/lark-slides/SKILL.md
  • skills/lark-drive/SKILL.md

📝 Walkthrough

Walkthrough

Five skill metadata files update each description to state Doubao (doubao.com) URLs are supported and that routing uses URL path patterns (e.g., /docx/, /wiki/, /sheets/, /slides/) rather than the domain name.

Changes

Doubao domain support

Layer / File(s) Summary
Skill metadata descriptions for Doubao domain support
skills/lark-doc/SKILL.md, skills/lark-drive/SKILL.md, skills/lark-sheets/SKILL.md, skills/lark-slides/SKILL.md, skills/lark-wiki/SKILL.md
Five skill metadata descriptions are updated to add Doubao domain support, each clarifying that the skill applies to doubao.com URLs with routing based on path patterns (/docx/, /wiki/, /sheets/, /slides/) rather than domain name.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • SunPeiYang996
  • wittam-01

Poem

🐰 I hopped through docs with nimble feet,
Doubao paths now make skills meet,
/sheets/, /slides/, /wiki/, /docx/ align,
Tokens guide routes — no domain sign,
A rabbit cheers for routes so neat!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: fixing agent routing for doubao.com URLs by updating skill descriptions.
Description check ✅ Passed The PR description comprehensively covers all required sections: Problem statement, Changes with a detailed table, Why this helps, and a thorough Test Plan with specific manual verification steps.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/skills-add-doubao-domain

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.92%. Comparing base (049ddf7) to head (2a1ccd1).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1082   +/-   ##
=======================================
  Coverage   67.92%   67.92%           
=======================================
  Files         603      603           
  Lines       55779    55779           
=======================================
  Hits        37888    37888           
  Misses      14755    14755           
  Partials     3136     3136           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 25, 2026

🚀 PR Preview Install Guide

🧰 CLI update

npm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@2a1ccd1728f9b4f8cc045e1a54763fd4d644b6be

🧩 Skill update

npx skills add larksuite/cli#feat/skills-add-doubao-domain -y -g

@fangshuyu-768 fangshuyu-768 changed the title docs(skills): add doubao.com domain support to lark-doc/drive/sheets/slides docs(skills): fix agent routing for doubao.com URLs May 26, 2026
@fangshuyu-768 fangshuyu-768 force-pushed the feat/skills-add-doubao-domain branch from 386bdc5 to 5bc44a3 Compare May 26, 2026 02:38
wittam-01
wittam-01 previously approved these changes May 26, 2026
@fangshuyu-768 fangshuyu-768 merged commit b9e5b50 into main May 26, 2026
22 checks passed
@fangshuyu-768 fangshuyu-768 deleted the feat/skills-add-doubao-domain branch May 26, 2026 09:41
@liangshuo-1 liangshuo-1 mentioned this pull request May 26, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain/ccm PR touches the ccm domain size/L Large or sensitive change across domains or core paths

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants