Move data/skills one level above spark/ to felderize root

wilmaontherun · claude · wilmaontherun · commit d596584ada0e · 2026-03-30T10:05:44.000-07:00
skills are dialect-agnostic rules; placing at data/skills/ makes the
directory structure cleaner for future dialects alongside spark/.

Co-Authored-By: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/python/felderize/README.md b/python/felderize/README.md
@@ -119,7 +119,7 @@ Environment variables (set in `.env`):
 
 ## How it works
 
-1. Loads translation rules from a single skill file (`spark/data/skills/spark_skills.md`)
+1. Loads translation rules from a single skill file (`data/skills/spark_skills.md`)
 2. Sends Spark SQL to the LLM with rules, validated examples, and relevant Feldera SQL documentation (from `docs.feldera.com/docs/sql/`)
 3. Parses the translated Feldera SQL from the LLM response
 4. Optionally validates output against the Feldera compiler, retrying with error feedback if needed
diff --git a/python/felderize/data/skills/spark_skills.md b/python/felderize/data/skills/spark_skills.md
diff --git a/python/felderize/pyproject.toml b/python/felderize/pyproject.toml
@@ -20,8 +20,6 @@ felderize = "spark"
 
 [tool.setuptools.package-data]
 felderize = [
-    "data/skills/*.md",
-    "data/skills/**/*.md",
     "data/samples/*.md",
     "data/demo/*.sql",
 ]
diff --git a/python/felderize/spark/skills.py b/python/felderize/spark/skills.py
@@ -10,7 +10,7 @@
 def load_skills(skills_dir: str | Path | None = None) -> str:
     """Load all SKILL.md files and return a combined prompt string."""
     if skills_dir is None:
-        skills_dir = Path(__file__).resolve().parent / "data" / "skills"
+        skills_dir = Path(__file__).resolve().parent.parent / "data" / "skills"
     else:
         skills_dir = Path(skills_dir)
 

Original file line number	Diff line number	Diff line change
`@@ -20,8 +20,6 @@ felderize = "spark"`
`20`	`20`
`21`	`21`	`[tool.setuptools.package-data]`
`22`	`22`	`felderize = [`
`23`		`- "data/skills/*.md",`
`24`		`- "data/skills/*/.md",`
`25`	`23`	`"data/samples/*.md",`
`26`	`24`	`"data/demo/*.sql",`
`27`	`25`	`]`