refactor(workflow): S16 legacy retry field demotion

[gsxdsm]

Stack Slice

• Slice: S16
• Milestone: Deletion
• Base branch: feature/workflow-owned-merge-s15-self-healing-policy-deletion
• Full plan: docs/plans/2026-06-09-003-refactor-workflow-owned-merge-full-migration-slices-plan.md

Goal

Demote task-level retry/merge counters to projections and remove policy reads that still treat them as authority.

Dependency

S9 retry state, S12 projections, and S15 self-healing policy deletion.

Expected Scope

core types, retry summary, manual retry reset, core store, project engine, self-healing, retry tests.

Expected Tests

Retry summaries derive from workflow state, manual retry emits workflow wake, task field changes alone cannot cause scheduler/recovery/merge action.

Exit Gate

Task retry fields are display-only compatibility data.

Status

Draft stack placeholder. This PR reserves ordering and review context; implementation should replace or extend the handoff artifact before this slice is marked ready.

Implementation Added

• Added hasAuthoritativeWorkflowWork and extended projection tests so workflow work beats legacy retry counters.\n- Legacy retry fields are now explicitly display-only fallback data in projection.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 4d21b8b7-5421-4f33-80ab-478cd9dfb17a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• ✅ Review completed - (🔄 Check again to review again)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/workflow-owned-merge-s16-legacy-retry-field-demotion

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[greptile-apps]

Greptile Summary

This PR adds hasAuthoritativeWorkflowWork — a new gate function that signals whether workflow work items should be considered authoritative over legacy task retry counters — and exports it from @fusion/core. The gate checks state !== "cancelled", which exactly mirrors when projectWorkflowWorkStatus returns source: "workflow", closing the alignment gap flagged in earlier review rounds.

• hasAuthoritativeWorkflowWork is added to workflow-work-projection.ts with a state-based predicate (any non-cancelled item is authoritative), replacing the previous kind-based filter that excluded "task" items and incorrectly included cancelled items.
• Two new tests verify alignment: cancelled-only items yield false/"legacy" on both the gate and projection, while kind: "task" runnable items yield true/"workflow" on both.

Confidence Score: 5/5

Safe to merge. The new gate function is a pure addition with no changes to existing behavior.

The state-based predicate state !== 'cancelled' is exactly the condition under which projectWorkflowWorkStatus returns source: 'workflow', so the two functions are provably aligned for every possible WorkflowWorkItemState. Tests cover the previously flagged edge cases (cancelled-only items, task-kind items) and a representative active-work scenario. No callers of the new export exist yet — it is staged for future slices — so there is no downstream blast radius from this change.

No files require special attention.

Important Files Changed

Filename	Overview
packages/core/src/workflow-work-projection.ts	Adds hasAuthoritativeWorkflowWork using state !== "cancelled" — correctly aligned with projectWorkflowWorkStatus's source: "legacy" condition (all items cancelled or empty). No logic bugs found.
packages/core/src/tests/workflow-work-projection.test.ts	Adds alignment tests for cancelled items and task-kind items. Alignment for the succeeded state is correct in implementation but not exercised by a new test case.
packages/core/src/index.ts	Exports hasAuthoritativeWorkflowWork alongside the existing projection exports — clean, single-line change.
docs/plans/workflow-owned-merge-stack/s16-legacy-retry-field-demotion.md	Draft stack-handoff doc for S16. Reserves the review slot; still carries placeholder language in Expected File Scope and Exit Gate per prior acknowledged threads.

_{Reviews (8): Last reviewed commit: "fix(FN-000): align authoritative workflo..." | Re-trigger Greptile}

[gsxdsm]

The gate function filters by kind alone while the projection additionally filters by state.

Addressed: hasAuthoritativeWorkflowWork is now aligned with projection authority by returning true only for non-cancelled authoritative work, with direct tests for empty, task-kind, retrying, and cancelled cases.

[stage-review]

Ready to review this PR? Stage has broken it down into 3 individual chapters for you:

	Title
1	Document S16 legacy retry field demotion plan
2	Introduce authoritative workflow work detection
3	Test workflow work projection authority

_{Chapters generated by Stage for commit edd43ce on Jun 16, 2026 9:19am UTC.}