Skip to content

docs: clarify doc block id lifecycle#1501

Closed
fangshuyu-768 wants to merge 1 commit into
mainfrom
docs/lark-doc-block-id-lifecycle
Closed

docs: clarify doc block id lifecycle#1501
fangshuyu-768 wants to merge 1 commit into
mainfrom
docs/lark-doc-block-id-lifecycle

Conversation

@fangshuyu-768

Copy link
Copy Markdown
Collaborator

Summary

  • add a Block ID lifecycle table for docs +update commands
  • make lark-doc SKILL.md require reading the update reference when editing existing docs
  • call out when old block IDs are invalidated, when new IDs must be fetched, and when move operations require refreshing position context

Tests

  • go test ./shortcuts/doc
  • node scripts/skill-format-check/index.js

@github-actions github-actions Bot added domain/ccm PR touches the ccm domain size/M Single-domain feat or fix with limited business impact labels Jun 17, 2026
@github-actions

Copy link
Copy Markdown

🚀 PR Preview Install Guide

🧰 CLI update

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

🧩 Skill update

npx skills add larksuite/cli#docs/lark-doc-block-id-lifecycle -y -g

@github-actions

Copy link
Copy Markdown

PR Quality Summary

CI did not complete successfully. Use the failed check links below to decide whether this PR needs a code change or a rerun.

Failed checks

@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.52%. Comparing base (c61acb5) to head (7384924).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1501      +/-   ##
==========================================
+ Coverage   73.48%   73.52%   +0.04%     
==========================================
  Files         782      783       +1     
  Lines       74555    74630      +75     
==========================================
+ Hits        54784    54874      +90     
+ Misses      15674    15662      -12     
+ Partials     4097     4094       -3     

☔ 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.

@fangshuyu-768

Copy link
Copy Markdown
Collaborator Author

Closing in favor of #1508. The block ID lifecycle guidance changes from this PR have been folded into #1508.

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

Labels

domain/ccm PR touches the ccm domain 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