Skip to content

fix(ci): update codegen-models workflow to use new Python output path#780

Open
khaliqgant wants to merge 2 commits intomainfrom
fix/codegen-ci-paths
Open

fix(ci): update codegen-models workflow to use new Python output path#780
khaliqgant wants to merge 2 commits intomainfrom
fix/codegen-ci-paths

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented Apr 24, 2026

Summary

  • Fixes the codegen CI job which was failing with fatal: pathspec 'packages/sdk-py/agent_relay/models.py' did not match any files
  • The Python codegen output was moved from sdk-py/agent_relay/ to sdk-py/src/agent_relay/ (to stop shadowing the real SDK package), but the workflow still referenced the old paths in its git diff check and git add step
  • Also drops agent_relay/__init__.py from the workflow since codegen no longer generates it

Test plan

  • Confirm codegen job passes on this PR

🤖 Generated with Claude Code


Open in Devin Review

After moving Python codegen output from sdk-py/agent_relay/ to
sdk-py/src/agent_relay/, the CI workflow was still referencing the old
paths in its git diff check and git add step, causing exit code 128.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
devin-ai-integration[bot]

This comment was marked as resolved.

…tion

- codegen-py.mjs was still writing to sdk-py/agent_relay/ (old path),
  making the CI diff check against sdk-py/src/agent_relay/ always quiet
- Stop generating __init__.py to avoid clobbering the hand-maintained
  SDK public API at src/agent_relay/__init__.py
- Update lint-staged git add in package.json to match the new path and
  drop the stale __init__.py reference

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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