[Refactor] Modular pipeline architecture - #31
Open
TecJesh wants to merge 3 commits into
Open
Conversation
…detect
Remove _fix_origin() that forced origin to point to triton-lang/triton-ascend,
breaking private fork workflows. Restore the pre-refactor behavior where:
- origin is left as-is (user may point it to their private fork)
- ascend_head is resolved from origin/{TA_BASE_BRANCH} with graceful fallback
- detect.py fetches from origin and re-resolves ascend_head (matching old
detect_commits.py which called it 'private fork')
- TA_BASE_BRANCH defaults to 'main' (matching old utils.py), not 'upstream_sync'
This ensures that when GITHUB_REPO points to a private fork with previously
merged commits, the merge-base calculation reflects actual progress instead
of always starting from the upstream origin.
TecJesh
force-pushed
the
main-fix-ir-patch-gen
branch
from
July 27, 2026 12:54
9876e51 to
b653dc5
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Refactor the monolithic flow.py (4446 lines) into a modular pipeline architecture.
Architecture Changes
Before
flow.pycontaining all logic (4446 lines)TA_Main2MainStatePydantic model with mutable state@start,@router,@listendecorators)TA_MODE: full, merge, fix)print_info,print_error, etc.)After
utils/package:TAConfig,WorkflowContext,TALogger,run_git,timed()pipeline/package: 13 focused modules, each with(ctx, config) -> ctxsignatureflow.py: 143-line orchestrator with simplewhileloopTAConfig.from_env()WorkflowContext.copy_with()patternTALoggerwith structured formatting (header, section, status, table)Preserved Features
All existing functionality is preserved across the new modules:
pipeline/ir_patch.pypipeline/build.py+fix.pypipeline/test.py+fix.pypipeline/fix.pypipeline/resolve.pypipeline/plan.pypipeline/ir_patch.py