Merge dev: resolve project board automation option IDs conflict (#305) by mpaulosky · Pull Request #306 · mpaulosky/MyBlog

mpaulosky · 2026-05-11T20:56:31Z

Closes #305

Context

Boromir (DevOps) has successfully resolved a merge conflict between local dev and origin/dev. The conflict arose because:

Upstream origin/dev was at commit 7e15cdd (feat: restrict blog post editing [Sprint 19] feat(app): restrict blog post editing to post author or Admin #300)
Local dev had 7 commits ahead, including project board automation fixes with correct option IDs
The upstream version had stale/incorrect project board option IDs

Resolution

✅ Conflict cleanly resolved, keeping the correct board automation option IDs
✅ markdownlint violation in .squad/decisions.md was also fixed during merge
✅ Merge commit verified: b65862e 'Merge dev: resolve project board automation option IDs conflict'
✅ Working tree is clean

Decision

As Aragorn (Lead Developer), authorizing this merge and push to keep dev synchronized with the correct project board automation configuration.

Related Decision

Board automation repair documented in .squad/decisions/decisions.md (lines 2341-2362)
Recent project board option ID fixes: commits 135f9fd, 3fb7406

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

github-actions · 2026-05-11T20:56:54Z

🏗️ PR Added to Squad Triage Queue

This PR has been labeled with squad and added to the triage queue.

Next steps:

The squad Lead will review and assign to an appropriate team member
A squad:member label will be added after triage

If you know which squad member should handle this, you can add the appropriate squad:member label yourself.

…TOKEN (#305) - Use GH_PROJECT_TOKEN instead of GITHUB_TOKEN in project board workflows - Update project board option IDs (IN_SPRINT, IN_REVIEW) to correct values - Add project-board-audit workflow improvements Closes #305 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

This PR merges dev to resolve a conflict around GitHub Project Board automation configuration, while also including a small Blazor edit-page behavior tweak and a markdownlint formatting fix in decisions docs.

Changes:

Blazor: redirect /blog/edit/{id} back to /blog when the post cannot be found, with a new bUnit test covering the behavior.
GitHub Actions: update project board automation workflows to use GH_PROJECT_TOKEN and adjust status option IDs.
Docs: fix a markdownlint spacing issue in .squad/decisions/decisions.md.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tests/Web.Tests.Bunit/Features/EditAclTests.cs	Adds a bUnit test ensuring the Edit page redirects to `/blog` when a post is not found.
src/Web/Features/BlogPosts/Edit/Edit.razor	Redirects to `/blog` when the queried post does not exist.
.squad/decisions/decisions.md	Inserts a blank line to satisfy markdownlint formatting.
.github/workflows/squad-mark-released.yml	Updates workflow env option IDs used for “Done → Released” project transitions.
.github/workflows/project-board-automation.yml	Updates workflow env option IDs for project board transitions.
.github/workflows/project-board-audit.yml	Adds header comments documenting the audit workflow and updated project ID.
.github/workflows/add-issues-to-project.yml	Switches token used by `actions/github-script` to `secrets.GH_PROJECT_TOKEN`.

  PROJECT_ID: PVT_kwHOA5k0b84BXZpa
  STATUS_FIELD_ID: PVTSSF_lAHOA5k0b84BXZpazhSmuGY
  DONE_OPTION_ID: "98236657"
-  RELEASED_OPTION_ID: "8e246b27"
+  RELEASED_OPTION_ID: "98236657"



  PROJECT_ID: PVT_kwHOA5k0b84BXZpa
  STATUS_FIELD_ID: PVTSSF_lAHOA5k0b84BXZpazhSmuGY
-  IN_SPRINT_OPTION_ID: 61e4505c
-  IN_REVIEW_OPTION_ID: df73e18b
+  IN_SPRINT_OPTION_ID: f75ad846
+  IN_REVIEW_OPTION_ID: 47fc9ee4
  DONE_OPTION_ID: "98236657"
-  RELEASED_OPTION_ID: 8e246b27
+  RELEASED_OPTION_ID: 98236657



      - name: Add issue to MyBlog project board
        uses: actions/github-script@v9
        with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
+          github-token: ${{ secrets.GH_PROJECT_TOKEN }}
          script: |
            const issue = context.payload.issue;


github-actions · 2026-05-11T21:06:15Z

Test Results Summary

361 tests ±0 360 ✅ ±0 19s ⏱️ -1s
6 suites ±0 1 💤 ±0
6 files ±0 0 ❌ ±0

Results for commit b352d5f. ± Comparison against base commit 7e15cdd.

♻️ This comment has been updated with latest results.

codecov · 2026-05-11T21:06:41Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.33%. Comparing base (7e15cdd) to head (b352d5f).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files

@@           Coverage Diff           @@
##              dev     #306   +/-   ##
=======================================
  Coverage   83.33%   83.33%           
=======================================
  Files          47       47           
  Lines        1248     1248           
  Branches      152      152           
=======================================
  Hits         1040     1040           
  Misses        148      148           
  Partials       60       60

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

mpaulosky · 2026-05-11T21:48:17Z

Triage Decision: ROUTE TO BOROMIR (Assign `squad:boromir`)

Status: ✅ Ready for Review

Analysis:

✅ Branch: squad/305-sync-board-option-ids (correct naming)
✅ GitHub issue link: Closes Sync: merge dev to resolve project board automation option IDs conflict #305
✅ CI fully passing (21 checks, all SUCCESS)
✅ Codecov gate passing (patch + project SUCCESS)
✅ No merge conflicts
✅ Copilot review: 3 comments, no flagged bugs/security issues

This PR resolves 3 concerns:

Merge conflict resolution (local dev vs origin/dev) — properly resolved, keeping correct board automation option IDs
DevOps/CI fix — switches from GITHUB_TOKEN to GH_PROJECT_TOKEN for project board access, updates stale option IDs
UX bug fix — includes Edit.razor redirect + bUnit test (identified in PR feat(app): restrict blog post editing to post author or Admin (#300) #304 Copilot review)

Assigned to: Boromir (DevOps) for code review focus on CI/workflow logic
Also requires: Aragorn review (architecture, code patterns, test coverage)

Added 'Released' status option to MyBlog project board (ID: 90af7f3b). DONE_OPTION_ID (98236657) remains unchanged. Previously RELEASED_OPTION_ID was set to 98236657, clashing with DONE. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

mpaulosky · 2026-05-11T22:06:55Z

🔧 Released Option Added — Blocker Fixed

Boromir here. I've resolved the blocker that prevented this PR from being correct.

Problem

RELEASED_OPTION_ID was set to 98236657 — the same ID as DONE_OPTION_ID. The Released status option did not exist on the project board at all.

Fix Applied

Added Released option to the MyBlog project board Status field via GraphQL mutation (keyring auth, project scope). New option ID: 90af7f3b.
Updated both workflow files on this branch:
- .github/workflows/project-board-automation.yml: RELEASED_OPTION_ID: 90af7f3b
- .github/workflows/squad-mark-released.yml: RELEASED_OPTION_ID: "90af7f3b"
DONE_OPTION_ID (98236657) is unchanged.

Board State Now

Option	ID	Color
Todo	`f75ad846`	GRAY
In Progress	`47fc9ee4`	YELLOW
Done	`98236657`	GREEN
Released	`90af7f3b`	BLUE

Commit

da163b4 — pushed to this branch after all 6 pre-push gates passed (49 tests, 0 failures).

PR #306 is now unblocked. DONE and RELEASED are distinct IDs — the board automation will correctly move items to Released on release, not back to Done.

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

Copilot AI review requested due to automatic review settings May 11, 2026 20:56

mpaulosky mentioned this pull request May 11, 2026

Sync: merge dev to resolve project board automation option IDs conflict #305

Closed

github-actions Bot added the squad Squad triage inbox — Lead will assign to a member label May 11, 2026

Copilot started reviewing on behalf of mpaulosky May 11, 2026 20:57 View session

Copilot AI reviewed May 11, 2026

View reviewed changes

mpaulosky force-pushed the squad/305-sync-board-option-ids branch from 8cf7981 to 6a384bf Compare May 11, 2026 21:00

mpaulosky mentioned this pull request May 11, 2026

fix(ui): redirect to /blog when post not found in Edit page (#307) #308

Closed

mpaulosky added squad:boromir Assigned to Boromir (DevOps / Infra) and removed squad Squad triage inbox — Lead will assign to a member labels May 11, 2026

docs(squad): log Released option board fix in boromir history

b352d5f

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

mpaulosky merged commit 9fcf7dc into dev May 11, 2026
18 checks passed

mpaulosky deleted the squad/305-sync-board-option-ids branch May 11, 2026 22:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Merge dev: resolve project board automation option IDs conflict (#305)#306

Merge dev: resolve project board automation option IDs conflict (#305)#306
mpaulosky merged 3 commits into
devfrom
squad/305-sync-board-option-ids

mpaulosky commented May 11, 2026

Uh oh!

github-actions Bot commented May 11, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

github-actions Bot commented May 11, 2026 •

edited

Loading

Uh oh!

codecov Bot commented May 11, 2026 •

edited

Loading

Uh oh!

mpaulosky commented May 11, 2026

Uh oh!

mpaulosky commented May 11, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

mpaulosky commented May 11, 2026

Context

Resolution

Decision

Related Decision

Uh oh!

github-actions Bot commented May 11, 2026

🏗️ PR Added to Squad Triage Queue

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

github-actions Bot commented May 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results Summary

Uh oh!

codecov Bot commented May 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

mpaulosky commented May 11, 2026

Triage Decision: ROUTE TO BOROMIR (Assign squad:boromir)

Uh oh!

mpaulosky commented May 11, 2026

🔧 Released Option Added — Blocker Fixed

Problem

Fix Applied

Board State Now

Commit

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions Bot commented May 11, 2026 •

edited

Loading

codecov Bot commented May 11, 2026 •

edited

Loading

Triage Decision: ROUTE TO BOROMIR (Assign `squad:boromir`)