Skip to content

Refactor - #5

Merged
manmit-s merged 18 commits into
mainfrom
refactor
Aug 4, 2026
Merged

Refactor#5
manmit-s merged 18 commits into
mainfrom
refactor

Conversation

@manmit-s

@manmit-s manmit-s commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary of Changes

Refactored the project structure to standard Python src/ layout, updated imports, and added PyPI release infrastructure with repository documentation.

Project Structure and Refactoring

  • Migrated Codebase: Moved core modules (agent, client, config, context, hooks, prompts, safety, tools, ui, utils) and main.py into src/flux_cli/.
  • Import Paths: Updated absolute imports across the codebase to use flux_cli. namespace to prevent top-level module collisions.
  • Package Discovery: Updated pyproject.toml to use where = ["src"] and ensured init.py files exist across subpackages.
  • Git Cleanup: Fixed .gitignore pathing (/flux_cli rule) to prevent ignoring src/flux_cli/, and removed legacy scratch scripts.

CI/CD and Community Guidelines

  • PyPI Release Pipeline: Added .github/workflows/publish.yml with OIDC (Trusted Publishing) for automated testing, package integrity checks (twine), and PyPI deployment on tag pushes.
  • Issue Templates: Added structured GitHub issue templates for Bug Reports and Feature Requests under .github/ISSUE_TEMPLATE/.
  • Repository Docs: Added CONTRIBUTING.md guide for open-source contributors.

Verification

  • Package installs cleanly in editable mode (pip install -e .).
  • CLI command (flux) builds and executes without import errors.
  • Build artifacts (sdist and wheel) generated successfully via python -m build.

@manmit-s
manmit-s merged commit df921ae into main Aug 4, 2026
3 checks passed
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