Skip to content

refactor(onboard): assemble FSM phase sequence#4485

Merged
cv merged 55 commits into
mainfrom
stack/onboard-fsm-flow-sequence
Jun 9, 2026
Merged

refactor(onboard): assemble FSM phase sequence#4485
cv merged 55 commits into
mainfrom
stack/onboard-fsm-flow-sequence

Conversation

@cv

@cv cv commented May 29, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Summary

Add an ordered onboarding FSM phase sequence builder. The builder composes the extracted phase wrappers into the canonical onboarding order while delegating actual phase behavior to supplied handlers.

Changes

  • Add buildOnboardFlowPhaseSequence() and OnboardFlowPhaseHandlers.
  • Assemble preflight, gateway, provider/inference, sandbox, OpenClaw, agent setup, policies, finalization, and post-verify phases.
  • Add tests for phase ordering and delegated handler execution.

Type of Change

  • Code change (feature, bug fix, or refactor)
  • Code change with doc updates
  • Doc only (prose changes, no code sample modifications)
  • Doc only (includes code sample changes)

Verification

  • npx prek run --all-files passes
  • npm test passes
  • Tests added or updated for new or changed behavior
  • No secrets, API keys, or credentials committed
  • Docs updated for user-facing behavior changes
  • npm run docs builds without warnings (doc changes only)
  • Doc pages follow the style guide (doc changes only)
  • New doc pages include SPDX header and frontmatter (new pages only)

Signed-off-by: Carlos Villela cvillela@nvidia.com

Summary by CodeRabbit

  • Tests
    • Added comprehensive test coverage for onboarding flow phases and sequence validation, including runtime execution and context propagation.

cv added 29 commits May 27, 2026 15:18
@cv
docs(onboard): document FSM migration target
6b753bf 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): centralize machine state metadata
fb1b32d 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): derive session step mapping from FSM metadata
c3e4ad6 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): derive progress labels from FSM metadata
603832c 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
fix(onboard): emit lifecycle events for onboarding start
4fad8e7 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
fix(onboard): emit machine events for resume conflicts
f99e9cb 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): introduce explicit state result types
2b60df4 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): apply explicit state results through runtime
30341b0 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): make finalization return FSM result
d4ad2d9 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): make agent setup return FSM result
356c947 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): make policy setup return FSM result
2296519 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): make preflight and gateway return FSM results
67a9a1e 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): make sandbox return branch FSM result
46f4a49 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): return FSM results from provider inference
9cc15f5 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): add FSM runner shell
dbbb273 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): consume handler FSM results compatibly
6b27a0b 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): allow step recording without machine transitions
44009ad 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): plumb step mutation options through runtime
cd6e5f7 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): add record-only FSM runner adapter
e266e3b 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): return ordered provider FSM results
bf4da0b 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): run live sequence with record-only steps
212ff4d 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): let FSM handlers return result sequences
f69f60a 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): add sequence runner adapter
727ac69 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): support FSM runner stop states
75f82f7 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): define FSM flow context
59deee6 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): extract preflight and gateway FSM phases
25c5abf 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): extract provider and sandbox FSM phases
4d9cc9f 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): extract agent policy finalization FSM phases
8a5b54a 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
refactor(onboard): assemble FSM phase sequence
bec7ef8 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv cv self-assigned this May 29, 2026
cv added 6 commits June 9, 2026 01:17
@cv
merge(onboard): sync provider sandbox phases with preflight phases
42eeb90 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
test(onboard): cover sandbox branch metadata passthrough
dc8c463 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
merge(onboard): sync finalization phases with provider sandbox phases
5c76573 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
test(onboard): run finalization phases through sequence runner
68722ed 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
merge(onboard): sync flow sequence with finalization phases
1775dc2 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv
test(onboard): validate assembled provider sequence
3de7633 
Signed-off-by: Carlos Villela <cvillela@nvidia.com>
@cv cv marked this pull request as ready for review June 9, 2026 08:48
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Selective E2E Results — ✅ All requested jobs passed

Run: 27194945418
Target ref: 3de76332c8312ad9d57d5afbca6ef33395da0e4b
Workflow ref: stack/onboard-fsm-agent-policy-finalization-phases
Requested jobs: cloud-e2e,onboard-resume-e2e
Summary: 2 passed, 0 failed, 0 skipped

Job Result
cloud-e2e ✅ success
onboard-resume-e2e ✅ success

jyaunches and others added 16 commits June 9, 2026 10:00
@jyaunches
Merge branch 'main' into stack/onboard-fsm-flow-context
5c7b7ee
@cv
Merge branch 'main' into stack/onboard-fsm-flow-context
83c1412
@cv
Merge branch 'stack/onboard-fsm-flow-context' into stack/onboard-fsm-…
6abbbb4 
…preflight-gateway-phases
@cv
Merge branch 'stack/onboard-fsm-preflight-gateway-phases' into stack/…
c07f0dd 
…onboard-fsm-provider-sandbox-phases
@cv
Merge branch 'stack/onboard-fsm-provider-sandbox-phases' into stack/o…
cf6c2a9 
…nboard-fsm-agent-policy-finalization-phases
@cv
Merge branch 'stack/onboard-fsm-agent-policy-finalization-phases' int…
4a297d0 
…o stack/onboard-fsm-flow-sequence
@cv
Merge branch 'main' into stack/onboard-fsm-flow-context
23e8577
@cv
chore: apply static formatting for FSM flow stack
abbd8c4
@cv
Merge remote-tracking branch 'origin/stack/onboard-fsm-flow-context' …
b592eac 
…into stack/onboard-fsm-preflight-gateway-phases
@cv
chore(onboard): format preflight gateway FSM phase
adbbdfa
@cv
Merge remote-tracking branch 'origin/stack/onboard-fsm-preflight-gate…
8a6bca2 
…way-phases' into stack/onboard-fsm-provider-sandbox-phases
@cv
chore(onboard): format provider sandbox FSM phase
baeab37
@cv
Merge remote-tracking branch 'origin/stack/onboard-fsm-provider-sandb…
5118bd7 
…ox-phases' into stack/onboard-fsm-agent-policy-finalization-phases
@cv
chore(onboard): format finalization FSM phases
a64242b
@cv
Merge remote-tracking branch 'origin/stack/onboard-fsm-agent-policy-f…
32059c9 
…inalization-phases' into stack/onboard-fsm-flow-sequence
@cv
chore(onboard): format FSM flow sequence
3562942
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Selective E2E Results — ✅ All requested jobs passed

Run: 27227889323
Target ref: 35629428b7d2835f2c6a28c84d595ce40ed3db9f
Workflow ref: stack/onboard-fsm-agent-policy-finalization-phases
Requested jobs: cloud-onboard-e2e,onboard-resume-e2e
Summary: 2 passed, 0 failed, 0 skipped

Job Result
cloud-onboard-e2e ✅ success
onboard-resume-e2e ✅ success

@cv cv changed the base branch from stack/onboard-fsm-agent-policy-finalization-phases to main June 9, 2026 18:59
@cv cv merged commit a7261f6 into main Jun 9, 2026
42 checks passed
@cv cv deleted the stack/onboard-fsm-flow-sequence branch June 9, 2026 19:10
@coderabbitai coderabbitai Bot mentioned this pull request Jun 10, 2026
Merged
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@cv cv

Labels

area: onboarding Onboarding FSM, provider setup, sandbox launch, or first-run flow refactor PR restructures code without intended behavior change v0.0.62 Release target

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cv @wscurran @jyaunches