Skip to content

refactor(torghut): normalize scheduler pipeline modules#10870

Merged
gregkonush merged 2 commits into
mainfrom
codex/torghut-scheduler-pipeline-quality-20260613b
Jun 14, 2026
Merged

refactor(torghut): normalize scheduler pipeline modules#10870
gregkonush merged 2 commits into
mainfrom
codex/torghut-scheduler-pipeline-quality-20260613b

Conversation

@gregkonush

Copy link
Copy Markdown
Member

Summary

  • Replace generated scheduler pipeline part_* modules with semantic modules for run-cycle, signal processing, decision lifecycle, submission policy, runtime gates, LLM review, LLM outcomes, support helpers, and typed request contexts.
  • Replace generated simple submission-preparation part_* modules with semantic quote sizing, quote routeability, direct submission, quote-value extraction, and shared request objects.
  • Keep existing public import surfaces and scheduler behavior stable while updating tests away from generated/private module imports.
  • Remove touched-file Pyright suppression in scheduler safety helpers and keep the Torghut quality plugin clean for this diff.

Related Issues

None

Testing

  • cd services/torghut && uv run --frozen ruff format app tests scripts migrations
  • cd services/torghut && uv run --frozen python -m compileall app scripts
  • cd services/torghut && uv run --frozen ruff format --check app tests scripts migrations && uv run --frozen ruff check app tests scripts migrations
  • cd services/torghut && uv run --frozen pylint app scripts tests migrations --disable=all --enable=too-many-lines --score=n
  • cd services/torghut && uv run --frozen pylint "$changed_python" --disable=all --enable=too-many-arguments,too-many-positional-arguments,too-many-branches,too-many-locals,too-many-return-statements,too-many-statements,too-many-nested-blocks --score=n, where $changed_python was the changed app and scripts Python file list from git diff origin/main plus untracked files.
  • cd services/torghut && uv run --frozen pylint --load-plugins=scripts.pylint_torghut_quality "$changed_python" --disable=all --enable=torghut-generated-split-filename,torghut-dynamic-globals-reexport,torghut-compat-module-wrapper,torghut-compat-module-registry,torghut-module-class-mutation,torghut-module-replacement,torghut-file-pyright-suppression,torghut-type-ignore,torghut-file-ruff-noqa,torghut-blanket-pylint-disable,torghut-dynamic-attribute-hook,torghut-dynamic-all,torghut-wildcard-import,torghut-custom-module-class --score=n, where $changed_python was the changed Torghut Python file list from git diff origin/main plus untracked files.
  • cd services/torghut && uv run --frozen pyright --project pyrightconfig.json
  • cd services/torghut && uv run --frozen pyright --project pyrightconfig.alpha.json && uv run --frozen pyright --project pyrightconfig.scripts.json
  • cd services/torghut && uv run --frozen pytest tests/pipeline/test_trading_pipeline_dspy_gate_a.py tests/pipeline/test_trading_pipeline_dspy_gate_b.py tests/pipeline/test_trading_pipeline_dspy_gate_c.py tests/pipeline/test_trading_pipeline_dspy_gate_d.py tests/pipeline/test_trading_pipeline_execution_llm_a.py tests/pipeline/test_trading_pipeline_execution_llm_b.py tests/pipeline/test_trading_pipeline_external_targets_a.py tests/pipeline/test_trading_pipeline_external_targets_c.py tests/pipeline/test_trading_pipeline_external_targets_d.py tests/pipeline/test_trading_pipeline_live_regime_a.py tests/pipeline/test_trading_pipeline_live_regime_b.py tests/pipeline/test_trading_pipeline_materialized_target_plan_a.py tests/pipeline/test_trading_pipeline_materialized_target_plan_b.py tests/pipeline/test_trading_pipeline_materialized_target_plan_c.py tests/pipeline/test_trading_pipeline_position_projection_a.py tests/pipeline/test_trading_pipeline_position_projection_b.py tests/pipeline/test_trading_pipeline_probe_exits_a.py tests/pipeline/test_trading_pipeline_probe_exits_b.py tests/pipeline/test_trading_pipeline_quote_outcome.py tests/pipeline/test_trading_pipeline_route_execution_a.py tests/pipeline/test_trading_pipeline_route_execution_b.py tests/pipeline/test_trading_pipeline_target_plan_source_a.py tests/pipeline/test_trading_pipeline_target_plan_source_b.py tests/pipeline/test_trading_pipeline_warmup_submission_a.py tests/pipeline/test_trading_pipeline_warmup_submission_b.py tests/pipeline/test_trading_pipeline_paper_route_quote_a.py tests/pipeline/test_trading_pipeline_paper_route_quote_b.py tests/pipeline/test_trading_pipeline_retry_profit_floor_a.py tests/simple_pipeline/test_part_01.py tests/simple_pipeline/test_part_03.py -q --tb=short --maxfail=10
  • git diff --check
  • bun run lint:argocd

Screenshots (if applicable)

N/A

Breaking Changes

None

Checklist

  • Testing section documents the exact validation performed.
  • Screenshots and Breaking Changes sections are handled appropriately.
  • Documentation, release notes, and follow-ups are updated or tracked.

@gregkonush gregkonush merged commit 66b1968 into main Jun 14, 2026
24 checks passed
@gregkonush gregkonush deleted the codex/torghut-scheduler-pipeline-quality-20260613b branch June 14, 2026 03:43
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