Skip to content

Conversation

@darrylb-github
Copy link

This PR adds a collapsible section to contain the schema diff in PR comments, improving readability by allowing users to expand/collapse the diff content as needed.

Changes

  • Wraps the diff content in HTML details and summary tags
  • Updates both the documentation example and the actual implementation in src/diff.ts
  • Maintains all existing functionality while improving UX

Benefits

  • Cleaner PR comment appearance
  • Users can choose when to view the full diff
  • Reduces visual clutter in PR conversations

@darrylb-github darrylb-github requested a review from a team as a code owner August 24, 2025 14:17
Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@gsalomao gsalomao force-pushed the feat/comment-diff-summary branch 2 times, most recently from 9bdd29b to e6af9c1 Compare September 25, 2025 08:13
@gsalomao gsalomao force-pushed the feat/comment-diff-summary branch from e6af9c1 to 6d3061e Compare September 25, 2025 08:50
@gsalomao
Copy link
Contributor

@darrylb-github, could you please run bun run format:write to fix formatting issues?

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