Skip to content

Commit 5ae45fd

Browse files
fix: give the generator its own venv on PATH
It shells out to ruff for post-processing. Finding none, it warns and exits 0, and the warning gate reports that as a spec it could not parse -- a clean regeneration on a runner without a global ruff failed with a message pointing at the wrong thing entirely. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014f9oEEYspPH4fmPULTnLkJ
1 parent a77ef6a commit 5ae45fd

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

tools/gen_sdk.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,10 @@ set -euo pipefail
1414

1515
REPO="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
1616
VENV="$REPO/.gen-venv"
17+
# The generator shells out to ruff for its own post-processing. Without this it
18+
# finds whatever ruff the caller happens to have, or none, and reports the miss
19+
# as a warning -- which the gate below reads as an unparsable spec.
20+
export PATH="$VENV/bin:$PATH"
1721
OUT="src/unstract/api_deployments/sdk_docstudio"
1822
# Pinned: unpinned, a generator upgrade and a spec change produce the same diff,
1923
# and the drift gate can no longer tell them apart.

0 commit comments

Comments
 (0)