Conversation
Change-Id: I6d3bc320255958f280922e595dc67f61a11f4b0b
|
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)
📝 WalkthroughWalkthroughUpdated the path-to-domain CODEOWNERS mapping in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Greptile SummaryThis PR adds two path-to-domain mappings for wiki-related directories ( Key changes:
Confidence Score: 5/5This PR is safe to merge — it only adds two configuration entries that follow existing patterns exactly. The change is a minimal, mechanical addition of two path→domain entries to a static lookup map. Both new entries follow the exact same pattern as their sibling entries, map to the already-used No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[PR file paths] --> B{getBusinessDomain}
B -->|shortcuts/wiki/*| C["ccm NEW"]
B -->|skills/lark-wiki/*| C
B -->|shortcuts/doc/*| D[ccm]
B -->|shortcuts/sheets/*| D
B -->|shortcuts/drive/*| D
B -->|skills/lark-doc/*| D
B -->|shortcuts/im/*| E[im]
B -->|shortcuts/base/*| F[base]
B -->|no match| G[shortcutDomainForPath / skillDomainForPath fallback]
C --> H{HEAD_BUSINESS_DOMAINS?}
D --> H
H -->|yes - ccm, im, contact, base, docx| I[headDomains count up]
I --> J[size/XL if 2+ headDomains + sensitive]
H -->|no| K[regular domain tracking]
Reviews (1): Last reviewed commit: "chore: map wiki paths in pr labels" | Re-trigger Greptile |
🚀 PR Preview Install Guide🧰 CLI updatenpm i -g https://pkg.pr.new/larksuite/cli/@larksuite/cli@a89b4414c5fe5f0b2505860b7613b2a9ae6e5f07🧩 Skill updatenpx skills add larksuite/cli#chore/pr-labels-wiki -y -g |
Summary
Add PR label routing for wiki-related changes so updates under
shortcuts/wiki/andskills/lark-wiki/are classified into the correct business domain automatically.Changes
shortcuts/wiki/->ccmpath mapping inscripts/pr-labels/index.jsskills/lark-wiki/->ccmpath mapping inscripts/pr-labels/index.jsTest Plan
lark xxxcommand works as expectedRelated Issues
Summary by CodeRabbit