docs: CLAUDE.md — idle-cost measurement note + refresh stale auto-merge pitfall (#32)#33
Merged
Merged
Conversation
…e pitfall - Add a "Workflow pitfalls" note: measure idle cost via interactive /context (the Custom agents slot), NOT `claude plugin details` -- the latter's Always-on projection counts agent descriptions (~50 tok/agent) and misses the loaded bodies (~2k/agent), under-reporting by ~15x. - Refresh the publish-dist pitfall: it claimed ci.yml doesn't run on auto-PRs and "those checks are the validation signal" -- stale since PR #32 wired the ak2k-ci App token, which makes the auto-PR fire pull_request -> ci -> auto-merge. Documents the App-token mechanism.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two
CLAUDE.mdcorrections, no code change.New "Workflow pitfalls" note — measuring idle cost. Use interactive
/context(the Custom agents slot) as the source of truth, notclaude plugin details: the latter's Always-on projection counts only agent descriptions (~50 tok/agent) and misses the full bodies Claude Code loads idle (~2k/agent), under-reporting by ~15× (upstream CE:plugin details~5.7k vs/context~88k). Elevates the note already ingenerate_wrappers.py's docstring to the contributor-facing doc.Refresh a now-stale pitfall. The first bullet claimed
ci.ymldoesn't run onpublish-distauto-PRs and "those checks are the validation signal" — true under the oldGITHUB_TOKENsetup, but fix(ci): open publish-dist PRs with ak2k-ci App token so auto-merge fires #32 changed it: the auto-PR is now opened with theak2k-ciApp token, so it firespull_request→ci.yml→auto-merge-publish-dist. Corrected, and documents the App-token mechanism + whyGITHUB_TOKENstalled it.