Skip to content

feat: add lark-openclaw integration skill#36

Open
Horacehxw wants to merge 3 commits intolarksuite:mainfrom
Horacehxw:feat/lark-openclaw-skill
Open

feat: add lark-openclaw integration skill#36
Horacehxw wants to merge 3 commits intolarksuite:mainfrom
Horacehxw:feat/lark-openclaw-skill

Conversation

@Horacehxw
Copy link
Copy Markdown

Summary

  • Add skills/lark-openclaw/SKILL.md — integration guide for using lark-cli inside OpenClaw agents
  • Add lark-openclaw entry to Agent Skills table in README.md

What This Skill Covers

  • Decision tree: when to use lark-cli vs @larksuite/openclaw-lark native tools
  • Docker sandbox installation: Dockerfile additions for OpenClaw agent sandboxes
  • Per-agent credential isolation: different Feishu apps per agent sandbox (multi-bot architectures)
  • Shortcuts quick reference: Mail, VC, Contact, and generic API commands for domains without native tools
  • Cross-references: links to peer skills (lark-mail, lark-vc, lark-contact, lark-openapi-explorer)
  • Practical workflows: native-first → lark-cli fallback patterns

Why

OpenClaw is listed as a supported agent in lark-cli's npx skills installer, but there's no integration guide. This skill teaches OpenClaw agents best practices for using lark-cli alongside the @larksuite/openclaw-lark plugin — specifically which tool to pick for each Feishu API domain.

Conventions Followed

  • Frontmatter: name, version, description (Chinese with trigger scenarios), metadata.requires.bins, metadata.cliHelp
  • CRITICAL reference to ../lark-shared/SKILL.md
  • Shortcuts table format with 身份 column
  • Cross-references to peer skill SKILL.md files
  • 权限表 with scope reference

Test Plan

  • Skill installs correctly: npx skills add larksuite/cli -s lark-openclaw --agent openclaw
  • SKILL.md follows conventions (frontmatter, CRITICAL reference to lark-shared)
  • README table renders correctly with the new row

Signed-off-by: Horacehxw <horacehxw@gmail.com>
Signed-off-by: Horacehxw <horacehxw@gmail.com>
Copilot AI review requested due to automatic review settings March 28, 2026 14:37
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new Agent Skill documenting how to integrate lark-cli into OpenClaw agents, and exposes it in the project’s Agent Skills index so users can discover and install it.

Changes:

  • Add skills/lark-openclaw/SKILL.md integration guide (tool choice decision tree, sandbox install, per-agent credential isolation, shortcuts/API workflows).
  • Add lark-openclaw entry to the Agent Skills table in README.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
skills/lark-openclaw/SKILL.md New OpenClaw integration skill guide (install, credentials isolation, command/workflow references).
README.md Adds lark-openclaw to the Agent Skills table for discoverability.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +45 to +51
在 Agent 的 Dockerfile 中添加:

```dockerfile
RUN apt-get update -qq && \
apt-get install -y -qq --no-install-recommends nodejs npm && \
npm install -g @larksuite/cli && \
apt-get clean && rm -rf /var/lib/apt/lists/*
Copy link

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dockerfile 安装段直接 apt-get install nodejs npm 可能在部分 Debian/Ubuntu 基础镜像上拿到 <16 的 Node,和本项目 @larksuite/cli 的 engines (node>=16) 不兼容,导致安装或运行失败。建议在文档里明确要求 Node.js>=16,或改用带版本的 Node 官方镜像/安装脚本来保证版本。

Suggested change
在 Agent 的 Dockerfile 中添加:
```dockerfile
RUN apt-get update -qq && \
apt-get install -y -qq --no-install-recommends nodejs npm && \
npm install -g @larksuite/cli && \
apt-get clean && rm -rf /var/lib/apt/lists/*
Node.js **必须为 16 或更高版本**。推荐直接使用官方 Node 镜像,避免从发行版仓库安装过旧的 `nodejs`
```dockerfile
FROM node:18-bullseye-slim
# 安装 lark-cli(全局)
RUN npm install -g @larksuite/cli && \
npm cache clean --force

Copilot uses AI. Check for mistakes.
Comment on lines +86 to +88
| `mail +search` | 搜索邮件 | user |
| `vc +meetings-list` | 列出已结束的会议记录 | user |
| `vc +note` | 获取会议纪要(摘要、待办、章节) | user |
Copy link

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shortcuts 表里列出的 mail +searchvc +meetings-listvc +note 在当前仓库的快捷命令中不存在/命名不一致(mail 搜索通常用 mail +triage --query ...,VC 为 vc +searchvc +notes)。建议改成仓库里实际提供的快捷命令名称,避免读者照抄后直接报错。

Suggested change
| `mail +search` | 搜索邮件 | user |
| `vc +meetings-list` | 列出已结束的会议记录 | user |
| `vc +note` | 获取会议纪要(摘要、待办、章节) | user |
| `mail +triage --query ...` | 搜索邮件 | user |
| `vc +search` | 列出已结束的会议记录 | user |
| `vc +notes` | 获取会议纪要(摘要、待办、章节) | user |

Copilot uses AI. Check for mistakes.
Signed-off-by: Horacehxw <horacehxw@gmail.com>
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 30, 2026

CLA assistant check
All committers have signed the CLA.

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.

3 participants