fix(ci): replace broken conda workflow with pip + CLI build validation - #325
fix(ci): replace broken conda workflow with pip + CLI build validation#325alfredo-petri wants to merge 1 commit into
Conversation
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>
|
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 |
|
Thank you @alfredo-petri for diagnosing the broken Conda workflow and adding CLI validation. I revalidated this against the latest |
Problem
.github/workflows/python-package-conda.ymlreferencesenvironment.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:
pip install flake8directlysrc/ui-ux-pro-max/scripts/(the actual source)search.pywith a real query to confirm the script is importable and functionalNew
cli-buildjob:oven-sh/setup-bun@v2(official Bun action)bun install+bun run build+npm pack --dry-runto catch packaging regressions🤖 Generated with Claude Code