Skip to content

docs: restore one-time authorization guidance in lark-apps skill - #1794

Merged
raistlin042 merged 1 commit into
mainfrom
docs/lark-apps-restore-auth-skill
Jul 8, 2026
Merged

docs: restore one-time authorization guidance in lark-apps skill#1794
raistlin042 merged 1 commit into
mainfrom
docs/lark-apps-restore-auth-skill

Conversation

@raistlin042

@raistlin042 raistlin042 commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

PR #1175 rewrote skills/lark-apps/SKILL.md and dropped the identity/one-time-authorization guidance. Without it, agents run apps shortcuts before authorizing, hit the openapi backend unauthorized, and get server errors. This restores a concise version of that guidance so agents authorize the apps domain once up front.

Changes

  • Add ## 身份与一次性授权 section to skills/lark-apps/SKILL.md, placed after the intro paragraph and before ## 意图路由
  • Section restores the up-front one-time authorization instruction lark-cli auth login --domain apps and explains the unauthorized-to-openapi failure it prevents
  • Reactive missing_scope handling references ../lark-shared/SKILL.md instead of duplicating it, matching the current file's delegation model

Test Plan

  • make unit-test passed (via validate)
  • validate passed (build / vet / unit / integration / convention guard)
  • local-eval skipped: lite mode, doc-only change with no code path (no sandbox E2E / skillave)
  • acceptance-reviewer passed (2/2 cases)
  • manual verification: ./lark-cli auth login --help — confirmed --domain flag exists and lists apps; git diff confirms single-file +10/-0

Related Issues

N/A

Summary by CodeRabbit

  • Documentation
    • Added guidance for one-time authorization before first use of the apps domain.
    • Clarified how to avoid repeated authorization prompts and missing_scope errors.
    • Linked to shared handling guidance for related authorization issues.

@raistlin042 raistlin042 added the documentation Improvements or additions to documentation label Jul 8, 2026
@github-actions github-actions Bot added the size/M Single-domain feat or fix with limited business impact label Jul 8, 2026
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: da6cca5a-3284-4c4c-bb97-a2eb4cb5be76

📥 Commits

Reviewing files that changed from the base of the PR and between 1a9f637 and c43c7d6.

📒 Files selected for processing (1)
  • skills/lark-apps/SKILL.md

📝 Walkthrough

Walkthrough

Added a new documentation section to skills/lark-apps/SKILL.md describing a one-time authorization step for the apps domain, using the lark-cli auth login --domain apps command, with guidance on avoiding missing_scope errors and a reference to shared error handling docs.

Changes

Documentation update

Layer / File(s) Summary
Authorization guidance section
skills/lark-apps/SKILL.md
Adds "身份与一次性授权" section instructing one-time authorization via lark-cli auth login --domain apps, with missing_scope guidance and a link to shared handling docs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • larksuite/cli#1283: Adds similar one-time auth login guidance for lark-doc operations.
  • larksuite/cli#1598: Expands the shared auth guidance referenced by this PR's new section.
  • larksuite/cli#1654: Updates the shared authentication instructions linked from this PR's new section.

Suggested reviewers: liangshuo-1

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately describes the main documentation change.
Description check ✅ Passed The PR description matches the template with Summary, Changes, Test Plan, and Related Issues filled in.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/lark-apps-restore-auth-skill

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.

@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.39%. Comparing base (f495cbb) to head (c43c7d6).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1794      +/-   ##
==========================================
- Coverage   74.40%   74.39%   -0.02%     
==========================================
  Files         860      860              
  Lines       89215    89481     +266     
==========================================
+ Hits        66384    66571     +187     
- Misses      17701    17750      +49     
- Partials     5130     5160      +30     

☔ View full report in Codecov by Harness.
📢 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

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#docs/lark-apps-restore-auth-skill -y -g

@raistlin042
raistlin042 merged commit 047d729 into main Jul 8, 2026
37 checks passed
@raistlin042
raistlin042 deleted the docs/lark-apps-restore-auth-skill branch July 8, 2026 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/M Single-domain feat or fix with limited business impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants