feat: support sub-page-list block for docx document#1168
Conversation
- lark-doc-xml.md §三「资源块」: add <folder-manager wiki-token="..."> entry with full sub-page schema (title / url / file-type+doc-id fallback / space-id / owner / owner-id / create-time / edit-time, ms timestamps, has-more="true" beyond 100 children) - lark-doc-xml.md §四「复制」: append folder-manager to copy support list (per spec FE-1 TC-D acceptance) - lark-doc-xml.md §八 完整示例: add folder-manager example - lark-doc-fetch.md: add 子页面列表 section explaining fetch behavior, url-first / file-type+doc-id fallback, container-only on wiki.core failure or no permission Spec ref: cli-docx-folder-manager FE-1 Change-Id: I746fbebcc3398c5ec0b144f2eb2a306e6d96fb74
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe PR updates Lark Doc XML documentation to introduce ChangesSub-page-list Element Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/lark-doc/references/lark-doc-fetch.md`:
- Around line 138-142: The <sub-page> fetch output is underspecified — update
lark-doc-fetch.md to document the complete <sub-page> element schema and
fallback rules: enumerate every attribute/key the CLI produces (e.g., title,
doc-id, file-type, updated-at/modified-by/owner, excerpt/summary, url/permalink,
has-children/child-count, mime-type, thumbnails/preview, permissions/visibility,
and any source/provider id), state which attributes are optional vs required,
and specify fallback/derivation precedence (e.g., use metadata.title then
fallback to first heading then filename; use doc-id if present else construct
from space-id+path; prefer explicit file-type then infer from mime-type),
reference the <sub-page> tag and the surrounding <sub-page-list> example, and
replace the minimal example XML with a full annotated example showing all fields
and an explicit note about parser behavior when fields are missing.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 32b23e98-b0fa-43fc-b0cd-03248c056998
📒 Files selected for processing (2)
skills/lark-doc/references/lark-doc-fetch.mdskills/lark-doc/references/lark-doc-xml.md
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@d2869fc69736ba747bcd6ca5c0aaa26982fb84e0🧩 Skill updatenpx skills add ZEden0/cli#feat-cli-docx-folder-manager -y -g |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1168 +/- ##
==========================================
+ Coverage 68.77% 68.90% +0.12%
==========================================
Files 628 628
Lines 58670 58735 +65
==========================================
+ Hits 40353 40473 +120
+ Misses 15021 14958 -63
- Partials 3296 3304 +8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Change-Id: I6a19d223a45b3d01f22ed194b98a4a18efa55424
8ec3f8a to
d2869fc
Compare
Summary
Document the CLI-facing Docx child page list resource block for the FolderManager rollout. This PR updates the
lark-docskill references so agents can read<sub-page-list>fetch output and produce<sub-page-list></sub-page-list>insert/edit input for Wiki documents.Changes
<sub-page-list>output, includingwiki-token,space-id,has-more, and nested<sub-page>metadata.<sub-page-list>as a resource block that can be inserted only in Wiki documents.sub-page-listto the documented resource block copy support list and complete XML example set.Test Plan
mainis limited toskills/lark-doc/references/lark-doc-fetch.mdandskills/lark-doc/references/lark-doc-xml.md.make unit-testSummary by CodeRabbit
<sub-page-list>resource block feature available in wiki documents.