Skip to content

Conversation

@alepane21
Copy link
Contributor

@alepane21 alepane21 commented Jan 28, 2026

Summary by CodeRabbit

  • New Features
    • Added copy buttons for operation hashes, enabling users to easily copy both truncated and full operation identifiers in header and detail views.

✏️ Tip: You can customize this high-level summary in your review settings.

Checklist

  • I have discussed my proposed changes in an issue and have received approval to proceed.
  • I have followed the coding standards of the project.
  • Tests or benchmarks have been added or updated.
  • Documentation has been updated on https://github.com/wundergraph/cosmo-docs.
  • I have read the Contributors Guide.

@github-actions
Copy link

Router-nonroot image scan failed

❌ Security vulnerabilities found in image:

ghcr.io/wundergraph/cosmo/router:sha-8d44724adea4152b0fa13f8f1254fd7dc46c9548-nonroot

Please check the security vulnerabilities found in the PR.

If you believe this is a false positive, please add the vulnerability to the .trivyignore file and re-run the scan.

@codecov
Copy link

codecov bot commented Jan 28, 2026

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 21.63%. Comparing base (b427211) to head (48a562d).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
studio/src/components/checks/operations.tsx 0.00% 10 Missing ⚠️

❌ Your project check has failed because the head coverage (21.63%) is below the target coverage (30.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##            main    #2475       +/-   ##
==========================================
+ Coverage   1.50%   21.63%   +20.12%     
==========================================
  Files        292      521      +229     
  Lines      46690    70552    +23862     
  Branches     431      431               
==========================================
+ Hits         703    15263    +14560     
- Misses     45704    53750     +8046     
- Partials     283     1539     +1256     
Files with missing lines Coverage Δ
studio/src/components/checks/operations.tsx 0.00% <0.00%> (ø)

... and 229 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@studio/src/components/checks/operations.tsx`:
- Around line 422-431: The CopyButton inside the AccordionTrigger currently only
stops click propagation via onClick, so keyboard events (Space/Enter) still
bubble and toggle the accordion; update the CopyButton usage in
components/checks/operations.tsx (the CopyButton instance that renders next to
the shortened hash in the AccordionTrigger) to also include an onKeyDown handler
that calls event.stopPropagation() to prevent the Space/Enter key events from
bubbling to the AccordionTrigger.

@alepane21 alepane21 force-pushed the ale/eng-8833-cosmo-docs-improve-mcp-graph_name-description branch from 48a562d to 90f43f4 Compare January 28, 2026 18:53
@github-actions github-actions bot removed the router label Jan 28, 2026
@wundergraph wundergraph deleted a comment from coderabbitai bot Jan 28, 2026
@alepane21
Copy link
Contributor Author

@coderabbitai full review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

✅ Actions performed

Full review triggered.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

Walkthrough

Modified the operations component to integrate copy functionality for operation hashes. Replaced the static hash display cell with a composed accordion header that includes a CopyButton component alongside the truncated hash. Added CopyButton to operation details area for full hash copying. Minor layout adjustments with flex containers accommodate the new controls.

Changes

Cohort / File(s) Summary
Copy Button Integration
studio/src/components/checks/operations.tsx
Adds CopyButton component to accordion header alongside truncated operation hash and to operation details area for copying full hash; imports CopyButton and configures it with preventDefault to prevent accordion click propagation; includes minor flex container wrapping for layout accommodation

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions improving MCP graph name documentation, but the actual changes are exclusively UI/copy functionality in the operations checks component with no documentation modifications. Update the title to reflect the actual changes, such as 'feat: add copy button to operation hash in checks section' to accurately represent the implemented feature.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants