skills: add malloy-materialization-inventory and -design - #945
Open
girishjeswani wants to merge 1 commit into
Open
skills: add malloy-materialization-inventory and -design#945girishjeswani wants to merge 1 commit into
girishjeswani wants to merge 1 commit into
Conversation
girishjeswani
force-pushed
the
skills/malloy-materialization-inventory-design
branch
from
July 31, 2026 19:16
de6dd14 to
812a271
Compare
Mirror the two sibling shared skills that flank malloy-materialization: - malloy-materialization-inventory: profile the warehouse before materializing (where the rows are, how selective each filter is, what the workload costs). - malloy-materialization-design: where the #@ persist tags go and how to shape sources around them, including the "narrow table + caller-varied filter => empty, not slow" correctness rule. Together with the existing malloy-materialization (tag syntax + build debugging) and Publisher's malloy-materialization-tuning (cost/perf), they form the what / where / how / tune set. Adds the two .claude/skills symlinks, lists both in skills/README.md's shared set, and regenerates skills_bundle.json (30 -> 32). No em-dashes, per the house-style test. Kept byte-identical with the upstream shared set (ms2data/agent-skills#69, tip f7213ea): generic illustrative examples (no client-specific schema/values/ figures), the draft-executor tool named bare (execute_query) with the portability note, no artifact-design cross-reference, the design frontmatter description single-quoted so YAML does not truncate it at ` #@`, and the denormalize example's second join declared. Signed-off-by: Girish Jeswani <girish@credibledata.com>
girishjeswani
force-pushed
the
skills/malloy-materialization-inventory-design
branch
from
August 3, 2026 18:44
812a271 to
187638b
Compare
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.
What
Adds the two sibling shared skills that flank
malloy-materialization:malloy-materialization-inventory- profile the warehouse before materializing: where the row volume actually is, how selective each hard-coded filter is, what the workload costs, and which tables to build.malloy-materialization-design- where the#@ persisttags go and how to shape sources around them, anchored on the correctness rule that a narrow table can only back a source that statically pins everything the table narrows on (otherwise a caller-varied filter returns empty, not slow - a silent wrong answer a parity check can't catch).Together with the existing
malloy-materialization(tag syntax + build debugging) and the Publisher-onlymalloy-materialization-tuning(cost/perf), they form a what → where → how → tune set. Each new skill hands off to the others byskill:reference; nothing is removed.Verified against the current compiler: persist-inheritance-through-
extend(refined-source.tsrecords the base persistable source's id for dependency tracking),query_source-only serve-time routing, andcompose()field-presence routing.Mechanics
.claude/skillssymlinks, both listed inskills/README.md's shared set, andskills_bundle.jsonregenerated (30 → 32; the house-style + sync spec passes locally,5 pass / 0 fail).Shared-set sync note
Synced from the upstream shared set with the standard shared-skill adaptations applied so it satisfies this repo's contract: em-dashes removed (house-style test), the draft-executor tool named bare (
execute_query) with the portability note, and theartifact-designcross-reference neutralized (it's a host-side skill, absent here). The upstream copy needs these identical edits to keep the two byte-identical.