Skip to content

Conversation

@adrianlyjak
Copy link
Contributor

@adrianlyjak adrianlyjak commented Nov 14, 2025

  • Moves workflows-py into packages
  • Refactors and adds to the workflows scripts, moving them into their own workflows-dev package
  • Updates coverage to run against all packages, not just llama-index-workflows
  • Updates release process. It's now triggered automatically by bumping the package version and pushing to main

tagging

PRs are automatically tagged based on changed folders with e.g. pkg:llama-index-workflows

Details of Release process

  • Update pyproject(s) versions
  • Github action will compare each package's version to the the last highest tag with its prefix
  • If increased, it will trigger a release
    • runs the build
    • publishes
    • pushes a prefixed tag, e.g. llama-index-workflows@v2.11.2
    • generates release notes by querying github api for the tags, and scooping up any PRs within the range that have a matching tag for the package
    • shoving the PR titles in the release notes. (This still uses the enhancement, bug, breaking-change tags, if applied)
    • triggering the openapi publishing if configured to do so

@adrianlyjak adrianlyjak force-pushed the adrian/multi-project branch 2 times, most recently from 8bb64cc to 51d5941 Compare November 14, 2025 22:59
@coveralls
Copy link

coveralls commented Nov 14, 2025

Pull Request Test Coverage Report for Build 19440894745

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.4%) to 87.266%

Totals Coverage Status
Change from base Build 19374624494: -2.4%
Covered Lines: 8683
Relevant Lines: 9950

💛 - Coveralls

@adrianlyjak adrianlyjak force-pushed the adrian/multi-project branch 9 times, most recently from bea4c39 to feb4c35 Compare November 15, 2025 01:39
@adrianlyjak adrianlyjak marked this pull request as ready for review November 15, 2025 01:48
@adrianlyjak adrianlyjak changed the title wip: make workflows more of a multi project Update workflows for better mono-repo build support Nov 15, 2025
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be part of workflows-dev or nah?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah geez, vibe coding gone wrong, that was the intent

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah this is a stray extra file

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR restructures the repository to better support mono-repo builds by moving the workflows Python package into a packages/ directory structure and extracting workflow development scripts into a dedicated workflows-dev package. The changes enable automated release processes triggered by version bumps and improve test coverage across all packages.

Key Changes

  • Reorganized project structure with packages/ directory containing llama-index-workflows, llama-index-utils-workflow, and workflows-dev
  • Extracted and refactored workflow build/release scripts into the workflows-dev package with improved CLI interface
  • Updated release automation to trigger on version bumps with automatic tag creation and release note generation
  • Migrated test files from root to appropriate package locations

Reviewed Changes

Copilot reviewed 28 out of 123 changed files in this pull request and generated no comments.

Show a summary per file
File Description
uv.lock Updated dependency lock file to reflect new package structure with editable installs from packages directory
pyproject.toml Converted root pyproject to workspace configuration, defining workspace members
tests/test_scripts.py Removed legacy script tests, replaced by workflows-dev package tests
scripts/*.py Removed scripts moved to workflows-dev package
packages/workflows-dev/* New package containing refactored workflow development tools with CLI interface
packages/llama-index-workflows/tests/* New comprehensive test suite covering workflows, server, runtime, and client functionality

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@run-llama run-llama deleted a comment from Copilot AI Nov 15, 2025
@@ -1,14 +1,11 @@
changelog:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, I think this should actually be deleted now

@adrianlyjak adrianlyjak merged commit 4d93f89 into main Nov 17, 2025
15 checks passed
@adrianlyjak adrianlyjak deleted the adrian/multi-project branch November 17, 2025 19:18
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.

5 participants