Skip to content

feat(vscode): add stateless codeful template - #9607

Open
Brian Lam (lambrianmsft) wants to merge 5 commits into
Azure:mainfrom
lambrianmsft:lambrian-microsoft-codeful-stateless-template
Open

feat(vscode): add stateless codeful template#9607
Brian Lam (lambrianmsft) wants to merge 5 commits into
Azure:mainfrom
lambrianmsft:lambrian-microsoft-codeful-stateless-template

Conversation

@lambrianmsft

@lambrianmsft Brian Lam (lambrianmsft) commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

Adds the missing Stateless workflow option and C# scaffold for codeful Logic Apps. Codeful workspace creation already supported Stateful, Autonomous agent, and Conversational agent workflows, while the shared workflow model defined a Stateless codeful type that was not exposed or generated. This change completes that end-to-end contract, discovers created Stateless workflows with the correct runtime kind, and makes them selectable in the existing Update debug mode command so WithStatelessRunHistory can be enabled and executed runs appear on Overview.

Impact of Change

  • Users: Can create Stateless codeful Logic App workflows and explicitly enable persisted run history for local Overview debugging.
  • Developers: Adds explicit codeful workflow-type normalization, template selection, source/debug-mode discovery, and focused tests.
  • System: Generated projects remain stateless by default and use the existing bundled Microsoft.Azure.Workflows.Sdk; run-history persistence stays opt-in through the existing debug-mode setting.

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in: 144 targeted Vitest tests across vs-code-react and vs-code-designer; generated Stateless C# template compiled against the bundled Microsoft.Azure.Workflows.Sdk with 0 warnings and 0 errors; VS Code extension bundle completed successfully. The repository type-check still reports pre-existing errors in unrelated files.

Contributors

Copilot

Screenshots/Videos

N/A - this adds an option to the existing workflow type dropdown and review summary without changing styling or layout.

github-actions Bot and others added 2 commits August 26, 2026 22:22
Auto-generated by Graphify after source code changes.

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 9, 2026 01:51
@lambrianmsft Brian Lam (lambrianmsft) added the risk:medium Medium risk change with potential impact label Sep 9, 2026
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

PR Title

  • Current: feat(vscode): add stateless codeful template
  • Issue: None — valid feat prefix with a vscode scope and a descriptive summary.
  • Recommendation: No change needed.

Commit Type

  • Exactly one type selected (feature - New functionality), consistent with the feat: title prefix.
  • Minor note: the checklist label reads feature while the template/title use feat. Harmless, but aligning to feat keeps things tidy.

Risk Level

  • Exactly one box selected (Medium) and the risk:medium label matches the body. This aligns with the advised estimate: the change ships to the VS Code extension distribution (apps/vs-code-designer) and its webview (apps/vs-code-react) — extension distribution changes are Medium. No shipped-product security/auth or breaking-API impact, so it is not High. Correctly declared.

What & Why

  • Current: Clear explanation of adding the missing Stateless codeful workflow option, C# scaffold, discovery, and debug-mode selection to complete the end-to-end contract.
  • Issue: None.
  • Recommendation: No change needed.

Impact of Change

  • All three audiences (Users, Developers, System) are addressed with meaningful detail.
  • Recommendation:
    • Users: Already covered — can create Stateless codeful apps and opt into persisted run history.
    • Developers: Already covered — normalization, template selection, discovery, tests.
    • System: Already covered — stateless-by-default, opt-in run-history persistence.

Test Plan

  • Unit tests added/updated are confirmed in the diff (new StatelessWorkflowsListStep.test.ts, plus additions across CreateLogicAppWorkspace.test.ts, codeful.test.ts, localCodefulOverviewPanel.test.ts, reviewCreateStep.test.tsx, workflowTypeStep.test.tsx, createWorkspaceSlice.test.ts). Manual testing also noted. Passes.

⚠️ Contributors

  • @Copilot is credited. Consider also tagging any PMs/designers who contributed to the workflow-type UX, if applicable. Never blocks.

⚠️ Screenshots/Videos

  • The diff touches apps/vs-code-react/src UI (workflowTypeStep.tsx, reviewCreateStep.tsx), but this only adds a new selectable option to an existing dropdown/review summary with no styling/layout change, so it's treated as a non-blocking nudge. A quick screenshot of the new "Stateless" option would still be a nice-to-have for reviewers.

Summary Table

Section Status Recommendation
Title No change needed
Commit Type Optionally align label to feat
Risk Level Medium is correct
What & Why No change needed
Impact of Change No change needed
Test Plan Unit tests confirmed
Contributors ⚠️ Credit PMs/designers if any
Screenshots/Videos ⚠️ Optional: add a screenshot of the new option

✅ This PR passes all required checks. Nice work — thorough tests and a well-structured description. The two warnings are optional and do not block merge.


Powered by: Copilot CLI (claude-opus-4.8) | Last updated: Wed, 09 Sep 2026 20:12:10 GMT

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-validated risk:medium Medium risk change with potential impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants