Update dev from main by marc-romu · Pull Request #444 · architects-toolkit/SmartHopper

marc-romu · 2026-05-03T09:27:51Z

No description provided.

…ion of closed milestone releases

…ell output commands

… output variable naming

… in version promotion workflow

…is closed

…init workflow

… stabilization init workflow

…to bypass GitHub security restrictions

…elease workflow

…h pushes

…flow

…rate across workflows

…r multi-branch commit fan-out

…els from aborting cherry-pick workflow

…rkflow

Add a two-layer GitHub Actions design: - Generic Mistral AI Chat Completions API wrapper action - Release-notes-specific action with domain prompts from .windsurf/workflows/release-notes.md Wire into release-3-pr-to-main-closed.yml to generate AI-powered release titles and notes, with GitHub auto-generated notes preserved in a collapsed details block.

…s, and other CI improvements - Add concurrency controls to prevent race conditions across 5 workflows - Paginate listReleases in release-promotion.yml using github.paginate() - Add auto-discover, include-main-branches, exclude-branches inputs to patch-propagate.yml - Add summary/notification job to patch-propagate.yml for failed cherry-picks - Add promotion: freeze label check in release-promotion.yml - Auto-create promotion-blocked issue when release >30 days but blocked - Add version-format validation before branch cleanup in stabilization-2-complete.yml - Dereference annotated tags in stabilization-0-init.yml - Post-filter issue search results in check-issues-for-version/action.yml - Target stabilization branch for hash PR in release-4-build.yml - Create required labels (automation, needs-attention, promotion: freeze/blocked, has-conflicts) - Update RELEASE_WORKFLOW.md documentation

Hash PRs must target main for GitHub Pages deployment.

Added guideline for new components to use a zeroed GUID.

Updated guidelines on unit tests related to Rhino/Grasshopper references.

Copilot

Pull request overview

This PR is a broad workflow-and-guidance sync from main into dev, mainly updating GitHub Actions release/stabilization automation and refreshing Windsurf rules/docs to match the current SmartHopper development process.

Changes:

Adds a new stabilization/patch-propagation release flow across GitHub Actions, plus related supporting actions.
Refactors release automation around manifest updates, release-note generation, Yak upload triggering, and branch/milestone handling.
Refreshes internal docs/rules and minor metadata files to document the new workflows and conventions.

Reviewed changes

Copilot reviewed 62 out of 62 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
`src/SmartHopper.Infrastructure/SmartHopper.Infrastructure.csproj`	Removes BOM/header formatting difference.
`hashes/1.4.2-beta.json`	Adds generated provider hash manifest for 1.4.2-beta.
`docs/index.md`	Adds Development docs links.
`docs/Development/patch-propagation.md`	Documents new multi-branch patch propagation workflow.
`CHANGELOG.md`	Adds unreleased note about rules/workflows clarification.
`.windsurf/workflows/pr-description.md`	Fixes wording/formatting in PR description workflow doc.
`.windsurf/workflows/new-branch.md`	Clarifies branch creation workflow text.
`.windsurf/workflows/fix-compilation-messages.md`	Refines compiler-fix workflow wording.
`.windsurf/workflows/docs-update.md`	Updates docs workflow scope and output paths.
`.windsurf/workflows/commit-message.md`	Clarifies Conventional Commit guidance.
`.windsurf/workflows/code-style.md`	Rewrites style-cleanup instructions with clearer rules.
`.windsurf/workflows/changelog-review.md`	Clarifies changelog review wording.
`.windsurf/rules/windsurf-rules.md`	Updates how Windsurf rule files should be edited.
`.windsurf/rules/webchat-cdn.md`	Removes old CDN-specific rule file.
`.windsurf/rules/webchat-architecture.md`	Adds consolidated WebChat architecture rule.
`.windsurf/rules/tool-result-envelope.md`	Adds tool JSON envelope rule.
`.windsurf/rules/testing-and-build.md`	Adds build/test expectations for platform constraints.
`.windsurf/rules/target-platform-and-frameworks.md`	Clarifies target TFMs/platforms.
`.windsurf/rules/solution-structure.md`	Expands solution structure guidance.
`.windsurf/rules/refactor-stop-and-confirm.md`	Clarifies stop-and-confirm refactor policy.
`.windsurf/rules/provider-authentication.md`	Adds provider auth/secrets handling rule.
`.windsurf/rules/infrastructure-aitools.md`	Refreshes AI tools infrastructure guidance.
`.windsurf/rules/infrastructure-aiprovider.md`	Refreshes provider architecture guidance.
`.windsurf/rules/infrastructure-aicall.md`	Rewrites AICall architecture overview.
`.windsurf/rules/general-guidelines.md`	Updates general SmartHopper development guidance.
`.windsurf/rules/documentation-and-changelog.md`	Clarifies docs/changelog expectations.
`.windsurf/rules/data-tree-processing.md`	Adds Grasshopper data-tree processing rule.
`.windsurf/rules/component-conventions.md`	Expands component conventions.
`.windsurf/rules/code-dedup-architecture.md`	Minor wording cleanup.
`.windsurf/rules/architecture-thinking.md`	Clarifies architecture-first workflow guidance.
`.windsurf/rules/ai-tool-conventions.md`	Expands AI tool conventions.
`.windsurf/rules/ai-provider-conventions.md`	Expands AI provider conventions.
`.github/workflows/stabilization-2-complete.yml`	Adds stabilization completion/backport workflow.
`.github/workflows/stabilization-1-cancel.yml`	Adds stabilization cancellation workflow.
`.github/workflows/stabilization-0-init.yml`	Adds stabilization branch initialization workflow.
`.github/workflows/release-promotion.yml`	Reworks staged-release promotion logic around stabilization milestones.
`.github/workflows/release-auto-upload-yak.yml`	Removes old standalone Yak auto-upload workflow.
`.github/workflows/release-5-deploy-pages.yml`	Switches Pages deploy trigger to hash pushes on `main`.
`.github/workflows/release-4-build.yml`	Adjusts build checkout/PR assignment and moves Yak trigger here.
`.github/workflows/release-3-pr-to-main-closed.yml`	Extends release creation to `main-*` and AI-generated release notes.
`.github/workflows/release-2-pr-to-dev-closed.yml`	Extends dev→main PR automation to stabilization branches.
`.github/workflows/release-1-milestone.yml`	Makes release-prep target stabilization branches and update manifest text.
`.github/workflows/pr-version-validation.yml`	Extends PR version validation to `main-`/`dev-`.
`.github/workflows/pr-validation.yml`	Extends general PR validation to stabilization branches.
`.github/workflows/pr-manifest-validation.yml`	Extends manifest validation and reuses shared manifest-text action.
`.github/workflows/pr-delete-auto-branches.yml`	Adds `patch/` branches to auto-delete list.
`.github/workflows/pr-build-hash-validation.yml`	Extends build/hash validation to stabilization branches.
`.github/workflows/patch-propagate.yml`	Adds manual multi-branch cherry-pick propagation workflow.
`.github/workflows/dev-update-manifest.yml`	Switches dev manifest updates to shared manifest-text action.
`.github/workflows/ci-dotnet-tests.yml`	Extends CI to stabilization branches.
`.github/workflows/RELEASE_WORKFLOW.md`	Documents new stabilization and patch-propagation flows.
`.github/actions/versioning/update-manifest-text/action.yml`	Adds reusable manifest note text update action.
`.github/actions/versioning/promote-version/action.yml`	Removes old promote-version composite action.
`.github/actions/versioning/move-milestone-items/action.yml`	Makes milestone-item moves stabilization-aware.
`.github/actions/versioning/manage-milestones/action.yml`	Simplifies milestone management behavior.
`.github/actions/versioning/check-issues-for-version/action.yml`	Tightens version-label matching for promotion checks.
`.github/actions/versioning/README.md`	Updates versioning action docs for new behavior.
`.github/actions/milestone/assign-pr/action.yml`	Adds optional explicit version input for milestone assignment.
`.github/actions/documentation/generate-release-notes/action.yml`	Adds AI-powered release-note generation action.
`.github/actions/cherry-pick-to-branch/action.yml`	Adds reusable cherry-pick-and-open-PR action.
`.github/actions/cherry-pick-to-branch/PR_TEMPLATE.md`	Adds PR template for propagated patches.
`.github/actions/ai/mistral-chat/action.yml`	Adds generic Mistral chat wrapper action.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

          git config --local user.name "github-actions[bot]"
          git add yak-package/manifest.yml
-          git commit -m "chore: update manifest text to ${{ steps.manifest_text.outputs.type }} version [skip ci]"
+          git commit -m "chore: update manifest text to ${{ steps.manifest_text.outputs.release-type }} version [skip ci]"


      with:
        pr-number: ${{ steps.hash-pr.outputs.hash_pr_number }}
        token: ${{ secrets.GITHUB_TOKEN }}
+        version: ${{ needs.prep.outputs.version }}


+          json=$(echo "$raw" \
+            | tr ',' '\n' \
+            | sed 's/^[[:space:]]*//; s/[[:space:]]*$//' \
+            | grep -v '^$' \


+            if (propagateResult === 'failure') {
+              // Create issue for manual attention
+              await github.rest.issues.create({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                title: `\ud83c\udf52 Patch propagate needs attention (run #${context.runNumber})`,
+                body: `One or more cherry-picks had conflicts or failures.\n\n` +
+                      `**Run:** [#${context.runNumber}](${runUrl})\n` +
+                      `**SHAs:** \`${shas}\`\n` +
+                      `**Source branch:** \`${{ inputs.source-branch }}\`\n\n` +
+                      `Please review the workflow run and resolve any draft PRs with conflict markers.`,


+                      `Please review the workflow run and resolve any draft PRs with conflict markers.`,
+                labels: ['automation', 'needs-attention']


+              // For each stabilization path, find the current staged release
+              // Priority: rc > beta > alpha (promote the most advanced stage that exists)
+              const stagePriority = ['rc', 'beta', 'alpha'];
+
+              for (const milestone of stabilizationMilestones) {
+                const base = milestone.title; // e.g. "1.4.2"
+                let currentStaged = null;
+
+                for (const stage of stagePriority) {
+                  const candidate = `${base}-${stage}`;
+                  if (releaseTags.has(candidate)) {
+                    currentStaged = candidate;
+                    console.log(`Stabilization path ${base}: current staged release is ${candidate}`);
+                    break;
+                  }
+                }
+
+                if (currentStaged) {
+                  versionsToCheck.push(currentStaged);


+            const frozen = issues.some(i =>
+              i.labels.some(l => l.name.startsWith(`version: ${baseVersion}`))
+            );


+                      `- This issue will NOT be auto-closed \u2014 close it manually once resolved`,
+                labels: ['automation', 'promotion: blocked']


…kflow

marc-romu and others added 30 commits March 24, 2026 23:05

ci: remove milestone check from promotion eligibility to allow promot…

b398f3e

…ion of closed milestone releases

ci: fix version grouping to use minor version and standardize PowerSh…

b6838c8

…ell output commands

ci: skip promotion for versions with closed target milestones and fix…

ae28638

… output variable naming

ci: standardize PowerShell output commands and improve PR URL parsing…

eb59bb5

… in version promotion workflow

ci: prevent older versions from being promoted when target milestone …

98fda0e

…is closed

ci: new stabilization-aware milestone workflow

9550ead

ci: add workflows write permission to stabilization init workflow

4c61ce1

ci: remove unnecessary workflows write permission from stabilization …

f37899d

…init workflow

ci: replace git commands with GitHub API calls for branch creation in…

75e4a6d

… stabilization init workflow

ci: use PAT_TOKEN for branch creation in stabilization init workflow …

d93232d

…to bypass GitHub security restrictions

chore: add provider hash manifest for version 1.4.2-beta (dual platform)

4bad881

ci: remove automatic branch deletion from hash manifest PR merge in r…

b02687f

…elease workflow

ci: simplify GitHub Pages deployment trigger to run on all main branc…

f7cc8da

…h pushes

ci: add validation to prevent editing stable releases in release work…

fb76c62

…flow

ci: checkout specific version tag in build jobs for release workflow

faca31d

chore: add provider hash manifest for version 1.4.2-beta (dual platform)

95b0e45

ci: add optional version input parameter to milestone assignment action

71fd802

Merge branch 'main' of https://github.com/architects-toolkit/SmartHopper

d1e38e5

ci: extract manifest text update logic into reusable action and integ…

306289b

…rate across workflows

ci: add reusable cherry-pick action and patch propagation workflow fo…

4fda03e

…r multi-branch commit fan-out

ci: apply PR labels best-effort after creation to prevent unknown lab…

061a4f7

…els from aborting cherry-pick workflow

ci: apply PR labels best-effort after creation to prevent unknown lab…

201f4b0

…els from aborting cherry-pick workflow

Merge branch 'main' of https://github.com/architects-toolkit/SmartHopper

83c39a5

build: replace GhJSON project references with NuGet package references

bd7e3c8

ci: add patch/ prefix to auto-delete branch patterns in PR cleanup wo…

77b6a5e

…rkflow

ci: revert hash PR target branch to always use main

84658a4

Hash PRs must target main for GitHub Pages deployment.

docs(rules): clarify Windsurf guidance

7a9d5bd

Specify zeroed GUID for new components

d9759e0

Added guideline for new components to use a zeroed GUID.

devin-ai-integration Bot and others added 4 commits May 3, 2026 09:26

docs(rules): clarify component GUID placeholder

a897928

Revise unit test guidelines for Rhino/Grasshopper

2b9a86f

Updated guidelines on unit tests related to Rhino/Grasshopper references.

docs(rules): align testing guidance

9e82b63

docs(rules): align tool envelope guidance

08f4f03

Copilot AI review requested due to automatic review settings May 3, 2026 09:27

Copilot started reviewing on behalf of marc-romu May 3, 2026 09:28 View session

Copilot AI reviewed May 3, 2026

View reviewed changes

ci: remove redundant JSON output from PR creation in contributors wor…

2456728

…kflow

marc-romu merged commit 11463cd into dev May 3, 2026
13 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update dev from main#444

Update dev from main#444
marc-romu merged 35 commits into
devfrom
main

marc-romu commented May 3, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		`Please review the workflow run and resolve any draft PRs with conflict markers.`,
		labels: ['automation', 'needs-attention']

		`- This issue will NOT be auto-closed \u2014 close it manually once resolved`,
		labels: ['automation', 'promotion: blocked']

Conversation

marc-romu commented May 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants