Skip to content

docs(skills): state when context-tree-read syncs and how to search - #31

Merged
yuezengwu merged 1 commit into
mainfrom
feat/read-skill-sync-timing
Nov 4, 2026
Merged

yuezengwu merged 1 commit into
mainfrom
feat/read-skill-sync-timing

Conversation

@liuchao-001

Copy link
Copy Markdown
Contributor

Part of #29.

  • Sync once per task: at task start (before planning, changing code, or answering about recorded decisions), before writing, and when content looks outdated or is reported updated. Not per node read, not for follow-ups or conversation without Context.
  • Keep the returned treeRoot for the whole task; navigate with context-tree read, search with rg/grep, read known files directly.
  • Description now triggers at the start of any task in a project with connected trees.

The sentence about a repeated sync reusing the recent fetch relies on #30; merge after it.

pnpm check and the skills/install tests pass.

🤖 Generated with Claude Code

Sync once per task at task boundaries, before writing, and when content
looks outdated; never per node read or for follow-ups. Keep one
treeRoot per task, navigate with context-tree read, and search with rg
or grep.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@liuchao-001
liuchao-001 requested a review from yuezengwu November 4, 2026 13:18

@yuezengwu yuezengwu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed d2c8c7c integrated with current maince8789e5. The sync timing and fixed-treeRoot read/search guidance are clear and preserve the evidence/authority boundary. Full118-test suite (with HTTPS Git transport blocked), check, typecheck and real package installation smoke all passed. No code or Skill defect found.

As declared in this PR, merge depends on #30; this approval does not waive that integration dependency. Revalidate the resulting actual base after #30 merges.

@yuezengwu
yuezengwu merged commit 1d9518f into main Nov 4, 2026
3 checks passed
@yuezengwu
yuezengwu deleted the feat/read-skill-sync-timing branch November 4, 2026 15:23
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.

2 participants