Skip to content

fix(ci): install stores extra in cypher-subset gate#110

Merged
se-jo-ma merged 1 commit into
mainfrom
fix/cypher-subset-extras
Jun 24, 2026
Merged

fix(ci): install stores extra in cypher-subset gate#110
se-jo-ma merged 1 commit into
mainfrom
fix/cypher-subset-extras

Conversation

@se-jo-ma

Copy link
Copy Markdown
Member

Problem

cypher-subset is red on main and every PR (it's what bricks release PR #104), failing with:

ModuleNotFoundError: No module named 'ryugraph'
src/stargraph/stores/ryugraph.py:292: ModuleNotFoundError

Root cause

The job set the stores extra via astral-sh/setup-uv's sync-args input — but setup-uv has no such input. CI logged it:

##[warning]Unexpected input(s) 'sync-args', valid inputs are ['version','python-version',...]

setup-uv only installs uv; it does not run uv sync. The actual sync comes from uv run, which only pulls default deps + default groups — never extras. So ryugraph (in the stores extra) was never installed and the loud-fail gate (NFR-4) blew up on import.

This is a latent bug, not a regression — the bogus sync-args never worked. Only cypher-subset surfaces it because it's the one gate that hard-imports ryugraph; the other sync-args jobs (test/serve/knowledge) pass without their declared extras and are left untouched (surgical scope).

Fix

Drop the no-op sync-args and pass extras to uv run directly:

- run: uv run --group dev --extra stores pytest ...

Verified locally (linux, py3.13)

  • uv run --group dev python -c "import ryugraph"ModuleNotFoundError (reproduces CI)
  • uv run --group dev --extra stores python -c "import ryugraph" → OK
  • ryugraph 25.9.2 has a linux cp313 wheel; installs in ms, no source build.

🤖 Generated with Claude Code

setup-uv has no `sync-args` input — it was silently ignored (CI logged 'Unexpected input(s) sync-args'), so `--extra stores` never installed and the cypher-subset gate hard-failed on `import ryugraph` (ModuleNotFoundError). Pass `--group dev --extra stores` to `uv run` directly. Verified locally: `uv run --group dev` reproduces the failure, `uv run --group dev --extra stores` imports ryugraph cleanly.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017pXsj4Q7q1Vn2mtHhakTtJ
Signed-off-by: se-jo-ma <mauk.sean@gmail.com>
@se-jo-ma se-jo-ma merged commit 4ca2af4 into main Jun 24, 2026
14 checks passed
@se-jo-ma se-jo-ma deleted the fix/cypher-subset-extras branch June 24, 2026 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant