From 097b50a5d1223f85a11768bb59c0e5454a3e26ad Mon Sep 17 00:00:00 2001 From: Safi Date: Tue, 2 Jun 2026 21:33:35 +0100 Subject: [PATCH] give the test job full git history and lock tomli for 3.10 test_skillgen.py (added in #1121) reads pre-split skill bodies from the immutable baseline commit via `git show`, and on Python 3.10 imports tomllib via the tomli fallback. The test job used a shallow checkout (no baseline tree) and the lock didn't carry tomli, so the 3.10 job failed two ways. Add fetch-depth: 0 to the test job (mirroring skillgen-check) and lock tomli under the python_version < 3.11 marker so --frozen sync installs it. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/ci.yml | 5 +++++ uv.lock | 14 ++++++++++---- 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index d93649c..0e0b1ab 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -54,6 +54,11 @@ jobs: steps: - uses: actions/checkout@v6 + with: + # test_skillgen.py reads pre-split skill bodies from the immutable + # baseline commit via `git show`; a shallow checkout omits that history + # and the baseline tests fail. Full history mirrors the skillgen-check job. + fetch-depth: 0 - name: Install uv uses: astral-sh/setup-uv@v8.1.0 diff --git a/uv.lock b/uv.lock index 1a78eb9..29ef7b0 100644 --- a/uv.lock +++ b/uv.lock @@ -1109,7 +1109,7 @@ wheels = [ [[package]] name = "graphifyy" -version = "0.8.24" +version = "0.8.28" source = { editable = "." } dependencies = [ { name = "datasketch" }, @@ -1121,7 +1121,6 @@ dependencies = [ { name = "tree-sitter-c" }, { name = "tree-sitter-c-sharp" }, { name = "tree-sitter-cpp" }, - { name = "tree-sitter-dm" }, { name = "tree-sitter-elixir" }, { name = "tree-sitter-fortran" }, { name = "tree-sitter-go" }, @@ -1160,6 +1159,7 @@ all = [ { name = "pypdf" }, { name = "python-docx" }, { name = "tiktoken" }, + { name = "tree-sitter-dm" }, { name = "tree-sitter-sql" }, { name = "watchdog" }, { name = "yt-dlp" }, @@ -1170,6 +1170,9 @@ bedrock = [ chinese = [ { name = "jieba" }, ] +dm = [ + { name = "tree-sitter-dm" }, +] gemini = [ { name = "openai" }, { name = "tiktoken" }, @@ -1234,6 +1237,7 @@ dev = [ { name = "ruff" }, { name = "safety" }, { name = "setuptools" }, + { name = "tomli", marker = "python_full_version < '3.11'" }, { name = "wheel" }, ] @@ -1279,7 +1283,8 @@ requires-dist = [ { name = "tree-sitter-c" }, { name = "tree-sitter-c-sharp" }, { name = "tree-sitter-cpp" }, - { name = "tree-sitter-dm" }, + { name = "tree-sitter-dm", marker = "extra == 'all'" }, + { name = "tree-sitter-dm", marker = "extra == 'dm'" }, { name = "tree-sitter-elixir" }, { name = "tree-sitter-fortran" }, { name = "tree-sitter-go" }, @@ -1308,7 +1313,7 @@ requires-dist = [ { name = "yt-dlp", marker = "extra == 'all'" }, { name = "yt-dlp", marker = "extra == 'video'" }, ] -provides-extras = ["mcp", "neo4j", "pdf", "watch", "svg", "leiden", "office", "google", "video", "kimi", "ollama", "bedrock", "gemini", "openai", "chinese", "sql", "all"] +provides-extras = ["mcp", "neo4j", "pdf", "watch", "svg", "leiden", "office", "google", "video", "kimi", "ollama", "bedrock", "gemini", "openai", "chinese", "sql", "dm", "all"] [package.metadata.requires-dev] dev = [ @@ -1325,6 +1330,7 @@ dev = [ { name = "ruff", specifier = ">=0.15.13" }, { name = "safety", specifier = ">=3.7.0" }, { name = "setuptools", specifier = ">=82.0.1" }, + { name = "tomli", marker = "python_full_version < '3.11'", specifier = ">=2.0" }, { name = "wheel", specifier = ">=0.47.0" }, ]