Skip to content

[operation-graph] Add Rush-compatible operation statuses - #5889

Merged
Bharat Middha (bmiddha) merged 2 commits into
mainfrom
bmiddha/ws0-t5-converge-operation-graph
Aug 13, 2026
Merged

[operation-graph] Add Rush-compatible operation statuses#5889
Bharat Middha (bmiddha) merged 2 commits into
mainfrom
bmiddha/ws0-t5-converge-operation-graph

Conversation

@bmiddha

Copy link
Copy Markdown
Member

Summary

Prepare @rushstack/operation-graph for eventual convergence with rush-lib's stateful operation graph by adding the four operation statuses currently defined only by Rush.

This is intentionally a small, non-blocking first increment. It does not change rush-lib or replace its OperationGraph.

Details

  • Add Queued, SuccessWithWarning, Skipped, and FromCache to OperationStatus, using the exact string values already used by rush-lib.
  • Extend the package's exhaustive requestRun status handling so the new values remain type-safe.
  • Keep existing execution behavior unchanged: no current @rushstack/operation-graph path starts emitting any new status.
  • Record the additive @beta API change as a minor change and update the API report.

The two graph implementations remain structurally different: Rush requires persistent iteration state, selective invalidation, weighted concurrency, and a larger hook/runner contract. Those pieces should converge through later additive APIs and compatibility adapters rather than a big-bang replacement.

How it was tested

  • node common/scripts/install-run-rush.js test --to @rushstack/operation-graph
  • cd libraries/operation-graph && node_modules/.bin/heft test --clean (27 passed, 0 failed)
  • cd apps/heft && node_modules/.bin/heft build --clean
  • node common/scripts/install-run-rush.js test --to @microsoft/rush-lib --to @rushstack/operation-graph
  • node common/scripts/install-run-rush.js change --verify

No snapshots changed.

Add Rush's queued, warning, skipped, and cache statuses to the standalone enum and cover them in exhaustive rerun handling. Existing execution paths continue to emit the same statuses.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Comment thread libraries/operation-graph/src/OperationStatus.ts Outdated
Comment thread libraries/operation-graph/src/OperationStatus.ts
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 8ebd5bf2-c44b-42d5-be25-e7936d4b0a14
@bmiddha
Bharat Middha (bmiddha) enabled auto-merge (squash) August 11, 2026 20:25
@bmiddha
Bharat Middha (bmiddha) merged commit 2ce4f59 into main Aug 13, 2026
9 checks passed
@bmiddha
Bharat Middha (bmiddha) deleted the bmiddha/ws0-t5-converge-operation-graph branch August 13, 2026 21:52
@github-project-automation github-project-automation Bot moved this from Needs triage to Closed in Bug Triage Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

2 participants