Skip to content

Cleanup constraints report#30

Merged
tylerpayne merged 2 commits intomainfrom
reports/constraints
Oct 11, 2025
Merged

Cleanup constraints report#30
tylerpayne merged 2 commits intomainfrom
reports/constraints

Conversation

@tylerpayne
Copy link
Collaborator

Use a table for summary, add line per passed constraint and line per violation in collapsible

Use a table for summary, add line per passed constraint and line per violation in collapsible
@tylerpayne tylerpayne requested a review from Copilot October 11, 2025 16:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Cleanup and enhancement of the constraint violations report to improve readability and organization. The changes replace the previous text-based summary with a structured table format and reorganize the constraint details into a more systematic presentation.

  • Replace text-based summary with a clear table showing errors, warnings, and passes
  • Reorganize constraint presentation to show all constraints (both passing and failing) in a consistent format
  • Simplify the collapsible section structure by removing nested collapsibles

Reviewed Changes

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

File Description
src/mcp_interviewer/reports/interviewer/constraint_violations.py Refactored report generation to use table format for summary and improved constraint organization
mcp-interview.md Updated report output showing the new table format and reorganized constraint details
mcp-interview.json Updated test data and expectations to reflect changes in report structure and tool behavior

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tylerpayne tylerpayne merged commit b8e4028 into main Oct 11, 2025
5 checks passed
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.

2 participants