Skip to content

fix(ci): replace broken conda workflow with pip + CLI build validation#325

Open
alfredo-petri wants to merge 1 commit into
nextlevelbuilder:mainfrom
alfredo-petri:fix/ci-python-workflow
Open

fix(ci): replace broken conda workflow with pip + CLI build validation#325
alfredo-petri wants to merge 1 commit into
nextlevelbuilder:mainfrom
alfredo-petri:fix/ci-python-workflow

Conversation

@alfredo-petri

Copy link
Copy Markdown
Contributor

Problem

.github/workflows/python-package-conda.yml references environment.yml (line 23) which does not exist in the repository. Every push triggers a failing CI job.

Additionally, there was no CI validation for the CLI package, so regressions in the TypeScript build or npm packaging could go undetected until a user tries to install.

Fix

Python job:

  • Remove conda dependency; use pip install flake8 directly
  • Scope lint to src/ui-ux-pro-max/scripts/ (the actual source)
  • Add smoke test: run search.py with a real query to confirm the script is importable and functional

New cli-build job:

  • Uses oven-sh/setup-bun@v2 (official Bun action)
  • bun install + bun run build + npm pack --dry-run to catch packaging regressions

🤖 Generated with Claude Code

The previous workflow referenced environment.yml which does not exist in
the repository, causing every push to fail. It also had no validation
for the CLI package.

Changes:
- Remove conda dependency; use pip to install flake8 directly
- Lint only src/ui-ux-pro-max/scripts/ (the actual Python source)
- Add a smoke test: run search.py to confirm the script is importable
- Add cli-build job: bun install + build + npm pack --dry-run to catch
  packaging regressions before they reach npm

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mrgoonie mrgoonie added agent:github-maintain Processed by github-maintain automation maintain:blocked Blocked by external condition or maintainer action pr:blocked-conflict PR blocked by merge conflicts labels Jun 21, 2026
@mrgoonie

Copy link
Copy Markdown
Contributor

Summary: I’m blocking this PR from the cron-safe maintainer lane because it is now stale against current main and reintroduces an obsolete CI file.

Decision: blocked / needs rebase or close

Evidence:

Next step: please rebase on current main and retarget the useful part only. If the goal is still a Python script smoke/lint job, add it as a new, clearly named workflow or fold it into the current workflow strategy without resurrecting python-package-conda.yml. If the release workflow already covers the intended CLI build validation, this PR can be closed as superseded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:github-maintain Processed by github-maintain automation maintain:blocked Blocked by external condition or maintainer action pr:blocked-conflict PR blocked by merge conflicts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants