Additional CI/CD workflow fixes (Issues #20, #28, #30)

[trsdn]

Summary

This PR contains additional critical fixes for CI/CD workflows that resolve multiple issues with workflow validation and functionality.

Issues Fixed

• Issue PR Summary workflow fails with invalid format and poor design #20: PR summary workflow JSON parsing failures
• Issue Status dashboard timestamp not expanding properly #28: Status dashboard timestamp not expanding properly
• Issue Invalid workflow file: pr-feedback.yml has YAML syntax error on line 200 #30: Invalid YAML syntax in pr-feedback.yml workflow

Key Changes

1. PR Summary Workflow JSON Parsing Fix (Issue #20)

• Problem: JSON output variables causing "Invalid format" errors
• Solution: Use file-based approach instead of output variables
• Files: .github/workflows/pr-summary.yml
• Impact: Eliminates workflow failures due to JSON parsing issues

2. Status Dashboard Timestamp Fix (Issue #28)

• Problem: Timestamp showing $( date -u '+%Y-%m-%d %H:%M UTC' ) instead of actual time
• Solution: Generate timestamp in shell variable before heredoc
• Files: .github/workflows/status-dashboard.yml
• Impact: Proper timestamp display in status dashboard

3. PR Feedback YAML Syntax Fix (Issue #30)

• Problem: Invalid YAML syntax on line 200 in multiline Python script
• Solution: Fix quoting in python -c command and rename workflow
• Files: .github/workflows/pr-feedback.yml
• Impact: Workflow validation passes, PR feedback works correctly

Technical Details

PR Summary Workflow

• Replaced complex JSON in output variables with file-based storage
• Added comprehensive error handling with continue-on-error flags
• Improved Python error handling with proper exception catching
• Added fallback mechanisms for missing data

Status Dashboard

• Changed heredoc from single quotes to double quotes for variable expansion
• Added proper shell variable generation for timestamps
• Fixed GitHub Actions variable escaping in URLs

PR Feedback Workflow

• Fixed YAML syntax by changing python script quoting
• Renamed workflow from "PR Feedback & Issue Reporting" to "PR Feedback"
• Changed internal Python strings from single to double quotes

Validation

• All workflow files now pass GitHub Actions syntax validation
• Fixes tested in development environment
• Error handling improved throughout

Test Plan

• Workflow files pass GitHub Actions validation
• PR summary workflow handles JSON data correctly
• Status dashboard shows proper timestamps
• PR feedback workflow executes without YAML errors
• Verify workflows run successfully in production

Breaking Changes

None - all changes are backward compatible improvements.

Additional Context

These fixes address critical workflow validation failures that were preventing proper CI/CD operation. All changes follow established patterns and maintain existing functionality while fixing syntax and parsing issues.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[github-actions]

🔍 PR Quality Summary

CI Status

⚪ Tests: pending
⚪ Docs: pending
✅ Security: success
⚪ Quality: pending

Metrics

Metric	Value	Trend
📊 Coverage	N/A	-
🧪 Tests	Test results pending	-
⏱️ Performance	No performance data	-

Quality Checks

• Format & Lint: Ruff formatting and linting
• Type Safety: MyPy strict type checking
• Security: Bandit, Safety, GitLeaks scanning
• MCP Protocol: Tool schema validation
• Documentation: Docstring coverage (80%+)

MCP Tools

• convert_file - Convert individual files to Markdown
• convert_directory - Batch convert directories
• list_supported_formats - Query supported file types

---

_{🤖 Auto-generated by CI • Last updated: 2025-09-17 19:44 UTC}

[github-actions]

🔍 CI Quality Gates Summary

Overall Status: ✅ All Passed

Check	Status	Details	Action Required
🎨 Format	✅ Passed	ruff format check	None
🔧 Lint	✅ Passed	ruff linting	None
📝 Types	✅ Passed	mypy type checking	None
🧪 Tests	✅ Passed	Unit tests	None
📊 Coverage	80.5%	Minimum: 80%	None
🔌 MCP	✅ Valid	Protocol compliance	None
🔒 Security	✅ Clean	Dependency audit	None

🔗 Quick Links

• View detailed analysis
• See workflow logs

🛠️ Quick Fix Commands

# Fix most issues automatically
ruff format .
ruff check . --fix

# Run tests locally
pytest tests/unit/ --cov=markitdown_mcp

# Check types
mypy markitdown_mcp

---

Last updated: 2025-09-17 19:45:02 UTC