Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 26, 2026

The cache-memory tool in .github/workflows/shared/mcp/chroma.md was using array format without the required key property, causing validation errors in workflows importing this shared configuration.

Changes

  • Added missing key property to cache-memory array configuration in shared/mcp/chroma.md
# Before (invalid - missing required key property)
tools:
  cache-memory:
    - id: chroma
      description: Persistent storage for Chroma vector database

# After (valid - includes both id and key)
tools:
  cache-memory:
    - id: chroma
      key: memory-chroma-${{ github.workflow }}
      description: Persistent storage for Chroma vector database

Context

Per the cache-memory schema, array format requires both id and key properties. The missing key caused validation warnings in any workflow importing shared/mcp/chroma.md.

All 140 workflows now compile with 0 errors. MCP Inspector and Research workflow runtime failures are execution environment issues, not compilation problems.

Original prompt

This section details on the original issue you should resolve

<issue_title>Workflow Health Dashboard - 2026-01-26</issue_title>
<issue_description>### Overview

  • Total workflows: 140 executable workflows
  • Shared imports: 59 reusable workflow components
  • Compilation coverage: 140/140 (100% ✅)
  • Outdated lock files: 9 workflows need recompilation
  • Healthy: ~137 (98%)
  • Critical: 2 (1%) - MCP Inspector, Research
  • Overall health score: 91/100 (→ stable from 91/100)

Critical Issues 🚨

MCP Inspector - Failing (P1) - Issue #11721

Research Workflow - Failing (P1) - Issue #11722

Recovered Workflows ✅

Daily News - RECOVERY SUSTAINED! (P0 → Healthy)

  • Score: 80/100 (→ stable from 80/100)
  • Status: RECOVERY SUSTAINED - 2/5 recent successes (40% success rate)
  • Latest success: §21280868153 (2026-01-23)
  • Previous issue: Missing TAVILY_API_KEY secret
  • Resolution: Secret added on 2026-01-22
  • Monitoring: ✅ Recovery sustained at 40% rate - continuing to improve

Healthy Workflows ✅

Smoke Tests - Perfect Health

All smoke tests: 100% success rate (all recent runs)

  • All CI/CD validation working perfectly
  • Score: 100/100

Systemic Issues

Issue: Outdated Lock Files - 9 Workflows Need Recompilation

Status: NEW FINDING - Code changes not yet compiled

Workflow Status Issue
daily-file-diet ⚠️ Lock file outdated Needs make recompile
go-fan ⚠️ Lock file outdated Needs make recompile
daily-code-metrics ⚠️ Lock file outdated Needs make recompile
agent-persona-explorer ⚠️ Lock file outdated Needs make recompile
sergo ⚠️ Lock file outdated Needs make recompile
copilot-cli-deep-research ⚠️ Lock file outdated Needs make recompile
ai-moderator ⚠️ Lock file outdated Needs make recompile
daily-repo-chronicle ⚠️ Lock file outdated Needs make recompile
typist ⚠️ Lock file outdated Needs make recompile

Root cause: Source .md files modified more recently than .lock.yml files
Impact: Workflows running with stale configuration
Recommended action: Run make recompile to regenerate all lock files

Issue: Tavily-Dependent Workflows

Status: MONITORING - 1 recovered, 2 still failing

Workflow Status Last Success Success Rate Issue
Daily News RECOVERED 2026-01-23 40% (↑) Resolved
MCP Inspector ❌ FAILING 2026-01-05 0% #11721
Research ❌ FAILING 2026-01-08 20% #11722

Key Finding: Daily News recovered after TAVILY_API_KEY was added, but MCP Inspector and Research did NOT. Hypothesis: workflows need recompilation with make recompile.

Recommendations

High Priority (P1 - Within 24h)

  1. Recompile all workflows - 9 outdated lock files detected

    • Command: make recompile
    • Duration: ~5-10 minutes
    • Impact: Updates all workflow configurations to latest code
  2. Fix MCP Inspector (MCP Inspector - Still failing after TAVILY_API_KEY fix (0% success rate, 20 days offline) #11721) after recompilation

    • Test workflow manually
    • Verify MCP Gateway starts successfully
    • Compare with Daily News (working)
  3. Fix Research workflow (Research Workflow - Still failing after TAVILY_API_KEY fix (20% success rate, 17 days offline) #11722) after recompilation

    • Apply same approach as MCP Inspector
    • Test manually 3-5 times for intermittent issues

Medium Priority (P2 - This Week)

  1. Monitor Daily News recovery (target: 80% success rate)
  2. Verify Scout workflow (also uses Tavily)
  3. Add pre-commit hook to detect outdated lock files

Trends

  • Overall health score: 91/100 (→ stable)
  • Daily News recovery sustained: 40% rate stable
  • MCP Inspector offline: 21 days (↑ 1 day)
  • Research low success: 18 days (↑ 1 day)
  • NEW: 9 workflows with outdated lock files detected

Actions Take...


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing MCP Inspector workflow in health dashboard Fix cache-memory configuration in chroma.md shared import Jan 26, 2026
Copilot AI requested a review from pelikhan January 26, 2026 03:45
@pelikhan pelikhan marked this pull request as ready for review January 26, 2026 03:47
@pelikhan pelikhan merged commit db5c5ea into main Jan 26, 2026
@pelikhan pelikhan deleted the copilot/fix-mcp-inspector-issue branch January 26, 2026 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Workflow Health Dashboard - 2026-01-26

2 participants