chore: Update MCP server configurations and squad export - #246
Conversation
…tories, archive aged decisions - Merged three inbox decision entries (Aragorn/Boromir/Frodo) into decisions.md - Updated agent history files with 2026-04-12 team sync learnings - Archived pre-April-1 decisions to decisions-archive.md for operational clarity - Released context by condensing aged decision records Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Session: 2026-04-12T19:57:04Z Tasks completed: 1. Orchestration log: frodo agent release-process deprecation task 2. Session log: release-skill-legacy-stub work summary 3. Decision inbox merged: frodo-release-process-legacy-stub.md → decisions.md 4. Frodo history.md updated: Added legacy stub deprecation entry 5. Inbox file deleted after merge Outputs: - .squad/orchestration-log/2026-04-12T19-57-04Z-frodo.md (new) - .squad/log/2026-04-12T19-57-04Z-release-skill-legacy-stub.md (new) - .squad/decisions.md (appended, 92896 bytes) - .squad/agents/frodo/history.md (updated with team context) No archival needed (92896 bytes < 20KB threshold would trigger archive, but this represents 2026-04 additions; all old decisions within 30d window). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Task 7: Summarized old entries (>30d) to '## Core Context' section. Files: - aragorn: 29KB → 21KB (kept March 28+ recent entries) - frodo: 16KB → 6KB (kept April 12+ recent entries) - legolas: 14KB → 4KB (consolidated CSS, styling, theme learnings) - pippin: 12KB → 4KB (consolidated Aspire, Playwright, team rule learnings) All summarized files now <12KB threshold except Aragorn (21KB still acceptable with Core Context structure). Maintains full context while improving readability. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Update .copilot/mcp-config.json with enhanced MCP server definitions - Add .mcp.json for MCP client integration (untracked) - Update squad-export.json with latest team metadata Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR updates squad governance artifacts and MCP configuration, including a new generic release-process skill and an IssueTrackerApp-specific release playbook, plus refreshed squad export and decision logs.
Changes:
- Added new release workflow guidance: a generic
release-process-baseskill, an IssueTrackerApp release playbook, and deprecated the legacyrelease-processskill. - Updated MCP server configurations in both
.mcp.jsonand.copilot/mcp-config.json. - Refreshed squad export and reorganized/updated squad decision and agent history artifacts.
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| squad-export.json | Refreshes exported timestamp and updates embedded agent charter/history payloads. |
| .squad/skills/release-process/SKILL.md | Adds a deprecation stub pointing to the new base skill and project playbook. |
| .squad/skills/release-process-base/SKILL.md | Introduces a generic, reusable release-process skill template. |
| .squad/playbooks/release-issuetracker.md | Adds an IssueTrackerApp-specific release playbook. |
| .squad/decisions.md | Updates decision log content with new release-process portability decisions and related notes. |
| .squad/decisions-archive.md | Moves/expands archived decision content (but currently has duplicated headers/intro). |
| .squad/agents/aragorn/history.md | Summarizes/restructures Aragorn’s history and adds release-process abstraction notes. |
| .squad/agents/boromir/history.md | Adds release-process discovery/inference notes and consolidates prior entries. |
| .squad/agents/frodo/history.md | Restructures Frodo history and adds release-process documentation work. |
| .squad/agents/legolas/history.md | Condenses and reorganizes Legolas history with styling and UI learnings. |
| .squad/agents/pippin/history.md | Summarizes historical learnings into a condensed “Core Context” section. |
| .squad/.ralph-state.json | Adds Ralph state tracking JSON (health check timestamp and placeholders). |
| .mcp.json | Adds/updates repository-level MCP server configuration entries. |
| .copilot/mcp-config.json | Aligns Copilot MCP config to the same MCP server set as .mcp.json. |
| | **Version System** | NBGV | Nerdbank.GitVersioning | | ||
| | **Version File** | `version.json` | At repo root | | ||
| | **Tag Prefix** | `v` | e.g., `v1.0.0` | | ||
| | **Package ID** | IssueTrackerApp | From `.csproj` | | ||
| | **Merge Strategy** | merge | Preserve commit history on main | | ||
|
|
||
| **version.json reference:** | ||
| ```json | ||
| { | ||
| "version": "1.0.0", | ||
| "publicReleaseRefSpec": [ | ||
| "^refs/heads/main$", | ||
| "^refs/tags/v\\d+(?:\\.\\d+)?$" | ||
| ] | ||
| } | ||
| ``` | ||
|
|
There was a problem hiding this comment.
This playbook claims IssueTrackerApp uses NBGV with a root version.json, but the repo has GitVersion.yml/global.json and no version.json at the root. Update the playbook to match the actual GitVersion-based release/versioning approach (or add the missing version.json/NBGV if that’s the intended migration).
| | **Version System** | NBGV | Nerdbank.GitVersioning | | |
| | **Version File** | `version.json` | At repo root | | |
| | **Tag Prefix** | `v` | e.g., `v1.0.0` | | |
| | **Package ID** | IssueTrackerApp | From `.csproj` | | |
| | **Merge Strategy** | merge | Preserve commit history on main | | |
| **version.json reference:** | |
| ```json | |
| { | |
| "version": "1.0.0", | |
| "publicReleaseRefSpec": [ | |
| "^refs/heads/main$", | |
| "^refs/tags/v\\d+(?:\\.\\d+)?$" | |
| ] | |
| } | |
| ``` | |
| | **Version System** | GitVersion | Repository uses GitVersion-based version calculation | | |
| | **Version File** | `GitVersion.yml` | Primary versioning configuration at repo root | | |
| | **SDK Pinning** | `global.json` | Pins the .NET SDK used by builds/releases | | |
| | **Tag Prefix** | `v` | e.g., `v1.0.0`, if tags are used for releases | | |
| | **Package ID** | IssueTrackerApp | From `.csproj` | | |
| | **Merge Strategy** | merge | Preserve commit history on main | | |
| **Versioning reference:** | |
| - Treat `GitVersion.yml` as the source of truth for branch/version rules. | |
| - Do not rely on a root `version.json` unless the repository is explicitly | |
| migrated to Nerdbank.GitVersioning and that file is added. |
| | **Build Verification** | release published | `.github/workflows/build.yml` | (logs only) | | ||
| | **Unit Tests** | release published | `.github/workflows/build.yml` | (logs only) | | ||
| | **Integration Tests** | release published | `.github/workflows/integration-tests.yml` | (logs only) | | ||
| | **Docker Image** | TBD | (not yet configured) | (not yet deployed) | | ||
| | **Documentation** | TBD | (not yet configured) | (not yet deployed) | | ||
| | **NuGet Package** | TBD | (not yet configured) | (not yet deployed) | | ||
|
|
||
| **Status:** Minimal release pipeline. Extend as needed. |
There was a problem hiding this comment.
The playbook references .github/workflows/build.yml and integration-tests.yml triggered by “release published”, but those workflows don’t exist in this repo (current release/test workflows are squad-*.yml, and release is tag-driven). Update workflow filenames and triggers to reflect the current CI/CD setup so the playbook is executable.
| | **Build Verification** | release published | `.github/workflows/build.yml` | (logs only) | | |
| | **Unit Tests** | release published | `.github/workflows/build.yml` | (logs only) | | |
| | **Integration Tests** | release published | `.github/workflows/integration-tests.yml` | (logs only) | | |
| | **Docker Image** | TBD | (not yet configured) | (not yet deployed) | | |
| | **Documentation** | TBD | (not yet configured) | (not yet deployed) | | |
| | **NuGet Package** | TBD | (not yet configured) | (not yet deployed) | | |
| **Status:** Minimal release pipeline. Extend as needed. | |
| | **Build Verification** | push/tag `v*` | `.github/workflows/squad-ci.yml` | (logs only) | | |
| | **Unit Tests** | push/tag `v*` | `.github/workflows/squad-ci.yml` | (logs only) | | |
| | **Integration Tests** | push/tag `v*` | `.github/workflows/squad-integration-tests.yml` | (logs only) | | |
| | **Docker Image** | TBD | (not yet configured) | (not yet deployed) | | |
| | **Documentation** | TBD | (not yet configured) | (not yet deployed) | | |
| | **NuGet Package** | TBD | (not yet configured) | (not yet deployed) | | |
| **Status:** Minimal tag-driven release pipeline. Extend as needed. |
| # IssueTrackerApp Decisions | ||
|
|
||
| Historical decisions archived from before 2026-02-01. See decisions.md for current decisions. | ||
| This file records team decisions that affect architecture, scope, and process. | ||
|
|
||
| --- | ||
|
|
||
| ## Archived Decisions | ||
| ## Decisions | ||
|
|
||
| ### MongoDB Connection String Fallback (2025-03-21) | ||
| ### Process & Planning | ||
| # IssueTrackerApp Decisions | ||
|
|
||
| **Author:** Sam (Backend Developer) | ||
| **Status:** Implemented | ||
| This file records team decisions that affect architecture, scope, and process. | ||
|
|
||
| The Web project crashed at startup with `System.TimeoutException` because the EF Core MongoDB provider reads `MongoDB:ConnectionString` (hardcoded to `mongodb://localhost:27017` in appsettings.Development.json), while the actual Atlas connection string lives in `ConnectionStrings:mongodb` (user secrets / Aspire injection). These two config paths never intersect. | ||
| --- | ||
|
|
||
| **Decision:** Added fallback logic in `AddMongoDbPersistence` that bridges the gap: | ||
| ## Decisions | ||
|
|
||
| 1. Before binding `MongoDbSettings`, check if `MongoDB:ConnectionString` is empty or equals `mongodb://localhost:27017` | ||
| 2. If so, read `ConnectionStrings:mongodb` and overlay it into the MongoDB config section | ||
| 3. Changed `appsettings.Development.json` to use empty string instead of the localhost default | ||
| ### Process & Planning |
There was a problem hiding this comment.
The file has duplicated top-level headers/intro content (# IssueTrackerApp Decisions appears twice and sections repeat). This looks like an accidental paste during the archive move; please collapse to a single header and ensure the archive’s intro accurately describes its purpose (archived decisions vs current decisions).
| "mongodb": { | ||
| "type": "stdio", | ||
| "command": "npx", | ||
| "args": [ | ||
| "-y", | ||
| "mongodb-mcp-server@latest" | ||
| ], |
There was a problem hiding this comment.
Several MCP servers are invoked via npx ...@latest, which makes the toolchain non-reproducible and increases supply-chain risk (each run may fetch different code). Consider pinning explicit package versions and updating them intentionally.
| "microsoft/playwright-mcp": { | ||
| "type": "stdio", | ||
| "command": "npx", | ||
| "args": [ | ||
| "@playwright/mcp@latest" | ||
| ], |
There was a problem hiding this comment.
npx is run without -y for some servers (e.g., Playwright MCP), while others include -y. Without -y, npm may prompt interactively to install packages, which can break non-interactive usage. Consider adding -y consistently for all npx-based servers.
| "mongodb": { | ||
| "type": "stdio", | ||
| "command": "npx", | ||
| "args": [ | ||
| "-y", | ||
| "mongodb-mcp-server" | ||
| "mongodb-mcp-server@latest" | ||
| ], |
There was a problem hiding this comment.
Several MCP servers are invoked via npx ...@latest, which makes the toolchain non-reproducible and increases supply-chain risk (each run may fetch different code). Consider pinning explicit package versions and updating them intentionally.
| "microsoft/playwright-mcp": { | ||
| "type": "stdio", | ||
| "command": "npx", | ||
| "args": [ | ||
| "-y", | ||
| "@playwright/mcp@latest" | ||
| ] | ||
| ], |
There was a problem hiding this comment.
npx is run without -y for some servers (e.g., Playwright MCP), while others include -y. Without -y, npm may prompt interactively to install packages, which can break non-interactive usage. Consider adding -y consistently for all npx-based servers.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #246 +/- ##
=======================================
Coverage 78.52% 78.52%
=======================================
Files 228 228
Lines 8481 8481
Branches 1166 1166
=======================================
Hits 6660 6660
Misses 1284 1284
Partials 537 537 🚀 New features to boost your workflow:
|
- Added orchestration log for Boromir PR creation/merge attempt (PR #246) - Added session log for PR workflow (squad/scribe-log-mcp-export) - Merged 3 decision inbox files into decisions.md: * Boromir: Branch strategy (dev/main model) feasibility audit * Boromir: MCP config commit safety assessment * Frodo: Documentation audit for dev/main branch model - Cleared .squad/decisions/inbox/ (all inbox files processed) This completes the spawn manifest processing for Boromir's two-pass PR flow attempt. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Added orchestration log for Boromir PR creation/merge attempt (PR #246) - Added session log for PR workflow (squad/scribe-log-mcp-export) - Merged 3 decision inbox files into decisions.md: * Boromir: Branch strategy (dev/main model) feasibility audit * Boromir: MCP config commit safety assessment * Frodo: Documentation audit for dev/main branch model - Cleared .squad/decisions/inbox/ (all inbox files processed) This completes the spawn manifest processing for Boromir's two-pass PR flow attempt. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* scribe: Log PR flow and merge decisions inbox - Added orchestration log for Boromir PR creation/merge attempt (PR #246) - Added session log for PR workflow (squad/scribe-log-mcp-export) - Merged 3 decision inbox files into decisions.md: * Boromir: Branch strategy (dev/main model) feasibility audit * Boromir: MCP config commit safety assessment * Frodo: Documentation audit for dev/main branch model - Cleared .squad/decisions/inbox/ (all inbox files processed) This completes the spawn manifest processing for Boromir's two-pass PR flow attempt. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * scribe: Add Aragorn adoption decision to decisions.md - Merged Aragorn's comprehensive two-branch strategy assessment - Includes GitVersion.yml configuration recommendations - Provides phased implementation roadmap (Phase 1 infrastructure, Phase 2 docs) - Clearcut approval gate: requires Matthew Paulosky sign-off - All decision inbox files now processed and cleared Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* chore: Update MCP server configurations and squad export (#246) * chore: merge release-process skill review decisions, update agent histories, archive aged decisions - Merged three inbox decision entries (Aragorn/Boromir/Frodo) into decisions.md - Updated agent history files with 2026-04-12 team sync learnings - Archived pre-April-1 decisions to decisions-archive.md for operational clarity - Released context by condensing aged decision records Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * .squad: Scribe logs for Frodo release-process legacy stub Session: 2026-04-12T19:57:04Z Tasks completed: 1. Orchestration log: frodo agent release-process deprecation task 2. Session log: release-skill-legacy-stub work summary 3. Decision inbox merged: frodo-release-process-legacy-stub.md → decisions.md 4. Frodo history.md updated: Added legacy stub deprecation entry 5. Inbox file deleted after merge Outputs: - .squad/orchestration-log/2026-04-12T19-57-04Z-frodo.md (new) - .squad/log/2026-04-12T19-57-04Z-release-skill-legacy-stub.md (new) - .squad/decisions.md (appended, 92896 bytes) - .squad/agents/frodo/history.md (updated with team context) No archival needed (92896 bytes < 20KB threshold would trigger archive, but this represents 2026-04 additions; all old decisions within 30d window). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * .squad: History file summarization (aragorn, frodo, legolas, pippin) Task 7: Summarized old entries (>30d) to '## Core Context' section. Files: - aragorn: 29KB → 21KB (kept March 28+ recent entries) - frodo: 16KB → 6KB (kept April 12+ recent entries) - legolas: 14KB → 4KB (consolidated CSS, styling, theme learnings) - pippin: 12KB → 4KB (consolidated Aspire, Playwright, team rule learnings) All summarized files now <12KB threshold except Aragorn (21KB still acceptable with Core Context structure). Maintains full context while improving readability. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update MCP server configurations and squad export - Update .copilot/mcp-config.json with enhanced MCP server definitions - Add .mcp.json for MCP client integration (untracked) - Update squad-export.json with latest team metadata Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: update frodo agent history --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: scribe log and decisions export (#247) * scribe: Log PR flow and merge decisions inbox - Added orchestration log for Boromir PR creation/merge attempt (PR #246) - Added session log for PR workflow (squad/scribe-log-mcp-export) - Merged 3 decision inbox files into decisions.md: * Boromir: Branch strategy (dev/main model) feasibility audit * Boromir: MCP config commit safety assessment * Frodo: Documentation audit for dev/main branch model - Cleared .squad/decisions/inbox/ (all inbox files processed) This completes the spawn manifest processing for Boromir's two-pass PR flow attempt. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * scribe: Add Aragorn adoption decision to decisions.md - Merged Aragorn's comprehensive two-branch strategy assessment - Includes GitVersion.yml configuration recommendations - Provides phased implementation roadmap (Phase 1 infrastructure, Phase 2 docs) - Clearcut approval gate: requires Matthew Paulosky sign-off - All decision inbox files now processed and cleared Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* chore: Sync dev with main (2 missing commits) (#248) Sync dev with main: brings in commits b529234 (#246) and 1c3b613 (#247). Establishes clean baseline for dev/main branching model. * chore: Sync dev with main (2 missing commits — #246, #247) (#249) * chore: Update MCP server configurations and squad export (#246) * chore: merge release-process skill review decisions, update agent histories, archive aged decisions - Merged three inbox decision entries (Aragorn/Boromir/Frodo) into decisions.md - Updated agent history files with 2026-04-12 team sync learnings - Archived pre-April-1 decisions to decisions-archive.md for operational clarity - Released context by condensing aged decision records Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * .squad: Scribe logs for Frodo release-process legacy stub Session: 2026-04-12T19:57:04Z Tasks completed: 1. Orchestration log: frodo agent release-process deprecation task 2. Session log: release-skill-legacy-stub work summary 3. Decision inbox merged: frodo-release-process-legacy-stub.md → decisions.md 4. Frodo history.md updated: Added legacy stub deprecation entry 5. Inbox file deleted after merge Outputs: - .squad/orchestration-log/2026-04-12T19-57-04Z-frodo.md (new) - .squad/log/2026-04-12T19-57-04Z-release-skill-legacy-stub.md (new) - .squad/decisions.md (appended, 92896 bytes) - .squad/agents/frodo/history.md (updated with team context) No archival needed (92896 bytes < 20KB threshold would trigger archive, but this represents 2026-04 additions; all old decisions within 30d window). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * .squad: History file summarization (aragorn, frodo, legolas, pippin) Task 7: Summarized old entries (>30d) to '## Core Context' section. Files: - aragorn: 29KB → 21KB (kept March 28+ recent entries) - frodo: 16KB → 6KB (kept April 12+ recent entries) - legolas: 14KB → 4KB (consolidated CSS, styling, theme learnings) - pippin: 12KB → 4KB (consolidated Aspire, Playwright, team rule learnings) All summarized files now <12KB threshold except Aragorn (21KB still acceptable with Core Context structure). Maintains full context while improving readability. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update MCP server configurations and squad export - Update .copilot/mcp-config.json with enhanced MCP server definitions - Add .mcp.json for MCP client integration (untracked) - Update squad-export.json with latest team metadata Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: update frodo agent history --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: scribe log and decisions export (#247) * scribe: Log PR flow and merge decisions inbox - Added orchestration log for Boromir PR creation/merge attempt (PR #246) - Added session log for PR workflow (squad/scribe-log-mcp-export) - Merged 3 decision inbox files into decisions.md: * Boromir: Branch strategy (dev/main model) feasibility audit * Boromir: MCP config commit safety assessment * Frodo: Documentation audit for dev/main branch model - Cleared .squad/decisions/inbox/ (all inbox files processed) This completes the spawn manifest processing for Boromir's two-pass PR flow attempt. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * scribe: Add Aragorn adoption decision to decisions.md - Merged Aragorn's comprehensive two-branch strategy assessment - Includes GitVersion.yml configuration recommendations - Provides phased implementation roadmap (Phase 1 infrastructure, Phase 2 docs) - Clearcut approval gate: requires Matthew Paulosky sign-off - All decision inbox files now processed and cleared Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: Update MCP server configurations and squad export (#246) (#250) Adopts dev/main two-branch model across all infrastructure (workflows, dependabot, pre-push, GitVersion, docs, skills). Replaces squad-promote from Node.js to .NET/GitVersion. * Add pre-push and merge playbooks, fix process documentation (#251) Add pre-push and merge playbooks, fix process documentation - New .squad/playbooks/pre-push-process.md (5-gate walkthrough) - New .squad/playbooks/pr-merge-process.md (8-step merge lifecycle) - Fix release-issuetracker.md (NBGV→GitVersion, two-branch model) - Rewrite pre-push-test-gate skill (all 5 gates, 10 test projects) - Flag 3 Squad CLI skills as non-applicable to IssueTrackerApp - Update routing.md with playbook-aware routing - Add playbook cross-references to ceremonies.md Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>" * chore: disable prerelease SDK in global.json (#254) Set allowPrerelease to false to use stable .NET SDK only. * fix: upgrade gittools/actions to v4.5.0 for GitVersion 6.7.0 compat v3.1.11 and v3 tags only support GitVersion <6.1.0, but 6.x now resolves to 6.7.0 which is out of range. Upgrade all four release workflows to gittools/actions@v4.5.0. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: upgrade gittools/actions to v4.5.0 for GitVersion 6.7.0 compat (#255) v3.1.11 and v3 tags only support GitVersion <6.1.0, but 6.x now resolves to 6.7.0 which is out of range. Upgrade all four release workflows to gittools/actions@v4.5.0. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: address 10 Copilot review items on PR #257 - Add configFilePath: GitVersion.yml to squad-promote.yml execute step - Fix shell-escaping risk: use --body-file instead of inline --body with command substitution - Replace stale embedded GitVersion YAML with link to actual file - Update Last Updated dates to 2026-04-13 - Fix Gate 0 docs: blocks main AND dev, not just main - Fix branch cleanup: reference origin/dev not origin/main (squad PRs merge to dev) - Fix conflict resolution: merge origin/dev not origin/main - Clarify tag prefix: lowercase v only triggers release workflow Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Summary
Updates to MCP server configurations and squad export artifacts.
Closes squad/scribe-log-mcp-export