Repository navigation
Add chalk-plan-cache plugin for debugging query plan cache misses - #4
Merged
Merged
Conversation
New plugin with the debug-plan-cache-misses skill: finds plan cache
miss log lines ("missed the plan cache" / "Computed plan for ...")
via the chalk logs CLI and diffs consecutive plan computations to
pinpoint which BatchQuery field (usually the given_features set) forces
the miss. Registered in both the Claude Code and Codex marketplace
catalogs.
Claude session: 754768b0-0140-45fa-821e-23b08105942d
There was a problem hiding this comment.
Pull request overview
Adds a new internal plugin, chalk-plan-cache, to help investigate Chalk query plan cache misses by guiding users through searching and diffing relevant engine log lines via the chalk logs CLI.
Changes:
- Introduces the
debug-plan-cache-missesskill with a step-by-step workflow for locating miss signals and diffing consecutiveComputed plan for …log entries. - Adds plugin metadata for both Claude and Codex plugin formats (name/version/description/author, plus Codex skill + interface wiring).
- Registers the plugin in both internal marketplaces (
.claude-plugin/marketplace.jsonand.agents/plugins/marketplace.json).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
plugins/chalk-plan-cache/skills/debug-plan-cache-misses/SKILL.md |
New skill documentation describing how to find and debug plan-cache misses from logs. |
plugins/chalk-plan-cache/README.md |
Plugin-level overview and basic usage examples for the skill. |
plugins/chalk-plan-cache/.codex-plugin/plugin.json |
Codex plugin metadata including skill directory and UI-facing interface fields. |
plugins/chalk-plan-cache/.claude-plugin/plugin.json |
Claude plugin metadata for marketplace consumption. |
.claude-plugin/marketplace.json |
Adds chalk-plan-cache to the internal Claude Code marketplace list. |
.agents/plugins/marketplace.json |
Adds chalk-plan-cache to the agents plugin marketplace list with git-subdir source. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
What
New
chalk-plan-cacheplugin with a single skill,debug-plan-cache-misses, that finds and debugs Chalk query plan cache misses from engine logs via thechalk logsCLI.Why
When a query keeps replanning on every request (an
Ad-hoc query request ... missed the plan cachewarning, latency from repeated planning), the cause is that some field in the normalizedBatchQuery— most often thegiven_featuresset — is varying between calls. This skill turns that investigation into a repeatable procedure.What the skill does
message:"missed the plan cache"(ad-hoc miss warning)message:"Computed plan for"(every plan computation — the useful one for diffing)chalk logsquery syntax (field:value, quoting, time flags,--aggregate,--follow).BatchQuery(planner options resolved to effective values, so unset == default never causes a spurious miss).Computed plan for … given …lines field-by-field,givenset first, to pinpoint the differing field.Files
plugins/chalk-plan-cache/.claude-plugin/plugin.json+.codex-plugin/plugin.jsonplugins/chalk-plan-cache/skills/debug-plan-cache-misses/SKILL.mdplugins/chalk-plan-cache/README.md.claude-plugin/marketplace.jsonand.agents/plugins/marketplace.jsonNotes
chalkas the CLI binary throughout (notchalkadmin --plane <p> cli <env>).0.1.0— initial release of a new plugin, consistent with the other plugins' initial versions (no prior release to bump from).claude plugin validate .passes.Claude session: 754768b0-0140-45fa-821e-23b08105942d