Skip to content

chore: map wiki paths in pr labels#249

Open
liujinkun2025 wants to merge 1 commit intomainfrom
chore/pr-labels-wiki
Open

chore: map wiki paths in pr labels#249
liujinkun2025 wants to merge 1 commit intomainfrom
chore/pr-labels-wiki

Conversation

@liujinkun2025
Copy link
Copy Markdown
Collaborator

@liujinkun2025 liujinkun2025 commented Apr 3, 2026

Summary

Add PR label routing for wiki-related changes so updates under shortcuts/wiki/ and skills/lark-wiki/ are classified into the correct business domain automatically.

Changes

  • Add shortcuts/wiki/ -> ccm path mapping in scripts/pr-labels/index.js
  • Add skills/lark-wiki/ -> ccm path mapping in scripts/pr-labels/index.js

Test Plan

  • Unit tests pass
  • Manual local verification confirms the lark xxx command works as expected
  • Not applicable: this PR only updates PR label path mapping and does not change CLI runtime behavior

Related Issues

  • None

Summary by CodeRabbit

  • Chores
    • Updated internal code ownership mappings for wiki-related directories.

Change-Id: I6d3bc320255958f280922e595dc67f61a11f4b0b
@github-actions github-actions bot added the size/M Single-domain feat or fix with limited business impact label Apr 3, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 3, 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: f91e0e16-1958-4938-be68-92d04370c8a1

📥 Commits

Reviewing files that changed from the base of the PR and between 5cf8667 and a89b441.

📒 Files selected for processing (1)
  • scripts/pr-labels/index.js

📝 Walkthrough

Walkthrough

Updated the path-to-domain CODEOWNERS mapping in scripts/pr-labels/index.js to associate two additional wiki-related directories with the ccm business domain: shortcuts/wiki/ and skills/lark-wiki/.

Changes

Cohort / File(s) Summary
CODEOWNERS Path Mappings
scripts/pr-labels/index.js
Added two new path-to-domain associations: shortcuts/wiki/ and skills/lark-wiki/ both mapped to ccm domain in the internal PATH_TO_DOMAIN_MAP constant.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Two paths through the wiki's hall,
Now mapped to ccm domain's call,
With whiskers twitching, we celebrate,
These shortcuts and skills taste great!
A simple hop, a config's fate! 📋

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: map wiki paths in pr labels' accurately describes the main change—adding path-to-domain mappings for wiki-related directories in the PR labels script.
Description check ✅ Passed The PR description includes all required template sections: Summary, Changes, Test Plan, and Related Issues. All sections are appropriately filled with relevant details for this configuration change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/pr-labels-wiki

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

@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Apr 3, 2026

Greptile Summary

This PR adds two path-to-domain mappings for wiki-related directories (shortcuts/wiki/ and skills/lark-wiki/) in the PR label classification script, routing them to the ccm (Cloud Content Management) business domain. This is consistent with how the existing doc, sheets, and drive shortcuts are classified.

Key changes:

  • shortcuts/wiki/ccm added after shortcuts/drive/ in PATH_TO_DOMAIN_MAP
  • skills/lark-wiki/ccm added after skills/lark-doc/ in PATH_TO_DOMAIN_MAP
  • Both entries follow established patterns and groupings in the map
  • The ccm domain is already in HEAD_BUSINESS_DOMAINS, so wiki-related PRs will be treated as touching a head business domain correctly

Confidence Score: 5/5

This 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 ccm domain, and are placed in the correct grouping position. There are no logic changes, no new code paths, and no risk of prefix collision with any existing entry.

No files require special attention.

Important Files Changed

Filename Overview
scripts/pr-labels/index.js Adds two new entries to PATH_TO_DOMAIN_MAP mapping shortcuts/wiki/ and skills/lark-wiki/ to the ccm domain, consistent with existing CCM path mappings.

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]
Loading

Reviews (1): Last reviewed commit: "chore: map wiki paths in pr labels" | Re-trigger Greptile

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 3, 2026

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#chore/pr-labels-wiki -y -g

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant